[DASH-Sonic-HLD] Reference dash-sonic-hld to sonic-net repo

I approve the request on condition that the DASH repo has a pointer to the HLD. Also, the DASH repo will stay in tact where it sits today as managed by Kristina Moore.

Also, Kristina needs the same permissions in the SONiC repo in order to make updates.

Also, Kristina needs the same permissions in the SONiC repo in order to make updates.

Kristina can create PRs for any updates in Sonic repo and it is managed by Linux foundation. For other rights, I suggest to discuss with @zhangyanzhao.

I approve the request on condition that the DASH repo has a pointer to the HLD. Also, the DASH repo will stay in tact where it sits today as managed by Kristina Moore.

Ack, thanks @GeraldatMicrosoft. @KrisNey-MSFT , would you signoff this?

I approve the request on condition that the DASH repo has a pointer to the HLD. Also, the DASH repo will stay in tact where it sits today as managed by Kristina Moore.

Ack, thanks @GeraldatMicrosoft. @KrisNey-MSFT , would you signoff this?

Leaving document where it is located per 10/2/2023
do-not-merge

I approve the request on condition that the DASH repo has a pointer to the HLD. Also, the DASH repo will stay in tact where it sits today as managed by Kristina Moore.

Ack, thanks @GeraldatMicrosoft. @KrisNey-MSFT , would you signoff this?

Leaving document where it is located per 10/2/2023 do-not-merge

@KrisNey-MSFT , the document itself is not removed. Change is to reference to Sonic repo location so that there is only one document to update for community.

I approve the request on condition that the DASH repo has a pointer to the HLD. Also, the DASH repo will stay in tact where it sits today as managed by Kristina Moore.

Ack, thanks @GeraldatMicrosoft. @KrisNey-MSFT , would you signoff this?

Leaving document where it is located per 10/2/2023 do-not-merge

@KrisNey-MSFT , the document itself is not removed. Change is to reference to Sonic repo location so that there is only one document to update for community.

-> Will send reference


antsRegistration command line options

what outputs are now expected when collapse-output-transformsandwrite-composite-transform` are enabled?
it seems like this has gone through a bunch of changes over the last few months, including naming convention changes.
at present when both are enabled i only see this as output (with version 2.1.0).
output_Composite.h5         output_warped_image.nii.gz  output_InverseComposite.h5

no genericaffine or inversewarp
complete command below.
antsRegistration --float --collapse-output-transforms 1 --dimensionality 3 --initial-moving-transform [
/data2/software/fsl/data/standard/MNI152_T1_2mm_brain.nii.gz,
/data2/scratch/ds002/openfmri/registration/_model_id_1_subject_id_sub001_task_id_1/stripper/highres0
01_brain.nii.gz, 1 ] --interpolation Linear --output [ output_, output_warped_image.nii.gz ] --transform
Rigid[ 0.1 ] --metric Mattes[ /data2/software/fsl/data/standard/MNI152_T1_2mm_brain.nii.gz,
/data2/scratch/ds002/openfmri/registration/_model_id_1_subject_id_sub001_task_id_1/stripper/highres0
01_brain.nii.gz, 1, 32, Regular, 0.3 ] --convergence [ 10000x11110x11110, 1e-08, 20 ] --smoothing-
sigmas 4.0x2.0x1.0vox --shrink-factors 3x2x1 --use-estimate-learning-rate-once 1 --use-histogram-
matching 0 --transform Affine[ 0.1 ] --metric Mattes[
/data2/software/fsl/data/standard/MNI152_T1_2mm_brain.nii.gz,
/data2/scratch/ds002/openfmri/registration/_model_id_1_subject_id_sub001_task_id_1/stripper/highres0
01_brain.nii.gz, 1, 32, Regular, 0.3 ] --convergence [ 10000x11110x11110, 1e-08, 20 ] --smoothing-
sigmas 4.0x2.0x1.0vox --shrink-factors 3x2x1 --use-estimate-learning-rate-once 1 --use-histogram-
matching 0 --transform SyN[ 0.2, 3.0, 0.0 ] --metric Mattes[
/data2/software/fsl/data/standard/MNI152_T1_2mm_brain.nii.gz,
/data2/scratch/ds002/openfmri/registration/_model_id_1_subject_id_sub001_task_id_1/stripper/highres0
01_brain.nii.gz, 0.5, 32 ] --metric CC[ /data2/software/fsl/data/standard/MNI152_T1_2mm_brain.nii.gz,
/data2/scratch/ds002/openfmri/registration/_model_id_1_subject_id_sub001_task_id_1/stripper/highres0
01_brain.nii.gz, 0.5, 4 ] --convergence [ 100x30x20, -0.01, 5 ] --smoothing-sigmas 1.0x0.5x0.0vox --
shrink-factors 4x2x1 --use-estimate-learning-rate-once 1 --use-histogram-matching 1 --winsorize-image-
intensities [ 0.005, 0.995 ]  --write-composite-transform 1

Those are the expected output.  The affine and warp files are combined in the hd5 files.

thanks @ntustison - so in prior versions, it used to produce both the hdf5 files and the combined GenericAffine and Warp files. i suspect this is now changed. also i noticed that the help still says Collapsed in the filename, but that has also been removed.
closing this.

There was a brief period where both sets of files were produced.  However, that was a result of some slightly faulty logic.  I don't know what happened to the 'Composite' naming---Hans and his crew are the ones that have done the hd5 work.

Composite is there, but a November git version also produced CollapsedComposite. the current release though only says Composite and i'm going by that for out updates.


Question: Deployment Guide

We have four steps to deploy the framework as mentioned in the https://github.com/SAP-samples/btp-bridge-framework/blob/main/documentation/set-up/Bridge%20Framework%20Deployment.md#bridge-framework-deployment
We only see the first step.
Is there any guide for the remaining steps?

We have four steps to deploy the framework as mentioned in the https://github.com/SAP-samples/btp-bridge-framework/blob/main/documentation/set-up/Bridge%20Framework%20Deployment.md#bridge-framework-deployment
We only see the first step.
Is there any guide for the remaining steps?

Hi Thien,
I apologize for the inconvenience caused by the incomplete documentation issue. We understand the importance of providing accurate and complete documentation, and we have taken steps to rectify the situation.
Currently with Bridge Framework V2, we provide the option to deploy the bridge framework apps and configure the principal propagation between MS Azure, SAP BTP, and SAP S/4HANA manually, or you can utilize our new automation pipeline docker image to automate the process.
To help you with the deployment, we have prepared detailed instructions that you can follow. Please click on this link to access the instructions and complete the deployment process.
Thank you for your understanding and patience.
Best Regards,
Weikun Liu

Hi @I561660,
We cannot access your link (https://github.wdf.sap.corp/SCE/bridge-framework/blob/master/README.md#getting-started) as I believe it's intranet site. Can you check the public link.
Thanks,
Thien

Hi Thien,
Our apologies, a full guide can now be found here.
If you are trying to set up Bridge manually, then please reference this guide.
If you are just trying to deploy Bridge's applications to Cloud Foundry and then register the app to MS Teams, please refer to this guide.
We do recommend using the automated setup tool (mentioned in the full guide), if you can. Please find the docker image to set it up under our release page. Note, we are currently experiencing some problems with the apple chip/M1 version of the container - we hope to release an updated docker image there soon.
If you have any further questions, please let us know!
Best,
Alex

Hi @AFK-Python (Alex)
Thanks a lot. We will try and revert if there is any challenges. For now, I will close the issue.
BR,
Thien


chore: Update cargo dist

#271  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @AngelOnFira and the rest of your teammates on  Graphite


Publish on F-Droid

https://f-droid.org/

That was discussed some time ago and I discarded it. I think it's enough to have it on Google Play and GitHub.
https://github.com/JGeek00/adguard-home-manager/issues/8


feat: add query suggestions

EX-6305

The export for the query-suggestions-request.mapper is missing in packages/x-adapter-platform/src/mappers/requests/index.ts file. And the export for the response mapper is also missing.
Could you add the index.ts files with the exports for schemas/models and types/models directories?

done

gj!


Add embeddings

Adds embeddings. Tested it locally with a client I have to ensure it works.

This update looks good and complete. Have you considered writing any unit tests? It's good that you're using interfaces for dependencies to make it easier to mock things out for tests.
 Andy W
Was this helpful? Yes | No

Reviewers will be notified any time you reply to their comments or commit new changes.
You can also request a full follow-up review from your PullRequest dashboard.

Yep, I'll add some tests.


[shim] Adding Condition Hints

Description
This PR isolates some changes introduced by https://github.com/microsoft/demikernel/pull/1351.

Bypassing this review as it concerns the SHIM Layer and PR https://github.com/microsoft/demikernel/pull/1351.


Introduce API for querying the classpath of a Kotlin build script via TAPI

I assume this is about providing a stable gsk-client JAR (similar to the tooling-api JAR) which is able to get the classpath in a forward/backward-compatible way? That would be great, because right now there is brittle "get GSK from the distribution" code in IntelliJ and Buildship.

A little clarification on the API usage. I image Buildship querying the Kotlin classpath model via the TAPI and then pass it to GSK via the environment map. This way GSK won't have to do another TAPI query. As a consequence, we spare and extra configuration for the build which can be significant for large for large projects.

Thanks, @donat.

@donat what's going to take care of invalidating the model cache in that case?
With the current implementation, the resolver also takes care of detecting changes to the buildscript and plugins block and automatically re-querying the model whenever they do.
How would that work with Buildship providing the model directly?

The process is driven by the Kotlin script editor, so Buildship would query a fresh TAPI model for each change in the build script.

@donat I assume by "for each change" you mean "each time the user saves the build script"?

Yes.

Querying the model involve setting some project property and system property.
We are thinking about putting the following in the gsk-tapi jar:

the model type declaration, akka. KotlinBuildScriptModel
a helper method to set the right properties on e.g. the client-side ModelBuilder

@oehme, @donat, does that make sense to you or so you have an existing pattern already in use in TAPI client code we could apply here?

It makes sense. Apart from the Buildship-specific code where we apply the project preferences on the TAPI connection we directly use ModelBuilder. I suppose your helper method would set some extra JVM arguments on the connection.

@donat and I had an enlightening chat about this issue.
The main outcome is that there was some confusion about the goal.
Buildship needs to provide the Eclipse-Kotlin support with the script template class (KotlinBuildScript) and the classpath required to load them.
It has nothing to do with the classpath for build scripts that is potentially different for each build script. It's simply the gradle-script-kotlin classpath, which is related to the Gradle distribution. Computing that classpath won't be costly as we won't need to configure any project. No need either for the optimisation trick mentioned ☝.
So, basically we need to expose a TAPI model builder for some GradleScriptKotlinClasspathModel. This model would include a classpath for the gradle-script-kotlin jar and its dependencies. This model builder won't have any system/project properties as input. Pretty simple.
For Buildship not to drag the whole gradle-script-kotlin jar and its dependencies we need to put that model and model builder in it's own jar and publish it.

Computing that classpath won't be costly as we won't need to configure any project.

Any tooling model request fully configures the project.

@oehme thanks! that's what I thought after reading some more code ..

Of course it shouldn't be that way, so one answer could be to improve that as part of this story. The GradleBuild model for instance also shouldn't need a configured project, only evaluated settings.

@oehme we probably won't handle this as part of this story
I opened an issue for this: https://github.com/gradle/gradle/issues/1977

A first cut of the model builder is available in #358
I built a distro with it and sent it to @donat for a preliminary evaluation.
Once we validate it works we'll adapt our build to produce a tapi-api jar, see #304 .

I integrated the template classpath loading locally and can confirm that it works as expected. We still see the issue with Kotlin-Eclipse not picking up the result of the resolver. I'll contact JetBrains on that matter by reviving the https://gradle.slack.com/archives/G0DGS35E2/p1491817311440691 thread.

Model: org.gradle.script.lang.kotlin.tooling.models.KotlinBuildScriptTemplateModel.
In master by e773d0d15942a72bb7fe28192b157480ceaa37cf


Ingress creation is single threaded

At one point we would go ALBIngress.SyncState(), lets look at enabling this again to improve throughput when creating/modifying many ingresses.
I think as long as we lock on ALBController.Onupdate() & ALBController.Reload() we should be safe.

Resolved in master


Allow custom ACME servers for certificate generation

Is your feature request related to a problem? Please describe.

It would be really nice to be able to add custom ACME servers to the certificate generation.
For example I have an acme server that generates internal signed certificates and I currently need to upload them every time I update them.
Describe the solution you'd like

Either allow acme servers added via the web interface or maybe a config file?
Describe alternatives you've considered

Additional context

Maybe as an addition :
documentation how to do this manually ?

I'd love to see this as well. I was able to connect to the console of the npm container and manually create certificates using certbot with my custom ACME server like this:
certbot certonly -d <domain> --agree-tos --no-eff-email \
--server <server_url> \
--eab-kid <kid> \
--eab-hmac-key <hmac>

But actually using them is kind of a nightmare. I have to get the files out of the container and onto my local machine just to then reupload them through the web UI. I had hoped to at least symlink the certs directly from /etc/letsencrypt to /data/custom_ssl but it looks like the web UI stores the certificates in the database and only writes them out to disk so that nginx can access them so I'm not sure that that would survive a restart. (and even if it did it would still require manually uploading a dummy

Looking at the code, it looks like npm creates /etc/letsencrypt.ini by default and it turns out you can just add the config in there and it works! I just added:
server = <url>
eab-kid = <kid>
eab-hmac-key = <hmac>

and then tried using the UI to generate a new certificate and it worked perfectly! I'd still love to see an option added to the UI to manage this, but this actually works right now which is great.

Does that allow multiple? For this case I have internal domains (not on a valid tld), and regular public domains. So can I set 2 acme servers?

You would have to modify it each time (or just create two copies and swap them out between invocations). Fortunately, once the certificate is created, there's a cert-specific .conf file created in /etc/letsencrypt/renewal which does store the server name, so renewals should continue to work even if you've changed the server in the config each time.

See #1054:

I am planning to add configurable CA's to v3.

Originally posted by @jc21 in https://github.com/NginxProxyManager/nginx-proxy-manager/issues/1054#issuecomment-870213956

+1

+1

+1

+1


Identity plugin for multiple blockchain protocols

Requirement
The identity plugin needs to be enhanced to handle requirements from different blockchain protocols. While the Ethereum identities are global, a single address string representing the unique account, Fabric identities are scoped to organizations.

example Ethereum identity: 0x170dcf0187fb2f8884b63cfa2b1573c37f32eeb8
example Fabric identity: Org1MSP::x509::CN=user001,OU=client::CN=fabric_ca.org1.example.com,OU=Hyperledger Fabric,O=org1.example.com,L=San Francisco,ST=California,C=US, which contain these segments:

MSP ID
certificate type
user id
CA ID

While both types of identities can be represented in a string format, it'd be very cumbersome to require developers to input the full Fabric string as the signer property in the API payload.
Practically, a Fabric client would be tied to a single Fabric CA with a particular org MSP, so the API payload only needs the user ID portion to be specified, user001 in the above example. But as soon as the identity gets into the FireFly core, it should be resolved to the fully qualified string before it gets replicated to other FireFly nodes.
In addition, the Fabric chaincode spits out the signer identity in the chaincode event in the fully qualified format, which means the matching logic in the aggregator won't work unless FireFly core resolves the supplied signer identity to use the same format.
Proposed Solution 1
Provide separate identity plugins for each protocol. this means a new protocol support requires at least 2 plugins, the blockchain plugin and the identity plugin. there will be some overlap between the two because the blockchain plugin is also responsible for validating the identity strings
Proposed Solution 2
Have the OnChain identity plugin consult the blockchain plugin for resolving the identity. this requires inter-plugin dependencies as the blockchain plugin must be initialized ahead of the identity plugin

Please see https://github.com/hyperledger/firefly-fir/pull/9


cherry-picking from other forks

Some people are still fixing stuff in their forks of poliastro, as one can see in the Network display:
https://github.com/poliastro/poliastro/network
I think it may be worth it to contact those people whose fixes you may want to incorporate. This may bring you additional contributors and spread the word that hapsira is the continuation of poliastro.

Indeed - got swamped due to moving. Will resume here shortly.


bug: cortex run URL command chain is broken

Cortex version
v83
Describe the Bug
cortex run chain of commands is broken when:

user invokes run with a url installion path
the cli tries to load the model after installing but gets the id wrong (it assumes url instead of actual model id)
Note: model downloaded perfectly, as running cortex run separately works

Error seems straightforward to fix:
Failed to read file: bad file: C:\Users\n\cortexcpp-nightly\models/https-//huggingface.co/cortexso/mistral/blob/main/model.gguf.yaml
Steps to Reproduce
(base) PS C:\Users\n\cortexcpp-nightly> cortex-nightly run https://huggingface.co/cortexso/mistral/blob/main/model.gguf
Validating download items, please wait..
Start downloading: mistral
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  1113  100  1113    0     0   3569      0 --:--:-- --:--:-- --:--:--  3578
100 4166M  100 4166M    0     0  10.3M      0  0:06:41  0:06:41 --:--:-- 10.7M
Model mistral downloaded successfully!
Failed to read file: bad file: C:\Users\n\cortexcpp-nightly\models/https-//huggingface.co/cortexso/mistral/blob/main/model.gguf.yaml

Screenshots / Logs
No response
What is your OS?

[ ] MacOS
[X] Windows
[ ] Linux

What engine are you running?

[X] cortex.llamacpp (default)
[ ] cortex.tensorrt-llm (Nvidia GPUs)
[ ] cortex.onnx (NPUs, DirectML)

@0xSage Can you try again with cortex v0.5.0-98?

works


The getServiceURL() method does not return expected string when using query and queryItems.

When specifying both, query and queryItems, in the replacements argument to the getServiceURL() method, the return value is not the expected one on Linux.
This issue was caused by differences in logic inside the custom SwiftyJSON library our team maintains. When iterating over the keys in a JSON object, the keys were returned in different order by the SwiftyJSON library on  OS X and Linux. To address this, refactoring in the Swift-cfenv library has been done to guarantee that replacements are always processed in a known order regardless of the platform.

This issue is now fixed in release 0.0.3.


Set the time zone to London

Trello: https://trello.com/c/WrpUNtsn
Config taken from: https://github.com/alphagov/content-publisher/commit/4ae1c4ed683bf0b24eb11d135aebd3cdd1a8ffa6
We store dates in the database in UTC, which means that when we're displaying
dates and times in the views they're an hour out of date during daylight saving
time. This sets the app's time zone to 'London' so that ActiveRecord
automatically converts dates and times to the UK's current time zone, whether
it's GMT or BST.
Also updates a test that was trying to get around this problem by converting the dates checked to "UTC +00:00"
Expected changes
Before

After

@AgaDufrat You're right, the line in_time_zone is no longer needed in the test. I've removed it and fixed up.

@kevindew I've updated the test to use Time.zone.parse as recommended. Time.new was using UTC so I've changed the expectation as well.


Getting started seems to be hardcoding file protocol

Steps to Reproduce:

set "enable-browser-code-loading": "bypassHeatCheck" in argv.json
restart
open getting started

Not allowed to load local resource: file:///Applications/Visual%20Studio%20Code%20-%20Insiders.app/Contents/Resources/app/out/vs/workbench/services/gettingStarted/common/media/colorTheme.png

We use: FileAccess.asFileUri('vs/workbench/services/gettingStarted/common/media/' + path, require);
https://github.com/microsoft/vscode/blob/840f19af066b9102b4ee1e852e2a6ad934298b6b/src/vs/workbench/services/gettingStarted/common/gettingStartedRegistry.ts#L144

asBrowserUri seems to do the trick.

In almost all cases you want asBrowserUri 👍


ReferenceError: window is not defined

Describe the bug
使用代码块报错
Reproduction
ReferenceError: window is not defined
Logs
No response
Screenshots
No response
Severity
annoyance

Did you use any third-party libraries?
Or can you provide more context, please?
Additionally, providing reproduction would be better.

`
onMount(() => {
import("epubjs").then(model) => {
});

});
`


Term request: ligation

In describing DNA assembly reactions, it would be useful to have a specific term for ligation of sequences.
It seems appropriate that this should be a child of SBO:conversion (SBO:0000182), as ligation is a modification of covalent bonds. It is also complementary with SBO:cleavage (SBO:0000178), which is the rupture of a covalent bond, whereas ligation is the formation of a covalent bond. It is also closely related to SBO:polymerization (SBO:0000652), except that refers to the joining of monomers to form a polymer whereas ligation is the joining of two or more polymers to form a larger polymer
Accordingly, I propose the following:

Name: ligation
Definition: Formation of a covalent bond resulting in the creation of a link between the ends of one or more linear polymer molecules.
Parent: SBO:conversion (SBO:0000182)

Note that this definition of ligation can apply not only to DNA ligation but also the RNA ligation and protein ligation. This definition also allows for ligation to describe circularization of a single DNA fragment, joining of two DNA fragments, and joining of multiple DNA fragments.

Thanks, @krish193 for tackling the request. Closed the request now. @jakebeal Sorry for the late response.

Thank you - the new term looks good to me.
When is this expected to go live for use? I see that at present, the EBI lookup service is not yet showing the new term.

There is a new version of OLS available (OLS4) in which the term is already visible after searching for ligation.


[Err] [RTShaderSystem.cc:489] Unable to find shader lib. Shader generating will fail

when I run  make px4_sitl_default gazebo

@Sunshinehualong I have exactly the same error. Can you please guide me how to solve it? I have Gazebo multi-robot simulator, version 7.14.0

@mrivi can you please guide I have same error and my gazebo version is also 7.14.0.

If you have exactly the same error, I would suggest exactly the same solution...?

@nicovanduijn he said to change the gazebo version, so change means upgrade or downgrade? By the way I am not PX4 package rather using another package but the problem is same as given above.

It means you should set your GAZEBO_RESOURCE_PATH correctly. Depending on what version of gazebo  you are running.
If you are running gazebo 7.0, run export GAZEBO_RESOURCE_PATH=/usr/share/gazebo-7.0.


Unable to disable camera

Type of issue

[x] Bug report
[ ] Feature request
[ ] Support request

Uploader type

[ ] Traditional
[x] S3
[ ] Azure

Bug Report
Fine Uploader version
5.15.0  (s3.fine-uploader.min.js)
Browsers where the bug is reproducible
Mobile Safari
Operating systems where the bug is reproducible
iOS 10.3.3
Detailed explanation of the problem
When running a test wage page (Based on your gallery demo) on mobile safari, after clicking on the button to select an image I am alway given the "Take Photo or Video" option, regardless of whether I set the option camera.ios = false, or omit the configuration altogether.

Feature Request
Feature request details
{why is this feature important, not just for you, but for many others?}

Support Request
Fine Uploader version
{example: 5.5.1}
Question
{ask your support question here & be as detailed and clear as possible}
Related code
{post any code related to your question here}

I don't believe this can be disabled anymore.


Added showCheckboxColumn parameter to DataTable and PaginatedDataTable

Description
The showCheckboxColumn property will determine whether or not selectable rows in DateTable or PaginatedDataTable will contain a leading CheckBox.
showCheckboxColumn == true:

showCheckboxColumn == false:

Related Issues
N/A
Tests
I added the following tests:

DataTable control test - no checkboxes

Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: Replace with a link to your migration guide

approach SGTM

Not sure why I'm having some golden tests fail. @Piinks would you happen to know if I'm just holding something wrong?

Looks some new golden files landed and weren't triaged. Should be all good now. I've re-run the failed tasks. :)

Looks some new golden files landed and weren't triaged. Should be all good now. I've re-run the failed tasks. :)

My hero! Thanks Kate :-)


bridge plugin crashing in the cleaner code

when running the new tests from kubevirt. see PR: https://github.com/kubevirt/kubevirt/pull/1342
run:
FUNC_TEST_ARGS='-ginkgo.focus=vmi_bridge_test -ginkgo.regexScansFilePath' make functest

logs from plugin pod:
panic: failed to bind unix socket: address already in use

goroutine 15 [running]:
github.com/kubevirt/kubernetes-device-plugins/pkg/network/bridge.(*cleaner).Run(0xc42000c1a0)
	/home/travis/gopath/src/github.com/kubevirt/kubernetes-device-plugins/pkg/network/bridge/cleaner.go:74 +0x149
created by github.com/kubevirt/kubernetes-device-plugins/pkg/network/bridge.(*NetworkBridgeDevicePlugin).Start
	/home/travis/gopath/src/github.com/kubevirt/kubernetes-device-plugins/pkg/network/bridge/plugin.go:62 +0xd3

The problem is that currently we try to create a monitor per resource (per device plugin) and not per pod (PID). So, a pod has multiple processes, it will try to create multiple monitors with the same unix socket address (the PID).

The original cleanup code was removed. Closing


"rdopkg patch" should switch over to the dist-git branch if necessary

If I am on a -patches branch, and I run rdopkg patch, I get an error:
$ rdopkg patch
## get_package_env

Invalid usage: This action must be run on a distgit branch.

It would be great if rdopkg could intelligently figure out what dist-git branch corresponds to my patches branch, and then switches over there, and then carries on with the patch operation. (This is how Debian's git-buildpackage tool works when I run gbp pq export when writing patches out for a Debian package.)
There might be some cases where one -patches branch could map to multiple dist-git branches, in which case an error is fine. But if there is only one match, then we should be able to do the Right Thing and just switch over.

Oh yes. Currently, this works for simplest case $DISTGIT and $DISTGIT-patches, but not for more complex cases like mitaka-rdo and mitaka-patches.
I don't currently have bandwidth for this quality of life improvement, so you or anyone else who wants it might write it:

add rdopkg.guess:find_distgit() function to find the correct distgit branch, see other such magic in guess module.
enrich '-patches' logic in the beginning of rdopkg.actions:get_package_env() function with 1)


Lasair settings - move configuration IP addresses and ports out of Hashicorp Vault to Ansible

Config IPs and config ports are buried in Hashicorp Vault secrets.  These should really be pulled by Ansible from Ansible variables or Ansible host groups (since these will change for every deployment).  Burying them into the Vault secrets makes new (e.g. development & test) deployments very difficult without lots of post install manual fixes.

This is all handled properly in lasair-deploy


Migrate helium-hardware-definitions

Move all imports that currently happen in Dockerfiles and/or manually copied into projects into hm-pyhelper. Import using PyPi instead.
Current location: https://github.com/NebraLtd/helium-hardware-definitions

Duplicate of #6


cannot read property 'Chart' of undefined

require('chart.js') by CommonJS way.

Fixed in version 2 with addition of UMD build in #1736


Support more RESTful API

Hello,
I'm just starting to use tiedot for an application on a low memory footprint device.  The results so far seem very promising!
The current API is definitely usable, but I was wondering if it would be possible to support a more RESTful API?  There are a couple of issues that this would solve.

Any HTTP method can be used for any request, which violates the intended use of the HTTP methods.  A RESTful API would use only the intended method for each request.
Documents are JSON (and only JSON), so it seems unnatural to URL encode them and submit them either with a GET (as URL encoded query params) or POST/PUT (as application/x-www-form-urlencoded content).  A RESTful API would use path parameters and application/json encoding only, which would simplify things.  Also, unnecessary overhead of URL decoding the documents would be eliminated.  For example, in the tutorial a document is inserted like this (lines omitted for brevity):

$ curl -v --data-ascii doc='{"a": 1, "b": 2}' "http://localhost:8080/insert?col=Feeds"
> POST /insert?col=Feeds HTTP/1.1
> Content-Length: 20
> Content-Type: application/x-www-form-urlencoded
>
* upload completely sent off: 20 out of 20 bytes
< HTTP/1.1 201 Created
< Cache-Control: must-revalidate
< Content-Type: text/plain
< Content-Length: 19
* Connection #0 to host localhost left intact
3587505840683728602

This would instead become something like:
POST /v1/insert/Feeds HTTP/1.1
Content-Type: application/json
{"a": 1, "b": 2}

The v1 prefix makes this a new API version so the existing API can still be supported.  The new version would use query path parameters, application/json content, and restrict to appropriate HTTP verbs.  For example:
# Collection management
POST   /v1/create                 # param(s) in request body JSON
POST   /v1/create/:col            # empty request body
POST   /v1/create/:col/:numparts  # empty request body
GET    /v1/all
PUT    /v1/rename                 # param(s) in request body JSON
PUT    /v1/rename/:old/:new       # empty request body
DELETE /v1/:col                   # empty request body
GET    /v1/scrub/:col
GET    /v1/sync

# Document management
POST   /v1/insert/:col
GET    /v1/get/:col/:id
PUT    /v1/update/:col/:id   # replace entire existing document
PATCH  /v1/update/:col/:id   # (optional) modify only specified fields in document
DELETE /v1/delete/:col/:id
GET    /v1/approxdoccount/:col
GET    /v1/getpage/:col/:page/:total

# Etc.

I can offer to help with documentation, but this project is my first foray into golang, so I doubt I could help much with implementation at this point.  Thanks!

I'm very glad to hear that tiedot could be useful to you. I'll think through it and get back to you soon.

I like the idea very much, but unfortunately at the moment I do not have time working on this feature. Please take a look at:
https://github.com/HouzuoGuo/tiedot/blob/master/httpapi/srv.go
Would you like to give it a try at implementing these APIs?

:+1: for the suggested API, it will work great with Angular Resource which will make a new control panel development and other tiedot + angular apps very smooth.

:+1:
As a starter, I think it would be quite useful  to have a simple RESTful API with a minimal feature set for basic operations. Since tiedot is using JSON internally, then json:api and RAML could be a good fit for modelling the API.

I might be able to work on this once I get a bit more experience with golang and some more free time.  We're just starting to port an existing Java application to golang in order to be able to run in a smaller footprint.  FYI, we're using tiedot to replace Hazelcast in the new architecture.

I just added all of you into collaborator list, please feel free to experiment with Restful API implementations however you like!
@mmindenhall if you see any vital feature of hazelcast missing from tiedot, or if you have any questions, feel free to ask!

In the next few weeks I have some free time to spare and I'd like to tackle this issue. If anyone of the previous commenters wants to join me then is welcome.
As a start I was thinking to model the API using RAML or OpenAPI Spec (ex Swagger) formats; it seems that OpenAPI/Swagger 2.0 is well supported by several Golang tools, while RAML isn't.
So, I vouch for OpenAPI/Swagger by using https://github.com/go-swagger/go-swagger. With go-swagger we can automate the creation of all the boilerplate code necessary for the client / server API, and automate the API validation too. All we need to do is to manually implement the models of the API. This way go-swagger can save us tons of time and the usual useless drama about API stability and its validation process.
Any preference / suggestion / alternative about it?

Hey Doctor.
I have heard about swagger and how easy it is to manage the entire API lifecycle with it, in fact another project of mine is also making use of swagger. It would definitely be a nice addition to tiedot, please feel very free to go ahead with implementing it.

I decided to work on this enhancement using httprouter.
Below you will see my proposed set of routes. They may change, but this is just the starting point.

 This will make the query object more scalable over time.
API Routes Versioning
I did not add a version number to the route. I think braking changes or reasons for a major version change should be controlled at the application level and not the API. Versioning an API may have its advantages on a service API, but I am not sure it will be beneficial for a database interface since it may create technical debt before getting to another major version. This should be controlled with tags and new version releases. Thoughts?
API Backwards Compatibility
I added an boolean argument "legacy-api-support" to the server in order to enable the old API routes. This should allows anyone with existing code to continue using the old routes while they migrate.
If everything goes well I may have a full working version soon.

Thanks very much @prods, please proceed with the proposal and I've added you to collaborator list.
Would it be more appropriate to use POST action for /sync, /shutdown, and /dump?

Thanks! and Yes, it makes perfect sense. Database engine specific operations should run on POSTS. I will update the route list and I will keep you posted. Let me know if you find anything else or you have any recommendation.

Below is an updated version of the API Routes.

In other hand after closer review I realized that error messages are being returned as plain text and I would like to propose, as part of this api change, to upgrade all api results to JSON objects. See my initial proposal below... I kind of like verbose api errors but the idea is to keep it as simple as possible...
{
  "collection": "<COLLECTION_NAME>",
  "operation": "<OPERATION>",
  "error": "<ERROR MESSAGE>"
}

Sample:
{
  "collection": "test_collection",
  "operation": "delete",
  "error": "Collection test_collection does not exist"
}

Thoughts?

Adding. Below is a better list of proposed error objects.
Collection Operations
{
  "collection": "test_collection",
  "operation": "delete",
  "error": "Collection does not exist"
}

Document Operations
{
  "collection": "test_collection",
  "document":<PHONE_NUMBER>2,
  "operation": "update",
  "error": "Document does not exist"
}

Engine Operations
{
  "operation": "dump",
  "error": "Specified Path cannot be found"
}

The error messages are not necessarily real. I added them in order to provide some examples.
Let me know.

Ok, just another update. I already created all the routes and successfully completed a first round of testing. Below you will see a few more changes in the proposal. As before I marked in bold the changes  and included some possible response payloads for functions that are silent today.

Method
Routes
Original Routes
Request
Response
Status

GET
/collections
/all
N/A
[ "c_name1", "c_name2", ... ]
Created

POST
/collection/:collection_name
/create
N/A
{ "done" : true } (Proposal)
Created

PUT
/collection/:collection_name/rename/:new_collection_name
/rename
N/A
{ "done" : true } (Proposal)
Created

DELETE
/collection/:collection_name
/drop
N/A
{ "done" : true } (Proposal)
Created

POST
/collection/:collection_name/scrub
/scrub
N/A
{ "done" : true } (Proposal)
Created

POST
/collection/:collection_name/query
/query
{ "c" : [...] }
{ "<id>" : {}, ... }
Created

POST
/collection/:collection_name/count
/count
{ "c" : [...] }
{ "count" : 100 } (Proposal)
Created

POST
/collection/:collection_name/doc
/insert
{  }
{ "id" : "" } (Proposal)
Created

GET
/collection/:collection_name/doc/:id
/get
N/A
{ }
Created

GET
/collection/:collection_name/page/:page/of/:total
/getpage
N/A
{ "<id>" : {}, ... }
Created

PUT
/collection/:collection_name/doc/:id
/update
{ }
{ "done" : true } (Proposal)
Created

DELETE
/collection/:collection_name/doc/:id
/delete
N/A
{ "done" : true } (Proposal)
Created

GET
/collection/:collection_name/count/approx
/approxdoccount
N/A
{ "count" : 100 } (Proposal)
Created

POST
/collection/:collection_name/index
/index
{ "path" : "" }
{ "done" : true } (Proposal)
Created

DELETE
/collection/:collection_name/index
/unindex
{ "path" : "" }
{ "done" : true } (Proposal)
Created

GET
/collection/:collection_name/indexes
/indexes
N/A
{ "", ... }
Created

POST
/sync
/sync
N/A
N/A
Created

POST
/shutdown
/shutdown
N/A
{ "done" : true } (Proposal)
Created

POST
/dump
/dump
{ "destination : "" }
{ "done": true } (Proposal)
Created

I will continue the testing and I will keep you posted. Please feel free to post any idea, comment and/or concern...

Looks really good so far!

Just an update. This continues to move forward. Finding time here and there to complete it. You can take a pick at My Fork Branch. The API is now fully working and I am close to jump into documentation and final testing. I will keep you posted.

Forgot mention, please  make sure to run it using the new boolean use-new-api argument. This will enable the new API. You will find a postman collection in the doc folder that you can use to test-drive it.

Many thanks for your effort, the code is looking very good already.
Would you mind creating a wiki article describing the usage of new API as well? Afterwards please merge the changes whenever you're ready.

No problem, already started working on the documentation. Will definitely add a Wiki article with all the details before merging the changes. Thanks! I will take a look at the other project, looks interesting from what I briefly saw.

thank You!
coincidentally, according to cloc program, the laitos project has 12345 lines of go code as of now :smile:


*: rename coreos to container linux

LGTM. Not a single instance of CoreOS was replaced in the wrong context, I'm impressed.
eg. The engineers at Container Linux have worked hard...

Yeah, this was a pain. I actually had to read every occurrence. I'm impressed that you read all of them as well. Let's not rename the OS anymore.


Provide instructions to fully automate mounting container or file storage with terraform

These instructions are too complicated and require sshing into the vm. It'd be really useful to be able to mount as a VM provisioning step.

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

ID: 6fde1939-787d-2aa5-4b29-d303f8ed49cb
Version Independent ID: c2838cb5-c34c-e0ff-b280-b86f29453cad
Content: Attach a data disk to a Linux VM - Azure Virtual Machines
Content Source: articles/virtual-machines/linux/attach-disk-portal.md
Service: virtual-machines-linux
Sub-service: disks
GitHub Login: @cynthn
Microsoft Alias: cynthn

@rbavery
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

#re-assign @TomArcherMsft Tom, can we provide a doc on adding and mounting a data disk for a new VM using Terraform?

#reassign @TomArcherMsft

#reassign:TomArcherMsft

Here's a script my colleague uses with terraform
#! /bin/bash
# Setup Azure CLI
curl -sL https://aka.ms/InstallAzureCLIDeb | sudo bash
sudo az login --identity
# Mount Data Disk
sudo parted /dev/sdc mklabel gpt
sudo parted -a opt /dev/sdc mkpart primary ext4 0% 100%
sudo mkfs.ext4 /dev/sdc1
sudo partprobe /dev/sdc1
sudo mkdir /workspace
sudo mount /dev/sdc1 /workspace
id=$(sudo blkid -s UUID -o value /dev/sdc1)
sudo echo "UUID=$id   /workspace   ext4   defaults,nofail   1   2" | sudo tee -a /etc/fstab

@wmgries Will, can you help with mounting an Azure file share during VM provisioning using Terraform?


trying to solve the issue base-wallet-unlocked.ts

Description
This pull request updates the BaseWalletUnlocked class with the following changes:

Modified signMessage: Updated to accept BytesLike instead of a string and removed automatic hashing.
Introduced signHashedMessage: Added a new method that hashes the message before signing, preserving the previous behavior.
Updated JSDoc Comments: Revised documentation to accurately reflect the new and updated functionality.

Issues Closed

Closes #base-wallet-unlocked.ts

Release Notes
In this release, we:

Modified signMessage to accept BytesLike directly, removing the need for automatic hashing.
Introduced the signHashedMessage method for scenarios requiring hashing before signing.
Updated internal documentation and comments for clarity.

Summary
These changes enhance flexibility and clarity in the BaseWalletUnlocked implementation by separating concerns for message signing and hashing. This update introduces breaking changes and requires a migration for any code relying on the old signMessage behavior.

Breaking Changes
This update introduces a breaking change:

signMessage no longer hashes messages automatically. Consumers must now use the new signHashedMessage method if hashing is needed before signing.

Checklist

[x] All changes are covered by tests (updated or newly added).
[x] All changes are documented in the code and supporting materials.
[x] I reviewed the entire PR myself.
[x] Breaking changes are described and highlighted.

Additional Notes
Please update the major version number to reflect the breaking nature of this change. Documentation and migration guides should be reviewed to ensure users are informed of the changes.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi, @crStiv 👋🏼
We already have an issue for this, which @Torres-ssf is working on. A number of downstream dependencies need to be considered before making this change. I'll close this PR for now, but thank you for your contribution. If you're still interested in contributing, please take a look at the open issue (without an assigned person).

Hi, @crStiv 👋🏼
We already have an issue for this, which @Torres-ssf is working on. A number of downstream dependencies need to be considered before making this change. I'll close this PR for now, but thank you for your contribution. If you're still interested in contributing, please take a look at the open issue (without an assigned person).

Hi, @crStiv 👋🏼
We already have an issue for this, which @Torres-ssf is working on. A number of downstream dependencies need to be considered before making this change. I'll close this PR for now, but thank you for your contribution. If you're still interested in contributing, please take a look at the open issue (without an assigned person).


Fix StructuredQuery tests by using Reflection

Part of #26

Coverage decreased (-0.1%) to 60.971% when pulling d4726a3ff4de3b5a2fe0fc2d36330158d93cd2de on nare469:ns-sq-issameas into f0bfae658c02334754b5ca4915a628d589866ac4 on graphflow:master.

Coverage decreased (-0.1%) to 60.971% when pulling 53227d757eab58ee36524ce939f01a3f36c37b2c on nare469:ns-sq-issameas into f0bfae658c02334754b5ca4915a628d589866ac4 on graphflow:master.

Coverage decreased (-0.07%) to 61.027% when pulling 876631ae57b21b17b6abe566e4c10f30b1da31f3 on nare469:ns-sq-issameas into f0bfae658c02334754b5ca4915a628d589866ac4 on graphflow:master.

Coverage decreased (-0.07%) to 61.027% when pulling 2c89d571498effb07ba8e5a908c22639d828bc77 on nare469:ns-sq-issameas into f0bfae658c02334754b5ca4915a628d589866ac4 on graphflow:master.

Coverage decreased (-0.07%) to 61.283% when pulling 68fbbd14aade55ad171859ea744e372942c94af4 on nare469:ns-sq-issameas into 6108acb3430e15363928a2e33e1fd2f79b6ff0ea on graphflow:master.

Coverage decreased (-0.07%) to 61.283% when pulling 6cb0bdc8c03fcac9966a3907890c49a0766df805 on nare469:ns-sq-issameas into 6108acb3430e15363928a2e33e1fd2f79b6ff0ea on graphflow:master.

Coverage decreased (-0.07%) to 61.283% when pulling b19e33359edaa9524ffdedbc376cefbd221de7f9 on nare469:ns-sq-issameas into 6108acb3430e15363928a2e33e1fd2f79b6ff0ea on graphflow:master.


Add coverage measurement on each test run

In order to implement coverage-guided strategies, we need to measure coverage each time a given test runs with a particular set of inputs. Coverage should be tracked so that a relation between coverage and inputs can be established. More details will be given as the development advances.

Depends on #146 and #169.

This PR is ready for review. It is worth mentioning that this PR purpose is to measure coverage on each test run and not making the input generation be guided by coverage. This last objective will be achieved in a future PR.


payment.id becomes negative when converting to Int

Hi @FelipeGangrel, thanks for your PR! If you're turning the ID value from int to string, we do need to modify the following files too:

Typings
IOS Payment
IOS Payment

Our idea is to give parity and enforce same return values from IOS and Android. Would you mind to adapt those too so we can merge this and release a newer version?

@JonatanSalas I just created a new PR containing the changes for Ios and Typescript types.
Cheers

@FelipeGangrel thank you very much for both PRs, I've merged both of them and released version 0.7.1.


API [Edit event] Changes in the event can be saved if the startDate/finishDate have past dates

Environment: OS Windows 11 Pro version 22H2, browser Chrome version 121.0.6167.86
Reproducible: always
Build found: 14.02.2024
Preconditions

User_1 is created and authorized.
Event_1 is created by the User_1.

Steps to reproduce

Create a request:

method - PUT
URL - greencity.greencity.social
endpoint - /events/update
body - form-data format; parameter eventDto
value with startDate/finishDate set as dates in the past, e.g.:
{
"title":"The quick, brown fox jumps over a lazy dog. DJs flock by when MTV ax q",
"description":"abc def ghi jkl mno pqrs tuv wxyz ABC DEF GHI JKL.",
"open":true,
"datesLocations":
[
{
"startDate":"2024-02-09T18:00:00+02:00",
"finishDate":"2024-02-09T23:59:00+02:00",
"onlineLink":"https://www.greencity.social/#/events/create-event",
"coordinates":{
"latitude":50.4501,
"longitude":30.5234
}
}
],
"tags":["Economic"],
"imagesToDelete":[],
"additionalImages":null,
"id":{{new_event_id}},
"titleImage":"img/habits/default-habit-image.png"
}

Send the request
Observe a response

Actual result
Status 200 OK is received in the response. The Event with the starDate/finishDate in the past is saved.
Expected result
Status 400 Bad Request is shown in the response.
User story and test case links
User story #3193
Swagger

bug already resolved


fluxengine rpm

Hi I'm trying to use your fantastic tool, I have two challenges, I did use the full windows precompiled setup and looks like as soon as I connect the programmed using the mini usb ( programming went well, successful programmed and all green ) kit to the PC I get in the control panel device manager an exclamation mark about the device driver not installed, what should be the device driver ? shall I install any additional drivers ?
I did try to connect the usb mini to another pc with linux and I don't get this
[22720.093091] usb 1-1: new full-speed USB device number 14 using xhci_hcd
[22720.244882] usb 1-1: New USB device found, idVendor=1209, idProduct=6e00, bcdDevice= 0.01
[22720.244887] usb 1-1: New USB device strings: Mfr=2, Product=1, SerialNumber=128
[22720.244889] usb 1-1: Product: FluxEngine
[22720.244891] usb 1-1: Manufacturer: Cowlark Technologies
[22720.244892] usb 1-1: SerialNumber: EA00021615121324
trying to run fluxengie rpm I get this :
gianrisa@promaster:~/fluxengie/fluxengine$ sudo ./fluxengine rpm
Error: this only makes sense with a real disk drive
same I get in Windows.
I did check all the steps in the Doc 10 times and looking at the connections are done correctly. What am I missing ?
I'm using the latest version dowloaded from git.
Thanks for your help in advance

That's kinda weird. Try fluxengine test bandwidth --- that'll test communication to the hardware without needing an actual drive connected.
One thing which could be wrong is that you may not have permissions to access the USB device, resulting in it not being detected. You may need something like this in your /etc/udev/rules.d/local.rules:
SUBSYSTEM=="usb", ATTRS{idVendor}=="1209", ATTRS{idProduct}=="6e00", MODE="0666"

I'll add a command to list detected devices.

Servus David,
I did some try I'm directly root:
root@promaster:/home/gianrisa/fluxengie/fluxengine# ./fluxengine test bandwidth
Using FluxEngine EA00021615121324
Reading data: transferred 1048576 bytes from device -> PC in 1070 ms (956 kB/s)
Writing data: transferred 1048576 bytes from PC -> device in 2298 ms (445 kB/s)
trying this:
root@promaster:/home/gianrisa/fluxengie/fluxengine# ./fluxengine rpm
Error: this only makes sense with a real disk drive
all commands execute with roots rights
With windows I gave up I get the device driver error in the control panned device manage ... strange ...
Thanks
VG Gian

Hi David,
I have only one drive and It is attached to connector floppy:0 nearest to the controller board. can also attach it to the other connector, and It will work if I add drive:1
root@promaster:/home/gianrisa/fluxengie/fluxengine# ./fluxengine rpm -s drive:0
Using FluxEngine EA00021615121324
Rotational period is 200 ms (300 rpm)
I don't know seems that ./fluxengine rpm in my case doesn't detect the floppy.
What about the windows setup why is windows not recognizing the device ?
Thanks a Million for your product and super direct support !
 Plus, I added that command --- get the latest version and then do fluxengine test devices and you'll get a dump of the detected devices.


fix(AppBar): fix selected state label color in bottom navigation bar

Changes description 🧑💻

Fix for the selected color state of the label in the bottom navigation bar
Context 🤔

#33
Checklist ✅

[x] Make sure you are requesting to pull a topic/feature/bugfix branch. Please, don't request directly from your main!
[x] Check commits & PR names matches our requested structure. It must follow the https://www.conventionalcommits.org pattern.
[x] Check compose contract convention. It must follow conventions described here.
[x] Check your code additions will fail neither code linting checks.
[x] I have reviewed the submitted code.
[x] I have tested on a phone device/emulator.
[x] I have tested on a tablet device/emulator.
[x] I have tested on a large screen device/emulator.
[ ] If it includes design changes, please ask for a review with a core team designer.

Screenshots 📸

Other info 👋

You can download the apk here :
sample-debug.apk :
https://app.bitrise.io/app/e9ae480151be4514/build/1a3a9de4-8b48-4b40-bcc8-63959a1b0ea5/artifact/05b261db3433b9b6/p/59f89782317c856dd2a529d872d47d92

You can download the apk here :
sample-debug.apk :
https://app.bitrise.io/app/e9ae480151be4514/build/64e37161-39c9-471e-bef2-0b3f6edc3f0a/artifact/c0dc568e30084991/p/e4721abe9dee9cab30169968c3a73480


Document the limitations of Audit Logging for policy rules

Starting with Antrea v1.13, logging is best-effort, which means that logging cannot typically be used for compliance purposes.
For older Antrea versions, traffic for which logging was enabled would be dropped after a certain rate was reached, creating issues for production workloads.

/skip-all


Doesn't work with raw_id_fields

Please provide:

An explanation of what is happening
An explanation of what you expected
Any debugging/troubleshooting information, like a traceback or any steps you have taken to try to resolve the problem on your own
The relevant models.py, forms.py, admin.py and any other relevant files that that are problematic for you.

I took a look at the Django documentation for raw_id_fields, and I don't understand how you think this should work, and you haven't given me any more information after prompting, so I'm closing this. Feel free to reopen with more information.


Specify DAX Query Time

It's frustrating to use the slicer sometimes to select Dax Query Timen especially when I want to specify by the millisecond
I want to write the exact time (Example: 1.13 seconds)
Keep the slicer but also add a box to write the number in seconds and milliseconds that I want to specify
Thank you so much for making this app, it's game-changing and it has been really helpful!!!

The latest version (1.6.1) allows you to specify up to the tenth of a second within the trackbar (i.e. DAX queries which took 2.1 seconds or more). Is there truly a need for more precision than this?


[AutoPR notificationhubs/resource-manager] typo: notificationhubs/resource-manager/Microsoft.NotificationHubs

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/4722

Build output:

This PR has been merged into https://github.com/Azure/azure-sdk-for-java/pull/2729


using --includeDeclarations  when there are no .d.ts declaration files causes typedoc to silently hang forever

The problem
Yes I have looked at #913 and the comment by @aciccarello also. But I still find that --includeDeclarations option would make typedoc hang like forever without any warnings if there are no d.ts declaration files.
Context
I was planning to add d.ts files later on but I did not want to edit typedoc config later too, so I just added includeDeclarations option to the config (typedoc.js). But the problem was typedoc would hang somewhere in its process, probably looking for .d.ts files.
I tested this problem each time with the option specified on cli (by specifying --includeDeclarations) and in typedoc.js in rootdir.
Reproducing the problem
You can test directly from c9.io here:
(on c9 cli)
typedoc --out docs test.ts

or git clone this repo to test it out:
git clone https://github.com/9oelM/typedoc-test.git
cd typedoc-test
npm install
typedoc --out docs test.ts

this is going to show something like
Using TypeScript 3.1.6 from /home/ubuntu/workspace/node_modules/typedoc/node_modules/typescript/lib

and hang forever.
This is the content of the config file used:
typedoc.js
module.exports = {
  module: "commonjs",
  mode: "modules",
  target: "es5",
  out: "docs/",
  name: "justTesting",
  exclude: ["node_modules/**", "**/"],
  includeDeclarations: true,
}

I waited for several minutes but it won't work. It is even clearer that includeDeclarations option is the problem when you test it out without it (just exclude the last line from the above config):
$ typedoc --out docs test.ts

Using TypeScript 3.1.6 from /home/ubuntu/workspace/node_modules/typedoc/node_modules/typescript/lib
Rendering [========================================] 100%

Documentation generated at /home/ubuntu/workspace/docs

It would work without includeDeclarations specified.
So..
I just hope there would be some kind of safety measures in case users put includeDeclarations option in advance of creating d.ts files, or by any other ignorance because it is typedoc is going to hang forever and not going to tell you the problem!

Thanks for the detailed report, I've merged this with #913


getController method changed to getControllerInstance

Dingo\Api\Routing\Route::getController() method no longer exists and was renamed to getControllerInstance().
Dingo\Api\Console\Command\Docs::getControllers() is making use of the old name, needs to be updated to getControllerInstance()

Just hit this too, thanks for posting.

Fix pushed, cheers.


Port audit_desc audit rules to a cop

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[x] Have you successfully run brew tests with your changes locally?

This is in continuation of the PR #1873 and issue #569
This PR ports audit_desc audit rules into a rubocop custom cop. The rules are

Check if desc exists in a Formula
Check if desc size is > 80
Check if desc begins with an article
Check for proper usage of command-line in desc
Check if desc contains the formula name

The previous tests for the ported audit rules in audit_spec.rb have been removed. I am working on adding tests for the Custom Cops. I have the basic code, but I'm currently trying to figure out where to place the code for the tests to get executed.
Code for some tests:
describe RuboCop::Cop::Homebrew::FormulaDesc do
  subject(:cop) {described_class.new}

  context "When auditing formula desc" do
    it "when there is no desc" do
      let(:source) do
        <<-EOS.strip_indent
        class foo < Formula
          url "http://example.com/foo-1.0.tgz"
          desc ""
        end
        EOS
      end
      let(:expected_offenses) do
        [{
          message: "Formula should have a desc (Description).",
          severity: :convention,
          line: 1,
          column: 0,
          source: source
        }]
      end
      inspect_source(cop, source)
      expected_offenses.zip(cop.offenses).each do |expected, actual|
        expect_offense(expected, actual)
      end
    end

    it "when desc is too long" do
      let(:source) do
        <<-EOS.strip_indent
        class foo < Formula
          url "http://example.com/foo-1.0.tgz"
          desc "#{"bar"*30}"
        end
        EOS
      end
      let(:expected_offenses) do
        [{
          message: "Description is too long. ",
          severity: :convention,
          line: 1,
          column: 0,
          source: source
        }]
      end
      insepect_source(cop, source)
      expected_offenses.zip(cop.offenses).each do |expected, actual|
        expect_offense(expected, actual)
      end
    end

    def expect_offense(expected, actual)
      expect(actual.message).to eq(expected[:message])
      expect(actual.severity).to eq(expected[:severity])
      expect(actual.line).to eq(expected[:line])
      expect(actual.column).to eq(expected[:column])
      expect(actual.location.source).to eq(expected[:source])
    end
  end
end

brew style without any args runs style checks on whole Homebrew Library which is why the build is failing as custom cops are running on files outside Library/Taps/homebrew/.
Instead we want Custom Cops to be run only on formulae files. Can I modify style.rb to achieve this? (To exclude custom cop checks on core code and include them on formulae)

rubocop-cask is checking checking if the file is a Cask,
https://github.com/caskroom/rubocop-cask/blob/eca041681dc554996a926173ec1d0c378efaecd3/lib/rubocop/cask/extend/node.rb#L14-L20
and skips it if it isn't.
https://github.com/caskroom/rubocop-cask/blob/cd020379b3a38d4dc20a23eb6c596b5f39414565/lib/rubocop/cop/cask/mixin/cask_help.rb#L9

@reitermarkus I too added a check for the same https://github.com/Homebrew/brew/pull/2242/commits/19f693d25ba514a6b604128474763e83de55b548#diff-166fafc45d1d65494760d2fb578d6e46R8
But the problem is there are formulae defined outside Library/Taps/homebrew/ https://travis-ci.org/Homebrew/brew/jobs/206899446#L266-L279

I too added a check for the same

Ok, so there is a on_class method, do you know if there is a on_file method? This way we could simply skip all files under Library/Homebrew.

This looks like it could do what we want: https://github.com/bbatsov/rubocop/blob/9754e06f468631e9145b72f30e23e4cc0883bdb8/lib/rubocop/cop/style/file_name.rb#L20-L27

@reitermarkus Yes, just found that too. Thanks! :)

This is looking pretty good to merge for me now. @reitermarkus, any thoughts?

Great work again @GauthamGoli! My suggestion for the next rules would be https://github.com/Homebrew/brew/blob/master/Library/Homebrew/dev-cmd/audit.rb#L251-L309; these are the ones I personally tend to wish to have in my editor the most.


Added Doc to replace content at https://support.microsoft.com/en-us/help/4021566/windows-10-send-feedback-to-microsoft-with-feedback-hub-app

The index page (speech bubble with a star) links to https://support.microsoft.com/en-us/help/4021566/windows-10-send-feedback-to-microsoft-with-feedback-hub-app.
I've created content to replace this as the content on the support site does not contain a reference to the new problem impact scale option in Feedback Hub.
Also fixed a formatting error on wip/at-home-whats-new-wip-at-home-1809 where the title "Cloud Clipboard" was not formatted correctly.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: rayd250592 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Hi @rayd250592 , I'm looking this pr over now. I think it might make more sense to update the support page so we don't have two topics that diverge slightly. That can be confusing to customers. I'll get back to you as soon as I can.
Dawn

We took your page. Thanks for contributing to the docs!!


Metric learning without labels

Is there a way to finetune dinov2 using OML with an unlabelled dataset. Does OML support that? Also, how will the STIR finetuning take place. My aim is to finetune dinov2 on my dataset, and also finetune the STIR. But all of this with an unlabelled dataset

@AlekseySh @leoromanovich @b0nce @DaloroAT @dapladoc
I want this finetuned model to generate embeddings. Not for classification

Hey, @JAVARSHA
We don't support unlabelled training in OML. I recommend you to find other solutions. You keywords are: Self Supervised Learning - SSL, Unsupervised Learning, Contrastive Learning. Check out this library: https://github.com/lightly-ai/lightly


Dev adhybridhealthservice microsoft.ad hybrid health service 2014 01 01 binilkk

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[X] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

@binilkk hello any update?

We had to make some changes in the product to align to the feedback. I'll have an update published next week.

@jhendrixMSFT , @KrisBash could you please review the PR again?

@binilkk - doesnt look like Kris's comments have been addressed? What is "riskyIp"? Is it a resource type? or is it an instance of "reports" resource type. Feel free to ping me to close on this - alias gauravbh

Signing off from ARM side. Service team will address the feedback around making the name segment enum will be addressed in a future PR.

Signing off from ARM side. Service team will address the feedback around making the name segment enum will be addressed in a future PR.

Thank you, Gaurav.

Waiting for deployment to finish before merging.

Deployment complete, @KrisBash can you please sign off?

Thank you!
Get Outlook for iOShttps://aka.ms/o0ukef

From: Joel Hendrix<EMAIL_ADDRESS>Sent: Saturday, March 2, 2019 9:36:55 AM
To: Azure/azure-rest-api-specs
Cc: Binil Kurian; Mention
Subject: Re: [Azure/azure-rest-api-specs] Dev adhybridhealthservice microsoft.ad hybrid health service 2014 01 01 binilkk (#5042)
Merged #5042https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-rest-api-specs%2Fpull%2F5042&data=02|01|binilk%40microsoft.com|bc50599069314e81cfb208d69f35aa04|72f988bf86f141af91ab2d7cd011db47|1|0|636871450170555592&sdata=ZtKNKSXabzbk%2FxKht5LUOaOR5v8fbn%2Fq6%2FCkzbO%2FjPM%3D&reserved=0 into master.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-rest-api-specs%2Fpull%2F5042%23event-2175869195&data=02|01|binilk%40microsoft.com|bc50599069314e81cfb208d69f35aa04|72f988bf86f141af91ab2d7cd011db47|1|0|636871450170555592&sdata=OKqC7%2BCltN95UmHKETHdqrIzl27auR%2B5Iwy4qzcEal8%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FArqItsyGxnJAW83EskUqtIXI58mjI_Bvks5vSra3gaJpZM4aCCaM&data=02|01|binilk%40microsoft.com|bc50599069314e81cfb208d69f35aa04|72f988bf86f141af91ab2d7cd011db47|1|0|636871450170565604&sdata=ByVD2OYb7QxX3iJcNyX7%2BAGQrkNjyRI7voleNrcUzlc%3D&reserved=0.


bump gophercloud/* and majewsky/schwift to v2

This requires carrying a context.Context argument to all Gophercloud methods that make HTTP requests. Also, some import paths got moved around.

coverage: 67.067% (+0.02%) from 67.046%
when pulling 96e578da2c99d884f235b592b3825c395b7d59be on gophercloud2
into efad024b4adb4ccee69f8f9cf203abd96f45eb14 on main.


"Added really cool feature"

Now the game can change the sprite, animation speed, sprite's size during running;

looks good


remove redefined parent property

Property $controller already defined in parent class \yii\base\Application, both console\Application and web\Application already inherit it.
No need to redefine it in both classes.

Thanks!

this is intentional to have better IDE type completing for web and console Controller...

Fixed via phpdoc.

Which IDE?
There are many other property inherited, why only need to redefine controller?
Can you show me some doc to read?

@matthewyang based on which app it is, IDE returned different Controller classes (either console or web).

@samdark , I see. So $controller in \yii\console\Application is actually the object of class \yii\console\Controller, right?

yes


use undefined as the default password

Problem
currently this plugin is not useable without a password, because for "falsy" passwords ("", 0, false, null and undefined) it always defaults to null
let PASSWORD = null;
[...]
PASSWORD = settings.ep_redis_publisher.password || PASSWORD;

leading to following error:
[2020-05-31 09:30:48.940] [ERROR] console - ReplyError: ERR AUTH <password> called without any password configured for the default user. Are you sure your configuration is correct?
    at parseError (/opt/etherpad-lite/node_modules/redis-parser/lib/parser.js:193:12)
    at parseType (/opt/etherpad-lite/node_modules/redis-parser/lib/parser.js:303:14)

because node-redis only considers undefined as no password.
https://github.com/NodeRedis/node-redis/blob/0041e3e53d5292b13d96ce076653c5b91b314fda/index.js#L240
 if (this.auth_pass !== undefined) {

Solution
with this pull request it will use undefined as the fallback value instead of null

Hi @alangecker , thanks for pointing that out.
Went to check this but I wasn't getting that error message:
[WARN] console - node_redis: Warning: Redis server does not require a password, but a password was supplied.
Is there any changes in you bigbluebutton setup?

I've just checked the error messages:
redis 3 to 5
<IP_ADDRESS>:6379> AUTH ""
(error) ERR Client sent AUTH, but no password is set

redis 6
<IP_ADDRESS>:6379> AUTH ""
(error) ERR AUTH <password> called without any password configured for the default user. Are you sure your configuration is correct?

I've upgraded my redis to 6.0 why I get the last message, but node-redis only ignores errors with the string  'no password is set' (individualCommands.js) and exits the process with anything else.
So this definitely also a problem within node-redis, but I guess this project can also avoid sending an auth request if it's not necessary :)


loop change "array_reverse" default php function

I think the point of writing an algorithm is defeated in some way by using a built-in function.  And I don't think this adds unique value.  Thank you for your pull request, but I'm going to decline this change.  Please refer to the Contribution guidelines


custom profile fields: Minor cleanups part 2.

Followups from #10303.

[ ] The date picker could use a little better styling; I assume we're just using a library for this, and maybe there are nicer libraries? The work for this will be useful elsewhere (reminder bot, searching by date, picking a time for a meeting, ..).
[x] "Date joined" and "Last seen" should be in the top section, where the avatar and name are, rather than in the custom profile fields section.
[x] Timezone should be another default piece of info shown

@rishig Should the last issue be split out into its own issue? Since it's not really a custom profile field issue but the date picker UI element in general

yeah, that's fair. Do you want to open it or should I?


j2ba : a jackd client that directly writes to bluealsa.

Hello,
Thank you for this project. For users like me that don't use pulseaudio it is realy usefull. Finally i have music from my PC on my BT headphone. Very happy :)
It works ok here with jackd using the alsa_out utility to stream to bluealsa_pcm.
Only point i have atm. is that i have to reconnect once or twice at startup to get an
a2dp connection.
I've looked at the code of the bluealsa_pcm plugin and i think that it won't be to
hard to write a jackd client that outputs directly to bluealsa, instead of using the alsa plugin.  I searched for such a project but couldn't find anything . (I'm in no way an experienced programmer but i'm gonna give it a try anyway ).
I hope you can give me some info.
To start i'd like to know what audio-format(s) and samplerate(s) are suppoted by bluealsa ? I only see signed short mentioned in the plugin code but there might be more than that ?

With that info i should be able to write some test code.
Thanks in advance.
Ed.

Hi.
what would the advantage be?
got alsa_out stable here. but only with "-p 10000".
I doubt there can be anything (much) done about the lag

This question has remained unanswered for quite some time now. If the OP is still interested, I suggest to checkout the bluealsa manual pages and wiki pages for info on supported codecs and audio formats etc. The bluealsa plugin works quite well with jack, there is also a wiki page here about that. Closing.


need some suggestions on searching my homepage by Google

Hi,
I have created my homepage based on your wonderful theme. I can do local development and have pushed it to my Github and thus I can see my homepage via https://jjiantong.github.io/. Thanks for your helpful instruction.
However, I had trouble when I searched my name (Jiantong Jiang) on Google. I expected my homepage link to appear on the search results but it didn't. Could you please give me some suggestions on how my homepage can occur when I search my name? Maybe, I guess, I need to set something somewhere in the configure file?
Thanks,
Jiantong

Hello Jiantong.
I think the best thing to do is to host your homepage and get a domain name for it then do some SEO to make it rank when you search for your name.
I hope this helps.


Add files via upload

@Jerem7 file/lib/domains/email/ahe.txt: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@Jerem7 Thank you for your patience while waiting for a review and processing your request. Pull request merged.
Please start requesting the licenses in about an hour, to let the changes to propagate through our system.


add * { box-sizing: border-box; }

There is a couple of uses of border-box in classes already, some are partial and only use non-prefixed version.
According to Paul Irish and the comments in his blog post this should work in all browsers foundation is supporting.
I'm suggesting to add * { -moz-box-sizing: border-box; -webkit-box-sizing: border-box; box-sizing: border-box; }  declaration to the globals.css and remove it in cases where you don't want it.

We've actually been looking into this (thanks to that same Paul Irish article) but we want to get to this in 3.0 rather than 2.x. For IE7 support it requires a JS polyfill so there's a good bit of testing we'll need to do.


Occasional failure of ChantProofreadViewTest

@jacobdgm editing here: We found an intermittently failing test in ChantProofreadViewTest, which we should try to recreate and then fix.
Here's a bit of traceback:
======================================================================
ERROR: test_view_url_and_templates (main_app.tests.test_views.ChantProofreadViewTest)
----------------------------------------------------------------------

Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/urllib3/connection.py", line 169, in _new_conn
    conn = connection.create_connection(
  File "/usr/local/lib/python3.9/site-packages/urllib3/util/connection.py", line 96, in create_connection
    raise err
  File "/usr/local/lib/python3.9/site-packages/urllib3/util/connection.py", line 86, in create_connection
    sock.connect(sa)
ConnectionRefusedError: [Errno 111] Connection refused

With, #1047, we no longer have a Chant Proofread View, and this test no longer exists. I think we're good to close this one!


Admin User should be able to set Attachment file types that Users can upload.

This is handled in the FileManagerInterface. The LocalFileManager implementation handles the allowed file types in the constructor:
public function __construct(
        $destinationPath,
        $uriPrefix = null,
        $allowedImageTypes = [
            IMAGETYPE_JPEG,
            IMAGETYPE_PNG
        ]
    )


give warn and exit with 0 if the script doesn't exist

Hello! Thank for your nice module.
This is my actual scenario;
I want to run prestart script just for development only and I don't want to specify prestart:production or prestart:stage or something with empty script string. but for now, per-env tries the script with unspecified env and it makes missing script error.
So I think it would be better that it doesn't try the script of unspecified env and just give warning.
The warning sentence in my code is just suggestion. I have no special idea for now.

Plus, empty script string also causes missing script error. I just checked it.

@ericclemmons
🔔 Is anyone there?

Yep, just been busy with work and family.
On Wed, Jan 2, 2019, 10:40 PM Jiwon Choi<EMAIL_ADDRESS>wrote:

@ericclemmons https://github.com/ericclemmons
🔔 Is anyone there?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ericclemmons/per-env/pull/17#issuecomment-451065660,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAA7ThKh51yIz8mW6GPSPY7ZwjHg7_ypks5u_aXCgaJpZM4ZX9d7
.

close this issue because of no activity for a long time.


Remove ellipsis from and group deployment context menu

Drop the ellipsis to match the other extensions and group them together.

@fiveisprime @EricJizbaMSFT @nturinski I think it's the other way around.  I think the other "Deploy" menu items should have the ellipses as well.  See https://vsuiguide.azurewebsites.net/topics/14 Evaluation Tools/14.01 Craftsmanship checklist.html:

Verify that buttons, commands, and command links that launch UI to capture user input, end in an ellipsis […]. Examples:
An [Advanced…] button on a dialog.
The command options under the Tools menu Tools > Options should not get an ellipsis, because
launching the dialog itself is the intent of the command.

We require user input to determine whether to deploy to.

I just want them to be the same. Add it to the others or remove it here.

I agree it should have an ellipses


Initial Database build performance

Describe the bug
The time taken to build the CVE database from scratch has increased dramatically.
Version of dependency-check used
5.3.2, gradle plugin.
Log file
This is an example of when it was fast: https://gist.github.com/mikehalmamoj/92726c89e5c1f0f4640ff069aa4fc5be
And an example of slow: https://gist.github.com/mikehalmamoj/067cf29c629dc0b1b084f46e9424281a
Note that the key difference is End database maintenance (52108 ms) compared to End database maintenance (1624822 ms)
To Reproduce
Run the gradle task dependencyCheckUpdate in a clean environment.
Additional context
Having watched the builds, the step that takes forever is Removed the CPE ecosystem on 5515 NVD records.
I traced that back to the UPDATE_ECOSYSTEM2 SQL statement: https://github.com/jeremylong/DependencyCheck/blob/master/core/src/main/resources/data/dbStatements.properties#L47
I wonder if some recent CVE additions have exposed a flaw in the SQL?

I am facing the same issue. Taking too long to build the database for the first time (in fact, it's still going on as I write this 😄)

Version of dependency-check used
CLI 5.3.2
I am trying to setup a centralized MySQL database, initialized using the corresponding script (on MacOS 10.15, if that helps). The initialization script worked perfectly & threw no errors.
Here is the exact command I ran to update the database
dependency-check.sh --updateonly --connectionString jdbc:mysql://MY_HOST:3306/dependencycheck --dbDriverName com.mysql.jdbc.Driver --dbPassword MY_DB_PWD --dbUser MY_DB_USER
Let me know if I can provide additional info to help figure this out!

Sorry to hear about your woes @aravindparappil46 but I don't think we have the same issue.  My problem is only with the database cleanup, not the building of the database which hasn't slowed down for me.
For example I see Processing Complete for NVD CVE - 2017  (109127 ms) whereas you see Processing Complete for NVD CVE - 2017  (47856652 ms).  That's more than 400 times slower than me.  I think you need to raise a separate ticket.

@aravindparappil46 - yes building the centralized database does currently take a long time. No one has had time to investigate this problem.
@mikehalmamoj this issue has already been reported and we have already adjusted the UPDATE_ECOSYSTEM2 statement to be more performant. See #2610

Thanks Jeremy - any timescales on the next release?


Fix list widget reordering

Pull Request Description
Fixes, but does not close #10651 - I will try adding some e2e tests for dragging (again).
The problem is, that removing/hiding element in the same frame as dragging starts, the drag is stopped immediately. It is presented here: https://jsfiddle.net/1g34jhe9/ (change HIDE_IN_NEXT_FRAME to compare).
In our case, the element removal was technically postponed to nextTick, but that is not enough after some version bump, as this tick occures after DOM recalculating, not necessarily after rendering - changed it to setTimeout with proper comment.
Important Notes

Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[x] The documentation has been updated, if necessary.
[ ] Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
[x] All code follows the
Scala,
Java,
TypeScript,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.
[ ] Unit tests have been written where possible.

A test for this behavior would be really helpful, as we rarely interact with it during normal development.