[ENH] Add case_when #736

Adding a case_when function
Please describe the changes proposed in the pull request:

Effectively, this is a wrapper around numpy.select but with an easier to manage mechanism for passing arguments using two element lists rather than separate lists for separate arguments.
The function does not mutate the original pd.DataFrame and is vectorized (there is a small part using a for loop but this is to generate the dictionary from the list that is passed through *args.
Can be method chained

**This PR resolves #736 **

PR Checklist

Please ensure that you have done the following:

[x] PR in from a fork off your branch. Do not PR from <your_username>:dev, but rather from <your_username>:<feature-branch_name>.

[x] If you're not on the contributors list, add yourself to AUTHORS.rst.

[x] Add a line to CHANGELOG.rst under the latest version header (i.e. the one that is "on deck") describing the contribution.

Do use some discretion here; if there are multiple PRs that are related, keep them in a single line.

Quick Check
To do a very quick check that everything is correct, follow these steps below:

[x] Run the command make check from pyjanitor's top-level directory. This will automatically run:

black formatting
flake8 checking
running the test suite
docs build

Once done, please check off the check-box above.
If make check does not work for you, you can execute the commands listed in the Makefile individually.
Code Changes

If you are adding code changes, please ensure the following:

[x] Ensure that you have added tests.
[x] Run all tests ($ pytest .) locally on your machine.

[x] Check to ensure that test coverage covers the lines of code that you have added.
[x] Ensure that all tests pass.

Relevant Reviewers

Please tag maintainers to review.

@ericmjl
@samukweku

I'm not exactly sure how to add tests here--is it the coverage.xml file? I ran pytest janitor/functions.py and everything seemed to check out.

@robertmitchellv There is a tests folder. Look for the functions subfolder and add a test file there. You could have a look at existing files as a guide.

@robertmitchellv , this may be useful link

@robertmitchellv thank you for the PR! My apologies for responding back late. Our first child just arrived, and I've been head underwater catching up with everything in life.
Because @samukweku has thought more about case_when than I have, I'm going to defer to his judgment here on the exact details of the implementation.
If possible, would you be kind enough to add in some tests, please? The best place for the tests is in the tests/functions/ directory, inside a test_case_when.py file. It's something we ask for, to maintain high test coverage of the library (this gives our end-users confidence that the functions behave as expected).
A final small friendly note -- you'll see the docs preview show up, but they build on every push to the PR, which does eat up into the build minutes on Netlify. I've been paying out of pocket for the build minutes (about $7/mo once I run out of the free tier that's associated with my account). So if possible, I'll be super appreciative of it if you could push "conservatively", i.e. push up a few commits at the same time, rather than push every commit. (And in case I come off the wrong way, this isn't a comment targeted towards you at all! I really appreciate the time you've spent crafting the PR. :smile: Just wanted to make sure you're aware of some of the constraints we're operating with in the project.)

@ericmjl , I thought you had cancelled the netliffy charge? personal opinion though, I think you should. We can go back to running make docs, view it on our local machine, before pushing to github.

I was thinking seriously about cancelling, but I later realized, actually it's not too bad. And I do use Netlify for other sites myself, like my personal site. It's two coffees worth per month, which I am happy to pay for the convenience 😃

@robertmitchellv there's some good stuff in here! Are you still planning to work on the PR?

@ericmjl sorry for the late response! I am going to finish this, but got swamped at work. I'll try to set time aside this week to push it through.
Also congrats on your first child 🎉🍾 so exciting

@samukweku Sorry for how long it took to add this! Work has been all consuming! Let me know if the test I added isn't sufficient. It passed, but I'm sure it could be better.

@samukweku Sorry for how long it took to add this! Work has been all consuming! Let me know if the test I added isn't sufficient. It passed, but I'm sure it could be better.

No worries, @robertmitchellv! Tis all good, that's just the nature of OSS :smile:. Glad you could come back to the PR!

@robertmitchellv @ericmjl I'd love to take a stab at your PR and make some changes (got a few ideas I would love to tinker and push through your PR) and get it over the finish line

@samukweku that would be wonderful! I was struggling to figure out how to incorporate your suggestions and then got buried with things at work. I would find a lot of value in seeing how you'd solve for the issues that remain here--I know @ericmjl asked me to merge latest changes--I'll have to look at what I have locally here because I don't know if I've made many changes outside of what I've already pushed (I may also not understand the ask and that could be where the confusion stems).

@robertmitchellv no worries. Started a new job as well ... been a busy one too - busy is good. I cant push commits to your branch, It seems you unchecked the box that says Allow edits from maintainers. Could you kindly check that box? Thanks

@samukweku sorry about that! Didn't realize I had that unchecked 😅 and congrats on the new job! That's great! Hopefully you won't run into any other issues--I'll try to get them resolved as soon as I can though

thanks @robertmitchellv ; for some reason, I still cant make any commits. @ericmjl any ideas on how I can pull robert's PR to my local branch, make commits, and push back into his PR? I guess you'll have some git-fu magic

@robertmitchellv ; picking this up to continue; let me know if you are back on it and I'll defer to you.

@robertmitchellv, got some updates on the case_when function that I'd like you to have a look at; I couldnt update your PR. @ericmjl I might need your git skills again to get into @robertmitchellv PR

@robertmitchellv made some edits to the code, and added some tests. Please feel free to make changes.

@ericmjl thank you! Big thanks to @samukweku 🙏🏻
This might be my first real OSS contribution 😅
Appreciate how welcoming and understanding you've both been; great community here 👍🏻


Update sync.yml

Add jhudsl/ITCR_Tables to list of repos receiving updates

Oh this url is no longer in existence?
https://www.datadreaming.org/post/r-markdown-theme-gallery/

@kweav I updated the broken URL. Feel free to merge after checks pass!


Bad Link

In the RollbackToTargetVersion section, there's a link to https://go.microsoft.com/fwlink/?linkid=2133918 which is 404.

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

ID: 96fddabd-2bac-2fab-a5ea-25c000b425fc
Version Independent ID: 41bba38e-bd24-0496-da04-012d6deb5db8
Content: Microsoft Edge Update Policy Documentation
Content Source: edgeenterprise/microsoft-edge-update-policies.md
Product: microsoft-edge
GitHub Login: @dan-wesley
Microsoft Alias: stmoody

@JasonNVSD Thank you for catching this. We'll follow up.

@dan-wesley , Please merge it PR #337, if it is correct

Link in documentation looks to be updated.  Closing.


Verified Trustchain endpoint

We want to minimise the amount of trust that the trustchain-mobile user is required to place in the Trustchain API endpoint configured in Settings. An obvious way to do this is to make it verifiable by including it as an endpoint in a dDID.
To implement this we need a verifiedTrustchainEndpoint boolean associated with the config page state (in config.dart). If this is false, and the root event date is set, the mobile app should send a GET request to a (new) API endpoint on the server that responds with the DID of the Trustchain server operator. The corresponding DID document should contain the configured trustchain server endpoint and should also be part of a verifiable dDID chain. If both of these check out, the server endpoint itself has been verified and this should be indicated with a new "verified endpoint" icon in the Settings. Otherwise, the endpoint should be marked as "unverified".

Since we verify data received from the Trustchain endpoint anyway, it's not necessarily obvious that there is any benefit to verifying the identity of the server operator. It provides only a marginal benefit (which is a good thing), but there is some benefit because the host may withhold information (e.g. DID revocations).


Question about Develop and Debug with CLion in wiki?

For "Click the link for Specify Clang-Tidy executable and point it to the binary installed by packages.sh"
Where could I find the binary installed by packages.sh in my ubuntu pc?

After use "whereis" command. I find /usr/bin/clang-tidy-8.  So I point to "/usr/bin/clang-tidy-8".


Issue#689

// Describe your PR here; eg. Fixes #issueNo

Make sure that:

[ ] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #758 (bb9371c) into master (81fc59a) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #758   +/-   ##
=========================================
  Coverage     65.20%   65.20%
  Complexity     1314     1314
=========================================
  Files           266      266
  Lines          5900     5900
  Branches        633      633
=========================================
  Hits           3847     3847
  Misses         1713     1713
  Partials        340      340

Continue to review full report at Codecov.

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


Fix in cross-references

This PR comes from 336, and the main goal is to evaluate a minor fix.
In resume: Cross-reference are not detected if this references are out of the scopes. In  domain model example index  fix it by using streamAllContent() instead of streamContent().  The initial proposal is to change DefaultAstNodeDescriptionProvider  to use streamAllContent(). It fix the issue, and cross-file cross-reference still works.
Probably there will be cases where we want that the cross-reference cannot be detected if it is not in the scope. In this case, maybe developing a way to express the scope of the cross-reference(global/local) in the grammar could be a better solution.

@JoanGi
If I recall it correctly we explicitly decided to reduce indexing to the root node + the first containment level which is basically what  the createDescriptions() currently does.
I would rather propose to change the AstNodeDescriptionProvider behavior in your current language and keep the lightweight default  implementation as it is.

Ok!,  have sense to keep lightweight the default implementation. Thank's for the efforts and the quick answer (that's really good!)!! :)

Thanks for contributing @JoanGi !
By the way, the every first implementation of createDescriptions() was exactly like you suggested here ;-)


I can submit an answer or reply with ctrl+enter

The key combination ctrl+enter should allow submitting.

Closed by #38


using react-live with mdx-deck

Hey,
first of all thanks for a great project!
I'm trying to add some live coding experience into a deck. Saw that the recommended way was to use react-live.
However when I add that and try add react-live to my deck I get an error:
Uncaught TypeError: Cannot read property 'tokTypes' of undefined
at Module.eval (buble-browser.es.js:269)
    at eval (buble-browser.es.js:10378)
    at Module../node_modules/buble/dist/buble-browser.es.js (main.js:970)
    at __webpack_require__ (main.js:725)
    at fn (main.js:102)
    at eval (react-live.es.js:26)
    at Module../node_modules/react-live/dist/react-live.es.js (main.js:3218)
    at __webpack_require__ (main.js:725)
    at fn (main.js:102)
    at eval (index.js:16)

// in index.mdx
import {
  LiveProvider,
  LiveEditor,
  LiveError,
  LivePreview
} from 'react-live'

---
<LiveProvider code="Hello World!">
  <LiveEditor />
  <LiveError />
  <LivePreview />
</LiveProvider>

mdx-deck version 1.7.7
any ideas??

Haven't had time to dig in, but I've seen that same error in other places. I suspect a change somewhere in the dependency tree is causing that

The current (albeit temporary) fix is to pin buble to 0.19.4. The current version of buble is what's causing this error and hasn't been fixed upstream yet.

@johno thanks, that fixes it even though it might be temp right!?

@johno thanks, that fixes it even though it might be temp right!?

@johno thanks, that fixes it even though it might be temp right!?

@johno thanks, that fixes it even though it might be temp right!?

@johno thanks, that fixes it even though it might be temp right!?

@johno thanks, that fixes it even though it might be temp right!?


Byte can be simplified to byte

Simplify Byte
Using byte instead of Byte.

Thank you for making the additional change, @hassanhabib. We'll merge your PR now.


Kythe releases should use fully static binaries

Right now the binaries for the Kythe release are "mostly static" which introduces a dependency on platform-specific C/C++ libraries.  We could increase the portability of our binaries by linking them entirely statically.

It looks like there is a still-unfixed Bazel bug with fully static binaries: https://github.com/bazelbuild/bazel/issues/8672
However, the workaround is pretty straightforward (explicitly specifying linkopts = ["-l:libstd++.a"]).  While the workaround is not portable to other standard C++ libraries (i.e. libc++) nor platforms that don't use .a for static libraries, it will work for building our Linux releases.

There are also a variety of POSIX libc functions which don't really work when statically linked (specifically, those which are controlled by nsswitch.conf) and a smattering of Kythe binaries which just segfault on start when built that way.

We also encountered errors like the following in our CI-indexing job:
external/kythe_release/extractors/bazel_cxx_extractor: /lib/x86_64-linux-gnu/libm.so.6: version `GLIBC_2.29' not found (required by external/kythe_release/extractors/bazel_cxx_extractor)
external/kythe_release/extractors/bazel_cxx_extractor: /usr/lib/x86_64-linux-gnu/libstdc++.so.6: version `CXXABI_1.3.11' not found (required by external/kythe_release/extractors/bazel_cxx_extractor)
external/kythe_release/extractors/bazel_cxx_extractor: /usr/lib/x86_64-linux-gnu/libstdc++.so.6: version `GLIBCXX_3.4.26' not found (required by external/kythe_release/extractors/bazel_cxx_extractor)

Is there documentation about what system library requirements are for those releases?
@mithro FYI

This is still an issue using kythe on various platforms and v0.0.60 binaries are still having the libstdc++.so.6 dynamically linked which makes it somewhat specific to the platform.
It looks like the aforementioned  bazelbuild bug is closed now; maybe it can be re-visited ?
What I did find is that this workaround actually seems to work quite well in one of my projects when building release binaries; the final binary is then statically linked to libstdc++.
bazel build --action_env=BAZEL_LINKLIBS=-l%:libstdc++.a:-lm --action_env=BAZEL_LINKOPTS=-static-libgcc //target

It's likely that re-applying #4607 would work, if the underlying issue has been fixed.

It seems fixed in Bazel, but whatever caused the resulting binaries to be unusable has not.
We'll happily accept contributions making this change, but it's unlikely to be high enough priority for me to tackle it anytime soon.

It looks like there is a still-unfixed Bazel bug with fully static binaries: bazelbuild/bazel#8672
However, the workaround is pretty straightforward (explicitly specifying linkopts = ["-l:libstd++.a"]). While the workaround is not portable to other standard C++ libraries (i.e. libc++) nor platforms that don't use .a for static libraries, it will work for building our Linux releases.

Another MAANG (minus G) uses Bazel only in spirit but moved away from it for this and a number of reasons. Personally, I don't use it either for exactly this gripe: mandating assumptions rather than making practical and necessary software configuration possible. I like being able to patch and compile Nginx with a custom static SSL and ZLib. Maybe in the future, we will have idempotent, deterministic, immutable, cached, linted, attested unification of software builds, package builds, and system conf mgmt. ¯\_(ツ)_/¯ /gripe. I came here to find magic args to cargo install cargo-kythe but can't seem to find working instructions. This looks like a neat project. I wish the friction for installation was less.


different step got different result

What did you do?
query the same metrics with different step
What did you expect to see?
got the same result at the same timestamp
What did you see instead? Under which circumstances?
step 60 result:

step 300 result:

System information
aws ec2
Prometheus version
Version	2.34.0
Revision	881111fec4332c33094a6fb2680c71fffc427275
Branch	HEAD
BuildUser	root@121ad7ea5487
BuildDate	20220315-15:18:00
GoVersion	go1.17.8

Prometheus configuration file
No response
Alertmanager version
No response
Alertmanager configuration file
No response
Logs
No response

正常现象，从当前时间开始每隔300s查询一次，数据被削峰了

Are the results really different? It's hard to spot in the graph. If you look at the raw result from the API (e.g. using curl to run the same query), you could do a precise comparison. Feel free to dump the result here.

Hello from the bug scrub!
With no follow up in more than a year, we are closing this.


Spring Cloud Zookeeper Updated

Rechecked and reformatted all code using eclipse formatting.
Made only string response for the producer application. For 4.5 the configuration file is required so the consumer application can look for zookeeper. Please check the section 5 it shows how to see the working solution should I still drop it.
Added the maven central links like you mentioned.
I rechecked the names for HelloWorld and Greeting Application, didn't notice any discrepancies.Please let me know if any came in your notice.

Resolved conflicts


302 bug python wheel builds

:tada: This PR is included in version 0.11.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Error from server (NotFound): error when creating "STDIN": namespaces "k6-operator-system" not found

Hello, was following the nice blog over at https://k6.io/blog/running-distributed-tests-on-k8s/ and stumbled upon the following issue:
karl@Karls-MacBook-Pro operator (main) $ make deploy
go: creating new go.mod: module tmp
go: downloading sigs.k8s.io/controller-tools v0.3.0
go: found sigs.k8s.io/controller-tools/cmd/controller-gen in sigs.k8s.io/controller-tools v0.3.0
go: downloading github.com/spf13/cobra v0.0.5
go: downloading k8s.io/apimachinery v0.18.2
go: downloading github.com/gobuffalo/flect v0.2.0
go: downloading k8s.io/apiextensions-apiserver v0.18.2
go: downloading k8s.io/api v0.18.2
go: downloading gopkg.in/yaml.v3 v3.0.0-20190905181640-827449938966
go: downloading golang.org/x/tools v0.0.0-20190920225731-5eefd052ad72
go: downloading github.com/fatih/color v1.7.0
go: downloading github.com/inconshreveable/mousetrap v1.0.0
go: downloading github.com/mattn/go-isatty v0.0.8
go: downloading github.com/mattn/go-colorable v0.1.2
/Users/karl/go/bin/controller-gen "crd:trivialVersions=true" rbac:roleName=manager-role webhook paths="./..." output:crd:artifacts:config=config/crd/bases
cd config/manager && /usr/local/bin/kustomize edit set image controller=ghcr.io/k6io/operator:latest
/usr/local/bin/kustomize build config/default | kubectl apply -f -
namespace/system created
customresourcedefinition.apiextensions.k8s.io/k6s.k6.io created
clusterrole.rbac.authorization.k8s.io/k6-operator-manager-role created
clusterrole.rbac.authorization.k8s.io/k6-operator-metrics-reader created
clusterrole.rbac.authorization.k8s.io/k6-operator-proxy-role created
clusterrolebinding.rbac.authorization.k8s.io/k6-operator-manager-rolebinding created
clusterrolebinding.rbac.authorization.k8s.io/k6-operator-proxy-rolebinding created
Error from server (NotFound): error when creating "STDIN": namespaces "k6-operator-system" not found
Error from server (NotFound): error when creating "STDIN": namespaces "k6-operator-system" not found
Error from server (NotFound): error when creating "STDIN": namespaces "k6-operator-system" not found
Error from server (NotFound): error when creating "STDIN": namespaces "k6-operator-system" not found
make: *** [Makefile:54: deploy] Error 1

Any ideas?

Hi! Thanks for opening the ticket :)
What version of Kustomize are you using?
Found this: https://github.com/operator-framework/operator-sdk/issues/3826. Could be related.

@dgzlopes Ah yes it might be!
Here is my outcome from the version:
karl@Karls-MacBook-Pro deploy (karl/load-testing) $ kubectl version
Client Version: version.Info{Major:"1", Minor:"17+", GitVersion:"v1.17.16-rc.0", GitCommit:"737e2c461a2999fa242d39e77b9252d0eee7167e", GitTreeState:"clean", BuildDate:"2020-12-09T11:14:02Z", GoVersion:"go1.13.15", Compiler:"gc", Platform:"darwin/amd64"}

karl@Karls-MacBook-Pro deploy (karl/load-testing) $ kustomize version
{Version:kustomize/v3.8.2 GitCommit:e2973f6ecc9be6187cfd5ecf5e180f842249b3c6 BuildDate:2020-08-29T17:44:01Z GoOs:darwin GoArch:amd64}

karl@Karls-MacBook-Pro deploy (karl/load-testing) $ make --version
GNU Make 4.3
Built for x86_64-apple-darwin19.6.0
Copyright (C) 1988-2020 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

karl@Karls-MacBook-Pro deploy (karl/load-testing) $ sw_vers -productVersion
10.15.7

I'm gonna try and see if the other issue will resolve this and if so close this one!

Ok I reverted to 3.8.1 and it seemed to not throw the kitchen sink at me.
For anyone else, you can use this to install a specific version of kustomize:
https://github.com/kubernetes-sigs/kustomize/blob/master/hack/install_kustomize.sh
Then run:
./install_kustomize.sh [WANTED_VERSION]
The downloaded bin file should be moved to the appropriate place.
In all honesty though, this should probably be fixed somehow.


forgot to define html5_data attribute column x error

I currently get a "It seems you have forgot to define html5_data attribute for column x" error message when any field/row in my serverside ajax/JSON returns null" as value. Any logic or suggestion to this, tested 0.8.6 and latest beta + latest datatables version.
Some related columns simply defined by -> "data": "6", "visible": false, "searchable": false and yadcf {    column_number: 6, filter_type: "autocomplete"  }

Hi,
You can't filter a column that is set to "searchable": false (not with yadcf and not with datatbles)
If you'll have any similar issues with a column that is not set to "searchable": false feel free to reopen this issue and provide additional information

Hi,
I get the same error. It occurs when a field in the Json Data array is null and the Data Array is no associative array. Tested in beta 0.8.7
I think the problem is in the function parseTableColumn line number 1487.
if (typeof col_inner_data === 'object') {
	if (columnObj.html5_data !== undefined) {
		col_inner_data = col_inner_data['@' + columnObj.html5_data];
	} else {
		alert('Looks like you have forgot to define the html5_data attribute for the ' + columnObj.column_number + ' column');
		return;
	}
}

if the Field Value data[j]._aData[columnObj.column_number] is null it is an object.
if (col_inner_data !== null && typeof col_inner_data === 'object') solved the problem.

I just experienced the same behavior @charliechihuahua observed and was able to solve it by using their solution with version 0.9.4.beta.35.
Do you see a reason a drawback in this solution, @vedmack ? I could open a PR if you like. :)

@SoerenWeber , yes that would be great

committed the fix in 0.9.4.beta.36


Documentation for Access Control and Page Security is invalid

Affected Version
Documentation site
Description
On the documentation site, the Access Control page doesn't contain a lot of useful information, and reads more like a stream of consciousness while someone was reading the code, rather than a useful developer guide. The whole page has this sort of content, but the 'Permission checking is at class level' section is probably the worst offender.
The page should ideally be re-written to properly cover the Access Control model that SilverStripe implements.
I'll try to do this in a PR, but no guarantees - adding this issue in case I don't get a chance to re-write it.

I came to this repo to raise the same issue. This page needs a full re-write!

Noting there's also another page of the same name: https://docs.silverstripe.org/en/4/developer_guides/controllers/access_control/#access-control
Might be worth looking at a rename of this file at the same time?


Change tracking frequency depends on the type

Change tracking frequency depends on the type. None of the types is actually hourly. See https://docs.microsoft.com/en-us/azure/automation/automation-change-tracking

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

ID: f054a0ea-fe3e-2460-e0c8-bf6475040fed
Version Independent ID: ce7a1b1d-9e00-c02d-a394-aaf1293e2e0d
Content: Data collection details for management solutions in Azure
Content Source: articles/monitoring/monitoring-solutions-inventory.md
Service: log-analytics
GitHub Login: @MGoedtel
Microsoft Alias: bwren

@slavizh Thanks for the feedback. We are actively investigating and will get back to you soon.

@slavizh Can you please elaborate your question to answer better or address your issue?

Hi
As you can see from Change tracking docs the frequency depends on the type of change. From below values none of them are 1 hour for sure.
Source: https://docs.microsoft.com/en-us/azure/automation/automation-change-tracking

Change type
Frequency

Windows registry
50 minutes

Windows file
30 minutes

Linux file
15 minutes

Windows services
30 minutes

Linux daemons
5 minutes

Windows software
30 minutes

Linux software
5 minutes

@slavizh Yeah, I can see that. What is your issue with that?

Hi,
This is just a suggestion but the information on the these pages is not correct.
https://docs.microsoft.com/en-us/azure/monitoring/monitoring-solutions-inventory
https://docs.microsoft.com/en-us/azure/log-analytics/log-analytics-add-solutions
Either list all change types and their intervals or write that it depends on the change type and have a link leading to https://docs.microsoft.com/en-us/azure/automation/automation-change-tracking to see the intervals for each type.
I think for people using the solutions it is important to know the intervals.

@slavizh Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

@slavizh - Brian Wren is addressing this and the updated article(s) should go live today or tomorrow the latest.  Thanks for bringing this to our attention.  #please-close.

Sorry for the delay, but this fix is done and going out today.  Most obvious solution was to indicate collection frequency varies for that solution and link to the other article.

#please-close


Fix CI/CD

Gonna use this branch to see if we can get CI/CD fixed up.

Errors seem random. Mostly the following fail tho:

Amazon Linux 2 arm64
Amazon Linux 2 x86_64
Debian arm64

I have seen some random image steps fail too. Again, random with postCreate


Fix/tests whitespace strictness

There's three challenges in the issue. The second one was already fixed. This fixes the other two.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[x] My changes do not use shortened URLs or affiliate links.

Closes #34344

We could add another test that doesn't strip white space and just checks for "Now I know React and Redux!", but if I agree that if we can check the console output, that's got to be better.   It's not always possible (we can't do it on #36705), but I prefer checking behaviour over code if we can.


Provide a button that leads the user to the Settings AI page when no provider is set up

When a user opens up Terminal Chat but does not have a provider set up, provide a button that sends them to the relevant settings page

/azp run


Improvement: Developer/Designer Notes in Godot Editor

Godot version:
3.*
Improvement description:
I think it would be very helpful while developing/designing your game in the Godot Editor, if you could attach developer notes/infos to nodes in the scene tab that appear as tooltips when you hover over them and with a small icon (like the script icon) - maybe a lightbulp - to show that here is a note attached.
One step further would be invisible note/info nodes that could be placed in the viewport window (game world), that are only visible in Editor and not exported or rendered when the game is running.

I often have ideas or improvements, that I want to explore later. Or if you are collaborating with other developers you could leave some notes to explain things.
Yes you could attach scripts with comments, but it's not the same and as practical.

Duplicate of #2082. See also #7361.


Option to exclude some url or maybe using regex in --urls option

Details
It would be useful to have an option to exclude specific URLs or to use regular expressions to filter which URLs are processed.
This can be inconvenient in cases where certain pages should be skipped.
One possible solution would be to add a new command-line option, such as --exclude-urls, which would accept a comma-separated list of URLs to exclude.
Alternatively, a regular expression could be used to match and exclude URLs based on a pattern.

The --exclude-urls /foo,/bar option is now available with support for regex in 0.6.0.
Let me know if you have any issues with it.


fix: properly handle the integration id for new integrations

Fixes https://github.com/syndesisio/syndesis-react/issues/415

PR Storybook available here


docker doesn't track changes of webapp source files on bind mount

I develop webapp served by services running on docker container. One of the services which is responsible for serving of components of the app (html, js, css, etc.) has bind mount folder with source files. When I modify  source files and try to test changes in web browser, browser receives unchanged files. It not happens always. In fact, a few times after start of docker files are refreshed correctly and changes are visible in browser. But after few cycles docker loses track of changes on bind mount and I must restart docker to bring it back to normal state.
In system log i found errors form Hyper-V-Integration:
"'DockerDesktopVM': The Data Exchange integration service is either not enabled, not running or not initialized."
with event id 4092.
However, Data Exchange integration service is not running but is started on demand. When I try to run it manually it starts and stops immediately with message that it will be run when needed.
What I need to do to make docker tracking changes on bind mount?

New update appeared (ver. <IP_ADDRESS>) and the problem seems to be solved.


fix archlinux useradd/adduser

Error when we try to create the vespene user on archlinux with adduser

Ha distros are great aren’t they?
To clarify, it doesn’t have the more low-level useradd at all there? Adduser likes to ask for names and email addresses by default on some platforms.

@mpdehaan yep it's nice to meet issue like this during other tests lol.
adduser is not available by default I need to add the AUR repo to install it.
I change my fix to add AUR and install adduser instead using useradd.

$ vagrant init archlinux/archlinux
$ vagrant up
$ vagrant ssh # connect to archlinux with vagrant
[vagrant@archlinux ~]$ sudo pacman --noconfirm -Sy git
[vagrant@archlinux ~]$ git clone -b fix-adduser https://github.com/4383/vespene.git ./vespene
[vagrant@archlinux ~]$ cd vespene/setup/
[vagrant@archlinux ~]$ sudo bash 1_prepare.sh

doesn't work...

So my changes works but adduser ask for user informations

@mpdehaan after a a lot of changes and tests (install adduser, try to use useradd, etc...), I think the more preferable solution is to use useradd.
pro:

useradd accept parameters and doesn't ask something.
useradd is installed by default on archlinux
cons:
adduser ask a lot of informations.
adduser is not installed by default

What do you prefer?

I can introduce useradd only for archlinux in a first time and after switch on all distros

I had already coded the install scripts to use useradd recently so yes useradd everywhere is good - so do we need the PR?

So the codebase should already be using useradd everywhere so does there need to be anything done at all?

@mpdehaan where is it?
If you share it I can rebase my code with it and we can close this PR

else merge it temporary and I adapte if needed

@4383 @mpdehaan How about something like this?
if [ "$DISTRO" != "MacOS" ]; then
if [ -x "$(command -v adduser)"  ]; then
sudo adduser vespene
else
sudo useradd vespene
fi
fi

@kdbeall personnally I agress

@kdbeall eventually for my temporary version but not for the next times

actually what i really want is "useradd" on all platforms unless it is not there at all.
useradd is the basic low level one
adduser is the annoying one that frequently goes interactive and asks about email addresses

https://github.com/vespene-io/vespene/commit/991a7bc1edc33cee36f187b2e4c737a7db1e51eb


Add Django-CMS plugin in a contrib application

At least we need a plugin to list "last X articles [for language]" and "Category listing [for language]".
Also an apphook would be very useful.


Use yaml.safe_load instead of yaml.load

This Tweet made me grep our code for yaml.load - https://twitter.com/anthonypjshaw/status/1014054816987803648
I was almost positive we already use yaml.safe_load everywhere, but it looks like a couple of yaml.load occurrences managed to creep into the tests.
It's not a big deal to do that in the tests. It's mostly problematic in places where we take user input, but it's still a good practice to use yaml.safe_load if there is no special reason / need to use yaml.load.
Thanks @tonybaloney

@tonybaloney https://github.com/StackStorm/st2/pull/4222 was first :stuck_out_tongue:

@armab ah, lol, I didn't even see that one. Will close this one as a duplicate of #4222.
Or maybe Github just doesn't correctly handle nano second precision and I was actually the first one :trollface:

looks like it's 6 mins in diff between 2 PRs :)

@armab No way - must be that we were routed to two different backend servers with local clocks out of sync :trollface:

Haha, that's a good one :+1:


Error in --select documentation

Document reads:
--select
Space-separated list of properties to return for each entity.
This is incorrect, list need to be comma separated.
Attempting to use space separated throws an error:
Term 'PartitionKey Row Active' is not valid in a $select or $expand expression.
RequestId:938109ca-b002-0014-5d9f-5e1748000000
Time:2022-05-03T03:37:20.7960453Z
ErrorCode:InvalidInput
az --version
azure-cli                         2.36.0
core                              2.36.0
telemetry                          1.0.6
Extensions:
azure-devops                      0.25.0
Dependencies:
msal                              1.17.0
azure-mgmt-resource               20.0.0
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 9a4d128f-a98f-d07b-0183-90ca79cc5fdc
Version Independent ID: 83084fff-ea1b-a9e7-b7a9-40023de43542
Content: az storage entity
Content Source: latest/docs-ref-autogen/storage/entity.yml
Service: storage
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

storage

Hi, @RDavis3000
When I try to use space separation, no error is raised. Can you provide more information about the command？

Hi, @RDavis3000 When I try to use space separation, no error is raised. Can you provide more information about the command？

Yep you are correct, I must have been doing something else wrong!
Please close.


Done 50

How do I transfer 0ara to the wallet?

Ok

Good 👍

👍

gh pr checkout 56

0x102eb8ccf709D519Ed6ba862b627924cE2693B18
On Sat, Apr 15, 2023, 4:49 AM HTCCAN @.***> wrote:

gh pr checkout 56
—
Reply to this email directly, view it on GitHub
https://github.com/pi-apps/pi-explorer/pull/56#issuecomment-1509489706,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXMWACO5VWLWY74SC2OJDKDXBILEJANCNFSM6AAAAAAQHFIWCE
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

C est koi sa ??? Rien compris
Le mar. 18 avr. 2023 à 22:07, Diamond57 @.***> a écrit :

0x102eb8ccf709D519Ed6ba862b627924cE2693B18
On Sat, Apr 15, 2023, 4:49 AM HTCCAN @.***> wrote:

gh pr checkout 56
—
Reply to this email directly, view it on GitHub
<https://github.com/pi-apps/pi-explorer/pull/56#issuecomment-1509489706
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AXMWACO5VWLWY74SC2OJDKDXBILEJANCNFSM6AAAAAAQHFIWCE
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub
https://github.com/pi-apps/pi-explorer/pull/56#issuecomment-1513848315,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APA3TREBNJ7SQHGKUHCXDTTXB4GCNANCNFSM6AAAAAAQHFIWCE
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

simontrinh0911:development


If the subscription is on subscribe a member for an every new app

This results in Follow.
If member mutes/unfullow the app he/she stops getting the notifications on the releases.

implemented weekly digest


Translation of service data

The service provider should have the possibility to translate its service metadata into platform languages. Right now only one language is available.
In the image below I have selected Romanian, but the services description appears in Engligh.

This should be analized further in order to see which are the information that could be translated into different languages.
For instance a lot of columns or even tables may be afected.


Why does error_state() return an int when the only allowable values are tinyint?

throw and raiserror accept a state parameter that is constrained to be a tinyint, yet error_state() is defined and documented to return the wider int. Why?

@IanKemp -- Ian, thank you again for your perceptive questions.  Are you following a particular docs.microsoft.com article?


[docs] fixed minor typos

Sorry, overlooked yesterday in #2116 that param name in CLI version in slight different.

@StrikerRUS Is it critical that @guolinke review this? Seems pretty non-controversial. I can give a review and merge, if you'd like

Oh, @jameslamb, of course it's not critical at all. Thanks a lot!


Update RakNetSession.java

Put a space after "session #GUID..." in the log

May I inquire why this space is important?


Two samples on communication between web parts through broadcasting events utilizing the Publish–subscribe pattern

Q
A

Bug fix?
no

New feature?
no

New sample?
yes X2

Related issues?
fixes #X, partially #Y, mentioned in #Z

What's in this Pull Request?
Two samples added with this pull request. They both are related to communication between web parts through broadcasting events utilizing the Publish–subscribe pattern, but the implementation is different for both.
The first one uses the SPFx Event Aggregator, API still in alpha. I will regularly update that to make sure it is aligned if any changes are made when it goes GA.
The second is my own implementation that utilizes the ReactiveX (RxJs) library, but works is a similar way as the Event Aggregator. It is added to the samples because the Event Aggregator, API is still in alpha and this one can be used as alternative until the Event Aggregator API is generally available.
Please refer the the samples README.md files for more information and feel free to contact me if any questions.

Thx Velin, even though this model technically does work, it's not supported in production usage. You have called that alpha status of the API in the code, but I'll modify this to be super clear in the SPFx even model readme, but other than that, this is absolutely fine as a starting point and when we get these APIs in more mature level, we can update this existing sample.
Good also to have a secondary sample, which shows an alternative option for the event handling. Thx for the submission, appreciated.


How to use

Sorry for bothering, I just cant figure out how to use this, I have nvidia, so I know I have to do something extra, but besides that I don't how it should work or what to do to make it work, so a brief explanation will be greatly appreaciated, I will buy you a coffee

For linux or Windows ?

Hi, srry for the late answer, Windows 10


Allow configuring a default remote

The goal of this PR is to allow the user to configure what should be the default remote. The current logic tries to guess what should be the default remote by looking at the "origin" and "upstream" remotes if defined, and at branches such as "main" and "master". But having those remotes and branches is not always the case, and even so they could hold a wrong value (for lab).
This PR adds the ability to explicitly configure the default remote, using a new core.default_remote parameter in the configuration. While it is probably not a good idea to define this globally, the configuration file can also be in the repositories .git directories, as well as in the .git/worktrees/ directories when using Git worktrees. In such cases, having the ability to explicitly set the default remote is quite handy.
This does not break any of the previous cases. When not set, lab will strill try to guess what should be the default remote. And commands allowing a remote as a parameter can still do so.
This PR starts with some cosmetic patches and then adds the new core.default_remote logic.
Thanks!

@atenart this looks good, if we can get the tests/build sorted out happy to build it in. Might just be a matter of bringing in the latest from master

@atenart this looks good, if we can get the tests/build sorted out happy to build it in. Might just be a matter of bringing in the latest from master

@atenart this looks good, if we can get the tests/build sorted out happy to build it in. Might just be a matter of bringing in the latest from master

The build actually fails because of an issue in the series: if skipInit is true in main(), the config isn't loaded and we can't access it. I'll push an update with this fixed. Thanks!

@atenart this looks good, if we can get the tests/build sorted out happy to build it in. Might just be a matter of bringing in the latest from master

The build actually fails because of an issue in the series: if skipInit is true in main(), the config isn't loaded and we can't access it. I'll push an update with this fixed. Thanks!


Analyze Android NDK projects without NDK installed

We should investigate whether it's somehow possible to analyze Android projects that depend on the Android NDK either without installing the NDK at all, or by bootstrapping the NDK, so we do not need to install it / add it to the Dockerfile beforehand.

Maybe a project from https://github.com/android/ndk-samples can be used for testing.

And maybe https://github.com/autonomousapps/dependency-analysis-android-gradle-plugin implements some ideas that we could reuse (also ping @mnonnenmacher).

The Android Gradle plugin can bootstrap the NDK so it should not be necessary to add it to the Dockerfile (relates to https://github.com/oss-review-toolkit/ort/pull/5059/commits/44ece47f73718a93725c7066bc3bc403e462e746).

Closed as part of backlog grooming. Feel free to comment if you would like to contribute to this.


DWIConvert: Conversion to FSL doesn't produce bvec/bval files

I have two sets of DWI scans for a patient, and running DWIConvert on them produces bvec/bval
files for one but not the other.  Converting to nrrd works fine for both sets.  Converting from nrrd to fsl
says I need input bvec/bval files, but those are exactly what I'm trying to generate.
Any hint on what the problem may be?

We would need to debug the program and reproduce the problem locally.  We have not run into this issue in the past.
Hans
From:  reckbo<EMAIL_ADDRESS>Reply-To:  BRAINSia/BRAINSTools<EMAIL_ADDRESS>Date:  Wednesday, May 11, 2016 at 7:36 PM
To:  BRAINSia/BRAINSTools<EMAIL_ADDRESS>Subject:  [BRAINSia/BRAINSTools] Conversion to FSL doesn't produce bvec/bval files (#270)
I have two sets of DWI scans for a patient, and running DWIConvert on them produces bvec/bval
files for one but not the other.  Converting to nrrd works fine for both sets.  Converting from nrrd to fsl
says I need input bvec/bval files, but those are exactly what I'm trying to generate.
Any hint on what the problem may be?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub

@reckbo do you get any error messages on the command line or the Slicer log view?
Are you able to share them locally? If so I can take a look.

We don’t use DWIConvert in the slicer environment.  I am not sure which version is being used there.
I would recommend trying the latest source build.
Hans
From:  Isaiah<EMAIL_ADDRESS>Reply-To:  BRAINSia/BRAINSTools<EMAIL_ADDRESS>Date:  Thursday, May 12, 2016 at 8:19 AM
To:  BRAINSia/BRAINSTools<EMAIL_ADDRESS>Cc:  Hans Johnson<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject:  Re: [BRAINSia/BRAINSTools] DWIConvert: Conversion to FSL doesn't produce bvec/bval files (#270)
@reckbo do you get any error messages on the command line or the Slicer log view?
Are you able to share them locally? If so I can take a look.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub

This was my mistake, I was missing the '--conversionMode DicomToFSL' flag.  Without it the app just generates a nifti volume.


[FEATURE]-Add methods for reading and writing files.

Is your feature request related to a problem? Please describe.
I would like helper methods to read and write files other than the essentials config. Plugins should have a decent reference for this functionality without having to recreate the logic each time.
Describe the solution you'd like
A method to search for files by path/name
A method that will return the data from a text file
A method that will write a text file when passed data and a path.

If something to implement as part of the core I would vote for decent config sharing, lookup, merging and deserialization functionality is needed.
While running multiple S#Pro programs I found need for:

Automatic config resolution per slot - when same plugin loaded into two or more program slots, right config should be picked.
Global and per-slot settings - ability to read global config and merge it with per slot config.
Automatic traversal - check program running folder, switch to other common areas where config can be located, i.e. check program0x first, check nvram, check user, etc...
Return as c# objects? I.e. read, merge, parse and instantiate config objects hiding file lookup, parsing from json, merge, de-serialize final config object....

Basically my two cents...
I did something like that in https://github.com/uoftdaniels/ConfigManagementLibrary
I really hope something like that can be implemented at core level for Essentials as well - so we can have global config and per room customizations... Right now I believe it is possible only at portal level where merge is happening....

@batourin that all sounds and looks great. I think maybe its a couple of iterations past this initial request. I encourage you to keep evolving this initial feature, or submitting your libraries as a separate feature request.


Resampler interval fix

(My) resampling logic was wrong, and only passed tests because it assumed that resample_interval % batch_size == 0, which was true for our testing, have added a test that the previous version would have failed.
Highlights the need for a resampler test that tests the whole section end to end.

Solved in #141


Feature/sparql proxy refactoring

To correct numerous flaws of SPARQLProxy. Based on the commit: 2e2e531a12c288661b2ed68233833c14eb704446
In particular:

https://github.com/odalic/sti/blob/2e2e531a12c288661b2ed68233833c14eb704446/sti-kbproxy/src/main/java/uk/ac/shef/dcs/kbproxy/sparql/SPARQLProxy.java#L431 was never satisfied, because the type was not returned for exact match queries with types, so this comparison was always false - so whenever entity in the constraint disambiguation part was queried, it was not successful, regex query was applied, which takes some time, so we were losing quite some time there I guess -> solved in Commit a7b9bf436196cb0e93ad4028b2241bade689dbe2
There is always one disambiguated entity suggested for the cell, because the exact query always took only one matched solution: https://github.com/odalic/sti/blob/2e2e531a12c288661b2ed68233833c14eb704446/sti-kbproxy/src/main/java/uk/ac/shef/dcs/kbproxy/sparql/SPARQLProxy.java#L468. So please adjust that part, so that it adds all matching entities to the result.
Check there is support was custom types defined in the config files - in the version I used, the "rdf:type" is hard coded, see: https://github.com/odalic/sti/blob/2e2e531a12c288661b2ed68233833c14eb704446/sti-kbproxy/src/main/java/uk/ac/shef/dcs/kbproxy/sparql/SPARQLProxy.java#L53 . I think that in the latest commits (not merged yet), this is already addressed
queryForLabel - https://github.com/odalic/sti/blob/2e2e531a12c288661b2ed68233833c14eb704446/sti-kbproxy/src/main/java/uk/ac/shef/dcs/kbproxy/sparql/SPARQLProxy.java#L189- is not supported for generic SPARQLProxy! Please provide one (copy the one from DBPediaProxy and remove DBPedia related stuff)

@JanVana Please:

review the changes briefly (1h)
merge the current develop (there are some conflicting changes, but for you should be easy to merge, if not, please ask) (1h)
address the issues above, which were not solved (1-3h)
test and assign to me, I will retest

I have moved the custom RDF types to a separate issue (#261).


STAR-1233: Use Guava concurrent hash set instead of Apache Mina

Apache Mina is not really in the direct dependencies
of the project. It compiled accidentally because it
got onto the classpath through one of the
transitive dependencies of Netty.

This ran clearly twice in a row.
Previous failures were both seen on different branches:
    tests-stage-1 / jvm-unit-tests / jvm-unit-tests / org.apache.cassandra.index.sai.disk.v1.KDTreeSegmentMergerTest.compactionQueryTest
    tests-stage-2 / dtests / dtest-offheap-bti / dtest-offheap-bti-39 / dtest-offheap-bti.gossip_test.TestGossip.test_2dc_parallel_startup_one_seed

TestGossip failed also in this run: https://jenkins-stargazer.build.dsinternal.org/job/ds-cassandra-pr-gate/job/PR-410/4/


feat(windows): crash handler that writes minidumps to ...

... ProgramData/dev.firezone.client/dumps/last_crash.dmp, in debug builds only for now
For #3111.
I was way off on my estimated dump size. It's only about 500 KB.
Please double-check that the Linux exes are still stripped properly. Windows, Linux, and Mac have different ways to do embedded or non-embedded debug info, so the Windows client should still be 20 MB but with a ~62MB pdb next to it, and the Linux client should still be 15 MB.

[ ] Check Linux artifacts weren't accidentally affected
[ ] Make sure connlib's panic handler doesn't interfere with this
[ ] Later on, change CI so pdb files are saved as artifacts
[ ] Later on, enable for release builds

Crud, it leaked a process somehow
I'll change this back to a draft, process leaks would be a disaster.


disable resize

disable resize

Fixed! 663e840cda7938541f333dbf742f8af5ee932e0d It'll be in the next release (v2.3) which will come out soon

v2.3 is now released
https://github.com/swharden/Scan-A-Gator/releases


Importing Italy Department of Civil Protection Covid-19 Data into Data Commons

Thanks. Clean code except for the region and province list cleaning.
I did not check the schema, which is Tiffany Jann's expertise. I think she said to separate schema from data import code. Does anything need to be separated out of this code?
Thanks.

Thanks for reviewing! I resolved the issues. There's no new schema definition in this PR.


Makefile?

Hi @shinyblink , @vifino , @g-rden .
$ make
cc -pedantic -Wall -Wextra -Os -lxcb -lxcb-image -o ffshot ffshot.c
/usr/bin/ld: /tmp/ccIPsFIF.o: in function «main»:
ffshot.c:(.text.startup+0x4b): undefined reference to «xcb_connect»
...

Maybe replace:
https://github.com/shinyblink/ffshot/blob/854c5371983e0f053a30d758fcef04c424ddabdd/Makefile#L14-L15
to
$(BINS): ffshot.c
        $(CC) $(CPPFLAGS) $(CFLAGS) $^ $(LDFLAGS) -o $@

?

PS:
$ uname -a
Linux devuan-zvezdochiot 5.10.0-14-amd64 #1 SMP Debian 5.10.113-1 (2022-04-29) x86_64 GNU/Linux
$ cc --version
cc (Debian 10.2.1-6) 10.2.1 20210110

Hey. I agree with the order of the linking arguments, but not the target changes.
I'll push a commit in a second.

By the way, how'd you find this? Did interest in farbfeld utilities suddenly peak? :)

Hi @vifino .
I know farbfeld for other reasons. It was tested by me along with others. I was looking for an intermediate format for fast packing / unpacking of images (large volumes, scans). As a result, I settled on imagezero, but now I prefer fpng.


Prioritization of MLE messages over data and handling of child re-attach

Summarizing two features:

Prioritize MLE traffic over data packet. This is particularly important for MLE "Child Id Response" message.

Current implementation removes the child/neighbor when we recive a "Parent Requst" from it (child tries to re-attahc). This in turn causes all queued data packets for the node to be cleared. It'd be better to keep all the data packets  but clear out the queued MLE packets.

Resolved by #1106.


[Templete question] How to get list item index?

Svelte doc about listView and Template:
https://svelte-native.technology/docs#template-component
How to get item index when an item is rendered, like in NativeScript doc?
https://docs.nativescript.org/ui-and-styling.html#listview

i have a PR pending for this https://github.com/halfnelson/svelte-native/pull/165

if you use svelte-native 0.9.7 you can do let:index on the template to get the item index


Build 8.0 beta

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

One build (Linux ARM Pypy Python 3.7) is failing with this message:
SafetyError: The package for pypy3.7 located at /home/conda/feedstock_root/build_artifacts/pkg_cache/pypy3.7-7.3.7-h7cff6a8_2
appears to be corrupted. The path 'bin/pypy3'
has an incorrect size.
  reported size: 6152 bytes
  actual size: 2646 bytes

Any idea what it means?

Maybe a temporary problem only? Let's see if retrying fixes it.

I merged the PR, but I don't see any sign of it building the new packages.  There's no status indicator at https://github.com/conda-forge/openmm-feedstock/tree/rc, and https://anaconda.org/conda-forge/openmm/files doesn't show any new packages.

Try pushing an empty commit directly to rc to kick a retry.

Manually started here: https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=534510&view=results

Thanks!

That same build failed again.  It's an obscure enough combination (Pypy on ARM Linux) that I won't worry about it for the beta, but we should figure out what's going on.  The error message indicates that the pypy3 executable is corrupted.

It should be possible to restart the failed builds if it is just an intermittent issue

Since it failed twice in a row, I suspect it's something more than that.  The next step would be to test the pypy3.7-7.3.7-h7cff6a8_2 package directly.  I don't have access to an ARM Linux system, but maybe it would be possible to create a VM under emulation.

If the feedstock is configured to build Linux aarch64 on Azure (and re-rendered), it should be possible to run the build locally using QEMU on x86_64 (Linux). May need to comment out failing bits and add a wait to make sure the Docker container and contents can be investigated

I was thinking of something much simpler than that: just install that pypy package, try to run the executable, and see whether or not it really is broken.

@conda-forge-admin, please rerender


Latest version Caffe with caffe-classification-opencl: seg fault at the end

Steps to reproduce

under linux
ck compile program:caffe-classification-opencl

select 8) opencl caffe

ck compile program:caffe-classification-opencl

it produces correct classification output, but have a seg fault at the end...
This week we should try to check that OpenCL Caffe works across different platforms ...

duplicate of #76


DoIP Transport Layer

Hi, do you have any plans to add DoIP as a transport layer?
I have some code that will support DoIP I have no idea however how to implement in your solution.
Regards
Andy

It is designed so that other transports and connection protocols should be easy enough to add. The interface is the TpFactory. Do you have python code that interfaces to DoIP?
I have no equipment to test with but I'm more than happy to disuss it :)

Hi,
Please see attached DoIP code, also a Server you can spin up to test.
This was originally from another GIT Repo https://github.com/htkim40/Shiba_DoIP_Tool but modified slightly to work with https://github.com/pylessard/python-udsoncan
We had a Kivy UI for the app we created but it proved too complicated in the end to perform asynchronously so we dropped it.
This works code works perfectly on physical vehicles also.  Well Volvo’s anyway.
Andy
From: richClubb<EMAIL_ADDRESS>Sent: 18 May 2020 16:11
To: richClubb/python-uds<EMAIL_ADDRESS>Cc: Andrew Pitt<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [richClubb/python-uds] DoIP Transport Layer (#54)
It is designed so that other transports and connection protocols should be easy enough to add. The interface is the TpFactory. Do you have python code that interfaces to DoIP?
I have no equipment to test with but I'm more than happy to disuss it :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/richClubb/python-uds/issues/54#issuecomment-630247801, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APCQPPMVWYJ4AYZL2GOQ22DRSFFYDANCNFSM4NEDKCWA.

In my current project I'm need for UDS using DoIP to send a specific UDS commands to an ECU for a car prototype. It would be really nice, if I can use this library in combination with the Robot Framework to test my ECU. Are there any plans to close this issue in the near future? It is possible to give some support here?

https://github.com/richClubb/python-uds/pull/63

hi, is DoIP available to be use with ipv6? i saw the default ip was an ipv4 but i dont know if is available to change to ipv6

@MultiName90 if you're asking about my proposal for #63  then no, it's ipv4 for now. It wouldn't be hard to add to the upstream but I have no means to test. This particular project appears to be dead though.

Apologies.
I'm hoping to do some work on this this month to get a few contributions
into the trunk and a new release out.
I'm not working in the automotive industry any longer so I have less time
to work on this project.
Rich.
On Tue, 11 Jan 2022 at 13:07, Jacob Schaer @.***> wrote:

@MultiName90 https://github.com/MultiName90 if you're asking about my
proposal for #63 https://github.com/richClubb/python-uds/pull/63 then
no, it's ipv4 for now. It wouldn't be hard to add to the upstream but I
have no means to test. This particular project appears to be dead though.
—
Reply to this email directly, view it on GitHub
https://github.com/richClubb/python-uds/issues/54#issuecomment-1009947039,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMQRVCKVIMJHOVKHRK3YJTUVQTQHANCNFSM4NEDKCWA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID:
@.***>


Metadata Table format changes for Flyway 4.0

Migrate INIT to BASELINE
Reconsider/reevaluate quoting columns
Reconsider ignoring line endings in checksums

Duplicate of #1117


Multiple external data table at spec level

I could only specify a single .csv as table parameter. Would be great to have multiple csv in one spec.

Closing this as this use case is not clear.


api doc

This is an initial design of the Sotopia API

Closes #
📑 Description

✅ Checks

[ ] My pull request adheres to the code style of this project
[ ] My code requires changes to the documentation
[ ] I have updated the documentation as required
[ ] All the tests have passed
[ ] Branch name follows type/descript (e.g. feature/add-llm-agents)
[ ] Ready for code review

ℹ Additional Information

My high level question is whether we should allow agent and scenario design to be of a custom schema?

Not for now? Imo the current schema is quite flexible. You can literally
squeeze so much info into public information and secret. I do think we
should re visit how we compose those schema tho
On Wed, Oct 30, 2024 at 6:13 PM Hao Zhu @.***> wrote:

My high level question is whether we should allow agent and scenario
design to be of a custom schema?
—
Reply to this email directly, view it on GitHub
https://github.com/sotopia-lab/sotopia/pull/242#issuecomment-2448556106,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE35IXOYEDPGBW6VHIQHM73Z6FKXVAVCNFSM6AAAAABQ422FJWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINBYGU2TMMJQGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

It would be great if we can allow people who use our apis to bring their own agent specifications, i.e. having a generic AgentProfile

Please give a concrete example in terms of fastAPI
On Wed, Oct 30, 2024 at 11:55 PM Hao Zhu @.***> wrote:

It would be great if we can allow people who use our apis to bring their
own agent specifications, i.e. having a generic AgentProfile
—
Reply to this email directly, view it on GitHub
https://github.com/sotopia-lab/sotopia/pull/242#issuecomment-2448964348,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE35IXI77J7B7DV73675N6DZ6GS3JAVCNFSM6AAAAABQ422FJWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINBYHE3DIMZUHA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Let's merge this imo

Cool It looks great. Just left some final request changes.
BTW, in all of endpoints /{get/post}/... seems to be not useful, i.e. we might just do GET /agents instead of GET /get/agents.

@ProKil why get is not useful? we want to disabiguiate from post right?

Cool It looks great. Just left some final request changes.
BTW, in all of endpoints /{get/post}/... seems to be not useful, i.e. we might just do GET /agents instead of GET /get/agents.

@ProKil why get is not useful? we want to disabiguiate from post right?

Since when you send an request, you will specify whether the method is GET/POST/PUT/DEL, have these method names in the end point is redundant.
For example,
import requests

# Make a GET request
response = requests.get('https://api.example.com/data')

# Make a POST request
data = ...
response = requests.post('https://api.example.com/data', json=data)

You don't need to use ' requests.get('https://api.example.com/get/data')'


[Autocomplete] Wrong T inference

I just updated to last lap version 4.0.0-alpha.40 which broke my <Autocomplete /> component so I'm trying to adapt to the changes. However I can figure out how to do it.
I've made a codesandbox example (code below):
import * as React from "react";
import { Autocomplete } from "@material-ui/lab";
import TextField from "@material-ui/core/TextField";

interface Person {
  firstName: string;
  lastName: string;
}

const initialValues: Person[] = [{ firstName: "John", lastName: "Doe" }];
const options: Person[] = [
  { firstName: "John", lastName: "Doe" },
  { firstName: "Jane", lastName: "Doe" }
];

export default function App() {
  const [values, setValues] = React.useState(initialValues);
  const handleSelect = (e: any, v: Person[]) => {
    setValues(v);
  };

  return (
    <div>
      <Autocomplete
        multiple
        options={options}
        getOptionLabel={option => option.firstName}
        value={values}
        onChange={handleSelect}
        renderInput={params => (
          <TextField {...params} label="Persons" fullWidth />
        )}
      />
    </div>
  );
}

It seems that options and onChange disagree on the types.

I don't think this issue should be closed. I'm having the same problem with it: it doesn't infer the right type for the option.
interface MyOption {
    someProp: number;
    someOtherProp: string;
}

const options: MyOption[] = [ /*...*/ ];

return (
    <Autocomplete
        options={options}
    />
);

In the example above, the component is going to infer MyOption[][] instead of MyOption[]. Although the suggested workaround makes the code compile again, it just feels weird and I believe it should be fixed in future versions.
This worked until 4.0.0-alpha-39 and stopped working with 4.0.0-alpha-40.

Do you have a reproduction?

@oliviertassinari  https://codesandbox.io/s/eager-blackwell-1vfve?fontsize=14&hidenavigation=1&theme=dark

@oliviertassinari for some reason, I can't get @akharkhonov 's example to fail, even after removing the explicit generic parameter, but in @mr-bjerre 's example, just change const handleSelect = (e: any, v: any]) => { to const handleSelect = (e: any, v: Person[]) => { and you'll be able to see it.

@andreivanea Type something into the textfield, press Enter and press button. You will see fail.

@akharkhonov I get it now. I thought you were trying to point out something else. Your example shows a runtime error, while I was talking about a compile-time one (TypeScript complains about incompatible types).

I am having this issue using the onChange function of an autocomplete with multiple = true. Hope this is the right place to post.
<Autocomplete
      multiple
      options={users} // TeamMember[]
      renderInput={params =><TextField {...params} /> }
      onChange={handleChange}
      getOptionLabel={option => option.firstName + " " + option.lastName}
    />

Seems multiple is messing up the type inference. So if my options is an array, the onchange value is not an array, right? So handleChange should be (event: any, value: TeamMember | null) => void; But it infers the type TeamMember[] for value. So if I provide TeamMember[] in the onChange it infers TeamMember[][] for options.
Commenting out multiple or onChange fixes it for Typescript.
Specifying the type <Autocomplete<TeamMember> ... seems good but then TeamMember[] is inferred in event value. But it shouldn't give an array for the value, should it?
So for now we have to provide any for onChange value which is not awesome.

https://github.com/mui-org/material-ui/pull/20949 fixes issue described in the first comment without having to specify explicit value type.
Issue described by @akharkhonov is something separate I believe.


some box/whisker plots not responding to global min/max p10/p90 setting

Box and whisker plots other than the measurement value chart do not seem to be responding to the global setting for p10/p90.
Seeing this in several charts:
#/cdm///condition_occurrence/- age at first diagnosis chart
#/cdm///measurement/ - age at first occurrence chart
appears to be inconsistent with measurement report.

Only the charts mentioned in the issue + those that had the button earlier https://github.com/OHDSI/Ares/issues/173 are affected by this setting.
I can add it to other charts. As far as I understand those should be all the bar charts using the p10/p90 and min/max values.


feat(config): persist credentials to file

Add a method to obtain the credentials if not present in the configurations. The credentials are stored in the current working directory or in a directory passed from the command line.
If the credentials are not present, it will try to register the device on Astarte with the pairing token.
Added tempfile crate to test using the file system.
Closes #96

The credentials and store could also be put in the config.toml.

This closes also #103


[enhancement](string column)copy string continuous

Proposed changes
Issue Number: close #xxx
Problem summary
Some query read string columns continuously, but the string will be copied to ColumnString one by one from page. There are two improvement in this PR:

Resize the chars using the avg size of 100 string rows.
Calculate the max continuous rows and copy using a single memcpy command.

I also add a padding size in page, so that we could using overflow memcpy in the future.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[ ] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[ ] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[ ] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

In lastes clang-14, our memcpy_overflow will be replaced by memcpy...
void my_memcpy(char * __restrict dst, const char * __restrict src, ssize_t n)
{
while (n > 0)
{
_mm_storeu_si128(reinterpret_cast<__m128i *>(dst),
_mm_loadu_si128(reinterpret_cast<const __m128i *>(src)));
dst += 16;
src += 16;
n -= 16;
}
}
You can find detail in this discussion: https://github.com/llvm/llvm-project/issues/56467


"API key is not valid" for fresh account

When connecting API key for a account that was never in the game api returns:
{'text': 'ErrBadData'}
so the plugin shows error: "API key is not valid". It is probably not the plugin problem but maybe you can bypass it somehow and pass bug request on to api developers?
This affects only those who just begin their gw2 journey, so not very common right now, but in being a FoG fork may make it more popular and new gw2 users will waste time trying to figure out what is going wrong, try to change API permission scope like me ect.
BTW please write what minimal permission scope is needed for plugin to run properly.

@FriendsOfGalaxy improved description for error and clarified permission scope

Thanks, that should be enough


Update AWS libraries to pick up a version of fast-xml-parser that addresses CVE-2023-34104.

Hi!
We would like to update the versions of the AWS SDK to one that depends on fast-xml-parser 4.2.5 instead of 4.1.2. 4.1.2 is vulnerable to https://www.cve.org/CVERecord?id=CVE-2023-34104, while 4.2.5 is not.
I ran the simple example against an MSK cluster we have and it authenticated without any issues. If there are any other changes you'd like or steps you want me to follow, just let me know.
Thanks!

I will be away for the week of July 10th so if there are any comments/questions/whatever feel free to leave them here and I'll check in when I'm back.


Add TryStreamExt::try_filter_map

A result-aware version of filter_map would be very useful

This looks great! Let me know when you've had a chance to fix the example and docs nits and I'll merge.


.removeAllListeners(eventName) argument should be optional

... to remove all listeners from all events if eventName is not specified.
tbd in #6

Fixed in #6. Thanks!


Implement backup and restore without storage permission

Hello. So, I have seen that this app advertises backup/restore feature but that functionality is missing in the actual app. Thus, I decided to look around in the code and I found out that the backup/restore code was actually present, but it seems to be tied to the storage permission. I have ported the code to use the ActivityResultContracts API, which requires no permission. I hope you can take a look at it.

That's awesome, @unbiaseduser! Thank you very much.
Please check if we can merge it, @arafaatqureshi.

@unbiaseduser  Thank you so much for your input, i was still working with the same issue. Highly appreciated.
Let me go through your code, will merge it after that :)

@arafaatqureshi Any progress so far? This is taking a bit too long for a simple change.
Keep in mind that it is you who wrote the main backup/restore code and I just adapted it a little bit.
I really don't think that it should take days for you to review...

Oh thank you, good sir. I was worrying you would be inactive.


Results offset when using a DateTime that is utc as input

When using calculateEvent or calculateForDay of the DaylightCalculator and using an input DateTime which already is UTC, the results are offset from the correct time by the difference between local and utc.
You should be able to easily reproduce this problem by using, e.g. final todayUtc = DateTime.utc(2020, 1, 19) as the input.

I got the same problem, while I need to get calculation at position which is not specifically the same as the phone timezone...
Any news about it?

Added a fix for this on #20


[c#]No warnings if there are duplicate binding arguments

Repro step:

Create QueueTrigger C#
Add input binding : documentdb
Add output binding : documentdb

This adds documentdb input and output bindings with  "name": "document"
Expected: Warning indicating duplicate names

What is the behavior at runtime in this scenario? We need to handle it well in the runtime, but the Portal needs to avoid getting people into this situation in the first place. I.e., it needs to use unique names for bindings. Recommend creating a portal bug for that.

This has been fixed.


Create a framework for testing the performance of various encoding methods.

In addition to correctness testing via unit- and integration tests, we need to be able to evaluate how different approaches (e.g. interleaving vs flat set-cherrypick) affect performance.

Is this something we could/should use the framework in apalache-bench for?

Well, first we need to have a discussion regarding what we consider to be a metric for performance. But ultimately, probably.


dotenv state diverge from .env file

Describe the bug
am using devenv with direnv integration.
when updating .env devenv doesn't rebuild.
To reproduce
create a dummy .env with a couple of envs.
enable dotenv.
  ...
  dotenv.enable = true;
  ...

build shell, update .env. then devenv is out of sync.
not sure if its a devenv problem or a direnv problem.
Version
devenv 1.3.1

@sandydoo huh, shouldn't that be detected via cache paths?

@sandydoo yes and no, when i reload it registers the change, but i have to do it every time i change .env file

@sandydoo apparently when i created a project recently but i used the flake template to instantiate devenv.
nix flake init --template github:cachix/devenv
``

it works just fine, it picks change to .env just fine


Parse odrdate in correct tz

Resolves #137

Since America/Chicago is subject to DST, I opted to use gettz() vs a static -5h/-6h offset.

I don't understand what you mean by isort?

I googled it. Found it. Ran it.


Fixes precedence of Squeak's platform attributes

In particular, the alpha-fallback got appended only after the attribute for the current system version, which is wrong because a more recent symbolic version could never be matched. Also increase the latest known Squeak 5 to version 5.3 to make that alpha-fallback work properly again in configurations that do not have a squeak6.x configured yet for 6.0alpha, which is the current Trunk.
(Authored by @marceltaeumel)

:shipit:


Use JDBC batches for inserting Raptor shard metadata

This improves performance, especially when there is substantial latency
between Presto and the metadata database.

Do all of the inserts still happen in a single transaction (assuming they did that before)?
Looks good

Yes, it is still in a single transaction.


Make columns space-filling again

See #196 and #172

The algorithm so far:

Sum the the width of locked columns
Sum the width of columns that have already the minimum width
Substract both sums from the available width of the container
Devide by the number of columns left -> new width
Set width columns that are flexible (not locked):

If new width > max width -> max width
If new width < max width -> new width
If new width < minimum width -> minimum width

There is still a bug with a wrong x-scale of the multiform visualization (see https://github.com/Caleydo/mothertable/issues/198)

Looks good so far. @bikramkawan, please also change the min width of the columns soon. However, we can merge this PR without those changes of course.

It looks good holger. I think you forgot something after removing column there are some white spaces as here

@bikramkawan Thanks for having a look. My implementation is correct, as you can see from the column header. The whitspace results from the wrong/not applied x-scale factor (issue #198).
If this is fine, I go ahead and merge this PR.


Restore original canvas context when destroying

Fixes #713
Inspired by the work done in #847 but addresses the use case of resetting CSS width/height regardless of whether we have a devicePixelRatio or not.
By setting the width/height of the canvas itself, the spec says it should completely reset any of the state applied to the canvas context.
Section 4.8.11:

When the canvas element is created, and subsequently whenever the width and height attributes are set (whether to a new value or to the previous value), the bitmap and any associated contexts must be cleared back to their initial state and reinitialized with the newly specified coordinate space dimensions.

+1 I will close #847


How to when click a plugin , it will show another plugin.

Hi, your editor is very nice, I really love it, but I have a problem, example:

I have a plugin 'format_text', then I click, it will show another plugin like bold, fontSize, fontColor, etc..

Is it possible ? if yes, how can I do it ?.
Thank you first.
Best regards,
ChiBach

Hi @BuiChiBach
Yes it is possible.
But it will be a bit complicated.
What is your HTML desired structure?

Hi @JiHong88 , It's like this.

Can you give me that link?

This is the 'New Message' form of Gmail. :D

The 2.26.0 version has been updated.
How to create custom plugin

Thank you @JiHong88  <3


Update bluefin-dx.md: s/Jetbrains/JetBrains

The company name is spelled as "JetBrains", not "Jetbrains".

Thanks!

@p5 Thank you for your work!


Question: Language switch

we would like to implement a language switch, so that the user is able to switch the needed language in the application.
For this i set  Windows.Globalization.ApplicationLanguages.PrimaryLanguageOverride, but it change only the language on application restart.
Is there a way to force reload the screen / window?
attached is my demo project
App1.zip
.

I believe this behavior is expected. See this description in the ApplicationLanguages.PrimaryLanguageOverride Property API doc:

When you set the PrimaryLanguageOverride, this is immediately reflected in the ApplicationLanguages.Languages property. However, this change may not take effect immediately on resources loaded in the app UI. To make sure the app responds to such changes, you can listen to the QualifierValues property on a default resource context and take whatever actions may be needed to reload resources. Those requirements may vary depending on the UI framework used by the app, and it may be necessary to restart the app.

Then, you can reload your strings from the default ResourceContext. There's an example in Updating strings in response to qualifier value change events that should be helpful.
Hope this helps!

Thanks for the response.
I also found those article earlier but the issue is how to reset the resource container.
Unfortunately winui 3 is not based on WindowsCore. Core unsupported classes
So that each time i try to get the Windows.ApplicationModel.Resources.Core.ResourceContext.GetForCurrentView() i get the exception "RPC_E_WRONG_THREAD", Even if i use the dispatcher and dispatcherqueue.
So question is is there a way in winui 3 to reset the ResourceContext to that my page is able to reload?

I see, sorry about missing the UWP/WinUI3 difference here. From the MRT to MRT Core migration doc, are you able to use the
Microsoft.Windows.ApplicationModel.Resources.ResourceManager.CreateResourceContext() API
rather than
Windows.ApplicationModel.Resources.Core.ResourceContext.GetForCurrentView()?
And this version of the ResourceLoader?
Microsoft.Windows.ApplicationModel.Resources.ResourceLoader

So far i see there is no option to reset / reinit the ResourceManager or the ResourceContext.
The current issue is that i set the language to a different one with Windows.Globalization.ApplicationLanguages.PrimaryLanguageOverride, but the app is not updating.
I also figured out that this only happens with the xaml x:uid markup.
<TextBlock  x:Uid="TestTextBlock" />
Is there an option to reset the resource so that the UI is updating to the other language ?

You can try the WinUI3Localizer:

Switch languages without app restarting
You/users can edit localized strings even after deployment
You/users can add new languages even after deployment
Use standard Resources.resw (see Microsoft docs)

You solved this problem? @Pupka-Lipinski

@MarcoOliveira2 :
all you need to do is:

set Windows.Globalization.ApplicationLanguages.PrimaryLanguageOverride
reload you page frame.Navigate(frame.CurrentSourcePageType);

I change to:
private async void Lang_PT(object sender, RoutedEventArgs e)
{
     ApplicationLanguages.PrimaryLanguageOverride = "pt";
     Type t = ContentFrame.Content.GetType();
     ContentFrame.Navigate(t);
}

Still the same problem :/

我为此发布了一个 NuGet 包。
WinUI3Localizer：

在不重新启动应用程序的情况下切换语言
您/用户即使在部署后也可以编辑本地化字符串
您/用户即使在部署后也可以添加新语言
使用标准 Resources.resw（请参阅 Microsoft 文档)

I have published a NuGet package for this.
WinUI3Localizer:

Switch languages without app restarting
You/users can edit localized strings even after deployment
You/users can add new languages even after deployment
Use standard Resources.resw (see Microsoft docs)

It perfectly solved my problem, so that I didn't need to consult the poor and even unknown WinUi documentation


Trigger build on default branch when saving a project

Close #5358

@stsewd
Tests are failing because every project don't have a default_branch.
I am thinking to add try.. except block.. if it fails then go for LATEST version.
Need your suggestions.

get_default_branch() is returning master for git based projects which is not present as version  for many projects.

Ok, looks like the method is used for another purpose (to get the version to which point to latest, not latest itself). We need something like projec.default_branch or LATEST I think

@stsewd
I have implemented that and also added the test.

@humitos
Updated the doc string.


New Colorbar

Automatic font sizing

Auto Scaling
Automatically scales for all resolutions.
Significant Figures
Rather than requiring the user to manually enter the number of decimals for every variable, it now uses significant figures so the default of 4 will round off to the first 4 digits and only include decimals if needed. This also solves the issue that the old color bar would show garbage values that are outside of the precision of floats. For example, below we have a color bar that goes from 42 to 1170000. On the left, is the old color bar and on the right the new one. The old shows garbage values that are outside of the float precision range.

Scientific notation

Pixel-Perfect Borders, Ticks, Padding, etc.
All graphical elements are corrected for aspect ratio and aligned to the pixel grid. Old on left, new on right.

Position Quick Select Widget
Custom widget which allows for quick and easy placement in commonly used annotation locations.

All settings play well together
All settings give a valid result. With the previous color bar it was very easy to end up with something like below. Often as you would fix one parameter, it would break other parameters, creating a frustrating feedback loop.

Correct tick placement and start/stop
It was not possible to have ticks at the min/max in the old color bar so it would never match up with the transfer function. The old ticks would also not actually point to the correct datapoint that the number shows. As you can see in the screenshot above the ticks are somehow outside of the color bar entirely. Both transfer functions below go from 42 to 1170000.

Titles
Auto-adjusts the size for titles. The old widget just rendered it outside the annotation window.

Simple intuitive use

Position works independent of size. (before if you adjusted one, you had to adjust the other. Also if position + size > 1 it would clip off the screen)
Height parameter adjusts height, while scale scales the entire widget. Both parameters seamlessly work together.

 It will nice to finally have this thing cleaned up. A couple of things I've noticed.
No colorbar is displayed for the wireframe renderer.

I think the wireframe renderer is breaking something in the OpenGL state. I am still looking into it.

You indicate that this PR closes #2178. I don't believe that is the case. I think what Pat is asking for is the ability to explicitly place the tick marks by setting, for example, a minimum value and an increment (similar to what is done for specifying contour lines). This allows you to put the tick marks at round numbers.

Here is why I did that:

The requests are:

Set a minimum value
Set an increment
Place ticks at round numbers
Use tick values from a list

With the old color bar, it was not at all possible to do any of those things.

Set a minimum value: No - All tick values are arbitrary[1]
Set an increment: No - All tick values are arbitrary[1]
Place ticks at round numbers: No - The tick values are always displayed with more precision than they actually have
Use tick values from a list: No

With the new color bar

Set a minimum value: Yes - The minimum value will be exactly the lower bound of the transfer function
Set an increment: Yes - Since you have the exact lower and upper bounds, you can set the number of ticks to 1 + (upper-lower) / increment
Place ticks at round numbers: Yes - With the above you can already get round numbers theoretically but floating point error would prevent this. If you lower the number of significant figures, it will round off the numbers.
Use tick values from a list: No but this seems like the least important use case but requires by far the most work and Pat seems to suggest it as an alternative to a minimum with an increment so I don't think it is worth the effort to add it.

For example if you have a TF with a range of 0 to 100, here is the default color bar:

Decrease the sig figs by 2:

If you want an increment of 10, increase the ticks from 6 to 11:

 [1] Technically they're not arbitrary, but as far as the end user is concerned they are.

I understand your point, but the implications are that if you want have tick marks at nice round numbers you have to define your transfer function accordingly, which may be a limitation that users do not want. @phawbeck what are your thoughts on this?

Hi Stas and John, I hope everything is well! I think this is a great improvement on the colorbars and am happy to see the new adjustments! I see what John is saying in that there are a lot of times I'll want the lowest rendered value to be whatever the minimum is (let's just say that's 0.34) and the maximum (e.g., 52), but would want ticks to be at some round numbers to make it clean (e.g., 10, 30, 50). Is there anyway to disassociate the minimum tick labels from the actual minimum of the TF?
Thanks a lot for looking into this!

Hi Stas and John, I hope everything is well! I think this is a great improvement on the colorbars and am happy to see the new adjustments! I see what John is saying in that there are a lot of times I'll want the lowest rendered value to be whatever the minimum is (let's just say that's 0.34) and the maximum (e.g., 52), but would want ticks to be at some round numbers to make it clean (e.g., 10, 30, 50). Is there anyway to disassociate the minimum tick labels from the actual minimum of the TF?
Thanks a lot for looking into this!

@phawbeck Thanks for your feedback and I see your point. For arguments sake and to explain my rational, I have recreated your scenario. Leaving the range as 0.34 to 52, below is the default Colorbar you would get. If you decrease the sig figs, you will get reasonable clean ticks.

In this case where you just decrease the sig figs until you round to the tens place, you have a maximum error of 4% in color gradient. Here is what 4% error looks like:

Another option is to change the range to 0 to 50. It will still render 0.34 to 52: 50-52 will not be clipped and the 0-0.34 will be 99.6% blue rather than 100% blue which would be indiscernible. In this case the last 50-52 will have the same color. Vapor's default colormap has had its right color control point not quite at 100% for years so the top of the TF range will have the same color and nobody's ever noticed.

I guess in conclusion @clyne maybe we need to have further discussion on what the abilities should be. Since there weren't any deliverables for this, my goal was to fix all the issues with the old Colorbar and make it do one thing well with a very simple interface.
Also I fixed the issue with the Colorbar not rendering for the wireframe renderer.

Not sure if it's intended, but introducing a while rectangle in the rendering doesn't seem cool.

Following the steps below you'll get a buggy colorbar:

Compile the code in "Release" mode
Open VAPOR with one of Leigh Orf's cm1.nc files.
Create a Slice renderer. Enable it with all default parameters.
Go to "Annotation" tab and click on "Show Colorbar"
You'll get the image blow:

@clyne I removed issue from the fixed list. I think we should see if the new functionality is sufficient and we can go back and add the addition features if needed.
@shaomeng I fixed the issue, thanks. It was due to a non-deterministic memory error which for some reason I could never reproduce on my computer but I was able to reproduce it on a different system.

Sounds good, @StasJ

@stasj this PR is good to go as soon as you resolve the conflicts. Thanks.

@clyne Done.


Your Ubuntu installation doesn;t work

Describe the bug
I am not a developer so I dont know what you are asking for. But I followed the instructions for Ubuntu installation. And it says falco package doesn't exist ( or something to that effect )

How to reproduce it

Expected behaviour

Screenshots

Environment

Falco version:

System info:

Cloud provider or hardware configuration:
OS:

Kernel:

Installation method:

Additional context

Hi,
Which version of Ubuntu are you using ?
Did you follow this guide ?
Might be a silly question but did you ensure to call sudo apt-get update after configuring the apt repository ?


Investigate Genesis Mismatch Issue

Description
Users are unable to sync mainnet , they are the nodes returns the following error
Thread 'tokio-runtime-worker' panicked at 'Storage root must match that calculated.', /root/.cargo/git/checkouts/polkadot-sdk-cff69157b985ed76/7049c3c/substrate/frame/executive/src/lib.rs:834

Tasks

[ ] Investigate issue and resolve
[ ] Document findings

Related Links

Discord Support Thread

@open-junius can you please update this ticket with your findings. Let also track the issue here and not on discord

I checked the both test and finney environments, actually the genesis hash is the same for old node and new polkadot-sdk v1.10 based node. The error is not from simple comparing the hash value.
Then I did some experiment in local env, build three different nodes based on v0.9.37, v1.0.0 and v1.10.0. I can see the v1.0.0 node can talk with v0.9.37 without any mismatched genesis error. It happened when I use the v1.10.0 node to connect v0.9.37 node, lots of mismatched genesis error from different peers.
Finally, I checked the network from the same v1.10.0 nodes, it is ok without any error.

Duplicate https://github.com/opentensor/subtensor/issues/655