[core] Update cli tool for incentivenet

default points to "production" (which is going to be arbitrum testnet)
identify the error when gcp is not around so that we can document it before people reach out to us
add a --devnet flag that points to devnet (aws)
maybe some way to auto upgrade the cli or maybe at least let people know they are not in the latest version (be mindful, not every single command, maybe once every N hours)

Our Goal
Our goal is to make it as simple as possible to switch networks when using the CLI.
The run command would use testnet by default (until mainnet):
lilypad run cowsay:v0.0.4 -i Message="moo"

An additional network option would set the network to devnet or dev:
lilypad run cowsay:v0.0.4 -i Message="moo" --network devnet

Proposal: Embed per network configs into the binary
We could embed configs with contract addresses for each network. The configs would be embedded directly into the binary as an embed.FS.
The network would be selected with a network option:

testnet. Contract addresses for testnet. (default)
devnet. Contract addresses for devnet.
dev. Contract addresses for local development.
(eventually we would add mainnet which would become the default)

We may add other values to the config like rpc_url. The configs would be in a standard format so we could extend for custom configs read from file. I propose TOML below.
Precendence
The configs would be used last after command line options and environment variables. The precedence would go:

CLI options. Should always take precedence because it is the user explicitly tweaking a configuration.
Environment variables. Should come second as a mechanism for dynamic configuration.
(Config file. Future work, but we could read configs from file using the same format as the embedded configs.)
Embedded configs. Should come last as default values when no other configuration has been provided.

Benefits
Embedding would give us an easy way to switch networks, but it would have a few additional benefits:

Nearly zero config for the most common use cases.

Private keys can't be done this way. We will likely need some environment variables or config files for them.

We could track deployed contracts by release. Each time we change contract code, we update the embedded configs and bump the version.

When helping users debug, we could ask them which CLI version they are using and instantly know which contracts they are using. (May have to check if the have environment variables set or if they used CLI options.)
We probably want to auto-update the CLI. With auto-updates, we could usher users into using the newest contracts without asking them to make config changes.

Tradeoffs
We would need to be disciplined about updating the config files every time we change the contracts. We might be able to automate.
Impact on other services
The solver, resource-provider, job-creator, and mediator all use the same mechanism for loading contract addresses. The run command instantiates a job-creator to run a job.
We probably want all services to use the embedded configs. That way we can version the services alongside the CLI. But we could use the embedded configs only for the run command with some refactoring.
Again, none of this will have an immediate impact because the environment variables we use today will override the embedded configs.
Alternatives
We could provide environment files or config files with the contract addresses per network:

Extra steps for users. UX isn't as nice.
Would have to implement config files if we wanted those.

We could inject contract addresses into user environments using Doppler:

Would need to think about Doppler access. Maybe something like Doppler service accounts would be appropriate.
Doppler ain't cheap.

Proposal: TOML
TOML would be a good choice for the configuration format. With an acknowledgement that there may be bikeshedding, here is how I would compare with the other popular formats:

JSON. Not really a good configuration language. Lack of comments without additional support for example.
YAML. More powerful and complex than we need. We won't need to re-use blocks for example.

Also, TOML is configs are mostly flat which makes it quite readable in my opinion. Here's a more thorough comparison if folks are interested: https://www.anbowell.com/blog/an-in-depth-comparison-of-json-yaml-and-toml
Private keys
We obviously can't use this approach for private keys or any other sensitive material. TBD what to do here. Open to ideas.
Migration from GCP
Once we shut down our GCP testnet, we may have users with environments that point to the old contracts.
We could embed a config with the GCP contracts and warn the user that they need remove the environment they were using, which will move them to the new testnet without configuring anything.
Probably the CLI should do nothing until they have cleared out their environment?
We can eventually drop this check from our code base.

LGTM.
I've also like to see a command to output the config actually being used.


Fix callout drawing that happens even if disabled

Fix #681

Some kind of regression test would be good to have, as this went through. A simple fixture is probably best, maybe as additional annotation in an existing one testing callouts?

Some kind of regression test would be good to have, as this went through. A simple fixture is probably best, maybe as additional annotation in an existing one testing callouts?

Yes, I'll commit it in next minutes.


add: linkzy

Description
Adding my profile

@linkzy Thank you so much for your contribution! Your efforts are greatly appreciated and will go a long way in improving our project. Please feel free to share it with others and Star the repo to help us grow even further!


[mle] unicast Link Request to 1.2.1 or earlier parent on router role promo

This commit adds a new mechanism to send a unicast MLE Link Request to
the device's former parent if its version is earlier than 1.3.0, when
the device is promoted from a child to a router role. This is to
address a potential compatibility issue with some non-OpenThread
stacks which may ignore MLE Advertisements from a previous/existing
child.

Related to SPEC-1131.

Proposed change looks good.
One related question: if by chance the former-Parent (mParent) got removed in the latest Router ID Mask from the Leader, then the SendAdvertisement(); call is not made I see. Which seems ok: if the Trickle timer for advertisement is reset then the device will anyway soon send out a new MLE advertisement.  Just to confirm, is the Trickle timer reset now that it becomes a Router?  And it seems like a corner case so normally doesn't happen.

One related question: if by chance the former-Parent (mParent) got removed in the latest Router ID Mask from the Leader, then the SendAdvertisement(); call is not made I see. Which seems ok: if the Trickle timer for advertisement is reset then the device will anyway soon send out a new MLE advertisement. Just to confirm, is the Trickle timer reset now that it becomes a Router? And it seems like a corner case so normally doesn't happen.

@EskoDijk Yes, this situation (the former parent not being in the router ID mask from the leader) should be unlikely. However, as you said, resetting the adv trickle timer is key and will make it work. The adv trickle timer should be reset from the UpdateRouterIdSet() call, which is before we check if the parent is present in the table (as we handle "address solicit response" in the HandleAddressSolicitResponse()).


[TODO] Migrate to pydantic V2

involves porting our custom BaseSettings class to use pydantic-settings
also involves migrating all our BaseModel instances to the new V2 structure as per https://docs.pydantic.dev/2.0/migration/

copier v9 depends on pydantic v2, so that's an easy fix.
uoft-core is gonna take a fair bit of work
the bigger issue is diffsync. diffsync still depends on pydantic v1. there's an unreleased diffsync v2 which depends on pydantic v2, but it's not out yet. we need to wait for this https://github.com/networktocode/diffsync/issues/232 and then wait for nautobot-ssot==1.3.2 [requires: diffsync>=1.6.0,<2.0.0] to cut a new release depending on diffsync v2, and then we can upgrade to pydantic v2


Questions about Elvish syntax in the cookbook

I work through the cookbook but get tripped up by the elvish syntax quickly
there's this example edit:rprompt = (constantly (edit:styled (whoami)✸(hostname) inverse)) and I just wanted to try to replace the ✸ so first I just wanted to delete it and then I get
Exception: arity mistmatch: want 2 arguments, got 3
got 3 args after I deleted one? how is that?
and then I tried edit:rprompt = (constantly (edit:styled (whoami) ">>" (hostname) inverse))
and now elvish gets 4 arguments huh
I like the idea of elvish but it's usability compared to modern languages seems to be a bit hit and miss or maybe the dice could be improved or I'm just stubborn?
it's also not clear to me why leaving out the inverse does not work, I'd expect this to be an optional argument.
thanks

and this confusion just continues like
edit:-matcher[''] = [p]{ edit:match-prefix &ignore-case $p }
the Fundamental were very easy nearly superfluous (as it should be) but now elvish is just throwing a bunch of weird symbols and unintuitive syntax at me with no comment? like what is that [''] supposed to do here, what is the & doing?
I find it is actually confusing and not so much Elvish is a friendly and expressive

your first example fn ls [@a]{ e:ls --color $@a } does not work for me btw but that is not a fault of elvish, my version of ls just has no --color option, not sure if that is special for me or if you have a special ls version. Probably BSD vs GNU but it's confusing because on another point you assume uname outputs Darwin in which case I'd not expect to have GNUs ls necessarily installed.

edit:rprompt = (constantly (edit:styled (whoami) ">>" (hostname) inverse))
These are arguments indeed.

(whoami)
">>"
(hostname)
inverse

Maybe you want to get a username>>hostname string. In this case, you shouldn't leave spaces between the first 3 arguments, because spaces are argument separators: (whoami)">>"(hostname). If you want spaces in the prompt between the items, you should put spaces in the string literal: `" >> ".

got 3 args after I deleted one? how is that?

Because you didn't delete the character -- you replaced it with a space (given the error message) So you ended up passing three arguments to a function that only accepts two.

BTW, You'd get a similar failure using fish which is arguably a friendlier shell for people with limited programming experience.  For example, here is a rough equivalent of what you were doing only using fish syntax:
$ function edit:style
        echo arg 1: $argv[1]
        echo arg 2: $argv[2]
        echo arg count: (count $argv)
  end
$ edit:style (whoami) (hostname) inverse                        0 ms
arg 1: krader
arg 2: macbook.skepticism.us
arg count: 3

Notice the edit:style function got three arguments but was only expecting two.

Hi, I am sorry that you find Elvish's syntax confusing. Learning materials are indeed lacking in many places, partially because I need to priotize coding over documenting at the stage, and partially because it is not always clear to me which aspects warrant more documentation. Feedback is appreciated; they are very valuable for my priotization. Also, the reference documents are a little bit more complete, and I encourage you to read them as well.
Answering your particular questions (I will also clarify the points in the documents later):

I guess you removed the star symbol and replaced with a space? That will indeed make it three arguments: (whoami)✸(uname) is one argument because it runs together; if you remove ✸ it becomes two separate arguments. What you want is not (whoami) ">>" (uname), but rather (whoami)">>"(uname). This is true of other shells as well; in bash for example, ls $a.txt lists one file, but ls $a .txt lists two.

The empty string key in edit:-matcher stands for the default matcher. The right-hand-side is a lambda, [p] being the parameter. The &ignore-case syntax is for command options; somewhat like --ignore-case, but for builtin functions and user-defined functions.

Yeah the difference in ls flags is confusing. I will clarify that.

@monouser7dig It seems to me you are simply too lazy to read and understand the full documentation. Understanding something is organizing the knowledge. If you don't have knowledge, you can't organize it, so you don't understand it. So read the documentation.
The --color option of ls probably expects an argument that specifies when to use color (e.g. always, auto, never). You may want to try exa, which is a modern replacement for ls.
The '' in edit:-matcher[''] = [p]{ edit:match-prefix &ignore-case $p } means an empty string. The & specifies named arguments (called options) for Elvish functions.
edit: is a namespace in edit:-matcher. The - in -matcher means that this function will change before 1.0 (the api, or only the internal implementation).

you shouldn't leave spaces between the first 3 arguments, because spaces are argument separators:

@notramo thanks that makes some sense
@krader1961 it's not like I have no experience programming, it's just that the examples are confusing/ assume much implicitly that is not said. As notramo explains it actually makes sense, would be nice if that was easier to discover. I'm actually coming back to elvish the third time now because 1. I like the idea and 2. it was just too confusing before so I have done several attempts as I'm now doing a bit go anyway I was taking a look again.
The space thing is especially strange because earlier it's said that spaces in the input do not matter (echo Hello     World)
@xiaq ok so what I still find confusing is the & thing what is it doing? thanks
I've even been doing some Haskell so I'd be inclined to say that if I do not understand it, chances are there are a couple other like me out there.

@notramo I actually read the docs and you'll find the BSD version of ls has no --color option if you read the docs/ manpage
sure '' is an empty string, but what is it doing in there? Is it telling the matcher to activate on an empty string? just would be nice to know.
- means unstable, ok, got it, thought everything before 1.0 would be subject to change eventually but ok, nice to know

IMO a shell should be as easy to learn as possible, I'm not programming to do bash scripting but bash scripting to get some stuff done. Bash is already awkward enough.
The structured data pipes/ multi assignment is nice and make my life easier but when I read about temporary assignment ... this just screams for implicit errors that are a pain to figure out when all I want to do is some script, not rocket science cpp.
Elements in braced lists can also be separated with whitespaces, or a combination of comma and whitespaces (the latter not recommended):
(In future, the syntax might be made more strict.)

yes PLEASE 👍 😄

@monouser7dig I hope your questions are answered now! As said above, I will make sure the documentations are improved. For further questions, discussion groups (listed on the repo homepage as well as https://elvish.io) can give you faster and more specific answers.
@krader1961 @notramo Thanks for chiming in!
One thing I do request is that we all strive to keep the conversation polite. Different people have different expectations, and Elvish is probably not going to make everyone happy (despite that being my wish). Let's work towards consensus, but if consensus is not reached, disagree respectfully.

sure, thanks, did not mean to sound rude, generally really appreciate your take on a new shell, was just torn between nice shell - language/ doc barrier

@monouser7dig I updated my above comment with links.

- means unstable, ok, got it, thought everything before 1.0 would be subject to change eventually but ok, nice to know

Everything is subject to change eventually, but these things will surely change.
The edit:-matcher map  contains the type of the completion (argument, variable, command, index etc.). If a completion type is not specified, (for example, the map don't have the argument key), the default matcher will be used (that is stored with the empty string key). The empty string don't have a special meaning, any key could be used for the fallback matcher (e.g. elephant, or default, or anything) , but @xiaq chose this.
By the way, @xiaq what about changing the key of the default matcher function from '' to something more obvious e.g. default or fallback in edit:-matcher?

@xiaq, Seems like this can be closed. The questions were answered and the documentation has been much improved since this issue was opened.


Fix AWS CloudFormation template and add VPC support

I note that the CloudFormation template provided here currently has a note saying that it fails to properly configure minions but will be fixed shortly.
I had modified the template to launch in a VPC, which had worked, but I had networking issues where setup-network-environment failed to come up and I didn't have connectivity to the outside world (no nameserver lookup seemingly - although a plain CoreOS instance was fine in the same subnet).
It would be very useful for production deployments for the example template to bring up Kubernetes in a VPC, which is probably best practice for isolation etc.
I could make a pull request against the fixed cloud formation, but am not sure if it will solve the connectivity issues, so perhaps it would be an idea to fix the current template at the same time as adding VPC support (probably no disadvantage to making it VPC-only for that matter).
For reference here are the relevant changes needed for a VPC (mostly using security group ids not names), I include the resource type and the config KV pairs:
"Type": "AWS::EC2::SecurityGroup",
        "VpcId": "vpc-9f2f27fd",

"Type": "AWS::EC2::SecurityGroupIngress",
        "GroupId": { "Fn::GetAtt" : [ "KubernetesSecurityGroup", "GroupId" ] },

"Type": "AWS::EC2::Instance",
        "SubnetId": "subnet-4cbc4515",
        "SecurityGroupIds": [{ "Fn::GetAtt": [ "KubernetesSecurityGroup", "GroupId" ] }],

"Type": "AWS::AutoScaling::AutoScalingGroup",
        "AvailabilityZones": ["eu-west-1a"],
        "VPCZoneIdentifier": ["subnet-4cbc4515"],

"Type": "AWS::AutoScaling::LaunchConfiguration",
        "SecurityGroups": [{ "Fn::GetAtt": [ "KubernetesSecurityGroup", "GroupId" ] }],

Fixed by #3848


Underlines are no longer drawn on Android

The underlines for the text input are currently drawn on Android.
This pull request makes them transparent such that they are no longer visible.

Oh, sry, I totally overread that sentence. Just saw the issue and needed it myself, so I thought I could help
But thanks for pointing it out, that helps a lot!


Fix a typo in TableUpsertMetadataManager

Codecov Report

Merging #9269 (fd68313) into master (dafa5bf) will increase coverage by 3.49%.
The diff coverage is n/a.

:exclamation: Current head fd68313 differs from pull request most recent head bf91919. Consider uploading reports for the commit bf91919 to get more accurate results

@@             Coverage Diff              @@
##             master    #9269      +/-   ##
============================================
+ Coverage     63.59%   67.08%   +3.49%
+ Complexity     4984     4813     -171
============================================
  Files          1809     1381     -428
  Lines         96993    71991   -25002
  Branches      14832    11524    -3308
============================================
- Hits          61678    48296   -13382
+ Misses        30757    20189   -10568
+ Partials       4558     3506    -1052

Flag
Coverage Δ

unittests1
67.08% <ø> (-0.01%)
:arrow_down:

unittests2
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...ore/query/scheduler/resources/ResourceManager.java
84.00% <0.00%> (-12.00%)
:arrow_down:

.../pinot/core/query/scheduler/PriorityScheduler.java
77.77% <0.00%> (-5.56%)
:arrow_down:

...form/function/BinaryOperatorTransformFunction.java
37.80% <0.00%> (-2.07%)
:arrow_down:

...ders/forward/VarByteChunkMVForwardIndexReader.java
77.57% <0.00%> (-1.87%)
:arrow_down:

...ler/recommender/data/generator/BytesGenerator.java

.../java/org/apache/pinot/client/utils/Constants.java

...main/java/org/apache/pinot/client/BrokerCache.java

...ker/failuredetector/ConnectionFailureDetector.java

...er/routing/segmentpruner/SegmentPrunerFactory.java

...mentpruner/MultiPartitionColumnsSegmentPruner.java

... and 425 more

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


Module bardapi not recognised anymore

I have been using this script that worked perfectly:
!pip install bardapi==0.1.23a
from bardapi import Bard
myToken = 'xxxxxxxx'
bard = Bard(token=myToken)
Now I get systematically an error 'ModuleNotFoundError: No module named 'bardapi'
at line ' from bardapi import Bard
Something changed?
Thanks for the attention, looking forward for feedback.
Cordially,
Alexandre

Yes, it seems there have been many changes. attributes like " bard.ask() " or " bard.generate() " are also do not work any more!

Thanks!
My understanding is that my issue is related to management of packages in my development environment which was Jupiter Notebooks. I was not able to restore the situation that worked without problems.
I changed the developing environment to Pycharm Community and was able to install bardapi package and got no more problems with the same python script I was using  in jupyter notebooks.
All these issues of establishing the right environment are still very confusing for me.
Thanks for the attention and feedback are welcome.
Cordially,
Alexandre


Code is science

Thanks for submitting your project to be featured in the Open Leadership Zone at MozFest 2017 :sparkles:
This is open to any project in Mozilla Open Leaders round 4 -- even if you're not coming to MozFest. This process ensures that you're following open practices and will help set you up for a sustained contributor community afterwards. If you have any questions, feel free to reach out to @acabunoc or ask questions in our chat room.
Open Project Checklist :clipboard:
Your project must complete the following to be a featured on Mozilla Pulse and the Open Leadership Zone at MozFest 2017. As you complete each exercise, check off the box and comment with a link to your completed resource. This template repository is here to help you if you get stuck!

[x] Complete Open Leadership 101
[x] Provide a GitHub repository for work and discussion on your project in a comment

new to GitHub?  Here's a step-by-step guide on using the #mozfest template

[x] Create file: README.md in your project repository. This file should help newcomers understand what your project is, why it's important, and kinds of help you're looking for.
[x] Create file: LICENSE to give your project an open license, allowing for sharing and remixing.
[x] Create file: CONTRIBUTING.md so others know how they can contribute. If you'd like, you can remix this template
[x] Create file: CODE_OF_CONDUCT.md. This can be the Mozilla Community Participation Guidelines or a code of conduct of your choice
[x] Turn on your Issue Tracker and create issues to describe each task that you need help with and how a contributor can get started on that task. If you're participating in Hacktoberfest, Create a label called hacktoberfest and apply it to your issues.

Ready to be featured :tada:

[x] Fill out this form to submit your project to Mozilla Pulse. Add the tag open leadership zone.
[x] Leave a comment on this issue with the text This is ready for the Open Leadership Zone. @acabunoc will review this issue. If accepted, your project will be approved on Mozilla Pulse and featured in the Open Leadership Zone.

If you get stuck at any point, feel free to look at the project templates or reach out to your mentor or @acabunoc. We're here to help you through this process.

Hey @yochannah! Gentle reminder that there's only one week to MozFest :) I'd love to see you project featured in the Open Leadership Zone!

This is ready for the Open Leadership Zone.
@acabunoc Thanks for the reminder!

Great repo! Thanks @yochannah 😄
You're now part of the Open Leadership Zone 🎉
Pulse: https://www.mozillapulse.org/entry/483


The context menu entry should also show up when right-clicking on the background of the explorer window inside a folder.

(Like, e.g. the powershell context menu entry.)

Same as #6414.

Very much so.


[Network] Add front-door load-balancing update

Add command: network front-door load-balancing update
Add test scenarios for backend-pool, frontend-endpoint, load-balancing, probe and route-rule

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

If this PR is for a new extension or change to an existing extension, use the following to try out the changes in this PR:
docker run -it microsoft/azure-cli:latest
export EXT=<NAME>
pip install --upgrade --target ~/.azure/cliextensions/$EXT "git+https://github.com/kairu-ms/azure-cli-extensions.git@fix-2093-add-front-door-load-balancing-update#subdirectory=src/$EXT&egg=$EXT"

Network

/azp run


Extension host terminated unexpectedly when open python files

I have encountered this issue when using VSCode(not insider version) and I try to reinstall it but nothing enhanced. So I try to using Insider version and it performs well at first but this problem repeats today.

VSCode Version: 1.22.0-insider
OS Version: Windows 10 Pro x64 10.0.16299

Steps to Reproduce:
1.Open a py file using VSCode
2.Extension host terminated unexpectedly.
I have installed these extensions:

Beautify
C/C++
Code Runner
Graphviz Markdown Preview
hexdump for VSCode
HTML Snippets
JavaScript(ES6) code snippets
Markdown All in One
Markdown Emoji
Markdown Preview Enchanced
Markdown Preview Mermaid Support
Markdown TOC
Markdown+Math
Python
SVG Editor
vscode-icon

Does this issue occur when all extensions are disabled?: No
And when I disable Python-extension I find it works well but I can't finish my work without python-extension.
I think It might be the wrong interaction between extensions but when I just enable python-extension only the problem stays still.

Oh I make a typo..
Does this issue occur when all extensions are disabled?: No
So sorry..

@Xusine1131 when this happens can you go to the Output Channel and select the Log (Extension Host) channel and provide the output.
Since you open a py file I suspect this might be caused by the Python extension. Does everything work if you disable that extension. If so please report the issue against the extension.

@dbaeumer Thank you for your suggestion! And I use Developer Tools and Output channel and finally I find the question is caused by JavaScript(ES6) Snippets and when it was removed(must be removed not disabled) everything went well.  Thank you anyway!


Make Qunit module available as 'ember-qunit' export

Just to be consistent with the other other parts and because I think it's useful to be uses in tests like
import {module} from 'ember-qunit';

module('My custom class test...');

Instead of using window.module which doesn't work anymore or QUnit.module, also in some cases in which moduleFor... doesn't fit.

Awesome, thank you!

@rwjblue yeah! thanks you for the repo and sorry for the spam 🚀

@rwjblue any change we can get a release out containing this?

@josemarluedke - Done!

@rwjblue thanks!

Thanks!


SD results not same?

what has happened to SD? I been using same models for while now and I use same prompts I've been using and I get totally different results than before....I want to say I cannot make any NSFW images...is safety checker now enabled or something?
also: what happened to the "PASTE" or "SAVE" button under prompt? delete/refresh are still there

check in the A1111 repo for answers https://github.com/AUTOMATIC1111/stable-diffusion-webui

Hi, I'm noticing big issues in the render quality, with old prompts, even model renders look different, getting tattoos on skin when there should not be and would not be a few days ago. It feels like lora are fire when not on prompt.
Sorry no idea what the repo is or where it is, from your link ^^

Yeah, I'd love to see more on this.  I couldn't find anything in the A1111 repo about this.  Do they say anything like they are going to improve this?  I know things will move towards SDXL, but there is a whole SD1.5 ecosystem of ckpts and Loras that produce great results that we should still be able to get.  I've completely stopped making anything online now (colab) until there is some clarity on this.

same problem.

Does anyone have any info on whether the developers (i.e. not Ben) are aware that they have messed up SD1.5 stuff and are planning to fix it in an update?

check in the A1111 repo for answers https://github.com/AUTOMATIC1111/stable-diffusion-webui

Hi Ben, I'm also facing the same issue with SD 1.5. The generated images on google colab are different from local.
Also, the prompts became unruly. I used same checkpoint, sampler, hires fix, resolution, seed, sd web UI version and everything. But no luck.
I think, the main culprit is the python dependency v3.10.12 on colab. As per the A1111 recommendation, I use v3.10.6 on my local machine. I have shared the screenshots of both the colab & local, please have a look at it. Thank you very much for your efforts. You are the Robbin hood for thousands of potato PC users like me. Please fix it.
My command-line arguments are --xformers --no-half --autolaunch
Google Colab A1111

Local A1111

I'm the original poster.  I figured out my problem thanks to this checklist: https://github.com/AUTOMATIC1111/stable-diffusion-webui/discussions/13093
In my case, I use a bunch of embeddings in my negative prompts and I hadn't moved a copy to the new SD embeddings folder when I updated A1111.  Now that I have, my old prompts are reproduced about 99% the same.  Hope that checklist is helpful to others too.


Work Package wp-fc.csv

Results of Phase 1:
wp-fc-results.csv

I would like to work on this please!

Results of Phase 1:
wp-fc-results.csv

I completed this work package.

Results of Phase 2:
wp-fc-results.csv

Results of Phase 2:
wp-fc-results.csv


ES v5.22 docs

Includes: Channel sidebar and team drag/drop

@devinbinnie can you help confirm the config values?

@devinbinnie can you help confirm the config values?

@esethna Yep, that's correct.

@esethna @devinbinnie Is it not called ExperimentalChannelSidebarOrganization? https://github.com/mattermost/mattermost-server/blob/master/model/config.go#L317

@esethna @devinbinnie Is it not called ExperimentalChannelSidebarOrganization? https://github.com/mattermost/mattermost-server/blob/master/model/config.go#L317

Sorry, yes you are correct. I forgot we had it under a different name in the code.

@devinbinnie can we please change it to "ExperimentalSidebarFeatures" as per the spec? It was chosen thoughtfully to avoid user error with the existing "ExperimentalChannelOrganization" setting

@devinbinnie can we please change it to "ExperimentalSidebarFeatures" as per the spec? It was chosen thoughtfully to avoid user error with the existing "ExperimentalChannelOrganization" setting

Sure, can you create a ticket for it and assign it to me? It will require code changes on 3 different repos.

@devinbinnie https://mattermost.atlassian.net/browse/MM-24059
Thanks for the catch Amy

@amyblais can we please merge this tomorrow and get it live on docs? I'd like to get the docs PR merged and live before the blog post for channel sidebar goes out (since the release blog links to it)

I updated the config name to ExperimentalChannelSidebarOrganization
It's very confusing with ExperimentalChannelOrganization still existing, but the name change got cut from the release, so updating the docs instead


Update links to npm packages

Description
Updates links to npm packages.

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Chore (minor updates related to the tooling or maintenance of the repository, does not impact compiled assets)

Motivation and context

How to test?
Screenshots (if appropriate)
Checklist

[ ] If my change required a change to the documentation, I have updated the documentation in this pull request.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Thank you, Bjarne!
Merging it right away!

@nielslyngsoe @iOvergaard it seems the readme in the recent packages added not link to npm package (except in the main readme), which I think is more useful than linking to the graphic, e.g. here: https://github.com/umbraco/Umbraco.UI/tree/dev/packages/uui-action-bar
Anywhere this can be changed when running npm run new-package to scaffold files in a new package, so the badge just link to the npm package?

@bjarnef
Well spotted! You are right, this should be added to the plop templates and fixed in all of the newer components. I'll create an issue for it.


Vuestic simple and multiselect problems

I'm trying to make these work in the project I've been working on and I ran into the following issues:

With the simple select, there is no way to un-select a value, would be nice to have an option to do so, since a field can be optional.
Tried to use multiselect instead, because at least you can un-select things there. Unfortunately there is no way to load the previously stored selections into the component via data or any other way (works with simple select).

@pfrohner thanks for mentioning, will be having a look into the issue

1)already implemented.
2)Could you tell more information on this point(maybe show the case, where we can use this feature)?

@vlad-shusterman

Where can I see that? I just pulled the repo, it doesn't work. I can click on an item twice but then I got an [object Object] as the value....
http://vuestic.epicmax.co/#/forms/form-elements there is the multiselect, let's say we have previously stored some values selected by the user and we'd like to display them on page load. Usually, that goes by setting the v-model to that stored value, and passing it to the component. How can I achieve that?

Hi,
Please, check our updates on selects and tell if all your problems are solved.


A method to add callbacks to the global event manager

resolves #131
Description
It was annoying me that we had a way to add loggers to the global event manager but not callbacks. I'm doing work in dbt-core that could benefit from this. So I decided to do the work here to give callbacks the same support as loggers.
Checklist

[X] I have read the contributing guide and understand what's expected of me
[X] I have signed the CLA
[X] I have run this code in development and it appears to resolve the stated issue
[X] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
[X] I have run changie new to create a changelog entry

Codecov Report
Attention: Patch coverage is 87.50000% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 56.37%. Comparing base (df4b4c0) to head (74d87a7).

Files
Patch %
Lines

dbt_common/events/event_manager.py
75.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #132      +/-   ##
==========================================
+ Coverage   56.29%   56.37%   +0.08%
==========================================
  Files          50       50
  Lines        3082     3090       +8
==========================================
+ Hits         1735     1742       +7
- Misses       1347     1348       +1

Flag
Coverage Δ

unit
56.37% <87.50%> (+0.08%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


I would like to add an important step in README.md.
After we clone our repo we should add remote reference upstream so that when there's an new update in the main repo or new branch has been merged our cloned repo is upto date with that repo.
This step is missing which creates unusual conflicts in the contributors repo. So, I would like to add it.

The image is just for sample purpose.

yes, assigning to you


docker image build fails

Hi!
After downloading the repo, on main branch, the following command fails:
docker image build . -t multiversx/deep-history:latest -f ./Dockerfile

with the following error:
[+] Building 113.3s (19/42)
 => [internal] load build definition from Dockerfile                                                                                0.0s
 => => transferring dockerfile: 37B                                                                                0.0s
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 2B                                                                                0.0s
 => [internal] load metadata for docker.io/library/ubuntu:22.04                                                                                0.7s
 => [internal] load metadata for docker.io/library/golang:1.17.6                                                                                0.7s
 => [builder  1/24] FROM docker.io/library/golang:1.17.6@sha256:fff998d0f7a46baeb15d8478ee909a8237377e07ccf1a8e77d77a32033021d62                           0.0s
 => [internal] load build context                                                                                0.0s
 => => transferring context: 70B                                                                                0.0s
 => [stage-1  1/13] FROM docker.io/library/ubuntu:22.04@sha256:9a0bdde4188b896a372804be2384015e90e3f84906b750c1a53539b585fbbe7f                            0.0s
 => CACHED [stage-1  2/13] RUN apt-get update && apt-get install -y wget python3.10                                                                        0.0s
 => CACHED [builder  2/24] RUN apt-get update && apt-get -y install python3-pip && pip3 install toml                                                       0.0s
 => CACHED [builder  3/24] WORKDIR /workspace                                                                                0.0s
 => CACHED [builder  4/24] RUN git clone https://github.com/ElrondNetwork/elrond-config-devnet --branch=D<IP_ADDRESS>-hf01 --single-branch --depth=1           0.0s
 => CACHED [builder  5/24] RUN git clone https://github.com/ElrondNetwork/elrond-config-mainnet --branch=v<IP_ADDRESS> --single-branch --depth=1               0.0s
 => CACHED [builder  6/24] WORKDIR /go                                                                                0.0s
 => CACHED [builder  7/24] RUN git clone https://github.com/multiversx/mx-chain-go.git --branch=$(cat /workspace/elrond-config-devnet/binaryVersion | sed  0.0s
 => CACHED [builder  8/24] RUN git clone https://github.com/multiversx/mx-chain-go.git --branch=$(cat /workspace/elrond-config-mainnet/binaryVersion | se  0.0s
 => CACHED [builder  9/24] RUN git clone https://github.com/multiversx/mx-chain-proxy-go.git --branch=v1.1.27 --single-branch --depth=1 mx-chain-proxy-go  0.0s
 => CACHED [builder 10/24] RUN git clone https://github.com/multiversx/mx-chain-proxy-go.git --branch=v1.1.27 --single-branch --depth=1 mx-chain-proxy-go  0.0s
 => CACHED [builder 11/24] WORKDIR /go/mx-chain-go-devnet/cmd/node                                                                                0.0s
 => ERROR [builder 12/24] RUN go build -i -v -ldflags="-X main.appVersion=$(git describe --tags --long --dirty --always)"                                112.5s

Any ideas?

Hey @rvignolo ,
We're on it. We already have a PR in work for it.
Thanks for the HU! :raised_hands:

Should be fixed in https://github.com/multiversx/mx-chain-deep-history/pull/6.


CORBS warning after 5minutes or more

My Activestorage routes outside
post 'upload', to: 'projects#upload'
my upload method
class ProjectsController < ApplicationController

 def upload
  @image = Project.new (image: params[:files].first)
   url = [
    {
     url: @image.image.service_url,
     thumbnail: @image.image.service_url,
     delete_url: @image.image.service_url
     delete_type: "DELETE"
    }
   ]
   end
   render json: url
end

Upload is working fine but after 5minutes or more i got a CORBS warning
GET https://api.cloudinary.com/v1_1/saintlucifer91/image/download?api_key=576932957151598&attachment=false&expires_at=1532661832&public_id=ndN5keoAccUwGzF7AUqyqSZ7&signature=033b7b7c4c95dcdbb54a77ef37caf32b96d0a657&timestamp=1532661532&type=upload 401 (Unauthorized)

my image link got this
{
"error": {
"message": "Stale request - expires_at 2018-07-27 03:23:52 +0000 has passed"
}
}

Seem the problem is like this
https://leavingharbor.com/posts/gotcha-with-active-storage-and-cloudflare

@jbarnobal, have you tried using the provided helpers, #url_for and/or #rails_blob_url? Either will provide you a permanent URL that points to your application and redirects your to the actual (short-lived) service url only when you want to access it.
Ideally, you shouldn't be accessing the #service_url directly but rather use either of the provided helpers. So in your case, you will want to do
...
url<EMAIL_ADDRESS>...

or
...
url<EMAIL_ADDRESS>...

The 5 mins expiration is an expected behaviour. Please see: https://edgeguides.rubyonrails.org/active_storage_overview.html#linking-to-files


remove "return to clients" link on profile page, redundant since "clients" is already in the header

Hi @HattieTavares ! could I work on this issue?


untested task table style v centering status

Link to Trello card (if applicable)
https://trello.com/c/kv3IuQwW/112-i-want-to-see-the-status-of-a-task
Impacted areas of website
Project Page - Task Status Table, in the Status column

 I want to see the status of a task.


DS - Make Repo Public

Description
As a gitHub user, I want to be able to see the va-mobile-library repo, so that I can see how the VA is building their mobile design system.
As part of this story we want to:

Make the va-mobile-library repo public to match other VA repos.

Acceptance Criteria

[ ] Make the va-mobile-library repo public

Notes & Open Questions

Dependencies/Roadblocks:

Any internal/external dependencies?
Test accounts needed?

Does this require QA?

Dev Notes:

Ticket Checklist

[ ] Acceptance criteria defined
[ ] Labels added (front-end, back-end, feature)
[ ] Linked to an Epic

Repo has been made public by https://github.com/department-of-veterans-affairs/github-user-requests/issues/19943#issuecomment-1967419029. Closing out.


[Testing] Fixed The Script to Work With New Options

Since the addition of the --analytical-place option, we can now use a single VTR version to pre-compute the files and run AP.

LGTM


Use out of source docs folder for generating docs artifacts

A recent PR added a docs/ folder into the root of the repo.
Because that PR did not change anything that would necessitate a build of the CLI, the CLI CI did not trigger.
Azure DevOps' default working directory is the root of the repo.
It's not uncommon for work done in Azure DevOps (like product builds, artifact generation, etc.) to work from the root of the repo and mutate the file structure... However, the docs folder does leave itself open to collisions in a way that other folders might not.
To fix: Do the work in the docs folder outside of the source directory.
Successful run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1881382&view=logs&s=2b1cf740-7e83-5b38-9f56-89c750cf5d77&j=dab3e848-2257-5774-eb9b-38eb96e299cc
fix https://github.com/Azure/azure-dev/issues/763

Fixes https://github.com/Azure/azure-dev/issues/763

@danieljurek should we undo my workaround? https://github.com/Azure/azure-dev/pull/765 ?

Azure Dev CLI Install Instructions
Install scripts
MacOS/Linux

May elevate using sudo on some platforms and configurations

bash:
curl -fsSL https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/770/uninstall-azd.sh | bash;
curl -fsSL https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/770/install-azd.sh | bash -s -- --base-url https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/770 --version '' --verbose

pwsh:
Invoke-RestMethod 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/770/uninstall-azd.ps1' -OutFile uninstall-azd.ps1; ./uninstall-azd.ps1
Invoke-RestMethod 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/770/install-azd.ps1' -OutFile install-azd.ps1; ./install-azd.ps1 -BaseUrl 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/770' -Version '' -Verbose

Windows
powershell -c "Set-ExecutionPolicy Bypass Process; irm 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/770/uninstall-azd.ps1' > uninstall-azd.ps1; ./uninstall-azd.ps1;"
powershell -c "Set-ExecutionPolicy Bypass Process; irm 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/770/install-azd.ps1' > install-azd.ps1; ./install-azd.ps1 -BaseUrl 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/770' -Version '' -Verbose;"

Standalone Binary

Linux - https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/770/azd-linux-amd64.tar.gz
MacOS - https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/770/azd-darwin-amd64.zip
Windows - https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/770/azd-windows-amd64.zip

Container
docker run -it azdevcliextacr.azurecr.io/azure-dev:pr-770


Fix {aarch64,arm}-android targets' CI

Let's see if it still happens.

@JohnTitor: no appropriate reviewer found, use r? to override

Oh well, s390x is failing here... But the android issue is fixed, let's merge this.


Replace wait that uses Selenium.Support

Something like
let wait timeout f =
  let sw = System.Diagnostics.Stopwatch.StartNew()
  let rec innerwait timeout f =
    if sw.Elapsed.TotalSeconds >= timeout then
      raise <| WebDriverTimeoutException("Timed out!")

    try
      let result = f()
      if result <> true then
        sleep 500ms
        innerwait timeout f
    with
      | :? WebDriverTimeoutException as ex -> reraise()
      | _ ->
        sleep 500ms
        innerwait timeout f

  innerwait timeout f

This will allow dropping of this dependency.

https://www.nuget.org/packages/canopy/0.9.47


fix email_activate views & Email2FA identity code -> secret

@MGatner HTML tags are the same as in the Email 2FA verify view email_2fa_verify

Ah gotcha! Thanks for pointing that out. Doubly-approved! 👍👍

A test failed. I will fix it.
There was 1 failure:

1) Tests\Controllers\ActionsTest::testEmailActivateShow
Failed asserting that '\n

Please click the link below to activate your account and start using the site.
\n
\n

https://example.com/index.php/auth/a/verify
\n
\n
' contains "/auth/a/verify?c=".

/home/runner/work/codeigniter-shield/codeigniter-shield/tests/Controllers/ActionsTest.php:221

https://github.com/lonnieezell/codeigniter-shield/runs/6075197185?check_suite_focus=true


Can't connect to open networks

When I want to extend open network, there isn't any way how to do it. I can't connect to station when it's not password protected.

Well, not entirely whatever password, it has to be a password shorter than 8 characters.


band service more restful

combine filter and get all

findbands works for getting all the bands and filters now


Add collaborators to this repo

Comment below to be added as a collaborator
We will all be using this repository today. Even though it's public right now, you will not be able to make changes until you're given the correct permissions. We've automated this process via the GitHub API. Once you comment, we will add you as a collaborator.
You'll start to receive a lot of emails. (❗) You can slow these down immediately by clicking the Unwatch button at the top of the page and selecting either the Participating and @mentions or Ignore option.

Hi!

;aldfj

Adding comment ...

Hello

Here's a comment

yep

New software!

Okay, catching up!

Hello

1

Greetings

Hello Vyas

Hello, missed the first part of yesterday's presentation. Do I need permission to create a branch? I


Selection of account type and its title

Prerequisites
Please answer the following questions for yourself before submitting an issue. YOU MAY DELETE THE PREREQUISITES SECTION.

[ ] I am running the latest version
[ ] I checked the documentation and found no answer
[X] I checked to make sure that this issue has not already been filed

Define You

[X] GSSoC22 Participant
[X] Contributor

Describe the bug
Selection of account type and its title
A clear and concise description of what the bug is.
Steps to Reproduce
Steps to reproduce the behavior:

Expected Behavior
Better selection with highlight just the image and not extra spaces.
Add title for accounts
Please describe the behavior you are expecting.
Current Behavior
What is the current behavior?
Screenshots Or Video
Attach screenshots or video if applicable

Failure Logs
Please include any relevant log snippets or files here.

@Phalak-Sharma any updates?

assign me
/assign

Please assign this issue to me @coding-geek21


Allow Flux 0.12

I just see it now. I will fix it soon!

@ChrisRackauckas Would almost ready to tag new version.

Please update to v0.7.6.

If it is checked, I just close this PR.


Add configurable replicas to stunner-auth template

Closes #39.
I've made two changes actually. First, the default behavior will be to not specify replicas at all. This is pretty much the same as replicas: 1 but it has the added benefit that you can use autoscaling and not have tools like ArgoCD or Flux fight with the autoscaling since from their perspective replicas is not managed by the IaC tools.
Second, replicas is now configurable via values.yaml. Any number specified is set as expected.
Replicas is intentionally left commented out in values.yaml to prevent any potential autoscaling from interfering in the default case.
I've tested this locally via helm template and verified it functions as described above.

@iterion I changed and committed the changes I requested. I'm working on uploading the chart to artifacthub and did not want to wait. I want to thank you for your participation. Let us know if you have some other suggestions as well.

@davidkornel Thanks for merging this! The reasoning for the if statement is actually in my original PR description. :see_no_evil: tl;dr is it helps with autoscaling. This is understandably an edge case, but when you do want to autoscale something it can be a bummer when your Gitops tool keeps fighting with your autoscaling.
Thanks for shipping a new version so quickly!

@iterion Ahh dang it. You're right. Does this affect you? If not I would leave it like this for now and if it indeed a problem we can adapt. WDYT?

Not a problem at the moment, we can definitely address it in a future PR. Thanks!


Add notice about Safari not being supported

Coverage decreased (-7.9%) to 67.042% when pulling 4e49eac4cf61abc08dc52ba2226afdb04541d86d on add-safari-notice into a593b23396761110badbd2264843a04bd63c9d42 on main.

Coverage decreased (-7.9%) to 67.042% when pulling 4e49eac4cf61abc08dc52ba2226afdb04541d86d on add-safari-notice into a593b23396761110badbd2264843a04bd63c9d42 on main.


Update list of paths with ExecutableFilePermissions

Fixes crash on launch on Linux:
[FATAL:double_fork_and_exec.cc(131)] execv .local-chromium/Linux-970485/chrome-linux/chrome_crashpad_handler: Permission denied (13)

See #1774

Were you able to merge into master with no approval? 🫢

@kblok No, haha, commit faec0b1 was already on master, GitHub got confused and closed my PR. Please refer to #2004


DOC: Should Result Classes be its own top level section?

Issue with current documentation:
Currently the SciPy documentation for version 1.10 shows Result Classes as its own, top level, section. I don't believe this is correct, as they are only results for stats. Should this come under scipy.stats.result_classes?
Idea or request for content:
Move results classes to scipy.stats.result_classes
Additional context (e.g. screenshots, GIFs)

It has been like that for a few versions but when I fight with Sphinx, Sphinx usually wins.
Ideally, it would not show up anywhere. These are classes that we want to be linked from the corresponding functions; e.g. in scipy.stats.contingency.relative_risk:

But these are not importable or intended to be used.
from scipy.stats import RelativeRiskResult  # should raise ImportError
# ImportError: cannot import name 'FitResult' from 'scipy.stats' (C:\Users\matth\scipy\scipy\stats\__init__.py)

So ideally, they would not appear anywhere except via links from the functions that use them.

Ok. Thanks for the explanation. I'm going to leave this open and, if someone comes along who wants to take a crack at it, great.

Well, if you can't completely fix it, would you move to the very bottom of the stats page? That way it is at least less visible.

I can try, although I have very little experience with Sphinx so I'm not sure how far I'll get.

You could include a note that the classes are not importable and they are documented only because instances are returned by other functions.

This is a very important point to make as not common in other libraries. Normally all specific exceptions or other return objects are also part of the public API. So as we go another way it needs to be documented.
About the toc, there are other elements I would remove from there as well, we have too many things in the sidebar ATM.

@tupui I've tried to remove it but haven't gotten very far. I've been able to delete the file and build the documentation which leads to the following warnings:
checking consistency... /home/kai/Projects/SciPy/kai/scipy/doc/source/reference/generated/scipy.stats._result_classes.BinomTestResult.rst: WARNING: document isn't included in any toctree
/home/kai/Projects/SciPy/kai/scipy/doc/source/reference/generated/scipy.stats._result_classes.FitResult.rst: WARNING: document isn't included in any toctree
/home/kai/Projects/SciPy/kai/scipy/doc/source/reference/generated/scipy.stats._result_classes.OddsRatioResult.rst: WARNING: document isn't included in any toctree
/home/kai/Projects/SciPy/kai/scipy/doc/source/reference/generated/scipy.stats._result_classes.PearsonRResult.rst: WARNING: document isn't included in any toctree
/home/kai/Projects/SciPy/kai/scipy/doc/source/reference/generated/scipy.stats._result_classes.RelativeRiskResult.rst: WARNING: document isn't included in any toctree
/home/kai/Projects/SciPy/kai/scipy/doc/source/reference/generated/scipy.stats._result_classes.TtestResult.rst: WARNING: document isn't included in any toctree
/home/kai/Projects/SciPy/kai/scipy/doc/source/reference/generated/scipy.stats._result_classes.TukeyHSDResult.rst: WARNING: document isn't included in any toctree
done
preparing documents... done
/home/kai/Projects/SciPy/kai/scipy/doc/source/reference/stats.rst:6: WARNING: toctree contains reference to nonexisting document 'reference/stats._result_classes'
/home/kai/Projects/SciPy/kai/scipy/doc/source/reference/stats.rst:6: WARNING: toctree contains reference to nonexisting document 'reference/stats._result_classes'
/home/kai/Projects/SciPy/kai/scipy/doc/source/reference/stats.rst:6: WARNING: toctree contains reference to nonexisting document 'reference/stats._result_classes'
/home/kai/Projects/SciPy/kai/scipy/doc/source/reference/stats.rst:6: WARNING: toctree contains reference to nonexisting document 'reference/stats._result_classes'
/home/kai/Projects/SciPy/kai/scipy/doc/source/reference/stats.rst:6: WARNING: toctree contains reference to nonexisting document 'reference/stats._result_classes'
/home/kai/Projects/SciPy/kai/scipy/doc/source/reference/stats.rst:6: WARNING: toctree contains reference to nonexisting document 'reference/stats._result_classes'

Any idea of where to go from here?

Thanks Kai. So yeah every document needs to be present in a toctree. What you need to do is to have a hidden tree. Basically on an existing tree you can add :hidden:. And it will not be included, but still there for Sphinx.

It looks like it's already hidden but being ignored:
https://github.com/scipy/scipy/blob/main/doc/source/reference/stats.rst

ok so you are saying you removed the file stats._result_classes.rst and you got the warnings?

So I found the culprit. It's due to the interaction of includehidden=True we have in doc/source/_templates/sidebar-nav-bs.html and the fact that in doc/source/reference/stats.rst we have on the same level the hidden sections.
First option is doing includehidden=False would mean that we need to rewrite the toctree in the index.rst (and other places maybe?).
So I propose instead to remove the hidden toctree and reference all that in the init of stats as Matt suggested. I will make a PR and ping you.
We might want to do similar things for other submodules. (e.g. linalg)


Simplified parallelism & cross-validation

Here's a sketch of a two-step cross-validation process where first we perform data preparation/imputation in parallel followed by parallel evaluation of the inner CV loop:
def transform_work(train_dataset, test_dataset):
    transformer = PrepareData()
    train2 = transformer.fit_transform(train_dataset)
    test2 = transformer.transform(test_dataset)
    return [Broadcast(train2), Broadcast(test2)]

 dataset_splits = kubeface.map(transform_work, cv_generator)

def cv_work(train_broadcast, test_broadcast, hyperparams):
        allele_to_model_dict = train_allele_specific_models(train_broadcast.value)
        allele_to_auc_dict = evaluate_aucs(models, test_broadcast.value)
        return (hyperparams, allele_to_model_dict, allele_to_auc_dict)

all_inputs = [
    (train_broadcast, test_broadcast, hyperparams) for
    for (train_broacast, test_broadcast) in dataset_splits:
    for hyperparams in hyperparameter_generator]

all_results = kubeface.map(cv_work, all_inputs)

grouped_results = # group all_results by hyperparams objects into lists of AUC & model dicts

The assumption here is that all allele-specific work happens inside train_allele_specific_models and evaluate_aucs -- the CV code doesn't have to know about it. In order to achieve CV splits that are well balanced across both allele sizes and binding labels we should probably use a stratified iterator over a label that combines allele and IC50<=500nM.

No longer doing imputation so closing for now


bugfix: pass error up if WALWathcer.segments() return err

Signed-off-by: YaoZengzeng<EMAIL_ADDRESS>

ping @cstyan

@YaoZengzeng did you run into an actual bug that was caused by this error not being bubbled up? If so, can you add a test?
Otherwise lgtm.

@cstyan
I have not encountered it in practice actually.
But in theory, if the segments are not sequential, the segments() would return error and we should bubble up the error ASAP, instead of replaying the WAL until encounter the missed segments.
In addition, the functions with similar function in TSDB bubbles up the error when list segments failed.
https://github.com/prometheus/tsdb/blob/master/wal/wal.go#L634
So it's reasonable to bubble up the error here :)

Yes, the error should be bubbled up. I was just wondering if you'd come across an actual bug.

But in theory, if the segments are not sequential, the segments() would return error and we should bubble up the error ASAP, instead of replaying the WAL until encounter the missed segments.

In this particular case Prometheus will exit with an error on startup before the WAL watcher code even starts :)

@cstyan
I just read through the WAL and TSDB related code to try to figure out why memory surge when WAL watcher and remote write are enabled in Prometheus v2.8.0
And find this bug by accident, haha...

2.8 included a rewrite of remote write that will use more memory in the happy path than previous versions, but in failure cases it should use less memory than before, and not lose samples or OOM your Prometheus.
There have been numerous small improvements since then though, if you're having issues with memory usage when using remote write try 2.10/2.11 and then open an issue (and ping me!) if you're still seeing something problematic.

@cstyan
Yes , thanks!
Also, I'd to contribute if there are issues to be fixed :joy:

LGTM from me too.


I'm building a SSH server using the Node.js library https://github.com/mscdex/ssh2 and I'm building the SSH client using the x/crypto/ssh library. The client aborts the SSH handshake when using the publickey authentication method.
I built an example project including server and client here: https://github.com/0x7f/ssh-authentication-bug
What did you see happen?
Even though the server offers [publickey] authentication methods when initiating the handshake with authentication method none, the client fails to connect. The client uses the publickey method and offers the key, and the server accepts the key, but the client still prints the error:
2024/03/21 09:07:40 Failed to dial: ssh: handshake failed: ssh: unable to authenticate, attempted methods [none publickey], no supported methods remain

What did you expect to see?
When using OpenSSH client, it connects successfully. I would expect the same from the Golang x/crypto/ssh client.
Maybe related to https://github.com/golang/go/issues/64785

@golang/security

Quick not about findings from the issue over at ssh2 (the Node.js SSH2 server library). When switching from RSA to ED25519 keys, the handshake works as expected. So seems the issue is related to RSA keys somehow.

@0x7f thank you for providing a reproducer. I think the linked CL fixes the reported issue. Can you please confirm?
According to RFC 4252 Section 7 the algorithm in SSH_MSG_USERAUTH_REQUEST should match that of the request, so our code was correct, but some server send the key type instead.
OpenSSH checks the key type (maybe to avoid problems with these buggy servers) so I think we should do the same

I ran into this problem too. Through some gross debugging I found that it was failing here. My public key algo was ssh-rsa and the server algo was rsa-sha2-256, as they weren't the same, it returned false here.
I managed to get around it by changing the code (I forked the repo to unblock myself) to check if the algo matched any of the algos returned in algorithmsForKeyFormat(msg.Algo). Not sure if it is the right thing to do here though.

I ran into this problem too. Through some gross debugging I found that it was failing here. My public key algo was ssh-rsa and the server algo was rsa-sha2-256, as they weren't the same, it returned false here.
I managed to get around it by changing the code (I forked the repo to unblock myself) to check if the algo matched any of the algos returned in algorithmsForKeyFormat(msg.Algo). Not sure if it is the right thing to do here though.

Can you please confirm that this issue is fixed by applying this CL? Thank you

@drakkan the CL fixes it for me 👍 Thanks a lot.
I had trouble referencing the CL in my test project, so I manually applied the changes to the local code in $GOPATH.

@drakkan any updates on this one? I'm not familiar with Gerrit, but to me it looks like the CL was not merged yet, right?

@drakkan any updates on this one? I'm not familiar with Gerrit, but to me it looks like the CL was not merged yet, right?

It is in the review queue. It should be merged soon, I think another Google employee should approve it

I encountered a similar issue. My Go version is go1.21.9 and crypto is v0.30.0. I fixed it during debugging by specifying the key .
	sshConfig := &ssh.ClientConfig{
		User: username,
		Auth: []ssh.AuthMethod{
			ssh.PublicKeys(signers...),
		},
		HostKeyCallback: ssh.InsecureIgnoreHostKey(),
		HostKeyAlgorithms: []string{
			ssh.KeyAlgoRSASHA256,
			ssh.KeyAlgoRSASHA512,
			ssh.KeyAlgoED25519,
			ssh.KeyAlgoECDSA256,
		},
		Timeout: 15 * time.Second,
	}

I encountered a similar issue. My Go version is go1.21.9 and crypto is v0.30.0. I fixed it during debugging by specifying the key . My situation involves accessing OpenSSH_8.8 from OpenSSH_7.4.
	sshConfig := &ssh.ClientConfig{
		User: username,
		Auth: []ssh.AuthMethod{
			ssh.PublicKeys(signers...),
		},
		HostKeyCallback: ssh.InsecureIgnoreHostKey(),
		HostKeyAlgorithms: []string{
			ssh.KeyAlgoRSASHA256,
			ssh.KeyAlgoRSASHA512,
			ssh.KeyAlgoED25519,
			ssh.KeyAlgoECDSA256,
		},
		Timeout: 15 * time.Second,
	}


Update Caliper and Adiak spack packages

Add versions 2.8.0, 2.9.0 and 2.9.1 of Caliper (update to a release in Jan 24, 2023).
Add version 0.2.2 of Adiak that is newer than the 0.3.0-alpha tag.

@tepperly do you know if this builds on clang compilers? im trying newer version on my Toss4 pr, but i seem to be getting libunwind errors
     589    /usr/WS2/meemee/serac/serac_libs/toss_4_x86_64_ib/2023_05_12_17_52_
            06/spack/lib/spack/env/clang/clang++ -O3 -DNDEBUG CMakeFiles/cali-s
            tat.dir/cali-stat.cpp.o -o cali-stat  ../util/libcaliper-tools-util
            .a ../../libcaliper-serial.a /usr/lib64/libdw.so /usr/tce/packages/
            clang/clang-14.0.6/lib/libunwind.so /usr/WS2/meemee/serac/serac_lib
            s/toss_4_x86_64_ib/2023_05_12_17_52_06/clang-14.0.6/adiak-0.2.2-rys
            bzdxdzdche5tjsqz7zy46kquzknbb/lib/libadiak.a /usr/tce/packages/mvap
            ich2/mvapich2-2.3.6-clang-14.0.6/lib/libmpi.so /usr/tce/packages/mv
            apich2/mvapich2-2.3.6-clang-14.0.6/lib/libmpicxx.so -ldl /usr/lib64
            /librt.so -pthread
     590    /opt/rh/gcc-toolset-10/root/usr/lib/gcc/x86_64-redhat-linux/10/../.
            ./../../bin/ld: ../../libcaliper-serial.a(Callpath.cpp.o): in funct
            ion `(anonymous namespace)::Callpath::callpath_service_register(cal
            i::Caliper*, cali::Channel*)':
  >> 591    Callpath.cpp:(.text+0x491): undefined reference to `_Ux86_64_getcon
            text'

i think we can just use the builtin spack recipes, since the only difference was the static patch.

Closing due to being taken care of in the TOSS4 PR by alex.


Build options

Fixes #10
Make GPU and MPI options available for simulation engines, but also create "vanilla" build for simple use in CI testing.
Separate simulation engines into different containers to make them lightweight

Fair enough, my thinking is that for actual use after the workshop but before we actually merge this code into an i.e. gromacs release, people would want to have a container with just the simulation engine they want to use rather than a massive one. Though it makes sense to me to have one demo/test container where we dump everything

@ljwoods2

I can create a "common" one. Should this have all the different GPU/threading options as well? Or just GPU and nothing else?

For common I would do one common all GPU and one common all CPU with our most useful config for each (no MPI)

@hmacdope Didn't end up doing a multi-stage build since this would've required installing another nearly-identical conda environment into the second stage without gcc, cmake, cudatools, etc (all the build specific dependencies) which felt redundant. This should be a "good enough" solution that makes the containers lighter by a few GB.

Going to merge and test this, made a backup of main on "archive" branch in case we need to hard reset for some reason


Not working on Mac

Maybe I'm missing something, but I just can not seem to get this to work on macOS. I can see the usb dongle in my /dev directory in macOS but I can't get it to connect to the container. Debugging a little I found the error: docker: Error response from daemon: linux runtime spec devices: error gathering device information while adding custom device "/dev/ttyUSB0": lstat /dev/ttyUSB0: no such file or directory.
I can resolve this error by adding --privileged to the run params, but the container still doesn't seem to mount the device internally. If I connect to the container and ls /dev, the device just isn't there.
What am I missing? I followed the directions under "Docker for Mac" to the letter, using explicit ports instead of shared host networking. I can connect to the Phoscon web app just fine but since the usb device doesn't exist there isn't much I can do.

If the problem is that Conbee is not created as /dev/ttyUSB0 on Mac host, so the Docker daemon can’t find it, what will be then the solution?
Sorry I’m just in the way of learning..
Thanks for your help

The solution is to pass the correct device into the container (ie sub /dev/ttyUSB0 for what your device is actually created as).

I appreciate your quick response. Unfortunately I'm not still able to run deCONZ on my Mac mini.
This is what I've done so far:

Installed Docker for Mac v. <IP_ADDRESS> (it is running)
Inserted CONBEE in a USB port (nothing else is connected)
Gave command:
docker run -d
--name=deconz
-p 80:80
-p 443:443
--restart=always
-v /opt/deconz:/root/.local/share/dresden-elektronik/deCONZ
--device=/dev/ttyUSB0
-e DECONZ_WEB_PORT=80
-e DECONZ_WS_PORT=443
marthoc/deconz

Then, the error pops up:
Error response from daemon: linux runtime spec devices: error gathering device information while adding custom device "/dev/cu.usbserial-DO01MNGG": no such file or directory.
as you can see, I tried to pass the correct device, searching into: ls /dev
but whatever I insert as device, the error is always the same.
Once again, thanks for your time and help

@fregol
I received a Conbee (1) USB stick the other day and trying to get it to work on my Mac also, running Mojave latest stable and Docker with marthoc/docker-deconz.
My programming skills are not good though.
Have you been able to get it to work? If so, how?
With ls /dev I find my Conbee is named tty.usbserial-DM01BF0Y. I can install with --privileged otherwise I get errors;
docker: Error response from daemon: error gathering device information while adding custom device "/dev/tty.usbserial-DM01BF0Y": no such file or directory.
But with privileged I can access Phoscon (via my Macs IP adress) but the Conbee stick shows up as "Firmware: Not connected" and I cannot add any sensors or other devices..
Hoping to get some help here.

@mrystrand USB passthrough with Apple Hypervisor might be the culprit, see https://github.com/marthoc/docker-deconz/issues/146#issuecomment-541205794


Unintended opening of editor windows when using the numbers T_[Unintended Action] P_[Medium] F_[Always]

Date of Detection

18/02/2021

Description

Hotkeys to open certain editor windows cause a conflict with inputing numbers in input boxes.

Reproduction

Pressing number keys.

Expected Behaviour

Not opening editor windows when inputing numbers.

Engine Version

0.1

Suggested Assignees

Angel Gonzalez

This Issue has been fixed.

This issue has been fixed.


remove the obsolete mspa in sepal

the MSPA implementation provided in this module is far more advanced than the previous one, MSPA analysis (R and Python) needs to be removed from the plateform
same for the mspa implementation in the gfc tool

Ok, maybe mention that the (feature-limited) MSPA-implementation can be downloaded from the MSPA-homepage: GIS-plugin with documentation and installation info for ArcGIS, QGIS3 and R.

@vogtpet Mentioning it in the "about" section of the module ?

yes, good idea!

done here : https://github.com/openforis/sepal/issues/83


Fix normal-related bugs exposed by computing a normal loss

Detach all inputs in compute_normal, similar to https://github.com/lab4d-org/lab4d/commit/f231a6a27c279b61ee14f240c19ae1b567452f77
Fix the CPU implementation of _quaternion_mul, previously it was not consistent with the CUDA version as it did not support quaternion_3D_mul_4D and quaternion_4D_mul_3D
Make input and grad tensors contiguous in lab4d/third_party/quaternion/quaternion.py. Computing a normal loss requires differentiating through lab4d/nnutils/nerf.py::compute_normal(), which previously caused the following error:

/home/jefftan/.miniconda3/envs/lab4d/lib/python3.9/site-packages/torch/autograd/__init__.py:200: UserWarning: Error detected in _Quaternion_mul_backwardBackward. Traceback of forward call that caused the error:
  File "/home/jefftan/.miniconda3/envs/lab4d/lib/python3.9/site-packages/torch/autograd/function.py", line 274, in apply
    return user_fn(self, *args)
  File "/home/jefftan/.miniconda3/envs/lab4d/lib/python3.9/site-packages/torch/cuda/amp/autocast_mode.py", line 123, in decorate_bwd
    return bwd(*args, **kwargs)
  File "/home/jefftan/papers/lab4d-clothing/lab4d/utils/../third_party/quaternion/quaternion.py", line 84, in backward
    grad_inputs_1, grad_inputs_2 = _quaternion_mul_backward(grad, inputs_1, inputs_2)
  File "/home/jefftan/.miniconda3/envs/lab4d/lib/python3.9/site-packages/torch/autograd/function.py", line 506, in apply
    return super().apply(*args, **kwargs)  # type: ignore[misc]
 (Triggered internally at /opt/conda/conda-bld/pytorch_1682343997789/work/torch/csrc/autograd/python_anomaly_mode.cpp:114.)
  Variable._execution_engine.run_backward(  # Calls into the C++ engine to run the backward pass
/home/jefftan/.miniconda3/envs/lab4d/lib/python3.9/site-packages/torch/autograd/__init__.py:200: UserWarning:

Previous calculation was induced by _Quaternion_mulBackward. Traceback of forward call that induced the previous calculation:
  File "/home/jefftan/papers/lab4d-clothing/lab4d/train.py", line 51, in <module>
    app.run(main)
  File "/home/jefftan/.miniconda3/envs/lab4d/lib/python3.9/site-packages/absl/app.py", line 308, in run
    _run_main(main, args)
  File "/home/jefftan/.miniconda3/envs/lab4d/lib/python3.9/site-packages/absl/app.py", line 254, in _run_main
    sys.exit(main(argv))
  File "/home/jefftan/papers/lab4d-clothing/lab4d/train.py", line 46, in main
    train_ddp(Trainer)
  File "/home/jefftan/papers/lab4d-clothing/lab4d/train.py", line 40, in train_ddp
    trainer.train()
  File "/home/jefftan/papers/lab4d-clothing/lab4d/engine/trainer.py", line 231, in train
    self.run_one_round(round_count)
  File "/home/jefftan/papers/lab4d-clothing/lab4d/engine/trainer.py", line 251, in run_one_round
    self.train_one_round(round_count)
  File "/home/jefftan/papers/lab4d-clothing/lab4d/engine/trainer.py", line 343, in train_one_round
    loss_dict = self.model(batch)
  File "/home/jefftan/.miniconda3/envs/lab4d/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/jefftan/.miniconda3/envs/lab4d/lib/python3.9/site-packages/torch/nn/parallel/distributed.py", line 1156, in forward
    output = self._run_ddp_forward(*inputs, **kwargs)
  File "/home/jefftan/.miniconda3/envs/lab4d/lib/python3.9/site-packages/torch/nn/parallel/distributed.py", line 1110, in _run_ddp_forward
    return module_to_run(*inputs[0], **kwargs[0])  # type: ignore[index]
  File "/home/jefftan/.miniconda3/envs/lab4d/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/jefftan/papers/lab4d-clothing/lab4d/engine/model.py", line 73, in forward
    results = self.render(batch, flow_thresh=config["train_res"])
  File "/home/jefftan/papers/lab4d-clothing/lab4d/engine/model.py", line 233, in render
    results = self.render_samples_chunk(samples_dict, flow_thresh=flow_thresh)
  File "/home/jefftan/papers/lab4d-clothing/lab4d/engine/model.py", line 303, in render_samples_chunk
    results_chunk = self.render_samples(
  File "/home/jefftan/papers/lab4d-clothing/lab4d/engine/model.py", line 341, in render_samples
    multifields_dict, deltas_dict, aux_dict = self.fields.query_multifields(
  File "/home/jefftan/papers/lab4d-clothing/lab4d/nnutils/multifields.py", line 333, in query_multifields
    ) = field.query_field(
  File "/home/jefftan/papers/lab4d-clothing/lab4d/nnutils/deformable.py", line 318, in query_field
    feat_dict, deltas, aux_dict = super().query_field(
  File "/home/jefftan/papers/lab4d-clothing/lab4d/nnutils/feature.py", line 106, in query_field
    feat_dict, deltas, aux_dict = super(FeatureNeRF, self).query_field(
  File "/home/jefftan/papers/lab4d-clothing/lab4d/nnutils/nerf.py", line 679, in query_field
    jacob_dict = self.compute_jacobian(
  File "/home/jefftan/papers/lab4d-clothing/lab4d/nnutils/nerf.py", line 773, in compute_jacobian
    jacob_dict["eikonal"], jacob_dict["normal"] = self.compute_normal(
  File "/home/jefftan/papers/lab4d-clothing/lab4d/nnutils/nerf.py", line 493, in compute_normal
    g = compute_gradient(fn_sdf, xyz_cam)[..., 0]
  File "/home/jefftan/.miniconda3/envs/lab4d/lib/python3.9/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
    return func(*args, **kwargs)
  File "/home/jefftan/papers/lab4d-clothing/lab4d/utils/torch_utils.py", line 10, in compute_gradient
    y = fn(x)
  File "/home/jefftan/papers/lab4d-clothing/lab4d/nnutils/nerf.py", line 482, in fn_sdf
    xyz = self.backward_warp(
  File "/home/jefftan/papers/lab4d-clothing/lab4d/nnutils/deformable.py", line 140, in backward_warp
    xyz, warp_dict = self.warp(
  File "/home/jefftan/.miniconda3/envs/lab4d/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/jefftan/papers/lab4d-clothing/lab4d/nnutils/warping.py", line 467, in forward
    out, warp_dict = super().forward(
  File "/home/jefftan/papers/lab4d-clothing/lab4d/nnutils/warping.py", line 326, in forward
    out = dual_quaternion_skinning(se3, xyz, skin_prob)
  File "/home/jefftan/papers/lab4d-clothing/lab4d/utils/geom_utils.py", line 71, in dual_quaternion_skinning
    pts = dual_quaternion_apply((qr_w, qd_w), pts)
  File "/home/jefftan/papers/lab4d-clothing/lab4d/utils/quat_transform.py", line 393, in dual_quaternion_apply
    return quaternion_translation_apply(q, t, point)
  File "/home/jefftan/papers/lab4d-clothing/lab4d/utils/quat_transform.py", line 280, in quaternion_translation_apply
    p = quaternion_apply(q, point)
  File "/home/jefftan/papers/lab4d-clothing/lab4d/utils/quat_transform.py", line 267, in quaternion_apply
    quaternion_point = quaternion_mul(quaternion, point)
  File "/home/jefftan/papers/lab4d-clothing/lab4d/utils/quat_transform.py", line 109, in quaternion_mul
    return _quaternion_mul_cuda(
  File "/home/jefftan/.miniconda3/envs/lab4d/lib/python3.9/site-packages/torch/autograd/function.py", line 506, in apply
    return super().apply(*args, **kwargs)  # type: ignore[misc]
 (Triggered internally at /opt/conda/conda-bld/pytorch_1682343997789/work/torch/csrc/autograd/python_anomaly_mode.cpp:121.)
  Variable._execution_engine.run_backward(  # Calls into the C++ engine to run the backward pass
Traceback (most recent call last):
  File "/home/jefftan/papers/lab4d-clothing/lab4d/train.py", line 51, in <module>
    app.run(main)
  File "/home/jefftan/.miniconda3/envs/lab4d/lib/python3.9/site-packages/absl/app.py", line 308, in run
    _run_main(main, args)
  File "/home/jefftan/.miniconda3/envs/lab4d/lib/python3.9/site-packages/absl/app.py", line 254, in _run_main
    sys.exit(main(argv))
  File "/home/jefftan/papers/lab4d-clothing/lab4d/train.py", line 46, in main
    train_ddp(Trainer)
  File "/home/jefftan/papers/lab4d-clothing/lab4d/train.py", line 40, in train_ddp
    trainer.train()
  File "/home/jefftan/papers/lab4d-clothing/lab4d/engine/trainer.py", line 231, in train
    self.run_one_round(round_count)
  File "/home/jefftan/papers/lab4d-clothing/lab4d/engine/trainer.py", line 251, in run_one_round
    self.train_one_round(round_count)
  File "/home/jefftan/papers/lab4d-clothing/lab4d/engine/trainer.py", line 345, in train_one_round
    total_loss.mean().backward()
  File "/home/jefftan/.miniconda3/envs/lab4d/lib/python3.9/site-packages/torch/_tensor.py", line 487, in backward
    torch.autograd.backward(
  File "/home/jefftan/.miniconda3/envs/lab4d/lib/python3.9/site-packages/torch/autograd/__init__.py", line 200, in backward
    Variable._execution_engine.run_backward(  # Calls into the C++ engine to run the backward pass
  File "/home/jefftan/.miniconda3/envs/lab4d/lib/python3.9/site-packages/torch/autograd/function.py", line 274, in apply
    return user_fn(self, *args)
  File "/home/jefftan/.miniconda3/envs/lab4d/lib/python3.9/site-packages/torch/autograd/function.py", line 522, in wrapper
    outputs = fn(ctx, *args)
  File "/home/jefftan/.miniconda3/envs/lab4d/lib/python3.9/site-packages/torch/cuda/amp/autocast_mode.py", line 123, in decorate_bwd
    return bwd(*args, **kwargs)
  File "/home/jefftan/papers/lab4d-clothing/lab4d/utils/../third_party/quaternion/quaternion.py", line 38, in backward
    _backend.quaternion_mul_backward_backward(grad_out_1, grad_out_2,
RuntimeError: grad_out_2 must be a contiguous tensor

Let's keep the changes away from the main branch for now.
The reason is as follows:

Those are not bugs that affect the performance of the main branch but rather new features if someone wants to supervise with normal loss.
We don't have a default testing workflow that verifies whether the system works after applying those changes. Before developing such a workflow, I'll be extra cautious in merging modifications on the basic operations.


Bug:  Database connection

Describe the bug
This is problem in DEVELOPMENT environment in app that use multiple database.
I try to insert record in other database table (not default) and insert failed  when I use is unique validation rule  'name'  => 'required|is_unique[' . $this->table . '.name],  set up on model.
During the validation process is used default DB-group, not custom that is set $this->db = \Config\Database::connect('custom_database);.

This problem does not appear on production and testing environment  or on dev env when I don't use unique validation rule
CodeIgniter 4 version
CI 4
Expected behavior, and steps to reproduce if appropriate
I expected that the record will be insert in custom database table
Context

Ubuntu 16
Web server: Apache 2.0 Handler
PHP version: 7.4

How are you handling validation? In the controller, or automatically in the model?
The model should automatically handle it. If doing it in the controller, it has no way to know, so you have to explicitly tell it so. Looks like we could add a parameter to the Controller's validate method for that, though.
In the meantime, you can always run validation manually in a controller, passing in the database group to the run method.

Validation is handle on model.
I have user model that is subclass of abstract class. (Class RoleModel extends AbstractCrud{...})
Abstract class is subclass of CI model class (CodeIgniter\Model). (abstract class AbstractCrud extends Model{..})
In user model and abstract class I called parent::__construct(); in constructot. In that case CI4 did zigzag with db groups (default custom). That (in combination with is_unique) produced err that table in default db does not exist.

#0 /var/www/html/app_folder/vendor/codeigniter4/framework/system/Database/MySQLi/Connection.php(331): mysqli->query()
#1 /var/www/html/app_folder/vendor/codeigniter4/framework/system/Database/BaseConnection.php(738): CodeIgniter\Database\MySQLi\Connection->execute()
#2 /var/www/html/app_folder/vendor/codeigniter4/framework/system/Database/BaseConnection.php(666): CodeIgniter\Database\BaseConnection->simpleQuery()
#3 /var/www/html/app_folder/vendor/codeigniter4/framework/system/Database/BaseBuilder.php(1845): CodeIgniter\Database\BaseConnection->query()
#4 /var/www/html/app_folder/vendor/codeigniter4/framework/system/Validation/Rules.php(234): CodeIgniter\Database\BaseBuilder->get()
#5 /var/www/html/app_folder/vendor/codeigniter4/framework/system/Validation/Validation.php(294): CodeIgniter\Validation\Rules->is_unique()
#6 /var/www/html/app_folder/vendor/codeigniter4/framework/system/Validation/Validation.php(189): CodeIgniter\Validation\Validation->processRules()
#7 /var/www/html/app_folder/vendor/codeigniter4/framework/system/Model.php(1443): CodeIgniter\Validation\Validation->run()
#8 /var/www/html/app_folder/vendor/codeigniter4/framework/system/Model.php(701): CodeIgniter\Model->validate()
#9 /var/www/html/app_folder/app/AbstractExtension/AbstractCrud.php(28): CodeIgniter\Model->insert()
....
When I call parent::__construct(); only in user model (or abstract class) or in case whitout calling parent::__construct();, I get this err

Only case when everything is OK is when I don't call methods in abstract class.
By my opinion,  problem is that CI returns errors in case of this architecture (model has parent class, parent class is subclass of CI model class (CodeIgniter\Model)) if working with multiple database

I solved my problem on this way:
public function __construct()
{
if (ENVIRONMENT !== 'testing') {
$this->DBGroup = 'CUSTOM DB GROP';
// $this->setDatabase();
// $this->db = \Config\Database::connect($this->getDatabase());
}
parent::__construct();
}
Set DBGroup not $this->db = \Config\Database::connect('CUSTOM DB GROP');

I believe that this is still a bug. The following code generates a Unable to connect to the database. error with the validation in place.
If I change the connection to $db = db_connect(); it works perfectly.
If I remove $model->setValidationRules($validationRules); it works fine - but obviously does not validate.
$db = db_connect('live'); $model = model('UserModel', true, $db); $validationRules    = [     'first_name'    => 'required',     'last_name'    => 'required',     'email'     => 'required|valid_email|is_unique[users.email]',     'password'     => 'required|min_length[8]',     'password_confirmation' => 'required_with[password]|matches[password]' ]; $model->setValidationRules($validationRules); $data = [     'first_name' => $this->request->getVar('first_name'),     'last_name' => $this->request->getVar('last_name'),     'email' => $this->request->getVar('email'),     'password' => $this->request->getVar('password'),     'password_confirmation' => $this->request->getVar('password_confirmation') ]; $id = $model->save($data);


Add ayush to contributors list

@ayushagg31, I'm quite elated about your PR. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a PR and learned that I merged it?

Thanks, @Roshanjossey  for merging my pull request.It's a great initiative for newcomers like me to help them create their first PR.
Keep up the good work. All the best.

We also have a slack team to help/mentor people trying to contribute to open source projects.
You can join the team by going to https://firstcontributions.herokuapp.com


SRAM compiler

Thank you for the work that you are doing. A question: for the Skywater-PDK, do free SRAM compilers exist that support the OpenROAD flow? VLSIDA/OpenRAM does not appear to support Skywater and I see that you folks have a "fake ram" module that looks like a single ported SRAM. Just wondering what people are using.

OpenRAM should support SkyWater "real soon now" -- please poke @mguthaus about the current status.

I think this is more in the scope of openlane (https://github.com/efabless/openlane) as openroad itself doesn't provide this functionality.


Back to last channel in Mattermost beta

Hi!
Using the Mattermost beta on Android. After opening the list of channels I don't see a way to get back to the last channel I had open. This was possible in the current/old app by just swiping the channel list (checking for unread messages) and then swiping it again to get back to the channel I was following last. It would be good to have a "back to last channel" feature in the new app.
Thanks,
Sebastian

Thanks Sebastian for your feedback.
The beta app - aka v2 app - is a redesign and yes, the navigation flow is a bit different.
@esethna I'm assigning you to this issue.  IIRC, you were looking into this together with @matthewbirtch ?

100% agree @schmitts, we should be supporting swiping gestures to return to the last viewed channel. I will open a tix for this: https://mattermost.atlassian.net/browse/MM-45904

@schmitts how is the rest of your experience with the app. Please share any feedback you have in our development channel: https://community.mattermost.com/core/channels/ship-gekidou--mobile-v2


Route stopped working when upgrading 4.0.1 -> 4.0.5

Version
Tell us which versions you are using:

react-native-router-flux v4.0.5
react 16.6.0-alpha.8af6728
react-native 0.57.3

Expected behaviour
Tab2 is a settings view. Clicking a row should open the settings for that row.
See router below.
Actual behaviour
When in tab2, nothing happens when pressing the row that should open tab2Scene2. This worked in version 4.0.1.
Steps to reproduce
When in tab2, I should be able to open the scene 'tab2Scene2', and from there open the modal 'tab2Scene2Popup'.
<Router>
<Scene key="root" hideNavBar>
    <Tabs key="tabs" >
        <Scene
            key="tab1"
            component={Tab1}
            title="tab1"
        />
        <Stack
            key="tab2"
            title="tab2"
        >
            <Modal hideNavBar>
                <Scene
                    key="tab2Scene2"
                    component={Tab2Scene2}
                />
                <Scene
                    key="tab2Scene2Popup"
                    component={Tab2Scene2Popup}
                />
            </Modal>

            <Scene
                key="tab2Scene3"
                component={Tab2Scene3}
                title="tab2Scene3"
            />
        </Stack>
    </Tabs>
</Scene>
</Router>

Try to give the Modal component a key prop, and navigate to that one instead. Navigating to the first scene in a Stack, Modal or Tabs would (at least in earlier versions) mount the scene twice, so navigating to the Stack/Modal/Tab directly to open the first scene is a better approach.


build/test: add back "etcd_setup_gopath"

Address https://github.com/coreos/etcd/issues/9338.
ETCD_SETUP_GOPATH=1 PASSES=unit ./test

@mkumatag Can you try this and check if it resolves the issue?

@gyuho the given solution is not 100% perfect, faced some issues running it. Please don't merge it, we should have some other way to deal with!

submitted #https://github.com/coreos/etcd/pull/9356 with all the changes, PTAL


Renewed junctionTable id's on every parent update

Hi,
The issue
This last day I have been debugging waterline to find out why records in a junction table get new id's on every update of any of it's two parent tables.
The reason (tl;dr)
The reason of new id behaviour is that after updating one of the two parent records ([Code] (https://github.com/balderdashy/waterline/blob/master/lib/waterline/query/dql/update.js#L207)) all nested associations get 'updated' ([Code] (https://github.com/balderdashy/waterline/blob/master/lib/waterline/query/dql/update.js#L222)); which means that all collection associations get removed and then re-added.
The reason (in depth)
While debugging I found out that a change set of associations is maintained by waterline while using objects. So when model.save() is called this is what happens:

update parent model [code] (https://github.com/balderdashy/waterline/blob/master/lib/waterline/model/lib/defaultMethods/save.js#L83)

update the model [code] (https://github.com/balderdashy/waterline/blob/master/lib/waterline/query/dql/update.js#L207)
remove all associations [code] (https://github.com/balderdashy/waterline/blob/master/lib/waterline/utils/nestedOperations/update.js#L185)
re-add all associations [code] (https://github.com/balderdashy/waterline/blob/master/lib/waterline/utils/nestedOperations/update.js#L191)

if there are new associations: add them [code] (https://github.com/balderdashy/waterline/blob/master/lib/waterline/model/lib/defaultMethods/save.js#L120)
if there are associations to be removed: remove them [code] (https://github.com/balderdashy/waterline/blob/master/lib/waterline/model/lib/defaultMethods/save.js#L138)

From this flow I deducted I should not be needed to remove and re-add the associations, so I created some usecases in which I skipped that part of the code (UpdateNestedAssociations(..),code). The results where as expected, adding and removing kept working but no new id's were introduced. So I'm starting to think that the UpdateNestedAssociations(..) is not needed as higher-level functions successfully deal with the adding and removing of associations.
My questions

Is it expected / desired behaviour that records in a junction table get new id's on every parent update? (I'm assuming it is not, because of key space exhaustion)
Is the UpdateNestedAssociations() necessary anyway? If so, can someone explain why?

Thanks in advance!
Marijn

@MarijnMensinga you're right about this flow.  However, it should only perform nested updates if Collection.update() is supplied values that imply setting of associations.  Collection.update() will reset associations to the parent record when associated records are specified to match the exact list of associated records provided in the Collection.update() values object.  Though Model.save() defers to Collection.update(), I believe that it does so in a way that does not trigger the nested  updates.
We have identified that this flow can be better optimized to minimize roundtrips to the db and data-touching.
Do you mind sharing some specific code where you're seeing this behavior?

Yes I have an example.
Agent
module.exports = {
  tableName: 'agent',
  migrate: 'safe',
  autoCreatedAt: false,
  autoUpdatedAt: false,
  schema	: true,
  attributes: {
    username: {
      type: 'string',
      required: true,
      maxLength: 50
    },
	groups: {
      collection: 'group',
      via: 'agents',
      through: 'group_agent'
    }
  }
};

Group
module.exports = {
  tableName: 'group',
  migrate: 'safe',
  autoCreatedAt: false,
  autoUpdatedAt: false,
  schema	: true,
  attributes: {
    name: {
      type: 'string',
      required: true,
      maxLength: 50
    },
    agents: {
      collection: 'agent',
      via: 'groups',
      dominant:true,
      through: 'group_agent'
    }
};

Group_Agent Junction table
module.exports = {
  tableName: 'group__agent',
  migrate: 'safe',
  junctionTable: true,
  autoCreatedAt: false,
  autoUpdatedAt: false,
  schema	: true,
  attributes: {
    agents: {
      columnName: 'agents',
      type: 'integer',
      foreignKey: true,
      references: 'agent',
      on: 'id',
      via: 'groups',
      groupBy: 'agent'
    },
    groups: {
      columnName: 'groups',
      type: 'integer',
      foreignKey: true,
      references: 'group',
      on: 'id',
      via: 'agents',
      groupBy: 'group'
    }
  }
};

Test
Test 1 and Test 2 both result in new PK's for all records belonging to group 1 within the junction table
var group,agent;
async.waterfall([
	function fetchGroup(done){
		Group.findOne(1).populate('agents').exec(function(err,dbGroup){
			group = dbGroup;
			done(err);
		});
	},
	function fetchAgent(done){
		Agent.findOne(4,function(err,dbAgent){
			agent = dbAgent;
			done(err);
		});
	},
	function test1(done){
		group.name = 'ICT group 2';
		group.save(function(err,dbGroup){ //This triggers new id's
			group = dbGroup
			done(err);
		});
	},
	function test2(done){
		group.agents.add(agent);
		group.save(function(err,dbGroup){ //This triggers new id's
			group = dbGroup
			done(err);
		});
	}
],function(err){
  //done
});

This is because of the "syncing" that takes place. We need to address this and turn it off or fix the save method to change this behavior. Remove the populate and this will be taken care of in the example above.


Client - Update documentation on how to override Content Type Header

With 1.0.0-RC1, the Client now sends the Content-Type header as application/graphql+json
We're trying to override the headers here, but it has no affect:
HttpGraphQlClient.builder(webClient)
        .url(url)
        .headers { header -> header.contentType = MediaType.APPLICATION_JSON }
        .build()

graphQlClient.documentName(documentName)
            .variables(variables)
            .retrieve("myQuery")
            .toEntityList(myQuery::class.java)

It's still coming over as application/graphql+json.
We need to do this, as the the Apollo graphql gateway we're using does not accept the graphql header.
Error: POST body missing, invalid Content-Type, or JSON object has no keys. (it's not the other two).

Overriding the content-type value (actually application/json) of request header is needed for me, too.