Use libffi 3.2.1 for ppc64(le)

Use libffi 3.2.1 for ppc64.

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
Non-GitHub Verified Committers
There are 1 commit author(s) whose commits are authored by a non-GitHub verified email address. Chef will have to manually verify that they are authorized to contribute.
Please sign the CLA here.

Be sure to bring in the updated omnibus-software now that https://github.com/chef/omnibus-software/pull/378 was merged.

:+1:

This PR has been replaced by: https://github.com/chef/omnibus-chef/pull/384


Error when applying new changes - Reverse DNS

I have found a bug where if you have pretty_ipv6_ptr enabled in settings -> basic, you encounter the "Error when applying new changes" error when trying to save a reverse DNS zone with an @ record (needed for NS records.) I can't have it enabled as it doesn't seem to let me add the @ NS records for the reverse DNS zone.

Thanks for reporting the issue. It has been fixed in the referenced commit above.


[bug] tauri v2.0.0-beta.3 failed to define permissions for path: failed to write file:

Describe the bug
Hi,
I'm trying to develop on Windows machine. I've got an error while executing cargo tauri dev
   Compiling tauri-utils v2.0.0-beta.2
   Compiling tauri-build v2.0.0-beta.2
   Compiling tauri-plugin v2.0.0-beta.2
   Compiling tauri-codegen v2.0.0-beta.2
   Compiling tauri v2.0.0-beta.3
   Compiling tauri-plugin-shell v2.0.0-beta.0
   Compiling tauri-macros v2.0.0-beta.2
   Compiling prs-ai-cht v0.0.0 (P:\projects\prs-ai-cht\src-tauri)
error: failed to run custom build command for `tauri v2.0.0-beta.3`

Caused by:
  process didn't exit successfully: `P:\projects\prs-ai-cht\target\debug\build\tauri-8dd888a98b32fd98\build-script-build` (exit code: 101)
  --- stdout
  cargo:rustc-cfg=dev
  cargo:rustc-cfg=desktop
  cargo:rustc-cfg=ipc_custom_protocol

  --- stderr
  thread 'main' panicked at C:\Users\micha\.cargo\registry\src\index.crates.io-6f17d22bba15001f\tauri-2.0.0-beta.3\build.rs:357:25:
  failed to define permissions for path: failed to write file: Nazwa pliku, nazwa katalogu lub składnia etykiety woluminu jest niepoprawna. (os error 123)
  note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
warning: build failed, waiting for other jobs to finish...
    Blocking waiting for file lock on build directory

2024-02-19T20:50:48.115895Z ERROR ❌ error
error from build pipeline

Caused by:
    0: HTML build pipeline failed (1 errors), showing first
    1: error from asset pipeline
    2: bad status returned from cargo artifacts request

Reproduction
cargo create-tauri-app --beta prs-ai-cht
cd .\prs-ai-cht\
cargo tauri dev

Expected behavior
Program compiles just fine
Full tauri info output
cargo tauri info
WARNING: no lock files found, defaulting to npm

[✔] Environment
    - OS: Windows 10.0.22631 X64
    ✔ WebView2: 121.0.2277.128
    ✔ MSVC: Visual Studio Community 2022
    ✔ rustc: 1.76.0 (07dca489a 2024-02-04)
    ✔ cargo: 1.76.0 (c84b36747 2024-01-18)
    ✔ rustup: 1.26.0 (5af9b9484 2023-04-05)
    ✔ Rust toolchain: stable-x86_64-pc-windows-msvc (environment override by RUSTUP_TOOLCHAIN)

[-] Packages
    - tauri [RUST]: 2.0.0-beta.3
    - tauri-build [RUST]: 2.0.0-beta.2
    - wry [RUST]: 0.36.0
    - tao [RUST]: 0.25.0
    - tauri-cli [RUST]: 2.0.0-beta.2
    - @tauri-apps/api : not installed!
    - @tauri-apps/cli [NPM]: 2.0.0-beta.2

[-] App
    - build-type: bundle
    - CSP: unset
    - frontendDist: ../dist
    - devUrl: http://localhost:1420/

### Stack trace

_No response_

### Additional context

_No response_

I couldn't reproduce, but note that tauri-plugin-shell is still not updated to tauri v2.0.0-beta.3, try removing that from your Cargo.toml and main.rs then try again

Could be related? https://github.com/tauri-apps/tauri/issues/8924

Looks like a different issue, but root cause may be similar.

I don't see this problem with v2.0.0-beta.16 anymore. I've got other problem, but it may be system configuration, webview related.

It looks like trunk update fixed the issue. Everything works. Thanks!


Can't see element while dragging

I have a basic sortable working using handle option on a list of divs. Unfortunately, the item is not visible while dragging it. Instead, I just see the default paper icon Chrome normally shows.
Is there some setting I'm missing?

Try dev-branch, perhaps the problem has been solved;
Use the search, maybe already have an answer;
If not found, create example on jsbin.com (draft) and describe the problem.

Thanks, I'll close this for now.


UI change - Button

Change the button design

@scorcism  please assign it to me.

@SamMrach Go for it


Kubernetes Ingress with Cert-Manager fails to run out of the box (on AKS)

I've been trying to run the example ingress with cert-manager on Azure AKS, but I can't get it to work.
I think the documentation can be improved to include more explicitly say you need to install the ClusterIssuers yourself, but aside from that I'm stuck with cert-manager logging:
cert-manager/controller/challenges "msg"="propagation check failed" "error"="wrong status code '404', expected '200'" "dnsName"="[REDACTED]" "resource_kind"="Challenge" "resource_name"="ingress-cert-3036365264-2507512598-1639085740" "resource_namespace"="istio-system" "type"="http-01"

Looking at the ingresses, I find the one needed for the certificate: cm-acme-http-solver-qlzvs. In its description, I find the url that should be routed to the solver service. When I request the url manually, I indeed get the 404:
$ curl dev2.digitaltwin.imec-apt.be/.well-known/acme-challenge/JJRrPbhzEnwA2dyVzG_k1QPTdmbUzb-0li5hl8E021c -v
*   Trying [REDACTED IP]:80...
* TCP_NODELAY set
* Connected to [REDACTED DNS NAME] ([REDACTED IP]) port 80 (#0)
> GET /.well-known/acme-challenge/JJRrPbhzEnwA2dyVzG_k1QPTdmbUzb-0li5hl8E021c HTTP/1.1
> Host: [REDACTED DNS NAME]
> User-Agent: curl/7.65.3
> Accept: */*
>
* Mark bundle as not supporting multiuse
< HTTP/1.1 404 Not Found
< date: Tue, 10 Mar 2020 10:23:30 GMT
< server: istio-envoy
< content-length: 0
<
* Connection #0 to host dev2.digitaltwin.imec-apt.be left intact

Apparently there is no route configured (I found this neat trick in another bug report, only while creating this issue: https://github.com/istio/istio/issues/20589):
$ istioctl proxy-config routes $(kubectl get pod --selector app=istio-ingressgateway --output jsonpath='{.items[0].metadata.name}' -n istio-system) -n istio-system --name http.80 -o json
[
    {
        "name": "http.80",
        "virtualHosts": [
            {
                "name": "[REDACTED DNS NAME]:80",
                "domains": [
                    "[REDACTED DNS NAME]",
                    "[REDACTED DNS NAME]:80"
                ],
                "routes": [
                    {
                        "match": {
                            "path": "/hello",
                            "caseSensitive": true
                        },
                        "route": {
                            "cluster": "outbound|5000||helloworld.default.svc.cluster.local",
                            "timeout": "0s",
                            "retryPolicy": {
                                "retryOn": "connect-failure,refused-stream,unavailable,cancelled,resource-exhausted,retriable-status-codes",
                                "numRetries": 2,
                                "retryHostPredicate": [
                                    {
                                        "name": "envoy.retry_host_predicates.previous_hosts"
                                    }
                                ],
                                "hostSelectionRetryMaxAttempts": "5",
                                "retriableStatusCodes": [
                                    503
                                ]
                            },
                            "maxGrpcTimeout": "0s"
                        },
                        "metadata": {
                            "filterMetadata": {
                                "istio": {
                                    "config": "/apis/networking.istio.io/v1alpha3/namespaces/default/virtual-service/dev2-digitaltwin-imec-apt-be-helloworld-ingress-istio-autogenerated-k8s-ingress"
                                }
                            }
                        },
                        "decorator": {
                            "operation": "helloworld.default.svc.cluster.local:5000/hello"
                        }
                    }
                ]
            }
        ],
        "validateClusters": false
    }
]

Expected behavior
To eventually resolve the certificate.
Steps to reproduce the bug
Just follow the example in your documentation: https://istio.io/docs/tasks/traffic-management/ingress/ingress-certmgr/
Version (include the output of istioctl version --remote and kubectl version and helm version if you used Helm)
$ istioctl version --remote
client version: 1.5.0
control plane version: 1.5.0
data plane version: 1.5.0 (2 proxies)

$ kubectl version
Client Version: version.Info{Major:"1", Minor:"14", GitVersion:"v1.14.2", GitCommit:"66049e3b21efe110454d67df4fa62b08ea79a19b", GitTreeState:"clean", BuildDate:"2019-05-16T16:23:09Z", GoVersion:"go1.12.5", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"14", GitVersion:"v1.14.8", GitCommit:"c52f59bbba5fbf21fbb18e9a06f96e563fe4c20a", GitTreeState:"clean", BuildDate:"2020-01-31T20:00:26Z", GoVersion:"go1.12.10", Compiler:"gc", Platform:"linux/amd64"}

$ helm version
version.BuildInfo{Version:"v3.1.1", GitCommit:"afe70585407b420d0097d07b21c47dc511525ac8", GitTreeState:"clean", GoVersion:"go1.13.8"}

How was Istio installed?
Istio was installed with (also see example):
istioctl manifest apply \
  --set values.gateways.istio-ingressgateway.sds.enabled=true \
  --set values.global.k8sIngress.enabled=true \
  --set values.global.k8sIngress.enableHttps=true \
  --set values.global.k8sIngress.gatewayName=ingressgateway

Environment where bug was observed (cloud vendor, OS, etc)
Environment is Azure AKS.
Btw: please update the curl command at the above url to use bash instead of sh, to get this dump:
curl https://raw.githubusercontent.com/istio/istio/release-1.5/tools/dump_kubernetes.sh | bash -s -- -z, sh is not always the same as bash.
istio-dump.tar.gz

You can't rename the resource, it has to be ingress-istio-autogenerated-k8s-ingress:  https://github.com/istio/istio/issues/13113

Interesting, thanks!
I'll try it out. Weird that the documentation tells me to use a different name though.

Thanks for the report @jwapt. Sorry for the confusion here. In 1.6 we have improved this a bit and hopefully the docs are better as well https://preliminary.istio.io/docs/ops/integrations/certmanager/ and https://preliminary.istio.io/docs/tasks/traffic-management/ingress/kubernetes-ingress/. Thanks!


Update to node16

Node 12 has an end of life on April 30, 2022.
This PR updates the default runtime to node16, rather then node12.
This is supported on all Actions Runners v2.285.0 or later so we will need to major version the action

@thboop could you please merge the main branch into yours?

Closed in favour of https://github.com/actions/publish-action/pull/9


fix(react): add missing react memory router

Pull request checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[ ] Build (npm run build) was run locally and any changes were pushed
[ ] Lint (npm run lint) has passed locally and any fixes were made for failures

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Does this introduce a breaking change?

[ ] Yes
[ ] No

Other information

Fixes #19910, right?

Fixes #19910, right?

yep, sorry forgot to tag it


New updates to pytest 5.4 removed output logging to junit files by de…

…fault, turning it back on.
Removed warnings, new junit 2.0 format is coming so specify we want orginal format.
https://docs.pytest.org/en/latest/reference.html#confval-junit_logging
#6058

old pytest still works


Debug CI failures, introduce task groups, error handling improvements

This is a bit of chaos, but to sum up:

we must not use unbounded retries in the client code operations
I'd like to introduce some more structured task control

Codecov Report
Base: 58.56% // Head: 58.61% // Increases project coverage by +0.04% :tada:

Coverage data is based on head (4e0e1c1) compared to base (8ae1513).
Patch coverage: 66.32% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #834      +/-   ##
==========================================
+ Coverage   58.56%   58.61%   +0.04%
==========================================
  Files         100      100
  Lines       16196    16277      +81
==========================================
+ Hits         9485     9540      +55
- Misses       6711     6737      +26

Impacted Files
Coverage Δ

client/cli/src/main.rs
0.29% <0.00%> (-0.01%)
:arrow_down:

client/clientd/src/main.rs
0.76% <0.00%> (-0.02%)
:arrow_down:

fedimintd/src/main.rs
1.36% <0.00%> (-0.06%)
:arrow_down:

fedimint-api/src/task.rs
67.50% <70.90%> (+4.53%)
:arrow_up:

modules/fedimint-wallet/src/lib.rs
92.61% <83.33%> (-0.09%)
:arrow_down:

client/client-lib/src/wallet/mod.rs
74.45% <91.66%> (+0.91%)
:arrow_up:

client/client-lib/src/lib.rs
85.16% <100.00%> (+0.18%)
:arrow_up:

fedimint-server/src/net/peers.rs
89.73% <0.00%> (-0.30%)
:arrow_down:

modules/fedimint-mint/src/lib.rs
90.12% <0.00%> (-0.18%)
:arrow_down:

... and 2 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

Looks landable to me, tests seem to be flaky again?

The only actual fix this includes is that timeout on that one function, and this fixes an issue I was frequently observing locally. Now when I run a test loop I'm hitting some other hangs, though much less frequently.
If you like it, we can land then keep on improving and keep hunting for other bugs that are still there.

I don't mind either way, up to you :)

nitpick: Rename PR to reflect the core change here being "an introduction of task control system"?

nitpick: Rename PR to reflect the core change here being "an introduction of task control system"?

Done

Squashed and rebased due to merge conflict.


Client identity storage service

Contains
The client to synchronize with a client identity storage service, as discussed here in the forum.
How to test
An implementation of the backend server is available here. You can install and configure it locally as described here, or use the public instance I'm hosting here.
To test, register an account on a server by navigating to it's address with a browser (or directly clicking the link above if you use the hosted instance). Then, start the game client (from this branch) and login by going to Settings > Player Settings. Now you can join a game server and as soon as you do, the new identity will be uploaded to the service; you can then start another client with empty/different configuration (e.g. use the 2nd or 3rd client run configurations in IntelliJ, or specify another data directory from the command line) and login into it too; your client identities will be synchronized and if you join the same game server, you will find your character in the same position you left it from the other client.
Outstanding before merging
Marking as WIP since I think there is still some minor improvement that can be done, e.g. localization of some UI messages. Also, the conflict resolving feature (which allows you which identity you want to keep for a certain server in case the locally stored one conflicts with the one stored on the service) is not fully working but this is due to a server issue which doesn't allow to overwrite the identities (will fix it soon).
Update: I just added the localization feature (the strings are in the menu translations files; the values are filled for English only but can be easily be added for the other languages via editing the proper files or - I think - with Weblate if this branch is merged). Also fixed the server to support replacing of already uploaded identities.
Tagging @msteiger since we talked about this recently on appear.in.

Hooray Jenkins reported success with all tests good!

Hooray Jenkins reported success with all tests good!

Hooray Jenkins reported success with all tests good!

Hooray Jenkins reported success with all tests good!

Thank you for the review! Left some comments and made a commit with various fixes.

Hooray Jenkins reported success with all tests good!

Uh oh, something went wrong with the build. Need to check on that

Hooray Jenkins reported success with all tests good!

Hooray Jenkins reported success with all tests good!

I added one more commit to fix a problem: trying to open the player's settings screen from the in-game pause menu was throwing a NullReferenceException due to the StorageServiceWorker instance (which the player settings screen polls) was being registered in the main menu's context, and thus not accessible from the in-game context.
I apologize if there is something bad in the fact that I added code that nobody asked for after the PR had been approved, possibly making the review/merge process longer, but I think this is still better than letting buggy code to propagate to other branches (develop/master). Unfortunately I didn't noticed this behavior when I tested before opening the PR because I didn't think about opening the settings from the pause menu.
Anyway, now it's fixed: the StorageServiceWorker instance is registered in the rootContext and thus is more decoupled from engine and user interface's states, and the player settings screen can be accessed both from the main menu and from the in-game pause menu.

Tried testing this out, but the server you're hosting seemingly took my registration (again I guess, tried earlier but the site looks much better now) then never sent me a confirmation key. Is it still working? :-)
Also wondering where we should host it. Since our Weblate instance runs out of Docker already maybe we could just add a couple more containers or something? Poking @qwc and @msteiger for ideas
Game itself seemed to run fine and I was able to test it successfully in the past!

@Cervator : it's OK that it allowed you to "register again" after the previous time you registered because I rebuilt the database about a week ago when I added email verification and reCAPTCHA (the only accounts were mine and your tests, so I guessed it wasn't a big deal; anyway, I have a dump). I had a look at the database now and saw you made an account with email address which is waiting for confirmation.
I tried to register another test account with my email and it worked, I got the activation mail - in the spam folder, however. Did you check yours? The reason it goes into spam, I guess, is that it doesn't come from a well-known/trusted mail service, but from a local SMTP server installed on the VPS, so Gmail thinks it's suspicious.
P.S. even if you find the "old" verification email in spam, if you want to retry you probably need to redo the registration (same username and email is fine) because an unverified account expires after 1-2 hours (not sure about the exact value now).

Can we merge this one, @Cervator?

@msteiger certainly - checked it out again and rested, got the confirmation email via junk mail, confirmed, and was able to validate the identity carried over :-) You don't need to wait for me if you have approved and feel you've tested decently, in that case merge ahead and just assign the PR (and any related issues) to the current milestone
Great work @gianluca-nitti !
One small suggestion: Make the UI element a little more consistent between different screens - the little thing in the top right is only visible on the main menu? Maybe it should be there on the settings screen etc too?
We should probably also find some places to update the docs, and a good on-going way to host the service. Will bump the forum thread :-)
Thanks again!

@Cervator Of course, spinning up docker containers is the easiest. That's the fancy thing with docker.
But we should think twice regarding resources usage, the weblate droplet may be to small for things like elastic search, etc.


Sometimes, clicking "back" or "forward" during the load of a large directory will include an item from the previous folder in the new one

Large directories are notorious in this. I currently have a cancellation token that triggers when navigation commands are invoked, but it still includes one random item from the previous directory in the directory being navigated to.

You can fix the problem quickly by moving the cancellation token check closer to the list addition code. If things get crazy you can use a lock for accessing the token but I don't think that will be necessary.
Imo it's a problem with the architecture though. Almost all of the fields are static, but you have a singleton field as well (ItemViewModel.ViewModel). To me both are evil and you should be using plain old instances when possible, i.e. some class should just be holding an instance of ItemViewModel and this instance should be called by everyone instead of using static or singleton.
This is very much a opinion thing though and each programmer has his own tastes developed over years. Just because I say something doesn't make it right (plus I only have 5 years exp and not in C# exclusively). I remember from reddit that you are in high school. You have a popular OSS project people are interested in and are contributing to, use this opportunity to learn good coding practices 👍

@duke7553 Thank you. It really makes me proud to hear all of this feedback. I have a solution which you can expect to come very soon for the "architecture problem". I followed your advice above so Files won't create an entirely new ItemViewModel every single time.

@duke7553 I created a new PR containing the changes. It's a small step in the right direction, but it still counts.

@duke7553 Is this issue solved? I couldn't reproduce in the latest master -   ##256a881ef1df035bdcd6898931f759c3125e262e.

This should be fixed now


fix cannot call clip_grads() at multi_gpu

When multi gpu FunctionSet is set in optimizer, I fail in optimzer#clip_grads()
from chainer import FunctionSet, cuda
from chainer.optimizers import SGD
import chainer.functions as F

cuda.init(0)
model = FunctionSet(l1=F.Linear(4, 4).to_gpu(0), l2=F.Linear(4, 4).to_gpu(1))
optimizer = SGD()
optimizer.setup(model.collect_parameters())
optimizer.clip_grads(5)
#=>raise LogicError: cuMemcpyDtoH failed: an illegal memory access was encountered

Thank you for your PR.
I recommend to fix an additional also here in this PR.
https://github.com/kikusu/chainer/blob/fix/optimizer_clip_grads_multi_gpu/chainer/optimizer.py#L158

Sorry, i miss PR. I fixed.

LGTM!
Thank you.


Building latest version (based on chromium-89.0.4389.90.tar.xz) fails on Arch

Building project on Archlinux fails with following error:
"In file included from ../../media/gpu/vaapi/vaapi_utils.cc:18:
../../third_party/libva_protected_content/va_protected_content.h:136:14: error: redefinition of '_VA_TEE_EXEC_FUNCTION_ID'
typedef enum _VA_TEE_EXEC_FUNCTION_ID {
^
/usr/include/va/va_prot.h:405:14: note: previous definition is here
typedef enum _VA_TEE_EXEC_FUNCTION_ID {
^
In file included from ../../media/gpu/vaapi/vaapi_utils.cc:18:
../../third_party/libva_protected_content/va_protected_content.h:163:16: error: redefinition of '_VAEncryptionSegmentInfo'
typedef struct _VAEncryptionSegmentInfo {
^
/usr/include/va/va.h:2044:16: note: previous definition is here
typedef struct _VAEncryptionSegmentInfo {
^
In file included from ../../media/gpu/vaapi/vaapi_utils.cc:18:
../../third_party/libva_protected_content/va_protected_content.h:183:16: error: redefinition of '_VAEncryptionParameters'
typedef struct _VAEncryptionParameters {
^
/usr/include/va/va.h:2064:16: note: previous definition is here
typedef struct _VAEncryptionParameters {
^
3 errors generated.
[24786/44930] CXX obj/components/policy/core/common/internal/cloud_policy_client.o
ninja: build stopped: subcommand failed.
==> ERROR: A failure occurred in build().
Aborting..."
After a brief investigation I found that chromium-89.0.4389.90 defines _VA_TEE_EXEC_FUNCTION_ID,  _VAEncryptionSegmentInfo and VAEncryptionParameters in a file "chromium-89.0.4389.90/third_party/libva_protected_content/va_protected_content.h". The very same symbols are defined in libva library (which is listed as a dependency) header file "va_prot.h".
Not sure what can be done on a ungoogled-chromium side; though, it seems that this issue was fixed in the most recent version of chromium as its "va_protected_content.h" no longer contains these conflicting  redefinitions.

You can just get the patch file from the commit that fixed it and apply it in the PKGBUILD: https://github.com/chromium/chromium/commit/e0b362edd9b49143b89fc76c4a31dd5603b6fbd0.patch
This is due to the latest libva update.

This has been addressed in the repository for Arch packaging.

Closing as resolved


[AMBARI-24556] Aggregation across clusters being done in AMS when multiple cluster support is enabled.

Duplicate of https://github.com/apache/ambari/pull/2313 for trunk.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/3948/
Test PASSed.


don't use echo -n in shell scripts

echo -n isn't portable.  some shells will write out a literal -n or just ignore it.
instead you can do things like:
-  echo -n "$1"
+  printf '%s' "$1"

printf also has %b in case you want escape sequences to be processed safely:
  printf '%b' '\nfoo\n\tbar %s\n'

Probably needs to be fixed sooner rather than later, if this ever gets reverted.

yes my intention is to revert that CL at some point.  but it's a bit of yak shaving i don't have time for atm :).


visual studio comments

I was also wondering if the comments (in the classes) for each variable or method can be added so it appears in visual studio when selecting a variable or method?
Currently there is no comment in visual studio.
For example in Set class (see text surrounded by **):
    /// <summary>
    /// **Gets the block the set is in.**
    /// </summary>
    public string Block { get; init; }

    /// <summary>
    /// **Gets the booster contents for this set.**
    /// </summary>
    public List<object> Booster { get; init; }

    /// <summary>
    /// **Gets the type of border on the cards, either “white”, “black” or “silver”.**
    /// </summary>
    public string Border { get; init; }

I think it's called Intellisense in visual studio. So the automatic selecting of variables and methods and the comments you see on the screen when selecting something.
See screenshot

the comments should be there so you might have an issue with VS

The comments appear in all .NET classes and in all 3rd party libraries I tried, except your library :(
So what can I do to make it work?

Will look into this, it might be something to do with generating the nugget package, when I have a direct reference to the library I see all the comments

Thx!
After testing on another PC, I think the problem is not in my visual studio itself.

Maybe it's possible to add the 'type' variable at the same time when making a new nuget package (see other issue).
I would love to use this.

On google I found a possible solution for this issue. Note I never made any nuget package myself, I just show you this link because I believe this is the solution.
https://stackoverflow.com/questions/40110687/intellisense-and-triple-slash-comments-in-packages

will be in next nuget release, closing issue


Retrieving structural force

Hi there, I would start off by saying this is an amazing library which is going to spend a lot of engineering time for my co-workers 😁.
I encountered an issue when trying to do some post-processing, on the original APDL code we used PRRSOL, FY which prints out the node and the force in the Y direction and some other details.
When calling it in python I get a string object, which makes completely sense, but I am only interested in the node number and force.
I can technically parse the string and extract the values that way but I wondered if there's a more elegant way of proceeding with this.
I tried retrieving the nodal forces from the rst file but there's a bit of lack in the documentation on what is the structure of the force list it's returns a bit confusing.
I would love to have some light spread on this subject.
Thanks very much!

Hello,
Thanks for your interest in pyansys, and I'm glad that it's been so helpful for you.  Could you please post a code snippet and/or upload a script so I can take a look at this today.

Yes, you should be able to grab structural force from both an active MAPDL session or the result file.  I've been meaning to add a simple beam example to the docs and I'll add one soonish.

Pinging @IdoLaster for the example script.

Closing stale issue


Parser: Inline HTML combined with code block (if no blank  line)

When using the parser, if a code block immediately follows inline HTML, they are both combined into a single HTML block.  See end for info on the CommonMark spec.  The CommonMark online editor handles both cases below fine, parsing things as two separate blocks.  Markdig for example, if markdown is:

    ```html
      <strong>Foo</strong>
    ```

The above is returned by the parser's Parse method as one whole HtmlBlock.  The way to fix this is to separate the inline HTML by blank line(s):

    ```html
      <strong>Foo</strong>
    ```

The above is returned as two separate blocks by the Markdig parser.
Is there a way for the Markdig parser to return two separate blocks regardless of format?
The CommonMark spec indicates:
A fenced code block may interrupt a paragraph, and does not require a blank line either before or after.

Is there a way for the Markdig parser to return two separate blocks regardless of format?
The CommonMark spec indicates:
A fenced code block may interrupt a paragraph, and does not require a blank line either before or after.

Nope, but it's indeed a corner case. I believe most implementers took the HTML rule as superseding the fenced block rule.
Almost CommonMark implementations are following the same behavior than markdig. Seems that GitHub flavored choose a different path on that particular case, which is unfortunate...
So we can't really change that, unless you are willing to make a PR with an extension to add an option to turn that off, but I'm not sure how easy it is to fit that in...


x-compiled 1.2.0 linux arm fastrand panic

Crosscompiled for linux arm runing on rasbpi B model arm v6 cpu.
pi@ronny ~/linux_arm $ ./siad version ; ./siad
Sia Daemon v1.2.0
Loading...
(0/6) Loading siad...
(1/6) Loading gateway...
(2/6) Loading consensus...
(3/6) Loading transaction pool...
(4/6) Loading wallet...
(5/6) Loading host...
(6/6) Loading renter...
panic: fastrand: argument to Intn is <= 0
goroutine 46 [running]:
github.com/NebulousLabs/fastrand.Intn(0xef798000, 0x10c39a0c)
/home/bjdo/go/src/github.com/NebulousLabs/fastrand/fastrand.go:127 +0x174
github.com/NebulousLabs/Sia/modules/renter/hostdb.(*HostDB).threadedScan(0x10cb4b60)
/home/bjdo/go/src/github.com/NebulousLabs/Sia/modules/renter/hostdb/scan.go:335 +0x53c
created by github.com/NebulousLabs/Sia/modules/renter/hostdb.newHostDB
/home/bjdo/go/src/github.com/NebulousLabs/Sia/modules/renter/hostdb/hostdb.go:180 +0x630

since you are able to compile+run, can you insert a statement that shows what the argument to Intn is? Curious if it's 0 or 1 or something more suspicious (like an overflow).
Also, can you paste the syntax you use to do the cross compilation?

we should probably have the panic report the received value

Looks to me like an overflow. The time.Duration values being used overflow a 32bit integer, so the fastrand library overflowed and got a negative number.

yup, that seems accurate to me. So we either use fastrand.BigIntn, or we add a fastrand.Uint64n function?

I registered this one on request from taek. Since sia requires 64bit to allow for mmap of large data, limiting the lib to require 64bit arithmetrrics is no limit to sia until the mmap is changed. Just saying.
It was compiled by setting env vars GOOS=linux GOARCH=arm then calling your makefile.

Ah, that is a good point. Officially we only support 64-bit. Still, it's best to keep the code as flexible as possible in the event that we do support 32-bit someday.

I write it again regarding emulation since this was an edit. Dont the compiler emulate 64bit if needed.

Like c compilers emulate float on non float cpus ( long time ago)

Go can perform 64-bit arithmetic on 32-bit machines, yes. However, in this case we are explicitly casting a 64-bit type to an int, and the size of int depends on the architecture. And of course mmap is limited by the 32-bit address space.

So if this fastrand lib explicitly use 64ints allover there is no problem in the libary. It will always work. (but the calling code of this lib need to be careful about the downcast which is likely needed )

We've fixed the problem.


Make proclaim.deepEqual spec-compliant

It wasn't adhering to point 7.3 of the CommonJS unit testing spec http://wiki.commonjs.org/wiki/Unit_Testing/1.0#Assert

LGTM 👍


rdkit 2022.03.5 not able to be imported (MacOS, osx-arm64, conda-forge)

Describe the bug
I am unable to import rdkit with version 2022.03.5 installed. If I downgrade to 2022.03.4, it works as expected
To Reproduce
(base) ➜  ~ conda list | grep rdkit
rdkit                     2022.03.5        py39h042da20_0    conda-forge

(base) ➜  ~ python
Python 3.9.13 | packaged by conda-forge | (main, May 27 2022, 17:00:33)
[Clang 13.0.1 ] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import rdkit
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ModuleNotFoundError: No module named 'rdkit'
>>>

(base) ➜  ~ conda install rdkit=2022.03.4 --force-reinstall
[output deleted for brevity]

(base) ➜  ~ conda list | grep rdkit
rdkit                     2022.03.4        py39he8580ed_0    conda-forge

(base) ➜  ~ python
Python 3.9.13 | packaged by conda-forge | (main, May 27 2022, 17:00:33)
[Clang 13.0.1 ] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import rdkit
>>> rdkit.rdBase.rdkitVersion
'2022.03.4'
>>>

Expected behavior
rdkit able to be imported
Configuration (please complete the following information):

RDKit version: 2022.03.5
OS: macOS 12.6, osx-arm64
Python version (if relevant): 3.9.13
Are you using conda? Yes, version 22.9.0
If you are using conda, which channel did you install the rdkit from? conda-forge
If you are not using conda: how did you install the RDKit? n/a

Posted to the wrong repo, I'm assuming. Sorry! I'll close this here and reopen it in conda-rdkit.

Okay, it looks like conda-rdkit hasn't been touched in a while, so I'm reopening here. Let me know if it should go somewhere else.

I can't reproduce this on the mac that I use for testing, so it's not a general problem.
I'd try reinstalling 2022.03.5 in that environment... if that doesn't work then please try creating a new conda environment to see if it works for you there; if so then it's a sign that something has gotten borked in conda environment you're using.

Out of curiosity, I created a new conda environment with only rdkit. It installed python 3.10, and everything worked fine.
I created a brand new python 3.9 environment using conda create -c conda-forge -n rdkit_39 python=3.9 rdkit. After activating the environment, I get the same ModuleNotFound error.
I updated my existing environment from python 3.9 to 3.10, which switched the rdkit build from 2022.03.5-py39h042da20_0 to 2022.03.5-py310h936bf1d_0. That seems to have fixed the issue.

Ah ha. This one I can reproduce. Thanks for that clarification


Cannot build docker image.

Alr hey guys. Im new to coding and idk how to do a lot of these things lol. Pls bear w me. Btw thanks Eric for making this its much appreciated. Oh sorry my english is bad too. When building the image it goes to step 9/11 and then says this:
pi@raspberrypi:~/inventory-hunter $ docker build -t inventory-hunter .
Sending build context to Docker daemon  125.4kB
Step 1/11 : FROM python:3.9
---> 1e73204649e2
Step 2/11 : RUN apt update
---> Using cache
---> 059ec8ea530b
Step 3/11 : RUN apt install -y chromium chromium-driver locales locales-all
---> Using cache
---> c26fc144649c
Step 4/11 : ENV LC_ALL en_US.UTF-8
---> Using cache
---> 836462816cda
Step 5/11 : ENV LANG en_US.UTF-8
---> Using cache
---> 3c4e49abe3c4
Step 6/11 : ENV LANGUAGE en_US.UTF-8
---> Using cache
---> 1a77b4929c62
Step 7/11 : WORKDIR /
---> Using cache
---> 85ce7862d489
Step 8/11 : COPY requirements.txt /src/requirements.txt
---> Using cache
---> 407463daf530
Step 9/11 : RUN pip install -r /src/requirements.txt
---> Running in e11994b8381a
Collecting beautifulsoup4
Downloading beautifulsoup4-4.9.3-py3-none-any.whl (115 kB)
Collecting lxml
Downloading lxml-4.6.2.tar.gz (3.2 MB)
Collecting pyyaml
Downloading PyYAML-5.3.1.tar.gz (269 kB)
Collecting requests
Downloading requests-2.25.0-py2.py3-none-any.whl (61 kB)
Collecting selenium
Downloading selenium-3.141.0-py2.py3-none-any.whl (904 kB)
Collecting soupsieve>1.2; python_version >= "3.0"
Downloading soupsieve-2.0.1-py3-none-any.whl (32 kB)
Collecting chardet<4,>=3.0.2
Downloading chardet-3.0.4-py2.py3-none-any.whl (133 kB)
Collecting urllib3<1.27,>=1.21.1
Downloading urllib3-1.26.2-py2.py3-none-any.whl (136 kB)
Collecting certifi>=2017.4.17
Downloading certifi-2020.11.8-py2.py3-none-any.whl (155 kB)
Collecting idna<3,>=2.5
Downloading idna-2.10-py2.py3-none-any.whl (58 kB)
Building wheels for collected packages: lxml, pyyaml
Building wheel for lxml (setup.py): started
Building wheel for lxml (setup.py): still running...
Building wheel for lxml (setup.py): finished with status 'error'
ERROR: Command errored out with exit status 1:
command: /usr/local/bin/python -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-mh277zjg/lxml/setup.py'"'"'; file='"'"'/tmp/pip-install-mh277zjg/lxml/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(file);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, file, '"'"'exec'"'"'))' bdist_wheel -d /tmp/pip-wheel-tbihpvsi
cwd: /tmp/pip-install-mh277zjg/lxml/
Complete output (84 lines):
Building lxml version 4.6.2.
Building without Cython.
Building against libxml2 2.9.4 and libxslt 1.1.32
running bdist_wheel
running build
running build_py
creating build
creating build/lib.linux-armv7l-3.9
creating build/lib.linux-armv7l-3.9/lxml
copying src/lxml/ElementInclude.py -> build/lib.linux-armv7l-3.9/lxml
copying src/lxml/cssselect.py -> build/lib.linux-armv7l-3.9/lxml
copying src/lxml/usedoctest.py -> build/lib.linux-armv7l-3.9/lxml
copying src/lxml/init.py -> build/lib.linux-armv7l-3.9/lxml
copying src/lxml/builder.py -> build/lib.linux-armv7l-3.9/lxml
copying src/lxml/doctestcompare.py -> build/lib.linux-armv7l-3.9/lxml
copying src/lxml/_elementpath.py -> build/lib.linux-armv7l-3.9/lxml
copying src/lxml/pyclasslookup.py -> build/lib.linux-armv7l-3.9/lxml
copying src/lxml/sax.py -> build/lib.linux-armv7l-3.9/lxml
creating build/lib.linux-armv7l-3.9/lxml/includes
copying src/lxml/includes/init.py -> build/lib.linux-armv7l-3.9/lxml/includes
creating build/lib.linux-armv7l-3.9/lxml/html
copying src/lxml/html/usedoctest.py -> build/lib.linux-armv7l-3.9/lxml/html
copying src/lxml/html/init.py -> build/lib.linux-armv7l-3.9/lxml/html
copying src/lxml/html/formfill.py -> build/lib.linux-armv7l-3.9/lxml/html
copying src/lxml/html/soupparser.py -> build/lib.linux-armv7l-3.9/lxml/html
copying src/lxml/html/builder.py -> build/lib.linux-armv7l-3.9/lxml/html
copying src/lxml/html/_diffcommand.py -> build/lib.linux-armv7l-3.9/lxml/html
copying src/lxml/html/_html5builder.py -> build/lib.linux-armv7l-3.9/lxml/html
copying src/lxml/html/ElementSoup.py -> build/lib.linux-armv7l-3.9/lxml/html
copying src/lxml/html/clean.py -> build/lib.linux-armv7l-3.9/lxml/html
copying src/lxml/html/_setmixin.py -> build/lib.linux-armv7l-3.9/lxml/html
copying src/lxml/html/diff.py -> build/lib.linux-armv7l-3.9/lxml/html
copying src/lxml/html/html5parser.py -> build/lib.linux-armv7l-3.9/lxml/html
copying src/lxml/html/defs.py -> build/lib.linux-armv7l-3.9/lxml/html
creating build/lib.linux-armv7l-3.9/lxml/isoschematron
copying src/lxml/isoschematron/init.py -> build/lib.linux-armv7l-3.9/lxml/isoschematron
copying src/lxml/etree.h -> build/lib.linux-armv7l-3.9/lxml
copying src/lxml/etree_api.h -> build/lib.linux-armv7l-3.9/lxml
copying src/lxml/lxml.etree.h -> build/lib.linux-armv7l-3.9/lxml
copying src/lxml/lxml.etree_api.h -> build/lib.linux-armv7l-3.9/lxml
copying src/lxml/includes/relaxng.pxd -> build/lib.linux-armv7l-3.9/lxml/includes
copying src/lxml/includes/etreepublic.pxd -> build/lib.linux-armv7l-3.9/lxml/includes
copying src/lxml/includes/xmlerror.pxd -> build/lib.linux-armv7l-3.9/lxml/includes
copying src/lxml/includes/tree.pxd -> build/lib.linux-armv7l-3.9/lxml/includes
copying src/lxml/includes/xmlparser.pxd -> build/lib.linux-armv7l-3.9/lxml/includes
copying src/lxml/includes/xinclude.pxd -> build/lib.linux-armv7l-3.9/lxml/includes
copying src/lxml/includes/config.pxd -> build/lib.linux-armv7l-3.9/lxml/includes
copying src/lxml/includes/htmlparser.pxd -> build/lib.linux-armv7l-3.9/lxml/includes
copying src/lxml/includes/xslt.pxd -> build/lib.linux-armv7l-3.9/lxml/includes
copying src/lxml/includes/c14n.pxd -> build/lib.linux-armv7l-3.9/lxml/includes
copying src/lxml/includes/uri.pxd -> build/lib.linux-armv7l-3.9/lxml/includes
copying src/lxml/includes/schematron.pxd -> build/lib.linux-armv7l-3.9/lxml/includes
copying src/lxml/includes/dtdvalid.pxd -> build/lib.linux-armv7l-3.9/lxml/includes
copying src/lxml/includes/xmlschema.pxd -> build/lib.linux-armv7l-3.9/lxml/includes
copying src/lxml/includes/init.pxd -> build/lib.linux-armv7l-3.9/lxml/includes
copying src/lxml/includes/xpath.pxd -> build/lib.linux-armv7l-3.9/lxml/includes
copying src/lxml/includes/etree_defs.h -> build/lib.linux-armv7l-3.9/lxml/includes
copying src/lxml/includes/lxml-version.h -> build/lib.linux-armv7l-3.9/lxml/includes
creating build/lib.linux-armv7l-3.9/lxml/isoschematron/resources
creating build/lib.linux-armv7l-3.9/lxml/isoschematron/resources/rng
copying src/lxml/isoschematron/resources/rng/iso-schematron.rng -> build/lib.linux-armv7l-3.9/lxml/isoschematron/resources/rng
creating build/lib.linux-armv7l-3.9/lxml/isoschematron/resources/xsl
copying src/lxml/isoschematron/resources/xsl/XSD2Schtrn.xsl -> build/lib.linux-armv7l-3.9/lxml/isoschematron/resources/xsl
copying src/lxml/isoschematron/resources/xsl/RNG2Schtrn.xsl -> build/lib.linux-armv7l-3.9/lxml/isoschematron/resources/xsl
creating build/lib.linux-armv7l-3.9/lxml/isoschematron/resources/xsl/iso-schematron-xslt1
copying src/lxml/isoschematron/resources/xsl/iso-schematron-xslt1/iso_dsdl_include.xsl -> build/lib.linux-armv7l-3.9/lxml/isoschematron/resources/xsl/iso-schematron-xslt1
copying src/lxml/isoschematron/resources/xsl/iso-schematron-xslt1/iso_schematron_skeleton_for_xslt1.xsl -> build/lib.linux-armv7l-3.9/lxml/isoschematron/resources/xsl/iso-schematron-xslt1
copying src/lxml/isoschematron/resources/xsl/iso-schematron-xslt1/iso_svrl_for_xslt1.xsl -> build/lib.linux-armv7l-3.9/lxml/isoschematron/resources/xsl/iso-schematron-xslt1
copying src/lxml/isoschematron/resources/xsl/iso-schematron-xslt1/iso_abstract_expand.xsl -> build/lib.linux-armv7l-3.9/lxml/isoschematron/resources/xsl/iso-schematron-xslt1
copying src/lxml/isoschematron/resources/xsl/iso-schematron-xslt1/iso_schematron_message.xsl -> build/lib.linux-armv7l-3.9/lxml/isoschematron/resources/xsl/iso-schematron-xslt1
copying src/lxml/isoschematron/resources/xsl/iso-schematron-xslt1/readme.txt -> build/lib.linux-armv7l-3.9/lxml/isoschematron/resources/xsl/iso-schematron-xslt1
running build_ext
building 'lxml.etree' extension
creating build/temp.linux-armv7l-3.9
creating build/temp.linux-armv7l-3.9/src
creating build/temp.linux-armv7l-3.9/src/lxml
gcc -pthread -Wno-unused-result -Wsign-compare -DNDEBUG -g -fwrapv -O3 -Wall -fPIC -DCYTHON_CLINE_IN_TRACEBACK=0 -I/usr/include/libxml2 -Isrc -Isrc/lxml/includes -I/usr/local/include/python3.9 -c src/lxml/etree.c -o build/temp.linux-armv7l-3.9/src/lxml/etree.o -w
gcc: fatal error: Killed signal terminated program as
compilation terminated.
Compile failed: command '/usr/bin/gcc' failed with exit code 1
creating tmp
cc -I/usr/include/libxml2 -I/usr/include/libxml2 -c /tmp/xmlXPathInit_he3bsco.c -o tmp/xmlXPathInit_he3bsco.o
cc tmp/xmlXPathInit_he3bsco.o -lxml2 -o a.out
error: command '/usr/bin/gcc' failed with exit code 1
ERROR: Failed building wheel for lxml
Running setup.py clean for lxml
Building wheel for pyyaml (setup.py): started
Building wheel for pyyaml (setup.py): still running...
Building wheel for pyyaml (setup.py): finished with status 'done'
Created wheel for pyyaml: filename=PyYAML-5.3.1-cp39-cp39-linux_armv7l.whl size=514839 sha256=c804dc7fbd8263026447af4b8be22e6e6d3c3db5831efcc487b996c5a42880ad
Stored in directory: /root/.cache/pip/wheels/69/60/81/5cd74b8ee068fbe9e04ca0d53148f28f5c6e2c5b177d5dd622
Successfully built pyyaml
Failed to build lxml
Installing collected packages: soupsieve, beautifulsoup4, lxml, pyyaml, chardet, urllib3, certifi, idna, requests, selenium
Running setup.py install for lxml: started
I did this 3 times and it says no docker image file found when doing step 3
Docker image list:
docker image list
REPOSITORY          TAG                 IMAGE ID            CREATED             SIZE
               407463daf530        4 hours ago         1.3GB
python              3.9                 1e73204649e2        11 days ago         694MB
hello-world         latest              851163c78e4a        11 months ago       4.85kB
pls help me to find out what is going on or what im missing.
oh btw docker is installed correctly i checked already.
thanks guys

Someone else had a very similar issue where lxml fails to build. Are you using an older Raspberry Pi? It's possible that there isn't enough memory available to build this module.

thanks for replyin. im using a rasberry pi 3 model b the same one ur using. I read that issue lol. it has 1 gb ram and i checked it to make sure.

I also got a 32 gb micro sd card that im using. So ik its not storage related problem.

i was also getting a build fail on my raspberry 4 8gb, but i was able to build it on my x5670 ubuntu server (console only) box so i've been using that to run my scrapers instead.
that being said, do you know if the chromium builds your testing now will work on a console only system?

I'm getting the same error trying to run the build on my RPie 3 B

It seems that the build step for lxml is brittle on some Raspberry Pi's. The build consumes almost all the available memory on my Raspberry Pi 3, but I have never seen it crash.
I'm considering making the lxml install optional (ie: if it builds, awesome.. if not, we can fall back to python's built-in parser). Thoughts?

Yes I think this is a wonderful idea. (Idk much about this coding stuff FYI) and another thought would be to make other files like the lxml optional if u can do that j incase if it crashes again. I’d hate for you to do all that work and it won’t work for some people. But again that’s just a thought and I think ur idea u presented is amazing. J pls make it easy to follow because for people like me who are below a noob level, it can be easy to follow like you have been doing with your posts now. Thanks man appreciate your work

Hey Eric, since you Said that the lxml uses a lot of ram I closed the chrome tab I had in the background and it worked for me! So thanks for the suggestion. Anyway, now I will keep this updated if it works on this thread.

Just made lxml optional: https://github.com/EricJMarti/inventory-hunter/issues/32#issuecomment-736225183


ibmcloud rate limit support - feature request

Hi there,
We have a bash script to run the 'ibmcloud' command several times in order to set up our IBM Cloud stuff.
Sometimes we are getting an error from the API server (status code 429) related to the rate limits.
FAILED
Error response from server. Status code: 429; description: The request frequency from this IP exceeded the allowable rate of 30 per 60000 ms with a minimum separation of 0 ms. Please check the 'X-RateLimit-Limit', and 'X-RateLimit-Reset' header values returned by the server..
Is it planned to add support for this in the upcoming releases of the ibmcloud cli? In this way, the ibmcloud manage the 429 status code, wait a while and re-attempt again X times.
It would be very interesting to have this support natively in the command line.

thanks for this feature request. Will move this to dev backlog and let you know once it is planned.

@jmgalvez would you mind if you could tell us the URL path in which you met this error? Thanks.

@archerzz We always obtain this 429 when calling CF related commands, not detected yet with IAM related commands. Mainly, when we try 'ibmcloud account org-role-set' and 'ibmcloud account space-role-set' commands. Likely with the 'unset' equivalent commands as well.
For example, yesterday, first we had got a 429 and when we tried again we got a 500.
FAILED
Error response from server. Status code: 500; description: Error while querying /v2/organizations?q=name%3Amy-beautiful-org&region=eu-de for region eu-de: "failed to GET https://api.eu-de.bluemix.net/v2/organizations?q=name%3Adev-bp-app-xsp-org&region=eu-de: Error: Parse Error".


Add all flickr.reflection.getMethodInfo responses

We'll use these to generate the full client.

Seems reasonable :+1:

With the power vested in me by Github, :+1:


Customizable Redis Client

The package.json file states redis in dependencies, but then uses ioredis in devDependencies?
Redis client dependencies should be dropped and the Redis client should load into the library via code.

For example, this was done on this repo...connect-redis
https://github.com/tj/connect-redis/blob/master/lib/connect-redis.js

We gotta roll this out ASAP! I'll try and submit a PR as soon as possible, this is urgent!

Totally agree with that. The following signature would indeed be better:
io.adapter(redisAdapter(pubClient, subClient, options));

My only concern is about the existing code examples on the Internet which display the current form... Do you think this breaking change is worth it?

For future readers, this was done in release 7.0.0:
const io = require('socket.io')(3000);
const { createClient } = require('redis');
const redisAdapter = require('@socket.io/redis-adapter');

const pubClient = createClient({ host: 'localhost', port: 6379 });
const subClient = pubClient.duplicate();
io.adapter(redisAdapter(pubClient, subClient));


race condition in async archive-push?

I'm seeing errors on a regular basis on a high-throughput database - seems to happen when two WAL segments are pushed in short succession. The segments are eventually pushed to the remote backrest server, so there's no data loss.
pg_backrest.conf on database side:
[global:backup]
backup-host=prd-ba-01
backup-user=backrest

[global:general]
repo-path=/data/postgresql/wal-spool
repo-remote-path=/data/postgresql/backups

[global:log]
log-level-console=info

[global:archive]
archive-async=y
archive-max-mb=4096

[prd-psql3-01]
db-path=/data/postgresql/9.3/main

pg_backrest log:
-------------------PROCESS START-------------------
2015-05-29 16:02:10.041 T00   INFO: archive to be copied to backup total 1, size 16MB
2015-05-29 16:02:11.980 T00   INFO: archive to be copied to backup total 2, size 32MB
2015-05-29 16:02:11.980 T00  ERROR: [116]: unable to open /data/postgresql/wal-spool/archive/prd-psql3-01/out/0000000100001ED3000000AD-753252ed2a19f94446a6b581ab3622b485e6a71c
 at /opt/backrest/bin/../lib/BackRest/Archive.pm line 173.
                                        BackRest::Archive::walInfo('BackRest::Archive=HASH(0x22fa9a8)', '/data/postgresql/wal-spool/archive/prd-psql3-01...') called at /opt/backrest/bin/../lib/BackRest/Archive.pm line 672
                                        BackRest::Archive::xfer('BackRest::Archive=HASH(0x22fa9a8)', '/data/postgresql/wal-spool/archive/prd-psql3-01...', '/data/postgresql/wal-spool/lock/prd-psql3-01...') called at /opt/backrest
/bin/../lib/BackRest/Archive.pm line 409
                                        BackRest::Archive::pushProcess('BackRest::Archive=HASH(0x22fa9a8)') called at /opt/backrest/bin/../lib/BackRest/Archive.pm line 66
                                        BackRest::Archive::process('BackRest::Archive=HASH(0x22fa9a8)') called at /opt/backrest/bin/pg_backrest.pl line 141
                                        eval {...} called at /opt/backrest/bin/pg_backrest.pl line 125

                                   2015-05-29 16:02:12.063 T00   INFO: archive to be copied to backup total 1, size 16MB

[..]

-------------------PROCESS START-------------------
2015-05-29 17:00:55.047 T00   INFO: archive to be copied to backup total 1, size 16MB
2015-05-29 17:00:57.029 T00   INFO: archive to be copied to backup total 2, size 32MB
2015-05-29 17:00:57.029 T00  ERROR: [116]: unable to open /data/postgresql/wal-spool/archive/prd-psql3-01/out/0000000100001ED400000035-6693e84ab158dd65387d352efd58e145327f2c6f
 at /opt/backrest/bin/../lib/BackRest/Archive.pm line 173.
                                        BackRest::Archive::walInfo('BackRest::Archive=HASH(0x7ca678)', '/data/postgresql/wal-spool/archive/prd-psql3-01...') called at /opt/backrest/bin/../lib/BackRest/Archive.pm line 672
                                        BackRest::Archive::xfer('BackRest::Archive=HASH(0x7ca678)', '/data/postgresql/wal-spool/archive/prd-psql3-01...', '/data/postgresql/wal-spool/lock/prd-psql3-01...') called at /opt/backrest/bin/../lib/BackRest/Archive.pm line 409
                                        BackRest::Archive::pushProcess('BackRest::Archive=HASH(0x7ca678)') called at /opt/backrest/bin/../lib/BackRest/Archive.pm line 66
                                        BackRest::Archive::process('BackRest::Archive=HASH(0x7ca678)') called at /opt/backrest/bin/pg_backrest.pl line 141
                                        eval {...} called at /opt/backrest/bin/pg_backrest.pl line 125

                                   2015-05-29 17:00:57.288 T00   INFO: archive to be copied to backup total 1, size 16MB

Relevant output from postgres log for the second occurence:
2015-05-29 17:00:54.580 T00   INFO: pushing WAL segment pg_xlog/0000000100001ED400000035 asynchronously
2015-05-29 17:00:54.731 T00   INFO: starting async archive-push
2015-05-29 17:00:56.566 T00   INFO: pushing WAL segment pg_xlog/0000000100001ED400000036 asynchronously
2015-05-29 17:00:56.715 T00   INFO: starting async archive-push
2015-05-29 17:00:55.047 T00   INFO: archive to be copied to backup total 1, size 16MB
2015-05-29 17:00:57.029 T00   INFO: archive to be copied to backup total 2, size 32MB
2015-05-29 17:00:57.029 T00  ERROR: [116]: unable to open /data/postgresql/wal-spool/archive/prd-psql3-01/out/0000000100001ED400000035-6693e84ab158dd65387d352efd58e145327f2c6f
2015-05-29 17:00:57.288 T00   INFO: archive to be copied to backup total 1, size 16MB
2015-05-29 17:01:01.086 T00   INFO: pushing WAL segment pg_xlog/0000000100001ED400000037 asynchronously
2015-05-29 17:01:01.244 T00   INFO: starting async archive-push
2015-05-29 17:01:01.559 T00   INFO: archive to be copied to backup total 1, size 16MB

Hmmm - it looks like more than one async process is getting spawned.

BTW - which commit are you running?

From a different server, identical configuration. Different error, maybe related?
Currently running 6d2f2d3f91227dffe635ab14de3db3d83c09e0c6
-------------------PROCESS START-------------------
2015-05-29 16:30:04.605 T00   INFO: archive to be copied to backup total 1, size 16MB
2015-05-29 16:30:05.236 T00   INFO: archive to be copied to backup total 3, size 48MB
2015-05-29 16:30:04.874 T00   INFO: archive to be copied to backup total 2, size 32MB
2015-05-29 16:30:06.948 T00   INFO: archive to be copied to backup total 1, size 16MB
2015-05-29 16:30:07.755 T00 ASSERT: no lock is currently held
 at /opt/backrest/bin/../lib/BackRest/Lock.pm line 107.
                                        BackRest::Lock::lockRelease() called at /opt/backrest/bin/../lib/BackRest/Archive.pm line 421
                                        BackRest::Archive::pushProcess('BackRest::Archive=HASH(0x1d43768)') called at /opt/backrest/bin/../lib/BackRest/Archive.pm line 66
                                        BackRest::Archive::process('BackRest::Archive=HASH(0x1d43768)') called at /opt/backrest/bin/pg_backrest.pl line 141
                                        eval {...} called at /opt/backrest/bin/pg_backrest.pl line 125

                                   2015-05-29 16:30:07.756 T00  ERROR: process terminated on signal or exception, 0 threads stopped
 at /opt/backrest/bin/pg_backrest.pl line 115.
                                        main::safe_exit() called at /opt/backrest/bin/pg_backrest.pl line 302

If it helps with debugging I can keep it running in trace mode on one node?

Just debug would be fine.  Trace is mostly protocol layer logging and I don't think that's the issue here.

OK - I see it.  I'll have a fix soon.

fwiw - here's a debug snippet:
-------------------PROCESS START-------------------
2015-05-29 18:01:13.477 T00  DEBUG:     File->manifest: db:absolute:/data/postgresql/wal-spool/archive/prd-psql3-01/out
2015-05-29 18:01:12.095 T00   INFO: archive to be copied to backup total 1, size 16MB
2015-05-29 18:01:12.095 T00  DEBUG:     archive<PHONE_NUMBER>001ED4000000BA-e39094dfc6246cc32eab0db38f0dc2e077252cd8, is WAL 1, source_compressed = 0, destination_compress 1,
 default_compress = 1
2015-05-29 18:01:12.096 T00  DEBUG:     Archive->pushCheck: backup:archive:0000000100001ED4000000BA
2015-05-29 18:01:12.099 T00  DEBUG:     File->copy:  local db:absolute:/data/postgresql/wal-spool/archive/prd-psql3-01/out/0000000100001ED4000000BA-e39094d
fc6246cc32eab0db38f0dc2e077252cd8 to remote backup:archive:/data/postgresql/backups/archive/prd-psql3-01/0000000100001ED4/0000000100001ED4000000BA-e39094df
c6246cc32eab0db38f0dc2e077252cd8.gz, source_compressed = false, destination_compress = true, ignore_missing_source = false, destination_path_create = true, modification_ti
me = [undef], mode = [undef], user = [undef], group = [undef]
2015-05-29 18:01:13.781 T00  DEBUG:     1 WAL segments were transferred, calling Archive->xfer() again
2015-05-29 18:01:13.782 T00  DEBUG:     File->manifest: db:absolute:/data/postgresql/wal-spool/archive/prd-psql3-01/out
2015-05-29 18:01:13.797 T00   INFO: archive to be copied to backup total 2, size 32MB
2015-05-29 18:01:13.797 T00  DEBUG:     archive<PHONE_NUMBER>001ED4000000BA-e39094dfc6246cc32eab0db38f0dc2e077252cd8, is WAL 1, source_compressed = 0, destination_compress 1,
 default_compress = 1
2015-05-29 18:01:13.798 T00  ERROR: [116]: unable to open /data/postgresql/wal-spool/archive/prd-psql3-01/out/0000000100001ED4000000BA-e39094dfc6246cc32eab
0db38f0dc2e077252cd8
 at /opt/backrest/bin/../lib/BackRest/Archive.pm line 173.
                                        BackRest::Archive::walInfo('BackRest::Archive=HASH(0xbd4978)', '/data/postgresql/wal-spool/archive/prd-psql3-01...')
called at /opt/backrest/bin/../lib/BackRest/Archive.pm line 672
                                        BackRest::Archive::xfer('BackRest::Archive=HASH(0xbd4978)', '/data/postgresql/wal-spool/archive/prd-psql3-01...', '/d
ata/postgresql/wal-spool/lock/prd-psql3-01-...') called at /opt/backrest/bin/../lib/BackRest/Archive.pm line 409
                                        BackRest::Archive::pushProcess('BackRest::Archive=HASH(0xbd4978)') called at /opt/backrest/bin/../lib/BackRest/Archive.pm line 66
                                        BackRest::Archive::process('BackRest::Archive=HASH(0xbd4978)') called at /opt/backrest/bin/pg_backrest.pl line 141
                                        eval {...} called at /opt/backrest/bin/pg_backrest.pl line 125

                                   2015-05-29 18:01:13.808 T00  DEBUG:     safe exit called, terminating threads
2015-05-29 18:01:14.092 T00   INFO: archive to be copied to backup total 1, size 16MB
2015-05-29 18:01:14.092 T00  DEBUG:     archive<PHONE_NUMBER>001ED4000000BB-d32f25ceb8382a1147fddadfbbb6a111bbf4421a, is WAL 1, source_compressed = 0, destination_compress 1,
 default_compress = 1
2015-05-29 18:01:14.093 T00  DEBUG:     Archive->pushCheck: backup:archive:0000000100001ED4000000BB
2015-05-29 18:01:14.096 T00  DEBUG:     File->copy:  local db:absolute:/data/postgresql/wal-spool/archive/prd-psql3-01/out/0000000100001ED4000000BB-d32f25c
eb8382a1147fddadfbbb6a111bbf4421a to remote backup:archive:/data/postgresql/backups/archive/prd-psql3-01/0000000100001ED4/0000000100001ED4000000BB-d32f25ce
b8382a1147fddadfbbb6a111bbf4421a.gz, source_compressed = false, destination_compress = true, ignore_missing_source = false, destination_path_create = true, modification_ti
me = [undef], mode = [undef], user = [undef], group = [undef]
2015-05-29 18:01:15.512 T00  DEBUG:     1 WAL segments were transferred, calling Archive->xfer() again
2015-05-29 18:01:15.513 T00  DEBUG:     File->manifest: db:absolute:/data/postgresql/wal-spool/archive/prd-psql3-01/out
2015-05-29 18:01:15.513 T00  DEBUG:     no archive logs to be copied to backup
2015-05-29 18:01:15.513 T00  DEBUG:     no more WAL segments to transfer - exiting
2015-05-29 18:01:15.514 T00  DEBUG:     safe exit called, terminating threads

This log makes it clear that there are two async process running at the same time.  I'm doing regression on the fix right now.

Fixed in e7d2d70.

Deployed in production, works fine!

Excellent - thanks!  0.70 has now been released.


Do not crash if image-data is missing

Skip conversion of ImageData if ImageCalibrated is already in structure.
Resolves #213

Is this enough or is something more needed, @lindamegner ?

That should probably do it. What is image data? A string of what?

That should probably do it. What is image data? A string of what?

ImageData is a binary blob of PNG-image data.

ok, so we store the image data twice? Both in IMAGE and ImageData?

ok, so we store the image data twice? Both in IMAGE and ImageData?

No, ImageData is dropped in L1b.


build(deps): bump go.etcd.io/etcd from v0.0.0-20211015134708-72d3e382e73c to v0.5.0-alpha.5.0.20230414071934-94593e63d45c

What this PR does
bump go.etcd.io/etcd from v0.0.0-20211015134708-72d3e382e73c to v0.5.0-alpha.5.0.20230414071934-94593e63d45c

Current dependencies on/for this PR:

main

PR #444   👈

PR #411

PR #412

PR #413

PR #414

This comment was auto-generated by Graphite.

Codecov Report
Patch coverage has no change and project coverage change: +0.01 :tada:

Comparison is base (9110264) 63.69% compared to head (243fc84) 63.71%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #444      +/-   ##
==========================================
+ Coverage   63.69%   63.71%   +0.01%
==========================================
  Files         131      131
  Lines       17928    17928
==========================================
+ Hits        11419    11422       +3
+ Misses       5962     5959       -3
  Partials      547      547

see 4 files with indirect coverage changes

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

@ijsong started a stack merge that includes this pull request via Graphite.

@ijsong merged this pull request with Graphite.


Update README.md

The previous source code link led to an HTTP 404 page.

Thanks!
But it is better to keep the link to the master branch. Docs in master branch should be in sync with code. Because otherwise it will be harder to keep docs in sync - at this point of time it is more likely that we add methods to this interface than rename this file.
I made changes in your PR and added a comment here. Also, because now we point to the master branch, better to not point to specific line.


Bad_alloc with pyroj import in QGIS plugin

Problem description
The following error may occur when trying to import pyproj into a QGIS plugin
terminate called after throwing an instance of 'std::bad_alloc'
  what():  std::bad_alloc
Abandon (core dumped)

I was able to identify that this error only occurs when the pyproj package has been installed using pip
The error does not occur if the package has been installed using a package manager, for example for ubuntu
sudo apt install python3-pyproj

This error was observed on several versions of linux (ubuntu/fedora) and on different versions of QGIS (3.28/ 3.34).
There doesn't seem to be any link with the pyproj version (installing an identical or more recent version using pip systematically causes the error)
To reproduce
Simply add :
import pyproj

to a plugin QGIS source files and install pyproj using pip. QGIS should stop while loading the plugin with the bad_alloc error.

I was able to identify that this error only occurs when the pyproj package has been installed using pip

fully expected and nothing one can do about, expect: never use pip with binary wheels, but in trivial use cases. And QGIS is not a trivial use case. The issue here is that pyproj installed through pip goes with its own libproj that conflicts with the libproj used by QGIS.


add cursor pagination

Added cursor pagination. Required feedback.

@sinisaos  Are you able to merge into this repo? If not, I need to give you permissions.

@dantownsend I think I can. Just one more question. Should I add github actions like this and is setup.py ok? Or you can check these things out after merging.

@sinisaos Yes, that should work.

@dantownsend Can you publish this package for PyPI, I'll be really appreciated that?
I’ve never done that before and I’ve followed a few tutorials but I haven’t been able to do that. I managed to upload to test PyPI (version is wrong because I delete 0.1.0 version) but all the dependencies are wrong when pip install it (maybe dependencies is wrong on test PyPI). I really doing something wrong or setup is wrong, I don't know.
Thanks in advance and sorry for interupption.

@sinisaos I've added an API key for PyPI and published a test release.
The easiest way to release from now on will be from GitHub. Is your PyPI username sinisaos? I'll add you as a maintainer in PyPI.

@dantownsend Thank you very much but problem is when I pip install piccolo_cursor_pagination in fresh virtualenv I get ModuleNotFoundError: No module named 'piccolo_cursor_pagination' because only piccolo_cursor_pagination-0.1.1.dist-info is in site-packages. There is no piccolo_cursor_pagination folder? I think this line is problem but I'm not shure.

@sinisaos The problem is this:
https://github.com/piccolo-orm/piccolo_cursor_pagination/blob/d533a096af7109be6c1b36b4a261dda38a2c1227/setup.py#L28
It isn't working for some reason. I'll push a fix.

@sinisaos Try it now

@dantownsend I'm not home at the moment but I'll try as soon as I get back and reported here. Sorry for the mess I made. Thanks again.

@sinisaos No worries. When you can, sign up for a PyPI account and give me your username so I can add you as a maintainer.

@dantownsend Everything work now. Thanks again. Username on PyPI is sinisaos.
Unfortunately when I tried cursor package I find error with newest Piccolo==0.73.0.
asyncpg.exceptions._base.InterfaceError: cannot perform operation: another operation is in progress
Future exception was never retrieved
future: <Future finished exception=ConnectionDoesNotExistError('connection was closed in the middle of operation')>
asyncpg.exceptions.ConnectionDoesNotExistError: connection was closed in the middle of operation

Piccolo==0.72.0 and lowest work fine.

@sinisaos I can't replicate that error. Do you get it when running the unit tests?

@dantownsend The unit test is good, but if you make a fresh virtualenv and use a sample project from piccolo_cursor_pagination an error occurs if Piccolo version is 0.73.0. If I install Piccolo==0.72.0 everything work ok. Maybe it's related to yesterday's changes in Piccolo.

@sinisaos I've tried it, and have been able to replicate it. In the most recent Piccolo version if you use run_sync, it will try and use the connection pool if one is available.
In the piccolo_cursor_pagination example project, I think weird things are happening because the app is async, but run_sync is being used for pagination.
What do you think about changing the pagination so it's async instead?

@dantownsend It’s weird and I don’t understand it. I change the get_cursor_rows method to be async and all unit tests pass both Postgres and Sqlite (both 0.72.0 and 0.73.0), but in the example app the same error occurs in Piccolo==0.73.0. In Piccolo==0.72.0 everything works ok, both sync or async get_cursor_rows method. I am deeply sorry for all these problems, but everything worked well before.

@sinisaos Interesting - I didn't consider the size of the connection pool.
It's definitely all connection pool related. If you like, you can upload your async version to a branch and I can test it.
I'll review the changes I made in the most recent Piccolo version in case I messed something up.

@dantownsend I make #2 that you can try async version, if it makes a difference.


[TESTED] Added Logging so we can find out whats going on with this error

@ExpDev07 My thing with the session seems to have very slightly improved the situation. I added some custom logging and a custom session provider, so I can properly log every crucial step, and we can find out what exactly is going wrong. If you can get this merged ASAP, we can start collecting some logs and finally get a step closer to solving this mystery.

@ExpDev07 there is also a bug that gets fixed with this commit that allowed invalid date of birth values to be saved into the database


SK calc error

First of all, so great to have a new version of squid to play around with.  I have an even greater appreciation for the "Group Me" labyrinth that needed to be untangled.
There seems to be a problem with the calculation of the SK ratios however.  What is being reported as the 7/6 ratio is actually the 7/4.  It should be SingleStagePbR(AGE, 1)/SingleStagePbR(Age, 0).  Likewise what is being reported as "8/6" ratio is actually 8/4.

Yes, and I suspect the error also occurs in the related arithmetic (i.e. using Nicole's example, the calculations are using common 207/206 = 15.6006 when the TRUE value of common 207/206 is actually (207/204)/(206/204) = 15.6006/18.1726 ~ 0.86). Analogously, TRUE common 208/206 values are near 2, not ~38.
This arithmetical error will also be the source of many of the large negative apparent ages, and the NaNs as well. I am optimistic that most of the rest of the arithmetic is fine (because the 4cor_206Pb238U data, which is the only value unaffected by the above, looks fine). Having said that, it is clear that I need to audit the arithmetic as it applies to the unknowns, starting with the SampleData ("all unknowns") output that predates v1.3.0.

Addressed in release v1.3.1 beta


load urls from environment in frontend

As a frontend dev
I want my url to get dynamically sourced from the environment
Because its an easy way to securely specifiy your urls, without revealing any information to the public while also being able to switch between urls for local development and different stages with ease.
NOTES:

Maybe you can introduce some kind of urlProvider-service?

Acceptance Criteria:

[ ] Urls get loaded from environment variables
[ ] The code is fully tested or a good explanation given why it should not or cannot be tested

awaiting review from @Straczi

Already addressed this issue in the comments for the commit https://github.com/corewar-teamprojekt/corewar/commit/fa702a4da573a1753221863826071e24596c821d#diff-de3f076909b4639b6c9005821614ad15fd4bba0ca675c9456d70517e855f7090 . We now have three different files where api requests are made: two component files and one RestService . Please address this issue in the comments for the other commit. Everything else looks good to me:)


Add kitchen sink Justfile to test syntax highlighting

I've added a file with all the different features I could think of, plus a line prefix in the GRAMMAR.md. The justfile is pretty messy, I mostly just copied things out of the docs while changing names as needed to get stuff parsing properly. I've definitely found some issues with the syntax highlighing during this exercise, so that's great!

Awesome, this is great! test-suite.just seems a little generic, so I renamed it to kitchen-sink.just.


WPFTweaksServices

Why would this tweak needs to change services from original value (example: automatic) to automatic?
So much bloat with those lines
https://github.com/ChrisTitusTech/winutil/blob/2354645b479b9ca7bcf377ea4e199b262f4b6bb4/config/tweaks.json#L117-L1540
Simply do
Set-Service AJRouter -StartupType Disabled
Set-Service AssignedAccessManagerSvc -StartupType Disabled
Set-Service CDPSvc -StartupType Manual
Set-Service StorSvc -StartupType Manual
Set-Service UsoSvc -StartupType Manual
Set-Service WpnService -StartupType Manual
Set-Service edgeupdate -StartupType Manual
Set-Service vm3dservice -StartupType Manual

The only reason I can think of why there are some services defined where the original and the target Startup Type are the same is, that like this you can be sure that if someone broke some functionality by changing stuff they don't know about, winutil would be able to reset and pretty much fix the problem, but for the sake of bloat I actually wouldn't be against removing the settings that effectively do not get changed from default.
Regarding your recommendation, I did write an answer outlining why it is smart to keep the object structure for registry keys in https://github.com/ChrisTitusTech/winutil/issues/1920#issuecomment-2093437488  This applies as well for services.


Inline runtime alloc

This PR:

makes MMTk's plan as a compiler option when building Julia
implements MMTk allocation fastpath for runtime allocations.

@udesou I have responded to your comments. Is there any change you would like to see for this PR? If everything is good, you can approve the PR, and I will start merging it.


⚠ MeeseOS has degraded performance

In 9b335fd, MeeseOS (https://aaronmeese.com) experienced degraded performance:

HTTP code: 523
Response time: 1641 ms

Resolved: MeeseOS performance has improved in 7fcde0c after 12 hours, 5 minutes.


Ignoring test branches

#608  👈
main

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

Join @CatChen and the rest of your teammates on  Graphite


Add OpenRewrite for automatic code cleanup

Description
This enables automatic migrations and code cleanup

It looks like OpenRewrite does not support mixed Java-Kotlin projects, so it can't parse Java files that call Kotlin classes:
jmeter/src/jorphan/src/main/java/org/apache/jorphan/gui/JFactory.java:111: error: cannot find symbol
    public static JEditableCheckBox small(JEditableCheckBox component) {
                  ^
  symbol:   class JEditableCheckBox
  location: class JFactory


Revert "Peter/goverance"

Reverts mspnp/architecture-center#566

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/cloud-adoption-guide/adoption-intro/azure-explainer.md
:white_check_mark:Succeeded
View

docs/cloud-adoption-guide/adoption-intro/governance-explainer.md
:white_check_mark:Succeeded

docs/cloud-adoption-guide/adoption-intro/overview.md
:white_check_mark:Succeeded
View

docs/cloud-adoption-guide/toc.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


pull-request-template and issue-template

Description
Created a Pull Request Template using Open Source Template
Created a IssueTemplate using Open Source Template
Related Issue

Create Community Documents #2
Motivation and Context

The project requires a pull request template to streamline submissions.
The project requires a issue template to streamline submissions.
How Has This Been Tested?

No Tests Required
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Merging since there is no feed back


feat: add goat plugin

Relates to:
https://github.com/ai16z/eliza/issues/735
https://github.com/ai16z/eliza/issues/644
Risks
Low
Background
What does this PR do?
Goat is an open-source framework for connecting AI agents to any onchain app.
This PR demonstrates how to integrate Goat with Eliza, enabling AI agents to execute a wide range of onchain actions with minimal effort. Using Goat’s plugin system, developers can enable agents to:

Mint NFTs on Zora
Swap tokens on Uniswap
Place bets on Polymarket
all with just a single line of code.

The framework supports multichain by either defining actions for a specific set of chains or dynamically selecting chains per action.
This concrete example simply enables transferring and checking the balance for USDC and ETH on the Base network.
What kind of change is this?
Features (non-breaking change which adds functionality)
Why are we doing this? Any context or related work?
Enable to easily add more onchain capabilities by leveraging Goat.
Documentation changes needed?
My changes do not require a change to the project documentation.
Testing
Where should a reviewer start?

Go to plugin-goat

Detailed testing steps

Make sure you have an EVM_PRIVATE_KEY, EVM_PUBLIC_KEY and EVM_PROVIDER_URL for Base set
Add plugin-goat to the dependencies of the package.json from the agent dir
Import it in agent/src/index.ts, exchange it for the evmPlugin
Run the agent and prompt it with: What's the USDC balance of 0x18Bead774f927Af586F86F6d054C269416E163DD
Should give you the balance of 0x18Bead774f927Af586F86F6d054C269416E163DD

To test actions on Base Sepolia simply:

Go to plugin-goat/src/index.ts and change

chain: {
    type: "evm",
    id: base.id,
},

to
chain: {
    type: "evm",
    id: sepolia.id, // including the viem import
},

Go to the plugin-goat/src/wallet.ts file and do the same
Make sure EVM_PROVIDER_URL works for Sepolia
Run the agent!

Discord username
aguspunk

@monilpat I moved to generateObjectV2 and added a wallet provider. Let me know if this works 🙌
Thank you for the review!


image display on iphone and ipad

Hi,
I am using your generator and it works great.
I have deployed my app on iphone and ipad and I noticed that images of the post sample are not displayed. Where as via a web browser they are displayed correctly.
thanks

I think this issue may be linked to is9 security and thus would not be linked to the plugin generation.


Extract code snippets from working-with-sheets.md

Extract the code snippets from docs/spreadsheet/working-with-sheets.md and use docfx snippets to retrieve them from the samples folder and audit the text.

closing as not needed


kubectl rollout status -f fails with "unable to decode" for file containing CRD reference

I’m finding the kubectl rollout status fails when I point it either a file (-f) or a kustomize directory (-k) with manifests that reference CRDs. An example would be if the manifests include an Istio VirtualService.
Here’s an example of what I see. I’ve got some CRDs (DestinationRule and VirtualService) in my setup along with deployments, services, etc. Nothing fancy:

$ kubectl kustomize overlays/sandbox > out.yaml
$ kubectl rollout status -f out.yaml
error: unable to decode "out.yaml": no kind "DestinationRule" is registered for version "networking.istio.io/v1alpha3" in scheme "k8s.io/kubernetes/pkg/kubectl/scheme/scheme.go:28"

I'm seeing a similar error when running kubectl auth reconcile -f on a file that contains any custom resource or APIServer resources.

I encountered the same issue as @bhcleek - when trying to run kubectl auth reconcile -f on a file containing custom resource definitions, kubectl aborts.
I created a small PR (over in kubernetes/kubernetes) which reproduces this failure, too: https://github.com/kubernetes/kubernetes/pull/85708
Explicitly attempting to use this example YAML file outside of the test, kubectl apply works fine, but kubectl auth reconcile fails.
$ kubectl auth reconcile -f rbac-resource-plus.yaml
clusterrole.rbac.authorization.k8s.io/testing-CR reconciled
        reconciliation required create
        missing rules added:
                {Verbs:[create delete deletecollection get list patch update watch] APIGroups:[] Resources:[pods] ResourceNames:[] NonResourceURLs:[]}
clusterrolebinding.rbac.authorization.k8s.io/testing-CRB reconciled
        reconciliation required create
        missing subjects added:
                {Kind:Group APIGroup:rbac.authorization.k8s.io Name:system:masters Namespace:}
rolebinding.rbac.authorization.k8s.io/testing-RB reconciled
        reconciliation required create
        missing subjects added:
                {Kind:Group APIGroup:rbac.authorization.k8s.io Name:system:masters Namespace:}
role.rbac.authorization.k8s.io/testing-R reconciled
        reconciliation required create
        missing rules added:
                {Verbs:[get list watch] APIGroups:[] Resources:[configmaps] ResourceNames:[] NonResourceURLs:[]}
unable to get type info from the object "*runtime.Unknown": no kind is registered for the type runtime.Unknown in scheme "k8s.io/kubectl/pkg/scheme/scheme.go:28"

$ kubectl apply -f rbac-resource-plus.yaml
Warning: kubectl apply should be used on resource created by either kubectl create --save-config or kubectl apply
clusterrole.rbac.authorization.k8s.io/testing-CR configured
pod/valid-pod created
Warning: kubectl apply should be used on resource created by either kubectl create --save-config or kubectl apply
clusterrolebinding.rbac.authorization.k8s.io/testing-CRB configured
Warning: kubectl apply should be used on resource created by either kubectl create --save-config or kubectl apply
rolebinding.rbac.authorization.k8s.io/testing-RB configured
Warning: kubectl apply should be used on resource created by either kubectl create --save-config or kubectl apply
role.rbac.authorization.k8s.io/testing-R configured
customresourcedefinition.apiextensions.k8s.io/foos.crds.example.com created

/reopen

/kind bug

/remove-lifecycle rotten
/sig cli
/area kubectl
/kind bug
/priority P2

This is probably obvious, but I'll ask anyways: was the CRD applied successfully before the attempt to create the custom resource?
Also, is it possible to report the version (e.g. kubectl version).

@seans3 Yes. The use-case here is deploying a pretty vanilla microservice to a cluster with Istio deployed. The Istio CRDs were created and in use long before attempting this rollout. My kubectl version at the time of opening this issue was 1.11.3.

I'm not sure if it's the same root cause (it's a similar error message), but I wrote a simple repro for kubectl auth reconcile -f failing when applied to CRDs in https://github.com/kubernetes/kubernetes/pull/85708.

and kubectl auth reconcile -f fails even if the CRD has already been applied successfully. Using the example I linked:
$ kind create cluster
...
$ kubectl create ns some-other-random
namespace/some-other-random created
$ kubectl apply -f rbac-resource-plus.yaml
clusterrole.rbac.authorization.k8s.io/testing-CR created
pod/valid-pod created
clusterrolebinding.rbac.authorization.k8s.io/testing-CRB created
rolebinding.rbac.authorization.k8s.io/testing-RB created
role.rbac.authorization.k8s.io/testing-R created
customresourcedefinition.apiextensions.k8s.io/foos.crds.example.com created
$ kubectl auth reconcile -f rbac-resource-plus.yaml
clusterrole.rbac.authorization.k8s.io/testing-CR reconciled
clusterrolebinding.rbac.authorization.k8s.io/testing-CRB reconciled
rolebinding.rbac.authorization.k8s.io/testing-RB reconciled
role.rbac.authorization.k8s.io/testing-R reconciled
unable to get type info from the object "*runtime.Unknown": no kind is registered for the type runtime.Unknown in scheme "k8s.io/kubectl/pkg/scheme/scheme.go:28"
$ kubectl version
Client Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.2", GitCommit:"52c56ce7a8272c798dbc29846288d7cd9fbae032", GitTreeState:"clean", BuildDate:"2020-04-16T11:56:40Z", GoVersion:"go1.13.9", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.2", GitCommit:"52c56ce7a8272c798dbc29846288d7cd9fbae032", GitTreeState:"clean", BuildDate:"2020-04-30T20:19:45Z", GoVersion:"go1.13.9", Compiler:"gc", Platform:"linux/amd64"}

/assign

/reopen
/remove-lifecycle rotten

/remove-lifecycle stale
Any progress with this?
Just encountered this issue with PrometheusRules CRDs in kubectl 1.18.10.

/remove-lifecycle stale
Any progress with this?
Just encountered this issue with PrometheusRules CRDs in kubectl 1.18.10.

/reopen

Any updates on that issue ?  I think this is still an on going issue

/reopen
/remove-lifecycle rotten


How to translate timeline date into japanese ?

Can someone give me an example to change timeline language to japanese ?
I found this setTimeWindow function at  ibm-gantt-chart/jquery/timeline.js  that can do this but I haven't figured out how to configure it. Please help me !!

You may look at https://github.com/IBM/gantt-chart/issues/21 for some info.

I've read it before, but it's not very clear. Can you help me explain a little more clearly. Or it would be nice if there are basic steps.

It seems that it should be
gantt.utils.setIntl({ locale : 'ja'} );
no ?

I tried this code and it works. Thanks for your advice !
gantt.utils.setIntl({locale : 'ja', formatMessage: () => {id: 'ja'}})

You're welcome !


Add vertical layout option

Seems to work reasonably well. I have no idea what I'm doing tho, I don't have much Rust experience, just found a nice tool and decided to add a must-have feature for me

Thank you for the PR! But I am rewriting a big part of mprocs right now. It would be very difficult to merge these changes into the branch I work on right. But I appreciate the proposal.

I will close this PR because next version of mprocs will include layout configuration which will allow making this what this PR does. But thank you anyway!


Make frontend form validation for support admin pages

There doesn't seem to be any frontend form validation against the required fields on this managed devices screen. I know we're assuming null values in the supported diseases to field to default to COVID until can update it manually, but unsure if this is intentional for the other fields. If it's not intentional, might be something we want to add to the other fields just to prevent human error (especially since I'll probably be the one interacting against this form 😅)

Originally posted by @fzhao99 in https://github.com/CDCgov/prime-simplereport/pull/3569#discussion_r839929793

Making this ticket to capture work related to form validation for the manage/create devices screens that support admin (Bob mostly) will be using to manage devices. Breaking this out from multiplex work


Enable seamless login for dinopark.

Coverage decreased (-0.06%) to 86.713% when pulling 6d6b09a9a994ec212df9329986d9fc3262016a2d on johngian:seamless-login-dinopark into 9e73435aca5caec713996725c04f0208a93728c3 on mozilla:master.


Needed : Portfolio worth indication

Some way of deciding how much a portfolio is worth.  I'm thinking this could be measured by:
"How much would I make if I sold everything instantly now?"
It would also be cool to track that.

Sum(value*number -number(number-1)/2)
On Sat, Feb 4, 2017 at 10:41 AM Brandon Davis<EMAIL_ADDRESS>wrote:

Some way of deciding how much a portfolio is worth. I'm thinking this
could be measured by:
"How much would I make if I sold everything instantly now?"
It would also be cool to track that.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/meme-exchange/server/issues/1, or mute the thread
https://github.com/notifications/unsubscribe-auth/AGgnAevacvZ9RJ2ILrrpOh5XnpilSw8wks5rZJwOgaJpZM4L3LqP
.

--
--Hastings Greer
UNC Class of 2017
919 525 6808 -- github.com/QuadmasterXLII

f8a271fef9d74da64d8ce86f39174723afde29c7


Include CQL Engine Information in CQL Test Runner Test Result

At present, the test result file generated by the CQL Test Runner doesn't have information regarding the specific CQL engine used for the tests. This proposal seeks to incorporate details about the CQL engine into the test results for better context and traceability.

Add simple, user-entered information to be included in the Result:

apiURL
cqlVersion
cqlTranslator
cqlTranslatorVersion
cqlEngine
cqlEngineVersion


EDM-390: Add decommissioning route to device API

A first step enabling a user to mark a device for decommissioning.

@hexfusion @fzdarsky Updated!

@fzdarsky What do you think?


feat: dynamic window title

add generateDocumentTitle option to Refine component; when enabled, Refine will generate titles in the same manner described in this issue #2810

options={{
    warnWhenUnsavedChanges: true,
    syncWithLocation: true,
    generateDocumentTitle: true
}}

Note: the default value is false

generateDocumentTitle also accepts a callback function that will get invoked whenever the url changes, and it should return a string that will be used to set the window title

options = {{
    warnWhenUnsavedChanges: true,
        syncWithLocation: true,
            generateDocumentTitle: ({
                resource, // posts
                action, // create
                params, // {id: 1}
                autoGeneratedTitle // Create new Post | Refine
            }) => {
                return `Refine ${action}` // => 'Refine create'
            }
}}

add useDocumentTitle hook which allows the user to change the title from any component

// to update the title as soon as the component is rendered
useDocumentTitle('New Post')

// to update the title at a later point use the updateTitle function returned from the hook
const {
    title, // current document title
    updateTitle
    } = useDocumentTitle()

updateTitle("New Post")

``

Hi @aliemir, thank you for your valuable feedback. I appreciate your point, and I will make the necessary changes as soon as possible. However, I'd like to mention two things:

You mentioned that the context is not necessary for persisting the title and I agree with you. However, I only utilized the context to prioritize title updates from the useDocumentTitle hook over the auto-generated ones. This is because, based on my understanding, the useEffect of child components executes before the parent's, which mean it will overwrite the title set by the child component.

Regarding the change I made to the @refinedev/react-router-v6 package: I found that useParams doesn't function outside of a Route component. That's why I replaced it with matchPath, but this necessitated checking if the resource and action (in other words, the params that I want to match ) were defined before executing the function.So I guess I should only use matchPath only if useParams has returned undefined ( which means it's been called outside of Route ). If you have any alternative solutions, please let me know.

Thank you once again for your feedback, and I'll be waiting for your opinion on the points I mentioned.

Hi @aliemir, thank you for your valuable feedback. I appreciate your point, and I will make the necessary changes as soon as possible. However, I'd like to mention two things:

You mentioned that the context is not necessary for persisting the title and I agree with you. However, I only utilized the context to prioritize title updates from the useDocumentTitle hook over the auto-generated ones. This is because, based on my understanding, the useEffect of child components executes before the parent's, which mean it will overwrite the title set by the child component.
Regarding the change I made to the @refinedev/react-router-v6 package: I found that useParams doesn't function outside of a Route component. That's why I replaced it with matchPath, but this necessitated checking if the resource and action (in other words, the params that I want to match ) were defined before executing the function.So I guess I should only use matchPath only if useParams has returned undefined ( which means it's been called outside of Route ). If you have any alternative solutions, please let me know.

Thank you once again for your feedback, and I'll be waiting for your opinion on the points I mentioned.

You're right, useDocumentTitle when used in multiple ways in the component tree may cause inconsistencies/mismatches. I actually meant to say that, instead of using the context in the core package, we can export it in our router packages and let the user wrap the <Refine> component if they want to. I thought maybe if the user wants to provide dynamic titles using useDocumentTitle and also set auto generated titles to false. Which in a way might make the context unnecessary. Does it make sense? We should still export the context though 😅
About the change in @refinedev/react-router-v6, yes it makes sense to use matchPath as a fallback and let users use useParams without resource and action if they want to. I think useParams doesn't throw an error, when used outside of <Route>, right? 🤔

I'm looking forward to see your updates on this PR 👀 I'm excited about how this PR will turn out. 🚀

Hi @aliemir, I've made the changes you requested. Here's a summary of the updates:
DocumentTitleHandler
The DocumentTitlteHandler is now exported from react-router-v6 and nextjs-router. However, I couldn't create one for remix and appdir in NextJS since their approach to handling document titles is outside the component itself.
useDocumentTitle
I've used useLayoutEffect in the DocumentTitleHandler to ensure that it updates the title before useDocumentTitle. This way, the hook can override the title set by the component.
i18n
I've introduced the key scope documentTitle. This scope includes the following keys:
    "documentTitle": {
        "default": "Refine",
        "suffix": " | Refine",
        "post": {
            "list": "Posts | Refine",
            "show": "#{{id}} Show Post | Refine",
            "edit": "#{{id}} Edit Post | Refine",
            "create": "Create new Post | Refine",
        }
    }

default: is used when the current route is not a resource
suffix: Appended to the default generated titles.
[resource]:  An object containing titles for each action within the resource.
The useDocumentTitle hook also supports i18n by accepting an object instead of a string, with an i18nKey property. For example:
useDocumentTitle({i18nKey: "documentTitle.default"})

Changeset
I tried to use changeset as mentioned in the docs but I got the following error:
> changeset

🦋  error Error: Failed to find where HEAD diverged from master. Does master exist?

Documentation
Once you confirm that these changes are as desired, I would be glad to update the documentation.
Thank you!

I think the problem came when I forked the repository; I accidentally chose to copy only the next branch. Any idea how I can fetch the master branch from the original repository?
Thanks

I tried git fetch --all, but unfortunately it didn't work because the master branch doesn't exist in the remote repository (the forked one ).

I tried git fetch --all, but unfortunately it didn't work because the master branch doesn't exist in the remote repository (the forked one ).

I think you can do try adding the main repo as a remote and checkout/push master 🤔
git remote add refineOrigin https://github.com/refinedev/refine.git
git fetch refineOrigin
git checkout master
git push origin master

This way we'll make sure we have master in origin (your fork). Then, you can checkout to feat/dynamic-window-title and changesets should work as expected 🙏

I tried git fetch --all, but unfortunately it didn't work because the master branch doesn't exist in the remote repository (the forked one ).

I think you can do try adding the main repo as a remote and checkout/push master 🤔
git remote add refineOrigin https://github.com/refinedev/refine.git
git fetch refineOrigin
git checkout master
git push origin master

This way we'll make sure we have master in origin (your fork). Then, you can checkout to feat/dynamic-window-title and changesets should work as expected 🙏

That did work! Thank you.
Please let me know if there are any further modifications that need to be done.

Hey @abdellah711, thank you for the changes 🙏
Can you add the necessary section to the docs?

https://refine.dev/docs/packages/documentation/routers/react-router-v6/#additional-components here for React Router v6
https://refine.dev/docs/packages/documentation/routers/nextjs/#additional-components and here for Next.js

Also if you have time, can you create separate changesets for core and router packages? One for each package so we can give more details about the changes 🚀

Hi @aliemir, I've made the changes you asked me for, and I also merged the next branch into the feat/dynamic-window-title branch.
Please let me know if I did anything wrong or if you need any additional modifications.
Thank you!

Thank you for your effort @abdellah711 🚀

I'm glad I could help, and thank you as well for allowing me to be part of your project. I look forward to working together on new features. 🚀