Setup page fails when proxy url is specified

As reported by a user, the setup page fails to conclude its process when a proxy is required for Internet Access.
Splunk Enterprise Version: 8.2.6
SEKOIA.IO for Splunk Version: 1.1.2

Find below the steps I executed to check the proxy support in SEKOIA.IO App for Splunk  works as expected.

I created a squid server that support Transparant HTTPs proxying.
with the browser I checked the squid works as expected
I !! switched off the docker of the proxy !!
I downloaded and installed a fresh Splunk Enterprise 8.2.6 instance
through the marketplace, I searched than installed the SEKOIA.IO app
I configured the app with the following info

my SEKOIA.IO API key
proxy url: http://<IP_ADDRESS>:3128
ipv4 lookup: search query:index=* and Field:ipv4
clicked on save and got redirected to the SEKOIA.IO Dashboard

The dashboard shows no indicator is downloaded (remember I switch off the proxy)
in the var/log/splunk/splunkd.log I see numerous error raised by the python scripts that fails to connect to the proxy server:

04-22-2022 15:47:25.824 +0200 INFO  ExecProcessor [33826 ExecProcessor] - message from "/home/georges/Applications/splunk/bin/python3.7 /home/georges/Applications/splunk/etc/apps/sekoia.io/bin/sekoia_indicators.py" Getting new events with the SEKOIA.IO modular input
04-22-2022 15:47:25.824 +0200 INFO  ExecProcessor [33826 ExecProcessor] - message from "/home/georges/Applications/splunk/bin/python3.7 /home/georges/Applications/splunk/etc/apps/sekoia.io/bin/sekoia_indicators.py" Fetch the indicators for input sekoia_indicators://feed
04-22-2022 15:47:25.832 +0200 INFO  ExecProcessor [33826 ExecProcessor] - message from "/home/georges/Applications/splunk/bin/python3.7 /home/georges/Applications/splunk/etc/apps/sekoia.io/bin/sekoia_indicators.py" Fetch indicators from feed_id=d6092c37-d8d7-45c3-8aff-c4dc26030608
04-22-2022 15:47:25.835 +0200 ERROR ExecProcessor [33826 ExecProcessor] - message from "/home/georges/Applications/splunk/bin/python3.7 /home/georges/Applications/splunk/etc/apps/sekoia.io/bin/sekoia_indicators.py" Traceback (most recent call last):
04-22-2022 15:47:25.835 +0200 ERROR ExecProcessor [33826 ExecProcessor] - message from "/home/georges/Applications/splunk/bin/python3.7 /home/georges/Applications/splunk/etc/apps/sekoia.io/bin/sekoia_indicators.py"   File "/home/georges/Applications/splunk/etc/apps/sekoia.io/bin/../lib/py3/urllib3/connection.py", line 170, in _new_conn
04-22-2022 15:47:25.835 +0200 ERROR ExecProcessor [33826 ExecProcessor] - message from "/home/georges/Applications/splunk/bin/python3.7 /home/georges/Applications/splunk/etc/apps/sekoia.io/bin/sekoia_indicators.py"     (self._dns_host, self.port), self.timeout, **extra_kw
04-22-2022 15:47:25.835 +0200 ERROR ExecProcessor [33826 ExecProcessor] - message from "/home/georges/Applications/splunk/bin/python3.7 /home/georges/Applications/splunk/etc/apps/sekoia.io/bin/sekoia_indicators.py"   File "/home/georges/Applications/splunk/etc/apps/sekoia.io/bin/../lib/py3/urllib3/util/connection.py", line 96, in create_connection
04-22-2022 15:47:25.835 +0200 ERROR ExecProcessor [33826 ExecProcessor] - message from "/home/georges/Applications/splunk/bin/python3.7 /home/georges/Applications/splunk/etc/apps/sekoia.io/bin/sekoia_indicators.py"     raise err
04-22-2022 15:47:25.835 +0200 ERROR ExecProcessor [33826 ExecProcessor] - message from "/home/georges/Applications/splunk/bin/python3.7 /home/georges/Applications/splunk/etc/apps/sekoia.io/bin/sekoia_indicators.py"   File "/home/georges/Applications/splunk/etc/apps/sekoia.io/bin/../lib/py3/urllib3/util/connection.py", line 86, in create_connection
04-22-2022 15:47:25.835 +0200 ERROR ExecProcessor [33826 ExecProcessor] - message from "/home/georges/Applications/splunk/bin/python3.7 /home/georges/Applications/splunk/etc/apps/sekoia.io/bin/sekoia_indicators.py"     sock.connect(sa)
04-22-2022 15:47:25.835 +0200 ERROR ExecProcessor [33826 ExecProcessor] - message from "/home/georges/Applications/splunk/bin/python3.7 /home/georges/Applications/splunk/etc/apps/sekoia.io/bin/sekoia_indicators.py" ConnectionRefusedError: [Errno 111] Connection refused
04-22-2022 15:47:25.835 +0200 ERROR ExecProcessor [33826 ExecProcessor] - message from "/home/georges/Applications/splunk/bin/python3.7 /home/georges/Applications/splunk/etc/apps/sekoia.io/bin/sekoia_indicators.py"
04-22-2022 15:47:25.835 +0200 ERROR ExecProcessor [33826 ExecProcessor] - message from "/home/georges/Applications/splunk/bin/python3.7 /home/georges/Applications/splunk/etc/apps/sekoia.io/bin/sekoia_indicators.py" During handling of the above exception, another exception occurred:

[...]

04-22-2022 15:47:25.836 +0200 ERROR ExecProcessor [33826 ExecProcessor] - message from "/home/georges/Applications/splunk/bin/python3.7 /home/georges/Applications/splunk/etc/apps/sekoia.io/bin/sekoia_indicators.py" requests.exceptions.ProxyError: HTTPSConnectionPool(host='api.sekoia.io', port=443): Max retries exceeded with url: /v2/inthreat/collections/d6092c37-d8d7-45c3-8aff-c4dc26030608/objects?match%5Btype%5D=indicator&limit=300 (Caused by ProxyError('Cannot connect to proxy.', NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7f5c8304d210>: Failed to establish a new connection: [Errno 111] Connection refused')))

I restart my squid docker
ProxyError in the splunk log stops and instead numerous logs show indicators are downloaded (for example, messages with message "WARNING Unsupported path …" shows the connection was succesfull
Almost 10min after the end of log errors, logs show "Saved KVStore Batch of" messages and the first indicators start to appear on the dashboard


Implement Logic in Symbol layer

Conditions
Constraints
Modifiers

Implementation of symbols was in 54e12b354b8bd8e0a1a0e0c6e7926c121011c108

Deep investigation into Querying was done in #45


take care of domain aliases

Current coverage is 79.47% (diff: 66.66%)

Merging #971 into master will increase coverage by 0.04%

@@             master       #971   diff @@
==========================================
  Files           141        141
  Lines          7264       7266     +2
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           5770       5775     +5
+ Misses         1494       1491     -3
  Partials          0          0

Powered by Codecov. Last update 37f784c...1f61d21


IDVA6-1187 show remove link based on roles

Link to Jira:
https://companieshouse.atlassian.net/browse/IDVA6-1187
Changes:

Logged in user role added
Remove column shown based on user role
Tests updated

CI: No security warnings found

CI: No security warnings found


Footer on main website directs to old link "https://github.com/apc-gmhs"

Good first issue for those learning github. Simple change of link in HTML code.

hey can you assign me for this issue? and which type of link you want to add

Assign what issue?


No timing reports generated

I am running syn_yosys.sh to reproduce the outlined flow. But looks like no timing reports were generated yet. However, I didn't see any obvious problems being reported up until the script looked for timing files.
The flow fails after running sta.
It fails inside: ./translate_timing_rpts.sh and the Python call-stack shows:
input in flex scanner failed
Traceback (most recent call last):
File "./python/build_translated_names.py", line 47, in
main()
File "./python/build_translated_names.py", line 32, in main
(new_cells_to_translate, path_info) = extract_path_info(csv_rpt)
File "/home/UNT/ndg0068/rv_design/ibex/syn/python/flow_utils.py", line 32, in extract_path_info
timing_in = csv.reader(open(timing_csv))
FileNotFoundError: [Errno 2] No such file or directory: './syn_out/ibex_22_06_2020_12_40_28/reports/timing/.csv.rpt'
Traceback (most recent call last):
File "./python/translate_timing_csv.py", line 41, in
main()
File "./python/translate_timing_csv.py", line 25, in main
(cells_to_translate, path_info) = extract_path_info(args.rpt_file)
File "/home/UNT/ndg0068/rv_design/ibex/syn/python/flow_utils.py", line 32, in extract_path_info
timing_in = csv.reader(open(timing_csv))
FileNotFoundError: [Errno 2] No such file or directory: './syn_out/ibex_22_06_2020_12_40_28/reports/timing/.csv.rpt'
The report folder is empty:
ndg0068@alveo-cse:~/rv_design/ibex/syn/syn_out/ibex_22_06_2020_12_40_28/reports/timing$ ls
ndg0068@alveo-cse:~/rv_design/ibex/syn/syn_out/ibex_22_06_2020_12_40_28/reports/timing$

Well.. I looked into this issue further and it turns out that the yosys synth is not successfully generating the netlist. And specifically, it reports:

Executing Verilog-2005 frontend.
ERROR: Parser error in line syn_out/ibex_22_06_2020_15_23_56/generated/ibex_controller.v:830: syntax error, unexpected '[', expecting ';' or '('

At this indicated line number (830) of ibex_controller.v is the function:
830         function automatic [5:0] sv2v_cast_6;
831                 input reg [5:0] inp;
832                 sv2v_cast_6 = inp;
833         endfunction
Can someone suggest why I am seeing this?
Thank you
Best Regards
Nagendra

I am closing this issue and opening a new specific one about s2v.


safetensors_rust.SafetensorError: Error while deserializing header: HeaderTooLarge

i install everything correctly and when i want to run an inference , i get this error.
People on the net with the same error told me that it can be corrupted file so i download everything again ( multiple time and i keep getting this error)  here the full error message
Traceback (most recent call last):
File "/home/azureuser/localfiles/MagicTime/inference_magictime.py", line 249, in
main(args)
File "/anaconda/envs/magictime/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
return func(*args, **kwargs)
File "/home/azureuser/localfiles/MagicTime/inference_magictime.py", line 61, in main
text_encoder = CLIPTextModel.from_pretrained(model_config.pretrained_model_path, subfolder="text_encoder").cuda()
File "/anaconda/envs/magictime/lib/python3.10/site-packages/transformers/modeling_utils.py", line 3284, in from_pretrained
with safe_open(resolved_archive_file, framework="pt") as f:
safetensors_rust.SafetensorError: Error while deserializing header: HeaderTooLarge

Thank you for your interest. I have never encountered this error. Maybe you can confirm whether the version information of the package is consistent with ours, or try to reconfigure the environment.

It seems that there is a problem with your torch or transformers library. You can try to reinstall the same version as ours.

It seems that there is a problem with your torch or transformers library. You can try to reinstall the same version as ours.

I already check the environnement but i will reinstall it from scratch i come home from work thx for your reply

My server is already running MagicTime normally, but there is no model called "safetendersorust.Safetensor" on my server. Please check when this model was installed during the installation process

Maybe a duplicate of #21 .  Sorry I posted a dup!  Try making sure git-lfs is installed.  Then redo your installation.  Easiest way is to do "git checkout ckpts" to restore order.  Note, issues with these scripts #23 .


Fix typo in de-DE

I saw this typo in the app and couldn't leave it like that
Resolved / Related Issues
No
Details of Changes

Fix typo

Validation
How did you test these changes?

[ ] Built and ran the app
[ ] Tested the changes for accessibility

Screenshots (optional)

@CodeDoctorDE before we merge this, can you add your agreement to #7625?

Added. MIT is one of the best licenses


unable to get build environment of the sdk on ubuntu 20.04

Description
Sorry newbie here,
cork create fails to build the sdk environment
Impact
Not able to get started with building custom images
Environment and steps to reproduce
download the latest cork and run
cork create
2020-10-07T17:39:49Z cork: Detecting SDK version
2020-10-07T17:39:52Z cork: Found SDK version 2643.0.0 from remote repo
2020-10-07T17:39:52Z cork: Downloading SDK version 2643.0.0
2020-10-07T17:39:56Z sdk: Unpacking SDK into /home/sanshand/flatcar/sdk/chroot
2020-10-07T17:42:47Z sdk: Unpacked
gpg: keybox '/home/sanshand/.repoconfig/gnupg/pubring.kbx' created
gpg: /home/sanshand/.repoconfig/gnupg/trustdb.gpg: trustdb created
gpg: key 16530D5E920F5C65: public key "Repo Maintainer<EMAIL_ADDRESS>imported
gpg: key 67B7E448692B382C: public key "Conley Owens<EMAIL_ADDRESS>imported
gpg: Total number processed: 2
gpg:               imported: 2
fatal: Cannot get https://gerrit.googlesource.com/git-repo/clone.bundle
fatal: error [Errno -2] Name or service not known
fatal: cloning the git-repo repository failed, will remove '.repo/repo'
2020-10-07T17:43:08Z cork: repo init failed: exit status 1
but I am able to reach and download the clone.bundle directly

Hi @skshandilya, which version of cork are you using? cork version

I just spun up a clean ubuntu 20.04 instance:
curl -OL https://github.com/flatcar-linux/mantle/releases/download/v0.15.1/cork-0.15.1-amd64
chmod +x cork-0.15.1-amd64
mv cork-0.15.1-amd64 /usr/local/bin/cork
apt update
apt install -y git build-essential
adduser vbatts
usermod -aG sudo vbatts

log in again as the limited user (cork environment does not work correctly as root):
mkdir flatcar
cd flatcar
cork create
cork enter

The output rolled right through where yours errored out:
2020-10-07T18:56:53Z cork: Detecting SDK version
2020-10-07T18:56:54Z cork: Found SDK version 2643.0.0 from remote repo
2020-10-07T18:56:54Z cork: Downloading SDK version 2643.0.0
2020-10-07T18:57:14Z sdk: Removing SDK at /home/vbatts/flatcar/chroot
2020-10-07T18:57:22Z sdk: Removed
2020-10-07T18:57:22Z sdk: Unpacking SDK into /home/vbatts/flatcar/chroot
2020-10-07T19:04:07Z sdk: Unpacked
gpg: keybox '/home/vbatts/.repoconfig/gnupg/pubring.kbx' created
gpg: /home/vbatts/.repoconfig/gnupg/trustdb.gpg: trustdb created
gpg: key 16530D5E920F5C65: public key "Repo Maintainer<EMAIL_ADDRESS>imported
gpg: key 67B7E448692B382C: public key "Conley Owens<EMAIL_ADDRESS>imported
gpg: Total number processed: 2
gpg:               imported: 2

Get https://gerrit.googlesource.com/git-repo/clone.bundle
Get https://gerrit.googlesource.com/git-repo
repo: warning: Python 2 is no longer supported; Please upgrade to Python 3.6+.

... A new version of repo (2.8) is available.
... New version is available at: /mnt/host/source/.repo/repo/repo
... The launcher is run from: /usr/lib/python-exec/python2.7/repo
!!! The launcher is not writable.  Please talk to your sysadmin or distro
!!! to get an update installed.

repo: Updating release signing keys to keyset ver 2.3
Downloading manifest from https://github.com/flatcar-linux/manifest.git
remote: Enumerating objects: 3, done.
remote: Counting objects: 100% (3/3), done.
remote: Compressing objects: 100% (3/3), done.
remote: Total 7794 (delta 0), reused 1 (delta 0), pack-reused 7791
[...]
remote: Counting objects: 100% (51/51), done.
remote: Compressing objects: 100% (42/42), done.
Fetching projects:  89% (26/29) flatcar-linux/dockerremote: Total 2357 (delta 16), reused 31 (delta 9), pack-reused 2306
Fetching projects:  93% (27/29) flatcar-linux/initremote: Enumerating objects: 10, done.
remote: Counting objects: 100% (10/10), done.
remote: Compressing objects: 100% (10/10), done.
remote: Total 43 (delta 4), reused 6 (delta 0), pack-reused 33
Fetching projects: 100% (29/29), done.
Updating files: 100% (9150/9150), done.tcar-linux/nss-altfilesUpdating files:  95% (8702/9150)
Checking out projects: 100% (29/29), done.
Your sources have been sync'd successfully. Note that only the revisions for the coreos-overlay, scripts, and portage-stable repositories are relevant. Revisions for the other repositories are specified in the ebuild files.
repo sync has finished successfully.

It is my guess that possibly a network connection or dns resolution failed. Possibly the dns servers <IP_ADDRESS> and <IP_ADDRESS> are not reachable.
Try again with the flag cork create --use-host-dns
(i'm closing as this does not appear to be a widespread issue. Feel free to re-open if it continues to be an issue)

Hi @skshandilya, which version of cork are you using? cork version

cork version
mantle/cork version 0.15.0+38-gdcd6f832

--use-host-dns worked

@skshandilya oh good


But how does it look in dark mode?

Nice, but how does it look in dark mode?
Originally posted by @danielsaidi in https://github.com/KeyboardKit/KeyboardKit/issues/285#issuecomment-881575984

This was accidentally created :)


Seaweed volume size is keeps on increasing

I have observed in my seaweed, volume size is keeps on increasing every hour without  initiating any upload or delete requests.
How to analyze the root cause of this issue?

This is by design.
see https://github.com/chrislusf/seaweedfs/wiki/Master-Server-API#force-garbage-collection


I should be able to init a model without migrations (no migrations directory)

I should be able to generate a model with a different type of id. id:uuid. this currently makes two id columns


GET /_scheduler returns 405

Observe:
$ curl -v -X GET http://admin:abc123@localhost:6002/_scheduler
Note: Unnecessary use of -X or --request, GET is already inferred.
*   Trying ::1...
* TCP_NODELAY set
* Connected to localhost (::1) port 6002 (#0)
* Server auth using Basic with user 'admin'
> GET /_scheduler HTTP/1.1
> Host: localhost:6002
> Authorization: Basic YWRtaW46YWJjMTIz
> User-Agent: curl/7.52.1
> Accept: */*
>
< HTTP/1.1 405 Method Not Allowed
< Server: CouchDB/2.1.0 (Erlang OTP/17)
< Date: Wed, 01 Nov 2017 21:20:11 GMT
< Content-Type: application/json
< Content-Length: 64
< Cache-Control: must-revalidate
< Allow: GET,HEAD
<
{"error":"method_not_allowed","reason":"Only GET,HEAD allowed"}
* Curl_http_done: called premature == 0
* Connection #0 to host localhost left intact

Same essential behavior observed when substituting GET with HEAD, or when requesting a specific (non-existent) URL below the /_scheduler/ path, i.e.:  /_scheduler/foo.
Expected Behavior
I would expect a valid response, or a 404 if the endpoint isn't supported.
Current Behavior
405 status (method not allowed), with the error message:
{"error":"method_not_allowed","reason":"Only GET,HEAD allowed"}

Possible Solution
I believe this is just a matter of returning the wrong error code; 404 is probably appropriate.
Steps to Reproduce (for bugs)
See above
Your Environment
CouchDB 2.1.0 official docker image.

I think the idea was to reserve _scheduler for the future to return replication stats or summary. But it hasn't been implemented yet.
Yeah 404 might make more sense here. Especially if we say "only GET/HEAD allowed" but then those method don't do anything.

This one was fixed


请问如何调整学习率与gpu个数的关系

在8gpu和16gpu的config中，只有batchsize的差异，学习率需要进行相应调整吗

No need to adjust the learning rate, our gradient reduction uses sum operation.

好的，谢谢！

No need to adjust the learning rate, our gradient reduction uses sum operation.

Exceuse me. I want to know the relation of 'batchsize' and 'learning rate'.When I set the value of 'batchsize' differently, the results of EAO is different too.So when I modify 'batchsize', how can I modify    'learning rate'correspondingly.There are too many 'LR',such as'BACKBONE:LAYERS_LR','TRIN:BASE_LR' etc.

No need to adjust the learning rate, our gradient reduction uses sum operation.

Exceuse me. I want to know the relation of 'batchsize' and 'learning rate'.When I set the value of 'batchsize' differently, the results of EAO is different too.So when I modify 'batchsize', how can I modify 'learning rate'correspondingly.There are too many 'LR',such as'BACKBONE:LAYERS_LR','TRIN:BASE_LR' etc.

I think you can find the answer in the paper.

Since no further question is asked, I'm closing it for now.
Please reopen it or create a new issue if you have further question.


[ignore] Testing CI force for signing server

Summary of the Pull Request
What is this about:
What is include in the PR:
How does someone test / validate:
Quality Checklist

[ ] Linked issue: #xxx
[ ] Communication: I've discussed this with core contributors in the issue.
[ ] Tests: Added/updated and all pass
[ ] Installer: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Docs: Added/ updated
[ ] Binaries: Any new files are added to WXS / YML

[ ] No new binaries
[ ] YML for signing for new binaries
[ ] WXS for installer for new binaries

Contributor License Agreement (CLA)
A CLA must be signed. If not, go over here and sign the CLA.

doesn't kick of sign server to check automatically.


fix: Delete stale application which is getting created for the failed API in case of fork, duplicate and import

Description

APIs for import, fork and duplicate application can fail at various stages like cloning pages, actions or JSObjects. This results in stale application getting created in this process. This PR makes sure if API fails for any reason the newly created application is also deleted so that we won't have any stale apps in DB.

Fixes #10490
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Manual test

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

/ok-to-test sha=fd76955

/ok-to-test sha=fd76955

/ok-to-test sha=7915fb7

@abhvsn It would be great if we can add few tests for this.

@nayan-rafiq I was looking for a way to add the test cases but was unable to do so, the reason being the above code kicks in only when we end up in an erroneous state for the mentioned APIs. Even for manual testing, I had to deliberately remove some fields from DB for the parent application so that these APIs will result in errors. But please let me know if there is any way to do this, I'll add those test cases 🙂

@abhvsn I think mocking some services will help here. For example, you can mock the few services that are called when importing application and raise error from the mocks. Then check whether partial application is created or not.

Closing this as this is being now handled with transactions, which is much better implementation than the current one.
Ref PR: https://github.com/appsmithorg/appsmith/pull/14939


[Bug]: 4.4.0 appears broken

Which packages are impacted?

[X] remix-validated-form
[ ] @remix-validated-form/with-zod
[ ] @remix-validated-form/with-yup
[ ] zod-form-data

What version of these packages are you using?

remix-validated-form: ^4.4.0
@remix-validated-form/with-zod: ^2.0.1

Please provide a link to a minimal reproduction of the issue.
https://github.com/airjp73/remix-validated-form/blob/main/packages/remix-validated-form/src/internal/state/createFormStore.ts#L105
Steps to Reproduce the Bug or Issue
Apologies that I don't have a 1-click repro. I'm pretty sure I'm doing exactly what's shown in the demo, although since the demo code doesn't ever show the full code I can't be totally sure. However, I can tell you that the code works in 4.3.1 and fails in 4.4.0, and when I used the debugger to investigate, I can see that setFormElement is being called with the proper element, but when validateField is called, get().formElement is null. So it seems like the draft is never being saved or something like that.
Expected behavior
The invariant should not be varied.
Screenshots or Videos
No response
Platform

OS: macOS
Browser: Chrome
Version: All

Additional context
No response

A reproduction would definitely be helpful here. The demo on the site always uses the latest version of the library since it's a monorepo, so there must be something different going on in your case.
Can you describe a bit more what you have to do to break it? Does it just break on mount or when you blur an input or something else?
If it helps, the FormInput in the demo is here, and all the other components imported in the demo are in the same folder.

Turns out it's caused by React.StrictMode. The attached zip file includes a repository with a minimal reproduction. Extract, run yarn; remix dev, open the page, click in the input, and blur it. The invariant violation will appear in the console. If you remove React.StrictMode from entry.client.tsx, the problem is bypassed. Documentation about React Strict Mode. In particular, as documented in Detecting unexpected side effects the "double-call of render" is likely the culprit here. It's important to note that this likely does describe a real bug, like incorrectly utilizing a reference that isn't tracked through standard hooks.
For easy reference, the dependencies listed in the test repo are:
{
  "dependencies": {
    "@remix-run/node": "^1.4.3",
    "@remix-run/react": "^1.4.3",
    "@remix-run/serve": "^1.4.3",
    "@remix-run/server-runtime": "^1.0.0",
    "@remix-validated-form/with-zod": "^2.0.1",
    "react": "^18.1.0",
    "react-dom": "^18.1.0",
    "remix-validated-form": "^4.4.0",
    "zod": "^3.15.1"
  },
  "devDependencies": {
    "@remix-run/dev": "^1.4.3",
    "@types/react": "^18.0.8",
    "@types/react-dom": "^18.0.3",
    "typescript": "^4.5.5"
  }
}

I experience the problem only on Safari: https://github.com/marcinczenko/remix-starter-with-tailwindcss.

I have a fix locally, but now have a broken test. Will probably be ready this week sometime.

I've published the fix as 4.5.0-beta.0. It's a big enough refactor that feels weird to release as a patch. I have a few other small features I want to squeeze into 4.5 so I'll do a full release once those are done.

Fix released in 4.5.0


[FEATURE] Provide multiarch image for longhorn-ui

Is your feature request related to a problem? Please describe.
When deploying Longhorn on arm64 everything deploys without issue with the exception of longhorn-ui. I edited the deployment to change the container image to be longhornio/longhorn-ui:v1.1.0-arm64 and it deployed just fine.
Describe the solution you'd like
If a multiarch image is provided it will "just work"
Describe alternatives you've considered
Manually editing the deployment for longhorn-ui to point to arm image.

@FlexibleToast longhornio/longhorn-ui:v1.1.0 should work for ARM64 now. Can you retry? Remember to set image pull policy to always.

@FlexibleToast longhornio/longhorn-ui:v1.1.0 should work for ARM64 now. Can you retry? Remember to set image pull policy to always.

I changed the image back to longhornio/longhorn-ui:v1.1.0 and changed the image pull policy. I can confirm it is working my arm64 cluster.

I changed the image back to longhornio/longhorn-ui:v1.1.0 and changed the image pull policy. I can confirm it is working my arm64 cluster.

Thanks @FlexibleToast for confirmation. I will close the issue then.

Thanks @FlexibleToast for confirmation. I will close the issue then.


Events not triggering

Checks

[X] Not a duplicate.
[X] Not a question, feature request, or anything other than a bug report directly related to React Splide. Use Discussions for these topics: https://github.com/Splidejs/splide/discussions

Version
0.5.3
Description
I have the following code:
<Splide
    onMove={() => console.log("move")}
    onMoved={() => console.log("moved")}
    onMounted={() => console.log("mounted")}
    onActive={() => console.log("active")}
>
    {/* ... */}
</Splide>

And active is triggered once at the beginning, as well as mounted.
move and moved is triggered never.
Reproduction Link
No response
Steps to Reproduce

Use the implementation above to reproduce

...
Expected Behaviour
All events should be triggered.

It looks working in 0.5.3.
https://codesandbox.io/s/event-8jus5
I've also tested the latest version and found no bug.

Hi @NaotoshiFujita,
thanks for the response + your example.
I found the issue: the events don't work anymore, as soon as you sync two sliders, see here:
https://codesandbox.io/s/event-forked-e6vsm

Thank you for the additional info. I've fixed the bug(^0.6.8).

I just noticed I never actually saved my example, sorry about that.
Thanks so much for the update!
I just updated the example to the new version, but now the sync is not working properly anymore?

In 0.6.7 the transitions are properly working, but the events are not triggered.
In 0.6.8 the events are triggered, but the images don't change (just the dots)

Okay. I will check it again 😥

Fixed in 0.6.9!

Looks good, thank you so much @NaotoshiFujita ! 🙌


Experimental escape mode for use with client-side katex.js

fix #109

@rogeryoungh, I made the changes I wanted. Please have a look.
If you don't have any issues, I will squash this into master.

The next step would be to add build instructions and potentially GitHub Action.

The only problem is the readme. When the title is changed to "Escape mode (experimental)", the link should be #escape-mode-experimental.

🎉 Great work, @rogeryoungh! Thank you for your contribution.


Avoid printing "Passphrases do not match" when passphrase is too short

Also, wrap the passphrase instructions paragraph at 80 columns, and
change the passphrase variable name in addKey to avoid a conflict with
the package name.
Signed-off-by: Aaron Lehmann<EMAIL_ADDRESS>

LGTM


Incorrect ID for signs and odd .* behavior

https://griefprevention.github.io/debug/?pkMoBlf0Np
If you look at this debug here, you can see that minecraft.oak_wall_sign is being used for the ID rather than the correct oak_sign ID. This is true for all signs as far as I can tell.
The issue with .* flags can be replicated by giving setting interact-block-primary.* to false and adding specific context flags set to true for individual blocks. For example, setting interact-block-primary.minecraft.oak_wall_sign to true will not allow you to interact with only oak sign's as intended.

These are all normal id's as per mojang's official id list. See https://minecraft.gamepedia.com/Sign


Added deleteTag for Model component

Dont understand how travis is failing when my junit and gradle headless all passed


Output of @properties.validator() functions on properties not saved

validator functions are able to coerce values, but the new values are not currently saved.

See the above commit - now property validators can coerce values and save changes to the change dictionary.


Overlapping IAPP label issue

ISSUE: When zooming into an area with many IAPP sites, the labels that currently have a white, semi-transparent background overlap and make it difficult to see the map underneath.
Suggested fix options (in order of preference by crystal):

[x] create a toggle on the side of the map to turn the labels on or off.
[ ] only have the labels show up when you get into a closer zoom level (ie. 200m or so?
[x] remove the white background of the labels and make them a bolded black or white colour so they only have text and not the white background - might be hard to see.

@micheal-w-wells please add comments/ideas/suggestions

what I did for second box is just not let user draw more than 1000 labels per layer at once, this stops screen from being covered in labels if zoomed too far out.
for data issues like many sites nearly on the same point - we'll have to keep thinking about how to do that.


programmatically set value on form builder textfield

hi , i'm doing auto generated Id to set on form builder textfield every time i press on submit button, it will generated a new id , how can i programmatically set a new id that generated on form builder textfield?
i tried this _formKey.currentState.fields['autoId'].didChange('$autoId'); it isn't work for me

Let me see if I understood your scenario
You have a field with a name/id like
FormBuilderField(name: 'first_id');
Then, on pressing a button, you generate a second id, without generating a new field?
So, you are trying to update the aforementioned field using the new id? Expecting something like:
FormBuilderField(name: 'second_id');
?
At this time, it seems the field with new id is not registered in the form. But, may I ask you, why are you changing the id of a field?

@WilliamCunhaCardoso it isn't what i want , i didn't want to change name of the field i need to change the value

I haven't been able to reproduce your error. This works for me:
FormBuilder(
  key: _formBuilderKey,
  child: FormBuilderTextField(
    name: 'text',
    decoration: const InputDecoration(labelText: 'Text'),
  ),
),
ElevatedButton(
  onPressed: () {
    _formBuilderKey.currentState!.fields['text']
        ?.didChange('New value');
  },
  child: const Text('Change value'),
),

Kindly confirm that the name of the attribute you are referencing in _formKey.currentState.fields[*] matches the name attribute of the FormBuilderTextField

@danvick solved by TextEditingController


Testing Redis Cluster 3.0.300 and having issues

First of all I had a cluster with 3 masters. I wanted to make one of them a slave so I used the CLUSTER REPLICATE command but it kept telling me I could not do it because there were keys in there.
I eventually got frustrated and did a bunch of CLUSTER RESET and CLUSTER MEET and eventually I ended up with 1 master and 2 Slaves but I can't get the cluster to not be in failed mode. This is the CLUSTER INFO now:
cluster_state:fail cluster_slots_assigned:0 cluster_slots_ok:0 cluster_slots_pfail:0 cluster_slots_fail:0 cluster_known_nodes:3 cluster_size:0 cluster_current_epoch:3 cluster_my_epoch:1 cluster_stats_messages_sent:1139804 cluster_stats_messages_received:1139804
What should be my next step?
Thx

Closing this issue since 3.0.300 is not supported anymore (latest release is 3.0.501).
Thank you.


[Error] Unable to create body for email message

[Error] Unable to create body for email message
Traceback (most recent call last):
File "kijiji.py", line 250, in
main()
File "kijiji.py", line 247, in main
scrape(url_to_scrape, old_ad_dict, exclude_list, filename, skip_flag)
File "kijiji.py", line 198, in scrape
MailAd(ad_dict, email_title) # Send out email with new ads
File "kijiji.py", line 132, in MailAd
msg = MIMEText(body, 'html')
File "/usr/lib/python2.7/email/mime/text.py", line 30, in init
self.set_payload(_text, _charset)
File "/usr/lib/python2.7/email/message.py", line 226, in set_payload
self.set_charset(charset)
File "/usr/lib/python2.7/email/message.py", line 262, in set_charset
self._payload = self._payload.encode(charset.output_charset)
UnicodeEncodeError: 'ascii' codec can't encode character u'\u2019' in position 451: ordinal not in range(128)

Yes, I was using 2.7.  I used version 3 and the script is working as expected.  Thank you.


Disable DTR for Ch340

Hello! I use your library to work with arduino. When connected to the port of Arduino, Arduino reboots. I tried serial.setDTR (false); - but it did not help me.

It is very important for me that Arduino does not reboot when connected to it.

@Blynskyniki DTR is disabled by default


fix: enable crawler for all mapped campuses as a workaround

Describe the pull request
This pull request addresses an issue where certain mapped campuses have invalid locations. As a workaround to ensure the accessibility and accuracy of campus data, we've opted to enable all crawlers for all mapped campuses.
Checklist

[x] I have made the modifications or added tests related to my PR
[x] I have run the tests and linters locally and they pass
[x] I have added/updated the documentation for my RP

Resolve #502


exclude polyfills for specific dependencies

Hi there, is there a way to exclude polyfills for a specific dependency ?
I've had issues with a library because it thinks it's running in node due to the process polyfill. In the source code of this repo banners are used to inject the process polyfill and according to esbuild docs there isn't a way to filter which files to inject the banner so i got stuck
So far i've tried

Excluding the dependency from optimizeDeps (didnt work because the library has require calls)
Doing a dynamic import instead (didn't work)

Sure thing. You can exclude imported polyfills with the exclude option, and you can exclude global polyfills (Buffer, process, or global) with the globals option. Check out this section of the readme.

Thanks for responding. I think i got misunderstood. I need the polyfills in some of my dependencies but others do not like it(issue explained in my first comment), is there a way to exclude polyfills file based ?


”Zăvoi”, ”Măgura”, ”Măru”, ”Poiana Mărului” are listed in the drop-down menu

Description:
When user writes ”Zăvoi”, ”Măgura”, ”Măru”, ”Poiana Mărului” the locations appear in the drop-down menu next to the comune and the county witch is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Zăvoi"

Expected results
The location appears in the drop-down menu as "Sat Zăvoi, CARAȘ-SEVERIN(Zăvoi)".

Step 3
Write in the search bar "Măgura"

Expected results
The location appears in the drop-down menu as "Sat Măgura, CARAȘ-SEVERIN(Zăvoi)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Măru"

Expected results
The location appears in the drop-down menu as "Sat Măru, CARAȘ-SEVERIN(Zăvoi)".

Step 6
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 7
Write in the search bar "Poiana Mărului"

Expected results
The location appears in the drop-down menu as "Sat Poiana Mărului, CARAȘ-SEVERIN(Zăvoi)".


Added searchResources and searchOne helpers to FhirRepository

This is a 100% pure refactoring / cleanup PR.
By default, search() calls return a FHIR Bundle.  Bundle is a wrapper type.  The meat of the results is inside Bundle.entry.  Bundle also includes a bunch of metadata (total count, next/previous links, etc).  However, in the common case, you just want the entry values.
In MedplumClient we have 2 extremely handy convenience methods: searchResources and searchOne.  Where the normal search() method returns a FHIR Bundle, searchResources returns a simple Resource[] array and searchOne returns a single Resource.
In the server, we did not have those convenience methods, and so we ended up doing a lot of bundle.entry?.map(...) calls.
This PR finally gets around to adding the convenience methods, and using it in a bunch of places.  I don't think I found all of them, so there might be some ongoing opportunities to replace search() + map() with searchResources().

Coverage: 93.96% (-0.04%) from 93.996% when pulling 9149bb84200933ca5060d6626ffb6f41d1657621 on cody-base-repository-search-resources into 22769370e46e4a357cfb4b136704b9629da6a668 on main.


Failed to build image for python 2 by "./gradlew :core:python2Action:distDocker"

The error message said python2Action was not found under core, but we actually have it. The Travis job failed at https://travis-ci.org/apache/incubator-openwhisk-runtime-python/builds/532323937?utm_source=github_status&utm_medium=notification.
The command to build python 3 runs well at "./gradlew :core:python3AiAction:distDocker", since we have python3AiAction.

OK. Python 2 will sunset in probably 6 months, so we decided to not keep python 2 build any more.

This runtime is gone.


demo.ipynb: Error in output extraction, sequence item 1: expected str instance, BWStateToT found

model:
TheBloke/Llama-2-70B-GPTQ
the final code block:
with open('examples/CoT/blocksworld/prompts/pool_prompt_v1.json') as f:
prompt = json.load(f)
evaluator = BWEvaluator(config_file='examples/CoT/blocksworld/data/bw_config.yaml',
domain_file='examples/CoT/blocksworld/data/generated_domain.pddl',
data_path='examples/CoT/blocksworld/data/split_v1/split_v1_step_4_data.json',
init_prompt=prompt)
evaluator.evaluate(reasoner_tot, shuffle_prompt=True, num_shot=4, resume=0)
Result: All of the outputs are empty strings?
Error in output extraction, sequence item 1: expected str instance, BWStateToT found
[+]: Saving plan in tmp_plan.txt
RESPONSE:::
Case #1: correct=False, output='', answer={'init': 'the blue block is clear, the hand is empty, the red block is on top of the yellow block, the blue block is on top of the orange block, the orange block is on top of the red block and the yellow block is on the table', 'goal': 'the orange block is on top of the blue block', 'plan': '\nunstack the blue block from on top of the orange block\nput down the blue block\nunstack the orange block from on top of the red block\nstack the orange block on top of the blue block\n[PLAN END]\n', 'question': '\n[STATEMENT]\nAs initial conditions I have that, the blue block is clear, the hand is empty, the red block is on top of the yellow block, the blue block is on top of the orange block, the orange block is on top of the red block and the yellow block is on the table.\nMy goal is to have that the orange block is on top of the blue block.\n\nMy plan is as follows:\n\n[PLAN]\n', 'instance_file': 'LLMs-Planning/llm_planning_analysis/instances/blocksworld/generated_basic/instance-176.pddl'};accuracy=0.000 (0/1)

Hi, sorry for the bug. We have updated the demo.ipynb.
The reason is that output_extractor was missing when initializing the BWEvaluator object. This function will transform the search results into the action sequence which can be understood by the evaluator. For more use cases, please refer to 1, [2] (https://github.com/maitrix-org/llm-reasoners/blob/main/examples/CoT/blocksworld/cot_inference.py)

thanks


Suggestion: web page that lists supported controllers

It would be useful for developers to be able to point people to a simple web page that lists the controllers that their SDL game supports. Maybe this repo could add a web page (with GitHub Pages) that lists the supported controllers given a commit hash (and optionally SDL version) by parsing the database from the given commit.

You can use sed and a simple regex to extract the names from the db.
Feel free to make a website that does such a thing. It is out-of-scope for this project.

Thanks for the quick response!


Incomplete code breaks parser assumptions therefore not recovering as the optimal way

Description:
 function foo() {
    xml a = xml
}

Above test case fails at,
https://github.com/ballerina-platform/ballerina-lang/blob/8322e00a8d67114f7c0a7e28c49f020d4cf15123/compiler/ballerina-parser/src/main/java/io/ballerina/compiler/internal/parser/BallerinaParserErrorHandler.java#L2357
Therefore recovering as,
 function foo() {
    xml a =  MISSING[any] MISSING[is]xml
 MISSING[;]}

We could rather recover this as an xml-template-expr.

Duplicate of #30002


get image from IOU when there is no source in transaction

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/29855
PROPOSAL: https://github.com/Expensify/App/issues/29855#issuecomment-1768484023
Tests

Click on FAB
Click on Request Money > Manual
Input an mount and click Next
Enter/select an option for the request
Click on the three dot menu at top right
Click on "Add Receipt"
Click on "Choose File"
Select an image
Verify that the image is shown as receipt

[x] Verify that no errors appear in the JS console

Offline tests

Click on FAB
Click on Request Money > Manual
Input an mount and click Next
Enter/select an option for the request
Click on the three dot menu at top right
Click on "Add Receipt"
Click on "Choose File"
Select an image
Verify that the image is shown as receipt

QA Steps

Click on FAB
Click on Request Money > Manual
Input an mount and click Next
Enter/select an option for the request
Click on the three dot menu at top right
Click on "Add Receipt"
Click on "Choose File"
Select an image
Verify that the image is shown as receipt

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native
https://github.com/Expensify/App/assets/102477862/e8caea96-7d02-40e2-9dce-8ca56f25fa3e

Android: mWeb Chrome
https://github.com/Expensify/App/assets/102477862/34524902-95fb-4cd3-9330-7f35daa92cc5

iOS: Native
https://github.com/Expensify/App/assets/102477862/ef041685-e958-47e6-b292-a6cbcef7075d

iOS: mWeb Safari
https://github.com/Expensify/App/assets/102477862/b0992216-41ed-4392-a2dc-f3b2a8d89664

MacOS: Chrome / Safari
https://github.com/Expensify/App/assets/102477862/dedc80ed-b062-43c0-864e-004505c6322f

MacOS: Desktop
https://github.com/Expensify/App/assets/102477862/835396e0-2eca-4af5-98bb-790d0b86d762

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Cherry-picked to staging by https://github.com/francoisl in version: 1.3.86-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/francoisl in version: 1.3.86-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/cristipaval in version: 1.3.88-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/chiragsalian in version: 1.3.88-11 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


Register other cases with crates.io

I've never submitted a crate, so I don't know that it's possible, but after spending a few minutes wondering why on Earth cargo wasn't finding inflector = x - for x as anything I could think to try - I think it would be a great idea to register 'inflector' as an alias for the crate 'Inflector', if that's possible?
If it's not possible, then perhaps consider changing the repo name to 'Inflector', so they're at least consistent?

Changed the repo name to Inflector instead of inflector. Thanks for the feedback, I honestly thought that I had named them the same.


Fix internal phpstan error

Closes https://github.com/Jan0707/phpstan-prophecy/issues/315

Thank you, @ivan-wolf and @jseparovic1!

@ivan-wolf @jseparovic1
The build is broken now. I have no interest in maintaining this project anymore.
Please refer to https://github.com/Jan0707/phpstan-prophecy/issues/285 and try to reach out to @Jan0707.


Add Getting Started section to Readme

This is probably obvious for most React devs but might not be so straightforward
to someone not used to a Node build process. Thoughts?

Great idea, and looks good to me. Could you add a reference to that section in the table of contents? Happy to merge it in if you include that. Thanks!

Updated! Thanks!

Thanks, merged!


Add a favicon

Description
A favicon would be nice and, in the interests of correctness, most browsers usually request a favicon.ico by default (which produces 404s when absent).
Tasks

Add a favicon

Closed by a4d824f.


feat: Reorganize manifests for CI based releasing

Supports https://github.com/open-services-group/peribolos-as-a-service/issues/70
Resolves manifests side of things. There will be a separate PR for CI. We have 3 overlays now:
dev: for manual deployment
stage: deploys released images, that will be handled by CI. Will be ArgoCD managed, requires a new namespace and app
prod: deploys promoted tags after release - this promotion will be handled by CI. Will be ArgoCD managed.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please ask for approval from tumido after the PR has been reviewed.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@tumido: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

pre-commit
6dbee0d2b470e814d6823608c22670bb13715881
link
true
/test pre-commit

Full PR test history. Your PR dashboard. Please help us and open an issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/cc @harshad16
please review :slightly_smiling_face:

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: harshad16
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [harshad16]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


saleor-checkout missing credit card form

Summary
I finally was able to get the SPA running and configured today for saleor-checkout, however I can place orders, there seems to be no way to pay for them.

Steps to reproduce
Latest Canary -- Follow the steps to setup everything and add the SPA in the dashboard and configure it.
A link to a reproduction repository
No response
Output from envinfo
No response

Can you elaborate on the problem? What do you mean by "no way to pay"? I can see payment methods and "pay" button in the screen shot. Do you get any errors when you click on it?

@mmiszy There is no credit card form regardless of the payment method -- I can click the pay button a checkout that is not paid for, and the subsequent button to pay does nothing.

What subsequent pay button?
Can you produce a full reproduction, including steps and description?

Add to card

Go to the saleor-checkout-app with said card

See the image where there is no credit card form.

Click Pay

See this page and get an E-mail that appears like you already paid:

Click the pay for order button which gets a 500 button because no stirpe webhook secret, but that was never required before.
Grep the code base for cardnum and see there is no form field for credit card number.

I am asking about if the credit card form even exists?

Ok I just figured out that if I add a webhook URL that it uses the stripe checkout website -- that is fine, but are there plans to add a credit card form as an alternative?

Is there a way to have the stripe checkout not auto-capture the payment?

same problem, i don't understand something maybe?


Menu button page object

Pull Request
🤨 Rationale
Create menu-button page object that is exported from nimble-components and from nimble-angular.
👩💻 Implementation

Create page object for the menu-button
Create new menu-button/testing entry point from nimble-angular and export the page object from there
Update menu button unit tests in nimble-components to use the page object where necessary

🧪 Testing
Unit tests still pass
✅ Checklist

[ ] I have updated the project documentation to reflect my changes or determined no changes are needed.

@m-akinc, will you buddy this PR for me?


CC0 not being recognized when following creativecommons.org instructions

First I'm sorry if this is the wrong place, the root of this issue is that choosealicense uses a slightly edited version of CC0 compared to the one CC distributes, so this might be the wrong place.
Currently licensee cannot detect CC0 added as per the instructions at https://wiki.creativecommons.org/wiki/CC0_FAQ#May_I_apply_CC0_to_computer_software.3F_If_so.2C_is_there_a_recommended_implementation.3F instead giving the following output
License file: COPYING
License hash: f5b7dd1f4796769f765ab4b8d40304c2bf1bcc47

Here's the closest licenses:
* LGPL-3.0 similarity: 29.60%

The reason for this appears to mostly be the disclaimer, if you remove that then it gets correctly recognized
License file: COPYING
License hash: f2d25c552d35d3e35b5041105b993da5a4b058c9
License: Creative Commons Zero v1.0 Universal
Confidence: 98.59%
Method: Licensee::Matchers::Dice

The license text used by licensee was introduced in https://github.com/github/choosealicense.com/pull/126 (licensee vendors choosealicense.com license texts) presumably by copying and pasting from https://creativecommons.org/publicdomain/zero/1.0/legalcode though I'm not sure how to explain the link at the end. Probably should be updated with https://creativecommons.org/publicdomain/zero/1.0/legalcode.txt or https://spdx.org/licenses/CC0-1.0.html ... latter marks disclaimer as optional and doesn't have first line "Creative Commons Legal Code" (probably should be present there but also marked as optional).
I'm not sure what the right solution for licensee is at this point given it doesn't know anything about optional sections, but I made https://github.com/github/choosealicense.com/pull/488 updating the license text from https://creativecommons.org/publicdomain/zero/1.0/legalcode.txt in case that's useful for testing.

why bot

Any chance this could be reopened? The issue of CC0 not being recognised is still happening.


navview data-on-menu-item-click no longer works as supposed

Starting from Metro UI CSS v4.4.0 data-on-menu-item-click is not working as supposed.
When click on menu items, this references navview node itself not the actual menu item clicked
codepen of v4.3.10
codepen of v4.4.0 and greater

use data-on-menu-item-click="onmenuitemclick"

https://metroui.org.ua/events.html

Great. Thanks olton


linux-8 - Replacing old G4 phone with new Pixel phone due to battery is starting failure.

Type of Request
bug
Infrastructure Environment
LUCI, Github, Cocoon scheduler, Autosubmit, etc...
What is happening?
linux-8 - Replacing old G4 phone with new Pixel phone due to battery is starting failure.
Steps to reproduce
Step 1:
Step 2:
..
Step n:
Expected results
I expect to see X when Y is finished.

Replacing old G4 phone with new Pixel 7 phone, due to battery on old ready to burst out.


update version v3 in template

update version v3 in template ,like this
from
"github.com/yunify/qingstor-sdk-go/config"
to
 "github.com/yunify/qingstor-sdk-go/v3/config"
Signed-off-by: Ubique0305<EMAIL_ADDRESS>

specs/qingstor should not be changed in this PR.


Fix #2233, Update data types in CFE msgs

Checklist (Please check before submitting)

[X] I reviewed the Contributing Guide.
[X] I signed and emailed the appropriate Contributor License Agreement to<EMAIL_ADDRESS>and copied<EMAIL_ADDRESS>
Describe the contribution
Fix #2233, Changed data types of some fields in certain message packets
Testing performed
Unit Testing
Expected behavior changes
no impact to behavior
System(s) tested on

OS: Ubuntu 18.04

Contributor Info - All information REQUIRED for consideration of pull request
Haven Carlson - NASA

Agreed - I just double checked and the intent was to update the EDS files to match FSW/the current database such that everything is in sync.


added support for Alutherm 2500, made Schedule optional

Fixed Alutherm config by making Schedule DPS optional, added support for 2500 device variant. Tested with the Alutherm 2000.
Closes #1315

Sorry, I forgot to push yesterday when I made the same change.


Display the verbose gc from entire time

Currently you still have to run any alternative tool to analyse verbose gc (e.g. gcmv). I would like to be able to see entire verbose gc in javacore analyser.
As a solution there can be a separate page generated only with verbose gc data.

I tried it with 100MB verbose gc but the charts did not appear.
waitData-many-verbose-gc2.zip
It seems to be a performance issue with JQuery.


feat: let user specify more parameters for storage

Purpose
Let user specify more parameters for storage. Closes #380.
What the code is doing

In abstract_grid.py: we add parameters LossFactor, terminal_min, and terminal_max.
In change_table.py: we refactor add_storage_capacity to follow the pattern set in add_plant etc.
In grid.py: ~we add the Lossfactor parameter to the ignored keys for grid.storage in the grid equality check.~ we automatically populate the ignored keys for grid.storage in the grid equality check from the new storage.py.
In transform_grid.py: we interpret the new change table schema.
We add a new file storage.py to handle usa_tamu storage constants.
In usa_tamu_model.py: we get our storage default values from the new storage.py file.
We update test_change_table.py and test_transform_grid.py accordingly

Testing
Unit tests are updated to the new format and now pass.
Time estimate
15-30 minutes. New tests to ensure that the new parameters are set properly are still incoming, but review can get started on everything else.

In addition to adding the LossFactor parameter, I've also added two more parameters, terminal_min and terminal_max, which constraint the amount of energy that the storage unit must end the interval with. Previously, this had been hardcoded to be between 50-100%; now the default values are 0-100% but the user can specify anything they wish.

In addition to adding the LossFactor parameter, I've also added two more parameters, terminal_min and terminal_max, which constraint the amount of energy that the storage unit must end the interval with. Previously, this had been hardcoded to be between 50-100%; now the default values are 0-100% but the user can specify anything they wish.

It reminds me something ...


Update assertion in property function

I believe the assertion in the property function should be using ~= for getter. Currently it is allowing the following calls without an assertion being raised:
property()
property({})

I also added "for property" to the error message since the stacktrace doesn't help you know where the issue is coming from since it gives you a line into the bundled amalgamated source. "for property" is maybe not the best wording but I think it helps.

Good eye! (oops =P)
Loading scripts has really bad error handling, I'll add a stack trace to it!


⚠ Bofrost.nl has degraded performance

In 2b0da7f, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 2199 ms

Resolved: Bofrost.nl performance has improved in 1acfb69.


How can I get the dataset？

Thx for your works. Can you tell me how to get the dataset？

Sorry for the late reply.
You can find the dataset in Kaggle datasets.


Add benchmark and estimators tests

Adds test cases for benchmark (smoke tests checking exit code 0) and for whitebox estimators through estimate_uncertainty.

Github Action gives the following warning:
=============================== warnings summary ===============================
test/test_estimators.py::test_lexical_similarity_bleu
/opt/hostedtoolcache/Python/3.10.15/x64/lib/python3.10/site-packages/nltk/translate/bleu_score.py:577: UserWarning:
The hypothesis contains 0 counts of 2-gram overlaps.
Therefore the BLEU score evaluates to 0, independently of
how many N-gram overlaps of lower order it contains.
Consider using lower n-gram order or use SmoothingFunction()
warnings.warn(_msg)


Docker CD Failing

Describe the bug
Docker CD fails
To Reproduce
Steps to reproduce the behavior:

Run Docker CD through a commit or pr

Expected behavior
Successful build and upload of Image

at this point i don't know what to do

fixed as of a0f22f9


Nav component shows scrollbars

Particularly noticeable on Windows:

This is also happening in dropdowns. Both should be fixed.

You can force MacOS to show scrollbars btw, and it's the same problem:


Two questions: How can I access all the data points of the camera's pose positions? Can I load a map?

Question 1: Is there a way I can access the coordinates of all the yellow pose positions?

Question 2: Can I load a map and localize the camera within the map?
Thanks in advance!

1: Yes you can! These are the keyFrames in the Mapping API output. In the replay_to_nerf.py script, they are used like this https://github.com/SpectacularAI/sdk-examples/blob/8ef5ae723a4442a2946bf1ee117ae6a4612d2367/python/mapping/replay_to_nerf.py#L278-L294
They are also stored to disk by the same script, in a Nerfstudio-compatible format and coordinate convention (COLMAP's output format), but if you want to use them for some other purpose, it's probably easiest to modify the replay_to_nerf script where everything is already correctly parsed and accessible in a documented coordinate convention
2: Possible, but currently experimental and not well-documented: You would first have to change the script to output the map in our internal .bin format (i.e., change this line from .csv to .bin, which breaks the rest of the script, but gives you a localizable map), which can then be loaded by the SDK for visual positioning. (Example here)

Awesome, thank you so much @oseiskar! I love your enthusiasm :+1:

Hey @oseiskar, quick question: can I somehow get the coordinates of the keyframe within the global map? Also, is it possible to update a loaded map with more coordinates?

Key frame (camera) coordinates are a part of the Mapping API output. The sparse co-visibility graph (which KFs share sparse key points) is not available in the current API

Thanks!


Helps when class name is a string and not an object

#patch#

Visual Acceptance tests:
No changes

Approved


[FEA] Add spark 3.4.1 snapshot shim

Is your feature request related to a problem? Please describe.
Add spark 3.4.1 snapshot shim

3.4.1 is likely to be out in the next couple weeks.

Spark 3.4.1 has been released: https://www.apache.org/dyn/closer.lua/spark/spark-3.4.1/spark-3.4.1-bin-hadoop3.tgz


What does the inv_temp variable mean?

i was looking at the code from
https://github.com/Parskatt/DeDoDe/blob/main/demo/demo_match.py
and at line 42 there is a variable named inv_temp  but there isn't much description for what it is.
Can someone clarify?

We match with softmax over scaled cosine similarly. We call the scaling inverse temperature. People often refer to temperature when scaling logits.


[YouTube Downloader] ...its showing download failed

Extension
https://www.raycast.com/vimtor/youtube-downloader
Raycast Version
1.73.2
macOS Version
No response
Description
Download failed
Steps To Reproduce
.
Current Behaviour
No response
Expected Behaviour
No response

Fixed in https://github.com/raycast/extensions/pull/13939.

@raycastbot close this issue


How to revoke token access

Hi there, less of an issue and more of a question..
I used your service this morning but now I wish to revoke the access/refresh token that was generated and that your site saw.
Is this possiblefrom azure devops? It doesnt appear to be (although my organisation might be disallowing it?).
I know you claim that your site doesn't see it but the security consious man in me wants to really make sure that is the case.
Cheers,
Ed

Totally respect the skepticism, it's the right security stance for sure. You can revoke authorization for an app like so: https://intercom.help/appcenter/en/articles/1338717-i-have-connected-to-the-wrong-azure-devops-account-how-can-i-re-connect-to-the-correct-one


MOL-619: CustomerService should use compatibility gateway

@boxblinkracer Why does it keep breaking during plugin install? It seems to trip over Shopware's own classes.

@boxblinkracer Is this still even necessary now that the minimum version of the plugin is <IP_ADDRESS>?


update-user api crashed when update fields include displayName

I found this error when using nodejs sdk to update displayName field. And I test it in admin dashboard by modifying request body in dev tool. Modifying some other fields is ok, only displayName will cause error.
Error log:
2024/09/09 07:52:11.603 [I]  (***: ) subOwner = built-in, subName = admin, method = POST, urlPath = /api/update-user, obj.Owner = ***, obj.Name = ****, result = allow
subOwner = built-in, subName = admin, method = POST, urlPath = /api/update-user, obj.Owner = ***, obj.Name = ***, result = allow
2024/09/09 07:53:08.233 [I]  (***: ) subOwner = built-in, subName = admin, method = POST, urlPath = /api/update-user, obj.Owner = ***, obj.Name = ***, result = allow
2024/09/09 07:53:08.235 [C]  the request url is  /api/update-user
2024/09/09 07:53:08.235 [C]  Handler crashed with error runtime error: invalid memory address or nil pointer dereference
2024/09/09 07:53:08.235 [C]  /usr/local/go/src/runtime/panic.go:884
2024/09/09 07:53:08.235 [C]  /usr/local/go/src/runtime/panic.go:260
2024/09/09 07:53:08.235 [C]  /usr/local/go/src/runtime/signal_unix.go:841
2024/09/09 07:53:08.235 [C]  /go/src/casdoor/object/user_util.go:435
2024/09/09 07:53:08.235 [C]  /go/src/casdoor/controllers/user.go:303
2024/09/09 07:53:08.235 [C]  /usr/local/go/src/reflect/value.go:586
2024/09/09 07:53:08.235 [C]  /usr/local/go/src/reflect/value.go:370
2024/09/09 07:53:08.235 [C]  /go/pkg/mod/github.com/beego/beego@v1.12.12/router.go:897
2024/09/09 07:53:08.235 [C]  /usr/local/go/src/net/http/server.go:2936
2024/09/09 07:53:08.235 [C]  /usr/local/go/src/net/http/server.go:1995
2024/09/09 07:53:08.235 [C]  /usr/local/go/src/runtime/asm_amd64.s:1598

@slow-groovin Could you please describe the steps to reproduce this bug in detail?
I used the following code to update the user's displayname and email, and the result was correct and the user information in my database was successfully changed.
import { SDK } from 'casdoor-nodejs-sdk'

// init
const authCfg = {
  endpoint: 'your endpoint',
  clientId: 'your clientId',
  clientSecret: 'your clientSecret',
  orgName: 'your orgName',
  certificate: `your certificate`,
}
const sdk = new SDK(authCfg)
const newDisplayName = "New Admin"
const newEmail =<EMAIL_ADDRESS>
// get-user
const { data: users } = await sdk.getUsers()
let originUser = users.data[0]

// update-user
originUser.displayName = newDisplayName
originUser.email = newEmail
const { data: updateResponse } = await sdk.updateUser(originUser)
if (updateResponse.data !== 'Affected') {
  throw new Error('Failed to update object')
}

// check update result
const {
  data: { data: updatedUser },
} = await sdk.getUser(originUser.name)
if (updatedUser.displayName !== newDisplayName) {
  throw new Error(
    `Failed to update object, description mismatch: ${updatedUser.displayName} != ${newDisplayName}`,
  )
}

console.log("end")

@slow-groovin your Casdoor is not latest version

@ZhaoYP-2001  @hsluoyz I update casdoor to the last version and start a replica of all service to compare to reproduce the problem and find the cause:   user setting page -> Account Items -> delete displayName , and then use update-user api  to update displayName will cause this error (I copy request from browser, user setting page, transfer it to curl and modified request body, it is same to using sdk)


Long term plan: Investigate strategies for non-owned types

Currently, it is impossible to have a S: Strategy where ValueOf<S> is non-owned.
Examples:

We have a base strategy T where ValueOf<T> is owned, and then we use that to make a strategy where ValueOf<S> = &'a ValueOf<T> - This will not type check.
Derived types, for example: Cow<'a, X>.

Generally, I think that, any type that has a lifetime can't be generated.
Being able to generate types with lifetimes in them is however useful when you have lifetimes somewhere deep in the hierarchy of a type. To be able to do this soundly, the ValueTree must perhaps have a notion of Owned (which defaults to Value) in addition to Value (which may be borrowed). Then TestRunner will return the Owned type instead of Value, but the function under test receives Value.
I tried to implement this and got TestRunner to compile, but no strategies would compile.
Solving this may require generic associated types (GAT). I'll experiment with it once we have GAT in nightly.
There's also *const T and *mut T. While it is possible to give a sound strategy for these by Box::leak, this may cause problems since tests for large data types may leak huge amounts of memory. This could perhaps be mitigated by letting the ValueTree store the previous value of .current(), and whenever .current() happens, the old value is deallocated. The last value needs to be persisted in the TestRunner somehow if the entire ValueTree did not fail so that it can be cleared.
I'd like your thought's on the desirability of these features.

I do think it'd be useful, but I also agree that this definitely requires GATs. (If trait aliases become a thing, there might also be a viable option of just adding a lifetime parameter to all the traits that most people wouldn't have to deal with.)
I think the problem bears a lot of similarity to "streaming iterators", so it might be best to wait and see how that gets solved and then follow suit.

Agreed, it is similar to streaming iterators. Once GATs lands I will do some prototyping on nightly to see how much the interface has to change to accommodate generation of borrowed types and how much we have to modify the strategies already provided.
If we find a good solution we can perhaps develop it as a separate branch and then merge it in once GATs for lifetimes are stabilized (which might be before GATs for types).

Hmm GATs would clearly be the correct way to represent this, but I'm curious as to why this wouldn't work if Strategy and ValueTree were generic on 'a.
It feels to me it would work, with ValueTree holding the owned data and generating the non-owned structures. current on ValueTree would take &'a self.
We would always implement ValueTree and Strategy for every 'a.

It’s be lovely to revisit this issue now that GATs have been stabilized.  Has anybody experimented with it? :)


Unit tests : setup

Goal :
Have the necessary parts to write and run unit tests on components
Technical requirements

the chosen framework for tests is Jest
Add instructions in the readme about how to run the test suite

Next steps :

Write unit tests for existing components  ( #33 )
Write unit tests for each new component

PR: #37


Fallback to LMT on missing source hash

Description

Feature / Bug Fix: (Brief description of the feature or issue being addressed)
For sync, when source hash is missing, AzCopy should fall back to LMT comparison instead of skipping the file.
AzCopy sync --compare-hash should fall back to LMT if MD5 not present on source and destination

Related Links:

Issues

Team thread

Documents

[Email Subject]

Type of Change

[x] Bug fix
[ ] New feature
[ ] Documentation update required
[ ] Code quality improvement
[ ] Other (describe):

How Has This Been Tested?

I have added an e2e test that should cover all scenarios for source hash

hash present and equal
hash present and not equal
hash missing and source LMTafter destination
hash missing and source LMT before destination
Please see the comments in the test for more info

Thank you for your contribution to AzCopy!

LGTM.  However, one end-to-end test is failing, and the pull request requires a code sync.


Enable screenshot recording by means of an extension property

I have an Android Monorepo with about a dozen components and hundreds of modules, including many Robolectric UI tests. To avoid having to "enable" screenshot recording and report generation on a per gradle.properties basis I looked for alternatives to run the report generation of screenshots by default, but could not find any suitable. I make heavy usage of build convention plugins, so the easiest was to have a custom "my.plugin.test.screenshots" build convention that would apply your plugin and, at the same time, enable screenshot reporting by default via the plugin's extension for the respective test tasks.
I figured it makes no sense to have such a configuration for the verify and compare tasks; my primary use case really is to automatically see what's wrong when a Robolectric UI test fails with some assertion, to aid debugging the problem. While one can omit the usage of the Gradle plugin altogether and just use the core library (together with a System.setProperty("roborazzi.test.record", "enabled") somewhere), I found it useful to re-use the HTML reporting capability that the plugin provides.
From what I see Roborazzi seems very feature-rich, eventually in the future things could get a bit more separated, so one thing does not require the existence of another.

Thanks. I'm interested in understanding more about your use case for managing screenshots in Roborazzi tests. Specifically, in what situations would you prefer setting recordEnabled = false instead of not applying the plugin at all? Could you elaborate on the scenarios or reasoning behind this choice?

Please take a look at the test report
FAILURE: Build failed with an exception.

* Where:
Build file '/private/var/folders/n2/pt_35rc53tdgkld9531s2tfh0000gn/T/junit4772681132119589584/app/build.gradle.kts' line: 83

* What went wrong:
Script compilation error:

  Line 83: }roborazzi {
            ^ Unresolved reference. None of the following candidates is applicable because of receiver type mismatch:
                public fun Project.roborazzi(configure: Action<RoborazziExtension>): Unit defined in org.gradle.kotlin.dsl

1 error

* Try:
> Run with --debug option to get more log output.
> Run with --scan to get full insights.

Please take a look at the test report
FAILURE: Build failed with an exception.

* Where:
Build file '/private/var/folders/n2/pt_35rc53tdgkld9531s2tfh0000gn/T/junit4772681132119589584/app/build.gradle.kts' line: 83

* What went wrong:
Script compilation error:

  Line 83: }roborazzi {
            ^ Unresolved reference. None of the following candidates is applicable because of receiver type mismatch:
                public fun Project.roborazzi(configure: Action<RoborazziExtension>): Unit defined in org.gradle.kotlin.dsl

1 error

* Try:
> Run with --debug option to get more log output.
> Run with --scan to get full insights.

Ah, I was unable to run the tests locally, so I trusted on the CI, fixed the wrapping issue.

Thanks. I'm interested in understanding more about your use case for managing screenshots in Roborazzi tests. Specifically, in what situations would you prefer setting recordEnabled = false instead of not applying the plugin at all? Could you elaborate on the scenarios or reasoning behind this choice?

Well, this is a good question. IMHO whenever the plugin is applied it should actually record screenshots by default, because this is the foundation of what the plugin always does, collect the individual screenshots that were taken in tests and create an overall report for it.
The reason why I introduced a boolean property here is mainly backwards compatibility. Users of the plugin are used to the fact that they have to specifically tell the plugin to do anything, and I did not want to break that behavior. Also, since running these tasks in addition to what the test execution does there is surely a bit overhead that people eventually not want to pay each time. I'd expect that screenshot and non-screenshot tests live side-by-side in a module and most of the time the non-screenshot tests don't need any tasks to be run to collect data from them (or even let the screenshot code in the tests be executed).

Ok, I figured I miss something still. Even though screenshot recording and report generation works with this change, during test execution this branch is executed anyways:
  if (taskType.isEnabled() && !roborazziSystemPropertyTaskType().isEnabled()) {
    println(
      "Roborazzi Warning:\n" +
        "You have specified '$taskType' without the necessary plugin configuration like roborazzi.test.record=true or ./gradlew recordRoborazziDebug.\n" +
        "This may complicate your screenshot testing process because the behavior is not changeable. And it doesn't allow Roborazzi plugin to generate test report.\n" +
        "Please ensure proper setup in gradle.properties or via Gradle tasks for optimal functionality."
    )
  }

I originally thought I would tackle this by this here https://github.com/takahirom/roborazzi/pull/293/files#diff-dc9dacb4aa08c2bfedbc7cbce3ce02898ab79e36f1dad6a7d5c314f819f3cb5bR322, however, that does not seem to work. Would you happen to know if I'm missing something else?

If users are looking to use Roborazzi for screenshot tests, they'll likely need to run the verify or compare tasks. As I understand it from this PR, setting recordEnabled = true means recording will always occur, which could be somewhat confusing. We should consider the options' priority and relevant use cases more carefully. I'll need some time to think over these changes.

Hi @takahirom, have you thought about how to continue with this?

Thanks for the updates. While I appreciate the convenience this could provide, I believe we need to consider the overall API design more comprehensively.
Currently, we have several options for enabling recording:

roborazzi.test.record=true in gradle.properties
recordRoborazziDebug Gradle task
RoborazziOptions(taskType = RoborazziTaskType.Record) in code

This PR introduces recordEnable = true within the Roborazzi DSL, which diverges slightly by using the term "enable." This might cause confusion or overlap with the existing options, and it could potentially limit us to only "record" actions, which might not be clear.
Perhaps we could consider something like roborazzi { taskType = RoborazziTaskType.Record } to align with existing configurations. Additionally, establishing a clear priority order—gradle.property < Gradle DSL < task < RoborazziOptions.taskType—and documenting this could help clarify their usage and justify this addition.

@takahirom I made some changes to my original PR - was it this what you imagined here? Where would be a proper place to document the behaviour?

Sorry for being super late. I think it looks great, and I would like to merge this. However, I would like to have a test that checks the priority of the options with tasks and gradle.properties. For the documentation, you can add it here. Perhaps we can include this Gradle extension setting for the default behavior of Roborazzi for the Gradle module The explanation would be here  https://github.com/takahirom/roborazzi/blob/main/docs/topics/build_setup.md
Gradle extension setting for the default behavior of Roborazzi for the Gradle module The explanation would be here

Sorry for being super late. I think it looks great, and I would like to merge this. However, I would like to have a test that checks the priority of the options with tasks and gradle.properties.

This is where I stumble - because frankly I didn't fully understood the current way it is designed right now. From what I read in the code there are warnings if the task is executed, but the property is not present (see processOutputImageAndReport.kt:46ff), if the current work mode is already that tasks override gradle.properties settings, this seems to be in conflict with this warning. I also don't see an existing test that would test tasks overriding gradle.properties settings, this is all a bit confusing :-/.
I know my change adds even more complexity to an already complex implementation, and I start to feel bad about this.

For the documentation, you can add it here. Perhaps we can include this Gradle extension setting for the default behavior of Roborazzi for the Gradle module The explanation would be here  https://github.com/takahirom/roborazzi/blob/main/docs/topics/build_setup.md

I added preliminary documentation, added it however to the table, and added a small preface that explains the concept behind.

Hi @takahirom, any comment on my last comment? Thanks!


kfp 2.3.0 upgrade metadata-writer rock

Context
Upgrade the the rock for 2.3.0 kfp
Related to https://github.com/canonical/kfp-operators/issues/581
What needs to get done
Upgrade the rock
Definition of Done

Rock is upgraded for kfp 2.3.0
Rock is on Dockerhub

closed by #143


How to manually increment timestep of simulation?

Hello,
I am writing a custom reinforcement learning plugin with PyTorch and have what I hope is a simple question... I would like to define my own "step" function for my training agent, so that it can perform an action based on its observed environment and then control the number of timesteps before the next observation.  This way, I can calculate an accurate reward for the model.  I have not been able to find a great way to do this without causing the simulation to crash/freeze.  Perhaps I could just track the number of timesteps elapsed between observations, but I would prefer to manually force the simulation to move forward from my plugin.
Hope this makes sense.  Any feedback would be very helpful.  Thanks!
Eric

Dear Eric,
You could make sure your update function is called every time step (dt=0.05) and then count the number of steps in that function.  This should have the same effect.
Best regards,
Jacco Hoekstra
On 24 Mar 2021, at 19:39, epaulz-vt @.***> wrote:

Hello,
I am writing a custom reinforcement learning plugin with PyTorch and have what I hope is a simple question... I would like to define my own "step" function for my training agent, so that it can perform an action based on its observed environment and then control the number of timesteps before the next observation. This way, I can calculate an accurate reward for the model. I have not been able to find a great way to do this without causing the simulation to crash/freeze. Perhaps I could just track the number of timesteps elapsed between observations, but I would prefer to manually force the simulation to move forward from my plugin.
Hope this makes sense. Any feedback would be very helpful. Thanks!
Eric
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_TUDelft-2DCNS-2DATM_bluesky_issues_265&d=DwMCaQ&c=XYzUhXBD2cD-CornpT4QE19xOJBbRy-TBPLK0X9U2o8&r=5JxzVuDw_d0JV-kYyto54d3K2zKNIvniSIeVeSDtif4&m=WZt3nt6_XbxSFHxnntDWBWWAvkIdG3K7GmWKNaOzDzo&s=nroYnKr4gdMOYFs3tw2AQ4X3wgS3VkpC7eCUTssvc5s&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_ABWT2BBUC7GWYFJI7DELY3TTFIWUXANCNFSM4ZX2SYVA&d=DwMCaQ&c=XYzUhXBD2cD-CornpT4QE19xOJBbRy-TBPLK0X9U2o8&r=5JxzVuDw_d0JV-kYyto54d3K2zKNIvniSIeVeSDtif4&m=WZt3nt6_XbxSFHxnntDWBWWAvkIdG3K7GmWKNaOzDzo&s=zQkEh3vOYcO_avk8HQKR8kq7L-r67B37ljy4FojIBQM&e=.

Thank you... I will experiment with that method.
As a follow-up question, what would be the best way to have a function within the plugin run just once at the beginning of the simulation, but not run again after n timesteps?  E.g. a function called train_agent() to be called when Bluesky initializes with a scenario.  The function runs as long as the simulation is running, and within it runs x number of episodes.  It seems I can only tell a function to run periodically (e.g. every 5 seconds) but not just "run it once as soon as the plugin initializes".  If I try to run something before returning the plugin config, then my plugin code runs but the simulation is blocked until the plugin is finished running.
Thanks,
Eric

Hi Eric,
In addition to periodic functions you can also define new stack functions in your plugin. You can then call this function once at the beginning of each scenario file.
To add to your previous question: if bluesky and your RL code are running asynchronously and you want to ensure that the simulation waits for the RL agent to send a new command you can use the SCHEDULE and DELAY commands. For instance, doing
DELAY 10 HOLD
would tell the simulation to pause after 10 seconds simulation time. You could do the same for a custom stack function you define.
Cheers,
Joost

Great, I think this information should allow me to do what I'm planning.  Thank you!


Syz-manager: add simple email support

Users can specify an email address to reveive notifications when a
bug is discovered the first time, without setting up a full fledged
dashboard. Supported mailer is mailx.
Signed-off-by: Tim Tianyang Chen<EMAIL_ADDRESS>
Before sending a pull request, please review Contribution Guidelines:
https://github.com/google/syzkaller/blob/master/docs/contributing.md

it looks like I used a different email for git. I'm sending another one.


Issue35-REST API only

Pull request with REST API changes to do semgrep scans for multiple git URLs and multiple config files.
Checks if semgrep exists and if it does returns a json body for every config and git URL.
REST call details
Endpoint - /api/v1/staticCodeAnalysis
Request parameters - {
"Path":"https://github.com/harshkolhatkar/gitscanner,https://github.com/MrWong99/gitscanner",
"ConfigFiles":["golang", "r2c-ci"]
}
Comma separated values of Path and ConfigFiles

@MrWong99


Migrate notification service to PostgreSQL

Conflicts here.
Also work with Graham to enable Travis here.

Conflicts here.
Also work with Graham to enable Travis here.

@awasum I have added the Travis configuration but the one file causes the build to fail during rat tests.
@aasaru can you help me to get the rat report on Travis to rectify the problem?
I have done the rat check on my local dev and all files have the License header but strangely Travis complains about 1 file (I think it might be a log file).

Conflicts here.
Also work with Graham to enable Travis here.

@awasum I have added the Travis configuration but the one file causes the build to fail during rat tests.
@aasaru can you help me to get the rat report on Travis to rectify the problem?
I have done the rat check on my local dev and all files have the License header but strangely Travis complains about 1 file (I think it might be a log file).

All is well now. I simulated the build on my personal Travis account to get the report.
Upon second thoughts. Can we add an after_failure configuration to publish logs and reports somewhere? I think it will come in handy for other developers.

Great point, discuss it on the mailing list and see others options. Then
maybe create an issue after that.
On Sat, May 11, 2019, 23:52 Ebenezer Graham<EMAIL_ADDRESS>wrote:

Conflicts here.
Also work with Graham to enable Travis here.
@awasum https://github.com/awasum I have added the Travis configuration
but the one file causes the build to fail during rat tests.
@aasaru https://github.com/aasaru can you help me to get the rat report
on Travis to rectify the problem?
I have done the rat check on my local dev and all files have the License
header but strangely Travis complains about 1 file (I think it might be a
log file).
All is well now. I simulated the build on my personal Travis account to
get the report.
Upon second thoughts. Can we add an after_failure configuration to publish
logs and reports somewhere? I think it will come in handy for other
developers.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/fineract-cn-notifications/pull/9#issuecomment-491549798,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADWPAYAQWA6EOMVTKIHQPR3PU5E3PANCNFSM4HLW2SYA
.

This has merge conflicts.
Pull develop into this branch to be up to date.
Pull when this PR is merged: https://github.com/apache/fineract-cn-notifications/pull/11

Thanks @awasum. I observed that too and will figure it out.

I have resolved the conflicts in PR #17. It was  much simpler to do so.

Closing this now. Given #17  is now in.


Kubernetes Application View, quick open for URL

In the application details view, we list the ports an application deployment is listening on, but there is no "quick launch" or even a "copy" button..
We should add a "copy" button next to the port to copy to clipboard.

Hi. Can I take a shot at this?

@KevinMarioGerard for sure, if you have any question feel free to ask them directly here. Otherwise we'd be happy to review a PR.

Hello!
This issue has been up for a little bit. Would it be okay for me to work on this?

@fish-dango you'd be welcome ! We have not given any thoughts to this one yet so if you have questions before starting working on it feel free to ask!

Hello!
Just starting working this a little bit ago. I think I was able to potentially figure something out, but at the moment I was having a hard time creating a dev environment particularly for kubernetes. Is their extra documentation or resource for setting up our dev environments?

@ncresswell  Hey i am new to opensource contribution but i have an knowledge of Kubernetes and docker  can i try it?

We're currently actually working to improve the accessibility from the applications list, which will result in something like:

Although that won't impact the application details page itself. So that could still be useful to change.


Point links to Synth Secrets on archive.org

Changes to docs, since Sound On Sound removed a lot of Synth Secrets articles and the links were dead. Links now point at the relevant articles on Archive.org
And Atom removed a fair bit of whitespace at the same time.

awesome thank you!
looks like you're working off a slightly older version of the dev branch. Can you please pull the latest from dev so that you're not reverting some commits.

Sorry but I'm not seeing it. The forked dev branch is 4 commits ahead, 0 behind the source. I've got nothing to pull 😕

Just made a comment in the file. Looks like Tone.defaultArg was changed to this.defaultArg in your commit of MetalSynth plus a couple of other small changes

Coverage remained the same at 97.566% when pulling 986358119cec82b394dcdf7ad8fc48aaa570ca93 on ajrussellaudio:dev into a4032ad65c40f94520a231944a875d6ec85ca805 on Tonejs:dev.

Atom was still on the master branch after I'd switched to dev. What a mess, totally my fault, sorry about that. Should be all good now.


[REVIEW] Improving Hugectr notebook

Adding MultiGPU for NVT Preprocessing
Removing freq-limit and using HashBucket to achieve better accuracy in DLRM.

rerun tests

@benfred
I have removed the datatype comment as you said in the review.  I was waiting to Ronay's PR to merge HashBucket and Categorify operations, but it is taking longer than expected.
Let's get this PR in, and I will merge those ops in next week PR together with Python for HugeCTR v2.3.


Running jobs locally

Have you tried trouble shooting?
N/A
Agent Version and Platform
Agent version: 2.140.2
OS of the machine running the agent? OSX/Windows/Linux/...
Windows
VSTS Type and Version
VisualStudio.com
If VisualStudio.com, what is your account name?
http://uipath.visualstudio.com
What's not working?
As per this issue, we know that some time ago the agent supported running jobs locally (it was some kind of a hidden feature, but this prospect excited us). What's the status on this? Is there any way of running builds locally? This would really help for speeding up testing!
Thank you!

I'll write up our current direction soon.

In the meantime, please move to azure-pipelines-yaml repo