Create Roslyn workspace with project references.

An overview of execution steps:

dotnet-aspnet-codegenerator tool evaluates the project information

It creates an IProjectContext for the current project.
It also then creates an IProjectContext for each of the projectReferences for the project.
All of this is packaged into ProjectInformation.

dotnet-aspnet-codegenerator invokes the InsideMan (dotnet-aspnet-codegenerator-design) with a port number to callback.

The InsideMan connects to the specified port and gets the ProjectInformation from the caller.

With the information regarding the root project and project references, the InsideMan builds a Roslyn Workspace.

The approach seems very expensive right now, but don't see a good alternative to avoid creation of so many projectContexts.
One optimization would be to expose an API in the Extensions.ProjectModel.MsBuild.Sources that provides all available TFMs in a project without actually building the ProjectContext
Additional Changes:

Removed imports from the Microsoft.VisualStudio.CodeGeneration.Tools package.
Added new packages to NuGetPackageVerifier.json

cc @mlorbetske @natemcmaster @NTaylorMullen

Fundamental design question:

the InsideMan builds a Roslyn Workspace.

Could this be done instead by the "outside man"? My concern is that putting roslyn in the inside man may introduce a dependency on Roslyn into the user's project. The design goal of the "inside man" should be to minimize its depedencies, as really it's only purpose is to load/read the user's assembly.

Could this be done instead by the "outside man"?

Inside man is where all the code generation/ manipulation happens currently. And the code generation relies heavily on Roslyn right now.

But is there some reason it has to be in the inside man? IIUC the only reason for it is that the tool needs to Assembly.Load to find existing types.
Current approach:
outside man --> push project info to --> inside man
This restricts scaffolding to runnable projects (e.g. you can't scaffold in class library projects)
What I'm suggesting:
outside man <--  push list of user types <-- inside man
This has the benefit of
(1) minimal dependencies required to be added to user project
(2)  you could use AssemblyLoadContext to scaffold class library projects
(3) the tool can do simple scaffolding operations without invoking the inside man (scenario: projects not using EF, operations that don't need much information, etc.)

This restricts scaffolding to runnable projects (e.g. you can't scaffold in class library projects)

This is currently intentional.

IIUC the only reason for it is that the tool needs to Assembly.Load to find existing types.

Another reason to make the inside man be the one to do the code generation is the way the current extensibility works for scaffolding.
The inside man looks at the user's dependencies to figure out if any custom scaffolder is installed and needs to be invoked.
Although this can be changed. We'd need to come up with a new contract for these custom scaffolders.

Can code generators return something to the roslyn workspace? IIUC "GenerateCode" is always a 'void' return type. I don't think my suggestion would break this model.

Yes, that might work. I will look into prototyping the new approach.
However for now, the workspace needs to live in the Inside man, because of another reason.
The Roslyn packages need imports and we cannot add that to tools.

:up: :date:

Code looks find, although I still think we should move roslyn into the outer guy.  :shipit: on this as the bigger design question is kinda beyond the scope of this PR.


Objectify monsters' usage limits.

Make objects out of the usage limits, same as what was done for spell usages.
Partially implements #120.

If you rebase, you should pick up the Travis CI changes.


Question: dirty property check

I want to enable/disable a button, if a property is dirty or not. How can I accomplish it in Aurelia?

check out the discussion here:  https://github.com/aurelia/templating-resources/issues/160#issuecomment-164277649


Pagination support

As a user, I want website list pagination that's intuitive and responsive, allowing me to easily navigate between pages and quickly access desired content.
There's already some commented code for it; you could adapt it. Look into how Alberto does it for transaction listing.

Fixed by #30


Fix envconfig parsing for ClusterStateMonitorInterval

It doesn't work if the variable starts with a lower-case letter,
resulting in an interval of 0s instead of the desired 10s. This causes
zero waiting between consecutive calls which is not what we want.

@oshercc make sure it will not make the test flaky again.


I have to log in over and over again

This started to happen again for me:

I loose my logged in sessions almost every hour for some reason.

Is this maybe because part of my GH auth is the GH login where we loose access every few hours and have to re-login via Okto verify?

/duplicate https://github.com/microsoft/vscode/issues/169914

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


cask/dsl/base: fix undefined method `method_missing_message'

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

[x] Have you successfully run brew tests with your changes locally?

This has popped up a fair bit in bug reports in the past couple years.
e.g. #8244

Thanks @Bo98!


Error after setting API key

How can I remove the API key?

I'm sorry for the problem, you can change the configuration with vim ~/.config/gpt-cli/.gptrc, I'll check the problem


[Reslice+otherTlbx] crash switching to an other toolbox after removal of data

How to create the crash:

In Reslice open a data, and remove it
Switch to an other toolbox: crash

Erreur de segmentation (core dumped)

Edit : MergeVolumes is in MUSICardio on which i did also the test. The crash happens on medInria (tested on 4.0 but it should be the same on 3.2) between Reslice and Cropping also


Will CycleIterator forward to dataset on resume for pretrain?

When resuming finetuning, I see that the CycleIterator is forwarded to the dataset where the iteration is to continue from:
https://github.com/Lightning-AI/litgpt/blob/f3343784bbd192490e2a70aa5ef75c52608b1d35/litgpt/finetune/full.py#L208-L219
However, for pretrain, this does not exist and the training seems to resume from the begining:
https://github.com/Lightning-AI/litgpt/blob/f3343784bbd192490e2a70aa5ef75c52608b1d35/litgpt/pretrain.py#L217-L271
Can I check in this case, it looks like when resuming, the pretraining will start from the first dataset, and not forwarded?

The pretraining code uses a stateful dataloader from LitData:
https://github.com/Lightning-AI/litgpt/blob/f3343784bbd192490e2a70aa5ef75c52608b1d35/litgpt/pretrain.py#L192-L198

I see, thank you so much!

Sorry, @awaelchli quick question, in that case, is there a reason why it is not implemented for finetuning?


Problems with ID

I get an error message about the unique QLab ID, also the buttons show a red square but they work. See attached screen shots.

Which version of QLab?

5.3 and 5.3.1Am 14.11.2023 um 21:09 schrieb John Knight Jr @.***>:
Which version of QLab?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

I have not upgraded that far. I'll take look soon.

QLab has changed something in their response to queries. The buttons will work but some of the feedback and variables will not.


Group menu items and individual menu items do not align correctly

Describe the bug
On Public Portal group menu items and individual menu items do not align correctly. Individual menu items are aligned to the top slightly.
Expected behavior
Group menu items and individual menu items should align properly, maintaining a consistent and visually appealing layout.
Screenshots

Hey @Lukeybooi, it seems this issue has come up again.


3.0 doc build fix

(cherry picked from commit<PHONE_NUMBER>1afff89762b8fd3c27c19d913df5c0)
What does this PR do?
Fixes issue with python module dependency versioning. Already done in master. docs in 3.0.x build will fail without it.
Fixes #2891
Which TC components are affected by this PR?

[x] Documentation
[ ] Grove
[ ] Traffic Analytics
[ ] Traffic Monitor
[ ] Traffic Ops
[ ] Traffic Ops ORT
[ ] Traffic Portal
[ ] Traffic Router
[ ] Traffic Stats
[ ] Traffic Vault
[ ] Other _________

What is the best way to verify this PR?
./pkg -v docs

Check all that apply

[ ] This PR includes tests
[x] This PR includes documentation updates
[ ] This PR includes an update to CHANGELOG.md
[ ] This PR includes all required license headers
[ ] This PR includes a database migration (ensure that migration sequence is correct)
[ ] This PR fixes a serious security flaw. Read more: www.apache.org/security

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2569/
Test PASSed.


Jobs top Jobname showing wrong data

Hey there,
when using jobs top <jobname> the data does not match at all with the real current top 10 users in a particular job.
This could be reproduced for all jobs.
We also have signs which show the top 4 players in each jobs. These work correctly.
Jobs 3.9.6
Spigot 1.8.8
Offline mode behind Bungee in online mode.
Other stuff that might be interesting:
storage-method: sqlite
MultiServerCompatability: false #As we use Jobs on one server only this is false
Optimizations:
  NewVersion: true
  DBCleaning:
    Jobs:
      Use: false
      Level: 1
    Users:
      Use: false
      Days: 60
  ShowToplist: false

As always
thank you for reading.

Should be fixed with next build


Angular 9: Compile error

WARNING in Invalid constructor parameter decorator in<EMAIL_ADDRESS>() => [
{ type: DomSanitizer, },
]

ERROR in getInternalNameOfClass() called on a non-ES5 class: expected PrismService to have an inner class declaration

package.json
    "@angular/animations": "^9.0.1",
    "@angular/cdk": "^9.0.0",
    "@angular/common": "^9.0.1",
    "@angular/compiler": "^9.0.1",
    "@angular/core": "^9.0.1",
    "@angular/forms": "^9.0.1",
    "@angular/http": "^7.2.16",
    "@angular/material": "^9.0.0",
    "@angular/platform-browser": "^9.0.1",
    "@angular/platform-browser-dynamic": "^9.0.1",
    "@angular/router": "^9.0.1",
    "@ngx-prism/core": "^2.0.1",
    "@types/prismjs": "^1.9.0",
    "prismjs": "^1.9.0",
    "rxjs": "~6.5.4",
    "zone.js": "~0.10.2"
  },
  "devDependencies": {
    "@angular-devkit/build-angular": "~0.900.2",
    "@angular/cli": "~9.0.2",
    "@angular/compiler-cli": "^9.0.1",
    "@angular/language-service": "^9.0.1",
    "@types/jasmine": "~3.5.4",
    "@types/jasminewd2": "~2.0.8",
    "@types/node": "~13.7.1",
    "codelyzer": "~5.2.1",
    "jasmine-core": "~3.5.0",
    "jasmine-spec-reporter": "~4.2.1",
    "karma": "~4.4.1",
    "karma-chrome-launcher": "~3.1.0",
    "karma-coverage-istanbul-reporter": "~2.1.1",
    "karma-jasmine": "~3.1.1",
    "karma-jasmine-html-reporter": "^1.5.2",
    "protractor": "~5.4.3",
    "ts-node": "~8.6.2",
    "tslint": "~6.0.0",
    "typescript": "~3.7.5"
  }```

Hi Hau,
I am also facing same issue. Looks like this package is no longer maintained by the authors. So either we need to find the alternate or try to upgrade it to support Angular 9 ourselves.
Will let you know if I find any luck. Please update me if you get around it too.
Thanks.

A
workaround is to disable Ivy compiler in Angular 9.
put below in tsconfig.json
  "angularCompilerOptions": {
    "enableIvy": false,
  }

Yeah, that works but the main reason to upgrade to Angular 9 would be to use Ivy compiler. So disabling it just for this component isn't a good idea. I have used ngx-highlightjs instead. It is not as cool as ngx-prism but does the job.

Any update on this ?

having same problem. I think its angular problem(ivy)

I'm also having the same issue, running on Angular 9 with Ivy enabled

Same here

I found a solution at least for my case
I added
{
  "scripts": {
    "postinstall": "ngcc"
  }
}

and the problem was gone
This is needed when you are using the new Ivy compiler
I hope this will help

Not sure why but I get the following error while running ngcc:
Warning: Invalid constructor parameter decorator in<EMAIL_ADDRESS> () => [
    { type: DomSanitizer, },
]
Error: Error on worker #2: Error: getInternalNameOfClass() called on a non-ES5 class: expected PrismService to have an inner class declaration

🤷

Same issue here, looks like lib is dead

Erro será resolvido consultando o seguinte link conforme a versão do Angular: https://github.com/ngx-translate/core

"postinstall": "ngcc" fix problem but not work in IE: Invalid character in function getColors.
Any ideas?

"postinstall": "ngcc"

where do you type that?

"postinstall": "ngcc"

where do you type that?

Package.json -> scripts


set up home link

From what I can see the issue with the navbar links has been fixed since it works as expected on my branch (search-movies-js) after I have pulled from the main. This fix is also responsive although what was implemented previously with the hamburger menu looks a bit better. In conclusion, I believe that is fix is out of date and you shouldn't merge this. Please talk to me at the next meeting so we can decide what we can do with this branch


Support for Jetson TX2?

Will any of the releases will work on Jetson TX2 (with CTI Orbitty carrier)?

@drjasonharrison-vp-eio It should work on all Linux systems. Are you encountering any issues with it?

@drjasonharrison-vp-eio  got it working on jetson TX2 ?

Uh, well, not sure, we were distracted by a number of other issues with our build on Jetson with Orbitty. None of them specifically due to balena. So I have confidence that it would work but I cannot confirm that it does.

[thgreasi] This issue has attached support thread https://jel.ly.fish/#/support-thread~7ead340c-f219-4ccd-b024-ca04ffa148b7

@balena-ci  i am not able to access this url / thread
https://jel.ly.fish/#/support-thread~7ead340c-f219-4ccd-b024-ca04ffa148b7

Hey @yogeshwar607 @balena-ci is part of our internal tooling to track support tickets, the link it posted is simply pointing to https://forums.balena.io/t/wifi-connect-issues-not-showing-access-point-on-jetson-tx2/31734


better stack traces

It would be nice if errors created and thrown by code running via sval could have better stack traces. Below I've shared an approach I used in a POC. All the information to create rich stack traces is available. I suspect it's even possible to have stack traces that combine traces from scripts within sval and scripts outside (i.e. when an outside script calls an inside script and vice-versa)

wow，I think this is very good！So you can send it out？Thank you very much！(^▽^)

any updates on this? :)

hi @doodlesbykumbi! your work is awesome. can you publish your work ? :)


Missing requirements

Hey Briggs. Big fan of the XLEAPP tools, thanks for all your efforts and sharing.
Requirements need updating following SGallery Calculator Vaults addition (https://github.com/abrignoni/ALEAPP/commit/4e44c18ec1fdad0ce0265444dc2b097ba1f86ea5). Standby for PR...

PR https://github.com/abrignoni/ALEAPP/pull/179 relates

What platform are you seeing the error in? Is mimetypes-magic==0.4.30 necessary? I don't remember having to pip install it nor would I know where would it be used. Current code identifies the mimetypes using the magic or filetype libraries. Maybe adding mimetypes-magic==0.4.30 causes some sort of conflict? Not sure what might be going on. Thoughts?

Ah, I probably misunderstood which magic library you're using. Is it python-magic-bin?
I was on a MacOS 12 (Intel) earlier, but also had the same issue with Windows 10.
Now I'm on a Mac with M1, which python-magic-bin doesn't support.
May have to come back to this on Monday...

You were right, mimetypes-magic was unnecessary!
I've removed it from https://github.com/abrignoni/ALEAPP/pull/179 and this works now...

Thanks for merging!
https://github.com/abrignoni/ALEAPP/commit/d7d3bfbe9f06ae52702e1561b99de570edecf7f4


Improve Vue3 Router catch all documentation is confusing

What problem does this feature solve?
The documentation regarding catch-all routes can be improved.  Specifically the section describing why the catch all parameter should be repeatable. Also, the suggested example parameter name "pathMatch" is a little generic and confusing.  The term does not build any immediate intuition for people coming into a new project that the route is a catch all.
What does the proposed API look like?
Improving "pathMatch" example parameter
I'm sure others have better ideas to improve "pathMatch" as the suggested catch all parameter name.  "Match" is implied because it wouldn't be captured if it wasn't a match, so I suggest "Match" be removed. To make it more clear we could use "completePath", "catchAllPath", "anyPath" or even just "path" for the example without repeating.  For the example with repeating we could use "pathPart" or something.
Clarifying the motivation and recommendation to repeat
Regarding the recommendation of making the catch all parameter repeatable.  If the regex matches "anything" not just any chars between "/", this is not clear.  We could change "anything" to be "anything including slashes" to make that more obvious.
The justification is also confusing for two reasons.
First, why would making a param repeatable make a "." regex not match slashes?  That seems counter intuitive because "." regex matches slashes.  It also looks strange because the "*" and "?" are reused out of a regex context. but perhaps that is just an artifact of the api.
Second, why is this array preferable for programmatic navigation, can we can just navigating using fullPath instead of reconstituting the parameters?  Also not sure when the need for programmatic navigation from a 404 or match all route come up.
Thank for you reviewing. I may have misunderstood the documentation, but I hope the issue it motivates one or two small changes to improve that section's clarity.

Alternatively we could consider keeping the syntactic sugar '*' and instead of warning the user just make it into the proper regex route.


Add shift changes to the staff

6 am and 6 pm are shift changes (tentatively), 12 hour shifts. The greeter nurse only works from 9 to 5. There are one of three arrival times. Morning, Afternoon (swing), and Evening.

I will need to make slight changes to certain aspects of the medical agents that allowed them to have more than one of itself. For instance, I need to revamp how their initial room is assigned.

The janitor now has shift change functionality, and I should be able to bring this over to the other medical agents.
@bricksphd It's probably fine, but I was wondering how extensively our 'garbage collection' is for the simulation. I noticed that when agents are leaving, the node has to have "Running" rather than "Success" (because otherwise they don't disappear and try to do other behaviors in their tree), but I was wondering if it's possible that there's some leftover stuff happening or hanging that could be part of the slowdown further in the simulation?
A console.log shows it doesn't actually run the node more than once, so it's probably a misplaced worry, but might as well be thorough!

I'll take a look at this. I think that the architecture is getting large enough now that more review really is necessary--and hopefully will reveal some places for improvement.

Now everybody except the Attending, Pharmacist, Phlebotomist, and XRay have shift changes! They also change shifts at 6 pm.
@bricksphd I also noticed that when the new agents came in and the old agents left, the simulation sped up to near the speed the simulation was when it started.


Java indexer: emit name nodes for array types [phab:D483]

Created by schroederc at 2015-08-31 22:58:43:

zarko accepted this revision at 2015-08-31 23:05:57

schroederc closed via commit 22dcf22d1cfd2ac9c7dfe393c85f216f82fde118 at 2015-09-01 15:53:59


New bootstrap popover does not rerender template

popover.js line 84
// POPOVER PLUGIN DEFINITION
// =========================
function Plugin(option) {
return this.each(function () {
var $this   = $(this)
var data    = $this.data('bs.popover')
var options = typeof option == 'object' && option
  if (!data && option == 'destroy') return
  if (!data) $this.data('bs.popover', (data = new Popover(this, options)))
  if (typeof option == 'string') data[option]()
})

}
I had this working updated bs to newest version and the tours js increments correctly and render correct templates but seems to skip the rerendering of the popover's content.
if (!data) $this.data('bs.popover', (data = new Popover(this, options))) if I force this new instance it does work.
Mind you, I'm not entirely sure this is the direct cause of tours not rerendering the template

Same Issue here..

Figured it out orphans need a parent element to re render now just give it the .popover.tour class


[docs] Update "motivation" section

I'm having trouble parsing this sentence

Lighthouse enables many "disabled-by-default" tracing categories while TracerBench can be instrumented without any "disabled-by-default" and with minimal impact on your application; as such TracerBench instrumentation can be "check-in" and left in your application without worry of negative performance impacts.

Why should I care about "disabled-by-default" as a consumer of TracerBench?
Why does TB avoiding "disabled-by-default" result in me being able to check instrumentation code into my project safely?

I've updated the messaging around this in other docs. How does this sound:
When comparing TracerBench to the most popular tool Chrome Developer Tools Lighthouse. The primary difference is TracerBench is focused on getting a low variance for a metric across many samples versus getting a hard to replicate “Lighthouse performance report”. Lighthouse is essentially a black-box, with developers unable to customize performance parameters in-depth and lacking proper statistical rigor. TracerBench on the other hand, can be highly instrumented, provides statistical rigor and adequate sampling of data. Additionally, TracerBench instrumentation has minimal impact on the overhead of the application; as such TracerBench instrumentation can be "check-in" and left in your application without worry of negative performance impacts.


[TR3FIR-198] add flash control

Needed for proper arbitration of flash access between the two CPUs.

Rebased on top of current token-main. No changes.


Guide: Core Validators - links errors

On top of the page http://www.yiiframework.com/doc-2.0/guide-tutorial-core-validators.html
The list of core validator (from 1 to 23) should be links to the anchors on the same page only:

boolean

should be

boolean

This will make it easier to read the correct information on the guide part.

Agree on this. Would you like to submit a PR?

Well, I don't know how to do, and it doesn't seem trivial. Can you handle it?

Okay

Fixed, thanks for reporting!


Adds CILogon code into new home page.

@DanielJMaher Please review.  Code manually updated on uframe.ooi for testing.


Add @hyva/react-checkout alias

To make sure @hyva-reacht-checkout dependencies always work, like in payment methods, regardless wheter they are installed directly into a full checkout clone or a CheckoutExample implementation, we should add the alias here too.
We should probably copy the logic from https://github.com/hyva-themes/magento2-checkout-example/blob/master/reactapp/config-overrides.js.
Any thougths on this @rajeev-k-tomy ?

Great idea! Never thought about it. This will reduce lot of work such as keeping separate branches for example template modules as well as for the original repo.


Guild Snap package

This is just an idea - I'm not sure it's a good one! Snap has emerged as a leading cross-system package manager with solid adoption by software publishers. Having a system level package would let Guild users use Guild without installing it (and it's now many dependencies!) into their Python environment.
This issue could be spelled "Guild system packages" but in the interest of NOT building system-specific packages (at the moment anyway) Snap is a sensible target for Linux distros.

Maybe we could call this "guild deployment strategies" instead? I think providing Docker images belongs in this vein, also. I think once we get a good deployment pipeline set up, it will be transparent to provide as many of these things as we want.
Also in this vein: decouple guild from things that are used in the actual run environment. Guild (the task runner) should be able to have minimal dependencies, and should be able to interact with several ways of providing the runtime environment (e.g. virtualenv/pip, pipenv, poetry, conda, docker/kubernetes/container runtimes). As a system package manager, I don't think Snap fits here, but I could be wrong. In general, I think that avoiding the need for admin access to the user's system is always a good thing to try for.

It's pretty standard to system level packages to require admin/root access so I don't think that we should shy away from that. As a format for Linux distros Snap seems to have enough traction to leverage (as opposed to myriad distro-specific packages, repos, etc.) For macOS there's brew and mac ports and of course Windows Installer for Windows.
I don't know how many people install software using docker (as opposed to docker for specific cases, operational deployment, etc.) but if people want that that would fit into this broader vein.
We could rename this perhaps to "System packages" - I wasn't thinking of this stream as quite as high level as generalized deployment strategies, but just a way to bring Guild into the world of actual software (as opposed to forever a pip install). E.g. see DvC install.
Maybe we could create an RFC on my.guild.ai and then add issues here for specific package types?


setgid/setuid/chroot?

See stud's command-line options:
  -r  --chroot=DIR           Sets chroot directory (Default: "")
  -u  --user=USER            Set uid/gid after binding the socket (Default: "")
  -g  --group=GROUP          Set gid after binding the socket (Default: "")

Maybe use release for this: https://github.com/andrenth/release

Not sure what the point of this would be, but if you do need to sandbox it, it should probably be done in a thorough fashion:

Care should also be taken to close inherited file descriptors and get rid of shared memory segments (shm*()).
When setting uid/gid, euid and egid should be set accordingly
Under Linux:

seccomp-bpf whitelisting should be used to limit system calls
Namespaces should be used (clean mount and network namespaces, probably process namespace too)

Additional resources:

http://www.toptal.com/linux/separation-anxiety-isolating-your-system-with-linux-namespaces
https://github.com/servo/servo/wiki/Linux-sandboxing
https://l3net.wordpress.com/projects/firejail/firejail-usage/
https://www.chromium.org/chromium-os/chromiumos-design-docs/system-hardening

patches welcome. I also stumbled upon https://github.com/haesbaert/hdhcp/commit/de2c28fcecac92d224d795643062cb2c1a773442 recently


Some question about testing on windows without GPU（CUDA）

Thank you for making your code public. Can your code be reproduced on a CPU basis, and how to do it. Because I had this problem during testing:
Exception: No GPU found, please run without --cuda.
Looking forward to your reply.

With 8 GPUS， you have to train the model for over a week to reach convergence. With on cpu it would be too long to train this model (probably longer than 1 or 2 month).

Thank you for your reply. I understand. It is of great help to me.


add description when timeout occurs.

Some additional information would be useful why it failed.

https://github.com/IdentityModel/oidc-token-manager/issues/65


Request to have the system wide copy/paste menu in BlueBubbles

This might not be possible in Flutter unfortunately. Will look into it though.

This might not be possible in Flutter unfortunately. Will look into it though.

I don't know much about programming but I found this and this, let me know if they're any help

I don't know much about programming but I found this and this, let me know if they're any help

@CSab6482 Unfortunately, I don’t think either of those really help. The first one looks as if it just allows you to hook into the copy and paste functions and do something specific with the data. The second link is a similar piece of code.
This issue is more surrounding the fact that the Cupertino Text Field in Flutter doesn't use the system copy/paste menu. I'm not sure if it's different in the material theme (on the nightly). If it is different, then that's good info to know. But if it's the same, it seems like a Flutter thing and less of something I can influence. But I can try?

@CSab6482 Unfortunately, I don’t think either of those really help. The first one looks as if it just allows you to hook into the copy and paste functions and do something specific with the data. The second link is a similar piece of code.
This issue is more surrounding the fact that the Cupertino Text Field in Flutter doesn't use the system copy/paste menu. I'm not sure if it's different in the material theme (on the nightly). If it is different, then that's good info to know. But if it's the same, it seems like a Flutter thing and less of something I can influence. But I can try?

Relating to issue #683, the system clipboard is accessible from the notification reply. I'm not sure if this is helpful in any way, but it was nice to discover.

Relating to issue #683, the system clipboard is accessible from the notification reply. I'm not sure if this is helpful in any way, but it was nice to discover.

Gotta wait until Flutter does this themselves I think.... unfortunately


[Bug Report] color-picker does not fit in input-append

Element UI version
2.12
OS/Browsers version
all
Vue version
2.6.10
Reproduction Link
https://codepen.io/ray007/pen/GRKOLXz
Steps to reproduce
put a el-color-picker into the append slot of an el-input.
What is Expected?
input control and color picker fitting seamlessly together
What is actually happening?
color picker is too high and destroys the geometry

<el-color-picker size="small" slot="append"></el-color-picker>
try this

No, this falls apart if I reduce the size of the input.
Shouldn't input and color-picker of the same size fit together?

Oh, if this, that's will depends on the ElemeFE plan.
You can use it like that first.

Shouldn't this land on a todo-list instead of just closing the issue?


codegen: Properly mangle nested anonymous enums with duplicated variants.

r? @fitzgen

@bors-servo r+

:pushpin: Commit 59987f1 has been approved by fitzgen

I added a few more fixups, mind re-reviewing @fitzgen?

@bors-servo r+

:pushpin: Commit ce0a254 has been approved by fitzgen

:hourglass: Testing commit ce0a254 with merge 673cb5f...

:sunny: Test successful - status-travis


"Brădetu" is listed in the drop-down menu

Description:
"Brădetu" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Brădetu" in the search bar.

Expected results
The location is listed in the drop-down menu as "Brădetu, Argeș (Brăduleț)".


[ID-434]Phone password auth type

resolve #434

Based on the latest changes in the code, the current progress on this PR and the priority of this issue I am closing this PR. We will open a new one when the focus is on it.
cc @stefanvit


docs: add proposal to build table from json format

Currently there is an issue of row explosion when using the our RAII wrapper on a builder pattern.
Eg:
  irw::Table::Builder{}
    .with_id("Table")
    .with_number_of_columns(4)
    .with_headers({"Account/Category", "Remaining Budget", "Change", "Total"})
    .add_row({"Assets", "", {"$-778.32", Red}, "$29,754.72"})
    .add_row({"|- Cash", "", {"$-2.85", Red}, "$236.30"})
    .add_row({"|- Checking Account", "", {"$-634.99", Red}, "$7,327.01"})
    .add_row({"|- Euro Account", "", {"-120.00 EUR", Red}, "5,403.00 EUR"})
    .add_row({"|- Investment Account", "", "0 CNY", "20,400.00 CNY"})
    .add_row({"|- Savings Account", "", "$0.00", "$12,800.00"})
    .add_row({"Liabilities", "", {"$-200.00", Green}, "$11,400.00"})
    .add_row({"|- Car Loan", "", {"$-200.00", Green}, "$11,400.00"})
    .add_row({"Incomes", "-", "$0.00", "$18,853.50"})
    .add_row({"|- Other", "-", "$0.00", "$5,853.50"})
    .add_row({"|- Salary", "-", "$0.00", "$13,000.00"})
    .add_row({"Expenses", {"$226.67 of $460.00", Green}, {"$578.32", Red}, "$4,532.67"})
    .add_row({"|- Bills", "-", {"$30.00", Red}, "$3,092.00"})
    .add_row({"|- Clothing", {"$-40.48 of $100.00", Red}, {"$140.48", Red}, "$949.48"})
    .add_row({"|- Groceries", {"$74.15 of $160.00", Green}, {"$85.85", Red}, "$155.20"})
    .add_row({"|- Leisure", {"$193.00 of $200.00", Green}, {"$7.00", Red}, "$21.00"})
    .add_row({"|- Other", "-", {"$314.99", Red}, "$314.99"})
    .build();

If one could define a table via the means of a json file, one could have something like:
  irw::Table::Builder{}
    .from_file(<path to the json file>)
    .build();

In this document we will try to define such a format.

I mean if we just want to abstract the add_row thing we could create a rows csv I guess it's simpler, and would do the trick. but how are we going to manage the linking between ui and the info to be written there?

After discussing with Luiz, yeah the UI will be the same for every user, so we can have a csv for the values 😃

closing this as we decided to use csv, and then adapt if needed.


[feat] Allow to specify the file extention for externalBin

Describe the problem
In additional to executable files, static/dynamic library files are also platform & arch dependent. Need a way to embed external binaries for other extentions such as: .dll, .so,  beside executable files.
Describe the solution you'd like
Support externalBin setup as following:
"externalBin": [
    "binaries/my-sidecar"
]

And in additional:
"externalBin": [
    {path: "binaries/my-sidecar-dll", ext: "dll"}
]

Alternatives considered
No response
Additional context
No response

I'd say the resources feature is what you're asking for https://tauri.app/v1/guides/building/resources/. They can be platform specific via https://tauri.app/v1/api/config#platform-specific-configuration.
Your pr also only works on windows because linux and macos typically don't have library files next to executables.

whoops, didn't mean to close this before you have a chance to respond

Hi @FabianLars, thanks for the reponse. The resource feature won't help bescause the library file is also arch dependent like the excutable file. Please take the wintun.dll file as an example https://www.wintun.net/builds/wintun-0.14.1.zip. It has to keep the name of wintun.dll and it has different version for x86, amd64, arm.. unless resource feature could be not only platform specific  but also arch specific.
On linux, library search path is LD_LIBRARY_PATH specified path,  executable's rpath, /lib folder. User could set the LD_LIBRARY_PATH if the executable's rpath is not set to be the same folder to executables.

The resource feature won't help bescause the library file is also arch dependent like the excutable file

right, maybe we could expand the platform specific configs to also be arch specific (optionally) but i can't think of a different usecase than this issue so maybe that's a bit too much. Using the --config/-c flag works but having to use a manual step for this feels awkward too.

On linux, library search path is LD_LIBRARY_PATH specified path, executable's rpath, /lib folder. User could set the LD_LIBRARY_PATH if the executable's rpath is not set to be the same folder to executables.

That's why i said your solution doesn't work. In the .deb bundle the sidecars will be installed into /usr/bin where .so files don't belong. In the .appimage bundle the rpath and iirc the ld_library_path are set but either way appimages also have an internal /usr/bin/ folder where it wouldn't be as bad to have .so files in but still unusual.

So yes, i agree that we need something but i also think it needs a bit more thinking. Abusing the externalBin/sidecar config for this doesn't seem right.

I have use case for this.
I am building Windows only application on a Mac (ironic I know). UI does not need native lib, so I could build myself abstraction which calls sidecar only when on Windows and still works for most other stuff on other platforms. Right now that is not possible.
Allowing platforms to be optional would be great.

@minht11 Your use-case sounds like it's already possible: https://tauri.app/v1/api/config#platform-specific-configuration
This issue here is about specific configs for platform+architecture combinations, so for example different configs for windows-x86_64 and windows-aarch64. If you only build the app for windows x64 (so like 99% of windows users) the current system should work fine.


For images dropped into markdown, configure how the path should be created

When I drop an image into a markdown file, I get a relative path like this:
![](../../assets/images/image.jpg)

I would like to be able to define a root for the workspace so that the path to file can be absolute from this root. In my case I have an eleventy static site where I have a folder structure like this:
/website-folder/src/assets/images
/website-folder/src/posts/2022

If there were an option to define the "web" root for the workspace as:
/website/src

Then when dropping an image into a markdown file the path could be created like this:
![](/assets/images/image.jpg)

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Create an issue for Calculator

Can you open an issue for the opening and closing of the Calculator?
I will take care of that

Thanks khagapati-bagh


[question] Docker Compose & Docker in generel

Hy, me again
related on running Gekko on Docker there are some missunderstandings, aswell as the Docu could need a little Update.
Something i stumbled over the first 30 minutes lookin at it:

In the Compose File there is used: build: ./ - This is not supported anymore with Compose V3, Images have to be prebuilt. restart has to be specified under "deploy" Tag.
When using a SSL Terminating Proxy (Traefik) - reconnectiong--.....js seems to got some strange problems with that

If you want me to, i could a little Update related to Docker.
Goal: Everybody should be able to fire up Gekko+SSL-Rev-Proxy with a one-liner within 1 minute..
.... may my English need some corrections.. :)

ping @cassvail, he added this functionality in #947.

try docker stack deploy -c docker-compose.yml gekko will throw some errors cause as you can see above eg. build is not supported with v3

Hi @Berndinox if you want to deploy on a swarm/stack service, yes! the configuration needs an update... I think you can feel free to move the needed configs under deploy
I only used the config on local servers, so I didn't test cloud configs... if also the build tag is not allowd, probably a separated bash script for building could be created and separated from the docker-compose (I still see the build config in the docs tough https://docs.docker.com/compose/compose-file/)
At this point the image will still be built locally, if needed publishing on docker hub would be also a possibility I guess
Hope this helps


Make nicer settings cards

I have made a small change to make the settings cards look better

Thanks, I have a different plan for the Settings page however so I won't merge this.
I plan to make it similar to how Twitter does it, so even if I merge the PR, it's gonna be undone either way.

I understand


Tweak CLI download instructions to reflect file store and layout changes

@hannes-ucsc commented on Tue Aug 13 2019
Tweak https://data.humancellatlas.org/guides/quick-start-guide#using-the-cli-to-download-files-listed-in-the-manifest to reflect @jessebrennan's recent download changes.
For example, consider mentioning the --layout option and explaining the file store a little bit.

Beta-2 feedback requested improved cli documentation.

closing cli is being sunset


Getting System.NullReferenceException

I just added this package to my service and tested it. I am getting following exception just by adding this package dependency.

System.NullReferenceException: Object reference not set to an instance of an object.
at Microsoft.ApplicationInsights.Wcf.Implementation.WcfOperationContext.DiscoverOperationName(OperationContext operationContext)
at Microsoft.ApplicationInsights.Wcf.Implementation.WcfOperationContext..ctor(OperationContext operationContext)
at Microsoft.ApplicationInsights.Wcf.Implementation.WcfOperationContext.GetContext()
at Microsoft.ApplicationInsights.Wcf.Implementation.WcfInterceptor.BeforeSendReply(Message& reply, Object correlationState)
at System.ServiceModel.Dispatcher.ImmutableDispatchRuntime.BeforeSendReplyCore(MessageRpc& rpc, Exception& exception, Boolean& thereIsAnUnhandledException)

@rahulrajatsingh Could you tells us a little information?

What .NET Version is your code compiled for, and what .NET Framework version is it running no?
Are you using duplex contracts?
Does your service code manipulate OperationContext in any way during the operation execution?
What behaviors are you enabling on your service?

From the stack trace, it looks like the exception is coming up when processing the response message generated by the service... it's not finding the same OperationContext as the one that was present when the request message came in, so likely the context is in the wrong state to look up the necessary information.

OK, found one possible repro scenario: This same exception will happen if the service operation that was called explicitly or implicitly ends up calling another remote service.
However, it will only happen if the calling code instantiates a new OperationContextScope and does not dispose it correctly, so something like this:
var channel = new SomeServiceClient();
// the following line should be in a using statement
var scope = new OperationContextScope((IContextChannel)channel);
// use OperationContext for something
OperationContext.Current.OutgoingMessageHeaders.Add(new SomeCustomHeader());

// call service
channel.CallSomeOperation();

Code like this causes the OperationContext for the child call to be stuck on the current thread, so when we look at OperationContext.Current later on, we're getting the wrong context.

Please find all the requested details below:

What .NET Version is your code compiled for, and what .NET Framework version is it running no?

4.5.1

Are you using duplex contracts?

No

Does your service code manipulate OperationContext in any way during the operation execution?

Yes we are pushing claims in the OutgoingMessageHeaders for other service calls.

What behaviors are you enabling on your service?

No custom behavior. Its configured to use Single instance with Concurrency mode as Multiple and some throttling using configurations.

@rahulrajatsingh Can you confirm if your code suffers by any chance of the issue mentioned above of not disposing correctly the OperationContextScope object?


MongoDB Connector generates queries which do not take advantage of indices.

Bug description
Overview
We noticed while developing a new feature using Prisma & MongoDB Connector, that simple ID $in queries were taking 450ms+ to query a single ID.
The _id field is of course indexed in MongoDB, so you'd expect the query Prisma generates to be one which will take advantage of indexing, however...
The docs for $expr state:

Indexes are not used for comparisons where the operand is an array or the operand type is undefined.

So here, our $expr takes an array to match $in, and results in a COLLSCAN through our entire collection:
  "planSummary": [
    {
      "COLLSCAN": {}
    }
  ],
  "keysExamined": 0,
  "docsExamined": 44855,
  "cursorExhausted": 1,
  "numYields": 350

If we rewrite our $match query to be much simpler and not use $expr, it will successfully make use of the indices:
{
  "_id": { "$in": [ ObjectId("5ffe656eed39c24704277f66") ] }
}

Questions / Observations

We're not managing the Mongo DB in prisma, it's already setup and configured, with the necessary indices. This all works fine in mongoose.
I'm assuming it's not necessary to explicitly declare that an @id prisma property is also an @@index.
Is there anything we can do to force prisma to not use $expr? Why is the generated query using $expr?

How to reproduce

Generate the prisma client using the supplied example and mongodb connector.
Run the provided typescript query.
Observe the generated mongo query results in a COLLSCAN.

Expected behavior
prisma.user.findMany(
{ where: {
     id: {
       in: ['id1','id2'],
      },
})

Should take advantage of _id indices so that query performance is fast.
Prisma information
Prisma Collection
model User {
  id        String   @id @default(auto()) @map("_id") @db.ObjectId
  createdAt DateTime @default(now())
  updatedAt DateTime @updatedAt

  email        String    @unique
  deactivate   Boolean   @default(false)
  bounce       Boolean   @default(false)
  notification Json?
  bannedAt     DateTime?
  lineId       String?

  pushToken PushToken[]
  Lead      Lead[]

  @@index([lineId])
  @@map("users") // mongodb collection name.
}

Calling Code
  async searchUsers(
    searchUsersBodyDTO: SearchUsersBodyDTO
  ): Promise<GetUserDTO[]> {
    // Setup conditions
    const conditions: Prisma.UserFindManyArgs = {}
    if (searchUsersBodyDTO.ids?.length > 0) {
      conditions.where = {
        id: {
          in: searchUsersBodyDTO.ids,
        },
      }
    }
    // Execute
    return (await this.prisma.user.findMany(
      conditions
    )) as unknown[] as GetUserDTO[]
  }

Prisma Generated Aggregate Pipeline Stage
{
  "$expr": {
    "$and": [
      {
        "$in": [
          "$_id",
          {
            "$literal": [
              {
                "$oid": "5ffe656eed39c24704277f66"
              }
            ]
          }
        ]
      }
    ]
  }
}

Environment & setup

prisma version: 4.4.0
mongo version: 4.2.23

Prisma Version
4.4.0

This is the same as [this issue](https://github.com/prisma/prisma/issues/12793#issuecomment-11174840750.

This performance problem is fixed in Prisma 5.14.0


[EasyCodingStandard] check for missing properties

# easy-coding-standard.neon

checkers:
    SomeSniff:
         missingProperty: 10

Throw error!

Use case missing.

Resolved by  https://github.com/Symplify/Symplify/pull/179/commits/383f8d73e9bd2310191d4b14d9129177e77dbb03#diff-475d379ac3db504eac75a51cc55bfceaR97


Using the latest tricks

Check List:

[X] Tests pass (./test.sh <username> shows no differences between expected and actual outputs)
[X] All formatting changes by the build are committed
[X] Your launch script is named calculate_average_<username>.sh (make sure to match casing of your GH user name) and is executable
[X] Output matches that of calculate_average_baseline.sh
[X] For new entries, or after substantial changes: When implementing custom hash structures, please point to where you deal with hash collisions (line number)

First changed the code to use Artsiom's ByteBuffer, but I've noticed Thomas has an even better idea, so stealing that :-)
All other changes pretty much reverted.. I'd love to make the code simpler, but changing it to use a Predicate<byte[]> for updating the table makes things worse, which is kind of the opposite point of the challenge, so not doing that.
Perhaps a MethodHandle or something could be faster, don't know...

00:02.313 now, nice!
Benchmark 1: timeout -v 300 ./calculate_average_royvanrijn.sh 2>&1
  Time (mean ± σ):      2.312 s ±  0.021 s    [User: 0.002 s, System: 0.003 s]
  Range (min … max):    2.270 s …  2.342 s    10 runs

Summary
  royvanrijn: trimmed mean 2.31353707266, raw times 2.27034278566,2.32108241566,2.29975882466,2.32893833366,2.30389421566,2.34196373966,2.30766307666,2.30471392566,2.33684425966,2.30540152966

Leaderboard

| # | Result (m:s.ms) | Implementation     | JDK | Submitter     | Notes     |
|---|-----------------|--------------------|-----|---------------|-----------|
|   | 00:02.313 | [link](https://github.com/gunnarmorling/1brc/blob/main/src/main/java/dev/morling/onebrc/CalculateAverage_royvanrijn.java)| 21.0.2-graal | [Roy van Rijn](https://github.com/royvanrijn) | GraalVM native binary |


Remove dependency of Jupyter notebook, so that can run in CLI

This is a duplicate of #2, now we can run in CLI mode


JOSS Review Overview

I am going to keep a list/links to individual issues here for the JOSS review to help keep things organized for myself as I go through. This is well put together - much of this will be stuff that I'd like to see as a user and a few things that stand out from the developer side.
I'll ping you here when I am done with my initial review.
Should be done:

[x] Minor text edits & cite networkx - #3
[x] Fix duplicate inputs bug - #8
[x] Custom GAF index use unclear - #12
[x] Document custom index should be made from gzipped GAF - #11

Would prefer to see done:

[x] Host web server - #4
[x] Utilize bottle via imports rather than inclusion in source - #5
[x] High resolution image download from GUI - #6
[x] Download annotations table directly from GUI - #10
[x] Leaf node coloring when filtering applied - #13

Worth considering:

[x] Support other gene identifiers - #7

Nitpicks/oddities:

[x] Input box resizing - #9

Alright @ialbert, I think I'm done with a first pass. All of my points have individual issues, so I'm happy to discuss any of them individually. Many are quick fixes, a few are questions - just let me know if anything is unclear.

@j-andrews7 all issues addressed,
thanks for the suggestion for using Shiny - it worked out really well and I learned a new skill, that of developing Shiny apps with Python.

Okay, I'm pretty happy with the changes. Will mark my review complete over on the JOSS issue as well. Nice job on the quick fixes.


Change top box flap behavior for responsiveness

Overarching user story:
As a disorganized person who's moving
I want a system to track what's in each box
So that I know what's in each box before I open it and scatter the contents
Issue user story:
As a disorganized person who's moving
I want to use a simple, visually-pleasing website
So that I can keep track of my boxes and their contents as I move
Acceptance criteria:
Given that user opens the a page with a box with flaps in it
The user is greeted with a responsive design
So that the user experience is fluid

already done

already done


Improve consistency of column names

PR created for discussion and continued work until it feels ready.

A majority of the ex.json file is very consistent and in excellent shape, this PR aims to meet a 100% shared param name for common names (eg: ClassJobLevel vs Level)
Recipes:

UnlockKey > SecretRecipeBook, would this make more sense? Most columns that link to another field (eg Placename, or RecipeLevelTable etc) use the name of the table they link to vs a "custom name".

Character Level / Class Job Level / "Level"
There can be areas of confusion when relating to the "Level", I think changing all params that do not read the "Level" CSV should be named more explicitly. Some examples:

ClassJobLevel
RetainerLevel
ItemLevel

Changes so far:

All "character" related levels changed to: ClassJobLevel

Some to think about, I'm unsure on

FishingSpot and FishParameter both have a "Level" but unsure if this is class level? I don't do fishing!
GatheringItemLevelConvertTable and GatheringPointBase both have a Level column.
MasterpieceSupplyDuty has MinLevel, should this be ClassJobLevel{Min} ? (it also has a MaxLevel field)
SpearfishingNotebook has a Level field, this a class level?

Thanks, I agree with all these changes including SecretRecipeBook.
FishingSpot, GatheringPointBase, and SpearfishingNotebook Level are the display level of that specific gathering point.  Definitely not character levels.  FishParameter Level is some other level value and I'm not sure what its use is beyond (likely) spirit bonding.  Probably to calculate difficulty and bait as well.
MasterpieceSupplyDuty Min/MaxLevel are definitely ClassJobLevels.

For Fishing/Gathinering we could use: NodeLevel as you have Gathering Nodes etc.
For FishParameter I was thinking DifficultyLevel ? Maybe, it goes up to 300+ and ItemLevel wouldn't really fit the meaning.
For GatheringItemLevelConvertTable is that level an ItemLevel?

I'm just thinking of the rules of linked content, so some examples:
Companion
There is:

Column: Behavior links to CompanionMove
Column: MinionRace links to MinionRace

It makes sense why 1 is not named CompanionMove because we (as dataminers) know "Companion" is a bit misleading and SE may fix this (as they've started adding Minion files), and no one refers to the Behavior as "MinionMove", as that isn't how it is in game. SO Behavior makes sense.
I wonder if it should be MinionBehavior to match MinionRace. The idea is that columns that are customised (eg do not match their linked file) always contain a Subject, eg:

FCRank subject = FC - matches file
CompanyCraftType subject = Company - matches file
AchievementKind subject = Achievement - matches file
SecretRecipeBook could be: RecipeMasterBook if we named it differently than the file

How about GatheringLevel for nodes and fishing spots (not the items in them; the nodes themselves)?
GatheringItemLevelConvertTable: Those are close to item levels but there are some deviations.  It also covers EventItem rows that do not have item levels.  GatheringItemLevel is probably the most accurate umbrella for these.  Perhaps FishParameter levels should be GatheringItemLevel too.


Refactor IBMCLOUD_API_KEY references

Addresses the git issue https://github.com/ppc64le-cloud/pvsadm/issues/666

/ok-to-test

/lgtm
Thanks @manju956 for the changes. :)


Fix readme code

The example code is broken.

Thanks @victorcoder, good catch, sorry for the late answer. Can you please squash and sign-off your commit?

Done @abronan

Thanks! LGTM


welcome back enter password screen - EditText curser does not move

this is not an issue, please close.


v1: algolia search result display issue

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Prerequisites

[X] I'm using the latest version of Docusaurus.
[X] I have tried the npm run clear or yarn clear command.
[X] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[ ] I have tried creating a repro with https://new.docusaurus.io.
[ ] I have read the console error message carefully (if applicable).

Description
I am using Docusaurus v1.14.7. I've been using Algolia DocSearch for awhile now and recently received email about DocSearch being migrated to their new infrastructure. That means I need to update DocSearch configuration for my site. After updating the configuration, I noticed there's an issue with the way search result is being displayed.
I think this bug is from Algolia's new index or their docsearch UI. I'm not sure where to raise this bug. This issue seems to be affecting Docusaurus v1 site also. I expect others might have the same issue.
Here's a screenshot of what the search UI looks like from Docusaurus v1 site:

I've highlighted the issue in the screenshot above. It seems the UI is concatenating everything from sidebars. It makes it very hard to understand where the search result is coming from compared to how it was before.
Steps to reproduce
n/a
Expected behavior
The search result UI to show results same way as it did before DocSearch migration.
Actual behavior
Search result UI has a bug. It combines everything from sidebars into the search result's UI instead of only showing the sidebar category where the result was found.
Your environment

Public source code: https://github.com/facebook/docusaurus
Public site URL: https://v1.docusaurus.io/docs/en/site-preparation
Docusaurus version used: 1.14.7
Environment name and version (e.g. Chrome 89, Node.js 16.4): Firefox 95.0.2
Operating system and version (e.g. Ubuntu 20.04.2 LTS): MacOS 12.0.1

Reproducible demo
https://v1.docusaurus.io/docs/en/site-preparation
Self-service

[ ] I'd be willing to fix this bug myself.

@shortcuts do you have any why this happens?

I'd say there's an issue with the lvl0 selector, we used xPath before but the equivalent might not properly work. Checking

Hey
I've fixed the lvl0, you should see proper results on Docusaurus v1. The doc is also updated: https://github.com/algolia/docsearch/commit/bfde46e140d368dcebb0f424f087a27b2ce7c04f

Thanks! I can see correct output now. Will close this if there are no further problems

Hey
I've fixed the lvl0, you should see proper results on Docusaurus v1. The doc is also updated: algolia/docsearch@bfde46e

@shortcuts - Thanks for looking into it and the fix for Docusaurus website. As I mentioned earlier, I manage multiple sites that are based on Docusuaurus V1. How do I get this fixed for my sites? Should I be applying that change through Algolia Crawler configuration myself for each of the sites?
I'm still somewhat new to the whole setup/docsearch-migration. I'd appreciate if you could clarify. Thank you.

How do I get this fixed for my sites? Should I be applying that change through Algolia Crawler configuration myself for each of the sites?

Hey, I'm out of office until next Monday so I can't apply changes myself on current Docusaurus v1 crawlers. The fastest would be if you could edit your configs directly from the web interface.
cc @bodinsamuel

I was able to make that change. I believe other sites built using Docusuarus V1 will have to do the same.

Go to crawler.algolia.com
Open the crawler for your site
Go into Editor
Apply the change according to the one done in Docusaurus' website crawler here https://github.com/algolia/docsearch/commit/bfde46e140d368dcebb0f424f087a27b2ce7c04f
Save the configuration and initiate a new crawl by clicking the "Restart crawling"

@shortcuts - When you're back, please let me know if there is anything else I need to do or if I missed anything.


App crashing after showInterstitial on Android

Describe the bug
I have an app that is showing an Interstitial with AdMob.showInterstitial();
After the Ad is shown completely and the user is closing the Ad the app is crashing.
Error message from Android Studio:
FATAL EXCEPTION: main                                                                                Process: xx.xxxxxxxx.xxxxxxxxxx, PID: 20558                                                                                java.lang.NullPointerException: Attempt to invoke virtual method 'void com.google.android.gms.ads.AdView.loadAd(com.google.android.gms.ads.AdRequest)' on a null object reference                                                                                at com.getcapacitor.community.admob.banner.BannerExecutor.lambda$updateExistingAdView$3(BannerExecutor.java:218)                                                                                at com.getcapacitor.community.admob.banner.BannerExecutor.$r8$lambda$rG8pevl4TYjNjEUjyIXkgzJ2UsA(Unknown Source:0)                                                                                at com.getcapacitor.community.admob.banner.BannerExecutor$$ExternalSyntheticLambda1.run(D8$$SyntheticClass:0)                                                                                at android.os.Handler.handleCallback(Handler.java:938)                                                                                at android.os.Handler.dispatchMessage(Handler.java:99)                                                                                at android.os.Looper.loopOnce(Looper.java:233)                                                                                at android.os.Looper.loop(Looper.java:334)                                                                                at android.app.ActivityThread.main(ActivityThread.java:8396)                                                                                at java.lang.reflect.Method.invoke(Native Method)                                                                                at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:582)                                                                                at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1068)
Desktop (please complete the following information):

OS: macOS Sanoma 14.2.1 (23C71)
Android Studio Android Studio Iguana | 2023.2.1

Smartphone (please complete the following information):

Device: vivo 1910
OS: Funtouch OS 10.5 (Android 12)

You are calling a show banner without preparing it.

Hey thanks for your answer.
For some reason I was so blind yesterday. Trying different things for hours.
And today after your message I found it immediately.
For some reason I have an listener for the appStateChange in my app.component (App.addListener('appStateChange'))
Where I remove the banner. And this listener is called when the Interstitial gets active.
I have to figure out why I have this listener. but it's working now.
thanks! 👍


Unable to start the rasa UI

I face below error while starting the software. Please find attached file for debugging info.
><EMAIL_ADDRESS>start /home/harshit/Downloads/RasaUI/rasaui
> node server/server.js

## There is an issue with `node-fibers` ##
`/home/harshit/Downloads/RasaUI/rasaui/node_modules/fibers/bin/linux-x64-64/fibers.node` is missing.

Try running this to fix the issue: /usr/local/bin/node /home/harshit/Downloads/RasaUI/rasaui/node_modules/fibers/build
/home/harshit/Downloads/RasaUI/rasaui/node_modules/fibers/fibers.js:20
	throw new Error('Missing binary. See message above.');
	^

Error: Missing binary. See message above.
    at Object.<anonymous> (/home/harshit/Downloads/RasaUI/rasaui/node_modules/fibers/fibers.js:20:8)
    at Module._compile (internal/modules/cjs/loader.js:678:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:689:10)
    at Module.load (internal/modules/cjs/loader.js:589:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:528:12)
    at Function.Module._load (internal/modules/cjs/loader.js:520:3)
    at Module.require (internal/modules/cjs/loader.js:626:17)
    at require (internal/modules/cjs/helpers.js:20:18)
    at Object.<anonymous> (/home/harshit/Downloads/RasaUI/rasaui/node_modules/asyncawait/src/fibers.js:1:75)
    at Module._compile (internal/modules/cjs/loader.js:678:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:689:10)
    at Module.load (internal/modules/cjs/loader.js:589:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:528:12)
    at Function.Module._load (internal/modules/cjs/loader.js:520:3)
    at Module.require (internal/modules/cjs/loader.js:626:17)
    at require (internal/modules/cjs/helpers.js:20:18)

npm ERR! Linux 4.4.0-124-generic
npm ERR! argv "/usr/bin/nodejs" "/usr/bin/npm" "start"
npm ERR! node v4.2.6
npm ERR! npm  v3.5.2
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>start: `node server/server.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script 'node server/server.js'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the RasaUI package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node server/server.js
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs RasaUI
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls RasaUI
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     [/home/harshit/Downloads/RasaUI/rasaui/npm-debug.log]

[npm-debug.log](https://github.com/paschmann/rasa-ui/files/2026232/npm-debug.log)

I am using v10 of node js.  Please find below terminal output
harshit@harshit-G41MT-S2:~/Downloads/RasaUI/rasaui$ node -v
v10.1.0

@harshitazilen Has your problem been solved?

@yuxuan2015 , no It is still there. I am unable to start.

Is linux-x64-64 in your directory, my directory is linux-x64-46?

I could find below directories
linux-x64-46
linux-x64-48
linux-ia32-46
linux-ia32-48

@pradeepmvn My directory  is linux-x64-46,not linux-x64-64.Do you know what happend?

@harshitazilen https://github.com/clofus/clofus-chatbot,another rasa-UI

This is particular to node v10 with fibers module. I was able to reporoduce it. Temporary solution would be to roll back to node V9 and do npm install. I tested this and works fine.


C4 Shapes have superfluous characters in the technology field

Preflight Checklist

[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

You must agree to search and the code of conduct. You must fill in this entire template. If you delete part/all or miss parts out your issue will be closed.
If you are technical, you should reporting bugs along the lines of https://marker.io/blog/how-to-write-bug-report. If you are not technical, we will make allowances, please try to make an effort to understand the process.
Describe the bug
Some C4 shapes have erroneous charters in the technology section. This includes but not limited to the database shape.
To Reproduce
Steps to reproduce the behavior:

draw.io
click on shapes
select c4
drag database shape onto the page.

Expected behavior
Doesn't have the additional characters.
Screenshots

draw.io version (In the Help->About menu of the draw.io editor):

draw.io version 24.2.0

Desktop (please complete the following information):

OS: OSX
Browser: Chrome
Browser Version: 123.0.6312.59

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

I tested the problem in incognito/private mode with all browser extensions switched off, write "yes" below:

Yes

Additional context
Add any other context about the problem here.

Fix in 24.2.1.


Update description on docs

This is a README change. Merging.

It's not "Google's", it's just that Google is the main employer of Flutter's contributors by volume. We have contributions from many people outside Google as well. I think it's fine to say it's "supported by Google" or things like that, but to say it's "Google's" IMHO unfairly dismisses the contributions of a large and growing community.


[Event Request] codeunit 1535 "Approvals Mgmt." - OnFindApprovalEntryForCurrUserOnAfterApprovalEntrySetFilters

Describe the request
procedure FindApprovalEntryForCurrUser(var ApprovalEntry: Record "Approval Entry"; RecordID: RecordID): Boolean
begin
    ApprovalEntry.SetRange("Table ID", RecordID.TableNo);
    ApprovalEntry.SetRange("Record ID to Approve", RecordID);
    ApprovalEntry.SetRange("Approver ID", UserId);
    OnFindApprovalEntryForCurrUserOnAfterApprovalEntrySetFilters(ApprovalEntry);

    exit(ApprovalEntry.FindFirst());
end;

[IntegrationEvent(false, false)]
local procedure OnFindApprovalEntryForCurrUserOnAfterApprovalEntrySetFilters(var ApprovalEntry: Record "Approval Entry")
begin
end;

Additional context
MissingInternal work item: AB#493928

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 14883.


Add Jenkinsfile whitelist

Signed-off-by: Adam Ludvik<EMAIL_ADDRESS>

This build is before I put all the names in the whitelist: https://build.sawtooth.me/job/Sawtooth-Hyperledger/job/sawtooth-core/job/PR-171/6/console
This build is after: https://build.sawtooth.me/job/Sawtooth-Hyperledger/job/sawtooth-core/job/PR-171/7/console

Could you add a little context to the commit message?  Whitelist for...?

Added more description to commit message.


fix: the missing of llava_shared library.

The osx llava libraries should be included on the cpu package.


Add casr-lua

@hkctkuy, can you, also, please add casr-lua calls into casr-libfuzzzer. Thus, output from luzzer could be triaged.

@ligurio, can you help testing Lua crash analysis added in this PR?

@SweetVishnya Sure, will do after weekend.


How does versioning work for waterline-sql-builder?

Looking at the releases at https://github.com/sailshq/waterline-sql-builder/releases, we have the latest listed as v1.0.0, and previous to that as v1.0.0-7.
Is 1.0.0-7 prior to 1.0.0?  Does the -7 suffix imply a non-production release?
I've seen similar version numbers in other sails-related nodejs modules; is the same convention followed for all of them?

Hey, @alxndrsn! The hyphen in a version name just indicates a pre-release. I believe this is conventional for all npm packages.

I believe this is conventional for all npm packages.

Of course, sorry for not realising!  I've never seen a number suffix without a word indicating what the number means (e.g. alpha, rc, next); interesting convention.

No problem! It's hard to keep track of all the minutiae, especially with everything changing all the time. Glad to be of assistance.


Inactive categories

Coverage decreased (-3.3%) to 72.875% when pulling 48950afc61536f817853fb7bc351ee87f0c67bca on AnjaneyuluBatta505:InactiveCategories into 72cdc256d24e6b6717d3cc326c6a1b700ad3df0f on MicroPyramid:master.


High increase in indexer queue used percentage when switching to otel

We've been using splunk connect for kubernetes for some time.
We've tried switching to otel, however when we switched we saw some increase in the GBs ingested coming from kubernetes, along with a very big increase in the % used rate for the queues in our indexers.
We are wondering where this increase in indexer load might be coming from. The ingestion in GBs doesn't seem big enough to cause the huge increase in queue usage that we are seeing.
Here's a hypothesis: We've noted that otel by default sends some new metadada to splunk, in particular this field: k8s.container.restart_count, that gingested as an indexed field, at index time.
Is it possible that by trying to index k8s.container.restart_count or other metadata as an indexed field, splunk is suffering a higher load?
Is it possible to stop otel from sending metadata? Our alternatively, is it possible to make splunk ignore this metadata?
Thanks in advance

Hello! I have seen your original post in https://github.com/signalfx/splunk-otel-collector-chart/issues/503. I am going to close this issue and will follow up there.


Refused to execute inline script because it violates the following Content Security Policy directive: "script-src 'self'

Deploying this to a working cloudfront distribution is causing inline javascript to fail. HTML page renders properly but javascript code does not. All scripts are local to the s3 bucket not from any external urls. Any help is much appreciated.
Refused to execute inline script because it violates the following Content Security Policy directive: "script-src 'self' https://code.jquery.com https://stackpath.bootstrapcdn.com". Either the 'unsafe-inline' keyword, a hash ('sha256-sSYP07KSXyZ0gQue3jDWS5nZaWUz7V/V0566xsjP90k='), or a nonce ('nonce-...') is required to enable inline execution.

Hi @pandrel
The default HTTP headers that this solution sets include a quite strict CSP––which seems to hinder you:
HttpHeaders:
    Type: String
    Description: The HTTP headers to set on all responses from CloudFront. To be provided as a JSON object
    Default: >-
      {
        "Content-Security-Policy": "default-src 'none'; img-src 'self'; script-src 'self' https://code.jquery.com https://stackpath.bootstrapcdn.com; style-src 'self' 'unsafe-inline' https://stackpath.bootstrapcdn.com; object-src 'none'; connect-src 'self' https://*.amazonaws.com https://*.amazoncognito.com",
        "Strict-Transport-Security": "max-age=31536000; includeSubdomains; preload",
        "Referrer-Policy": "same-origin",
        "X-XSS-Protection": "1; mode=block",
        "X-Frame-Options": "DENY",
        "X-Content-Type-Options": "nosniff"
      }

The easiest way to get going fast: redeploy the stack again (causing a stack update), this time provide the value "{}" for parameter "HTTPHeaders". This effectively removes the default HTTP headers.
The right way forward: determine which CSP headers you need exactly, and pass those in as HTTPHeaders.
Let me know if that helps.

You are awesome. I updated the stack by adding 'unsafe-inline' in the script-src section, reconfigured cloudfront behaviors with new version of the lambda and everything worked fine.
FYI .. i am using existing cloudfront distribution so had to manually update the behaviors.
Thanks so much for your help. !!!

In case anybody comes across this, here are two header sets you can feed to the CFN stack that I've found useful until you can develop a real working CSP.
#1 - this set puts CSP in report-only mode.  It will moan about every violation, but still let you run.  Very useful in the process of tracking down problems on the way to a valid CSP.  Provides no CSP protection.
{
  "Content-Security-Policy-Report-Only": "default-src 'none'; img-src 'self'; script-src 'self'; style-src 'self'; object-src 'none'; connect-src 'self' https://*.amazonaws.com https://*.amazoncognito.com",
  "Strict-Transport-Security": "max-age=31536000; includeSubdomains; preload",
  "Referrer-Policy": "same-origin",
  "X-XSS-Protection": "1; mode=block",
  "X-Frame-Options": "DENY",
  "X-Content-Type-Options": "nosniff"
}

#2 This policy has no CSP but retains other important security items like transport security, referrer policy, etc.  No CSP protection, no report warnings:
{
  "Strict-Transport-Security": "max-age=31536000; includeSubdomains; preload",
  "Referrer-Policy": "same-origin",
  "X-XSS-Protection": "1; mode=block",
  "X-Frame-Options": "DENY",
  "X-Content-Type-Options": "nosniff"
}


Fix docs and comments

Some suggestions I thought of while reading over the docs. I'm not sure if all of these changes are correct or even wanted, so feel free to only keep the ones you like, or let me know if I should change this PR in any way.
Thanks for making this library.

LGTM thanks!

👍 Glad I could help.


Support for all Cordova plugins

I'm considering Crosswalk, and I'm wondering why only few Cordova plugins are supported. Is there any technical reason why Crosswalk can't support all Cordova plugins? It seems to me that theoretically it can support any plugin and it's just a matter of integration tools missing. Am I right?

At the moment Crosswalk for iOS is not integrated with the Cordova infrastructure. Crosswalk has its own extension mechanism, supported on all platforms including iOS, and as a convenience we provide a bridge between Crosswalk extensions and Cordova plugins. However not all the features available to Cordova plugins work when running as a Crosswalk extension, and that's why not all plugins are supported. We don't plan to spend any effort in trying to make additional plugins to work.
If Cordova plugins are required, an alternative way is to use the WkWebView webview plugin implemented by the Cordova project. This is not that different than using Crosswalk for iOS, because on iOS we are restricted by the App Store terms to use the WebKit web engine that comes with the platform. I haven't tried the plugin myself, but given that it's maintained by the Cordova project, I assume that it supports all existing plugins. We actually started to implement a crosswalk-cordova iOS webview plugin ourselves, but it didn't really add much value compared to the WkWebview plugin and so we abandoned that effort.
@jondong anything to add?

Thanks for your answer, so it's not a real technical issue. From what I understand the problem is how the cordova add plugin CLI works and the problem of integrating with their CDVViewController and their structure in general that is quite rigid.
To clarify I'm already using Cordova + WKWebView plugin, I just want to add support for crosswalk plugins too.

I just want to add support for crosswalk plugins too.

Could you clarify what you mean by that? Are you trying to use some Crosswalk extensions (i.e. APIs) in your app? Which ones?

It's quite a complicated project, but to sum up, imagine a container native app that runs custom js code.
And you can add any plugin coming from both Cordova and Crosswalk and use them in the js without ever seeing the Xcode project. (so adding/removing plugins will be a completely automated process)


Refactored addresses update, and updating these for constructor.

Stack:

#3897
#3896 ⬅

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

This change is

crates/cairo-lang-runner/src/casm_run/mod.rs line 883 at r1 (raw file):

        let (res_data_start, res_data_end) = if let Some(constructor) = &contract_info.constructor {
            let old_addrs =
                self.starknet_state.open_caller_context(deployed_contract_address.clone());

this should be based on deploy\_from\_zero
https://github.com/starkware-industries/starkware/blob/afdb639f5ed71322630c6f4c48d8cc2065bd0ece/src/starkware/starknet/core/os/execution/execute_syscalls.cairo#L472
Code quote:
deployed_contract_address.clone()


Memory leak while looping through a Dataset

I'm encountering a detrimental memory leak when simply accessing data from a Dataset repeatedly within a loop. I'm opening netCDF files concatenated in time, and looping through time to create plots. In this case the x-y slices are about 5000 x 5000 in size
import xarray as xr
import os, psutil
ds = xr.open_mfdataset('*.nc', chunks={'x': 4000, 'y': 4000}, concat_dim='t')
for k in range(ds.dims['t']):
    data = ds.datavar[k,:,:].values
    print('memory=', process.memory_info().rss)

>>> memory= 566484992
>>> memory= 823836672
>>> memory= 951439360
>>> memory=<PHONE_NUMBER>

I tried explicitly dereferencing the array by calling del data at the end of each iteration, which reduces the memory growth a little bit, but not much.
Strangely, in this simplified example I can greatly reduce the memory growth by using much smaller chunk sizes, but in my real-world example, opening all data with smaller chunk sizes does not mitigate the problem. Either way, it's not clear to me why the memory usage should grow for any chunk size at all.
ds = xr.open_mfdataset('*.nc', chunks={'x': 1000, 'y': 1000}, concat_dim='t')
>>> memory= 514043904
>>> memory= 499363840
>>> memory= 502509568
>>> memory= 522133504

I can also generate memory growth when cutting dask out entirely with open_dataset(chunks=None) and simply looping through different variables in the Dataset:
ds = xr.open_dataset('data.nc', chunks=None) # x-y dataset 5424 x 5424
for var in ['var1', 'var2', ... , 'var15']:
    data = ds[var].values
    print('memory =', process.memory_info().rss)

>>> memory = 246087680
>>> memory = 280604672
>>> memory = 285810688
>>> memory = 315834368
>>> memory = 344510464
>>> memory = 374530048
>>> memory = 403742720
>>> memory = 403804160
>>> memory = 404140032
>>> memory = 403660800
>>> memory = 404262912
>>> memory = 403513344
>>> memory = 404115456
>>> memory = 403636224

Though you can see that, strangely, the growth stops after several iterations. This isn't always true. Sometimes it asymptotes for a few interations and then begins growing again.
I feel like I'm missing something fundamental about xarray memory management. It seems like a great impediment that arrays (or something) read from a Dataset are not garbage collected while looping through that Dataset, which kind of defeats the purpose of only accessing and working with the data you need in the first place. I have to access rather large chunks of data at a time, so being able to discard that slice of data and move onto the next one without filling up the RAM is a big deal.
Any ideas what's going on? Or what I'm missing?

print(ds) # from open_mfdataset()
<xarray.Dataset>
Dimensions:                                 (band: 1, number_of_image_bounds: 2, number_of_time_bounds: 2, t: 4, x: 5424, y: 5424)
Coordinates:

y                                       (y) float32 0.151844 0.151788 ...
x                                       (x) float32 -0.151844 -0.151788 ...
t                                       (t) datetime64[ns] 2018-05-25T00:36:02.796268032 ...
Data variables:
data                                 (t, y, x) float32 dask.array<shape=(4, 5424, 5424), chunksize=(1, 4000, 4000)>

INSTALLED VERSIONS
commit: None
python: 3.6.5.final.0
python-bits: 64
OS: Linux
OS-release: 4.16.8-300.fc28.x86_64
machine: x86_64
processor: x86_64
byteorder: little
LC_ALL: None
LANG: en_US.UTF-8
LOCALE: en_US.UTF-8
xarray: 0.10.4
pandas: 0.22.0
numpy: 1.14.3
scipy: 1.1.0
netCDF4: 1.4.0
h5netcdf: 0.5.1
h5py: 2.8.0
Nio: None
zarr: None
bottleneck: 1.2.1
cyordereddict: None
dask: 0.17.4
distributed: 1.21.8
matplotlib: 2.2.2
cartopy: 0.16.0
seaborn: None
setuptools: 39.1.0
pip: 10.0.1
conda: None
pytest: None
IPython: 6.4.0
sphinx: None

The memory management here is handled by python, not xarray.  I know that doesn't help solve your problem...

Yes, I understand the garbage collection. The problem I'm struggling with here is that normally when working with arrays, maintaining only one reference to an array and replacing the data that reference points to within a loop does not result in memory accumulation because GC is triggered on the prior, now dereferenced data.
Here, it seems that under the hood, references to arrays have been created other than my "data" variable that are not being dereferenced when I reassign to "data," so stuff is accumulating in memory.

I'm now wondering if this issue is in dask land, based on this issue: https://github.com/dask/dask/issues/3247
It has been suggested in other places to get around the memory accumulation by running each loop iteration in a forked process:
def worker(ds, k):
    print('accessing data')
    data = ds.datavar[k,:,:].values
    print('data acquired')

for k in range(ds.dims['t']):
    p = multiprocessing.Process(target=worker, args=(ds, k))
    p.start()
    p.join()

But apparently one can't access dask-wrapped xarray datasets in subprocesses without a deadlock. I don't know enough about the internals to understand why.

I've discovered that setting the environment variable MALLOC_MMAP_MAX_ to a reasonably small value can partially mitigate this memory fragmentation.
Performing 4 iterations over dataset slices of shape ~(5424, 5424) without this tweak was yielding >800MB of memory usage (an increase of ~400MB over the first iteration).
Setting MALLOC_MMAP_MAX_=40960 yielded ~410 MB of memory usage (an increase of only ~130MB over the first iteration).
This level of fragmentation is still offensive, but this does suggest the problem may lie deeper within the entire unix, glibc, Python, xarray, dask ecosystem.

I might try experimenting with setting autoclose=True in open_mfdataset(). It's a bit of a short in the dark, but that might help here.
Memory growth with xr.open_dataset('data.nc', chunks=None) is expected, because by default we set cache=True when not using dask. This means that variables get cached in memory as NumPy arrays.

Using autoclose=True doesn't seem to make a difference. My test only uses 4 files anyway.
Thanks for the explanation of open_dataset() - that makes sense.

@meridionaljet I might've run into the same issue, but I'm not 100% sure. In my case I'm looping over a Dataset containing variables from 3 different files, all of them with a .sel and some of them with a more complicated (dask) calculation. (still, mostly sums and divisions) The leak seems mostly happening for those with the calculation.
Can you see what happens when using the distributed client?  This leads to many distributed.utils_perf - WARNING - full garbage collections took 40% CPU time recently (threshold: 10%) messages being shown for me, indeed pointing to something garbage-collecty.
Also, for me the memory behaviour looks very different between the threaded and multi-process scheduler, although they both leak. (I'm not sure if leaking is the right term here). Maybe you can try memory_profiler?
I've tried without succes:

explicitly deleting ds[varname] and running gc.collect()
explicitly clearing dask cache with client.cancel and client.restart
Moving the leaky code in its own function (should not matter, but I seemed to remember that it sometimes helps for garbage collect in edge cases)
Explicitly triggering computation with either dask persist or xarray load and then explicitly deleting the result

For my messy and very much work in process code, look here: https://github.com/Karel-van-de-Plassche/QLKNN-develop/blob/master/qlknn/dataset/hypercube_to_pandas.py

This might be the same issue as https://github.com/dask/dask/issues/3530

Hey folks, I ran into a similar memory leak issue. In my case a had the following:
for num in range(100):
    ds = xr.open_dataset('data.{}.nc'.format(num)) # This data was compressed with zlib, not sure if that matters

    # do some stuff, but NOT assigning any data in ds to new variables

    del ds

For some reason (maybe having to do with the # do some stuff), ds wasn't actually getting cleared. I was able to fix the problem by manually triggering garbage collection (import gc, and gc.collect() after the del ds statement). Perhaps this will help others who end up here...

For what it's wroth, the recommended way to do this is to explicitly closer the Dataset with ds.close().
Or with a context manager, e.g.,
for num in range(100):
    with xr.open_dataset('data.{}.nc'.format(num)) as ds:
        # do some stuff, but NOT assigning any data in ds to new variables
        ...

I just stumbled across the same issue and created a minimal example similar to @lkilcher. I am using xr.open_dataarray() with chunks and do some simple computation. After that 800mb of RAM is used, no matter whether I close the file explicitly, delete the xarray objects or invoke the Python garbage collector.
What seems to work: do not use the threading Dask scheduler. The issue does not seem to occur with the single-threaded or processes scheduler. Also setting MALLOC_MMAP_MAX_=40960 seems to solve the issue as suggested above (disclaimer: I don't fully understand the details here).
If I understand things correctly, this indicates that the issue is a consequence of dask/dask#3530. Not sure if there is anything to be fixed on the xarray side or what would be the best work around. I will try to use the processes scheduler.
I can create a new (xarray) ticket with all details about the minimal example, if anyone thinks that this might be helpful (to collect work-a-rounds or discuss fixes on the xarray side).

Hello,
I am facing the same memory leak issue. I am using mpirun and dask-mpi on a slurm batch submission (see below). I am running through a time loop to perform some computations. After few iterations, the code blows up because out of memory issue. This does not happen if I execute the same code as a serial job.
from dask_mpi import initialize
initialize()

from dask.distributed import Client
client = Client()

#main code goes here
ds = xr.open_mfdataset("*nc")

for i in range(0, len(ds.time)):
        ds1 = ds.isel(time=i)
        # perform some computations here

        ds1.close()
ds.close()

I have tried the following

explicit ds.close() calls on datasets
gc.collect()
client.cancel(vars)

None of the solutions worked for me.  I have also tried increasing RAM but that didn't help either. I was wondering if anyone has found a work around this problem.
@lumbric @shoyer @lkilcher
I am using dask 2022.2.0 dask-mpi 2021.11.0 xarray 0.21.1

For what it is worth, since this issue is quite old. I also noticed high memory usage for netcdf files (specifically with many variables, for some reason). The memory usage is way lower with engine="h5netcdf", but this is also way slower for these particular datasets (containing quite a lot, 2410, variables). It seems that the amount of variables really matters. When I open netcdf files that are 10 times larger (35GB vs 3GB), but with many less variables, the memory usage is way less with the default engine="netcdf4".


Show how org/org permissions affect user permissions in the Invitation wizard

Context
When inviting a user, you might not realise that org/org permissions affect the user's permissions.
This PR is @AgaDufrat's work — added final fix and raising in her absence.
Changes proposed in this pull request
Add the org/org permissions under each checkbox.
Guidance to review
Test manually through the UI.
Link to Trello card
https://trello.com/c/OnfnXcmz/2709-when-inviting-a-user-show-how-org-org-permissions-will-affect-the-permissions-youre-giving-them
Things to check

[X] This code does not rely on migrations in the same Pull Request
[X] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[X] API release notes have been updated if necessary
[X] New environment variables have been added to the Azure config

I'm going to put this behind the org/org feature flag to avoid confusion pre-shipping


Support sizer with default value

If not specified, the sizer argument (RowMultiplier or ConstantParameter) ought be replaced by the default value, which is given by the user when defining a table function.
Examples
@omnisci('int32(Column<double>, ConstantParameter<10>, OutputColumn<double>)')
def row_copier(x, m, y):
    n = len(x)
    for i in range(m):
        y[i] = x[i]
    return m

-- Specifying a sizer argument (=5)
SELECT * FROM TABLE(row_copier(cursor(SELECT f8 FROM table_name), 5))

-- Omitting! sizer will be equals to the default value (=10)
SELECT * FROM TABLE(row_copier(cursor(SELECT f8 FROM table_name)))

The same should to be supported using RowMultiplier:
@omnisci('int32(Column<double>, Column<double>, double,'
         ' RowMultiplier<1>, OutputColumn<double>)')
def add_columns(x, y, alpha, m, r):
    input_row_count = len(x)
    for i in range(input_row_count):
        for c in range(m):
            j = i + c * input_row_count
            r[j] = x[i] + alpha * float(y[i])
    return m * input_row_count

-- Specifying a RowMultiplier argument (=1)
SELECT * FROM TABLE(add_columns(
	cursor(select f8 from table_name),
	cursor(select f8 from table_name),
	cast(2 as double),
	1));

-- Ommiting it
SELECT * FROM TABLE(add_columns(
	cursor(select f8 from table_name),
	cursor(select f8 from table_name),
	cast(2 as double)));

Signature Specification
One can specify a default value by putting <N> after the sizer argument, where N is an integer value.
# RowMultiplier
@omnisci('int32(args..., RowMultiplier<N>, OutputColumn<T>)')
@omnisci('int32(args..., RowMultiplier, OutputColumn<T>)')
@omnisci('int32(RowMultiplier, args..., OutputColumn<T>)')

# ConstantParameter
@omnisci('int32(args..., ConstantParameter<N>, OutputColumn<T>)')
@omnisci('int32(args..., ConstantParameter, OutputColumn<T>)')
@omnisci('int32(ConstantParameter, args..., OutputColumn<T>)')

Limitations
When a default value is specified, the sizer parameter ought to be the last argument. Although we could support sizer with default value in any arg location. Parsing would be a bit more difficult, but it can be done.

Closing since this is not needed anymore


Update gradle-wrapper.properties

@RocketChat/android
Hi,
This small PR adds the distributionSha256Sum property to gradle-wrapper.properties. It allows for verification of the downloaded Gradle distribution via SHA-256 hash sum comparison.
You can find some documentation about this security improvement here, and the sums for each version of gradle here.
Note that you need to update this property every time you change your version of gradle.
#26

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


Clarifying for other language

Closes https://github.com/MicrosoftDocs/OfficeDocs-SharePoint/issues/2299

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@JoanneHendrickson PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


router API schema validation

The router's API should use JSON schema to validate its input. That way, things like blank service names would be detected before trying to shove them into the database.

Flynn is unmaintained and our infrastructure will shut down on June 1, 2021. See the README for details.


Bignoli ES 2.1

This allows the transport plugin to be installed with the newly released ES 2.1.0.
Caveats:

Users of the plugin will have to use the %JAVA_HOME%/bin/policytool to modify their permissions file to get the plugin to run.  I will begin editing the README.MD file with instructions as soon as I create this PR.

Some warnings are thrown by ES while trying to clean up the temporary indices created during the initial transport process.  I don't think these are fatal, but I'm not 100% sure.  This only seems to happen for ES 2.1.0...

So this PR superceeds the old one (https://github.com/couchbaselabs/elasticsearch-transport-couchbase/pull/104) correct?
The only thing missing here is support for 1.7, which requires changes to the CAPI lib as well, right?


Save/Restore caches in tmp/starter for remaining tests

I think we can eventually put this into the testing orb, but I'd rather put it here manually until the orb is merged in.
Opening a draft to see the results in CircleCI.

It's not a massive margin, but this will cut down about 20ish seconds (mainly the bundler cache) on our tests:

If I can get this to work on the orb, I think I'll try to apply the orb test jobs along with a new cache job to all of our packages to slim down the config files.


[WIP] Working on Travis passing

Remove palettable and haversine from conda install

Coverage remained the same at ?% when pulling aa1916c418c356b68c8d7d9ef9c5c5e574bb66c7 on fix/ci into ed25180a879b8be71b6468983149ffe485f430c9 on master.


'bst workspace open -f' does not behave as documented

See original issue on GitLab
In GitLab by [Gitlab user @phildawson] on Jun 28, 2018, 12:19
Summary
Steps to reproduce

Open a workspace for an element bst workspace open element.bst workspace_dir
Make a change to the source code checked out to workspace_dir
Close the workspace bst workspace close element.bst
force open a new workspace in the same directory at the previous on
bst workspace open -f element.bst workspace_dir
Notice the changes made at step 2 will have persisted.

What is the current bug behavior?
If there is an existing workspace for element.bst in the directory workspace_dir which has been closed,
using bst workspace open -f element.bst workspace_dir does not checkout an unmodified copy of the
elements source code. I.e any changes previously make to the workspace checked out it workspace_dir
persist, despite the force option.
What is the expected correct behavior?
Using bst workspace open -f element.bst workspace_dir should overwrite an existing workspace's contents with an unmodified copy of the element's source code.
Other relevant information
This was observed in version 1.1.3+171

In GitLab by [Gitlab user @knownexus] on Jul 18, 2018, 16:10
mentioned in merge request !549

In GitLab by [Gitlab user @phildawson] on Aug 10, 2018, 09:33
closed

In GitLab by [Gitlab user @valentindavid] on Aug 13, 2018, 14:13
This is a bug that affects 1.2. It should be backported.

In GitLab by [Gitlab user @valentindavid] on Aug 13, 2018, 14:33
reopened

In GitLab by [Gitlab user @valentindavid] on Aug 13, 2018, 15:15
mentioned in merge request !651

In GitLab by [Gitlab user @valentindavid] on Aug 13, 2018, 21:23
Backported by !651

In GitLab by [Gitlab user @valentindavid] on Aug 13, 2018, 21:23
closed