Fix when BoundingBoxGizmo issues warning about setting attachedNode

PR to move the warning about setting the attachedNode property of the BoundingBoxGizmo into the setter for attachedNode to make sure that the warning is only logged when the attachedNode property is set. Currently there are a few commonly occurring cases when the warning will be issued even when the property isn't being used.
Forum: https://forum.babylonjs.com/t/warning-logged-when-gizmomanagers-boundingboxgizmo-is-enabled/31644/4
Test PG's: 1. https://playground.babylonjs.com/#6RARFS#7
2. https://playground.babylonjs.com/#6RARFS#5

Hmm I wonder why the build failed? I'm able to build locally successfully using  "npm run build:dev"

This is the error:
https://dev.azure.com/babylonjs/ContinousIntegration/_build/results?buildId=20769&view=logs&j=275f1d19-1bd8-5591-b06b-07d489ea915a&t=b4978d5e-5988-54a5-1480-26654484e930&l=276
Super cannot be used in assessors like that, especially not if it is seeing anything private. The get is not needed, the set can be modified to use a protected variable in the parent
I wonder why it compiles locally. When was the last time you ran npm install?

Super cannot be used in assessors like that

Aah my bad, I was doing it a way that was recommended for ES6 that lets the typing get sorted out automatically. I'll trying running install again too, it's been quite a while... Thanks for the help!

Will convert to draft for now, not as straightforward as I thought it would be...

Will just change this to a draft for now, it's not as straightforward a solution as I thought...


feat(image): adds method to create Associations from remote sources

Signed-off-by: Jennifer Power<EMAIL_ADDRESS>Description
This PR adds methods to allow AssociationSet information to be gathered from local and remote sources.
Related to #302
Type of change
Please delete options that are not relevant.

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

How Has This Been Tested?

[x] Unit tests added for remote layer association collection

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

cc @afflom this is ready for logic review

@soltysh PTAL (priority #1). Thanks!


Removed CSC specific inventory items

Removed Austin lab specific inventory items from project.

Can you also update ansible.cfg with inventory = inventory/hosts.ini.  No reason to have it pointing back to wiley and `hostfile' is depreicated.

Code review :+1:

:ok_hand:


setting the timer to 0 leads to constant buzzing from the watch

It fires immediately, auto-flips, and buzzes until the app is closed.

Setting the timer to 0 now does nothing. The user is sent back to the grill menu and state is set to INIT. Alternatively, we could send the user back to the picker.


weightless docker image

Description of Problem: rasa/rasa image is big (~1.3GB). I'm trying to shrink the image, would you be interested if the weight difference is important ?

Overview of the Solution: build requirements from scratch on an alpine image and remove non-necessary runtime system dependencies

Examples (if relevant):

Blockers (if relevant):

Definition of Done:

[ ] install scipy / openblas https://github.com/scipy/scipy/issues/9481#issuecomment-510540522
[ ] install tensorflow / bazel
[ ] clean up non-necessary runtime dependencies once builded

@tormath1 sure, that would be interesting to see. Not sure how many of the requirements you can actually get rid of. Is this something you want to test out yourself?

Yes, totally. I would be glad to share the result with you !

@tormath1 Thanks for your offer! Could you please also check with https://github.com/RasaHQ/rasa/pull/3332 ? In this pr I'm trying to achieve the same :-) Let me know if you have suggestions!

@tormath1 One more question:
Building tensorflow from scratch in an alpine image is quite tedious. Don't you think the resulting image definition would be quite complex / hard to maintain?

@wochinge, well I guess I won't be harder than any dependencies. Actually, tf should release an Alpine wheel. :sweat_smile:
Building dependencies from sources could optimize built wheels as we are compiling through GCC. I'll let you know how's going.
Good work for your slim image. :+1:

This is my first stage:

Do you have some estimates regarding the image size?

Hi guys, I have already built an alpine version and the estimated size is a bit under 2G decompressed as opposed to the 2.7G from rasa/rasa:latest-full. I think it's not a huge reduction and definitely not worth the build time if it's just for space optimization, however, there are other arguments in favor of alpine based images, particularly for enterprises. In our case, our private registry vulnerability scan complains about anything but alpine and from scratch. Too paranoid? Maybe, as I say it takes about an hour to build on CI/CD runners.

@kronos-cm seems about the space reduction I got in https://github.com/RasaHQ/rasa/pull/3332

Hi guys, I have already built an alpine version and the estimated size is a bit under 2G decompressed as opposed to the 2.7G from rasa/rasa:latest-full. I think it's not a huge reduction and definitely not worth the build time if it's just for space optimization, however, there are other arguments in favor of alpine based images, particularly for enterprises. In our case, our private registry vulnerability scan complains about anything but alpine and from scratch. Too paranoid? Maybe, as I say it takes about an hour to build on CI/CD runners.

@kronos-cm: We are running into the same vulnerability scan issues and most of the issues are around Debian OS. I am trying to built on alpine version. It seems you were able to successfully built BOT Servers on alpine. Could you please share your docker file. Thank you.

Hi @HariTeneti, yes, we were able to do it compiling tf==1.14 from scratch with bazel, and rasa=1.2.3. Unfortunately, per my contract, I am not able to share any files publicly, though I would love to I've you a hand if possible.


Add missing transaction names

Dennis reviewed this when it was part of #287, so I'm going to force-merge.


Blind Fighting does not work on large enemies right next to me

I am in darkness (no darkvision), a troll right next to me My fighter has blind fighting but still has disadvantage on attacking the troll.

A friend of mine hosted the game, I asked him for the save and will add it as soon as I get it.

Game has this limitation. Distance calculation is done in a very odd way with large enemies. We tried to fix that in other places. Were you using the lightning and obscurement setting in mod?

Yes, we have "lighting, obscurement and vision" enabled

BLIND_FIGHTING_BUG.zip
Finally got the save. Sorry it took so long.

This is a very hard use case to fix because engine has some limitations. Sorry.

It's okay. At least I now know what I can do to avoid it. Thanks for looking into it.


Add telemetry option

This PR adds the telemetry option to consumer so that consumer can opt in/out the registry telemetry when using operator to deploy the registry.
Signed-off-by: Jingfu Wang<EMAIL_ADDRESS>

@johnmcollier Comments addressed, please review again.


Question: how to run icons through loaders?

For the extension that I'm building, I've got one core icon which is an SVG, and I was hoping to use vite-imagetools to process that into a bunch of different sized PNGs at build time, but I couldn't tell how to get the two vite plugins to work together.
Is it possible to do something like #52 for the icons as well?

What happens when you try to use that plugin with this one? Are the directives not applied to images or is there some other issue?


Feature Request: Mesh thickening

I'd like to be able to thicken the mesh within vedo such that the pointdata is retained.
For inspiration: https://pymadcad.readthedocs.io/en/latest/_modules/madcad/generation.html#thicken

isn't it already possible with extrude():
from vedo import *
letter = Text3D("A") #can be any Mesh
letter.c("blue6").pickable(True)
letter.pointdata["mydata"] = range(letter.npoints)
letter.cmap("Set1")
eletter = letter.clone().extrude(0.1)
eletter.cmap("Set1")
eletter.print()
show(letter, eletter, N=2, axes=1)

The extrusions are made based on the normals of the points rather than in a constant direction. It should also handle overlapping regions when the points collapse on top of each other. You can do a contour offset in equal amounts in both the positive and negative directions of the normal so the centre of the thickened object remains the same.

this is only available in the forthcoming release:
git clone https://github.com/marcomusy/vedo.git
cd vedo
git checkout restruct_mem
pip install -e .

from vedo import *
letter = Text3D("A")
letter.pointdata["mydata"] = range(letter.npoints)
letter.cmap("Set1")
eletter = letter.clone().extrude(0.3, direction=(1, 1))
print(eletter)
show(letter, eletter, N=2, axes=1)

The thickening is moreso like inflating and deflating a balloon by a centimetre. Unless the point normals can be passed into direction?
This visualisation might help: https://pymadcad.readthedocs.io/en/latest/reference/generation.html#madcad.generation.thicken

oh for that i see  2 options:

use https://vedo.embl.es/autodocs/content/vedo/vedo/pointcloud.html#Points.implicit_modeller
compute_normals(cells=False) then manually move the vertices along the normals. To avoid self collisions I would move them by averaging the normals over a region of the mesh (by 3d closeness, or better by looking at the connecting vertices).

(this madcad is pretty cool btw!)

The link you shared doesn't seem to point to anything. I couldn't see implicit_modeller.
For the second option, it'll be too computationally slow in Python for dense meshes. The normal approach is to do the contour offsetting and merge points that collapse into each other to my knowledge.
A third option (that I haven't explored yet) could be supporting madcad (glad you like it, it's been quite nifty!). I'm not sure if it's possible to include attributes and collect them back after the operation is done within madcad.

Ops sorry, another miss in my release docs.. it's:
https://vedo.embl.es/autodocs/content/vedo/vedo/pointcloud.html#Points.generate_surface_halo
How "dense" is dense for you? I would say it should be ok up to 100k vertices.
Third option, would be cool to explore, i'm also sure it should be possible (it's just a bunch of numpy arrays in the end).

Thanks. The surface halo is in all directions rather than just the normal so it doesn't fit the particular use case (which is to extrude a flat mesh boundary and then thicken it).
In terms of density, most cases I'm working with will be probably 120-200k but I also need to be able to accommodate for the outliers.
I've created a feature request for madcad. I'm not too familiar with it under the hood but happy to give it a go in a week or two.

Moving vertices along the normal is very fast even for meshes of 438k:
from vedo import *
msh = Mesh("https://www.dropbox.com/scl/fi/l28380iuhudaanl5uajf4/dragon.ply")
msh.compute_normals(cells=False, points=True)
for i in progressbar(10):
    normals = msh.pointdata["Normals"]
    msh.vertices += normals*0.0005
show(msh)

the problem is to deal with self collisions..

Hi @marcomusy! Finally working on this now. Just writing it with vedo itself. All that's left is to copy across the cell and point data to the new mesh (or to update the points, edges, faces on the existing one. Is there a simple/recommended way to do that?

Hey, in principle you don't even need to do anything if you just replace/modify the vertex coordinates, all data stays the same.

Because we're offsetting the vertices in both directions, there will be double the number of vertices, so there will also be new ones (and new faces) [will be less than double for intersecting offsets].


Leave original element when cloning

Using the data-clone-enabled option, is there an easy way to leave the element in its place while the drag is in progress?  It seems to temporarily remove the element and drop it back in place once the drag/clone is complete

@lewism36 I don't quite catch your question. Using clone-enabled, the original element should not have been changed.

Hi,
As a basic example I added the data-clone-enabled attribute to the trees in the ‘connected trees’ example.
When dragging a node from the left tree to the right, the left list reduces to 3 items until the node is dropped on the right list, then it appears again in the left list once it has been dropped on the right - I am looking for a way for the left list to remain intact at all times - i.e. the clone occurring as soon as the drag starts.
A further issue I found is that the underlying model object for the node is not actually cloned - it is just added to the other array - so the same object exists in both arrays - this is ok until it is dragged back again or the same item is cloned once more.
This then causes a Duplicate Key in Repeater error in Angular because the same item exists twice in the same array.  I worked around this second issue by cloning the item myself in the dropped event but it would be useful if this happened automatically (using track by $index on the repeater caused strange issues when the trees had nested items).
hope that makes sense
Lewis
Sent from Windows Mail
From: Niels Dequeker
Sent: Saturday, 2 May 2015 16:14
To: angular-ui-tree/angular-ui-tree
Cc: lewism36
@lewism36 I don't quite catch your question. Using clone-enabled, the original element should not have been changed.
—
Reply to this email directly or view it on GitHub.

I've got into exact same issues with duplicating upon dragging a clone. Could you please share your solution?
This should be done be default, though.
And item should never disappear from source when cloning is enabled, or at least there should be an option for that. That makes a lot sense.

@fyodorvi  It's a bit messy, but I basically keep a local copy of my source array (it never needs to change after the controller initializes and populates it from the server), then in the drop event handler of the source tree I empty the array that the source tree is bound to, and fill it with clones of each item from the local array.  I then also update the event.source.nodeScope.$modelValue.$$hashKey to be something unique.
I did first try cloning only the dropped item from the bound array, removing the original, then adding back the clone, but this didn't seem to work.

@lewism36 I though about doing it your was, but yeah, it messy. So I edited ui-tree source. I modified dragInfo.appy function a bit, the last line that actually inserts node now inserts a copy of it if cloning is enabled:
Was:
this.parent.insertNode(this.index, nodeData) ;

Now:
this.parent.insertNode(this.index, this.source.$treeScope.cloneEnabled === true ? angular.copy(nodeData) : nodeData) ;

Which to my mind makes much more sense.
To solve problem with disappearing item (just like you described) I modified dragStart function a bit: I don't remove source element if cloning is enabled (which is, again, reasonable for me), and don't add placeholder to source list:
Was:
   scope.$element.after(placeElm);
   scope.$element.after(hiddenPlaceElm);
   dragElm.append(scope.$element);

Now:
   if (scope.$treeScope.cloneEnabled) {
      dragElm.append(scope.$element.clone());
   } else {
      scope.$element.after(placeElm);
      dragElm.append(scope.$element);
   }

I hope this might help you.

Last modification is not so smart, though. It assumes that you don't allow drop of clone on the source tree, so it should be treated as a must-have. Ideally there should be separate setting for that.

thanks @fyodorvi - that's a good solution

Actually I think there's a line missing from the else.  should it be:
if (scope.$treeScope.cloneEnabled) {
dragElm.append(scope.$element.clone());
} else {
scope.$element.after(placeElm);
scope.$element.after(hiddenPlaceElm);
dragElm.append(scope.$element);
}

@lewism36, sorry, it should be like that:
scope.$element.after(hiddenPlaceElm);

 if (scope.$treeScope.cloneEnabled) {
      dragElm.append(scope.$element.clone());
   } else {
      scope.$element.after(placeElm);
      dragElm.append(scope.$element);
   }

hiddenPlaceElm should always be there

great, thanks

Why was this reopened?  If it was an accident, can OP close it again?

Is it fixed?
On Sat, 13 Jun 2015 02:53<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:

Why was this reopened? If it was an accident, can OP close it again?
—
Reply to this email directly or view it on GitHub
https://github.com/angular-ui-tree/angular-ui-tree/issues/446#issuecomment-111653798
.

Closing this issue due to inactivity. See #815 for more information.
Please create a new issue is this still occurs with the current version of the component.


wrong declaration in allon_sensor_tflm.h

I downloaded your SDK yesterday. I see that by default the application is allon_sensor_tflm_freertos.
This did not build because the .h file declares int rtos_app(void); which does not exist. When I changed this to int app_main(void); it builds.

Thank you for your reminder. We update it.


Prepare for 0.2.0 release

We plan to release 0.2.0 soon. A checklist:
[x] update readme
[ ] fix CI of the examples

Fix the seed of random_split. Hopefully this fixes the randomly occurring batchnorm issue.


Credits section

Description
Credits section at the bottom of the page gives honor to our open-source contributors.

Should we really need it?
it is already present at the readme

Hey @Pavan-Sai-padavala After 1 month, after considering active open source contributor. I will add Official Team Section on Frontend Freaks.
We will not add whole open source contributor. So for now let's close this issue


For fans work in progress

dev
JIRA

Main JIRA ticket

Code reviewers

[ ] @github_username

Second Level Review

[ ] @github_username

Summary of issue
ToDo
Summary of change
ToDo
Testing approach
ToDo
CHECK LIST

[ ]  СI passed
[ ]  Сode coverage >=95%
[ ]  PR is reviewed manually again (to make sure you have 100% ready code)
[ ]  All reviewers agreed to merge the PR
[ ]  I've checked new feature as logged in and logged out user if needed
[ ]  PR meets all conventions

I see from name that this PR is in progress, but I'd suggest to use Draft PRs instead


Mapping to ISO codes

Is there an easy way to map the area codes in the data to the GB ISO-3166-2 codes (https://de.wikipedia.org/wiki/ISO_3166-2:GB)? Help is much appreciated!

Not seen any tables using those ISO 3166 codes, but the stuff at https://geoportal.statistics.gov.uk/datasets/lower-tier-local-authority-to-upper-tier-local-authority-april-2019-lookup-in-england-and-wales is quite useful for navigating up and down the geographical hierarchy the UK govt does use (at least in England and Wales); I think counties and boroughs are in there.  For Scotland https://statistics.gov.scot/data/data-zone-lookup might be useful.

Thank you very much. I have another dataset that uses the ISO codes and I need to match them. I can share the matching table once I've created it if that is interesting to you.

@danielw2904 please share! Thanks.


How to debug this case

Hi,
I created a custom template for Launching RDS inside  a VPC from quickstart-magneto. When I tried to debug with -v option, however  this Error Message is not helping me.
DEBUG  ] :jsonstatus = True
[PASS   ] :Validated [defaults.json]
[INFO   ] :|PREPARING TO LAUNCH => defaults
[INFO   ] :No cleanup value set
[INFO   ] : - (Defaulting to cleanup)
[INFO   ] :|Acquiring tests assets for .......[defaults]
[DEBUG  ] :|S3 Bucket     => [taskcat-test-ababa]
[DEBUG  ] :|Project       => [rds]
[DEBUG  ] :|Template      => [https://s3-ap-southeast-1.amazonaws.com/taskcat-test-ababa/rds/templates/rds.yaml.template]
[DEBUG  ] :|Parameter     => [https://s3-ap-southeast-1.amazonaws.com/taskcat-test-ababa/rds/ci/defaults.json]
[DEBUG  ] :|TemplateType  => [yaml]
[DEBUG  ] :|Defined Regions:
			 - [ap-southeast-1]
[PASS   ] :(Completed) acquisition of [defaults]

[INFO   ] :Preparing to launch in region [ap-southeast-1]
[DEBUG  ] :AutoGen values for $[alfred_genpass_32]
[DEBUG  ] :Auto generating password
[DEBUG  ] :Pass size => 32
[DEBUG  ] :Pass type => default alpha-numeric
[ERROR  ] :expected string or bytes-like object
[FAIL   ] :Cannot launch rds.yaml.template

The template and Parameter file I used are: https://github.com/chrisduong/quickstart-magento/blob/rds-only/templates/rdsmysql-only.yaml.template, https://github.com/chrisduong/quickstart-magento/blob/rds-only/ci/rdsmysql-only.json
What does this error mean?  I had tested manually my template in CFN console, it works with the current Parameter JSON file I used.
[ERROR  ] :expected string or bytes-like object

@sshvans can you take a look at this and see if its related to https://github.com/aws-quickstart/taskcat/issues/39 (If so please re implements that fix) as it was reverted to an emergency merge)

@chrisduong I investigated the issue and found that the problem is the parameter value of "Subnets" in rdsmysql-only.json file. The value should be of type string. I would suggest to update it to something like "subnet-cccccc,subnet-dddd" instead of ["subnet-cccccc", "subnet-dddd"].

Thanks @sshvans

This issue is also triggered when json stack input value are not wrapped in double quotes
example:
[
  {
    "ParameterKey": "Param1",
    "ParameterValue": 1
  }
]

Working on fix to detect and convert

Fixed in https://github.com/aws-quickstart/taskcat/commit/fc2017fb07510c6eae8541cb892ff8c1bf7c31c4


Should we stop suppressing the override mistake for Function.prototype.apply?

At https://github.com/Agoric/SES-shim/blob/9a7a0975036d8d938263ba265f747876d7d91599/packages/ses/src/enablements.js#L67 we say that we added Function.prototype.apply to the enablements whitelist because of tape.
At https://github.com/Agoric/SES-shim/pull/473#issuecomment-697887937 @ljharb says that tape does not do an overriding assign to apply. If tape is no longer a problem, or if we were mistaken about whether it was a problem in the first place, and if we do not encounter any other such problems, we should remove Function.prototype.apply from the enablements whitelist.

As far as I can discern, tape has never done an assignment to apply.


Build problem: wrong maven dependencies

Hi,
I'm trying to build this project but some problem with dependencies rised.

Order and Payment modules use <camunda.version>7.7.3-ee</camunda.version> but this version doesn't exist in public repository
I've solved problem changing version to
<camunda.version>7.7.0</camunda.version>

These modules use

<dependency>
			<groupId>com.camunda.consulting.webapp</groupId>
			<artifactId>camunda-webapp-customized-webjar</artifactId>
			<version>7.7.2</version>
			<scope>provided</scope>
		</dependency>

this dependecy include another dependency which not exists
<dependency>
      <groupId>org.camunda.bpm.webapp</groupId>
      <artifactId>camunda-webapp-ee-plugins</artifactId>
      <version>7.7.2-ee</version>
      <type>jar</type>
      <classifier>classes</classifier>
    </dependency>

I've solved this problem removing this dependency camunda-webapp-ee-plugins from pom

this project use ee version of camunda by default.  just check the comment in the pom. you will find the answer.

Sorry for that - accidentially commited. Fixed now!


Links to classes are broken in Monitoring documentation

Steps to reproduce:

Visit Google Cloud documentation
Browse to Monitoring > V3 > MetricServiceClient #create_metric_descriptor
Click on the return type Google::Api::MetricDescriptor to see the class documentation

Expected result:

The documentation for Google::Api::MetricDescriptor is shown

Actual result:

404 Not Found

@remi Thanks for reporting this. I suspect we can generalize this issue, as it will probably be true for any linked type that does not have a namespace that includes the service name (in this case, monitoring).
This should only be a problem in the google-cloud package docs, because they are created by copying docs from the individual packages. In the google-cloud-monitoring docs, the page for google/api/metricdescriptor is present.

I can confirm the class documentation is defined in google-cloud-monitoring/lib/google/cloud/monitoring/v3/doc/google/api/metric.rb. Agree this looks to be a problem with the umbrella packaging... le sigh

@quartzmo Can you verify whether this affects the ML or GA candidate APIs? If so, we need to raise the priority of this issue.

To be clear, this affects only the GCP hosted documentation, and does not affect the user's local documentation or documentation hosted elsewhere.
The documentation application's use of an "umbrella" package has been problematic for us. The approach is basically duplicating documentation. So the same docs are found under google-cloud-monitoring that are also found under google-cloud. Because of the many issues we have had with this approach I do not believe it is sustainable, and we have proposed GoogleCloudPlatform/gcloud-common#218 as a way to link to navigation across packages instead of duplicating it.

@bjwatson I started work on this yesterday and should have the fix complete today. The answer to your question is yes, but I hope to have it closed before you need to change the priority.

@quartzmo Thanks! But too late to stop me from changing the priority. ;)


NAT Gateway Template

#303

I wish to see a test/example where an instance without a public IP is created and internet is accessible via the NAT Gateway.

@googlebot please verify my CLA status again.


Outdated Documentation?

Does this documentation need to be updated to include Jest (https://docs.microsoft.com/en-us/visualstudio/releasenotes/vs2017-relnotes)

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

ID: 3fdc4321-1bc2-215a-7696-c25bb6e5bf04
Version Independent ID: 13350cc1-b3f5-f01d-44c8-18dd1019ae22
Content: Write JavaScript code in Visual Studio without a solution or project - Visual Studio
Content Source: docs/javascript/develop-javascript-code-without-solutions-projects.md
Product: visual-studio-dev15
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

I'll ask about it

@Mikejo5000 We have added Jest support in release 15.9. It needs to be added to the list of supported frameworks.

@armanio123 OK, thanks, I'll add it to the list

Could it please be noted that the jest support does not work in TypeScript due to how tests are discovered.

@Saulzi Do you mean that the tests are not discovered? Can you be more specific?

They are not discovered when file is typescript but are when using javascript, this is because of how the 'glue' code for the new out of process testing works (https://github.com/Microsoft/nodejstools/blob/abe9726ef173b8d5546aa90e8735f190f679ddd8/Nodejs/Product/TestAdapter/TestFrameworks/Jest/jest.js).
Basically you are using the jest-editor-support package in the test discovery. (this means that you need to have this in your package.json or it will not detect tests in javascript code, does it need to be documented? as a pre-req?) unfortunately this npm package will not work with typescript files as it cannot strip types / traverse ast to find tests.
Perhaps a solution like chutzpah does for discovering tests may be required i.e. run jest and parsing output for discovery / running as this will have less dependencies.
One more point
likewise I have noticed in the 2019 documentation the following statement
'We have added support for enabling JavaScript debugging within unit tests in Node.js projects'
(https://docs.microsoft.com/en-gb/visualstudio/releases/2019/release-notes#jsts)
Is this javascript only or is running nodejs with sourcemaps also supported?

it does say about tests being compiled in test root to js (on existing documentation), jest is a bit of a funny one here as it can transpile typescript itself using babel7 or the ts-jest plugin before running tests..

@Saulzi OK, thank you @armanio123 For now, it seems I should include a caveat that jest is only supported in javascript, and that you need to include jest-editor-support npm package? That correct?
Also, see Saulzi's last question about using sourcemaps with JS unit tests. Thanks!

may sound stupid but note source maps are in the context of debugging unit tests written in typescript

Thanks for the details @Saulzi, TypeScript is supported in jest tests as well because Visual Studio first transpiles the files and then discovers test. By then jest-editor-support is able to find the test since now is a JavaScript file. I wasn't aware that jest is able to transpile through babel. It might be worth checking how can we implement this so that we remove one dependency.
Debugging is supported through sourcemaps and it should work on your jest tests.

More information overload below
Please see https://jestjs.io/docs/en/getting-started#using-typescript, I am currently using ts-jest
for transpilation for testing as it gives type / javascript errors as part of the test output. using jest with babel wont do this and the suggestion is to give the typescript compiler the noemit option and only use it for type checking here.
Previously we used chutzpah with jasmine using requirejs to load modules into the browser / phantomjs for testing but due to the large amount of tests for the frontend of the application to do a full test run would take over 30 mins (this is less than a min when running through jest and required little changes to get running). This was also somewhat flaky if there was a error in one of the files / loading a file as that suite would not run / get discovered.
My current solution is to use vs-code to run jest tests with tasks to allow debugging tests in node and launching the application attached to chrome, this is far from ideal as there currently I have found no nice way to debug individual scenarios within a module, I have to run all of the tests in the current module.
@armanio123 have you seen the new Vuejs templates in visual studio, out of the box when creating a typescript application the noemit flag is specified in the typescript compiler. I don't think that this is a uncommon scenario, likewise it is also common to compile source to different folder.
Our solution is similar to this, for development we watch all of the source files and emit them as AMD modules to a different folder which the browser can load in this way we get quick feedback without having to rebuild a lot of stuff (we have legacy / third party javascript alongside typescript source which is moved to this location), for deployment we then bundle / minify these artefacts as barrels and load functionality on demand.

I thought that I would give you an update on getting test discovery to work.
I managed to get some some of the tests written in typescript to be discovered although they will not run (spinner spins forever) in Visual Studio 2019 / 2017 by making the following changes to my solution.
Moved package.json to top of project and set options in project file to ./ and Jest. (would not work pointing at source subfolder)
Changed tsconfig to emit files alongside source as discovery only seems to support this scenario,
Changed module type to UMD so that it can be used in both node and the browser.
Changed jest config to run tests using emitted UMD rather than using ts-jest.
Please note that it only discovered 1304 (in project) +285 (in external?) out of 2046. note that this is just the core of the application.
The test hierarchy is somewhat incorrect in visual studio.

That doesn't sounds right. I suggest to file an issue in in https://github.com/Microsoft/nodejstools/ and share your project, or a project that reproduces if possible. That would make things much easier.

Note that documentation at https://docs.microsoft.com/en-us/visualstudio/javascript/unit-testing-javascript-with-visual-studio?view=vs-2017 is still missing jest.
I am in currently creating a list of issue's that I have with your current jest integration and I will put that onto the nodejs tools repo and link here with some examples in another repo on github.

note that the documentation at https://docs.microsoft.com/en-us/visualstudio/javascript/unit-testing-javascript-with-visual-studio?view=vs-2017 does not say that jest is supported.

as discussed examples etc can be found in this ticket https://github.com/Microsoft/nodejstools/issues/2140

Updating doc


Add exponential backoff

For files that hit rate limit (error 403), keep retrying with twice the delay each time. Fixes #5 fixes #8.

@loongyh Looks good, thank you!

Published as 0.2.0.


ttrt testing

Since ttrt is going to be a core development tool, let's brainstorm ways we can test it APIs so eventually we can put it in CI.
These should be push commit level tests, so quick checks to make sure things aren't broken.  Ideally we can run these tests on non-silicon machines

Blocked by: https://github.com/tenstorrent/tt-mlir/issues/217

blocked by: https://github.com/tenstorrent/tt-mlir/issues/286

Tons of silicon + ttrt api testing has gone into tip. Closing issue.


Task: Fix Security Warnings

Although it's just a maven plugin, but we can't be sure in which env this plugin will be used. So better to update the vulnerable jars.
Dev Tasks:

[x] Update the vulnerable checkstyle jars
[ ] Upgrade pdfbox version
[ ] Release Patch if needed for release-1.0.0.

~Blocked on #43~

unblocking it as travis staged build is now present


support read-tuples endpoint

In the OpenFGA documentation there is also an endpoint stores/{id}/read-tuples to list all currently existing tuples in the store.
As for read requests an object is required, I did not find a way to call the read-tuples endpoint via the sdk.
Please add support for this endpoint.
https://openfga.dev/api/service#/Relationship Tuples/ReadTuples

Hi @dabdada. Thanks for the request.
We intentionally block ReadTuples from the SDKs. We are planning on sun-setting it and merging its functionality into Read

I will move this ticket to the sdk-generator as it affects all SDKs. We probably won't support the ReadTuples, but I will leave this issue open until ReadTuples has been merged into Read


Follow GTK/Qt color scheme for frame and background and text

Description of the new feature/enhancement

Proposed technical implementation details (optional)

You're more than welcome to customize the colorScheme and theme however you see fit ☺

You're more than welcome to customize the colorScheme and theme however you see fit ☺

Thing is that would define custom colors where I talk here rather about a mechanism to follow color scheme


How to retrieve coins from Trinity + Ledger ?

Hi.
I have some Iota on Trinity linked to a Ledger. It looks like I can't activate the send button to transfer and sell the cryptos on an exchange.
I tried Firefly as I was suggested to do but I have no seed, password or file I can recopy to retrieve my wallet.
(Trinity says : 'Seed export not available for Ledger account').
How can I retrieve my coins ?
Regards.

See https://blog.iota.org/firefly-token-migration/ to learn about the migration. Support for Ledger Nano devices is planned for later this month.

Thank you.

Hi,
I have exactly the same problem explained by MacGregorsk on May 14:
I have some Iota on Trinity linked to a Ledger. It looks like I can't activate the send button to transfer and sell the cryptos on an exchange.
I tried Firefly as I was suggested to do but I have no seed, password or file I can recopy to retrieve my wallet.
(Trinity says : 'Seed export not available for Ledger account').
How can I retrieve my coins?
MacGregorsk: did you solve the issue?  Can you please tell me how you did it?
Many thanks in advance!
Regards,
Javier

Ledger integration is still work in progress, so a little more patience is needed. Join the official IOTA discord at discord.iota.org for further questions. 🙂


Naming

findRight
findIndexRight
findLast
findIndexLast
findEnd
findIndexEnd

***Right -> reduce & reduceRight

***End-> trimStart & trimEnd

***Last -> lastIndexOf but no xxxLast

Don't forget String.prototype.padEnd, which was added most recently along with trimEnd.

Should be introduced

includesRight

someRight

everyRight

in this case?

Nope. These method return boolean only. That means we don't really care the order. Well it's may faster.

I think this matters, from performance perspective, when need check big chunk of data beginning from end

I have a strong preference for findLast and findLastIndex, to match lastIndexOf, which is the closest analogy.
The fact that Lodash and Rambda both use that name suggests other people also find it to be the natural choice. Those are also the names that previous proposals have reached for.

@bakkot Thanks for the reminding.
I'm also preference **Last too
BTW: Seems the original thread is by @Nokel81 @kdex @ljharb.

I'd like to rename to **Last if there aren't anyone disagree.

Oh cool, my idea is being taken up again. Yeah, I think **last would be better here. Especially since Right implies left to right but we only really care about first to last.


feat: Add Engine tab to dashboard

📝 Changed routes:

/dashboard/engine/config

(view full size)

Commit b8c0ee9b64193fb1b61bbdd1490fcfbb3a0e54b9 (https://thirdweb-l0yp9fljl.thirdweb-preview.com).

Superseded by #1815


Unhandled exception when checking if URL exists

Impact: Cannot build or serve on some network isolated machines.
Issue: Code expects that request is always made and only evaluates the final HTTP code:
def url_exists(url:str, local_base_dir:str='') -> bool:
    "Checks that a url exists"
    if url.startswith('http'):
        request = requests.get(url)
        return request.status_code == 200

Discovered on ZScaler networks, where the product fails to inject SSL certificates. This part is not why this issue is raised.
Based on reading the code I would expect there to be a try and error message: "Cannot find Mermaid library: %s" % javascript
I would like to also request an option to skip this check - this URL is available in the browser itself due to the way ZScaler works differently to python environment. This seems to align with the intention to just load the remote URL later with:
import mermaid from<EMAIL_ADDRESS>I was able to reproduce the request, confirming the SSL issue is not related to this project.
>>> import requests
>>> request =<EMAIL_ADDRESS>cadmwXcAs6MGMHWs
Traceback (most recent call last):
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\urllib3\connectionpool.py", line 468, in _make_request
    self._validate_conn(conn)
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\urllib3\connectionpool.py", line 1097, in _validate_conn
    conn.connect()
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\urllib3\connection.py", line 642, in connect
    sock_and_verified = _ssl_wrap_socket_and_match_hostname(
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\urllib3\connection.py", line 783, in _ssl_wrap_socket_and_match_hostname
    ssl_sock = ssl_wrap_socket(
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\urllib3\util\ssl_.py", line 471, in ssl_wrap_socket
    ssl_sock = _ssl_wrap_socket_impl(sock, context, tls_in_tls, server_hostname)
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\urllib3\util\ssl_.py", line 515, in _ssl_wrap_socket_impl
    return ssl_context.wrap_socket(sock, server_hostname=server_hostname)
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\ssl.py", line 513, in wrap_socket
    return self.sslsocket_class._create(
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\ssl.py", line 1071, in _create
    self.do_handshake()
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\ssl.py", line 1342, in do_handshake
    self._sslobj.do_handshake()
ssl.SSLCertVerificationError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1007)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\urllib3\connectionpool.py", line 791, in urlopen
    response = self._make_request(
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\urllib3\connectionpool.py", line 492, in _make_request
    raise new_e
urllib3.exceptions.SSLError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1007)

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\requests\adapters.py", line 486, in send
    resp = conn.urlopen(
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\urllib3\connectionpool.py", line 845, in urlopen
    retries = retries.increment(
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\urllib3\util\retry.py", line 515, in increment
    raise MaxRetryError(_pool, url, reason) from reason  # type: ignore[arg-type]
urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='unpkg.com', port=443): Max retries exceeded with url<EMAIL_ADDRESS>(Caused by SSLError(SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1007)')))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\requests\api.py", line 73, in get
    return request("get", url, params=params, **kwargs)
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\requests\api.py", line 59, in request
    return session.request(method=method, url=url, **kwargs)
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\requests\sessions.py", line 589, in request
    resp = self.send(prep, **send_kwargs)
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\requests\sessions.py", line 703, in send
    r = adapter.send(request, **kwargs)
  File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\requests\adapters.py", line 517, in send
    raise SSLError(e, request=request)
requests.exceptions.SSLError: HTTPSConnectionPool(host='unpkg.com', port=443): Max retries exceeded with url<EMAIL_ADDRESS>(Caused by SSLError(SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1007)')))

https://github.com/fralau/mkdocs-mermaid2-plugin/pull/102 - Pull request for handling the exception,

Came for the same issue. I wanted to remove use of extra_javascript and extra_css but since I'm building our documentation in an isolated network, the build fails checking the URL of mermaid on unpkg.
I had a look at #102 @BansheeHero and it looks good but I would recommend a simpler strategy to allow not even checking the URL and warn that it might lead to non-functional graphs. The reason is that it would avoid doing any connections at all thus no exception handling, timeouts, etc to deal with. What do you think?

Thank you. The business case makes sense and the approach appears sensible.

@BansheeHero Your approach has some good points.You complemented it with two procedures warning() and critical(), something I had just done with Mkdocs-Macros (https://github.com/fralau/mkdocs-macros-plugin/commit/4f96dd5b56817275fa7d5597b4bbac394d07b20e), in a slightly more general way.
Maybe you wish to compare?
The question we need to ask at this point, is how we want this issue to initially appear in the build process: as a warning to the admin (which would break only if the --strict option is used), or as an error?
(For more background, see https://github.com/fralau/mkdocs-macros-plugin/issues/226.)

Since our documentation has a CI/CD pipeline, it is easy ok for us to make it fail in case there is an error in the documentation (missing documents, broken links, etc). As long as we can skip some specific errors that are expected due to network isolation.

Thanks for all the input and PRs. To sum it up, we have an issue of

Unhandled exception.
A mechanism for gracefully failing or not.

In general, this puts into question the mechanism for calling the library. In principle, if there is a local file, it should not make an http call., and that might solve the problem, I believe.
Or, in the normal case just issue a warning and continue. If there is an --strict option (as mentioned) we could make it fail.
@EvaSDK Would that second solution work for you?


improve handling of bufr-conversion related packages

All bufr-conversion related tasks are compiled into the wis2box-api , we need to be able to update these package from time without getting stuck on dependencies between one another. Let's find a way to improve the packaging of these interdependent packages:

pymetdecoder
bufr2geojson
csv2bufr
synop2bufr

Something like
|---bufrtools
|   |---__init__.py
|   |---csv2bufr
|       |---__init__.py
|   |---synop2bufr
|   |---bufr2geojson


Franknoirot/help menu

Creates a little menu to go in the lower right corner of the app for version number, helpful links, the new home of the NetworkHealthIndicator, and a new HelpMenu. Based on these Figma mockups (and HelpMenu is very inspired by Figma's lol).
Lays the groundwork for a full control cluster down there with a view gizmo, scale control, and ortho/perspective toggle. It will likely collide with the orientation gizmo from engine if we fix the bug with that before I'm able to make a client-side gizmo.
Demo
https://github.com/KittyCAD/modeling-app/assets/23481541/7a745377-f014-4ecf-8bad-6d8a340f945f

Oh and this polishes up the Tooltip component to have a better look and be able to be corner-anchored, for those top-right anchored tooltips for the help menu and network health. I'm way happier with that component now.


visualizer bar

Someone and I have the same problem of the most bar in the visualizer stuck about halfway up and it happens sometimes.
Version:
Windows 10
Rainmeter 4.2 Beta
Monstercat Visualizer 2.0.2 & before update

I think the problem is about headphone and speaker, here's the screenshot of I use the headphone


tsserver doesn't handle new files correctly some of the time

I haven't been able to figure out exactly why this happens, but sometimes tsserver doesn't work properly with new files, and then after a while it does. Imports and such don't work, and then later they do.

My hunch is that a didSave event needs to be sent when new files are created.

In my case - I can constantly reproduce this issue.

start neovim
open typescript file, so tsserver starts
copy some correct file to another location using file manager
open copy with neovim

ale highlights local imports.
also seems that tsserver don't use project tsconfig.json for new files.

I'm having the same issue when following the steps from @rapthead. Any imports that require baseUrl from tsconfig.json get highlighted as errors, while imports from node_modules seem to be fine.

I think the problem lies in tsserver. It doesn't watch the filesystem for changes, so it doesn't know what to do with new files. I have added a command for stopping tsserver and all other lsp-like things to work around this now. See :help ALEStopAllLSPs.

I verified that the same issue exists in vscode, so you're probably right that it might be a tsserver thing. I was hoping to find that it works in vscode so I could figure out the difference, but alas...
When I looked through the tsserver logs it looked like it was assuming my new file to be its own inferred project root, not sure what's up with that.

@w0rp tsserver has watch mode. Passing -w and I believe --watch would solve this problem.

I think that option exists for tsc, but not for tsserver.

Well, I have modified ale/ale_linters/typescript/tsserver.vim and added -w flag to command. That solved the problem for me of ale not working anymore on file rename.

Okay. I couldn't see it documented anywhere. I'll have to dig around to figure out what versions of tsserver support that.

 As well I have looked into tsserver code and there shouldn't be -w option unless it is well hidden somehow.  One possibility is that it is somehow related to the fact that another plugin that starts tsserver is https://github.com/mhartington/nvim-typescript.
Could it be that my naive approach how to pass "-w" to tsserver in Ale broke tsserver for Ale and in result it somehow ended using tsserver instance that was run by nvim-typescript's? I see that nvim-typesript is issuing TSServer reload command before executing any other TSServer command. Maybe that would be the correct fix?


Inconsistent result of flesch_reading_ease() between version 0.7.2 and 0.7.3

Here's the steps to reproduce this issue using docker:

grab and run any docker image that has no python package pre-installed and have python>=3.6, for instance

 docker run -it refazul/python3.9.0 bash

download 0.7.3

root@9dbeec82e01e:/# pip3 freeze
root@9dbeec82e01e:/# pip3 install textstat==0.7.3

run commands below, test with a sentence

root@9dbeec82e01e:/# python3.9
Python 3.9.0 (default, Nov 11 2020, 20:23:31)
[GCC 9.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import textstat
>>> textstat.__version__
(0, 7, 2)
>>> textstat.flesch_reading_ease("\nSo now, Marne, our Chief Business Officer is going to take us through some of them. \nThanks, Mark. We'll see you in a minute! \nWhat if you could learn about anything in the world just by bringing it closer to you? \nSo, we're going to have an astrophysicist in the family. \nActually, I have to write this paper. Will you help me? \nLet's take a closer look. \nWhat part of the solar system are we talking about? \nSaturn. \nIf you were taking astrophysics, you could study in the Meta-Verse. \nDid you know the rings are made up of billions of icy particles? \nReally? \nLook at this. \nYou ready to do that paper now, right? \nYeah. \nIn the Meta-Verse, \nyou’ll be able to teleport not just to any place, but any time as well. Ancient Rome. Imagine standing on the streets, hearing the sounds, visiting the markets, to get a sense of the rhythm of life over 2,000 years ago. Imagine learning how the Forum was built by actually seeing the Forum get built, right in front of you.")
84.78
>>> exit()

download 0.7.2

root@9dbeec82e01e:/# pip3 install textstat==0.7.2

run the same commands as step 3

root@9dbeec82e01e:/# python
Python 3.9.0 (default, Nov 11 2020, 20:23:31)
[GCC 9.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import textstat
>>> textstat.__version__
(0, 7, 2)
>>> textstat.flesch_reading_ease("\nSo now, Marne, our Chief Business Officer is going to take us through some of them. \nThanks, Mark. We'll see you in a minute! \nWhat if you could learn about anything in the world just by bringing it closer to you? \nSo, we're going to have an astrophysicist in the family. \nActually, I have to write this paper. Will you help me? \nLet's take a closer look. \nWhat part of the solar system are we talking about? \nSaturn. \nIf you were taking astrophysics, you could study in the Meta-Verse. \nDid you know the rings are made up of billions of icy particles? \nReally? \nLook at this. \nYou ready to do that paper now, right? \nYeah. \nIn the Meta-Verse, \nyou’ll be able to teleport not just to any place, but any time as well. Ancient Rome. Imagine standing on the streets, hearing the sounds, visiting the markets, to get a sense of the rhythm of life over 2,000 years ago. Imagine learning how the Forum was built by actually seeing the For)m get built, right in front of you.")   u
60.72

voila

Sorry for the long paragraph as the example. But it is that very text that produced this issue.
I am not very familiar with Flesch reading ease formula, but I still think such discrepancy is worth having another look.
Please enlighten me if I did something wrong or if you cannot reproduce this issue.

I can confirm I'm also encountering this issue inside a docker container on 0.7.2.

Hi @XyLearningProgramming, thank you for raising this. Sorry I haven't had chance to investigate it yet.
Some changes between 0.7.2 and 0.7.3 that could be responsible for the change in score:

Update syllable_count to use .split() and not .split(' ') so to split on all whitespace. https://github.com/shivam5992/textstat/pull/161
Fix issue with remove_punctuation(). https://github.com/shivam5992/textstat/pull/172

Thanks for looking into this... So, to my understanding, there are some word-splitting & punctuation related changes possibly causing this issue. Since they are all "improved" towards performing the tasks in the intended way, can I say the reading ease score is more accurate, even though the score is different in different versions?
Also, feel free to close this issue if you investigate it and think this behavior is intended. Thank you again.

the reading ease score is more accurate

That's correct. As we make these changes, the hope is we're moving towards more accurate scores.
I'll close this issue, but if you find something wrong, feel free to open it or raise a new one.


Update dashboard Kubernetes / Views / Global ( 31 → 35 )

This PR contains the following updates:

Package
Update
Change

Kubernetes / Views / Global
major
31 -> 35

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Edited/Blocked Notification
Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.
⚠ Warning: custom changes will be lost.


Locking migration generation to specified tables

When designing a new database layout, I often get myself into a situation, where a part of changes made to tables is already done (ready for a commit), whereas other tables are still being a wip. Is it possible to restrict the migration generation only to tables specified? I'm looking at how
git add * && git commit -m "some work"
./vendor/bin/phinx-migrations generate --name 'some work' --overwrite

could compare compare to, if a parameter such as --tables would exist
git add x1 x2 && git commit -m "some work"
./vendor/bin/phinx-migrations generate --name 'some work' --overwrite --tables x1 x2

is this an existing feature?

This feature does not exist.
Usually I do this by generating a smaller migration when I know that the change is done.
Then I change the next table(s) and generate a new migration for it.

I tried and failed miserably (to generate smaller migrations). I'm too unstructured when updating databases and have a tendency to work on several things in parallel. This would definitely help my more chaotic workflow.


React Player GitHub is a broken link in readme.md

Hello!
In the readme.md, under Plugins and Components, there is a link to a react repo https://github.com/xDae/react-plyr, but it is a broken link. @xDae, are you still working on this project? Do you know where it is located?
There are a couple of alternatives too if we are looking to replace the link with a new one.

Feel free to PR with an updated link 👍


When tests are stopped by timeout, pending promises continue execution

🐛 Bug Report
When tests are stopped by timeout, pending promises continue execution
To Reproduce
Run the following test:
jest.setTimeout(3000);

beforeAll(async () => {
  await longFunction();
});

test("Test name", async () => {
  expect(1).toBeTruthy();
});

afterAll(async () => {
  await sleep(5000);
  console.log(3)
});

async function longFunction() {
  console.log(1);
  await sleep(5000)
  console.log(2);
}

function sleep(ms: number) {
  return new Promise((resolve) => setTimeout(resolve, ms));
}

Actual behavior
'2' is written in console

Expected behavior
longFunction execution is stopped after 3 sec, therefore '2' is not written in console.
envinfo
λ npx envinfo --preset jest
npx: installed 1 in 1.668s

  System:
    OS: Windows 10 10.0.18362
    CPU: (8) x64 Intel(R) Core(TM) i5-8300H CPU @ 2.30GHz
  Binaries:
    Node: 10.16.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.17.3 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 6.9.0 - C:\Program Files\nodejs\npm.CMD
  npmPackages:
    jest: 25.1.0 => 25.1.0

This often leads to Jest did not exit one second after the test run has completed. behavior and various errors. F.e., I close my logger in afterAll function and if longFunction tries to log something in this moment, network error will occur.

@4ekki You missed the jest.setTimeout(3000); in the pasted example, which kind of makes it harder to understand your example since the default is 5000. Would be good if you update. I noticed that by looking at the image. Also, are you using jest-circus?

@thernstig, thank you for pointing this out. For some reason, github hides everything in the same line, where triple-quote code macro starts. Added new line and it was ok=)
Yes, I use jest-circus.

@4ekki No worries. Also, if you add this to the start triple single-quote it looks even nicer. Like this.
```javascript

I also found these, do you recon this is a duplicate?
https://github.com/facebook/jest/issues/8442
https://github.com/facebook/jest/issues/8688
https://github.com/facebook/jest/pull/8654 (someone trying to fix it?)

Not stale

I'm running into this issue also


Update prereqs and fix quotes in readme.md

Hello, I added aria2 to the prereq list (maybe a minimum version requirement?) and fixed quotes in the input file input example (same as PR #283 for master branch)

Thanks you for the help :D
Sorry for the late response and merge


Sidebar buggy on mobile

What browser and device are you using, it looks fine on my phone

iphone xr safari

also in chrome

probably has something to do with apples web-kit

This also happens in macOS Safari 17.1. Not sure why but likely a WebKit issue


Add mt5 cpe kmutt thai sentence sum

Add mt5 cpe kmutt thai sentence sum
Model: https://huggingface.co/thanathorn/mt5-cpe-kmutt-thai-sentence-sum

Hello @wannaphong! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file pythainlp/summarize/core.py:

Line 9:80: E501 line too long (85 > 79 characters)
Line 51:80: E501 line too long (94 > 79 characters)
Line 87:80: E501 line too long (122 > 79 characters)
Line 99:80: E501 line too long (114 > 79 characters)

In the file tests/test_summarize.py:

Line 26:80: E501 line too long (90 > 79 characters)

@thanathornboss @thanathorn What do you think about this pull request?

Coverage decreased (-0.05%) to 97.057% when pulling 06e9084e49bee732236a21c2110c3240448a6a65 on thanathorn:add-mt5-cpe-kmutt-thai-sentence-sum into efcbe7f1881351fadf62d28cc0415d22156a2e71 on PyThaiNLP:dev.

@wannaphong  We are delighted to have our model merged with PyThaiNLP. We hope this integration could expand the potential and capabilities of PyThaiNLP!

We are delighted to have our model merged with PyThaiNLP.
We hope this integration could expand the potential and capabilities of
PyThaiNLP!
On Mon, 6 Jun 2565 BE at 00:17 Wannaphong Phatthiyaphaibun <
@.***> wrote:

@thanathornboss https://github.com/thanathornboss @thanathorn
https://github.com/thanathorn What do you think about this pull request?
—
Reply to this email directly, view it on GitHub
https://github.com/PyThaiNLP/pythainlp/pull/679#issuecomment-1146851416,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABKN6UGKRGIGBAKQPBVWYR3VNTOJXANCNFSM5X5KOFIQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


Fix service-spec targetPort and container api tls flag

Pull Request Test Coverage Report for Build 5812

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 43.325%

Totals

Change from base Build 5810:
0.0%

Covered Lines:
15943

Relevant Lines:
36799

💛  - Coveralls

Pull Request Test Coverage Report for Build 5812

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 43.325%

Totals

Change from base Build 5810:
0.0%

Covered Lines:
15943

Relevant Lines:
36799

💛  - Coveralls


add mso algorithm support on traffic ops

#1344

will update file name of<PHONE_NUMBER>0000_add_multisite_algrithom.sql when this PR is reviewed

@mitchell852 @mtorluemke Could you help to review? Thanks:)

yep, i'm reviewing today

just curious. why not just store the text values (consistent_hash, false, strict, true) in the database for multi_site_origin_algorithm rather than using the numbers 0,1,2,3. when querying the database sometimes it's hard to remember what 2 stands for...for example...

@mitchell852 Thanks for your comments. Yes, it looks to be more direct to store the text (consistent_hash, false, strict, true) in the database. Initially I was following the "Protocol" field with values of "http", "https", and etc., which is stored as 0, 1, 2... Now I get that mso algorithm is a bit different story so your suggestion looks better. But the change from number to text is big. If the current implementation is ok, could we use it? :-) Of course if you prefer to use text value, I will modify it in one or two days. Thanks.

i think it's fine to use 0..4, plus it's consistent with other implementations (i.e. protocol as you mentioned). thx.

can you run perltidy against a bunch of your files. looks like your formatting is a bit off.

@mitchell852 Just renamed the goose sql file name. And also I reformatted the perl files (except Result/Deliveryservice.pm) with perltidy. Please help to review if they are good. Thanks.

Does this fix #1377 ?

It does but I want to do a little more testing on #1377 and then i'll close it.

Tested in the lab with all four MSO configuration possibilities, as well as ranking configuration for primary/secondary configuration.  Parent config files formatted as expected.

I just found a small bug - it looks like we get an entry in the parent.config on the mid for each regex entered into the delivery service.  They're all duplicate entries.

@dg4prez Thanks Derek. This duplicated issue seems to exist before my pull request, but let me verify this later.


Update setup.py to remove version duplication

This change addes my well vetted ast parsing solution to finding the version string that lives in the package source code, without needing to have the package installed, allowing for it to be used in setup.py.
Additionally, I added a minimum python version to the setup call. f-strings are used throughout the code base and therefore we need to be using at least python3.6

Could you format the setup.py with black? Other than that let's merge this.

 Shouldn't be an issue in the future since #58 has been merged.

 Shouldn't be an issue in the future since https://github.com/r-three/git-theta/pull/58 has been merged.

The perils of out of order execution I guess lol


Incomplete Update of Project Fields in Edit Endpoint

The edit project endpoint ("/task-tracker/v1/task/projects/:id/") is not functioning as expected. When submitting new values through the form, only "title" and "description" fields are being updated successfully. However, other fields such as "tags," "assignees," and "attachments" remain unchanged despite sending new values through the form.

Project
Postman:

Tested for both POST and PATCH methods.
Assignees (the plural form in the name of model is unfortunate though) are ForeignKey - meaning only one assignee for one task as discussed previously.
Please let me know if you encountered any issue on PATCH for Task.

Should be working as expected now. Please check.


Include Dinh's Bulwark in Special Attack Counter

With the recent changes to the Dinh's bulwark, it can now be used to lower stats at places like corp instead of a BGS. It now lowers offensive stats by 5% at a time. As was the issue with the spec weapons previously, if using it to spec down corp, the player must count all specs. Since this is now viable, I think it should be added to the counter.

This was already added in e60ce99081d5908cb3e2d7f87c9ce7cba0f9051b, and will be in the next release


**.databinding no exist

In my demo.  all databinding  can import automatically. but when it ruuning
Error msg show that :
Error:(10, 44) 错误: 程序包com.xhyy.lxr.listviewdb1.databinding不存在(“不存在”means not exist)

Make sure you have your  in your xml layout, *.binding packages are generated automatically AFTER you create your XML binding.


Update enpass to 5.5.5

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Thank you for the contribution. It was merged directly as 9e0a864781d31ea11d4f885e5adb8778cfc69007. Your contribution is still credited to you.


Custom Widget not supported

https://github.com/mottosso/Qt.py/blob/cc5e730cc3e5e901a3bd1d4df343006ebd77e3e0/Qt.py#L894
We have:
class ManagerTable(QtWidgets.QTableView)

and
class AssetTable(ManagerTable)

This throws an error Custom widget ManagerTable not supported.
Modifications I did to Qt.py:
                try:
                    widget = Qt._QtUiTools.QUiLoader.createWidget(self,
                                                                  class_name,
                                                                  parent,
                                                                  name)

                except Exception:
                    raise Exception("Custom widget '%s' not supported"
                                    % class_name)

Thanks @michimussato, would it be possible to put your modification into a PR? That way, our automated tests will take it for a spin and make sure it runs on all bindings, platforms and Python interpreters.

Sorry. My changes do not do the trick.

That's allright; once it's in a PR, we can start iterating on a solution that does the trick, where every change is being run through the same series of tests. Once the tests pass, odds are we've got a winner.

It looks like the current state of master (1.2.0.b3) has solved this?  I have a Designer .ui file that I load through QtCompat.loadUi(), which "promotes" a TableView object to a custom subclass MyTableView.  This works with my Python 2.7/PyQt4 setup, but when I run it inside Maya 2016 (pyside) or Maya2018 (pyside2), I get the above custom widget unsupported error. When I bring in the current Qt.py code, it works (at least superficially).
Is there any timeline for a 1.2.0 release to PyPi?  It would be nice to be able to do simple "pip install" to get this.

Is there any timeline for a 1.2.0 release to PyPi? It would be nice to be able to do simple "pip install" to get this.

This isn't working for you?
$ pip install Qt.py==1.2.0.b3


does createFromString handle a very big csv file?

(Fill in the relevant information below to help triage your issue.)

Q
A

Version
9.6.0

Question
I want to read csv file hosted on S3. It works fine with createFromString (https://github.com/thephpleague/csv/issues/306). However, I wonder if createFromString can handle a very big csv file.
Is it smart enough to read line by line instead of dumping everything?

@michaelnguyen547 thanks for using the library. Did you try your solution before asking ? I don't use S3 so I can not answer a question I never faced before even though I have an idea on how it will behave you better test it for yourself IMHO. If it fails or behave not according to what the documentation state then please don't hesitate to open an issue or a pull request if you succeed in fixing the found issue.


Feature request: Collapse job data

I have queues with large payloads and the bull board pages load very slowly. I did not investigate why it is so slow. I suspect rendering the job data as json takes for ever. Thus it would be nice if large objects/arrays would be auto-collapsed when rendering (similar to collapsing code lines in IDEs). This would also make job data more readable because you don't have to scroll down thousands of lines to see the properties that come after these big chunks. So collapsible json would be a nice regardless if it fixes the performance or not.

It probably will solve the performance, but It should be only a UX consideration.
If your json data is huge, you should use data formatter in order to reduce the size of the JSON, the app won't be able to reduce it for you.
But this is totally a nice UX optimization, it should be settings configurable.

I face the same issue.
I write a lot of data and some payloads are huge and they completely break the Dashboard. Even when they are not being processed anymore or they should be on the second page, I cannot get it to load without my Browser killing the tab.
Only solution to make it usable again currently is to just kill the redis-container, delete and recreate it, which works and is fine in this use-case, but probably not in others.

That could be a option. I'll need to figure out first if the data needs to be shown there, normally I wouldn't want to check each entry in the queues, but for debugging it's perfect.
Anyways, I'll talk back with my team and see if it's a viable option. Thanks for the suggestion.

We are running into the same issue here!
We have very big payloads and bull-board is basically unuseable to us, as the browser just crashed because the DOM can't render the big JSONs!
My recommendation would be to just create a setting (in the top right menu), where you can collapse the JSONs by default. Like this, hopefully, the DOM should not crash and it is then the user's job to just open the JSON (job data) for a single job as needed.

@Twisterking will able to prepare a PR for this?

Maybe, not sure yet! I was not aware of the queueAdapter.setFormatter() option, so we might be using just that and be happy with it! 👍

I'll add an additional optimization which will use webworker if available for highlighting code

@sroettering can you test v4.12.1 which includes web worker highlight & collapse functionality (enabled from settings).
Try your heavy queue pls :]

At first glance it looks very nice! Would have expected to be able to collapse the data again, but I guess it's ok like this. Performance is also good in my first test.
Thanks for the quick implementation. I will open more feature requests soon :-)

Did you tested the board without the collapse state?
Is it much better?

yes, it's much better now. Also the paging is really fast as well!


[3.0] Villages spawn with anvils

This allows for free 3x3 crafting at spawn. They also have loot chests that can give you free tools and a free chest at spawn, though that is possibly okay and not more OP than a drop from a zombie or a desert temple.

the anvil has been removed


Doesn't work with nGinx. Connections are reset. Regular nodeJS server are working fine

2018/07/21 08:07:24 [error] 8617#8617: *808 recv() failed (104: Connection reset by peer) while reading response header from upstream.
While nodejs sockets are working fine, even with TLS.

This is normal error. When client disconnected abnormally you'll see this error.
If you don't care about lost connection you can ignore it. But if you want to implement some functionality to restore connection or notify about it then you could use this error.

Hi I tried. I like the multi-thread but the server just doesn't work on nGinx with load balancing. Other libraries (also PHP) work fine. There something wrong with the code.

Ah, I got it. This server is "standalone". It's completely independent. No need to nginx or other web-server. You just have to start it once. This component dedicated for small simple chats.
If you're looking for load-balanced websocket-server then you could use another repo

Yes I tried that one as well because it was plain. Same issues. It doesn't even accepts the connection request. Thats how far I got. Then I was done with it. I know use this one: https://github.com/heminei/php-websocket but that one isn't multi-threading :(

I will give it a last try tomorrow. Maybe I will reach out for your help ;)

Great! Anyway you can post piece of log here and I'll try to find/explain what's going on.

See you here ;) https://github.com/ollyxar/websockets/issues/7


DevTrack Problem

I downloaded the extension and while installing it all the files of my project got deleted. I tried it again and same happened. As you can see these files were there earlier but aren't there now.

Hey I'm glad you were able to resolve the issue! What fixed it?

Can you explain me how to use this extension? I just have to download and authenticate it from GitHub. Do I need to create any repo on my github account as well for this to work?

You just download it and authenticate it from GitHub.
You don’t have to create any repository for it to work. Once you
authenticate it to your GitHub account it creates a repo for you and it
will save automatically to that repository.
On Mon, Dec 23, 2024 at 11:30 PM Shashank @.***> wrote:

Can you explain me how to use this extension? I just have to download and
authenticate it from GitHub. Do I need to create any repo on my github
account as well for this to work?
—
Reply to this email directly, view it on GitHub
https://github.com/Teamial/DevTrack/issues/7#issuecomment-2560630701,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASWR6CCJHCENHWJBUPIPFPD2HDPOPAVCNFSM6AAAAABUEB5AHKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNRQGYZTANZQGE
.
You are receiving this because you commented.Message ID:
@.***>

When I install it in the working directory of any current project opened in VS Code, all the files in that project are deleted

May I ask what version of dev track do you have installed when you look in
the extensions? Also do you have any uncommitted changed in your working
directory?
On Mon, Dec 23, 2024 at 11:57 PM Shashank @.***> wrote:

When I install it in the working directory of any current project opened
in VS Code, all the files in that project are deleted
—
Reply to this email directly, view it on GitHub
https://github.com/Teamial/DevTrack/issues/7#issuecomment-2560648985,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASWR6CHLKZ2ALU4FDEIUG2T2HDSVJAVCNFSM6AAAAABUEB5AHKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNRQGY2DQOJYGU
.
You are receiving this because you commented.Message ID:
@.***>

Version 4.1.11. I had not initialized any repo for that project. Yes, I was working on windows.


Gamepad Rework

This reworks gamepads so that all button and axis mappings go through the same enum, instead of controller specific controls.
Desktop and UWP Gamepads have been tested and work correctly. Desktop (on windows, if GLFW mapping works here, it should work everywhere else) has been tested with an XB1 and PS4 controller, and UWP just with the XB1.
Web has not been tested as I am unable to test, but what I have written comes straight from the w3 docs and so should be correctly mapping buttons and axis to the new enum's aswell.

Just noticed all the examples need updating... I shall get on that.

Ok, just answered two of my three questions (TABS, gamepad examples).
Third question: What about Android gamepad? I just been investigating it right now and it seems gamepad inputs on Android come in the form of key inputs... not sure how to deal with that properly... neither have an Android gamepad for testing... probably we can skip that part for now.

I have ignored Android gamepads for now as I did not find them implemented and I am unsure as to how to add Android support to it.

ok, do you want me to wait for the examples review before merging?

Yeah please, I am just in the middle of updating the gamepad example to the correct enums, but nothing more. My aim is for it to build and the example to work, not worrying about controller identification for the moment.

That should be it sorted, I'm going to wait for the build to complete before giving the go-ahead.

take it easy, no hurries... :)

It seems there is some issue on PLATFORM_WEB:
/src/src/core.c:3488:57: error: use of undeclared identifier 'k'
                const GamepadAxis axis = GetGamepadAxis(k);

Oh, I see, 2 seconds

That should fix it

@Rover656 Thnak you very much for another great contribution! :)


Change structs to package private

AppWrapper and eventHandler.New() do not need to be public.

Coverage remained the same at 79.145% when pulling c9228eefd88140338dd0a8a1dfd81b7c72362ad5 on reduce_scope into a6d4e3cb299b8cd9b9180dc229d9c2293ad14e03 on develop.

👍


docs(install): update instructions for Arch Linux

I moved cargo-dist to the official repositories: https://archlinux.org/packages/extra/x86_64/cargo-dist/
Signed-off-by: Orhun Parmaksız<EMAIL_ADDRESS>

oh awesome, thanks!


Small perfomance improvments for Dictionary

Use TryGetValue instead of ContainsKey and Getter for Dictionary to prevent double calculation of HashCode.

@thecodejunkie Can you review and merge it please? It is simple improvement of work with Dictionary.


add option for setting output file name & chunk name

This PR adds an optional options object as parameter to the constructor.
The module now automatically ignores modules which are not containing in a node_modules folder.
If no options are provided, the module will collect node modules from all chunks.
If a chunk name is provided via the options object, only modules in chunks with the given name will be collected.
The options object also allows setting a different output file name, but still defaults to npm-modules.md.

Please review my small comments.


no module "nt-web-button" ?

npm ERR! 404 Registry returned 404 for GET on https://registry.npmjs.org/nt-web-button

published it to npmjs.com?

Sorry for published with a wrong name, I have republished it right now.


warning: implicit conversion changes signedness: 'int' to 'unsigned int' [-Wsign-conversion]

Compiling with clang yields following warnings.
Can we address these warnings? If so, I want to send a patch for this.
In file included from src/teavpn2/server/linux/tcp.c:16:
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:211:15: warning: implicit conversion loses integer precision: 'unsigned int' to '__u8' (aka 'unsigned char') [-Wimplicit-int-conversion]
        sqe->flags = flags;
                   ~ ^~~~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:218:16: warning: implicit conversion loses integer precision: 'int' to '__u8' (aka 'unsigned char') [-Wimplicit-int-conversion]
        sqe->opcode = op;
                    ~ ^~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:255:64: warning: implicit conversion changes signedness: 'int64_t' (aka 'long') to '__u64' (aka 'unsigned long long') [-Wsign-conversion]
        io_uring_prep_rw(IORING_OP_SPLICE, sqe, fd_out, NULL, nbytes, off_out);
        ~~~~~~~~~~~~~~~~                                              ^~~~~~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:256:23: warning: implicit conversion changes signedness: 'int64_t' (aka 'long') to '__u64' (aka 'unsigned long long') [-Wsign-conversion]
        sqe->splice_off_in = off_in;
                           ~ ^~~~~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:276:62: warning: implicit conversion changes signedness: 'off_t' (aka 'long') to '__u64' (aka 'unsigned long long') [-Wsign-conversion]
        io_uring_prep_rw(IORING_OP_READV, sqe, fd, iovecs, nr_vecs, offset);
        ~~~~~~~~~~~~~~~~                                            ^~~~~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:283:63: warning: implicit conversion changes signedness: 'off_t' (aka 'long') to '__u64' (aka 'unsigned long long') [-Wsign-conversion]
        io_uring_prep_rw(IORING_OP_READ_FIXED, sqe, fd, buf, nbytes, offset);
        ~~~~~~~~~~~~~~~~                                             ^~~~~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:284:19: warning: implicit conversion loses integer precision: 'int' to '__u16' (aka 'unsigned short') [-Wimplicit-int-conversion]
        sqe->buf_index = buf_index;
                       ~ ^~~~~~~~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:291:63: warning: implicit conversion changes signedness: 'off_t' (aka 'long') to '__u64' (aka 'unsigned long long') [-Wsign-conversion]
        io_uring_prep_rw(IORING_OP_WRITEV, sqe, fd, iovecs, nr_vecs, offset);
        ~~~~~~~~~~~~~~~~                                             ^~~~~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:298:64: warning: implicit conversion changes signedness: 'off_t' (aka 'long') to '__u64' (aka 'unsigned long long') [-Wsign-conversion]
        io_uring_prep_rw(IORING_OP_WRITE_FIXED, sqe, fd, buf, nbytes, offset);
        ~~~~~~~~~~~~~~~~                                              ^~~~~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:299:19: warning: implicit conversion loses integer precision: 'int' to '__u16' (aka 'unsigned short') [-Wimplicit-int-conversion]
        sqe->buf_index = buf_index;
                       ~ ^~~~~~~~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:389:22: warning: implicit conversion changes signedness: 'int' to '__u32' (aka 'unsigned int') [-Wsign-conversion]
        sqe->accept_flags = flags;
                          ~ ^~~~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:396:22: warning: implicit conversion changes signedness: 'int' to '__u32' (aka 'unsigned int') [-Wsign-conversion]
        sqe->cancel_flags = flags;
                          ~ ^~~~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:418:65: warning: implicit conversion changes signedness: 'int' to '__u64' (aka 'unsigned long long') [-Wsign-conversion]
        io_uring_prep_rw(IORING_OP_FILES_UPDATE, sqe, -1, fds, nr_fds, offset);
        ~~~~~~~~~~~~~~~~                                               ^~~~~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:426:51: warning: implicit conversion changes signedness: 'off_t' (aka 'long') to '__u64' (aka 'unsigned long long') [-Wsign-conversion]
                        (const uintptr_t *) (unsigned long) len, mode, offset);
                                                                       ^~~~~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:426:45: warning: implicit conversion changes signedness: 'int' to 'unsigned int' [-Wsign-conversion]
                        (const uintptr_t *) (unsigned long) len, mode, offset);
                                                                 ^~~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:433:20: warning: implicit conversion changes signedness: 'int' to '__u32' (aka 'unsigned int') [-Wsign-conversion]
        sqe->open_flags = flags;
                        ~ ^~~~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:444:57: warning: implicit conversion changes signedness: 'off_t' (aka 'long') to '__u64' (aka 'unsigned long long') [-Wsign-conversion]
        io_uring_prep_rw(IORING_OP_READ, sqe, fd, buf, nbytes, offset);
        ~~~~~~~~~~~~~~~~                                       ^~~~~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:450:58: warning: implicit conversion changes signedness: 'off_t' (aka 'long') to '__u64' (aka 'unsigned long long') [-Wsign-conversion]
        io_uring_prep_rw(IORING_OP_WRITE, sqe, fd, buf, nbytes, offset);
        ~~~~~~~~~~~~~~~~                                        ^~~~~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:460:21: warning: implicit conversion changes signedness: 'int' to '__u32' (aka 'unsigned int') [-Wsign-conversion]
        sqe->statx_flags = flags;
                         ~ ^~~~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:466:58: warning: implicit conversion changes signedness: 'off_t' (aka 'long') to '__u64' (aka 'unsigned long long') [-Wsign-conversion]
        io_uring_prep_rw(IORING_OP_FADVISE, sqe, fd, NULL, len, offset);
        ~~~~~~~~~~~~~~~~                                        ^~~~~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:467:24: warning: implicit conversion changes signedness: 'int' to '__u32' (aka 'unsigned int') [-Wsign-conversion]
        sqe->fadvise_advice = advice;
                            ~ ^~~~~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:466:53: warning: implicit conversion loses integer precision: 'off_t' (aka 'long') to 'unsigned int' [-Wshorten-64-to-32]
        io_uring_prep_rw(IORING_OP_FADVISE, sqe, fd, NULL, len, offset);
        ~~~~~~~~~~~~~~~~                                   ^~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:474:24: warning: implicit conversion changes signedness: 'int' to '__u32' (aka 'unsigned int') [-Wsign-conversion]
        sqe->fadvise_advice = advice;
                            ~ ^~~~~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:473:53: warning: implicit conversion loses integer precision: 'off_t' (aka 'long') to 'unsigned int' [-Wshorten-64-to-32]
        io_uring_prep_rw(IORING_OP_MADVISE, sqe, -1, addr, length, 0);
        ~~~~~~~~~~~~~~~~                                   ^~~~~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:481:19: warning: implicit conversion changes signedness: 'int' to '__u32' (aka 'unsigned int') [-Wsign-conversion]
        sqe->msg_flags = flags;
                       ~ ^~~~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:480:53: warning: implicit conversion loses integer precision: 'size_t' (aka 'unsigned long') to 'unsigned int' [-Wshorten-64-to-32]
        io_uring_prep_rw(IORING_OP_SEND, sqe, sockfd, buf, len, 0);
        ~~~~~~~~~~~~~~~~                                   ^~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:488:19: warning: implicit conversion changes signedness: 'int' to '__u32' (aka 'unsigned int') [-Wsign-conversion]
        sqe->msg_flags = flags;
                       ~ ^~~~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:487:53: warning: implicit conversion loses integer precision: 'size_t' (aka 'unsigned long') to 'unsigned int' [-Wshorten-64-to-32]
        io_uring_prep_rw(IORING_OP_RECV, sqe, sockfd, buf, len, 0);
        ~~~~~~~~~~~~~~~~                                   ^~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:503:59: warning: implicit conversion changes signedness: 'int' to '__u64' (aka 'unsigned long long') [-Wsign-conversion]
        io_uring_prep_rw(IORING_OP_EPOLL_CTL, sqe, epfd, ev, op, fd);
        ~~~~~~~~~~~~~~~~                                         ^~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:503:55: warning: implicit conversion changes signedness: 'int' to 'unsigned int' [-Wsign-conversion]
        io_uring_prep_rw(IORING_OP_EPOLL_CTL, sqe, epfd, ev, op, fd);
        ~~~~~~~~~~~~~~~~                                     ^~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:510:66: warning: implicit conversion changes signedness: 'int' to '__u64' (aka 'unsigned long long') [-Wsign-conversion]
        io_uring_prep_rw(IORING_OP_PROVIDE_BUFFERS, sqe, nr, addr, len, bid);
        ~~~~~~~~~~~~~~~~                                                ^~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:510:61: warning: implicit conversion changes signedness: 'int' to 'unsigned int' [-Wsign-conversion]
        io_uring_prep_rw(IORING_OP_PROVIDE_BUFFERS, sqe, nr, addr, len, bid);
        ~~~~~~~~~~~~~~~~                                           ^~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:511:19: warning: implicit conversion loses integer precision: 'int' to '__u16' (aka 'unsigned short') [-Wimplicit-int-conversion]
        sqe->buf_group = bgid;
                       ~ ^~~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:518:19: warning: implicit conversion loses integer precision: 'int' to '__u16' (aka 'unsigned short') [-Wimplicit-int-conversion]
        sqe->buf_group = bgid;
                       ~ ^~~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:524:54: warning: implicit conversion changes signedness: 'int' to 'unsigned int' [-Wsign-conversion]
        io_uring_prep_rw(IORING_OP_SHUTDOWN, sqe, fd, NULL, how, 0);
        ~~~~~~~~~~~~~~~~                                    ^~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:531:22: warning: implicit conversion changes signedness: 'int' to '__u32' (aka 'unsigned int') [-Wsign-conversion]
        sqe->unlink_flags = flags;
                          ~ ^~~~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:538:61: warning: implicit conversion changes signedness: 'int' to 'unsigned int' [-Wsign-conversion]
        io_uring_prep_rw(IORING_OP_RENAMEAT, sqe, olddfd, oldpath, newdfd,
        ~~~~~~~~~~~~~~~~                                           ^~~~~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:540:22: warning: implicit conversion changes signedness: 'int' to '__u32' (aka 'unsigned int') [-Wsign-conversion]
        sqe->rename_flags = flags;
                          ~ ^~~~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:547:66: warning: implicit conversion changes signedness: 'off_t' (aka 'long') to '__u64' (aka 'unsigned long long') [-Wsign-conversion]
        io_uring_prep_rw(IORING_OP_SYNC_FILE_RANGE, sqe, fd, NULL, len, offset);
        ~~~~~~~~~~~~~~~~                                                ^~~~~~
/home/ammarfaizi2/project/now/teavpn2/src/ext/liburing/src/include/liburing.h:548:26: warning: implicit conversion changes signedness: 'int' to '__u32' (aka 'unsigned int') [-Wsign-conversion]
        sqe->sync_range_flags = flags;
                              ~ ^~~~~
40 warnings generated.

There are many of them in C files of liburing sources. But at least, address the warnings in header as we expose it to user.

Temporary workaround
#if defined(__clang__)
#  pragma clang diagnostic push
#  pragma clang diagnostic ignored "-Wimplicit-int-conversion"
#  pragma clang diagnostic ignored "-Wshorten-64-to-32"
#  pragma clang diagnostic ignored "-Wsign-conversion"
#endif

#include <liburing.h>

#if defined(__clang__)
#  pragma clang diagnostic pop
#endif