Questions and thanks

Hello and thanks for your wonderful code. I'm reading and loving it!
I have 3 questions:

if I manage to understand how this works and how to adapt to my need, do you mind if I add you directly to my special thanks page?
the current working draft SVG 1.1 does not support 3d transform functions on SVGPathElement, not yet as of 2020, does this means your code basically compensates the Z axis transformation by calculating the 2D coordinates to MIMIC the 3D presentation, do I understand correctly?
I haven't finished reading the code yet, so I'm not sure: can this code be adapted to work with a given 3d matrix?

Thank you and GOD bless man, you made my day!

Hello
First, thank you for your interest and that evaluation of my code.

if I manage to understand how this works and how to adapt to my need, do you mind if I add you directly to my special thanks page?

You can of course adapt it and everything else, and it would be a honor to be on your thanks page :-).

the current working draft SVG 1.1 does not support 3d transform functions on SVGPathElement, not yet as of 2020, does this means your code basically compensates the Z axis transformation by calculating the 2D coordinates to MIMIC the 3D presentation, do I understand correctly?

Yes that is what the code does, it has a 3D scene of objects in memory, and it projects it on the screen in order to mimic the 3D.
That code was made a long time ago, when svg did not have much interoperability with CSS. Then the team of thisequalsthat asked me some improvements, so I added some code.

I haven't finished reading the code yet, so I'm not sure: can this code be adapted to work with a given 3d matrix?

At the time I coded it, there was no easy access to 3D rendering code, like the excellent three.js library. I had no knowledge of quaternions, 4*4 matrix, etc... My projection code is not correct when camera moves behind the screen and lots of other situations.
I would be very happy if the library could be recoded with all the real algorithms of 3D rendering, keeping the principles of 3D declaration of the svg like I wanted it. I may be able to help you.
There is also code in three.js in order to render a scene in SVG. I do not know the state of it, i do not think it can support everything.
Best regards!

Ok I read the other question you asked on the svg spec repo.
You can apply any matrix on the "node" DOM element with the following code :
var shape = svg3d.shapeFactory(node);
shape.transform([3dmatrix]);

the matrix is an array of 12 numbers, as in the example :
svg3d.setTranslationMatrix = function(x, y, z) {
   return [
	1, 0, 0, x,
        0, 1, 0, y,
        0, 0, 1, z
        ];
    }

See the example :
https://github.com/ndebeiss/svg3d/blob/e620e04db3401af57ab848c600966ef710bf9252/examples/rg1024_metal_barrel.svg
with the associated javascript :
https://github.com/ndebeiss/svg3d/blob/e620e04db3401af57ab848c600966ef710bf9252/examples/js/rg1024_metal_barrel.js
with the code :
	var shapeAxex = svg3d.shapeFactory(document.getElementById("axex"));
	shapeAxex.transform([svg3d.setTranslationMatrix(0, 0, 0)]);

Wow man, take it easy! Thank you for the time you took to reply :D
I made a cool little tool to manipulate SVG path data, something of a hybrid of Raphael.js (for path command operations), SvgPath (the best svg path string parser I know), some Paper.js (for determining the shape draw direction), and the entire codebase is modernized ES6+. Heck I don't even use SVGMatrix API, I use DOMMatrix to calculate path command transformation.
My additions to the bunch is the ability to reverse path commands, transform path commands, calculate total path length (not possible in Node.js, currently working with a Raphael.js tool, but looking to finish my own implementation very soon)
I'm looking for ways to enable more creativity for my users in creating SVG Morph animations, I was thinking of a way to transform path commands to 3D, so I came here :)
I only know I can apply a "tuple" multiplication, or "multiply a vector by a matrix" but I don't know what I need from your code, not yet, so far this part seems the right part, just I'm not sure where to begin, a little help might be of use.

Ok, I see, that looks good. You may add features in order to apply filters, they are powerful but not easy to write.
The code you see parses the d attribute, and rewrites it into the svg. There is extra complexity because if threeD attribute is present, I allow the path to be declared with 3 coordinates on each point.
The projection of the 3d point on the screen is done there :
https://github.com/ndebeiss/svg3d/blob/master/svg3d/svg3d.js#L1014-1018
but is not good, it would be better to use code like that :
https://github.com/mrdoob/three.js/blob/ff48de58e216bc72abb4085a080c5acb66f2f584/examples/js/renderers/Projector.js#L844
meaning you multiply the vector by a projection matrix created by a PerspectiveCamera :
https://github.com/mrdoob/three.js/blob/cd41804aa436bb2cfd79797c04985f75c4c63e63/src/cameras/PerspectiveCamera.js#L182
The matrix4 creates its perspective :
https://github.com/mrdoob/three.js/blob/1983589c07c6ae833a23763562496e5197b48516/src/math/Matrix4.js#L772
I do not know about all the other matrix which are calculated...

Maybe we can also optimize your code a little bit, how about a 60fps animation with 3D SVG? I've never seen one :)

The setInterval is fixed, I think it can be reduced to have a 60 fps animation. Not sure it takes a lot of CPU.

That resource you're linking from Three.js is awesome, so I put some thinking before replying to you, so in y mind this is too far from our scope and objective. Please allow me to explain:

first, we don't need a Vector4 + Vector3 + Vector2 or and entire Silvester Math library to compute what we already have at hand: DOMMatrix, gets you the computed matrix, instantly, we only polyfill it in Node.js, but only for a quick transformation, while Three.js is a doom escape ready library for WebGL, the best JS library probably ever;
second, we only need to adjust some code to work with this type of pathArray [['M',0,0],['L',0,0]] which to my eyes, your code seems to be close to
lastly and possibly most importantly, the ARC path command is pretty trixxy, if you change its radius it's a hell of a problem, a problem I managed to solve by converting the arc commands to CUBIC bezier, the problem arises when you're looking to calculate the arc length for other operations like getTotalLength for which I'm also working on a polyfill for Node.js, but overall I believe you're on my side in this development.

Cheers man.

I'm testing your script and checking the values computed. I'm trying to understand how it works.
Ready for more questions?

Why is the computed matrix a 12 numbers array? Isn't a matrix3d supposed to work with 16 values?
If it's 12, then you're removing the perspective matrix, or what exactly is missing?
For some reasons, the script completely disregards the viewBox attribute and always compute values way out of the viewport, is there something I'm missing?
Why sometimes I get NaN and undefined in the path data? Probably you should skip cases where the matrix isIdentity? (This is something I had to fix in my code as well)

This is what I do with your script added to my demo page:

var shape = svg3d.shapeFactory(document.getElementByTagName('path')[0]);

// and apply the matrix array you want by calling :
var matrixArray = [];
matrixArray[0] = svg3d.setAnglesRotationMatrix(0,0,45);
shape.transform(matrixArray);

console.log(shape,matrixArray)

I have it to work a little bit, just I can't seem to figure out this matrixArray, what would be the equivalent of that into a DOMMatrix instance with 16 values please?

Yea, I have it working but a hell to configure, probably need some rest. I implemented the transform method from CSSMatrix polyfill:
transform = function(t /* tuple */ ){
 	let m = this,
 		x = m.m11 * t.x + m.m12 * t.y + m.m13 * t.z + m.m14 * t.w,
 		y = m.m21 * t.x + m.m22 * t.y + m.m23 * t.z + m.m24 * t.w,
 		z = m.m31 * t.x + m.m32 * t.y + m.m33 * t.z + m.m34 * t.w,
 		w = m.m41 * t.x + m.m42 * t.y + m.m43 * t.z + m.m44 * t.w;

 	t.x = x / w;
 	t.y = y / w;
 	t.z = z / w;

 	return t;
 }

Cheeeeeeeeeeeeeerrrrrrrrrrrssssssssssssssss !!

Hello
I do not have much time to answer, the 4th line of a 4*4 3d matrix is full of 0 and a 1 at the last column, see
https://stackoverflow.com/questions/29079685/how-does-4x4-matrix-work-in-3d-graphic
I will look at the demo, glad you made it work!

I have some projections going on but because the SVG coordinates system which is different from the standard HTMLElement box model, this projection isn't working with the DOMMatrix properly, I think would have to implement your matrix constructor.
With the identity matrix3d I'm getting the path projected to an 90 degrees angle on X axis and I can't see it, or sometimes it's going out of the parent SVG viewport.
So perhaps this isn't going the way we want.

I have the good news at hand: I'm rolling this feature in the next version, crazy?
I found the script that does the job, exactly what I wanted.

@ndebeiss aside from having it to work perfect with all other path commands, I can't find a solution for A and your code here isn't very helpful.

I'm going to put the A on hold, meanwhile...
https://thednp.github.io/svg-path-commander/

Hi @ndebeiss how you doin?
I remembered about this topic and wanted to check on you, are you still developing stuff or active in any way on Github?
I recently found some code to work with Arc path commands and the only thing left to solve is the 3D transformation (the point at length and segment length are done on my local machine), so I came to the only one in the world that might have what I hope to be a solution.
I'm looking at your half_pipe_3d_declared example and your code, I don't understand what you're doing with the xRadius, yRadius, xAxisRotation parameters. Also in the demo, the arcs seem to move up and down once a cycle, is that intentional?
In brief, this is what I understand from your code so far:

you're parsing and rebuilding the path strings on the fly in the same function body
you're using mostly Absolute values to compute X2, Y2, both before and after computation
you seem to leave some parameters unchanged

Other sources seem to do something different, if I understand correctly they create an ellipse, apply transformation and convert back to Arc segment, which is something I might have to try eventually.
So how does your example work? Can you make a simple arc transformation demonstration? That would be so awesome :)
Cheers

Hello
I am doing well thank you, and you ?
I am not actively developing on Github, but I still work in software development.
I read the code again, and I do not think that works. It only transforms rx and ry so if one of those values become very small, it will create a big arc, and it is not intentional at all.
The source you give looks very good !
Maybe try adding the code :
    const reflected = matrix.getDeterminant() < 0;

    // reverse the 'clockwiseness' if our transform includes a reflection
    // TODO: check reflections. swapping angle signs should fix clockwiseness
    const anticlockwise = reflected ? !this._anticlockwise : this._anticlockwise;
    const startAngle = reflected ? -this._startAngle : this._startAngle;
    let endAngle = reflected ? -this._endAngle : this._endAngle;

    if ( Math.abs( this._endAngle - this._startAngle ) === Math.PI * 2 ) {
      endAngle = anticlockwise ? startAngle - Math.PI * 2 : startAngle + Math.PI * 2;
    }

regards


Mixed-type fields?

Is it possible to define a field on a Model that may contain more than one type of Model, or an array containing different types of Models.
Something like normalizr's schema mapping function, giving control over how entities are contructed/deconstructed:
https://github.com/paularmstrong/normalizr/blob/master/docs/api.md#schema
"If your input data is an array of more than one type of entity, it is necessary to define a schema mapping."
Thanks.

The first requirement for this issue – Many To Many – relation is now available with Vuex ORM 0.17.0! 🎉  Please see the doc on how to define it.

Just started using this package for the past few days and just wanted to say thanks for the superb effort @kiaking ! Also went down the route implementing a similar thing manually using normalizr and things was getting messy, this package really simplifies that.
Also wanted to add that "Many To Many Polymorphic Relations" would be super useful for our use-case, but trying to workaround it now using plain many-to-many relations.

@DennisOng Thank you for using Vuex ORM! Yes, I'm working on with morph relation currently : ) It's bit complicated, so it might take time but stay tuned! I wouldn't think it takes too long though.

Polymorphic relations are implemented by #86! It will be available on next release 🚀

Now the Vuex ORM 0.20.0 supports polymorphic relation 🎉
https://github.com/vuex-orm/vuex-orm/releases/tag/v0.20.0
Thank you all for being patient on this, and thank you all for bringing this up!


Hotfix/1215/add test parameter file

For issue ga4gh/dockstore#1215
Tests are missing from this because of merge conflicts.  See feature/1215/addTestParameterFile for working tests.

Codecov Report

:exclamation: No coverage uploaded for pull request base (hotfix/2.1.2@d8b3d01). Click here to learn what that means.
The diff coverage is 0%.

@@               Coverage Diff               @@
##             hotfix/2.1.2     #258   +/-   ##
===============================================
  Coverage                ?   60.04%
===============================================
  Files                   ?      115
  Lines                   ?     3489
  Branches                ?      451
===============================================
  Hits                    ?     2095
  Misses                  ?     1166
  Partials                ?      228

Impacted Files
Coverage Δ

...container/version-modal/version-modal.component.ts
35.97% <0%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d8b3d01...aa94fbd. Read the comment docs.


Add make gardener-dev as a skaffold-based dev experience

How to categorize this PR?

/area dev-productivity
/kind enhancment
What this PR does / why we need it:
Developers can now use make gardener-dev to start a skaffold-based dev loop which can trigger new builds and deployments when changing source files.
Specific skaffold modules can be selected by setting SKAFFOLD_MODULE to reduce turnaround times.
Which issue(s) this PR fixes:
Part of https://github.com/gardener/gardener/issues/6016
Special notes for your reviewer:
With https://github.com/gardener/gardener/pull/7633, it is finally possible to select specific skaffold modules, because we are now using named image references instead of {{.IMAGE_TAG1}} etc.
Release note:

Developers can now use `make gardener-dev` to start a skaffold-based dev loop which can trigger new builds and deployments when changing source files. See the [documentation](https://github.com/gardener/gardener/blob/master/docs/deployment/getting_started_locally.md#developing-gardener) for more details.

/kind enhancement

/assign

/retest


Improve spark row written performance by optimizing memory access

Changelog category (leave one):

Performance Improvement

Can one of the admins verify this patch?

Deprecated PR. Close now.


Detail re Schema Limitations in Synapse Analytics

I have added a little detail in two places indicating that the Preview feature for Azure Synapse Analytics will only work in the DBO schema. The Development team is aware of this and indicated that they would be updating the documentation. https://feedback.azure.com/forums/307516-azure-synapse-analytics/suggestions/18434083-rename-a-column-name-in-sqldw#{toggle_previous_statuses}

@JamesMcG-MSBI : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Apologies. I was not aware that I should include the #sign-off keyword in the PR.

@JamesMcG-MSBI: I'm sorry - only the author of this article, @markingmyname, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

Apologies. I was not aware that I should include the #sign-off keyword in the PR.

Hi @JamesMcG-MSBI - That comment is for the article author. 👍🏻

@julieMSFT to help confirm

#sign-off

@julieMSFT: I'm sorry - only the author of this article, @markingmyname, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Change documents

Is your feature request related to a problem? Please describe.
no
Describe the solution you'd like
In the guide it saids you can’t make the bootable drive in windows or Linux but I tried making it in windows today following the open core guide using macrecovery (big sur)and it worked.

I actually prefer the online installer because it’s faster since you don’t have to flash it to your external flash drive. Also it’s nice to have the USB stick ready so fast because u can just leave ur pc on and do other stuff but if you used offline you would have to wait for it finish flashing the USB then boot into the installer and stuff

I haven't updated the guide in a very long time. It didn't work when Big Sur was out. I'll update the guide when i get the time.
You can also edit the readme and issue a pull request. I've been really busy and haven't gotten any time to update stuff

ok I'm issuing a pull request now thanks

Did you update to macOS 12 on the Thinkpad?

Yes I wrote this guide because I made the hackintosh for my brother and I
can’t ask for his computer to take a screenshot because it’s midnight but I
did it for fun

Writing this in the email glitched it out

Yes I wrote this guide because I made the hackintosh for my brother and I
can’t ask for his computer to take a screenshot because it’s midnight but I
did it for fun

With 0.6.8?

Yes

Ok. I've merged the changes.


fix README Week2-3 Description

What problem does this PR solve?

Disambiguate readme
What have you done or changed in this PR?
Update README Week2-3 Description
What test did you complete before submitting the PR?
Nothing

update README Week2-3 Description


Adds tree.py

PR Description
Description
Adds tree data structure
Related Issue
Closes: #65
Checklist
Before submitting this pull request, please ensure that you have:

[x] Followed the code style and conventions.
[x] Added appropriate comments and documentation.
[x] Updated or added unit tests, if necessary.
[x] Checked for any linting errors or warnings.
[x] Verified that the code builds and runs without errors.

Author
Raghav Sharma (@rghvsh)

Hey @Tinny-Robot @matt-wisdom is this fine?

Hey!
@Tinny-Robot @matt-wisdom is this fine?


QUEST queries

We also said we’d share an example search we’d like to do via the API:

How many and which interventional studies with at least one German location/recruitment site were started (or completed) in 2022?
How many and which interventional studies with at least one German location/recruitment site were started between 2000 and 2022?
Subanalysis by different sponsors/sponsor categorization
Subanalysis by different intervention types
Subanalysis by different status

https://crmdr.ecrin.org/api/Study/ByCountries-StartYear-StudyType?Countries=Germany&StartYear=1980&StudyType=Interventional
gives null result


Set up documentation on ReadTheDocs

Resolved: https://wellcome-aws-utils.readthedocs.io/en/latest/


Getting started

Please add a getting started sectiion to help people

added getting started section

Associating this Issue #3 with this commit


Better message username and time layout/size

Better aligned and font size exactly like Slack

Automated message from Dropbox CLA bot
@borisyankov, it looks like you've already signed the Dropbox CLA. Thanks!


Read only root fs

Towards: https://github.com/giantswarm/giantswarm/issues/31179
This PR ensures file system is accessed with root only
Checklist

[x] Update changelog in CHANGELOG.md.

Similar grafana agent PR tested here https://github.com/giantswarm/grafana-agent-app/pull/73

@TheoBrigitte can you take care of testing this change? Because I'm not sure what's the state of alloyLogs

You can test this on garfish where its already running, you would need to deploy the new version of the app and monitor:

alloy-logs logs: for any error, using kubectl logs should be enough
alloy-logs pods resources: to ensure resource usage stays the same, you can use the  Kubernetes / Compute Resources / Pod dashboard for this.
The amount of logs received on Loki. You can use the following query  sum(rate({scrape_job=~".+", cluster_id="golem"} [1m]))by(scrape_job) which count the amount of log lines received for each scrape job.

But I found out I cannot because security context and others are configured in the logging operator :D

So this PR works quite fine with alloy rules for instances :)

@TheoBrigitte could you take another look at this? This only sets readonlyrootfs and does not change the resource usage at all :)

Have you tested this with Alloy for logs ?

Yes :)


Fix missing symlinks in unzip

Fixes: #923
Description
This uses macOS built in unzip tool to unzip packages. On some macOS environments, the javascript compressing utility isn't restoring symlinks

@ayushmanchhabra please let me know if you're ok with this. I'd like to get it merged ASAP


Dusty up does not display issues form docker-compose and docker-machine

When running dusty up you sometimes get errors in docker-machine or docker-compose.  Often the real cause of the error is hidden by dusty.  Instead, you see the command that dusty tried to run but failed.
Two examples:
...
Stopping all running containers associated with Dusty
WARNING: docker-compose stop failed
Command '['docker-machine', 'env', 'dusty', '--shell', 'bash']' returned non-zero exit status 1
Compiling together the assembled specs
Compiling the port specs
Compiling the nginx config
Creating setup and script bash files
Compiling docker-compose config
Saving port forwarding to hosts file
Configuring NFS
Saving updated nginx config to the VM
Saving Docker Compose config and starting all containers
ERROR: Command '['docker-machine', 'env', 'dusty', '--shell', 'bash']' returned non-zero exit status 1

and
...
WARNING: docker-compose stop failed
Command '['docker-machine', 'env', 'dusty']' returned non-zero exit status 1
Compiling together the assembled specs
Compiling the port specs
Compiling the nginx config
Creating setup and script bash files
Compiling docker-compose config
Saving port forwarding to hosts file
Configuring NFS
Saving updated nginx config to the VM
Saving Docker Compose config and starting all containers
ERROR: Command '['docker-machine', 'env', 'dusty']' returned non-zero exit status 1

Instead of just saying that docker-machine or docker-compose failed, dusty should grab the error from the run and output it to the user.
Right now the only way to figure out what is wrong is to type the command that dusty failed on manually into your terminal. This is tedious, time consuming, and error prone.

Right now the only way to figure out what is wrong is to type the command that dusty failed on manually into your terminal.

I was under the impression the daemon generally logs stderr from the subprocesses, making the error output visible at /var/log/dusty.log. Is it not doing that?
Agree it would be nice to surface the issues more immediately to the user, not sure though whether it's worth whatever extra wrangling we'd need to do. Would need to think of how we could implement it before making a decision.


Document SIG Docs New Contributor Ambassador Program

cc: @RinkiyaKeDad @natalisucks @reylejano @jimangel
We require more documentation around the New Contributor Ambassador role to set off with the formalization of the process. Opening up this issue to track the progress in these potential areas

[x] Modify README
[ ] Discuss potential for inclusion in Contributor guide for SIG Docs
[ ] If outcome of above discussion is a yes, suitable edits to be made on the Contributor guide.

These are a few places I could think of. Please use this issue as an avenue for discussing any other areas that might benefit from including details about the role.

/assign

Thanks for opening the issue to track this @divya-mohan0209! By "Contributor guide" do you mean https://kubernetes.io/docs/contribute/?

/triage accepted

@RinkiyaKeDad : Yes, I don't know if it'd be a good fit given the nature of the contributor guide. But we need to have it documented somewhere as a role like we have the Release Team shadow/roles under the sig-release repo.

/language en
/sig docs

Ping to update the checklist :) Readme has been updated, will start a discussion on Slack to see if the Contributor guide needs to be updated next.

/remove-lifecycle stale
I can get to this next month!

/remove-lifecycle rotten

@RinkiyaKeDad - gentle reminder on this one. Do you need anything from our side?

@RinkiyaKeDad - gentle reminder on this one. Do you need anything from our side?

Thanks! This totally fell off radar. I'll get the discussions for this going this week!

Started discussions here: https://kubernetes.slack.com/archives/C1J0BPD2M/p1666775238623839

/remove-lifecycle stale

Hello @RinkiyaKeDad , please may I know if you have had a chance to work on this? Please let us know if you need anything from our side.

Hey @divya-mohan0209, if you see the Slack thread in the comment above, I need someone to confirm if it is okay to proceed with adding the section on that page. LMK if yes, and I will make the PR!

@RinkiyaKeDad thank you for clarifying. However the expectation was to timebox the consensus seeking process to a few days/weeks. After which, the PR you submit for amending the page would be put through a formal review by the website approvers & reviewers. So, if there is no dissent/feedback, it'd be okay for us to now have a PR opened to k/website per the initial idea & then figure out the placement/duplication there onwards.

Hello! I talked with Sreeram on one of our New Contributors Meet and Greet calls, and they expressed an interest in picking this issue up so I'm assigning them for now. I have communicated what needs to be done over the call to them and given them the required context. @sreeram-venkitesh please feel free to ping me if you have more questions and tag me for a review on the PR!
/assign @sreeram-venkitesh

/assign @sreeram-venkitesh

@RinkiyaKeDad @sreeram-venkitesh : Any updates on this? Please can we prioritise work on this?

Sorry for the delay! Will work on this on priority.

@sreeram-venkitesh - Any updates on this, please?

@divya-mohan0209 I have raised a PR and asked for review from @RinkiyaKeDad . Sorry for the delay in getting this done!

All the objectives to track the formalisation of the New Contributor Ambassador have been completed with the merge of #42481 . Hence closing this as completed.
/close


In Roku the function val() when receive a string with no numeric content returns zero

The code below must print:
1
0
1
0
And is currently printing:
1
NaN
1
0
sub main()
	print val("1")
	print val("notanumber")
    print strToI("1")
	print strToI("notanumber")
end sub

Fixed by #163


Needs an upgrade

I am using "@angular/core": "^2.1.2", and I get.......

core.umd.js?e2a5:334Uncaught TypeError: Reflect.getMetadata is not a function(…)ParamDecorator @ core.umd.js?e2a5:334(anonymous function) @ base-cookie-options.js?d615:23__decorate @ base-cookie-options.js?d615:16(anonymous function) @ base-cookie-options.js?d615:60(anonymous function) @ base-cookie-options.js?d615:67__webpack_require__ @ bootstrap 12ee094…?36af**:19(anonymous function) @ services.js:8__webpack_require__ @ bootstrap 12ee094…?36af**:19(anonymous function) @ core.js:11__webpack_require__ @ bootstrap 12ee094…?36af**:19(anonymous function) @ deps.ts:8__webpack_require__ @ bootstrap 12ee094…?36af**:19(anonymous function) @ bootstrap 12ee094…?36af**:39(anonymous function) @ bootstrap 12ee094…?36af**:39

The weird thing is it doesn't seem to cause any issues just seems to be annoying. I tried updating my polyfills to...
import "core-js/es6";
import "reflect-metadata";
import "core-js/es7/reflect";

But it didn't change anything

Sorry I also looked at other branches and didn't see anything from this arena so I didn't think it was getting worked on

This is actually caused by what I described in #31


Adding Push Agend Standart JAR file

Instead of using a Java managed, WebContainer, we use standard Kafka client and iOS Push feature
See the reduced memory/

Quality is POC :grin: with some copied sources, it's more to proof the point

@maleck13  @david-martin   FYI

@ziccardi yeah - it's POC / all in flow :)

for that, I think there will be a bit of reusable code (the fcm client will
be similar)
On Mon, Feb 5, 2018 at 4:55 PM, Massimiliano Ziccardi <
<EMAIL_ADDRESS>wrote:

@ziccardi commented on this pull request.
LGTM for a POC, just one major comment about exposing mutable internals
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aerogear/aerogear-unifiedpush-server/pull/964#pullrequestreview-94039460,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJnzvUzaEYiS_b2CrZa8HN97MmoEKDBks5tRyRrgaJpZM4R5f6K
.

--
Matthias Wessendorf
github: https://github.com/matzew
twitter: http://twitter.com/mwessendorf


python310Packages.bellows: 0.31.3 -> 0.32.0

Description of changes
https://github.com/zigpy/bellows/releases/tag/0.32.0

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 186332 run on x86_64-linux 1

4 packages built:

home-assistant-component-tests.homeassistant_yellow
home-assistant-component-tests.zha
python310Packages.bellows
python39Packages.bellows


Weird quote marks thing on livewire refresh

Thanks for a great package! :)
Running into a weird thing where quote marks gets screwed up after a livewire refresh.
So a json string starts out looking something like this in the browser:
<x-button wire:click="$emit('openModal', 'update', {"rowId":1})" />
After a livewire refresh it looks like this in the browser:
<x-button wire:click="$emit('openModal', 'update', {" />
I guess it's trying to close the wire:click="xxx" with the first quote mark of the json string and not return anything after that.
I tried using single quotes in wire:click='xxx', but then I got weird unexpected END IF problems with my button component.
My current solution is the following but there must be a better way :D
<x-button wire:click="$emit('openModal', 'update', {{ str_replace('"', '\'', e(json_encode(['rowId' => $rowId]))) }})" />
Thank you!

Switching to single quotes will fix this issue I believe:
<x-button wire:click='$emit("openModal", "update", {"rowId":1})' />

Thanks!
That gives me syntax error, unexpected token "endif", expecting end of file
When I run <x-button wire:click='$emit("openModal", "edit", {{ json_encode(['id' => $rowId]) }})'>Edit</x-button>
An error that I don't get with double quotes.
This gives no erors, but then I don't get my parameter:
<x-button wire:click='$emit("openModal", "edit">Edit</x-button>
Ideas? Thanks!


Batch mode tagging: Changes to view should unselect all items

In batch mode, it's easy to accidentally apply a tag to unintended items because the view has changed to filter them.
I believe the behavior would be less surprising if changing the view (filtering, randomizing) cleared all selected items.

Thank you for the feedback.
First, a user requested a good feature: to show how many items are currently selected. I think it's relatively easy to do and is worth doing. This may address a part of the problem you're describing.
I sort-of feel like clearing the selected items while searching isn't the behavior I want. Specifically: I suspect someone might search for "comedies" and filter by something else, click "select all", and then remember that there are a few other videos that fall under the same tag, perform a separate search (e.g. "rofl movies") and then click to apply the tag 🤔
Perhaps, as long as there is a "number of videos selected" indicator next to the "Batch mode" button, it will be clear that many movies are selected besides the ones that are seen 🤔
Though I also see how it is easy-enough to forget. And it's similarly easy to just do a separate search and click the tag a 2nd time (required since every search would clear the currently-selected) ...
I'm unsure since I don't use tags and don't know what a comfortable 'workflow' would be.
You're probably right that clearing out all tags on search is useful.
I hope others weigh in 👂

💡 I should also add a little notification (aka "toaster") that appears when you click on a tag - to notify user "N tags have been added" when they click on a tag 👍


Interactive should not fail when it encounters corrupted/unparsable cache.

This may occur naturally with a stale cache

This recently became a problem again, something is not being caught by our exception handling from https://github.com/Azure/azure-cli/pull/6381
I will look into and make improvements to the flow involved in reading/writing to the cache.

Shouldn't this be in the extensions repo now that interactive is an extension?

@tjprescott it is also still currently in the CLI.
I'll move the issues over once the interactive command -> extension wiring is complete.


The installation worked except for avast plugin. I tried to restart the docker elk but know I get this error
Get http://localhost:9200/: dial tcp [::1]:9200: getsockopt: connection refused.

Steps to reproduce the issue:

docker stop malice-elk
malice elk start
malice -D scan ~/go/bin/malice

Describe the results you received:
DEBU[0000] Malice config loaded from: /home/algosecure/.malice/config.toml
DEBU[0000] Malice plugins loaded from: /home/algosecure/.malice/plugins.toml
DEBU[0000] Using 2 PROCS
DEBU[0000] Malice Version: 0.2.0-alpha
DEBU[0000] Running inside Docker...
DEBU[0000] Connected to docker daemon client             ip=localhost port=2375
DEBU[0000] Searching for container: malice-elk           env=development
DEBU[0000] name:  malice-elk   container.Name:  loving_mccarthy
DEBU[0000] MATCH:  false
DEBU[0000] name:  malice-elk   container.Name:  cocky_lumiere
DEBU[0000] MATCH:  false
DEBU[0000] name:  malice-elk   container.Name:  objective_kalam
DEBU[0000] MATCH:  false
DEBU[0000] name:  malice-elk   container.Name:  modest_stallman
DEBU[0000] MATCH:  false
DEBU[0000] name:  malice-elk   container.Name:  malice-elk
DEBU[0000] MATCH:  true
DEBU[0000] Container FOUND: malice-elk                   env=development
DEBU[0000] ELK is running.                               image=blacktop/elastic-stack:malice ip= network=default
DEBU[0000] Attempting to connect to: http://localhost:9200
2017/02/06 12:01:36 Get http://localhost:9200/: dial tcp [::1]:9200: getsockopt: connection refused
Describe the results you expected:
A normal scan with avast not working
Additional information you deem important (e.g. issue happens only occasionally):

I like that you are scanning the malice binary with malice I haven't thought to try that yet 😉

This might be related to #39. I have noticed that the blacktop/elastic-stack:malice takes a little while to fully boot so I tried to be clever and wait-for-it in golang, but it seems to be exiting once it sees it is running the very first time.  Can you do a:
docker ps -a

To see what state malice-elk is in?
Thank you

You can also do a docker rm -f malice-elk to clear it out, the data should persist to the malice docker volume.

Thank you, the last command worked! For those who have the same issue : rm -f malice-elk and then malice elk
I'll just have to figure out how to install avg


feat: implement UNIQUE constraints creation mechanism

The GA is showing 15 new failures. Some of these are clearly related to the insert bug. However, others look legitimate. Could you show the tests results when run locally with the bug fixes?

@larkee, yeah, it looks like test_update_from_round_trip cleanup has failed and made other tests failing as well. Taking a look at it...

@larkee, I propose to override the test method, which defines tables for CTE tests. Originally it creates a foreign key without a name. Spanner gives a generated name to the constraint, but it's not reflected in the test case (as it's really small and other tests of the class are skipped - it doesn't see a reason to sync once again). So, there is no way to drop the constraint before dropping the table, as we don't have its name.
As it's only a single test cleanup problem (the test itself is passing), I assume we better override testing data to explicitly set the FK name, instead of adding calls to backend into visit_drop_table to force reflect the data.
The strange thing is that in workflow a lot of tests are failing, while locally I have only expected failures, caused by the insert but, and the failures, which are not yet processed (all the reflection tests are now executed, but not all of them are processed in fact). Should we move the test workflow to kokoro?

I propose to override the test method, which defines tables for CTE tests. Originally it creates a foreign key without a name. Spanner gives a generated name to the constraint, but it's not reflected in the test case (as it's really small and other tests of the class are skipped - it doesn't see a reason to sync once again). So, there is no way to drop the constraint before dropping the table, as we don't have its name.

As it's only a single test cleanup problem (the test itself is passing), I assume we better override testing data to explicitly set the FK name, instead of adding calls to backend into visit_drop_table to force reflect the data.

SGTM 👍

The strange thing is that in workflow a lot of tests are failing, while locally I have only expected failures, caused by the insert bug, and the failures, which are not yet processed (all the reflection tests are now executed because of import, but not all of them are processed in fact). Should we move the test workflow to kokoro?

Based on your screenshot, I suspect you are running the tests against prod. The workflow is running against the emulator. Unlike prod, the emulator cannot infer the type for null values. This is what is causing the additional failures. There was a fix being worked on in the DB API driver but it's not finished yet.
I am very unfamiliar with setting up a Kokoro workflow correctly so I'm hesitant to move away from GitHub Actions but I'll talk to the team about it.

@larkee,

Should we override the asc column name since it's a reserved keyword?

I was going to take a look at keywords escaping system (I saw something like this in DB API). We can override this particular test any time, it's not gonna be a problem, but I assume there are more similar cases in tests (at least I've caught my eyes on one earlier), so I'd like to deal with the trouble overall (if it can be done overall).

Based on your screenshot, I suspect you are running the tests against prod

Yes

Are there unique index specific tests?

No, no, I don't see any index-type-specific tests. They are just about indexes overall, like test_get_indexes. Constraints, yes, for every constraint type there is a separate test, but not for indexes.

Merging to unblock further work


'query' is not implemented and it is a pity : 'query' are really missing to have OSM contributor functional

Hello
I ve tried to use the  'query'. This is documented this way in the spec :
$.groups.$.item.$.query
The overpass query allowing to fetch the current item. This field is optional.
This is quite short and not very self explainable  !
The query I wish is the following
[out:json][timeout:25];(node["advertising"="board"]"legal_type:FR"="préenseigne";);out body;
I note for information, that for dynamic umap, query are like this one :+1:
http://overpass-api.de/api/interpreter?data=[out:json][timeout:25];(node["advertising"="board"]["legal_type:FR"="préenseigne"]({south},{west},{north},{east}););out body;
I ve tried several query syntaxes, none worked well. LOrtola conformed later that the 'query' is in fact not implemented.  The query is needed anyhow to know if the object when want to set and add in OSM already exists or not. If it does already exist, there is obviously no need to add it again in OSM, but user of OSM contributor needs to know that !
Thank you to see if this 'query' can be added in implementation !

Hi @pdfext, I just noticed that this issue is on the wrong repository.
Please post it on the repository of osm-contributor.


Add update methods for NetworkRouter

Added update and raw_update methods for Network Router model in order to expose update operations in Automate engine: https://github.com/ManageIQ/manageiq-automation_engine/pull/54

@miq-bot assign @blomquisg

@blomquisg changed

@bdunne yes, fixed it

According to discussion https://github.com/ManageIQ/manageiq/pull/15720#issuecomment-320034194 we're not going to merge this


[Question] How to get the id of all players in a world instance?

I'm making a program to find specific users.
It doesn't seem to exist in the VRChat API, how did you do it?

You can't, VRCX only knows about player userId's in the instance that are your friends with by finding matching displayName from friends list. Only other way is using the player search endpoint to find a user with a matching displayName from the result, the search isn't great and doesn't work all the time also would be a lot of extra API calls.

So, how was it possible to check the id of non-friend users in gamelog?

The companion mod has userId's as well as if you have API logging enabled and select the user in-game VRCX will fetch that users details.

Thanks for answer!


Build status should take into account re-running a failed test

Currently, when we re-run a devicelab test, the build dashboard goes green because the test is no longer red -- it's orange (re-running).  However, in such cases, we should treat the test as the status of whatever the first run was.  In other words, re-running a failed test should not make the tree go green until that test passes.

I am closing this in favor of https://github.com/flutter/flutter/issues/45735.


Update AnnotationController.java

@VincentStriegel, Please when you have some time, test this functionality, because of @BruderSimon I was able to implement it very fast 👊🏼, So We can tell team A about it too

This is a minor issue:  when you delete the picture you are currently viewing, the visual annotation stays.

For now i have merged the implementation, because Group A wanted to use the tool, but i will fix this small bug today :)

For now i have merged the implementation, because Group A wanted to use the tool, but i will fix this small bug today :)

We have some other bugs, created an issue in Jira about the arrow and delete button not working.


SOLR-17066: Switch "LB" clients away from core URLs

https://issues.apache.org/jira/browse/SOLR-17066
Description
Bootstrapping Solr clients with core URLs has several downsides: it prevents the created client from making core-agnostic requests, it prevents the client from making both v1 and v2 requests, etc.  For all these reasons, SOLR-17066 has been moving our client implementations away from using these "core URLs" and towards using a "default collection" instead.
Solr's "Load balancing" client is an interesting case though.  A common (maybe even the primary?) use-case of the "LB" client is to round-robin requests across a set of replicas.  Each replica in a set has a unique core name, and distinguishing between them is important to ensure uniform load distribution.  So users can't rely on the "default collection" feature to meet their needs.
Any attempt to move the LBClients away from "core URLs" must also retain a way for them to specify a distinct set of replicas/cores.
Solution
This PR solves this problem by creating a new LBClient-specific abstraction called an "endpoint".  Endpoints are thin wrappers around a Solr base URL and optional core name, and users now provide these "Endpoint" objects when creating their LBClient, instead of passing raw String URLs.
Using a strongly typed object instead of raw String URLs allows core-based clients to still make core-agnostic requests as needed, while still supporting the "target a set of distinct replicas" use case.
Tests
Changes to TestLBHttpSolrClient and TestLBHttp2SolrClient.  New unit tests for the Endpoint class.  Existing tests continue to pass.
Checklist
Please review the following and check all that apply:

[x] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[x] I have created a Jira issue and added the issue ID to my pull request title.
[x] I have given Solr maintainers access to contribute to my PR branch. (optional but recommended)
[x] I have developed this patch against the main branch.
[ ] I have run ./gradlew check.
[x] I have added tests for my changes.
[x] I have added Javadoc documentation

Question Is this change worth the added complexity?  I'm excited to move away from core URLs, but want to make sure the net effect is still positive given some of the complexity this pulls in.  Curious what others think.
If we do go this route, this change should target main/10.0 only; I don't think 9.x is an option, since we still support arbitrary "context root"s on that branch.  (Though of course, we'll need to make some sort of branch_9x change to deprecate the LBClient method signatures that this PR modifies)


Sysctls are required for cis-hardening to work

Summary

These six sysctls are required, matching the source code.
It's well hidden in your documentation and it only mentions two of them.
What Should Happen Instead?

Enabling cis-hardening should set them, or at least give a hint about them.
Reproduction Steps

Install a fresh Ubuntu Server 23.04.
sudo snap install microk8s --classic --channel=1.28
sudo microk8s enable cis-hardening
sudo reboot
Observe that it's unstable.

Introspection Report

None
Can you suggest a fix?

I could if you agree first on what should be done.
Are you interested in contributing with a fix?

Depends on its scope. I spent many hours to track down that issue, I could give some more.

Hi @KoltesDigital, thank you for reporting this and finding the root of the cause in this issue.
Assuming that anyone enabling CIS-hardening has some level of expertise we could ask him to check that the node is setup correctly and perform any changes. The microk8s snap could  perform these checks/changes too but this is an approach we would rather avoid as it would mean we would need to break snap isolation and touch the host operating system.
So the action items we should have is to:

update the cis-hardening message to inform the users of the extra configuration they potentially need to perform https://github.com/canonical/microk8s-core-addons/blob/main/addons/cis-hardening/enable#L387
update the docs page https://microk8s.io/docs/how-to-cis-harden

Would this be enough @KoltesDigital? Wdyt?

Hi @ktsakalozos,
Updating the docs is mandatory of course :)
I didn't thought about snap isolation. So I guess a message after install is the best available solution.
Would it be violated if the enable script only checks, not touches, the host OS? With the aim of displaying the message only when the OS isn't already configured.

@KoltesDigital we are patching the deployment message in the addon and the docs. I am keeping this issue open incase you are interested in contributing some logic to improve the messaging by  inspect the OS. Thank you.


Chatbot using Speech Recognition [GSSoC23]

The user will be able to ask questions using their speech and based on questions, the chatbot would answer correspondingly.
Chatbot will recognize the speech and using that the answers would be given by the chatbot.

Hey, I'm a GSSOC'23 participant. I would like to do this.

Hey, I'm a GSSOC'23 participant,
I can implement the chatbot with the help of Deep Learning API's. I will be going to combine Speech recognition and Test to Speech conversion to build a complete chatBot that will answer your queries and play music for you.

i amm intereted in this can you please assign it to me under gssoc

Hi, I am very much interested and would like to contribute as a part of @GSSOC'23.
Please assign the task to me. Thank you.

@dishamodi0910  Go Ahead !!

Thank you very much @neelshah2409 for assigning this issue to me!

@neelshah2409 I want to contribute.Please assign me


profile-time: more result reporting, and learn to specify other backends

Output looks like:

which renders to this on Github:

feature class
evaluation count

evaluate.feature
19,939,641

evaluate.feature.and
4,441,407

evaluate.feature.rule
4,124,464

evaluate.feature.api
2,385,944

evaluate.feature.bytes
1,756,958

evaluate.feature.match
1,546,698

evaluate.feature.or
1,443,142

evaluate.feature.number
1,246,595

evaluate.feature.mnemonic
1,205,911

evaluate.feature.regex
271,779

evaluate.feature.os
264,511

evaluate.feature.string
192,866

evaluate.feature.characteristic
178,392

evaluate.feature.some
163,596

evaluate.feature.operand[1].number
155,261

evaluate.feature.substring
127,813

evaluate.feature.arch
127,381

evaluate.feature.operand[0].offset
104,100

evaluate.feature.operand[1].offset
78,648

evaluate.feature.offset
56,995

evaluate.feature.range
31,907

evaluate.feature.property
21,125

evaluate.feature.format
7,604

evaluate.feature.not
6,108

evaluate.feature.operand[2].number
425

evaluate.feature.section
5

evaluate.feature.export
3

evaluate.feature.import
2

evaluate.feature.operand[0].number
1

label
count(evaluations)
min(time)
avg(time)
max(time)

5390e1a0 be2: insn: polish thunk handling a bit (dirty)
19,939,641
81.45s
83.54s
86.74s

Checklist

[x] No CHANGELOG update needed

[x] No new tests needed

[x] No documentation update needed

@s-ff FYI, this is a small script I've used in the past to help evaluate performance changes to capa. It benchmarks the rule matching phase and shows the number of times each type of feature was evaluated. This enables two things:

if we reduce the overall count, then it means we're doing less work, so capa is running faster, and
we can identify hotspots (features that are evaluated a huge number of times) and optimize those

For example, API features are evaluated around 2 million times in the above example, while import features are only evaluated twice, so its probably worthwhile to spend more time optimizing API features than import features, if possible.
This is all just background info for you, nothing expected at this time :-)


capacity of logstash container

I noticed a problem where logstash container stop working and kibana throwing
ERR Syslog Sink syslog://<IP_ADDRESS>:12345: Error when dialing out. Backing off for 1.016ms. Err: dial tcp <IP_ADDRESS>:12345: connection refused
restarting container just that container only fixed the problem .. the service instance bind with mutiple application like 15 at same time, do you see any issues in capacity like how many apps can push data at same time or CPU or memory assigned to container ...

@amulyas did you ever solve this?

Yes its not the case anymore this should be closed


State machine refactor

This is a major change aimed at speeding up re-renders by moving event triggers and complex state into mobx-state-tree.

Lingering issues:

[ ] Endbin calculation is still overshooting
[ ] Link Navigation is very precisely targetted
[ ] test if always loading more than one chunk
[ ] screen/scroll width reduced as well as expanded
[ ] update on jsonName change


Run a case study

In order to test usability and usefulness of CFF, a case study with a (smaller) repository/community should be run, which uses citation-relevant metadata, but not in a standardized format.
@npch has suggested ASCL.

This has (implicitly) been done during the ASCL Registries Workshop in Nov 2019, during which CFF support has been implemented.


Call to undefined function Workerman\pcntl_signal()

hi, when I am trying to start it, I am getting an error
Fatal error: Uncaught Error: Call to undefined function Workerman\pcntl_signal() in /Applications/MAMP/htdocs/site/vendor/workerman/workerman/Worker.php:1058
Stack trace:
#0 /Applications/MAMP/htdocs/site/vendor/workerman/workerman/Worker.php(505): Workerman\Worker::installSignal()
#1 /Applications/MAMP/htdocs/site/worker.php(40): Workerman\Worker::runAll()
#2 {main}
  thrown in /Applications/MAMP/htdocs/site/vendor/workerman/workerman/Worker.php on line 1058

Help me please!

Please install pcntl extension for php.


Extract max length values of SelectOption and Button

Pull Request Etiquette

[x] I have checked the PRs for upcoming features/bug fixes.
[x] I have read the [contributing guidelines][contributing].

Changes

[x] Internal code
[ ] Library interface (affecting end-user code)
[ ] Documentation
[ ] Other: _____

Description
Introduces public fields in SelectOption that contain the maximum length of Label, Value and Description.
Introduces public fields in Button that contain the maximum length of Label, ID and URL.
Identical to MessageEmbed, these fields can then be used by developers without having to update themselves when changes occur.

Resolved merge conflict

It seems select option label and description limits are being raised to 100: https://github.com/discord/discord-api-docs/discussions/3409#discussioncomment-1128066
We should probably address that in this PR as well.


Clean up the bundle archive after extracting it

Make some more disk space available to future steps by deleting the CodeQL bundle archive once we've extracted it.  We tolerate this step failing since deleting files doesn't always work, particularly on Windows where an antivirus program might lock the file.
Merge / deployment checklist

[x] Confirm this change is backwards compatible with existing workflows.
[x] Confirm the readme has been updated if necessary.
[x] Confirm the changelog has been updated if necessary.

I reran https://github.com/github/codeql-action/actions/runs/5730681885/job/15530162792?pr=1820 in debug.  Logs are looking good.

Just curious: how exactly did you figure out the disk space savings here?

I did a fairly basic calculation: I added a bunch of print statements to dump the available disk space, and summed the amount of disk space made available by deleting the archived bundle and deleting the extracted bundle from the temporary directory.


Don't log loaded PackageConfig

It was added (7c5770d) when moving from Marshmallow v2 to v3 and it's not needed since we already log raw config.

recheck


Forbid casting of struct types

C does not allow casting struct types by value, so we should do the same in BPFtrace.
Resolves #1869.

Checklist

[ ] Language changes are updated in docs/reference_guide.md
[x] User-visible and non-trivial changes updated in CHANGELOG.md
[x] The new behaviour is covered by tests

Updated to block both casts from and to struct types.


bug when trying to lauch azure cli in interactive mode on windows 10

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az interactive
Errors:
No module named 'prompt_toolkit.interface'
Traceback (most recent call last):
pip-install-rtk110ve\knack\knack\cli.py, ln 206, in invoke
cli\core\commands\__init__.py, ln 603, in execute
cli\core\commands\__init__.py, ln 661, in _run_jobs_serially
cli\core\commands\__init__.py, ln 654, in _run_job
Temp\pip-install-rtk110ve\six\six.py, ln 693, in reraise
cli\core\commands\__init__.py, ln 631, in _run_job
cli\core\commands\__init__.py, ln 305, in __call__
azure\cli\core\__init__.py, ln 485, in default_command_handler
cli\command_modules\interactive\custom.py, ln 41, in start_shell
cliextensions\interactive\azext_interactive\__init__.py, ln 10, in start_shell
    from .azclishell.app import AzInteractiveShell
interactive\azext_interactive\azclishell\app.py, ln 27, in <module>
    from prompt_toolkit.interface import Application, CommandLineInterface
ModuleNotFoundError: No module named 'prompt_toolkit.interface'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az interactive --style {}

Expected Behavior
Environment Summary
Windows-10-10.0.17763-SP0
Python 3.6.6
Shell: powershell.exe

azure-cli 2.0.70

Extensions:
interactive 0.4.3

Additional Context

@jiasli Please take a look

Hi @gisairo , it seems that your prompt_toolkit version is not right. We use 1.0.x in interactive extension, for somehow, your version is 2.x or 3.x.
You can reinstall it by this command:
pip install prompt_toolkit~=1.0 --force-reinstall -t C:\Users\<USERNAME>\.azure\cliextensions\interactive
Let me know if you have other problems. Thanks

Hi @gisairo , close this first. You can reopen it or submit a new one if you have other concerns.


Trackme not working

Hello,
I am trying to use trackme to get the user's location on the map, but it is not working on any browser or device I tried. As far as I understand it, by sending this command, the user's location will be shown as a marker on the map?
The tracking button on the left of the map changes to tracking, but no marker is shown(I tried with a custom marker to no avail).
I am using NR 3.0.2 and Worldmap 2.37.4

My bad, needs TLS cert on NR server for this to work.


Maintenance takeover request

Hi,
this project is not as active as it can be.
So I think if you won't mind to pass project to other maintainer.
I would be glad to take maintenance of this project, fix bugs and improve it.

I created this ages ago as a little helper program and don't really use it any more. @KeNaCo if you'd like to take over maintenance and continue developing it, I'd be happy to make you a maintainer.


"import type" statements are left in the code

So it seems not all type-related code is removed before letting the actual node process take over.
Example:

$ flow-node apps/retrieve-imap-emails.js
C:\Users\WhyEvenTry\OneDrive\Projects\One\lib\storage-versioned-objects.js:22
import type {SHA256Hash, OneObject, OneObjectTypes} from './object-recipes';
^^^^^^
SyntaxError: Unexpected token import
at Object.exports.runInThisContext (vm.js:78:16)
...

My mistakee, I forgot // @flow at the top of that file.


🐞 - emitEvent config for control does not work

Playground Link
https://stackblitz.com/edit/angular-5sxteh?file=src%2Fapp%2Fapp.component.ts&hideExplorer=1
Description

Angular version
12.2.17
Taiga UI version
1.2.0
Which browsers have you used?

[ ] Chrome
[ ] Firefox
[ ] Safari
[X] Edge

Which operating systems have you used?

[ ] macOS
[X] Windows
[ ] Linux
[ ] iOS
[ ] Android

@waterplea Hello, I understand correctly that there is no bug here, because this is how reactive forms work?


docs: add aspect ratio docs for CldImage component

Description

This PR adds documentation for the aspectRatio prop of the CldImage component, highlighting the constraints and cases where aspectRatio prop is applicable and will work properly.
The constraint is that width and height props are required by NextJS and aspectRatio prop does not function properly when they are set. So, for proper functioning of aspectRatio, the fill prop must also be present on the component, and height and width props must be absent.
More details here
Issue Ticket Number
Fixes #475

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Fix or improve the documentation
[x] This change requires a documentation update

Checklist

[x] I have followed the contributing guidelines of this project as mentioned in CONTRIBUTING.md
[x] I have created an issue ticket for this PR
[x] I have checked to ensure there aren't other open Pull Requests for the same update/change?
[x] I have performed a self-review of my own code
[x] I have run tests locally to ensure they all pass
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes needed to the documentation

thanks @DevRish - i think the only thing missing here is mentioning about it requiring a specific crop mode
if you check this out, you can see where the code itself requires it: https://github.com/cloudinary-community/cloudinary-util/blob/main/packages/url-loader/src/plugins/cropping.ts#L185

@colbyfayock  Thanks for reviewing and providing the code reference! I have made the necessary changes and also added some further explanation following your comment on the issue.
I also created a separate Usage constraints sub-section for it under the aspectRatio section to make it more readable.

looks good to me! thanks @DevRish
thanks for your contribution. this PR is eligible for free Cloudinary Hackatoberfest swag. Please send me an email at<EMAIL_ADDRESS>with your name, GitHub username, and a link to the PR where I'll send further instructions.
Happy Hacktoberfest!

@all-contributors please add @DevRish for doc

@colbyfayock
Thanks for adding me to the contribitor list! I have also sent the mail for the swags.
Happy Hacktoberfest!


DetectedLongRunningApplicationTask warning on each websocket connection

Describe the bug
All connections (using only websockets) to Azure SignalR hub causes DetectedLongRunningApplicationTask warning.
Looks like it was introduced with https://github.com/Azure/azure-signalr/pull/1332.
To Reproduce
Repro with server and client here https://github.com/standsed/SignalRRepro
Please change ConnectionString in Startup.cs to your Azure SignalR instance.
Exceptions (if any)
warn: Microsoft.Azure.SignalR.ServiceConnection[26]
      The connection L8g2QG7jU6iwwgPOCEUrDg1d404c941 has a long running application logic that prevents the connection from complete.

Further technical details

Your Azure SignalR SDK version: 1.9.1
Your Server ASPNETCORE version 5.0.301
Your SignalR Client SDK version 5.0.7

Thanks for reporting the issue, investigating...

Fixed in 1.9.2, thanks!


Skip empty tracks

Closes https://github.com/foobuzz/statify/issues/5

Thanks for this. Out of curiosity, do you know what such empty track represent in Spotify?


Alt + click not working [OSX 1.0.3]

It seems that alt + click does not work on OSX version 1.0.3

It works finally.


netprofm holds on to process handles and leaks zombie processes

Item
Value

OS, Version / Build
Win32NT             10.0.18363.0 Microsoft Windows NT 10.0.18363.0

Processor Architecture
AMD64

Processor Type & Model
Intel Xeon CPU E5-2690 v3 (2 processors)

Memory
96 GB

Storage Type, free / capacity (e.g. C: SSD 128GB / 512GB)
1 TB SSD, 430 GB free

Relevant apps installed
Visual Studio, Chrome, full Chromium repo, Edge Canary, Edge Dev, ???

Description
After using this workstation for a while to develop Chrome I have noticed zombie processes accumulating. This is indicated through increasingly large process IDs and then verified by running FindZombieHandles.exe from an elevated command prompt. This shows:
1115 zombies held by svchost.exe(2496)
    1081 zombies of svchost.exe (process: 1081 - thread: 0)
    20 zombies of MicrosoftEdgeUpdate.exe (process: 20 - thread: 0)
    14 zombies of MicrosoftEdgeUpdate.exe (process: 14 - thread: 0)

Checking with Task Manager shows that svchost.exe(2496) is netprofm, the Network List Service. This machine has been running for twelve days, so roughly 100 zombies generated per day.
Steps to reproduce
Unknown.
Expected behavior
There should be zero zombie processes. Each zombie process consumes roughly 64 KB of kernel memory.
Actual behavior
A non-trivial number of zombie processes where netprofm is holding the process handle of the dead process.
For more details see:
https://randomascii.wordpress.com/2018/02/11/zombie-processes-are-eating-your-memory/
FindZombieHandles.exe can be found at https://github.com/randomascii/blogstuff/tree/main/FindZombieHandles

Note that WmiPrvSE.exe also leaks process handles - 220 random processes. OpenConsole.exe (the new terminal?) has leaked another 99. Other Microsoft processes also leak a few handles but those are the three main ones.

Hey Bruce! Thanks for filing. I've reached out to the team who owns the process and have filed a bug. I'll update this thread when I hear back from them on potential investigations/solutions/other updates.


Node count of 1 is recommended, but not supported. Docs could be a lot clearer about this.

Recommended node count for Bronze tier is said to be one, yet buried further down is a very important note that basically means that 1 node clusters are unusable in all practical sense and will require re-creation to do almost anything. Even deploying applications to it doesn't work reliably (because it will fail EnsureAvailability check and get stuck).
This is such an important limitation it should be the first paragraph in bold.

The one node cluster, runs with a special configuration and so, scale out of that cluster is not supported. The one node cluster, has no reliability and so in your Resource Manager template, you have to remove/not specify that configuration (not setting the configuration value is not enough). If you set up the one node cluster set up via portal, then the configuration is automatically taken care of. One and three node clusters are not supported for running production workloads.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 658de695-0927-175f-0419-dcf7b6bece9d
Version Independent ID: 8166322c-774b-9434-3df5-5501b7dd9cf1
Content: Planning the Service Fabric cluster capacity - Azure Service Fabric
Content Source: articles/service-fabric/service-fabric-cluster-capacity.md
Service: service-fabric
GitHub Login: @athinanthny
Microsoft Alias: pepogors

@egorpavlikhin
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@athinanthny
Can you please check and add your comments on this doc update request as applicable.


How to Run current script in node debug (Alt + D)?

How to Run current script in node debug (Alt + D)?

Yes @ggslove, you can hit (Alt+D) or (Ctrl+D), depending on your system, and you'll get desired :) But there are a more 3 serious problems:

you cannot communicate with an output of debugging session, for now
hence the (Alt+D) is useless, for now
node --debug deprecated since version 7.7.0
I'll need to do some observation for the first two problems.

@ggslove the issue is fixed in new release 2.0.0


Add AWS ECS Compatibility to DefectDojo

:warning: Note on feature completeness :warning:
Is your feature request related to a problem? Please describe.
As of now, DefectDojo doesn't provide built-in support for deploying on AWS ECS (Elastic Container Service), which makes it difficult for users running their services on ECS to easily deploy and manage DefectDojo containers. Managing ECS compatibility would streamline containerized deployments for users with AWS infrastructures.
Problem statement:
I am frustrated by the manual setup required to deploy DefectDojo on ECS. The current setup is more tailored for Kubernetes or Docker Compose, but there's no dedicated guide or configuration for AWS ECS.

Describe the solution you'd like
Solution overview:
To support users who prefer AWS ECS for container management, I propose introducing ECS-compatible task definitions, service configurations, and documentation that guides users through deploying DefectDojo on ECS.
Key additions:

ECS Task Definitions: Add a JSON template for ECS task definitions to launch DefectDojo containers.
Service Setup: Provide AWS-specific instructions for setting up Fargate or EC2-based ECS services.
Networking: Ensure documentation includes necessary configurations for security groups, load balancers, and IAM roles for AWS ECS compatibility.
Documentation: Extend the current deployment documentation to include a section for AWS ECS, complete with step-by-step instructions.

Describe alternatives you've considered

Manually configuring ECS: While it's possible to manually configure ECS services, task definitions, and networking for DefectDojo, this process is cumbersome and error-prone, especially for users not familiar with AWS ECS configurations.
Using EKS (Elastic Kubernetes Service): While EKS is an option for AWS users, some organizations prefer ECS due to its simplified container orchestration model and Fargate's serverless container management.

Additional context
Updates to existing documentation:

How to Test

Deploy the updated version of DefectDojo using the provided ECS task definition.
Ensure the application runs without issues on ECS, both using Fargate and EC2-based instances.
Verify logging works with CloudWatch Logs based on the provided configurations.
Test scalability by increasing the task count and ensuring that the ECS service properly handles the additional load.

Checklist:

[x] ECS task definitions added.
[x] Documentation updated with ECS deployment instructions.
[x] Tested on both AWS ECS Fargate and EC2-backed ECS.

@Busola-TR As a project, we've tried to stay away from vendor specific deploys - AWS, GCP, Azure, OpenShift, etc. directly in this repo.
We are happy to take contributions of guides, code or other ways to deploy DefectDojo - that's why we created the Community Contribs repo where you'll find several items that have been contributed by the community but aren't strictly DefectDojo source code. We have things like GKE, Rancher and Cloud Formation templates already in there.
https://github.com/DefectDojo/Community-Contribs
I'd be happy to approve a PR for this in Community Contribs - it appears like you've done this and have some experience you could share with the community which would be greatly appreciated by those with similar situations.


[FEATURE] Enhance LLM-Powered Explanations for Edge Cases and Customization

Description
Currently, ExplainableAI's LLM-powered explanations provide general insights into model predictions, which are helpful in most cases. However, there is a need for more tailored explanations when the model encounters edge cases, such as poor performance or imbalanced datasets. Additionally, there is no option for users to customize the level of detail in the explanations, which limits the tool's usability for both technical and non-technical users.
Problem it Solves
This feature will address the problem of generic LLM explanations by offering more specific insights when models perform poorly or encounter problematic data (e.g., outliers, imbalanced classes). It will also solve the issue of inflexible explanations by allowing users to control the depth of explanation, catering to both novice users and more advanced technical users who need detailed analyses.
Proposed Solution
Enhance LLM explanations to identify and explain edge cases, such as:
Poor model performance (e.g., low accuracy, high error rates).
Imbalanced datasets leading to biased predictions.
Outliers or anomalies in the data.
Add an optional parameter (e.g., explanation_level) that allows users to select between a high-level summary for non-technical users or an in-depth analysis for those who require detailed insights, such as feature importance breakdowns and model-specific diagnostics.
Alternatives Considered
One alternative would be to manually explain these edge cases by interpreting the results post-hoc, but this process can be time-consuming and inefficient, especially for non-experts. Additionally, separate documentation or tutorials could be provided to explain these scenarios, but having it built into the tool as an automated feature would be much more user-friendly and efficient.

@ombhojane I've completed the enhancement with EDA and SHAP integration for the LLM explanations. Let me know if you need any changes!
Thanks!

hey @nazeyanehal can you mension the link of PR


Cannot edit workflow version and set Workflow Path

Bug Report
Cannot edit workflow version and set Workflow Path
Expected behaviour
New Workflow Path is displayed in dialog
Actual behaviour
Error message is displayed:
The webservice encountered an error.
[HTTP 413] OK:
413 Request Entity Too Large

413 Request Entity Too Large
nginx/1.13.1

Steps to reproduce the behaviour
Go to https://staging.dockstore.org/my-workflows/github.com/DataBiosphere/topmed-workflows/UM_variant_caller_wdl
Click on feature/freeze8-wdl on Versions tab
Click Edit link
Change Workflow Path to /variant-caller/variant-caller-wdl/topmed_freeze8_caller.wdl
Click Save Changes
Environment (Browser or OS and Dockstore version)
https://staging.dockstore.org/api/static/swagger-ui/index.html#
https://github.com/dockstore/dockstore-ui2/releases/tag/2.4.0-rc.0
Google Chrome Version 76.0.3809.100 (Official Build) (64-bit)
Mac OS Mojave

┆Issue is synchronized with this Jira Story
┆Issue Type: Story
┆Fix Versions: Dockstore 1.7
┆Sprint: Seabright Sprint 18 Skipjack
┆Issue Number: DOCK-961

Probably  a duplicate of #2833
Also @wshands, please remove the template instructions

I'm guessing it's the same API caused from a different place as in #2833; I wonder if it's worth creating an Angular interceptor to handle this instead updating all calls to this particular API. OTOH, these may be the only two calls, so maybe it's overkill.

➤ Gary Luu commented:
Much easier if someone has access to that repo reviews this instead.

@wshands could you invite @garyluu  to DataBiosphere?

@garyluu you should get an email invite

➤ Gary Luu commented:
After creating another workflow with the same info and then changing the workflow path, I no longer get the error.


Translate system to System on connect arguments

This shouldn't be needed long term once the driver fix is in place.  Not even sure this is the best way to do this.

merged in 5ac827d6e411a500600dd24ac2ccb37a2d2ea070


MIgration of //kccapi:spec

The first issue has been solved. In particular, SpecExtractor has been refactored and comments have been added. The relative WORKSPACE and BUILD files have also been updated to reflect these changes. It is now possible to run bazel build //:spec in the root folder as requested.
Tests for the SpecExtractor have not been added as the only ones I could come up with would only check if the outputs files were indeed generated - my reasoning for not adding them is that the relative path of the output results will change depending on where the .jar file (and corresponding bazel command) is executed; therefore they are easy to break and not really meaningful in terms of the task's success. However, I am open to a second opinion!

Could you merge this into a single commit?


test: Removed no longer supported -U option from tox.ini

Codecov Report

Merging #209 (fd8a677) into master (f18ec3b) will decrease coverage by 0.05%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##           master     #209      +/-   ##
==========================================
- Coverage   94.21%   94.16%   -0.05%
==========================================
  Files          66       66
  Lines        6223     6223
==========================================
- Hits         5863     5860       -3
- Misses        360      363       +3

Impacted Files
Coverage Δ

stix2validator/v20/shoulds.py
95.65% <0.00%> (-0.47%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Added MySQL Pool metrics

The current MySQL probe doesn't work with MySQL Pools, this enables that functionality.

@rwky before we can accept your pull request, please can you fill out a CLA agreement https://github.com/RuntimeTools/appmetrics/blob/master/CONTRIBUTING.md

Would love to except it says I need to print, sign and scan the doc, I won't be any where near my printer/scanner until August. Can I digitally sign it?

@rwky yes, assuming it looks OK, that should be fine, thanks.

Attached doc gpg signed will that do?
Archive.zip

@rwky Can you send that document to<EMAIL_ADDRESS> I had a look but couldn't see any details in it related to you

Check the .asc file it has my gpg signature.

Sorry @rwky, I'm not able to resolve your .asc file to the personal details requested in the CLA form.. I suggest you print it down, complete, scan in/photo it, then send to<EMAIL_ADDRESS>.
(I realise this process is a little awkward, I'm investing moving to the CLA process built into github, hopefully this will be much smoother in future).

Forget it I stopped using this module it caused issues when in production.


Weird sample variance when creating a lognromal sample with pre-defined mean and variance

I have been working with lognormal distributions as a proposal distribution for some MCMC routines which require the proposal distribution to have some pre-defined sample mean and variance, m and v, respectively. Considering that I'm using scipy.stat.lognorm to generate a sample from the proposal distribution, I have to define mu and sigma^2, parameters of the underlying normal distribution.
I've determined mu and sigma^2 using the relations: mu=ln(m / sqrt(1 + v / m^2)) and sigma^2 = ln(1 + v / m^2). This relation is easy to derive and its also on the Wikipedia article about the Lognormal distribution.
This has worked when m >= 1, however when m < 1 the variance of the generated sample becomes smaller than it was supposed to be (v) and I am not sure why.
To reproduce this behaviour, I ran the code
def get_underlying_normal_params(m, v):
    m2 = m * m
    nrm_mean = np.log(m2 / np.sqrt(v + m2))
    nrm_var = np.log((v + m2) / m2)
    return nrm_mean, nrm_var

lognrm_mean = 1e-5
lognrm_var = 10
norm_mean, norm_var = get_underlying_normal_params(lognrm_mean, \
        lognrm_var)
sample = [lognorm.rvs(scale=np.exp(norm_mean), s=np.sqrt(norm_var)) \
        for _ in range(50000)]
print("Sample mean: ", statistics.mean(sample))
print("Sample variance: ", statistics.variance(sample))

the produced output is:
Sample mean:  1.3404377964314877e-06
Sample variance:  5.009857166696648e-09

We can see then, that the created sample has variance ~5e-9, far from the desired variance, 10.
If however, I run the same code setting lognrm_mean = 1, the output is:
Sample mean:  1.0077730236251317
Sample variance:  9.250922743416222

And now the sample variance is actually close to the desired variance.
Why was I able to pre-determine the variance of the sample on the second case but not on the first case?
I am not sure that what I did is conceptually correct, but I suspect that this could have been caused by floating point errors; if that was the case, does anyone have an idea of how to investigate further?

Hi,
I did not check your description in detail but maybe the following hint in the documentation helps:

A common parametrization for a lognormal random variable Y is in terms of the mean, mu, and standard deviation, sigma, of the unique normally distributed random variable X such that exp(X) = Y. This parametrization corresponds to setting s = sigma and scale = exp(mu).

Hi!
Thanks for the reply! I was aware of this detail, so I think that is not the issue. You can see that I have addressed this possible issue in the line:
sample = [lognorm.rvs(scale=np.exp(norm_mean), s=np.sqrt(norm_var)) \
        for _ in range(50000)]

The SciPy lognorm variable X(s) corresponds to the one on Wikipedia with s=Sigma and mu=0.
You can adjust s to match your desired variance of 10 solving the formula of the variance given on Wikipedia for sigma. it should give you a value of ~1.144
lognorm(1.144).var() # 9.99955146185357
Now you can use loc to shift X(s) to get the desired mean by considering loc + X(s). Note that shifting does not affect the variance. The mean of X(1.144) is:
lognorm(1.144).mean() #1.9239262116984597
So take loc = 1e-5 - 1.9239262116984597.
x = lognorm.rvs(s=1.144, loc=loc, random_state=1234, size=int(1e6))
x.mean() # 0.0009755198570579801
x.var() # 9.943471830901856

I hope that helps. Closing as there seems to be no issue.

Thank you, but I needed to generate positive numbers, and the loc parameter allows the distribution to have negative numbers with non-zero pdf.
I guess this could work if I change my application to allow negative numbers with small probability.

You are right, the above approach gives you a shifted lognormal distribution but you are looking for a random variable that can be written as exp(sigma*N + mu) where N is standard normal.
If you use the scale parameter, you get the following random variable in SciPy:
scale * exp(s*N) = exp(log(scale) + s*N), i.e. mu = log(scale) and s = sigma as stated in the documentation.
m = 1e-5
v = 10

mu = np.log(m/np.sqrt(1+v/m**2)) # -24.2
sigma = np.sqrt(np.log(1+v/m**2)) # 5.0

scale = np.exp(mu)
s = sigma

# mean and var are as expected
lognorm(s=s, scale=scale).mean() # 9.99999999999998e-06
lognorm(s=s, scale=scale).var() # 9.999999999999947

Note that these are quite extreme parameters for mu and sigma. If you sample from them, the empirical var is indeed not close to 10. (as you observed before)
x = lognorm.rvs(s=s, scale=scale, random_state=1234, size=int(1e7))
x.var() # 0.000248654531158192

I don't know if this is due to numerical issues or if you need more samples for such parameters.
So let me reopen the issue… I don't have time to investigate it at the moment

I've traced the numbers through the random variate generation and I'm not seeing evidence of a loss of precision or other numerical issue.
We get the same sort of thing with np.random:
rng = np.random.default_rng(8932598345)
x = rng.lognormal(mu, sigma, size=int(1e7))
x.var()  # 0.00024264448171329683 != 10

Most convincing to me that this is not due to a bug in SciPy/NumPy is that the mean and standard deviation of the log of the random variates are as expected.
y = np.log(x)
y.mean()  # -24.177120820652654 ~ -24.2
y.std()  # 5.032397460979327 ~ 5

So if the formulas for the required normal mean and variance are correct, it seems that the "problem" is inherent; that is, the distribution of the sample variance is unfavorable, and as @chrisb83 suggested, we've need (lots) more samples to observe sample variance closer to the population variance.
Come to think of it, we have observed a related issue before in gh-15723, in which lognorm and gibrat (special case of lognorm with s=1) were two distributions that failed the sample variance test.
Perhaps @rkern, @josef-pkt, or @steppi have deeper thoughts here?
Either way, I think this can be closed. Please feel free to reopen if I've missed something.


Box Drive broken functionality

Description
When Files first loads, the cloud drive sidebar does not show Box Drive or OneDrive initially; when toggling the dropdown to hide and then show, the drives will become visible. When Box is selected, the root directory is shown but refreshes/reloads every few seconds. When entering one of the folders before it refreshes, its contents are empty.
Steps To Reproduce

launch Files
toggle cloud dropdown
select Box
enter folder in Box Drive before refresh

Expected behavior

launch Files
Box Drive should appear in cloud drive sidebar dropdown menu
after selecting Box Drive from the sidebar, the root directory should be static and not keep refreshing
entering a Box Drive folder should show its contents and work as a typical directory

Files Version
v2.0.13
Windows Version
Windows 10 Enterprise 20H2
Relevant Assets
debug.log
GetDrivesAsync|Drive added: C:\, Fixed
EnumerateDrivesAsync|Adding cloud provider "Box" mapped to C:\Users\username\Box
EnumerateDrivesAsync|Adding cloud provider "OneDrive" mapped to C:\Users\username\OneDrive

debug_fulltrust.log (nothing specifically Box related shown)
ConnectionRequestReceived|Argument: GetIconOverlay
[repeat x60]
.
.
ConnectionRequestReceived|Argument: Terminate

Merging with #7428


Hillshading source/layer

It is currently not possible to add raster_dem source and hillshading type of layer as in this example:
https://www.mapbox.com/mapbox-gl-js/example/hillshade/
Available since mapbox-gl-js 0.43.

Closed by #263


Reimplement Japanese segmenter

Pull Request
What does this PR do?
Fixes #89
PR checklist
Please check if your PR fulfills the following requirements:

[x] Does this PR fix an existing issue?
[x] Have you read the contributing guidelines?
[x] Have you made sure that the title is accurate and descriptive of the changes?

Notes

This PR adds a japanese feature flag which depends on lindera and lindera-core. These are the crates performing the actual segmenting. The two Japanese dictionary features of lindera are used. I have not checked whether both are strictly necessary.
I have added only one unit test. The benchmarks seemed to already include Japanese text, so I assumed that there was no need to add another one.
The existing Chinese normaliser was previously triggered on script ==  Cj, which meant it was also triggered on Japanese text. I have made it stricter to avoid normalising Japanese text. Now it only triggers when the language is known as Cmn.
lindera's tokenize_str method returns a Result. It is not clear to me on what conditions this method fails. A previous implementation of the Japanese segmenter on this repo unwraped the result.  I have done the same thing, but have low confidence on the correctness of that approach.
Japanese compounded words are decomposed by using the following configuration:

TokenizerConfig { mode: Mode::Decompose(Penalty::default()), ..TokenizerConfig::default() }

I have tried to avoid initialising Linderaevery time by storing it in a static Lazy, as recommended in the dummy example.

Good job @loiclec! Thank you for your contribution!
bors merge

@ManyTheFish you need to approve to be able to merge 😇


Additional description in the frontend

The additional description does not show in the frontend
This would be great to display a message to the customer about that payment method like a surcharge

Something like this:

Hi, thank you for your input, we have put it on our backlog.

Will this be fix, cos this looks like is a small bug

Hey @TaiKamilla it's coming in the next release, going to be published this week.

Closing, since it has been released.


Update main.go

added mount options to allow other users to access directory especially root
this is useful in situations such as nginx that uses uid 1 to ping a directory yet user: www-data is the one performing IO options

I added a bool flag --allow-other to specify if other users can access the folder.


Python: Enable core skills, retry mechanisms and template engine protocols to be used as pydantic fields

Summary
This is part 1 of the Serialization work for semantic kernel.
Corresponding issue: https://github.com/microsoft/semantic-kernel/issues/131
Motivation and Context
This PR allows core skills (all except 2), retry mechanisms and template engine protocols to be used
as valid pydantic fields. This allows me to convert the classes that depend on these classes to pydantic
models, and thus be able to serialize them.
No user facing code changes are expected from this PR.

Description

Contribution Checklist

[x] The code builds clean without any errors or warnings
[x] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[x] All unit tests pass, and I have added new tests where possible
[x] I didn't break anyone :smile:

I thought that the PR was getting too big and that I should move it to a
separate PR, but now that I say it, I realize it sounds very silly. I will
update this pr with those changes as well. Thank you for pointing this out.
On Wed, Aug 2, 2023 at 4:54 PM Mark Karle @.***> wrote:

@.**** approved this pull request.
In python/tests/unit/test_serialization.py
https://github.com/microsoft/semantic-kernel/pull/2287#discussion_r1282511734
:

+from semantic_kernel.core_skills.web_search_engine_skill import WebSearchEngineSkill
+from semantic_kernel.sk_pydantic import PydanticField, SKBaseModel
+from semantic_kernel.template_engine.protocols.code_renderer import CodeRenderer
+from semantic_kernel.template_engine.protocols.prompt_templating_engine import (

PromptTemplatingEngine,
+)
+from semantic_kernel.template_engine.protocols.text_renderer import TextRenderer

+PydanticFieldT = t.TypeVar("PydanticFieldT", bound=PydanticField)
+
+
@.***(

"sk_type",
[

   pytest.param(

       ConversationSummarySkill, marks=pytest.mark.xfail(reason="Contains data")

Why does the conversation summary skill not get serialized if it contains
data?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/semantic-kernel/pull/2287#pullrequestreview-1559980151,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABI5YNKJRZKML3QAE53IN73XTLSCFANCNFSM6AAAAAA3BW4ZHU
.
You are receiving this because you authored the thread.Message ID:
@.***>


fix: fix the problem that Black background when converting SVG to PNG

When I parse a style attribute with a global style tag and a stop tag in a Graid component, I will encounter the problem of the above attribute parsing failure,and resulted in the problem that Black background when converting SVG to PNG. I would classify it as the parsing problem of the style attribute, so I submitted this PR to solve the above problem.

stari, can you please check if pull request #40, Support top level style solves this problem? I just merged it today into the main branch.

@srwiley , I just merged the above top style code #40 , but I also solved another problem other than GradStop's rgb color not being resolved, you can see the changes I made to StopF function.

@srwiley asked for my input in #40, so I'm giving it:
I don't know a lot about svg gradients, so I can't say for sure that this solution is good. However, the implementation seems plausible to me. Parsing the contents of style tags within a gradient to obtain gradient related information sounds good.
This in addition to the added tests leads me to believe that this PR is good and should be merged.
I am a bit confused about the chinese comments in the tests though. Since the rest of the project is in english, these should probably be translated.
I did translate the comments using google translate and they are things like "read svg" and "define length and width", which are the kinds of comments that can in my opinion be removed entirely.
Apart from that, the PR looks good to me.

Sorry, I accidentally submitted some Chinese comments into the code.
I removed the comment just now.
Thanks!

Sorry for taking so long to get back. I've been busy elsewhere.
I can't accept this pull request as is. The main problem is that in order fix the GradStop bug you have rearranged 8 files. This makes it hard for me to see exactly what you changed because such larger blocks have been moved around. In addition, generated pngs should not be included with source code. Also, I can't see where the fix alters the output from the last merge in the example svgs. You also add the style function to the draw function maps twice, which is unnecessary.
I am sure there is something good here, but please do the minimal alterations to the the existing structure, such as do not split out functions into individual files.