Visual studio 2015 high CPU and memory usage

Hello.
I'm working on a project on Angular2 with Typescript. Now I have around 80 typescript files. The problem is when I editing files CPU usage goes to 25% and memory usage starts to grow. CPU usage stays on this level almost permanently during coding and memory usage is growing up to 3GB until it crashes with OutOfMemoryException. It can happen after every 10-20 minutes.
It started when I was using Typescript 1.7. Upgrade to 1.8 didn't solved this problem. I'm not using any extension like ReSharper.
Is there a way to fix it?

can you share a sample project where we can diagnose the issue?

I'm afraid I can't share my code. I can provide any other information you need to fix this problem.

We would be willing to sign an NDA to get access to your project. if that is not possible, can you share a sample project that exhibits similar behavior?

Ok, how can I send my project directly to you?

I'm having the same problem. VS starts out around 1GB of memory usage.
As soon as I start typing in a .ts file my memory usage skyrockets to around 2.5GB and after that VS will just stop working.
I'm also working on a Angular2 project, and the problem started this morning after upgrading to Angular2 beta 7, and this seems to be causing the issue.
After returning to Angular2 beta 3 the problem stopped and my memory usage is back to normal.

Apparently returning to Angular2 beta 3 didn't solve the problem.
It was looking good at first, but now I'm back at 2.8GB memory usage...

Also tried to return to Angular beta 2 and it didn't help at all. Same problems with memory and CPU

if project is not very big you can mail it to vladima at microsoft dot com. Otherwise you can use any existing file sharing solution (OneDrive, Dropbox etc..) and then send a link to the email above.

I've sent project to your e-mail

Were you able to reproduce this issue? Maybe you need any other information?

yes, thanks for the repro.

Exactly the same problem here: Angular2, TypeScript 1.8. Opening a .ts files memory start growing up to 2Gb and CPU as well. All the Visual Studio functionality start to be slow (especially IntelliSense). then, after 15-20 min Visual Studio crashes.
Any news on this problem?

Having the same problem as of 4/11.  Angular2 beta 12 with typescript 1.8.2.  Any update on this issue?

We tracked this issue to RxJS 5.0.0-beta.5, but it should be resolved with 5.0.0-beta.6. what version of RxJS are you using?

I'm using RxJS 5.0.0-beta.2 I upgrade and let you know.

I've tried with RxJS beta.6. Memory usage looks ok, but CPU still spikes on Intellisense.

the spike is not intellisense, it is more of edit. on every edit we need to do a full check on all files, open and closed. so depending on the size of the project, you can see some CPU utilization. this should subside at some point though, if not, then this is an issue.

We are seeing the same thing here.   Seemed fine for the last few weeks and (without any change we know of) our VS 2015 is killing memory.  Suspiciously we are using the same basic projects you are above (ng2, RxJS, etc).   We are currently leaning towards it being TS related either due to compilation behind the scenes or intellisense (thus looking for answers here).   My VS taps out at about 2.7gig of memory after opening the project and then crashes.

is there a chance you can share the project, or a set of repro steps to get to the same behavior?

also what version of RxJs do you use?

Reading through the above we reverted back to RxJs beta 2.  Testing now but it appears to have solved our issue (we are on Ng2.beta.15).

I thought we had it fixed yesterday but today it's back (rxJS Beta 2 and NG2.beta.15 just as stated in the last post).   Memory skyrockets to about 2.5gig per developer upon opening the solution.   Intellisense for Typescript is almost non-existent (extremely slow).   Has been working great for months and no issues until the last few days on our end.

I ended up deleting my local repo and cloning local and haven't had the problem again... yet.  Seems like a short term solution.  One thing of note is that I do not currently have Intellisense for rxjs (.map as an example).  When I was having this problem the Intellisense for jxjs was working.  I think there may be a correlation.

Any changes we can have a look at your projects to figure out the root cause of such behavior? We can sign NDA if necessary?

Unfortunately not.  It’s a client project and they have been less than willing to share it for similar purposes in the past.   If it was something I had control over I’d gladly sign an NDA and share.
-Kevin
From: Vladimir Matveev<EMAIL_ADDRESS>Sent: Friday, April 22, 2016 11:00 AM
To: Microsoft/TypeScript<EMAIL_ADDRESS>Cc: Kevin Grossnicklaus<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [Microsoft/TypeScript] Visual studio 2015 high CPU and memory usage (#7208)
Any changes we can have a look at your projects to figure out the root cause of such behavior? We can sign NDA if necessary?
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHubhttps://github.com/Microsoft/TypeScript/issues/7208#issuecomment-213490590

is there a tsconfig.json in the project? can you share that file?

We are using this tsconfig.json file:
{
"compileOnSave": true,
"compilerOptions": {
"noImplicitAny": false,
"noEmitOnError": false,
"removeComments": false,
"sourceMap": true,
"target": "es5",
"experimentalDecorators": true,
"emitDecoratorMetadata": true,
"module": "commonjs"
},
"exclude": [
"node_modules",
"typings/main",
"typings/main.d.ts"
]
}
From: Mohamed Hegazy<EMAIL_ADDRESS>Sent: Friday, April 22, 2016 6:52 PM
To: Microsoft/TypeScript<EMAIL_ADDRESS>Cc: Kevin Grossnicklaus<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [Microsoft/TypeScript] Visual studio 2015 high CPU and memory usage (#7208)
is there a tsconfig.json in the project? can you share that file?
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHubhttps://github.com/Microsoft/TypeScript/issues/7208#issuecomment-213628818

is this the same file you use for building as well? what do you see when you run the compiler with tsc --p <your tsconfig.json> --diagnostics? does disabling compileOnSave make any difference?

I am experiencing the same issue with Angular 1.5.3 and Typescript 1.8 - not using RxJS.
VS2015 is on 2.3-2.5 GB RAM while the project is loaded. As soon as I close the project it goes back to normal.

might be related: #8152

@Aides359 can you share your project?

It seems as if in my case nodejstools#44 and nodejstools#88 seemed to be related in my case.
I have now added node_modules to my analysis ignore (<AnalysisIgnoredDirectories>node_modules</AnalysisIgnoredDirectories>) and VS is back to 1 GB RAM
@mhegazy sharing the project would be quite a fight with my project manager and since it is working now I'd like to avoid that. In any case, I will get back to you if it the problems reoccur.

@Aides359 thanks for getting back to us on this. please let us know if you run into other issues.

Just to bring it to notice, VS is still working fine.
Apparently for me the problem was really caused by nodejstools including my libraries in its analysis.

closing since it was fixed on RxJS side.


Split Horizon EDS multi-cloud single mesh support

Q
A

Bug fix?
no

New feature?
yes

API breaks?
no

Deprecations?
no

Related tickets
fixes #33

License
Apache 2.0

What's in this PR?
It implements Split Horizon EDS based multi-cloud single mesh support.
Checklist

[x] Implementation tested
[x] User guide and development docs updated (if needed)

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 3 committers have signed the CLA.:white_check_mark: Laci21:white_check_mark: waynz0r:x: matyixYou have signed the CLA already but the status is still pending? Let us recheck it.


[elasticsearch] master service template

This pr adds masterService template which automatically set master service name.

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

jenkins test this please


Add Azure DevOps Task to install NLU.DevOps extension

Add a task called NLUInstallExtension that will install a .NET Core CLI tool package in the correct location for integration with NLU.DevOps.

Not sure this is needed as it's reasonably trivial to just use:
- task: DotNetCoreCLI@2
  displayName: Install dotnet-nlu-mock
  inputs:
    command: custom
    custom: tool
    arguments: install dotnet-nlu-mock --add-source $(Build.ArtifactStagingDirectory) --tool-path $(Agent.TempDirectory)/.dotnet

Re-opening, given this is something we may want.

Closing again, now that we have toolPath input, it's trivial to install the tool to the correct location.


ProjectRootElement.RawXml Never contains updated comments

As the title says, updated comments aren't reflected in the output of ProjectRootElement.RawXml. This causes https://github.com/dotnet/roslyn-project-system/issues/743. If there's a better api for getting what XML would be actually written out when saved, I'd be more than happy to use it, as RawXml seems to not be intended for that purpose.

I think what you want to use is ProjectRootElement.Save(TextWriter).  The whitespace preservation tests, for example, call Project.Save(TextWriter), which just forwards the call to the method on ProjectRootElement.

RawXml actually calls Save under the hood with a StringWriter. I tried it anyway just to be sure, and it has the same behavior.

I suspect the root of the problem here is that the MSBuild side of the object tree doesn't know anything about comments, so if we walk it to update we won't see comment nodes.

Strange.  The whitespace preservation tests use the same APIs to save to validate that not only are comments preserved, but that the whitespace around them is preserved (if preserveWhitespace is enabled).  I suspect the difference lies in how the project is loaded.

Could also be a post-clone problem.

@dsplaisted , RawXml, and by extension Save, are also not showing the whitespace.

@333fred / @dsplaisted
Can you please write out a minimal repro that, using MSBuild's APIs shows this issue? I'm asking because the MSBuild APIs do not expose the xml comments, so I am curious how you get into this state. I would expect the editor to tear down and recreate the Project / ProjectRootElement objects after the contents got changed. Otherwise the underlying XmlElement object model does not get updated, hence Save does not reflect the updated comments.

This is an issue with the way project live reload works. A new ProjectRootElement is created from the changed project on disk and then deep copied into the existing project. This is not an ideal solution as it is expensive and DeepCopyFrom doesn't seem to preserve a lot of the formatting (including comments). Also looks like it doesn't even update RawXml.
As @cdmihai has suggested in earlier discussions, we should swap the ProjectRootElement out with the newly loaded one. Unfortunately, the only way I have found to "swap" out the elements would be via ProjectCollection.UnloadProject, which means unloading all Project objects (one per CPS ConfiguredProject) that reference the ProjectRootElement. This is not feasible for CPS. Instead I think ProjectCollection should offer an API to swap out ProjectRootElement. It can then update all necessary Project.Xml to point to the new xml. This is fine to perform in CPS as it will happen within a write lock, so no one else should be holding references to these objects at the time. Another option is to have ProjectRootElement be able to re-build itself from new xml.


Create CTI test for reverse engineering

To catch cases like #15414

Notes from triage: We should also review the existing tests to make sure they are using as much product code as possible and add comments letting you know that if you change certain parts of the test code, you're doing it wrong and should be changing product code instead.

Also add case here for using named connection string and user-secrets. See #17375


Add view to query replicator documents in terminal states

Currently there are two terminal states - completed and failed.
These states are reached when scheduler decides it will no longer pay
attention to the document. Either because it represents a completed replication,
or an unrecoverable failure occured, such as specifying an invalid replication
parameter. In case of failure, user is expected to fix the cause of the failure
and then recreate the document.
Since scheduler forgets about these documents, the only way to inspect is to
look in the database. So create a view to help users perfom that query.
The view is  _replicator/_design/_replicator/_view/terminal_states
Example of usage:
http http://adm:pass@localhost:15984/_replicator/_design/_replicator/_view/terminal_states
{
    "offset": 0,
    "rows": [
        {
            "id": "rdyno_0001_0001",
            "key": "completed",
            "value": null
        },
        {
            "id": "rdyno_0001_0002",
            "key": "completed",
            "value": null
        },
        {
            "id": "badrep",
            "key": "failed",
            "value": null
        }
    ],
    "total_rows": 3
}

+1


HDP 2.6.3 Spark 2.2.0

Hello,
It would be possible to upload a version built for Spark 2.2.0 (that is already available on HDP 2.6.3) to the Hortonworks public repo?
If it's not possible any deadline on this matter?
Thanks!

Hi @Neuw84 We have uploaded a version for Spark 2.2.0, HDP 2.6.3, you may want to use it (http://repo.hortonworks.com/content/repositories/releases/com/hortonworks/shc/shc/)
--packages com.hortonworks.shc:shc-core:<IP_ADDRESS>.6.3.0-235
--repositories http://nexus-private.hortonworks.com/nexus/content/groups/public
spark-submit  --class your.application.class --master yarn-cluster --packages com.hortonworks.shc:shc-core:<IP_ADDRESS>.6.3.0-235 --num-executors 4 --driver-memory 512m --executor-memory 512m --executor-cores 1 --files /usr/hdp/current/hbase-client/conf/hbase-site.xml --repositories http://nexus-private.hortonworks.com/nexus/content/groups/public /your/application/jar

Lets know if you hit any issues.
We are planning to publish new releases around Jan 2018.

New version for HDP 2.3.x and HDP 2.4.x http://nexus-private.hortonworks.com/nexus/content/repositories/IN-QA/com/hortonworks/shc/shc/

Thanks!
Will test it and let you know if I hit any issues.


1px border on alpha export

^^ cannot remove the 1px border created by the key color.
Any idea if this is possible? Thanks.

did you fine a solution to this.

GIF as a format only supports one-bit transparency, but you can get rid of this color artifact by passing an image where every pixel is already fully opaque or transparent.
You can apply a threshold, like so, using canvas:
var canvas = document.createElement("canvas");
var ctx = canvas.getContext("2d");
canvas.width = image.naturalWidth;
canvas.height = image.naturalHeight;
ctx.drawImage(image, 0, 0);

var imageData = ctx.getImageData(0, 0, canvas.width, canvas.height);
var threshold = 128;
for (var i=0; i<imageData.data.length; i+=4) {
	imageData.data[i+3] = 255 * (imageData.data[i+3] > threshold);
}
ctx.putImageData(imageData, 0, 0);

// and add the canvas as the frame instead of the image
gif.addFrame(canvas);

You can play around with the threshold between 0 and 255 to find what works best.

@1j01 Ah yes, 255 * true = 255 and 255 * false = 0
I love js :)

GIF as a format only supports one-bit transparency, but you can get rid of this color artifact by passing an image where every pixel is already fully opaque or transparent.
You can apply a threshold, like so, using canvas:
var canvas = document.createElement("canvas");
var ctx = canvas.getContext("2d");
canvas.width = image.naturalWidth;
canvas.height = image.naturalHeight;
ctx.drawImage(image, 0, 0);

var imageData = ctx.getImageData(0, 0, canvas.width, canvas.height);
var threshold = 128;
for (var i=0; i<imageData.data.length; i+=4) {
	imageData.data[i+3] = 255 * (imageData.data[i+3] > threshold);
}
ctx.putImageData(imageData, 0, 0);

// and add the canvas as the frame instead of the image
gif.addFrame(canvas);

You can play around with the threshold between 0 and 255 to find what works best.

This works well, but the way gif.js works I think you want to correct the pixel so it's exactly the key color, rather than setting the canvas opacity to 0.
I also managed to get these artifacts fixed on a webgl canvas by first copying to an offscreen canvas. Kind of expensive, but it works.
`var offscreenCanvas = document.createElement('canvas');
var ctx = offscreenCanvas.getContext("2d");
ctx.drawImage(canvas,0,0);
var imageData = ctx.getImageData(0,0, offscreenCanvas.width, offscreenCanvas.height);

console.log(imageData.data[1]);

for (var i=0; i<imageData.data.length; i+=4) {
    const rDistance = Math.abs(alphaCol[0] - imageData.data[i + 0]);
    const gDistance = Math.abs(alphaCol[1] - imageData.data[i + 1]);
    const bDistance = Math.abs(alphaCol[2] - imageData.data[i + 2]);

    if (rDistance < threshold && gDistance < threshold && bDistance < threshold) {
        imageData.data[i + 0] = alphaCol[0];
        imageData.data[i + 1] = alphaCol[1];
        imageData.data[i + 2] = alphaCol[2];
    }
}
ctx.putImageData(imageData, 0, 0);`


Login loop rendering the service useless

Upon logging in, I get the "logging you in... succsess", but it just takes me back to the login page
this is probably not relevent, but this error appears in console
(node:28666) DeprecationWarning: Mongoose: mpromise (mongoose's default promise library) is deprecated, plug in your own promise library instead: http://mongoosejs.com/docs/promises.html express deprecated req.host: Use req.hostname instead node_modules/lodash/lodash.js:4814:28 ^C

+1, also working fine in internet explorer


[WFCORE-6814] Adjusting ContentRepositoryTest to be functional on all JDKs on Fedora 40

https://issues.redhat.com/browse/WFCORE-6814

Thanks @ropalka


Some changes

I’m experimenting with building luaharfbuzz into LuaTeX, here is some of the changes I made.

I believe you have commit rights to this repo. So feel free to push this to master. I am a bit busy nowadays, but I might be able to find some time to do a release to LuaRocks, just to keep things consistent.

OK, I wanted another eyeball. I’m also still changing stuff as I go with my experiment, so will keep this open until things settle down and then merge.

I will look through the code later this week, and leave comments if I have any.

A couple of things:

If you are changing the names of public fields, (like the Drop redundant… commits), do update the luadoc file: https://github.com/ufyTeX/luaharfbuzz/blob/master/src/harfbuzz.luadoc

I haven’t been following Harfbuzz development very closely, but it seems there is a 2.0 version out. Moreover, with the kind of changes you are making it would make sense to bump the major version. So, if/when I release, I would tag the release as luaharfbuzz 2.0. Makes sense?

I did update the luadoc file for any name change (I also added aliases to the Lua file to not break any existing code).
Version 2.0 sounds good.

I’m still working on that though,  I keep adding stuff as I continue exploring the integration with LuaTeX, so I’d hold on on merging and releasing for now since things are still evolving. Once the dust settles and may LuaTeX integration is working and more thoroughly tested, I might go more systematic and try to map all HarfBuzz API (at least what makes sense for the Lua module) and then version 2.0 would even be more appropriate.

Ok sure. I won’t do anything until you commit to master.

I think this PR is stable enough and can be merged if you don’t have any objections. I’ll work on another PR that adds more API in a systemic way (right now I was only adding stuff I have immediate need for).

Go for it!
I will try and find some time to do a release eventually, hopefully by the end of the year.


move away from the private ci registry

move to the dev registry that can then be accessed from multiple clusters

[test]


MongoDB Atlas Search support

Problem
It would be useful if MongoDB Atlas Search could be queried directly via the Realm Flutter/Dart SDK. This will reduce the development time needed i.e. there will be no need to spin up a Search API service that utilises MongoDB Atlas Search under the covers.
Solution
See above
Alternatives
See above
How important is this improvement for you?
Would be a major improvement
Feature would mainly be used with
Atlas Device Sync

This is already captured (although admittedly not in obvious way) by https://github.com/realm/realm-dart/issues/784. Since Altas Search is just a stage in an aggregation pipeline, once we have support for aggregations, that will enable $search along with all the other aggregation stages.
It is our plan to add support for aggregations, along with all the other remote MongoDB operators but we don't have exact timeline yet.
I'm going to close this issue as a duplicate but feel free to continue the discussion in https://github.com/realm/realm-dart/issues/784 if you have further questions/comments.

Thank you @nirinchev, thats very good news to provide a better 'online' search experience without building out a search sevice. Many thanks again.


[main] Upgrade to latest dependencies

Cron -knative-prow-robot
/cc knative-sandbox/channel-wg-leads
/assign knative-sandbox/channel-wg-leads
Produced by: knative-sandbox/knobots/actions/update-deps
Details:
bumping knative.dev/hack b96d65a...11d193b:
  > 11d193b 1.0 Backports (# 99)
bumping knative.dev/eventing 34c08a4...1e4fa35:
  > 1e4fa35 upgrade to latest dependencies (# 5865)
  > 5263765 upgrade to latest dependencies (# 5864)
  > 4de0da0 Manually update dependencies to bring in hack update (# 5857)
bumping knative.dev/eventing-kafka c380a06...bc68a05:
  > bc68a05 upgrade to latest dependencies (# 963)
  > 4850240 Fix wrong KafkaSource test with CE overrides (# 961)
  > 01d891b manual update (# 962)
bumping knative.dev/pkg 5d9d300...96c0204:
  > 96c0204 upgrade to latest dependencies (# 2337)
bumping knative.dev/reconciler-test cff9b53...9839937:
  > 9839937 Backport deps to release-1.0 (# 254)

/test pull-knative-sandbox-eventing-kafka-broker-integration-tests

/lgtm
/approve

/hold


Hamburger icon needs to be visible in mobile view

Is your feature request related to a problem? Please describe.
Hamburger icon needs to be visible in mobile

Describe the solution you'd like
A clear and concise description of what you want to happen.
Additional context
Add any other context or screenshots about the feature request here.

Hello @Priyaaa1 please assign this issue to me under gssoc 24


Up object properties

Refactored motion blur attributes (now read from VRay Properties panel)
Removed render stats (primary viz, shadows) from plugin (now read from Object Properties panel)
Added Kill List attribute
Taking into account 3ds max system unit when a .vrscene is imported
Added help url in readme

You cannot delete parameters from a parameter block or from the parameters enum; this breaks the loading of old scenes saved with previous versions and may crash 3ds Max.


FATA[0005] failed to initialize project with version "2"

Bug Report
What did you do?
operator-sdk init --project-version=2 --domain example.org --license apache2 --owner "The Kubernetes authors"
What did you expect to see?
project scaffolded since has the option
What did you see instead? Under which circumstances?
$ operator-sdk init --project-version=2 --domain example.org --license apache2 --owner "The Kubernetes authors"
Writing scaffold for you to edit...

go fmt ./...
go vet ./...
go build -o bin/manager main.go
Next: define a resource with:
$ operator-sdk create api
Error: failed to initialize project with version "2": error updating Makefile: unsupported plugin key ""
Usage:
  operator-sdk init [flags]

Examples:
  # Scaffold a project using the apache2 license with "The Kubernetes authors" as owners
  operator-sdk init --project-version=2 --domain example.org --license apache2 --owner "The Kubernetes authors"

Flags:
      --domain string            domain for groups (default "my.domain")
      --fetch-deps               ensure dependencies are downloaded (default true)
  -h, --help                     help for init
      --license string           license to use to boilerplate, may be one of 'apache2', 'none' (default "apache2")
      --owner string             owner to add to the copyright
      --plugins strings          Name and optionally version of the plugin to initialize the project with. Available plugins: ("ansible.sdk.operatorframework.io/v1", "go.kubebuilder.io/v2", "helm.sdk.operatorframework.io/v1")
      --project-version string   project version, possible values: ("2", "3-alpha") (default "3-alpha")
      --repo string              name to use for go module (e.g., github.com/user/repo), defaults to the go package of the current working directory.
      --skip-go-version-check    if specified, skip checking the Go version

Global Flags:
      --verbose   Enable verbose logging

FATA[0006] failed to initialize project with version "2": error updating Makefile: unsupported plugin key ""

Environment

operator-sdk version: 4739c1fc5bfb61db28b1bc52b66a63d9493a1ec2
go version: go1.13.12

Since this is an issue in the init command that results in an immediate failure, I think we could consider the fix for this (whatever it is) a non-breaking bug fix.

Closed by #3697

Hi @estroz,
Should we not address it to 0.19.X as well?


M3-1827 Extract and test IP sorting

Coverage increased (+0.01%) to 32.19% when pulling 1ed9f733cd3b7a2fc04a3167f479911b20a17ce7 on Jskobos:M3-1827-ip-sorting into 4e3e699445f506743aa20c5059d0807cce1fa93a on linode:develop.


[otbn] Implement/evaluate modular multiplication for p384

Current implementation for p256 relies on a subroutine implementing modular multiplication including a Barrett reduction.
Also subroutine uses a single addm instruction at the end which is not available for numbers wider than WLEN
We need to port this to a support 384 bit wide numbers.
Split out of #2982

estimate 8

Evaluation:
Currently it is 152 cycles for 384 bit vs. 37 cycles for 256 bit.
This is with using the fully unrolled multiplication.
There is room for optimization here:

For a generic implementation (no specific curve) we would implement Karatsuba for the multiplication kernel
For the p384 specific implementation we would either move to Solinas or use an optimized Barret implementation (using the special form of the pre-computed barrett parameter for p384). This could use Karatsuba on top. Using Solinas should be faster but I am not sure yet how efficiently it can be implemented on OTBN.


test($rootScope): add tests clarifying $watchGroup oldValues behavior

Closes #16024
The first group seem related but have always passed.  The second group fail in v1.6.x as noted in the comments.  IMO these tests (with the notes about 1.6) show how weird the oldValues were before.
Still need to add the BREAKING CHANGE message.

I've updated the commit message and removed the extra comments from the tests.

Also created https://github.com/angular/angular.js/issues/16514 for updating the changelog like we discussed

Is the BC message 100% clear? There must be at least one typo in there! :laughing:

Is the BC message 100% clear?

It is to me.

There must be at least one typo in there!

Well...if you absolutely want to fix a typo, you can fix "will will lways" :grin:

Will fix that. Now I know you read it! 😄

Fixed the typo in the commit message.


Add build_sif target

Initial Draft
The goal is to allow the Makefile to build sif files based on built targets if BUILD_SIF=1 is passed in on make command line

Please check that I accurately resolved your concerns above.


Add example

Depends on https://github.com/bazelbuild/bazel-central-registry/pull/1202

To trigger the problem that https://github.com/lalten/rules_appimage/pull/115 is trying to solve:

cd examples
bazel test //...

This will make examples be the root module, which will mean that the 3.12 version will be picked up, and no wheel for that version will be found.


⚠ SceneTime has degraded performance

In 3388b61, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1904 ms

Resolved: SceneTime performance has improved in aca5ff0 after 13 minutes.


[BUG] Torch datasets not loading

Torch datasets are not loading in 0.14.2 and 0.14.1 it seems. 0.14.0 seems to not have the issue.
import fiftyone as fo

dataset = foz.load_zoo_dataset("cifar10", split="test")

Output:
Downloading split 'test' to '/home/ben/fiftyone/cifar10/test'
Files already downloaded and verified
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-2-a56cbfe97d58> in <module>
----> 1 dataset = foz.load_zoo_dataset("cifar10", split="test")

~/venvs/fnew/lib/python3.6/site-packages/fiftyone/zoo/datasets/__init__.py in load_zoo_dataset(name, split, splits, label_field, dataset_name, dataset_dir, download_if_necessary, drop_existing_dataset, overwrite, cleanup, **kwargs)
    201             overwrite=overwrite,
    202             cleanup=cleanup,
--> 203             **download_kwargs,
    204         )
    205         zoo_dataset = info.get_zoo_dataset()

~/venvs/fnew/lib/python3.6/site-packages/fiftyone/zoo/datasets/__init__.py in download_zoo_dataset(name, split, splits, dataset_dir, overwrite, cleanup, **kwargs)
    119         splits=splits,
    120         overwrite=overwrite,
--> 121         cleanup=cleanup,
    122     )
    123

~/venvs/fnew/lib/python3.6/site-packages/fiftyone/zoo/datasets/__init__.py in download_and_prepare(self, dataset_dir, split, splits, overwrite, cleanup)
   1002                     num_samples,
   1003                     classes,
-> 1004                 ) = self._download_and_prepare(split_dir, scratch_dir, split)
   1005
   1006                 # Add split to ZooDatasetInfo

~/venvs/fnew/lib/python3.6/site-packages/fiftyone/zoo/datasets/torch.py in _download_and_prepare(self, dataset_dir, scratch_dir, split)
    196             download_fcn,
    197             get_class_labels_fcn,
--> 198             sample_parser,
    199         )
    200

~/venvs/fnew/lib/python3.6/site-packages/fiftyone/zoo/datasets/torch.py in _download_and_prepare(dataset_dir, scratch_dir, download_fcn, get_class_labels_fcn, sample_parser)
    534         )
    535     else:
--> 536         raise ValueError("Unsupported label class %s" % label_cls)
    537
    538     # Write the formatted dataset to `dataset_dir`

TypeError: not all arguments converted during string formatting

label_cls is now a tuple in this case and the loging in _download_and_prepare no longer works.

It is still giving the same error, any reason for this?

@yinde0 this bug has been fixed on develop, but there hasn't been a release since 0.14.2 yet, so your current options are to:

Downgrade to 0.14.0, download the dataset, then upgrade to 0.14.2
Use a source install

fiftyone==0.14.3 was released earlier this week and contains the fix for this


purescriptls provides no diagnostics info

nvim --version:

NVIM v0.5.0-dev
Build type: Release
LuaJIT 2.1.0-beta3
Compilation:
Compiled by nixbld

Features: +acl +iconv +tui
See ":help feature-compile"

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "
/nix/store/28wqibhxh7scsk2ai6lhi16y2v0jm7yw-neovim-unwrapped-master/share/nvim
"

Run :checkhealth for more info

nvim-lsp version(commit hash): fc9d94ef006e082596c2e8724eb3f1c92ff203c7

:checkhealth result

health#vimtex#check
========================================================================
## vimtex
  - OK: Vim version should have full support!
  - OK: Zathura should work properly!
  - OK: Compiler should work!

health#nvim_lsp#check
========================================================================
## Checking language server protocol configuration
  - INFO: purescriptls: configuration checked.

health#deoplete#check
========================================================================
## deoplete.nvim
  - OK: exists("v:t_list") was successful
  - OK: has("timers") was successful
  - OK: has("python3") was successful
  - OK: Require Python 3.6.1+ was successful
  - OK: Require msgpack 1.0.0+ was successful
  - INFO: If you're still having problems, try the following commands:
    - $ export NVIM_PYTHON_LOG_FILE=/tmp/log
    - $ export NVIM_PYTHON_LOG_LEVEL=DEBUG
    - $ nvim
    - $ cat /tmp/log_{PID}
    - and then create an issue on github

health#denite#check
========================================================================
## denite.nvim
  - OK: has("python3") was successful
  - OK: Python 3.6.1+ was successful
  - OK: Require msgpack 1.0.0+ was successful

health#defx#check
========================================================================
## defx.nvim
  - OK: has("python3") was successful
  - OK: Python 3.6.1+ was successful

health#nvim#check
========================================================================
## Configuration
  - WARNING: Missing user config file: /home/nick/.config/nvim/init.vim
    - ADVICE:
      - :help |init.vim|

## Performance
  - OK: Build type: Release

## Remote Plugins
  - OK: Up to date

## terminal
  - INFO: key_backspace (kbs) terminfo entry: key_backspace=\177
  - INFO: key_dc (kdch1) terminfo entry: key_dc=\E[3~
  - INFO: $COLORTERM='truecolor'

health#provider#check
========================================================================
## Clipboard (optional)
  - OK: Clipboard tool found: xclip

## Python 2 provider (optional)
  - INFO: Using: g:python_host_prog = "/nix/store/3g1cbcf14bhzvdrka5mzg1las9dl3604-neovim-master/bin/nvim-python"
  - INFO: Executable: /nix/store/3g1cbcf14bhzvdrka5mzg1las9dl3604-neovim-master/bin/nvim-python
  - INFO: Python version: 2.7.18
  - INFO: pynvim version: 0.4.1
  - OK: Latest pynvim is installed.

## Python 3 provider (optional)
  - INFO: Using: g:python3_host_prog = "/nix/store/3g1cbcf14bhzvdrka5mzg1las9dl3604-neovim-master/bin/nvim-python3"
  - INFO: Executable: /nix/store/3g1cbcf14bhzvdrka5mzg1las9dl3604-neovim-master/bin/nvim-python3
  - INFO: Python version: 3.8.5
  - INFO: pynvim version: 0.4.1
  - OK: Latest pynvim is installed.

## Python virtualenv
  - OK: no $VIRTUAL_ENV

## Ruby provider (optional)
  - INFO: Ruby: ruby 2.6.6p146 (2020-03-31) [x86_64-linux]
  - INFO: Host: /nix/store/3g1cbcf14bhzvdrka5mzg1las9dl3604-neovim-master/bin/nvim-ruby
  - WARNING: Gem "neovim" is out-of-date. Installed: 0.8.0, latest: 0.8.1
    - ADVICE:
      - Run in shell: gem update neovim

## Node.js provider (optional)
  - INFO: Disabled (g:loaded_node_provider=1).  This might be due to some previous error.
  - WARNING: `node` and `npm` (or `yarn`) must be in $PATH.
    - ADVICE:
      - Install Node.js and verify that `node` and `npm` (or `yarn`) commands work.

## Perl provider (optional)
  - WARNING: `perl` and `cpanm` must be in $PATH.
    - ADVICE:
      - Install Perl and cpanminus and verify that `perl` and `cpanm` commands work.

What language server(If the problem is related to a specific language server): purescriptls
Operating system/version: NixOS 20.09.git.3878497ec2fM (Nightingale)
Terminal name/version: kitty 0.18.3
$TERM: xterm-kitty

How to reproduce the problem from neovim startup
Try to save a purescript file.
Actual behaviour
Nothing
Expected behaviour
Compile warnings are annotated in the buffer.
purescript-language-server emits diagnostics info on building; is this sort of information used for nvim-lsp currently, or does it need to be some kind of dedicated LSP diagnostics interface?

Thanks for your report. The configs here are best-effort:

It is hoped that these configurations serve as a "source of truth", but they are strictly best effort. If something doesn't work, these configs are useful as a starting point, which you can adjust to fit your environment.

If you have an improvement send a PR. If you have a specific question we'll try to answer it. Thanks.


Category product filter is not implemented

I want to know how to implement 'filter-box' which is already implemented on demo site :)
(I can't find it on current repository)
https://github.com/mage2/laravel-ecommerce/blob/a2896060883dc865b0c7cdbac625692641003d16/themes/mage2/default/views/catalog/category/options.blade.php#L18

@darron1217 That feature doesn't work in this new database version. I am working on it but it will take some time.

@darron1217 Category filter feature is almost finished. I am sure it should have many bugs so please try install again and let me know if you face any bugs.


Invisible Ship

Sorry for the duplicate, but I can't find the old issue. I looked in my mods because you seem to have been thinking it came from an imcompatibility with an other mod, and as you suspected, it does. It seem to be an incompatibility with Immersive Portals.

Immersive Portals isn’t compatible for now, so yeah. It’s known-

Duplicate of #77


Add DDL documentation for shard rerouting/allocation

ALTER TABLE t ...

[x] : REROUTE MOVE af3364d21886314721c403fa4979a2aabc7d2e93
[x] : REROUTE ALLOCATE REPLICA ae1e3e040f9aa8ce18393c9aeeac47902d2f8ca3
[x] : REROUTE CANCEL 6a452d13816fceb483934d244b3c9a48729db861

Codecov Report

Merging #6444 into master will decrease coverage by 0.03%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #6444      +/-   ##
============================================
- Coverage     82.82%   82.79%   -0.04%
- Complexity    13027    13028       +1
============================================
  Files          1497     1497
  Lines         48502    48502
  Branches       5870     5870
============================================
- Hits          40173    40156      -17
- Misses         5573     5587      +14
- Partials       2756     2759       +3

Impacted Files
Coverage Δ
Complexity Δ

...exceptions/GroupByOnArrayUnsupportedException.java
66.66% <0%> (-33.34%)
1% <0%> (-1%)

...in/java/io/crate/lucene/match/CrateRegexQuery.java
43.47% <0%> (-30.44%)
4% <0%> (-4%)

...a/io/crate/exceptions/ContextMissingException.java
81.81% <0%> (-18.19%)
1% <0%> (-1%)

...ticsearch/indices/recovery/BlobRecoveryStatus.java
63.63% <0%> (-9.1%)
3% <0%> (-1%)

...io/crate/protocols/postgres/types/VarCharType.java
91.66% <0%> (-8.34%)
10% <0%> (-1%)

...rc/main/java/io/crate/analyze/UnboundAnalyzer.java
76% <0%> (-8%)
2% <0%> (ø)

...peration/collect/BatchIteratorCollectorBridge.java
65.78% <0%> (-7.9%)
2% <0%> (ø)

.../java/io/crate/planner/node/dql/join/JoinType.java
64.7% <0%> (-5.89%)
6% <0%> (ø)

.../projectors/BatchIteratorBackpressureExecutor.java
92.75% <0%> (-4.35%)
19% <0%> (-1%)

.../distributed/TransportDistributedResultAction.java
78.46% <0%> (-3.08%)
9% <0%> (ø)

... and 26 more

Continue to review full report at Codecov.

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


private key seed appears in log

per security best practices, we shouldn't log private keys.
This log from HostCore.WebAssembly.Imports dumps the seed field on error. The seed should be blanked out or replaced with "(omitted)" or "S(57 chars omitted)". A general solution is preferred rather than fixing the call in this location, for example, if whatever function converts the struct to json below could be overridden to accept a list of fields to mask, then it would apply to all debug or error logs.
Alternate solutions:

It's rare that the actual key is really needed for debugging, but If it's really important to confirm a specific key is present, then one of the following would be acceptable:

show the first 3 chars, compromising only a small portion of the bit length
compute a sha256 of the private key/seed and display that
enable key logging only with a special environment variable set, and by practice, only set that variable in test environments that share no keys with production and contain no sensitive data

15:02:29.186 [error] GenServer #PID<0.4774.0> terminating
** (FunctionClauseError) no function clause matching in HostCore.WebAssembly.Imports.authorize_call/1
    (host_core 0.50.0) lib/host_core/web_assembly/imports.ex:212: HostCore.WebAssembly.Imports.authorize_call(%{agent: #PID<0.4773.0>, authorized: false, binding: "default", claims: %{__struct__: HostCore.WasmCloud.Native.Claims, call_alias: nil, caps: [], issuer: "ABCEKIBKCOEN7K2FRRNACKON4OLM3FQ72ZYFAY22OJJSFOE6MQ6LEXF3", name: "pinger", public_key: "MDCX6E7RPUXSX5TJUD34CALXJJKV46MWJ2BUJQGWDDR3IYRJIWNUQ5PN", revision: 0, tags: [], version: "v0.1.0"}, namespace: "wasmcloud/examples/ponger", operation: "Ping", payload: <<129, 165, 118, 97, 108, 117, 101, 11>>, prefix: "default", seed: "SCAH4B44WHN7RJSGPRRI2DEVZENNNWEPXMG7BAE46PSSHMX4WSQZVGYW3M", source_actor: "MDCX6E7RPUXSX5TJUD34CALXJJKV46MWJ2BUJQGWDDR3IYRJIWNUQ5PN", state: %HostCore.Actors.ActorModule.State{api_version: 0, claims: %{__struct__: HostCore.WasmCloud.Native.Claims, call_alias: nil, caps: [], issuer: "ABCEKIBKCOEN7K2FRRNACKON4OLM3FQ72ZYFAY22OJJSFOE6MQ6LEXF3", name: "pinger", public_key: "MDCX6E7RPUXSX5TJUD34CALXJJKV46MWJ2BUJQGWDDR3IYRJIWNUQ5PN", revision: 0, tags: [], version: "v0.1.0"}, guest_error: nil, guest_request: nil, guest_response: nil, host_error: nil, host_response: nil, instance: #PID<0.4774.0>, instance_id: "6f30bac0-6c4d-42bc-976e-bb01cb8939fb", invocation: %HostCore.Actors.ActorModule.Invocation{operation: "HandleRequest", payload: <<133, 171, 113, 117, 101, 114, 121, 83, 116, 114, 105, 110, 103, 160, 164, 112, 97, 116, 104, 161, 47, 166, 109, 101, 116, 104, 111, ...>>}, ociref: "", subscription: 9}, target: {:actor, "MBMOM2EZZICFYM4KATRMH2JUO5QWE3YWCHGFZVRQQ2SQI4I5BKWIGMBS", "wasmbus.rpc.default.MBMOM2EZZICFYM4KATRMH2JUO5QWE3YWCHGFZVRQQ2SQI4I5BKWIGMBS"}, verified: true})
    (host_core 0.50.0) lib/host_core/web_assembly/imports.ex:144: HostCore.WebAssembly.Imports.host_call/11
    (wasmex 0.5.0) lib/wasmex.ex:339: Wasmex.handle_info/2
    (stdlib 3.15) gen_server.erl:695: :gen_server.try_dispatch/4
    (stdlib 3.15) gen_server.erl:771: :gen_server.handle_msg/6
    (stdlib 3.15) proc_lib.erl:226: :proc_lib.init_p_do_apply/3
Last message: {:invoke_callback, "wapc", "__host_call", %{memory: #Reference<0.508212900.3147431936.143714>}, [1056245, 7, 1056252, 25, 1056277, 4, 1114376, 8], #Reference<0.508212900.3147431936.143713>}
State: %{imports: %{"wapc" => %{"__console_log" => {:fn, [:i32, :i32], [], #Function<14.90078472/3 in HostCore.WebAssembly.Imports.wapc_imports/1>}, "__guest_error" => {:fn, [:i32, :i32], [], #Function<19.90078472/3 in HostCore.WebAssembly.Imports.wapc_imports/1>}, "__guest_request" => {:fn, [:i32, :i32], [], #Function<15.90078472/3 in HostCore.WebAssembly.Imports.wapc_imports/1>}, "__guest_response" => {:fn, [:i32, :i32], [], #Function<18.90078472/3 in HostCore.WebAssembly.Imports.wapc_imports/1>}, "__host_call" => {:fn, [:i32, :i32, :i32, :i32, :i32, :i32, :i32, :i32], [:i32], #Function<13.90078472/9 in HostCore.WebAssembly.Imports.wapc_imports/1>}, "__host_error" => {:fn, [:i32], [], #Function<20.90078472/2 in HostCore.WebAssembly.Imports.wapc_imports/1>}, "__host_error_len" => {:fn, [], [:i32], #Function<21.90078472/1 in HostCore.WebAssembly.Imports.wapc_imports/1>}, "__host_response" => {:fn, [:i32], [], #Function<16.90078472/2 in HostCore.WebAssembly.Imports.wapc_imports/1>}, "__host_response_len" => {:fn, [], [:i32], #Function<17.90078472/1 in HostCore.WebAssembly.Imports.wapc_imports/1>}}, "wasmbus" => %{"__console_log" => {:fn, [:i32, :i32], [], #Function<23.90078472/3 in HostCore.WebAssembly.Imports.wasmbus_imports/1>}, "__guest_error" => {:fn, [:i32, :i32], [], #Function<28.90078472/3 in HostCore.WebAssembly.Imports.wasmbus_imports/1>}, "__guest_request" => {:fn, [:i32, :i32], [], #Function<24.90078472/3 in HostCore.WebAssembly.Imports.wasmbus_imports/1>}, "__guest_response" => {:fn, [:i32, :i32], [], #Function<27.90078472/3 in HostCore.WebAssembly.Imports.wasmbus_imports/1>}, "__host_call" => {:fn, [:i32, :i32, :i32, :i32, :i32, :i32, :i32, :i32], [:i32], #Function<22.90078472/9 in HostCore.WebAssembly.Imports.wasmbus_imports/1>}, "__host_error" => {:fn, [:i32], [], #Function<29.90078472/2 in HostCore.WebAssembly.Imports.wasmbus_imports/1>}, "__host_error_len" => {:fn, [], [:i32], #Function<30.90078472/1 in HostCore.WebAssembly.Imports.wasmbus_imports/1>}, "__host_response" => {:fn, [:i32], [], #Function<25.90078472/2 in HostCore.WebAssembly.Imports.wasmbus_imports/1>}, "__host_response_len" => {:fn, [], [:i32], #Function<26.90078472/1 in HostCore.WebAssembly.Imports.wasmbus_imports/1>}}}, instance: #Wasmex.Instance<#Reference<0.508212900.3147300865.155272>>}

In production, state is never logged. Not a bug.


Ensure Yarl <0.8 is installed

What do these changes do?
Prevents Yarl 0.8 from being installed as it breaks aiohttp installs.
Are there changes in behavior for the user?

Related issue number
https://github.com/KeepSafe/aiohttp/issues/1447
Checklist

[ ] I think the code is well written
[ ] Unit tests for the changes exist
[ ] Documentation reflects the changes
[ ] Add yourself to CONTRIBUTORS.txt

The format is <Name> <Surname>.
Please keep alphabetical order, the file is sorted by names.

[ ] Add a new entry to CHANGES.rst

Choose any open position to avoid merge conflicts with other PRs.
Add a link to the issue you are fixing (if any) using #issue_number format at the end of changelog message. Use Pull Request number if there are no issues for PR or PR covers the issue only partially.

Current coverage is 98.89% (diff: 100%)

Merging #1448 into 1.1 will increase coverage by 0.04%

@@                1.1      #1448   diff @@
==========================================
  Files            29         29
  Lines          6779       6779
  Methods           0          0
  Messages          0          0
  Branches       1128       1128
==========================================
+ Hits           6701       6704     +3
+ Misses           35         32     -3
  Partials         43         43

Powered by Codecov. Last update 5412d9e...7d8824e

Although this fixes it for a newer release of aiohttp, it would probably be better to remove the Yarl 0.8 release from PyPi (since @asvetlov maintains both aiohttp and Yarl) because all old releases of aiohttp will no longer be installable, breaking a bunch of builds everywhere.

Closing this as Yarl 0.8.1 will reinstate the quote and unquote methods https://github.com/KeepSafe/aiohttp/issues/1447#issuecomment-264659978


please clarify GO:0009346 citrate lyase complex

GO:0009346 name (citrate lyase complex) is very general but its description applies only to the bacterial type.
id: GO:0009346
name: citrate lyase complex
namespace: cellular_component
def: "Citrate lyase is a multienzyme complex with three constituents: the alpha subunit, citrate-ACP transferase; the beta subunit, citryl-ACP lyase; and the gamma subunit, an acyl-carrier protein which also carries the prosthetic group components. All three subunits are required for citrate lyase enzyme activity." [EC:<IP_ADDRESS>, MetaCyc:ACECITLY-CPLX]

Indeed all the bacterial/plant/animal enzymes are complexes so the general name makes sense but you can see from the following that the description only applies to bacteria:
https://www.semanticscholar.org/paper/A-novel-architecture-of-600-kDa-bacterial-citrate-Forouhar-Lew/b467491dd6a1598d92a0a47cde0bde51cc35da7f where is stated:
There are two types of citrate lyases, ATP-dependent and ATP-independent. The latter, which exists only in bacteria, converts citrate to acetate and oxaloacetate using a two-step reaction catalyzed by a stable heterotrimeric protein complex comprising alpha-beta-gamma subunits. So bacteria use ACP (acyl-carrier protein) but no ATP with phosphorylation. The entry definition clearly is for bacteria. It is not for plants because there the complex is heterodimeric (PMID:12376641) nor animals where it is homotetrameric (IPR014608).

[x] GO:0009346 citrate lyase complex: changed label to ATP-independent citrate lyase complex
[x]  Added comment in def: "This enzyme has only been found in bacteria."
[x]  Added taxon constraint only in bacteria: GO:0008815 citrate (pro-3S)-lyase activity and GO:0009346 ATP-independent citrate lyase complex
[x]  Created new term ATP-dependent citrate lyase complex
+[Term]
+id: GO:0140615
+name: ATP-dependent citrate lyase complex
+namespace: cellular_component
+def: "A protein complex that catalyzes the cleavage of citrate into oxaloacetate and acetyl-CoA." [PMID:12376641]
+intersection_of: GO:0032991 ! protein-containing complex
+intersection_of: capable_of GO:0003878 ! ATP citrate synthase activity
+property_value: term_tracker_item https://github.com/geneontology/go-ontology/issues/21028 xsd:anyURI
+created_by: pg
+creation_date: 2021-03-08T14:15:18Z

@tberardini
When the new term is available, can you move the 2 TAIR annotations from GO:0009346 citrate lyase complex
to
GO:0140615 name: ATP-dependent citrate lyase complex
Thanks, Pascale

Thanks.

@pgaudet Updating the annotations is on my to do list now.

@pgaudet Annotations transferred.


[Automation] Generate Fluent Lite from Swagger elastic#package-2024-03-01

[Automation] Generate Fluent Lite from Swagger elastic#package-2024-03-01

/azp run java - elastic - ci

/azp run prepare-pipelines

/azp run java - elastic - ci

API change check
APIView has identified API level changes in this PR and created following API reviews.
com.azure.resourcemanager:azure-resourcemanager-elastic


[HxTabPanel] Set order in which tabs will be displayed

I want to set a specific order in which the tabs would show up in the tab panel. In my scenario one tabs rendering is delayed because it is wrapped in AuthorizeView component therefore it will render last in the tab panel.
<HxTabPanel>
    <HxTab Title="First tab" Order="1">
        <Content>This is the first tab.</Content>
    </HxTab>

    <HxTab Title="Second tab" Order="2">
        <TitleTemplate>Second tab</TitleTemplate>
        <Content>This is the second tab. This tab is initially active.</Content>
    </HxTab>

    <HxTab Title="Third tab" Order="3">
        <TitleTemplate>Disabled tab</TitleTemplate>
    </HxTab>
</HxTabPanel>

This is already implemented for HxGridColumns and we should "just" replicate the same ordering logic here.
cc @jirikanda


HoldClick timing problem

When first started, the program seems to obey the 500msec delay that signals that a click is really a HoldClick.  But after some usage (a few menu operations) the required delay seems to increase.  Sometimes the Teensy locks up!?

This may have been solved by adding 0.1uF caps to the RE pins A, B and the switch.  Performs as expected now.  Close this.


When using generator with MongoDB the default port is 5432 rather than 27017

Gem version: 4.2.2

PR: https://github.com/backup/backup/pull/732


There should be a way to use already connected Socket in HttpClient connectionFactory.

Dart SDK Version 3.3.4
Issue related to dart core library "dart:io"

My library (socks proxy client) worked before class modifiers existed by implementing ConnectionTask making it constructor public.
Now ConnectionTask is final class and I am unable to pass already connected socket to connectionFactory.
Some code that used to work:
class SocketConnectionTask implements ConnectionTask<Socket> {
  SocketConnectionTask(this.socket);
  @override
  final Future<Socket> socket;

  @override
  void cancel() => socket.ignore();
}

httpClient.connectionFactory =
      (uri, proxyHost, proxyPort) async {
        // Returns instance of SocksSocket which implements Socket
        final client = SocksTCPClient.connect(...);

        // Secure connection after establishing Socks connection
        if(uri.scheme == 'https')
          return SocketConnectionTask((await client).secure(uri.host, ...));

        return SocketConnectionTask(client);
      };

I have no idea how to fix it.

I have the same problem, and is it possible to provide an approach for constructing a ConnectionTask instance?

I think laziest solution will be making ConnectionTask constructor public. Anyway it only used in Socket and considering we can extend/implement our own Socket I don't see why ConnectionTask shouldn't be available to us.

Let me see if I can recover our reasoning for making ConnectionTask final. It was probably an oversight.

Here is the relevant comment: https://dart-review.googlesource.com/c/sdk/+/291343/4/sdk/lib/io/socket.dart (line 533):

Final, data class.
Nobody needs to create another version.

This really is a data class so we should probably just make the constructor public. Any objections @lrhn ?

Reasonable to make constructor public.
A little worried that <S> is not constrained in any way. This is basically a genetic CancelableOperation, which isn't linked to sockets in any way except the naming of the future variable.
If anyone can use it as a pair of Future<AnyType> and void Function(), the name no longer makes much sense.
Not sure what to do about that. It's there any reasonable supertype of sockets that the type variable can be bounded by? If not, should we add one?

Not sure what to do about that. It's there any reasonable supertype of sockets that the type variable can be bounded by?

I don't think so. RawSocket only implements Stream, which seems overly broad.

If not, should we add one?

That seems more like a @lrhn question ;-) Would this be a supertype that exists only to indicate that the subclass is Socket-like?

As I remember it, we have several places where we generalize over different socket types, in one way or another, maybe it's only the underlying implementation classes that share something.
Maybe we should have a supertype, the problem is that it should be called Socket, which we already use for something different from a SecureSocket.
So no great ideas off the top of my head.

Can't this be done already? It doesn't seem like much work (If you, of course, do not overthink it). Sorry for being impatient but I really want to use Dart 3 features.
I need to fix my socks package and this is one of the most useful parts of it (being able to connect it with built-in HTTP client).

Ditto
@lrhn @brianquinlan is this issue still on your radar?
Thanks

Yep, there is a PR out now that I have to do a bit more work on: https://dart-review.googlesource.com/c/sdk/+/367820


[DEVOPS-1177] Windows cross test failures

This should fix

nix-tools.tests.x86_64-pc-mingw32-cardano-crypto.cardano-crypto-golden-tests.x86_64-linux
nix-tools.tests.x86_64-pc-mingw32-cardano-sl-cluster.cardano-sl-cluster-test.x86_64-linux

Description

Linked issue

Type of change

[ ] 🐞 Bug fix (non-breaking change which fixes an issue)
[ ] 🛠 New feature (non-breaking change which adds functionality)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🏭 Refactoring that does not change existing functionality but does improve things like code readability, structure etc
[ ] 🔨 New or improved tests for existing code
[ ] ⛑ git-flow chore (backport, hotfix, etc)

Developer checklist

[ ] I have read the style guide document, and my code follows the code style of this project.
[ ] If my code deals with exceptions, it follows the guidelines.
[ ] I have updated any documentation accordingly, if needed. Documentation changes can be reflected in opening a PR on cardanodocs.com, amending the inline Haddock comments, any relevant README file or one of the document listed in the docs directory.
[ ] CHANGELOG entry has been added and is linked to the correct PR on GitHub.

Testing checklist

[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

QA Steps

Screenshots (if available)

How to merge
Send the message bors r+ to merge this PR. For more information, see
docs/how-to/bors.md.

bors r+


Add permission support for workbook

I have taken the work that @DepthDeluxe did in #292 and tried to implement most of the suggestions left in the code review, as well as adding some more tests.
I've restricted the scope to workbooks as suggested in #292 for ease of review. I will add the support for projects and datasources once I've gotten some feedback.
Here is a sample usage:
server.workbooks.populate_permissions(wb)
print(wb.permissions)

permisssion_collection = TSC.PermissionsCollection(
    [
        TSC.PermissionsRule(
            TSC.PermissionsGrantee(TSC.Permission.GranteeType.Group, "<group-id>"),
            {
                TSC.Permission.WorkbookCapabilityType.AddComment: TSC.Permission.CapabilityMode.Deny
            },
        ),
        TSC.PermissionsRule(
            TSC.PermissionsGrantee(TSC.Permission.GranteeType.User, "<user-id>"),
            {
                TSC.Permission.WorkbookCapabilityType.AddComment: TSC.Permission.CapabilityMode.Deny
            },
        ),
    ]
)

server.workbooks.delete_permission(wb, permisssion_collection)

server.workbooks.update_permission(wb, permisssion_collection)

Thank you for the PR!
I have another MR in progress that changes the design a bit, and covers support for workbooks (#429) that would supersede this.
I'd appreciate your review or thoughts/concerns on the design changes; but 429 covers all the new types coming soon to Server, and plays with some of the existing patterns well.

Thanks for your response @t8y8
Sorry, I didn't see that PR. Not a problem, I will close this one.


Simplify recommendations for using packages.yml and dependencies.yml

Contributions

[X] I have read the contribution docs, and understand what's expected of me.

Link to the page on docs.getdbt.com requiring updates
https://docs.getdbt.com/docs/collaborate/govern/project-dependencies#use-cases
What part(s) of the page would you like to see updated?
db users have (4) choices as it relates to packages.yml and dependencies.yml files:

Neither packages.yml nor dependencies.yml
Only packages.yml
Only dependencies.yml
Both packages.yml and dependencies.yml*

* If both files exist, then only one of them can define the packages: key (i.e., packages.yml !).
To me, the most simple instructions would be:

Add any package dependencies to packages.yml
Add any project dependencies to dependencies.yml

That would keep things very understandable without needing to get into the details of caveats like Jinja, private packages, etc.
Additional information
See also: https://github.com/dbt-labs/docs.getdbt.com/pull/6029

hey @dbeatty10 , this is a great issue! however, i believe we added additional caveat details like jinja, etc. because users raised this as a point of confusion and wanting to understand why jinja rendering isn't allowed in packages.yml (slack thread).
so based a couple of user thread/tickets i recall, it think it's best to keep the jinja,private packages caveats since they did come up multiple times before we added those caveats. so erring on the side of overcommunicating so users can self-answer any questions they have via the docs, rather than raise a ticket or question.
I'll close this issue for now, but let me know if you have any thoughts and i appreciate you opening this up!

erring on the side of overcommunicating so users can self-answer any questions they have via the docs, rather than raise a ticket or question.

Very good points!
My main thought is that there's no special information needed if users just split their packages and mesh dependencies across two files: packages.yml and dependencies.yml.
But right now, a user would have to read the entire page to come to that conclusion. So I'm wondering if there's some way we could lead with the "short story" (just use two different files!)? And then provide the full details either in a separate page or further down on the same page.
Maybe something like this?

The most simple heuristic that will work for every dbt project is:

Add any package dependencies to packages.yml
Add any project dependencies to dependencies.yml

But many folks can consolidate both into a single dependencies.yml file. Read below to learn more.

oh i see, yea that's a good idea! I've re-opened this, thanks for explaining @dbeatty10 !


Load error (comfyUI)

Retry to install several times, got this :
Warning: Missing Node Types
When loading the graph, the following node types were not found:
LayerColor: Brightness Contrast
No selected item
Nodes that have failed to load will show as red on the graph.
.. Any ideas please ?

Perhaps the presence of the '&' character in the node name caused an unexpected JSON decoding issue in comfyUI. Please create a new node to replace the one that cannot be loaded.


Incorporating xlang error handling into cppxlang

Updating the cppxlang projection to represent the state of error handling in xlang
Addressing some PR feedback from the previous PR for error handling in xlang with how to represent HRESULT errors from functions whose interface can't change like QueryInterface by introducing com_interop_result

Build status not being reflected, but it passed.


Feature request: add more options to Luong attention

Currently, the best results of the Luong attention paper (Minh-Thang Luong, Hieu Pham, Christopher D. Manning. "Effective Approaches to Attention-based Neural Machine Translation." EMNLP 2015.) cannot be reproduced with the implementation of tf.contrib.seq2seq.LuongAttention. Features that are missing:

Local attention: attend to a window of time steps, rather than to all of the time steps of the encoder output (global attention). The window size should be a hyperparameter that the user can tune.
Different scoring functions: currently, the scoring function is limited to a dot products between each encoder output and the decoder output. The paper shows better results with "general" scoring (all of the encoder outputs are multiplied by one learnable matrix) and also explores the option of using Bahdanau-like scoring (concatenate and multiply by a learnable matrix, then apply tanh and take a dot product with a learnable vector).
Predictive alignments: while the probability function can be replaced, it would be nice to add predictive alignment as a function, and make the implementation of both monotonic and predictive alignments behave well with local attention limited to a time window (changes shape of learnables).
Input-feeding approach: (please correct me in this one if I am mistaken) the current implementation is missing the final step that computes a prediction by concatenating the context vector with the decoder output, weights them and applies tanh (let this be s_t=tanh(W [c_t; h_t])). Passing s_t through a softmax layer gives the prediction distribution, but passing it as is to the next input improved performance in the paper.

On a sidenote: it also seems to be that there is no difference in the key and query vectors between the Bahdanau and Luong attention mechanisms, when there should be. Bahdanau attention has a computation pathway starting from the previous decoder output h_{t-1} -> a_t -> c_t -> h_t, while Luong attention starts from the current output h_t -> a_t -> c_t -> s_t.

@ebrevdo. Is anyone going to work on this? Should we mark this contributions welcome if we want this included but won't do it at google.

Thang, can you comment?
On Jun 26, 2017 2:30 PM, "Andrew Selle"<EMAIL_ADDRESS>wrote:

@ebrevdo https://github.com/ebrevdo. Is anyone going to work on this?
Should we mark this contributions welcome if we want this included but
won't do it at google.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/10842#issuecomment-311187626,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABtim6WvPh_pJ7bk__DQcD-GJavXts62ks5sICL5gaJpZM4N_QJ8
.

Nagging Assignee: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @lmthang, @ebrevdo: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Hi all,
Sorry for a late reply. Here's my brief answer:

Local attention, predictive alignments: unfortunately, we don't have the bandwidth for this. Local attention, while being an interesting idea, is more complicated to implement and train. Contributions are welcome though I expect the contributors need to demonstrate convincing gains on over nowadays strong baselines and new advances in attention.

Input-feeding approach: yes, we missed the tanh() part but the performance is found to be similar. See answers in  https://github.com/tensorflow/nmt/issues/257.

Different scoring functions: in fact, we are implementing the general approach for Luong attention, see this line. Similarly for Bahdanau attention if you read the code carefully.

Hope that clarifies!

Nagging Assignees @lmthang, @ebrevdo: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @lmthang, @ebrevdo: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @lmthang, @ebrevdo: It has been 15 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @lmthang, @ebrevdo: It has been 59 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @lmthang, @ebrevdo: It has been 74 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.


socket.io handshakeData.headers.cookie should be used

https://github.com/LearnBoost/socket.io/wiki/Authorizing says that socket.io sends cookies using an auxillary HTTP connection during handshake even if a transport incapable of transporting cookies such as flash sockets is used later.
So, SS should use this capability instead of current getCookie RPC to support session sharing with Connect.

Thanks again. Implemented this in the next release, hopefully out tomorrow.


nixos/unifi: add test and additional config options [WIP]

Motivation for this change
This allows having the controller listen on custom ports + to set advanced options via system.properties.
We remove the ability to configure where the data is stored and just force things to be under /var/lib/unifi as someone with specific database requirements can use an external mongodb instance now and configure that to place the data in a custom location.
Furthermore, we add a proper nixos test.
The thing I would love some input on is how we handle migration in case a custom dataDir was set in the past.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @erictapen

Thanks for the PR. Unforunately I have no time currently to review the cnange. In general I'd be glad if someone could take the maintainership for the package.

Just rebased this ontop of current master and migrated to python tests: https://github.com/erictapen/nixpkgs/tree/unifi-add-test
Will test in the next few days and maybe open a new PR.

You should be able to push to my branch @erictapen so we can get this merged. If you want to open a new PR, that's cool too.

Exciting. Never tried that before.
But I can't push to f/unifi on git@github.com:peterhoeg/nixpkgs.git? Maybe I'm not enough of a maintainer in Nixpkgs? I can't create branches in NixOS/nixpkgs e.g..

But I can't push to f/unifi on git@github.com:peterhoeg/nixpkgs.git?

Ah, it's the nixpkgs-committers flag that gives access to this, sorry about that.
I can incorporate your changes or you create a PR - up to you.

This seems like a valuable addition to the unifi module. Any chance to revive this PR to get it merged, @peterhoeg? Happy to lend a hand 🙂

@veehaitch I'd also love to have this. But I guess it makes sense to rebase this first before reviewing? There are probably quite a few merge conflicts by this time.

@veehaitch I'd also love to have this. But I guess it makes sense to rebase this first before reviewing? There are probably quite a few merge conflicts by this time.

Definitely 👍🏻

Without the dataDir option we need a configuration to define all persistent paths.

No, without this option is just goes to /var/lib/unifi

This seems like a valuable addition to the unifi module. Any chance to revive this PR to get it merged, @peterhoeg? Happy to lend a hand 🙂

We absolutely should. I got lazy and simply used my own version...

No, without this option is just goes to /var/lib/unifi

I think you're right, this should work 🙂

No, without this option is just goes to /var/lib/unifi

Love to see this merged as well, but I'm undecided whether the elimination of dataDir is good.
It is more explicit and many services define such an option, thus, we would break the convention here.
Not to mention the needed migration as well.

dataDir itself isn't necessarily needed. We have it for stuff like databases where you very likely want control over
where it's stored, but for services without special requirements, we just stick it under /var/lib/foo and ideally run it
with DynamicUser = true;
That doesn't change the fact, that we need proper migration or at the very least guard it behind stateVersion.


The codecarbon does not run

CodeCarbon version:All
Python version:All
Operating System:Windows 10

Description
Describe what you were trying to get done.
Tell us what happened, what went wrong, and what you expected to happen.
What I Did
Paste the command(s) you ran and the output.
If there was a crash, please include the traceback here.

The codecarbon is incompatible with the Unity program.
Please tell me how to solve this

Hello, could you tell us more about what you are trying to do ?
Unity use C langage, CodeCarbon is a Python library.


White/blank window

Hi! Thanks for the cool library. I tried using it on Windows 10 (21H2 - 19044.2006), and I just have a white/blank window:

However, weirdly it shows fine (w/ acrylic) in the taskbar hover preview.
My code on init (before app ready):
vibe.setup(require('electron').app);

My code in window setting:
const w = new BrowserWindow({
    frame: true,
    center: true,
    backgroundColor: '#00000000',
    autoHideMenuBar: true
});
// ...
vibe.applyEffect(w, 'acrylic');

Thanks again!

Have you set both the html and body elements to transparent in your CSS?
html, body {
    background-color: transparent !important;
}

They're set to transparent yeah.

Hmm, alright. What's your Electron version?

Discord should be 13.6.6 atleast on WIndows


adding language verilog #490

Adding language verilog.
Verilog Hardware Description Language
Idea for icon from Visual Code verilog file icon here

Note that tokei has issue identify verilog files
I was surprised too, usually use *.v but tokei wants *.vg so keep that in mind. That shouldn't impact the usage here, only remind users that the issue is upstream.
TESTING
git clone https://github.com/alexforencich/verilog-ethernet.git
cd verilog-ethernet
find . -name *.v -exec mv \{\} \{\}.vg \;

@o2sh
Thanks, all valuable feedback.


ISO string as input

Hi,
On this method:
    fromIso8601(iso8601String: string): jmoment.Moment | null {
        const d = jmoment(iso8601String, jmoment.ISO_8601).locale('fa');
        return this.isValid(d) ? d : null;
    }

I get config._f.match is not a function !!!
And these are the tests:
    it('should create dates from valid ISO strings', () => {
        expect(adapter.fromIso8601('1985-04-12T23:20:50.52Z')).not.toBeNull();
        expect(adapter.fromIso8601('1996-12-19T16:39:57-08:00')).not.toBeNull();
        expect(adapter.fromIso8601('1937-01-01T12:00:27.87+00:20')).not.toBeNull();
        expect(adapter.fromIso8601('1990-13-31T23:59:00Z')).toBeNull();
        expect(adapter.fromIso8601('1/1/2017')).toBeNull();
      });

Thanks a lot.

Hi mate,
see the issue in moment.js
its not necessary pass format for ISO_8601 strings
const d = jmoment(iso8601String).locale('fa');

Ok,
Then this section of test expect(adapter.fromIso8601('1/1/2017')).toBeNull(); does not pass.
It simply creates a moment object from non-ISO string.

This is implemented in moment.js and this library has not changed it.


Update generating cert doc for new tls options

Update docs for the changes in Cargill/splinter#727
This should be merged after Cargill/splinter#727

@chenette  that is a unrelated change but I added a commit for it.


Fixed relative refs & jquery cdn

Hey, nice work on the API!
I've fixed the reference to 'common.js' (and copy pasted it in), and have also changed the jquery cdn link to the Microsoft cdn, as the original was 404'ing.
The example is now functional but not pretty because the 'style' js and css files are still missing.

Hi Mark,
Very very happy to receive our very first pull request. We did not foresee problem you picked up where if someone else forks the repo, their scripts and styles will cease to work due to the relative references.
There is however a few issues with this pull request:

Is there any reason the why MS CDN is used as opposed to jQuery's official CDN?
The problem of relative references still remain for style files. It would be better to reference all of them as ://ausftas.github.io/example-api-tariffs/... e.g. ://ausftas.github.io/example-api-tariffs/common.js etc instead of copy pasting them onto the repo.

Cheers

Oh, that's weird. The jQuery CDN was giving me a 404 last night (according to devtools in chrome). Must have just been temporary downtime (or my workplace blocks it for some strange reason...).
I'll close this pull request. Page looks nice btw :)

Resolved by #2


Add a shortcut to toggle the ability of finding project files or all files

Check for existing issues

[X] Completed

Describe the feature
The file finder does not support for finding non-project files. It would be nice to have a shortcut to enable and disable the ability of finding all files (hidden files like .env or other files) in the project.
If applicable, add mockups / screenshots to help present your vision of the feature
No response

Well, I was wrong, it does support for finding all files. But some of them are ignored by .gitignore.

Yeah, that's a current limitation in Zed.  You can upvote any of these issues if you are looking to have some control over that:

https://github.com/zed-industries/community/issues/1813
https://github.com/zed-industries/community/issues/1538
https://github.com/zed-industries/community/issues/70


tmux 2.1, strange interaction

Description
I'm using the latest version of iterm2 nightly build and everything works properly in there. When I'm in my tmux session don't show up or all the images pile onto each other and at the end of the output they're cut off.
Reference
Visual: https://vimeo.com/157512145

Wow, weird indeed. I don't think it's related to gifi, as it just uses the official imgcat script by iTerm team. I googled and found this stackoverflow question, which is related to your situation. So apparently it does not work or does not work properly in a tmux session.

Came here in search of a solution to get this working in tmux. Sad it doesn't work but glad I found an answer. Thanks.


Cant change meter color

Hi,
I apologize if this is the wrong forum.
This is my current dashboard.

I'm trying to change the default "green" to a darker color.
I tried changing the "$background-color" value in "meter.scss", that didn't change anything.
What am I missing? Can someone please point me in the right direction?
This project is amazing!
Thanks!
Tzvi

Add a data-bgcolor property on the widget in your Dashboad .erb file. This enables you to specify different colours for individual widgets.


Add TON NFT's support

[ ] https://github.com/gemwalletcom/core/issues/136 (needs to be implemented first)

Use https://docs.tonconsole.com/tonapi/api-v2 to return NFT's

@kehiy can you comment here? so I can tag you

Yep, you can.


[BUG] Assertion failure in IndexShard.updateGlobalCheckpointOnReplica() when remote translog is enabled

PublishCheckpointAction fails with following stacktrace.
java.lang.AssertionError: supposedly in-sync shard copy received a global checkpoint [4] that is higher than its local checkpoint [-1]
	at __randomizedtesting.SeedInfo.seed([16C715B7EF601310]:0)
	at org.opensearch.index.shard.IndexShard.updateGlobalCheckpointOnReplica(IndexShard.java:3044)
	at org.opensearch.index.shard.IndexShard.lambda$innerAcquireReplicaOperationPermit$40(IndexShard.java:3763)
	at org.opensearch.action.ActionListener$3.onResponse(ActionListener.java:130)
	at org.opensearch.index.shard.IndexShardOperationPermits.acquire(IndexShardOperationPermits.java:309)
	at org.opensearch.index.shard.IndexShardOperationPermits.acquire(IndexShardOperationPermits.java:254)
	at org.opensearch.index.shard.IndexShard.lambda$acquireReplicaOperationPermit$38(IndexShard.java:3698)
	at org.opensearch.index.shard.IndexShard.innerAcquireReplicaOperationPermit(IndexShard.java:3818)
	at org.opensearch.index.shard.IndexShard.acquireReplicaOperationPermit(IndexShard.java:3692)
	at org.opensearch.action.support.replication.TransportReplicationAction.acquireReplicaOperationPermit(TransportReplicationAction.java:1204)
	at org.opensearch.action.support.replication.TransportReplicationAction$AsyncReplicaAction.doRun(TransportReplicationAction.java:849)
	at org.opensearch.common.util.concurrent.AbstractRunnable.run(AbstractRunnable.java:52)
	at org.opensearch.action.support.replication.TransportReplicationAction.handleReplicaRequest(TransportReplicationAction.java:697)
	at org.opensearch.transport.RequestHandlerRegistry.processMessageReceived(RequestHandlerRegistry.java:106)
	at org.opensearch.transport.InboundHandler$RequestHandler.doRun(InboundHandler.java:453)
	at org.opensearch.common.util.concurrent.ThreadContext$ContextPreservingAbstractRunnable.doRun(ThreadContext.java:806)
	at org.opensearch.common.util.concurrent.AbstractRunnable.run(AbstractRunnable.java:52)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
	at java.base/java.lang.Thread.run(Thread.java:829)

Draft PR is created but could not get reviewed/merged. Moving to 2.7.0 release.

Hey @sachinpkale. Is it safe to close this issue as completed since the backport #6978 was merge in 2.x line and is ready to be released for v2.7.0


Aggregation layer tooltips show incorrect entries

Aggregation layer tooltips are showing unrelated information  -- properties of other objects that haven't been picked --- instead of null values (which is the expected result given the current implementation).
Tooltips on these layers should include the point count (if available) and the aggregated value (if available).

Tooltip removed from aggregation layer params. Fixed in #56


Apply pyupgrade suggestions

Changes proposed in this pull request
Apply pyupgrade suggestions:
pyupgrade --py38-plus

Documentation that should be reviewed
-

Codecov Report
Patch coverage: 18.18% and no project coverage change.

Comparison is base (e329aa1) 49.49% compared to head (535936a) 49.49%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3043   +/-   ##
=======================================
  Coverage   49.49%   49.49%
=======================================
  Files          46       46
  Lines        3598     3598
=======================================
  Hits         1781     1781
  Misses       1817     1817

Impacted Files
Coverage Δ

fmriprep/__main__.py
0.00% <0.00%> (ø)

fmriprep/cli/run.py
4.58% <ø> (ø)

fmriprep/cli/tests/test_parser.py
100.00% <ø> (ø)

fmriprep/interfaces/gifti.py
53.33% <ø> (ø)

fmriprep/interfaces/patches.py
27.27% <0.00%> (ø)

fmriprep/interfaces/reports.py
50.00% <0.00%> (ø)

fmriprep/interfaces/workbench.py
80.23% <0.00%> (ø)

fmriprep/utils/bids.py
15.09% <0.00%> (ø)

fmriprep/utils/telemetry.py
18.68% <0.00%> (ø)

fmriprep/workflows/base.py
10.25% <ø> (ø)

... and 2 more

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


how to use class MyRateController?

I am using instaloader on windows, how do I go about invoking the class below? is this just a matter of changing the code within the .py file, or do I need to put this in an args file? thanks, the documentation isn't really all that clear.
import instaloader
class MyRateController(instaloader.RateController):
def query_waittime(self, query_type, current_time, untracked_queries=False):
return 28 + super().query_waittime(query_type, current_time, untracked_queries)
L = instaloader.Instaloader(rate_controller=lambda ctx: MyRateController(ctx))

god forbid anyone answer.

Im not asking for programming help, or how to use python, i was asking for how this class is invoked, is it automatic; or do i have to call it from a python script. you dont list anything in the documentation to address these questions. this is an appropriate place to ask how to use the product in question, because the documentation is so crappy.

do I need to put this in an args file?

@god43, what the hell is "an args file"?

Im not asking for programming help, or how to use python

You may not be, but you should, as you clearly need help with both, as demonstrated here:

do i have to call it from a python script

Where else would you call it from?

you dont list anything in the documentation to address these questions.

True, they probably dont mention that you need to put Python code in a Python script. Hopefully every single person who has ever coded Python would already know that.

this is an appropriate place to ask how to use the product in question, because the documentation is so crappy.

Im not asking for programming help, or how to use python

You may not be, but you should, as you clearly need help with both, as demonstrated here:

do i have to call it from a python script

Where else would you call it from?

you dont list anything in the documentation to address these questions.

True, they probably dont mention that you need to put Python code in a Python script. Hopefully every single person who has ever coded Python would already know that.

this is an appropriate place to ask how to use the product in question, because the documentation is so crappy.

So are your Python skills.

well you call instaloader from the command line after loading the python module, why would code something that is already in the main class file? the documentation says nothing about needing to use python, simply call your parameters from the command line. this is how the documentation says to use instaloader, no mention of using a python script, simply a cmd window; or terminal window if nix based? what now internet tough person, maybe understand the issue before you pop off.
https://instaloader.github.io/cli-options.html#cmdoption-arg-args-txt


Remove MPU6500 driver due to insufficient testing with vibration

The MPU6500 driver configures the sensor not correctly, which will lead to bad flight performance. It has to be removed. Instead the MPU9K driver should be used with similar arguments:
if mpu9250 -s -R 8 start
then
fi

Note that the -R param is lightly correctly set at -R8, if not it needs to be changed. This applies to the MindPX config which needs to be changed accordingly.
Also note that the SPI chip selects need to be updated to match.

Fixed


ViewPortTexture in button freezes system

Godot version
v4.1.1.stable.arch_linux
System information
Arch Linux
Issue description
In this kind of specific situation that I set up in my test project having a SubViewportContainer visible while loading the SubViewport as an expanding texture in a button next to it in an HBoxContainer the whole system freezes.
Steps to reproduce

Open my test project
Get ready to force quit godot (if you don't close it quickly, you might have to reboot)
Toggle visibility of the SubViewportContainer

Minimal reproduction project
subviewport_freeze_demo.zip

It's probably a bit less specific than what I have here but I didn't have time to test it until I have the minimal project


Coop and config fixes

all coop tests now work
@johnjosephhorton

Removed some a config var no longer in use
Removed some config vars such as RPM_OPENAI. Config vars are supposed to be vars that are absolutely crucial for the package to work. Everything else (including LLM API keys) should be set as env vars (or in code) by the users, and then pulled as need (and if they're absent, their absence should dealt with as needed) in code.
I've also made a change in edsl/language_models/registry.py where a value was previously set to 'test' if a config var was missing. That's not right - the config var DEFAULT_MODEL should always tell us what the default model should be. Users can override it using an env var (or code), and the tests suite can override it with the pytest.ini (or code)
Speaking of (3), I've "incorrectly" set in tests/pytest.ini the default model to be "gpt-4o" (it should be "test"). The reason is that if it is set to "test", result.example() creation and evaluation takes forever -- or something like that. What would be the reason for that? We'd like to fix that.
I really like that you've added a test model and I'd like to use it in tests. I think a good practice here would be that it lives in fixtures and not in the main code of the package.

I believe 2-5 are good practices, but ymmv. Let me know if you disagree!


Updates to /users call

Found a couple fixes

The regex for users was failing. Need to include the "/"
"Authorization" was misspelled
"apikey" was null because it wasn't using config

Wow I wonder how I could code so badly, thanks for fixing them!

@fvdm thanks for the project!


Question - Run selenium tor with remote web driver (selenium grid)

Question
Is there any possibility of running Tor using remote web driver? like:
var remoteWebDriver = new RemoteWebDriver(DesidedCapabilities.Tor());

Assuming you mean tor browser selenium by Tor, I don't know. Feel free to experiment yourself and share your findings with the others.


Change perform_validations signature.

This change is

Codecov Report

Merging #1126 (37c1cc5) into main (029dd08) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main    #1126   +/-   ##
=======================================
  Coverage   67.16%   67.16%
=======================================
  Files          51       51
  Lines        6578     6578
  Branches     6578     6578
=======================================
  Hits         4418     4418
  Misses       1761     1761
  Partials      399      399

Files
Coverage Δ

crates/native_blockifier/src/py_validator.rs
0.00% <0.00%> (ø)

:mega: Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!


Wrong border thickness when drawing TouchableOpacity with borderRadius

Is this a bug report?
Yes. It only occurs 0.51.0 version

Have you read the Contributing Guidelines?
Yes
Environment
Environment:
OS: macOS High Sierra 10.13.2
Node: 8.9.1
Yarn: Not Found
npm: 5.5.1
Watchman: Not Found
Xcode: Xcode 9.2 Build version 9C40b
Android Studio: 3.0 AI-171.4443003
Packages: (wanted => installed)
react: 16.0.0 => 16.0.0
react-native: 0.51.0 => 0.51.0
Target Platform: Android 4.1.2 (API 16)

Steps to Reproduce

<TouchableOpacity       style={{             				borderColor: 'blue',             				borderWidth: 1,             				borderRadius: 4,             				alignItems: 'center',             				justifyContent: 'center',             				flexDirection: 'row',             				height: 44,                             width: 200, 			        }}>         				{         					<Text style={{         						fontSize: 16,         						color : 'blue',         						textAlign: 'center',         						backgroundColor: 'transparent',         					}}>         						This is a flat button         					</Text>         				} 			    </TouchableOpacity>
Expected Behavior

Actual Behavior

Reproducible Demo

(Paste the link to an example project and exact instructions to reproduce the issue.)

+1


This mod is INCREDIBLY UNDERRATED!

Holy sh3t I had no idea something like this was even possible, this is incredible, the modding community is going places, but could you please tell me if there's any risk of corrupting a save file if I accidentally load a one where I had previously used the mod but without the mod enabled?

I see, I would also like to report what I believe to be a bug, when I was using Noxious Shroom in the colosseum of fools, whenever it spread the cloud would not spawn at another enemy's location, but randomly nearby the fog first fog.
Also, Lush Moss is very OP when put in the head slot and being overcharged, especially when combined with Paving Stone and Quick Slash, it makes face-tanking an actually viable strategy, if combined with Stalwart Shell it makes it very easy to not die.

Lush Moss has been nerfed slightly in the latest update, and Noxious Shroom could be targeting dead enemies which should be fixed in the next update.


Added Vector2i

Redone the 2D integer vectors. Added a small example in the demo project.
Note: I've had to change the runtime version so that the projects would work on my old laptop (VS 2013 and .NET 4.5). This should be changed back to 4.5.2.

Alright, I'll change the version after merging

I've just done some quick doc on the wiki for Vector2i, so you might just want to check it to make sure it makes sense.

Looking good, nothing I would change


chore: Add feature flag for VM 2.0 node and platform CVEs

Description
Title
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
Load the new central image and verify inclusion of the feature flag in the network response:

#10315
#10304  👈
master

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

Join @dvail and the rest of your teammates on  Graphite


Add a Gitter chat badge to README.md

openworm/PyOpenWorm now has a Chat Room on Gitter
@slarson has just created a chat room. You can visit it here: https://gitter.im/openworm/PyOpenWorm.
This pull-request adds this badge to your README.md:

Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

dupe


Don't color diff output when stdout is not a tty.

Fixes #435

Superceded by https://github.com/rust-lang-nursery/rustfmt/pull/1023.


Terraform providers fail to instantiate after multiple use

To Reproduce
From the same cloudshell, apply multiple terraform configurations from different directories. After multiple terraform configurations have been applied, Terraform will start failing to initialize providers. This only happens on cloudshell. This can be consistently reproduced using #AzureSandbox.
This issue makes cloudshell unusable for complex Terraform scenarios.
Observed Behavior
Error: Failed to load plugin schemas

Error while loading schemas for plugin components: 2 problems:

- Failed to obtain provider schema: Could not load the schema for provider registry.terraform.io/hashicorp/azurerm: failed to instantiate provider
"registry.terraform.io/hashicorp/azurerm" to obtain schema: Unrecognized remote plugin message:

This usually means that the plugin is either invalid or simply
needs to be recompiled to support the latest protocol..
- Failed to obtain provider schema: Could not load the schema for provider registry.terraform.io/hashicorp/random: failed to instantiate provider
"registry.terraform.io/hashicorp/random" to obtain schema: fork/exec .terraform/providers/registry.terraform.io/hashicorp/random/3.4.3/linux_amd64/terraform-provider-random_v3.4.3_x5:
exec format error..

Expected behavior
Terraform providers should instantiate without errors.
Is this specific to Cloud Shell?
Yes
Interface information
portal.azure.com
Additional context
https://github.com/hashicorp/terraform/issues/28354
https://github.com/hashicorp/terraform-provider-azurerm/issues/11543

Im not running in CloudShell but I am intagrated with app.terraform.io and I get the same error when init and plan from my VSCode workstation.
Any work arounds?
terraform plan ╷ │ Error: Failed to load plugin schemas │ │ Error while loading schemas for plugin components: Failed to obtain provider schema: Could not load the schema for provider       │ registry.terraform.io/hashicorp/azurerm: failed to instantiate provider "registry.terraform.io/hashicorp/azurerm" to obtain       │ schema: Unrecognized remote plugin message: │ │ This usually means that the plugin is either invalid or simply │ needs to be recompiled to support the latest protocol... ╵

I have the exact same issue.  Surprising that there are so few comments on this given how popular CloudShell is and how otherwise smooth the experience using it through VS Code is otherwise.  Really hope this can get some attention!
One thing I did notice when this issue kicks in, is that successive runs of 'terraform validate' yield oscillating results.  Perhaps this helps someone resolve the issue!
alan [ ~/clouddrive/theLab/Virtual Networks - Network Security Groups ]$ terraform validate
╷
│ Error: failed to read provider configuration schema for registry.terraform.io/hashicorp/azurerm: failed to instantiate provider "registry.terraform.io/hashicorp/azurerm" to obtain schema: Unrecognized remote plugin message:
│
│ This usually means that the plugin is either invalid or simply
│ needs to be recompiled to support the latest protocol.
│
│
╵
alan [ ~/clouddrive/theLab/Virtual Networks - Network Security Groups ]$ terraform validate
╷
│ Error: failed to read schema for azurerm_resource_group.rg-app1-sc-us-01 in registry.terraform.io/hashicorp/azurerm: failed to instantiate provider "registry.terraform.io/hashicorp/azurerm" to obtain schema: Unrecognized remote plugin message:
│
│ This usually means that the plugin is either invalid or simply
│ needs to be recompiled to support the latest protocol.
│
│
╵
alan [ ~/clouddrive/theLab/Virtual Networks - Network Security Groups ]$ terraform validate
╷
│ Error: failed to read schema for azurerm_network_interface.NIC1 in registry.terraform.io/hashicorp/azurerm: failed to instantiate provider "registry.terraform.io/hashicorp/azurerm" to obtain schema: Unrecognized remote plugin message:
│
│ This usually means that the plugin is either invalid or simply
│ needs to be recompiled to support the latest protocol.
│
│
╵
alan [ ~/clouddrive/theLab/Virtual Networks - Network Security Groups ]$ terraform validate

The only other detail I can give is that when I first ran into this I wondered if it was somehow related to directory names because copying and renaming full directories of various .tf files, at times resolved the issue.  Once resolved the issue stayed resolved.  Once it starts, it stays.

I don't have an exact diagnosis but it sounds like some local Terraform data may have gotten into a bad state.
This StackOverflow thread offers a couple of workarounds that may be useful:
https://stackoverflow.com/questions/70407525/terraform-gives-errors-failed-to-load-plugin-schemas


Account emails

Adds the ability for accounts to have an email address for password recovery and important alerts.
May also be used in the future for anti-spam mechanisms, such as detecting ban evasion and requiring email verification for suspicious accounts. But those are out of scope of this PR.

[X] HTML email templates
[X] Text email templates
[X] The ability for emails to be rendered and sent
[X] Create new token system (our current one sucks and won't work for email actions)
[X] The ability for email addresses to be added to accounts
[X] The ability for email addresses to be verified
[X] The ability for account passwords to be reset using an account's verified email address
[x] The ability for account security actions to be reverted and for the account to be locked when a suspicious change happens
[x] Send password change email alerts
[x] Send multi-factor authenticator adding/removing email alerts

Some of these will require frontend changes in meower-media/website. Email-related pages will be stored under emails in the website repo.

I need to add the ability for email addresses to be removed before this gets merged.