MSID: 91597 Version: 2 DOI: 10.48550/arXiv.2306.03218v3

MSID: 91597
Version: 2
Preprint DOI: https://doi.org/10.48550/arXiv.2306.03218v3
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=91597
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.48550/arXiv.2306.03218v3

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/91597v2
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.48550/arXiv.2306.03218v3
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/91597/v2/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

Hi @fred-atherden the Public Reviews have gone to the v1 on this one

Ah thanks @KJT08. Looks like we have the same issue with the author response as for 92080:

The author response has made its way to https://arxiv.org/abs/2306.03218 instead of https://arxiv.org/abs/2306.03218v3, which has messed everything up.
I'll move it over to the v3 URI

Forgot that the XSL library doesn't run for non-bioRxiv files.


Add current school question to maths and physics

This is the first PR in a series for maths and physics eligibility.
Obtains the current_school using the existing school search. Although I looked at the specs around the school search, I decided not to add additional tests for maths and physics as the school search is independant of which journey, but let me know if that was a bad call.
Most of this is taken straight from student loans.

 Build out the Eligibility part of the Maths & Physics claim journey

 Build out the Eligibility part of the Maths & Physics claim journey

Should the user actually be providing their "current school", rather than their "claim school"? i.e. the school they are currently employed at?

There's also already a perhaps entirely re-useable claims/current_school template.

Feel like I got the completely wrong end of the stick on this one, apologies @lawrence-dxw @pezholio.
Switched this to use current_school now.

For the ineligible feature spec I wanted a helper which meant re-naming the existing one, as part of this I updated a similar reuest spec helper even though I was not using it, it felt like it was part of the renaming though.

@lawrence-dxw I am hopeful I have them all now as well as the ineligiblity coverage. Thanks for the pointers and patience.


[Gtk] Fix a crash when tooltip query runs right after a native window…

… has been released
Fixes: https://devdiv.visualstudio.com/DevDiv/_workitems/edit/1184538

Bumped to 2020-02 with https://github.com/mono/bockbuild/commit/d37dbfe1c69911483cfcc9e2294f43c8dc639604


There's a problem with the gui text.

 But there's an error in the name of the skin. Is there an easy way to solve it

?

Please tell me how to solve it easily.

Now supports all Asian languages.


Not possible to completely remove the borders for a Stacked Bar Chart

Expected behavior
borderWidth set to 0 should remove all borders from a stacked bar chart.
Current behavior
Examples:

Reproducible sample
https://www.chartjs.org/docs/latest/samples/bar/stacked.html
Optional extra steps/info to reproduce
No response
Possible solution
No response
Context
It should be possible to remove these borders.
These 1 line pixels can give the impression that there is a gap between the two datasets.
chart.js version
v0.3.24
Browser name and version
Chrome Version 113.0.5672.126 (Official Build) (arm64)
Link to your project
No response

Duplicate: https://github.com/chartjs/Chart.js/issues/9906


finer grained --wait for ressources to be considered ok

Output of helm version:
version.BuildInfo{Version:"v3.11.1", GitCommit:"293b50c65d4d56187cd4e2f390f0ada46b4c4737", GitTreeState:"clean", GoVersion:"go1.18.10"}

Hello there!
I actually use helm with a lot of crds, and all of them are not that well designed, the current wait argument is not enough to ensure that what you released is actually released correctly, you'll have to make some custom checks with the kube apis, like checking a specific status field to be here, another resource to be created (operator pattern I see you here), etc...
So I was thinkin, why can't helm have at least one of the following solutions?

"--wait-for-ressource-attribute" command line option, where you can check anything you want to be here like
"status.whatever=whatever" (and add multiple of them)
helm annotations to add to your ressources to ensure the current ressource should be considered ok or not, like the following:  annotations:
        helm.sh/resource-check-spec.someattribute: "meh"    # check if spec.someattribute equal "meh"
        helm.sh/resource-check-status.somelist.value: "meh"  # check if one or all the values are equal to "meh"
        ...

ability to check if another resource status is ok before trying to continue with one of the following solution:

with the annotations thing above  + a new annotation like "helm.sh/weight": "5", without the hook thing here, be able to put a weight on any ressources
like with the annations thing above, but only for this purpose, with a specific annotation like:    "wait-for-resource-mynamespace.resourcename": "" # Check if the ressource default status is ok (if there's the annotations things mentionned before, it should be default + custom checks)
    "wait-for-resource-mynamespace.anotherresourcename.spec.someattribute": "meh"  # check it the anotherressource.spec.someattribute is equal to meh before putting this ressource

This would be great to have this feature in helm, for at least 3 reasons:

the helm engine can then use this to have a generic way to handle any ressources lifecycles, whatever it is, making it simpler to implement new helm default ressources checks
the custom ressources would have a way to handle their statuses in a standard way without having to put some jobs to handle the ressource lifecycle for no reason
this would be of great benefit for any tools like argocd, flux, terraform-helm-provider, it would let the helm chart handle the lifecycle of its ressources in a defined way, so you would not be bothered by having to implement custom configurations/checks inside this tools, your helm chart defines the way it should behave, and when it's deployed or not when you're waiting for it to be released.

Hope that's something in the scope of helm, because I'm currently using terraform + argocd right now, and it can be hard to handle stuff like operators, deploying a simple crd can be a hurdle from my point of view, and it would be easier with this feature
Just for information, vault is using annotations for secret injection in pods as an example, so I do think it's doable at some point

Looks fairly well thought out. If you can contribute this feature, please start by writing a helm improvement proposal (hip).
Thanks!

Thanks you for your quick réponse, I'll take some time to write the whole Idea correctly, I think it can be of great use for the helm community, as it would provide more mature/granular ressources state management on release


[1213] Fix broken cart JS

Resolves #1213

adds qualifications_hash parameter from #1207 to the following files:

app/views/cart_js/cart_dates_reload.js.erb
app/views/cart_js/reload_all.js.erb

looks good!

Great, merging!


Limit Clojure's LazySeq output

Hi, I'm experimenting with Conjure and in my current project I have some big LazySeq.
Printing it in lein repl with ultra is fine because it's truncated, but on Conjure it consumes a lot of cpu and memory and never prints anything.
Is there any configuration to control this behaviour? Should LazySeq be truncated?

Oh interesting! There's an nREPL setting to apply a hard limit on the number of bytes printed (quota) which I could start using, the hard part is how to let the user request the rest of it (when the size is unknown!) in a nice way.
There's also clojure.core/print-length which may be easier to use and configure from the user's standpoint.

; --------------------------------------------------------------------------------
; eval (current-form): (set! *print-length* 5)
5
; --------------------------------------------------------------------------------
; eval (current-form): (range 30)
(0 1 2 3 4 ...)

So I'm thinking about integrating the nREPL quota to have a hard limit on output as well as print-length, all through config.
My thinking is this: There's a default limit, if you exceed it then Conjure will warn you that it's been exceeded and which option you should set to remove the limit or raise it (if you need to). What do you think? So no special prefixes or commands to request the rest of the data, just "this has been trimmed for reason X, set this option to nil or a higher number to see more"?
You can then always run ,v1 to see the last result so you don't need to recompute it.

I've added limits to the depth and length of data structures on the develop branch. Let me know what you think :smile:
You can use :helpgrep print_options#level to show the docs on the two options, they default to a depth of 50 and a length of 500 at the moment.

Thank you so much for the fast response.
I just gave it a try and It is great, I can finally use Conjure on this project.
Informing the user the output is trimmed and how to configure it would be great. I don't really need to load more (or the rest) of the data because it's usually just to peek on it.
Thanks for this awesome plugin, really good stuff.

Awesome! That's great to hear! And yeah, I'd love to provide that but I
don't think there's any way for me to know it's been elided... maybe I can
check the string for "..." but that could misfire. I'll have a think!
I hope you enjoy :D also if you have any suggestions on what the default
limits should be, let me know. I took a complete guess.
On Sat, 4 Jul 2020, 23:35 Diego B. Fernandez<EMAIL_ADDRESS>wrote:

Thank you so much for the fast response.
I just gave it a try and It is great, I can finally use Conjure on this
project.
Informing the user the output is trimmed and how to configure it would be
great. I don't really need to load more (or the rest) of the data because
it's usually just to peek on it.
Thanks for this awesome plugin, really good stuff.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/Olical/conjure/issues/104#issuecomment-653819398, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AACM6XM5UGT2X7DROSHOWLDRZ6VBVANCNFSM4OOJU2OA
.

Closing this for now, I didn't add the "hey set this to raise the limit" yet because I can't think of a fool proof way to solve it so I'll give it some more hammock. If you :helpgrep print_options for now it'll show you everything you can configure.
This is released in v4.1.0 :smile: :rocket:
Have a great day!

The defaults seems fine and I usually search the output for what I want. Configuring this is not that valuable as the feature itself. Thanks again.


Node.js  Loaders Team Meeting 2022-08-30

Time
UTC Tue 30-Aug-2022 18:00 (06:00 PM):

Timezone
Date/Time

US / Pacific
Tue 30-Aug-2022 11:00 (11:00 AM)

US / Mountain
Tue 30-Aug-2022 12:00 (12:00 PM)

US / Central
Tue 30-Aug-2022 13:00 (01:00 PM)

US / Eastern
Tue 30-Aug-2022 14:00 (02:00 PM)

EU / Western
Tue 30-Aug-2022 19:00 (07:00 PM)

EU / Central
Tue 30-Aug-2022 20:00 (08:00 PM)

EU / Eastern
Tue 30-Aug-2022 21:00 (09:00 PM)

Moscow
Tue 30-Aug-2022 21:00 (09:00 PM)

Chennai
Tue 30-Aug-2022 23:30 (11:30 PM)

Hangzhou
Wed 31-Aug-2022 02:00 (02:00 AM)

Tokyo
Wed 31-Aug-2022 03:00 (03:00 AM)

Sydney
Wed 31-Aug-2022 04:00 (04:00 AM)

Or in your local time:

https://www.timeanddate.com/worldclock/fixedtime.html?msg=Node.js+Foundation+Loaders Team+Meeting+2022-08-30&iso=20220830T1800
or https://www.wolframalpha.com/input/?i=06PM+UTC%2C+Aug+30%2C+2022+in+local+time

Links

Minutes Google Doc: https://docs.google.com/document/d/1dKJhYs3mB_zzf89yE3m1yBoupvn7FmbML-WQE6-KlfE/edit

Agenda
Extracted from loaders-agenda labelled issues and pull requests from the nodejs org prior to the meeting.
nodejs/loaders
Invited

Loaders team: @nodejs/loaders

Observers/Guests
Notes
The agenda comes from issues labelled with loaders-agenda across all of the repositories in the nodejs org. Please label any additional issues that should be on the agenda before the meeting starts.
Joining the meeting

link for participants: https://zoom.us/j/97506450082

For those who just want to watch: https://www.youtube.com/c/nodejs+foundation/live

Removing #95 from the agenda as it’s settled. Anything else to discuss? Off-thread work?
For visibility I also opened https://github.com/nodejs/node/pull/44323 to investigate async_hooks behavior in ESM. Nothing we need to discuss, and it’s a lower priority than our existing roadmap, but I wanted everyone to be aware of it.

I think no? I owe Maël a review for his ambient loaders PR and need to try Guy's suggestion (from a Twitter DM) for the locks. My family visitors leave tomorrow, so I should have time this weekend.

I owe Maël a review for his ambient loaders PR

I think the main question there is whether off-thread needs to land first.

need to try Guy’s suggestion

If that doesn’t unstick you, maybe push a draft PR? With a list of issues you’re stuck on, so that others can try to solve them.


Fix issue with multiple connections in the same thread interfering wi…

…th each others'
autoPurge. This happens quite easily in a vibe-d environment with
multiple fibers processing requests.

Not sure what the CI is doing, I need to give some attention. But the tests passed locally, and also on the majority of the CI runs.


apis: add Telemetry and ExtensionService APIs

Description:
Adds 2 new APIs based on the Telemetry design doc

Telemetry to configure telemetry related config, starting with Envoy access logs.

ExtensionService to specify configuration pertaining to external services that OSM can integrate with, starting with config necessary to connect to an OpenTelemetry collector.

Part of #5136

Testing done:
E2e

Affected area:

Functional Area

New Functionality
[X]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? no

Did you notify the maintainers and provide attribution?

Is this a breaking change? no

Has documentation corresponding to this change been updated in the osm-docs repo (if applicable)? not yet

Codecov Report

Merging #5144 (e0c2ae4) into main (6d11fd2) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #5144      +/-   ##
==========================================
- Coverage   69.24%   69.22%   -0.02%
==========================================
  Files         196      196
  Lines       15559    15559
==========================================
- Hits        10774    10771       -3
- Misses       4730     4733       +3
  Partials       55       55

Flag
Coverage Δ

unittests
69.22% <ø> (-0.02%)
:arrow_down:

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

Impacted Files
Coverage Δ

pkg/messaging/workqueue.go
89.28% <0.00%> (-10.72%)
:arrow_down:

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


Permission failed with the var/cache/nginx/* folder using Zotero

I use your docker with Zotero to be able to keep my files uploaded and sync on my server.
I couldn't make it work until I changed the chmod for  /var/cache/nginx/*  from 700 to 775.
Without that I got permission denied because it tried to push file in that folder.
I use the env variable PUID and GID.
I have no idea if it can be helpful and/or if the code needed to be changed but I thought it was worth mentioning it

Hi, and thanks for reporting your issue! First things first, which image version are you using?
I use PUID and GID as well in the example docker-compose.yml.
These are the permissions I see:
docker conatiner exec nginxwebdav ls -la /var/cache/nginx
total 36
drwxr-xr-x 1 root root 4096 Jan 30 18:35 .
drwxr-xr-x 1 root root 4096 Jan 30 18:35 ..
drwx------ 1 abc  1000 4096 Jan 30 18:35 client_temp
drwx------ 1 abc  1000 4096 Jan 30 18:35 fastcgi_temp
drwx------ 1 abc  1000 4096 Jan 30 18:35 proxy_temp
drwx------ 1 abc  1000 4096 Jan 30 18:35 scgi_temp
drwx------ 1 abc  1000 4096 Jan 30 18:35 uwsgi_temp

So, correct that these files have 700 for permissions with PUID as their owner. So, they are not supposed to give issues (but we can look into it). The directory is owned by root but has 751 as permission.
The directory would miss w permissions if used, I agree. I cannot reproduce your issue, however. I allowed 10G for body size (CLIENT_MAX_BODY_SIZE=10G), created a 10GB test file, and uploaded it with Cyberduck. While the file was uploading (I checked 5 times), there were no changes to /var/cache/nginx:
docker container exec nginxwebdav ls -la /var/cache/nginx
total 36
drwxr-xr-x 1 root root 4096 Jan 30 18:35 .
drwxr-xr-x 1 root root 4096 Jan 30 18:35 ..
drwx------ 1 abc  1000 4096 Jan 30 18:35 client_temp
drwx------ 1 abc  1000 4096 Jan 30 18:35 fastcgi_temp
drwx------ 1 abc  1000 4096 Jan 30 18:35 proxy_temp
drwx------ 1 abc  1000 4096 Jan 30 18:35 scgi_temp
drwx------ 1 abc  1000 4096 Jan 30 18:35 uwsgi_temp

The file uploaded successfully.
Could you tell me a bit more on how to reproduce this issue, including what you see and what brought you to think you needed to change permissions to /var/cache/nginx? That might help. Thanks!

I used the github version (pulled yesterday) and build it.
I reproduced the issue (with the 700 permission)
My docker compose is nothing fancy, just PUID=1000 (my user) and PGID=100 (users group) to access the external data folder mounted in the data. CIENT_MAX_BODY_SIZE=120M
Here an example of the error I receive.
nginx-webdav-zotero    | 2022/01/30 19:28:38 [crit] 324#324: *110 unlink() "/var/cache/nginx/client_temp/0000000205" failed (13: Permission denied), client: <IP_ADDRESS>, server: localhost, request: "PUT /zotero/zotero-test-file.prop HTTP/1.1", host: "zotero.my.domain.com" nginx-webdav-zotero    | <IP_ADDRESS> - zotero [30/Jan/2022:19:28:38 +0100] "PUT /zotero/zotero-test-file.prop HTTP/1.1" 500 177 "-" "Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Firefox/60.0" nginx-webdav-zotero    | 2022/01/30 19:28:38 [crit] 324#324: *110 unlink() "/var/cache/nginx/client_temp/0000000205" failed (13: Permission denied) while closing request, client: <IP_ADDRESS>, server: <IP_ADDRESS>:80 nginx-webdav-zotero    | 2022/01/30 19:28:38 [crit] 324#324: *111 unlink() "/var/cache/nginx/client_temp/0000000207" failed (13: Permission denied), client: <IP_ADDRESS>, server: localhost, request: "PUT /zotero/zotero-test-file.prop HTTP/1.1", host: "zotero.my.domain.com"
Not sure if it is the info you require

Seems to be working here..
This is up to linking Zotero to WebDAV:
docker compose up
[+] Running 1/0
 ⠿ Container nginxwebdav  Created                                          0.0s
Attaching to nginxwebdav
nginxwebdav  | [s6-init] making user provided files available at /var/run/s6/etc...exited 0.
nginxwebdav  | [s6-init] ensuring user provided files have correct perms...exited 0.
nginxwebdav  | [fix-attrs.d] applying ownership & permissions fixes...
nginxwebdav  | [fix-attrs.d] done.
nginxwebdav  | [cont-init.d] executing container initialization scripts...
nginxwebdav  | [cont-init.d] 01-envfile: executing...
nginxwebdav  | [cont-init.d] 01-envfile: exited 0.
nginxwebdav  | [cont-init.d] 01-migrations: executing...
nginxwebdav  | [migrations] started
nginxwebdav  | [migrations] no migrations found
nginxwebdav  | [cont-init.d] 01-migrations: exited 0.
nginxwebdav  | [cont-init.d] 02-tamper-check: executing...
nginxwebdav  | [cont-init.d] 02-tamper-check: exited 0.
nginxwebdav  | [cont-init.d] 10-adduser: executing...
nginxwebdav  | usermod: no changes
nginxwebdav  |
nginxwebdav  | -------------------------------------
nginxwebdav  |           _         ()
nginxwebdav  |          | |  ___   _    __
nginxwebdav  |          | | / __| | |  /  \
nginxwebdav  |          | | \__ \ | | | () |
nginxwebdav  |          |_| |___/ |_|  \__/
nginxwebdav  |
nginxwebdav  |
nginxwebdav  | Brought to you by linuxserver.io
nginxwebdav  | -------------------------------------
nginxwebdav  |
nginxwebdav  | To support LSIO projects visit:
nginxwebdav  | https://www.linuxserver.io/donate/
nginxwebdav  | -------------------------------------
nginxwebdav  | GID/UID
nginxwebdav  | -------------------------------------
nginxwebdav  |
nginxwebdav  | User uid:    501
nginxwebdav  | User gid:    20
nginxwebdav  | -------------------------------------
nginxwebdav  |
nginxwebdav  | [cont-init.d] 10-adduser: exited 0.
nginxwebdav  | [cont-init.d] 30-config: executing...
nginxwebdav  | [cont-init.d] 30-config: exited 0.
nginxwebdav  | [cont-init.d] 90-custom-folders: executing...
nginxwebdav  | [cont-init.d] 90-custom-folders: exited 0.
nginxwebdav  | [cont-init.d] 99-custom-scripts: executing...
nginxwebdav  | [custom-init] no custom files found exiting...
nginxwebdav  | [cont-init.d] 99-custom-scripts: exited 0.
nginxwebdav  | [cont-init.d] done.
nginxwebdav  | [services.d] starting services
nginxwebdav  | [services.d] done.
nginxwebdav  | <IP_ADDRESS> - user [31/Jan/2022:14:08:19 +0100] "OPTIONS /zotero/ HTTP/1.1" 200 0 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.16; rv:60.0) Gecko/20100101 Firefox/60.0"
nginxwebdav  | <IP_ADDRESS> - user [31/Jan/2022:14:08:19 +0100] "PROPFIND /zotero/ HTTP/1.1" 404 153 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.16; rv:60.0) Gecko/20100101 Firefox/60.0"
nginxwebdav  | <IP_ADDRESS> - user [31/Jan/2022:14:08:19 +0100] "PROPFIND / HTTP/1.1" 207 217 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.16; rv:60.0) Gecko/20100101 Firefox/60.0"
nginxwebdav  | <IP_ADDRESS> - user [31/Jan/2022:14:08:22 +0100] "MKCOL /zotero/ HTTP/1.1" 201 0 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.16; rv:60.0) Gecko/20100101 Firefox/60.0"
nginxwebdav  | <IP_ADDRESS> - user [31/Jan/2022:14:08:22 +0100] "OPTIONS /zotero/ HTTP/1.1" 200 0 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.16; rv:60.0) Gecko/20100101 Firefox/60.0"
nginxwebdav  | <IP_ADDRESS> - user [31/Jan/2022:14:08:22 +0100] "PROPFIND /zotero/ HTTP/1.1" 207 224 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.16; rv:60.0) Gecko/20100101 Firefox/60.0"
nginxwebdav  | 2022/01/31 14:08:22 [error] 324#324: *1 open() "/data/zotero/nonexistent.prop" failed (2: No such file or directory), client: <IP_ADDRESS>, server: localhost, request: "GET /zotero/nonexistent.prop HTTP/1.1", host: "<IP_ADDRESS>:32080"
nginxwebdav  | <IP_ADDRESS> - user [31/Jan/2022:14:08:22 +0100] "GET /zotero/nonexistent.prop HTTP/1.1" 404 153 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.16; rv:60.0) Gecko/20100101 Firefox/60.0"
nginxwebdav  | <IP_ADDRESS> - user [31/Jan/2022:14:08:22 +0100] "PUT /zotero/zotero-test-file.prop HTTP/1.1" 201 0 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.16; rv:60.0) Gecko/20100101 Firefox/60.0"
nginxwebdav  | <IP_ADDRESS> - user [31/Jan/2022:14:08:22 +0100] "GET /zotero/zotero-test-file.prop HTTP/1.1" 200 1 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.16; rv:60.0) Gecko/20100101 Firefox/60.0"
nginxwebdav  | <IP_ADDRESS> - user [31/Jan/2022:14:08:22 +0100] "DELETE /zotero/zotero-test-file.prop HTTP/1.1" 204 0 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.16; rv:60.0) Gecko/20100101 Firefox/60.0"
nginxwebdav  | <IP_ADDRESS> - user [31/Jan/2022:14:08:26 +0100] "OPTIONS /zotero/ HTTP/1.1" 200 0 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.16; rv:60.0) Gecko/20100101 Firefox/60.0"
nginxwebdav  | <IP_ADDRESS> - user [31/Jan/2022:14:08:26 +0100] "PROPFIND /zotero/ HTTP/1.1" 207 224 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.16; rv:60.0) Gecko/20100101 Firefox/60.0"
nginxwebdav  | 2022/01/31 14:08:26 [error] 324#324: *1 open() "/data/zotero/nonexistent.prop" failed (2: No such file or directory), client: <IP_ADDRESS>, server: localhost, request: "GET /zotero/nonexistent.prop HTTP/1.1", host: "<IP_ADDRESS>:32080"
nginxwebdav  | <IP_ADDRESS> - user [31/Jan/2022:14:08:26 +0100] "GET /zotero/nonexistent.prop HTTP/1.1" 404 153 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.16; rv:60.0) Gecko/20100101 Firefox/60.0"
nginxwebdav  | <IP_ADDRESS> - user [31/Jan/2022:14:08:26 +0100] "PUT /zotero/zotero-test-file.prop HTTP/1.1" 201 0 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.16; rv:60.0) Gecko/20100101 Firefox/60.0"
nginxwebdav  | <IP_ADDRESS> - user [31/Jan/2022:14:08:26 +0100] "GET /zotero/zotero-test-file.prop HTTP/1.1" 200 1 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.16; rv:60.0) Gecko/20100101 Firefox/60.0"
nginxwebdav  | <IP_ADDRESS> - user [31/Jan/2022:14:08:26 +0100] "DELETE /zotero/zotero-test-file.prop HTTP/1.1" 204 0 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.16; rv:60.0) Gecko/20100101 Firefox/60.0"

After adding one of my open access papers by DOI (10.7717/peerj.289) and forcing a sync
nginxwebdav  | 2022/01/31 14:16:01 [error] 324#324: *4 open() "/data/zotero/BFHTWTLC.prop" failed (2: No such file or directory), client: <IP_ADDRESS>, server: localhost, request: "GET /zotero/BFHTWTLC.prop HTTP/1.1", host: "<IP_ADDRESS>:32080"
nginxwebdav  | <IP_ADDRESS> - user [31/Jan/2022:14:16:01 +0100] "GET /zotero/BFHTWTLC.prop HTTP/1.1" 404 153 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.16; rv:60.0) Gecko/20100101 Firefox/60.0"
nginxwebdav  | <IP_ADDRESS> - user [31/Jan/2022:14:16:01 +0100] "PUT /zotero/BFHTWTLC.zip HTTP/1.1" 201 0 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.16; rv:60.0) Gecko/20100101 Firefox/60.0"
nginxwebdav  | <IP_ADDRESS> - user [31/Jan/2022:14:16:01 +0100] "PUT /zotero/BFHTWTLC.prop HTTP/1.1" 201 0 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.16; rv:60.0) Gecko/20100101 Firefox/60.0"

Two files BFHTWTLC.{prop, zip} are created in the WebDAV folder.
If I add another PDF manually to the newly created record and force sync:
nginxwebdav  | <IP_ADDRESS> - user [31/Jan/2022:14:18:24 +0100] "OPTIONS /zotero/ HTTP/1.1" 200 0 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.16; rv:60.0) Gecko/20100101 Firefox/60.0"
nginxwebdav  | <IP_ADDRESS> - user [31/Jan/2022:14:18:24 +0100] "PROPFIND /zotero/ HTTP/1.1" 207 224 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.16; rv:60.0) Gecko/20100101 Firefox/60.0"
nginxwebdav  | 2022/01/31 14:18:24 [error] 324#324: *8 open() "/data/zotero/nonexistent.prop" failed (2: No such file or directory), client: <IP_ADDRESS>, server: localhost, request: "GET /zotero/nonexistent.prop HTTP/1.1", host: "<IP_ADDRESS>:32080"
nginxwebdav  | <IP_ADDRESS> - user [31/Jan/2022:14:18:24 +0100] "GET /zotero/nonexistent.prop HTTP/1.1" 404 153 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.16; rv:60.0) Gecko/20100101 Firefox/60.0"
nginxwebdav  | <IP_ADDRESS> - user [31/Jan/2022:14:18:24 +0100] "PUT /zotero/zotero-test-file.prop HTTP/1.1" 201 0 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.16; rv:60.0) Gecko/20100101 Firefox/60.0"
nginxwebdav  | <IP_ADDRESS> - user [31/Jan/2022:14:18:24 +0100] "GET /zotero/zotero-test-file.prop HTTP/1.1" 200 1 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.16; rv:60.0) Gecko/20100101 Firefox/60.0"
nginxwebdav  | <IP_ADDRESS> - user [31/Jan/2022:14:18:24 +0100] "DELETE /zotero/zotero-test-file.prop HTTP/1.1" 204 0 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.16; rv:60.0) Gecko/20100101 Firefox/60.0"

If I put both PDF in the trash, empty the trash, and force sync:
nginxwebdav  | <IP_ADDRESS> - user [31/Jan/2022:14:19:59 +0100] "DELETE /zotero/BFHTWTLC.zip HTTP/1.1" 204 0 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.16; rv:60.0) Gecko/20100101 Firefox/60.0"
nginxwebdav  | <IP_ADDRESS> - user [31/Jan/2022:14:19:59 +0100] "DELETE /zotero/BFHTWTLC.prop HTTP/1.1" 204 0 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.16; rv:60.0) Gecko/20100101 Firefox/60.0"

Unless I am missing some use case, the issue is on your side. I can still try to help you. Let's verify the obvious one: a typo in your options.
You wrote:

PUID=1000 (my user) and PGID=100

PGID=100 is unusual. Are you certain that it's not 1000 instead? This would be the easiest explanation.

@Oliph, any news on this?

@Oliph it would really help if you could find a way for me to replicate the issue. I have just forced UID 1000 and GID 100 on a fresh container on my Raspberry Pi, and it is working fine..

Closing it for now as I can't replicate, but feel free to reopen in case.

Don't know if I should open another issue or piggy back off of this based upon what has been discussed.
I am running this docker container on a Synology NAS and receive this error. I moved from a separate non Synology environment where it worked before, simply copied my folders and same compose block. I am primarily using this for Paperless document consuming, thus the paperless naming conventions. However the configuration should be isolated.
  paperless-webdav:
    image: dgraziotin/nginx-webdav-nononsense:latest
    container_name: paperless-webdav
    restart: unless-stopped
    networks:
      - apps-net
    environment:
      - PUID=${puid}
      - PGID=${pgid}
      - TZ=${tz}
      - WEBDAV_USERNAME=${paperless_webdav_user}
      - WEBDAV_PASSWORD=${paperless_webdav_password}
      - SERVER_NAMES=scans.*
      - TIMEOUTS_S=1200 # these are seconds
      - CLIENT_MAX_BODY_SIZE=2G # must end with M(egabytes) or G(igabytes)
    volumes:
      - ./paperless/webdav:/config
      - ./paperless/consume:/data

PUID and PGID are ones that I use for other containers on the NAS. I can see that the startup of the container will show the proper ID's
-------------------------------------
GID/UID
-------------------------------------
User uid:    1032
User gid:    65538
-------------------------------------

Here is the "consume" directory on the host, where data should be getting dropped.
$ ls -laF

drwxrwxr-x 5 captain docker 4096 Jan 18 21:40 ./
drwxrwxr-x 8 captain docker 4096 Feb  8 13:36 ../
drwxrwx--- 2 captain docker 4096 Feb  8 14:01 consume/
drwxrwx--- 3 captain docker 4096 Feb  8 13:22 webdav/

With user captain being the 1032 id used for the container
$ id captain
uid=1032(captain) gid=100(users) groups=100(users)

I am getting the same error when trying to write a file. I can connect, browse and delete files just cant write, tried through various methods like an app called Quickscan on iOS or just Finder connection on Mac
<IP_ADDRESS> - paperless [08/Feb/2023:14:01:46 -0600] "PROPFIND /statement.pdf HTTP/1.1" 404 153 "-" "WebDAVFS/3.0.0 (03008000) Darwin/22.2.0 (arm64)"
<IP_ADDRESS> - paperless [08/Feb/2023:14:01:46 -0600] "PROPFIND /statement.pdf HTTP/1.1" 404 153 "-" "WebDAVFS/3.0.0 (03008000) Darwin/22.2.0 (arm64)"
2023/02/08 14:01:46 [crit] 138#138: *84 open() "/var/cache/nginx/client_temp/0000000003" failed (13: Permission denied), client: <IP_ADDRESS>, server: scans.*, request: "PUT /statement.pdf HTTP/1.1", host: "scans.mydomain.com"
<IP_ADDRESS> - paperless [08/Feb/2023:14:01:46 -0600] "PUT /statement.pdf HTTP/1.1" 500 177 "-" "WebDAVFS/3.0.0 (03008000) Darwin/22.2.0 (arm64)"

For good measure, I set this up on a a raspberry pi and it failed for the same file. Note that this has no reverse proxy (above uses SWAG/nginx), was just accessing directly.
  webdav:
    image: dgraziotin/nginx-webdav-nononsense:latest
    container_name: webdav
    ports:
      - 9999:80
    environment:
      - PUID=1001
      - PGID=995
      - TZ=America/Chicago
      - WEBDAV_USERNAME=test
      - WEBDAV_PASSWORD=qwerty21
      #- SERVER_NAMES=test.*
      - TIMEOUTS_S=1200 # these are seconds
      - CLIENT_MAX_BODY_SIZE=2G # must end with M(egabytes) or G(igabytes)
    volumes:
      - ./test/webdav:/config
      - ./test/consume:/data

Error from Mac OSX finder.
<IP_ADDRESS> - test [08/Feb/2023:14:24:28 -0600] "PROPFIND /statement.pdf HTTP/1.1" 404 153 "-" "WebDAVFS/3.0.0 (03008000) Darwin/22.2.0 (arm64)"
<IP_ADDRESS> - test [08/Feb/2023:14:24:28 -0600] "PROPFIND /statement.pdf HTTP/1.1" 404 153 "-" "WebDAVFS/3.0.0 (03008000) Darwin/22.2.0 (arm64)"
2023/02/08 14:24:28 [crit] 138#138: *4 open() "/data/statement.pdf.0000000002" failed (13: Permission denied), client: <IP_ADDRESS>, server: localhost, request: "PUT /statement.pdf HTTP/1.1", host: "<IP_ADDRESS>:9999"
<IP_ADDRESS> - test [08/Feb/2023:14:24:28 -0600] "PUT /statement.pdf HTTP/1.1" 500 177 "-" "WebDAVFS/3.0.0 (03008000) Darwin/22.2.0 (arm64)"

Really curious what is going on and how it worked for me before but then broke when I moved to a different host.
Thanks.


Looking at a Warrior on a SpawnPoint reveals the SpawnPoint

If a NPC is standing on a SpawnPoint then tileAtPosition (and therefore look) will return a SpawnPoint not a Warrior. The same is true for a NPC on an Exit (I'm not sure if NPCs moving on to an Exit should leave the map or stay?). Items placed on SpawnPoints or Exits are also invisible.
Below is a test where I am placing an Item and a NPC on a SpawnPoint - I expect to see a Warrior but actually see a SpawnPoint. If I remove the SpawnPoint then I see the Warrior, and if I remove the Warrior too then I see the Item.
If this is a valid bug then I'm happy to write a PR, please can you confirm what you expect:

Can a NPC move to or spawn on a SpawnPoint?
Can an Item be placed on a SpawnPoint?
Can a NPC move to or spawn on an Exit?
Can an Item be placed on an Exit?

Expectation
Builder.init { rows = 1, columns = 5 }
    |> Builder.withSpawnPoint { x = 4, y = 0 }
    |> Builder.withItem { x = 4, y = 0 } Item.Potion
    |> Builder.withNpc "Dummy" { x = 4, y = 0 } Dummy.takeTurn
    |> Builder.build
    |> Map.tileAtPosition { x = 4, y = 0 }
    |> Expect.equal (Tile.Warrior "Dummy")

Failure
    SpawnPoint
    ╷
    │ Expect.equal
    ╵
    Warrior "Dummy"

Bonus Thoughts
I can envisage some alternative ProgressionFunctions where:

Multiple players work together to reach all exits on the map (rather than first player to reach one exit).
Player races a NPC to reach the exit.
Player races to the Exit before the NPC gets to the SpawnPoint.

A follow up thought, if an Item can be placed on a non-empty Tile then perhaps lookDown should return a tuple of ( Maybe Item, Tile ) (e.g.,  ( Just Item.Potion, Tile.SpawnPoint ) and ( Nothing, Tile.Empty )).

This is a very good question. Both Items and Warriors can exist on top of a SpawnPoint and Exit however, you should still be able to see the SpawnPoint or Exit if that's what's most important to your AI.
Not sure how to solve this though. I'm open for suggestions.


FEAT(curve): Add Stark252 curve

Describe the changes
This PR adds a curve config for the Stark252 curve and operations over the Stark252 field element used in starknet and the  stone prover.

Hi @PatStiles
Thanks for the PR!
Just fyi: we're currently working on adding support for STARK fields in general, and I assume Stark252 is a STARK field and not a curve, right? I think it'll be much easier to support this field once all the infrastructure for it is ready, which should be next week (I think).

@DmytroTym this is specific to the curve used for ECDSA in starknet https://docs.starkware.co/starkex/crypto/stark-curve.html  I hit a hurdle as arkworks does not currently support the stark252 curve and the tests require the corresponding arkworks curve.

@PatStiles gotcha!
Reg failing Rust tests:

I think NTT tests fail because you run them in the wrong field. My understanding is that you want to run them in the 2^251+17⋅2^192+1 field which happens to be the base modulus of the curve. However, you instantiate them for the scalar field of the curve, which only has 2-adicity of 1, so doesn't support any NTTs, which is why arkworks throws an error when trying to get_root_of_unity before all the NTT tests. I think you need to move NTTs to the base field.
MSM probably fails because icicle version of the curve differs from arkworks version. Unfortunately icicle doesn't allow for curves with non-zero linear term just yet so instead of 𝑦^2≡𝑥^3+𝛼⋅𝑥+𝛽(mod𝑝) you effectively get 𝑦^2≡𝑥^3+𝛽(mod𝑝) in icicle, so the results of MSM are wrong.

My understanding is that the system you linked works roughly like this: some operations that happen on the curve (like ECDSA, Pedersen commitments etc) get compiled into a STARK circuit whose native field is F_{2^251+17⋅2^192+1}. Then you produce a STARK proof using NTTs and Poseidon hashes (probably without any extension fields). Let me know if I'm missing something but if that's more or less accurate then:

If you want to use icicle for the first part: ECDSA, Pedersen etc. (now or in the future), one thing that we need to add support for is elliptic curves with non-zero linear terms. And also probably ECDSA, Pedersen etc. as primitives.
If you only want icicle for the STARK prover part, I'd suggest that you don't really need the full curve but only the F_{2^251+17⋅2^192+1} field with NTT and Poseidon implemented for it. I'd suggest that you implement it on top of https://github.com/ingonyama-zk/icicle/tree/V2 branch similar to how we implemented baby bear there (one difference is that you probably don't need the extension field). I think your current code is good, just need to move things around and probably delete curve-related functionality. Btw your solution with implementing stark252 for arkworks from scratch is probably better than our ad-hoc tests for baby bear. V2 should be released really soon too so you won't have to wait for a long time. If this approach fits, please let me know if there are any issues or questions along the way.

@DmytroTym Thank you for the helpful suggestions will tackle this this weekend!

Closing in favor of #493 which implements the Stark252 base field on top of icicle V2


Add more logs for HNSW and DiskANN build stage

Re: title
Requested by @liliu-z

2.2 PR: #260 #261 #262


Fix workflow name for PerformWorkflowStep API

Description Of Change
This PR helps fix the workflow name being set while calling the PerformWorkflowStep API. This is a remnant from the time we had workflow parent.
Testing Procedure

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/1

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
0

Sum
1


can not use @iconify/svelte in vue3+vite project

my project use vue3 and vite
i want to use my own icon repository
I see this doc: https://iconify.design/docs/api/providers.html
I use pmpm add @iconify/svelte install it
then
i use import { addAPIProvider } from '@iconify/svelte'; in my code
but i got an error
[plugin:vite:import-analysis] Failed to resolve entry for package "@iconify/svelte". The package may have incorrect main/module/exports specified in its package.json: No known conditions for "." specifier in "@iconify/svelte" package
why?what can i do to reslove it

Why are you using Svelte component in a Vue project?

Why are you using Svelte component in a Vue project?
Code sample in docs is generic, replace component with correct one, in your case @iconify/vue

I used in main.js，import after import App from "./App.vue"
and used like this `
  app.mount("#app");

  addAPIProvider("", {
    resources: ["http://127.0.0.`:3000", "https://api.iconify.design"],
  });

I try change to@iconify/vue,it can success import,but addAPIProvider not use my custom provider

There are 3 errors in your code:

Always change config before creating app.
Typo in address, making it unusable.
If you are using custom API, why are you also adding Iconify API server to config? That doesn't make sense.

There are 3 errors in your code:

Always change config before creating app.
Typo in address, making it unusable.
If you are using custom API, why are you also adding Iconify API server to config? That doesn't make sense.

this is my new code ,changed the problem,but it also request the https://api.iconify.design/svg.json,
I run the server with https://github.com/iconify/api
the api server also not receieve any request
my chrome's developetool network tab also not found request with http://<IP_ADDRESS>:3000
  addAPIProvider("", {
    resources: ["http://<IP_ADDRESS>:3000"],
  });
  app.mount("#app");


Add new rule to default rules array

I just would like to know if there is a way to add a custom rule to the default array of rules (defined in M6Web\CS\Config\php71::getRules())

This question seems to have been answered.


Improve type casting

Thanks you @MathiasReker. I must close this PR because it doesn't bring any value.


Add Lua plugin to list of available plugins.

Add Lua plugin to list of available plugins under the heading "Third party plugins have been built for the following languages".

@Stratus3D I refactored the readme instead. Added a table of plugins. Thank you for creating the plugin ~!

Thanks @HashNuke! I'll be adding support for LuaJIT to the plugin later this week.


mms dont use pre_processor it uses toekizer so i how can i solve this?

Still in colab when i run this cell as suggated by @atulpokharel-gp :
`from transformers import AutoTokenizer, AutoModelForTextToWaveform
tokenizer = AutoTokenizer.from_pretrained("facebook/mms-tts-eng")
model = AutoModelForTextToWaveform.from_pretrained("facebook/mms-tts-eng")`
and then i run: !accelerate launch run_vits_finetuning.py /content/finetune-hf-vits/training_config_examples/finetune_mms_kor.json
i get this error: OSError: facebook/mms-tts-som does not appear to have a file named preprocessor_config.json. Checkout 'https://huggingface.co/facebook/mms-tts-som/tree/main' for available files.
as we know TTS of mms model use uses a tokenizer to pre-process the text inputs to token ids. It doesn't need a preprocessor_config.json how can i do this my config file is:
`
{
"project_name": "so-finetuning",
"push_to_hub": true,
"hub_model_id": "facebook/mms-tts-som",
"report_to": ["wandb"],
"overwrite_output_dir": true,
"output_dir": "./tmp/vits_finetuned_Som",
"dataset_name": "khederwaaOne/s",
"audio_column_name": "audio",
"text_column_name":"transcribe",
"train_split_name": "train",
"eval_split_name": "train",
"full_generation_sample_text": "Labadaas qaybood oo aanu soo sheegnay haddii la falanqeeyo",
"max_duration_in_seconds": 28.5,
"min_duration_in_seconds": 2.4,
"max_tokens_length": 300,
"model_name_or_path": "facebook/mms-tts-som",
"preprocessing_num_workers": 4,
"do_train": true,
"num_train_epochs": 30,
"gradient_accumulation_steps": 1,
"gradient_checkpointing": false,
"per_device_train_batch_size": 4,
"learning_rate": 2e-5,
"adam_beta1": 0.8,
"adam_beta2": 0.99,
"warmup_ratio": 0.01,
"group_by_length": false,
"do_eval": true,
"eval_steps": 50,
"per_device_eval_batch_size": 4,
"max_eval_samples": 25,
"do_step_schedule_per_epoch": true,
"weight_disc": 3,
"weight_fmaps": 1,
"weight_gen": 1,
"weight_kl": 1.5,
"weight_duration": 1,
"weight_mel": 35,
"fp16": true,
"seed": 456
}
`

@C0deXG
create a file name : preprocessor_config.json
with :
{
"feature_extractor_type": "VitsFeatureExtractor",
"feature_size": 80,
"hop_length": 256,
"max_wav_value": 32768.0,
"n_fft": 1024,
"padding_side": "right",
"padding_value": 0.0,
"return_attention_mask": false,
"sampling_rate": 16000
}
inside the model like like this :
!

@atulpokharel-gp thank you so much for the reponse i solve the issue but my dataset is only 100 of one speaker so i remove all the things the refer or has speaker_Id in the config but someone am still getting speaker_Id error :
Traceback (most recent call last):   File "/content/finetune-hf-vits/run_vits_finetuning.py", line 1494, in <module>     main()   File "/content/finetune-hf-vits/run_vits_finetuning.py", line 1099, in main     speaker_id=batch["speaker_id"],   File "/usr/local/lib/python3.10/dist-packages/transformers/tokenization_utils_base.py", line 257, in __getitem__     return self.data[item] KeyError: 'speaker_id' wandb: 🚀 View run polished-cosmos-2 at: https://wandb.ai/jjhjjff4/so-custom-model-TTS/runs/4f060m6d wandb: Find logs at: wandb/run-20241028_174011-4f060m6d/logs Traceback (most recent call last):   File "/usr/local/bin/accelerate", line 8, in <module>     sys.exit(main())   File "/usr/local/lib/python3.10/dist-packages/accelerate/commands/accelerate_cli.py", line 48, in main     args.func(args)   File "/usr/local/lib/python3.10/dist-packages/accelerate/commands/launch.py", line 1174, in launch_command     simple_launcher(args)   File "/usr/local/lib/python3.10/dist-packages/accelerate/commands/launch.py", line 769, in simple_launcher     raise subprocess.CalledProcessError(returncode=process.returncode, cmd=cmd) subprocess.CalledProcessError: Command '['/usr/bin/python3', 'run_vits_finetuning.py', '/content/finetune-hf-vits/training_config_examples/finetune_mms.json']' returned non-zero exit status 1.
my config:
`{
"project_name": "so-custom-model-TTS",
"push_to_hub": true,
"hub_model_id": "catsOfpeople/V5.01",
"report_to": ["wandb"],
"overwrite_output_dir": true,
"output_dir": "./tmp/vits_finetuned_guj",
"dataset_name": "khederwaaOne/s",
"audio_column_name": "audio",
"text_column_name":"transcribe",
"train_split_name": "train",
"eval_split_name": "train",

"full_generation_sample_text": "Labadaas qaybood oo aanu soo sheegnay haddii la falanqeeyo mid waliba wuxuu noqonayaa bad aan xeeb lahayn iyo berri aan laga hoyan karin iyo biyo aan laga bixi karin.",

"max_duration_in_seconds": 28.5,
"min_duration_in_seconds": 2.4,
"max_tokens_length": 500,

"model_name_or_path": "catsOfpeople/V5.01",

"preprocessing_num_workers": 4,

"do_train": true,
"num_train_epochs": 35,
"gradient_accumulation_steps": 1,
"gradient_checkpointing": false,
"per_device_train_batch_size": 4,
"learning_rate": 2e-5,
"adam_beta1": 0.8,
"adam_beta2": 0.99,
"warmup_ratio": 0.01,
"group_by_length": false,

"do_eval": true,
"eval_steps": 50,
"per_device_eval_batch_size": 4,
"max_eval_samples": 25,
"do_step_schedule_per_epoch": true,

"weight_disc": 3,
"weight_fmaps": 1,
"weight_gen": 1,
"weight_kl": 1.5,
"weight_duration": 1,
"weight_mel": 35,

"fp16": true,
"seed": 456

}`

So i dont have speaker_ID in my dataset since i only have one speaker is there way to solve this issue @atulpokharel-gp

@C0deXG  if you do not want to modified anything in the file : "run_vits_finetuning.py"  you can just add a column name Speaker_id and assign as default value 1 . if you prefer to change in the "run_vits_finetuning.py" then you need to  modified the code  that handle the batch , and the processing part so i  suggest you to add the column in your data rather then modifing the script of @ylacombe

@C0deXG if you do not want to modified anything in the file : "run_vits_finetuning.py" you can just add a column name Speaker_id and assign as default value 1 . if you prefer to change in the "run_vits_finetuning.py" then you need to modified the code that handle the batch , and the processing part so i suggest you to add the column in your data rather then modifing the script of @ylacombe

Do i have to add the speaker_Id into my dataset too and make it all 1?

yes !!

so i altho set speaker_id to one iam getting errror:
config:
"project_name": "so-custom-model-TTS",     "push_to_hub": true,     "hub_model_id": "catsOfpeople/V5.01",     "report_to": ["wandb"],     "overwrite_output_dir": true,     "output_dir": "./tmp/vits_finetuned_guj",     "Speaker_d": 1,     etc..
Error:
`ValueError: Some keys are not used by the HfArgumentParser: ['Speaker_id']`

@atulpokharel-gp

@C0deXG  can you share the dataset  structure with all the features in  it

@C0deXG can you share the dataset structure with all the features in it

@atulpokharel-gp
here:
DatasetDict({
train: Dataset({
features: ['transcription', 'audio', 'speaker_id'],
num_rows: 75
})
})
btw i a have qestion the reason my dataset is small is my point is to clone my voice so i made this dataset will the model have my voice or tone after the traning

"Speaker_d": 1

can you check this in the config.json

@atulpokharel-gp

@C0deXG   I suspect you just have a typo in the call you are making to the script so can you check the name in dataset  feature  and the config.json

@C0deXG I suspect you just have a typo in the call you are making to the script so can you check the name in dataset feature and the config.json

@atulpokharel-gp
I dont know what is wrong you can check out my dataset here https://huggingface.co/datasets/catsOfpeople/owoV4.000
and here is my config:
`{
"project_name": "so-custom-model-TTS",
"push_to_hub": true,
"hub_model_id": "catsOfpeople/V5.01",
"report_to": ["wandb"],
"overwrite_output_dir": true,
"output_dir": "./tmp/vits_finetuned_guj",
"dataset_name": "catsOfpeople/owoV4.000",
"audio_column_name": "audio",
"text_column_name":"transcribe",
"train_split_name": "train",
"eval_split_name": "train",
"speaker_id_column_name": "speaker_id",
"override_speaker_embeddings": true,
"filter_on_speaker_id": 1,

"full_generation_sample_text": "Labadaas qaybood oo aanu soo sheegnay haddii la falanqeeyo mid waliba wuxuu noqonayaa bad aan xeeb lahayn iyo berri aan laga hoyan karin iyo biyo aan laga bixi karin.",

"max_duration_in_seconds": 28.5,
"min_duration_in_seconds": 2.4,
"max_tokens_length": 500,

"model_name_or_path": "catsOfpeople/V5.01",

"preprocessing_num_workers": 4,

"do_train": true,
"num_train_epochs": 35,
"gradient_accumulation_steps": 1,
"gradient_checkpointing": false,
"per_device_train_batch_size": 4,
"learning_rate": 2e-5,
"adam_beta1": 0.8,
"adam_beta2": 0.99,
"warmup_ratio": 0.01,
"group_by_length": false,

"do_eval": true,
"eval_steps": 50,
"per_device_eval_batch_size": 4,
"max_eval_samples": 25,
"do_step_schedule_per_epoch": true,

"weight_disc": 3,
"weight_fmaps": 1,
"weight_gen": 1,
"weight_kl": 1.5,
"weight_duration": 1,
"weight_mel": 35,

"fp16": true,
"seed": 456

}`
and here is how am calling it in colab:
!accelerate launch run_vits_finetuning.py /content/finetune-hf-vits/training_config_examples/finetune_mms.json
i chnage the info in the /finetune_mms.json to my info but still getting the exact same error i try removing the speaker_id and i try to copy it from the exmaples but same issue i really dont know what to do now other then just quite :(

@atulpokharel-gp


Allow customization of validator / Apply defaults in request body

Description
I need Connexion to use default value if a property is missing in request body. Although it is done with URL query, in body 'default' is just for documentation purposes (that is nature of JSON Schema). In order to enforce defaults (as described here https://python-jsonschema.readthedocs.io/en/latest/faq/#why-doesn-t-my-schema-that-has-a-default-property-actually-set-the-default-on-my-instance) I need to overwrite validator used in connexion/decorators/validation.py. It would be even better if Connexion offered defaults enforcement on its own.
Expected bahaviour

Access to body validation class in similar way as we have it with Resolver.

OR

A flag to set 'defaults enforcement'. When 'defaults' is defined for a non-required property and the property is not passed, default value should be used.

Actual behaviour
We don't have tools for that. When non-required property is not passed, it is not included in data provided for controlled function, even if there are defaults.
Example
...
    post:
      operationId: "controllers.post_foo"
      parameters:
      - in: "body"
        name: "foo"
        required: true
        schema:
          $ref: "#/definitions/Foo"
      responses:
        200:
          description: "OK"
definitions:
  Foo:
    type: "object"
    properties:
      name:
        type: "string"
        default: "BOB"

If -d '{}' was sent, controller should receive {'name': 'BOB'}.
Another example
It should be enough to provide defaults in bottom-most layer.
definitions:
  Foo:
    type: "object"
    default: {}
    properties:
      name:
        type: "object"
        properties:
          first:
            type: "string"
            default: "BOB"
          last:
            type: "string"
            default: "DOE"

If -d '{}' was sent, controller should receive {'name': {'first': 'BOB', 'last': 'DOE'}}.

I think it could be enough to keep RequestBodyValidator  as a Operation's attribute, then we can replace it inside a custom implementation of Resolver. I will take a look at that and hopefully post a draft of needed changes.

@Ruwann you can use a custom validator since #352. I don't think we should add a flag to add defaults to Connexion. I would close this one if you agree.

Yes, indeed. And there is an example about how to use default values. However, I was considering adding a flag to enable this behaviour because for query parameters, we do provide the default values:
https://github.com/zalando/connexion/blob/803cf638785223b3a9321c4ad5bbe371a61e38b6/connexion/operations/swagger2.py#L257-L259
https://github.com/zalando/connexion/blob/803cf638785223b3a9321c4ad5bbe371a61e38b6/connexion/operations/openapi.py#L352-L362

When in the code myself, it was this that appears to start with the defaults and update the dict with what the request has.
I'd have to get in there with the debugger/logging to figure out why that doesn't cause the desired behavior... unless you already know.

I think this only takes the default at the top level of the body and doesn't take into account any nested defaults,
https://github.com/spec-first/connexion/blob/87a0fed4dcd164eb779cd11afd838845910b1e32/connexion/operations/openapi.py#L297

FWIW, I just stumbled on a flaw with the approach suggested in some of the other issue related to this (https://github.com/spec-first/connexion/blob/main/examples/swagger2/enforcedefaults/enforcedefaults.py)
If you have a schema with oneOf, the defaults for every type contained within the oneOf will be set. This is because the jsonschema validator invokes the validators for each instance of oneOf, without regard for the provided discriminator value.
I considered seeing if there was a way to strip out the validators which didn't match the provided discrinator, but it appears the the necessary context (mapping the type to the validator) is missing at the point where the validation occurs.
In particular, the schema has nothing like class_name for well-typed objects. With that, I think I could make this work for oneOf polymorphism.

Fixed since https://github.com/spec-first/connexion/pull/1616


Fix for darkTheme not getting changed correctly if default value is set in GetMaterialApp

Simple fix to avoid that ThemeData doesn't get changed correctly when standard value for darkTheme is set in GetMaterialApp and Theme should be changed to a ThemeData with Brightness.dark.
To test this issue, you would need two ThemeData with Brightness.dark and try to switch between those two with Get.changeTheme  AND have a default value for dark Theme in GetMaterialApp.
runApp(
    GetMaterialApp(
      themeMode: ThemeMode.system,
      theme: CustomThemes.lightStandardTheme,
      darkTheme: CustomThemes.darkStandardTheme,
    ),
  );

ThemeChanges work perfectly if you have NO default value set for darkTheme, but I don't think this is the correct behaviour.

Thanks for this PR.
LGTM


Fix in handling of screens and behaviour of menu

Fix duplication of players in GameOver
Add clearAll() function which has to be implemented for Screen
Use return Screen::gotoScreen(<SCREEN_STATE>) in order to switch to another screen
A bit of refactoring

works on my end on my ongoing branch.
small nitpick is I think you're not supposed to use the virtual keyword to override clearAll() on game.hpp and gameover.hpp.

works on my end on my ongoing branch. small nitpick is I think you're not supposed to use the virtual keyword to override clearAll() on game.hpp and gameover.hpp.

You're right. There're a few other mistakes I'll fix in a next PR. Thank you.


Fix axis ordering in UnivariateFinite example

In docs, it seems like the axis order in the doc as been reversed: https://github.com/alan-turing-institute/MLJModelInterface.jl/blob/a064ba36cd5474a88e09a907401e8d891301462e/src/data_utils.jl#L393
The provided example fails:
v = categorical([:x, :x, :y, :x, :z, :w])
probs = rand(3, 100)
probs = probs ./ sum(probs, dims=1)
UnivariateFinite([:x, :y, :z], probs, pool=v)

While the adjusted example with observation being a row works:
v = categorical([:x, :x, :y, :x, :z, :w])
probs = rand(100, 3)
probs = probs ./ sum(probs, dims=2)
UnivariateFinite([:x, :y, :z], probs, pool=v)

Yes, indeed. The docs are wrong. Good catch, thanks!

Done already
@OkonSamuel


Prevent duplicate log entries when both DDASLLogCapture and DDASLLogger are used

When both DDASLLogCapture and DDASLLogger are active and you have another logger (for instance the file or tty logger),
using a DDLog(xxx) statement would result in 2 log entries with the same message in other loggers.
Indeed, the first (expected) entry would be sent to the registered loggers and then the DDASLLogger would send it to the ASL facility, after that the DDASLLogCapture would pick it up and it would again be sent to the registered loggers (note that in that case DDASLLogger would rightfully skip it since it comes from the capture).
So here I've marked messages sent by DDASLLogger with a custom key that's used for filtering in DDASLLogCapture. Hence preventing the duplicate messages.

Could you please rebase this? We just merged ASL capture code from #512 that may be conflicting.

Rebase done.

Still marked as conflicting below.

Oh indeed, cause more stuff has been merged since I did the rebase ;)

Should be good now!

Ok, merging this. Please give it a good try.
We will need some cleanup and fix unit tests before making a new release.


doc: zbus: samples: Use new Sphinx extension to document samples

Use the new code-sample directive and roles to document the zbus samples so that they show up as "Related samples" when browsing zbus API.

Only small things to adjust. Great summaries! Thanks for the PR.

thanks for the quick review! should be good now :) live result should eventually be visible here: https://builds.zephyrproject.io/zephyr/pr/62360/docs/services/zbus/index.html#api-reference

@rodrigopex please revisit when you have a chance :)


refactor to hopefully improve performance

Removed all double calls on trackedShortBlockPositions.get and trackedIntBlockPositions.get...
Hopefully that improves the performance of playertracking.

Il tell you tomorow if i can finish that:-)

After talking back to @D3-M0N we noticed that it is pretty much impossible to change the class over to using the FastUtil maps - we have two options:

Make a copy of the Map in the set()-Method (which would break if someone makes changes to the map he passed to theset()-Method as they would not be reflected into the copied Map)
Create Wrapper classes which relay all calls to the parent Map<Integer/Double, PlayerTracker> (which would require a huge boilerplate and has terrible performance both due to the strengthened iteration-methods)

...why is it impossible?

Why would it be impossible? We do this in several cases.

Sorry for the bad word choice - by impossible i basically mean we are unsure wether to break functionality someone might depend on or create a lot of boilerplate that will be removed again in the near future anyways ...
Oh and could you clarify what "We do this in several cases." is supposed to mean? (That sounds bad ...)

...it's a simple replacement of one map with another.

Its not that easy - we have to convert the Map<Integer/Double, Playertracker> fields to Int/Double2ObjectMaps in order to be able to use its methods with primitives - thos in return breaks the setters available in the class as they still take a basic map that we can no longer directly assign to the map fields - thats why i was asking which way those (old) setters should be changed

@Zidane i will provide an update tomorrow:-)

Hello, after a long time back and forth I would have 2 suggestions:

Make a copy of the Map in the set () - Methods (@TechnicianLP)
change the IMixinChunk interface set () - Methods to be deprecated and
create 2 new methods and use fastutils maps

which should i use?

Manually merged into stable-7 with some changes of my own. Thanks!


Switch on travis ci on prism

Hi, we have the travis ci file, but i doint see any tests ran (nothing is ran on travis when creating pulls either).

https://docs.travis-ci.com/user/getting-started/

CI tests only occur when master branch is updated and when tags are pushed (i.e. a new release is made).

hmm, should we also enable it on the gh-pages branch?

Ah, you're right, the PR I was looking at was for the master branch.
So I guess we could add the gh-pages branch explicitely. Since

By default, the gh-pages branch is not built unless you add it to the safelist.

As long as we don't change the deployment rules, it should be ok.

Thank you :)


Allow dynamic changes of extraData

As it stands it's impossible to send e.g. the value of a checkbox along
with the files - my example was to have a 'allow file replacement' box,
but this only works if you can examine the checkbox as the file is
dropped. My solution I think is more general than Fazioli's by allowing
the values in the extraData to be functions, and if they are to call
them as the formData is created and use the return values for the value
of the parameter.

Very interesting, hopefully i'll be able to catch up with merges and other project related tasks next weekend.
Thanks

@danielm anything new about it? its so simple fixed that you can just accept without worry like https://github.com/danielm/uploader/pull/37 https://github.com/danielm/uploader/pull/33

Any update on this? As the OP noted, it's impossible to pass any dynamic data, such as an ID, along with the file as it currently sits.

thank you very much @davidearl , but the code hadn't been update


Have DynamicEndpointGroup ignore updates that do not change the list …

…of endpoints and order them for more stable comparison.
Currently, DynamicEndpointGroup listeners are notified anytime an endpoint refresh happens, even if the endpoints didn't change. This doesn't seem useful so now DynamicEndpointGroup compares the old and new endpoints and only notifies when they change.
I guess insertion sort can be used for addEndpoint to avoid a copy but the complexity didn't seem worth it for a method that shouldn't be called with that much concurrency, but let me know if you'd prefer it.
Fixes #1216

Thanks, @anuraaga and reviewers!


Place TFloat type in the tesseract namespace

Place TFloat type in the tesseract namespace, same as has been done with all other tesseract defined types, to prevent collisions with thirdparty software.

Merged by cherry picking the relevant commit, thanks.


App settings subtitle, we don't provide "country"

Fixed in https://github.com/openfoodfacts/smooth-app/commit/781a6761dda40b67b7f0599566363d7a45bea3a6


test_models.py:246-247: Create fixtures for refarm tests....

The puzzle 162-e3264da6 from #162 has to be resolved:
https://github.com/fidals/refarm-site/blob/512cfa82ec777bb9db66fda2f3ad1580905f0ffa/tests/catalog/test_models.py#L246-L247
The puzzle was created by duker33 on 25-Aug-18.
Estimate: 60 minutes,
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

rm https://github.com/fidals/refarm-site/issues/166 pdd task too

The puzzle 162-e3264da6 has disappeared from the source code, that's why I closed this issue.


Wrong mixup… (cf german and spanish)

Thanks fewieden.

Please send your pull requests the develop branch.
Don't forget to add the change to CHANGELOG.md.

Note: Sometimes the development moves very fast. It is highly
recommended that you update your branch of develop before creating a
pull request to send us your changes. This makes everyone's lives
easier (including yours) and helps us out on the development team.
Thanks!

Does the pull request solve a related issue?

Mixup between german and spanish : https://github.com/MichMich/MagicMirror/pull/1352#issuecomment-403960925

If so, can you reference the issue?
What does the pull request accomplish? Use a list if needed.
If it includes major visual changes please add screenshots.

Warnings

:warning:

Please include an updated CHANGELOG.md file.This way we can keep track of all the contributions.

  Generated by :no_entry_sign: dangerJS


FIX Adjust HTMLEditorField tests to support alt attr changes in assets

The default behaviour of the alt attribute has changed from using the filename to applying an empty value.
See the test failure.

CI failure on the PR is caused by the Travis config pointing to 4.4.x-dev for core dependencies, instead of 4.x-dev, causing the tests to failure in the opposite direction 😅The Travis config will need to be updated to get this green.

This PR is a follow up on https://github.com/silverstripe/silverstripe-assets/pull/338/files


Fix table layout in Startup guide documentation

1. Why is this change necessary?
The table under "The sources" on https://docs.shopware.com/en/shopware-platform-dev-en/system-guide/startup-guide is broken:

2. What does this change do, exactly?
The changes contained in this pull request will fix the table layout:

3. Describe each step to reproduce the issue or behaviour.

Go to https://docs.shopware.com/en/shopware-platform-dev-en/system-guide/startup-guide
Scroll down to the table, shown below the headline "The sources"

4. Please link to the relevant issues (if any).
:heavy_minus_sign:
5. Checklist

[X] I have written tests and verified that they fail without my change
[X] I have squashed any insignificant commits
[ ] I have created a changelog file with all necessary information about my changes
[X] I have written or adjusted the documentation according to my changes
[X] This change has comments for package types, values, functions, and non-obvious lines of code
[X] I have read the contribution requirements and fulfil them.

Will create an internal PR.


Upgrade spark 2.0 and prepare 0.4.0-SNAPSHOT

This PR migrates spark-xml to Spark 2.0.
This PR,

deprecates saveAsXmlFile and promote the usage of write().
deprecates xmlFile and promote the usage of read().
drops 1.x compatibility from 0.4.0.
makes not supporting UserDefinedType as it became private.

Hi @liancheng and @JoshRosen , this PR resembles the ones in spark-redshift and spark-avro. Could you please review this?

Current coverage is 90.28% (diff: 100%)

Merging #150 into master will increase coverage by 0.08%

@@             master       #150   diff @@
==========================================
  Files            14         14
  Lines           663        669     +6
  Methods         605        612     +7
  Messages          0          0
  Branches         58         57     -1
==========================================
+ Hits            598        604     +6
  Misses           65         65
  Partials          0          0

Powered by Codecov. Last update 0c74b41...29acc30

@JoshRosen @liancheng Could you please take a quick look?

@JoshRosen @liancheng Could you just quickly scan this out for some not sensible changes here please?

Maybe I will merge this after double-checking this by myself if you don't have time to review this (but I will wait more just in case) - @JoshRosen @liancheng

Will this be released soon?

Yeap, I am thinking doing this on this weekend.

Thanks for the information
On Thu, Sep 8, 2016 at 3:03 PM, Hyukjin Kwon<EMAIL_ADDRESS>wrote:

Yeap, I am thinking doing this on this weekend.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/databricks/spark-xml/pull/150#issuecomment-245607483,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AS_MPqLonai9E_j-uHQjlsORBenBhJVIks5qoBWzgaJpZM4JZOUW
.

I am going to merge this as soon as the tests pass.

I am using spark-xml_2.10-0.2.0.jar but still facing the same issue


VisualScript: Fixed VariableGet vs VariableSet node names, fixes #6325

Fixes #6325 name mismatch.

Please do not merge visual script stuff for some days, still working on it
and doing large changes
On Aug 30, 2016 12:08, "J08nY"<EMAIL_ADDRESS>wrote:

Fixes #6325 https://github.com/godotengine/godot/issues/6325 name
mismatch.
You can view, comment on, or merge this pull request online at:
https://github.com/godotengine/godot/pull/6332
Commit Summary

VisualScript: Fixed VariableGet vs VariableSet node names, fixes
#6325

File Changes

M modules/visual_script/visual_script_nodes.cpp
https://github.com/godotengine/godot/pull/6332/files#diff-0 (6)

Patch Links:

https://github.com/godotengine/godot/pull/6332.patch
https://github.com/godotengine/godot/pull/6332.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/pull/6332, or mute the thread
https://github.com/notifications/unsubscribe-auth/AF-Z25ZuUHkOf8U3CxkAoAqVcQLKYbQxks5qlELPgaJpZM4JwnPO
.

I guess there are a lot of enthusiastic developers on Godot who take it easy :)


Download isn't available

The download is not available:
https://www.microsoft.com/download/confirmation.aspx?id=23734

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

ID: 83c5b210-2ecd-fc40-54c9-38a8ba9c7493
Version Independent ID: a9236354-4965-a214-17f7-746c20de8ec4
Content: Connect to data in an Access database (Windows Forms) - Visual Studio
Content Source: docs/data-tools/connect-to-data-in-an-access-database-windows-forms.md
Product: visual-studio-windows
GitHub Login: @gewarren
Microsoft Alias: gewarren

Thanks, I'll try to find a better target and/or determine if that step is still required.

Thanks, I got an error message "Unrecognized database format....".
I installed Microsoft Access Database Engine 2016 Redistributable from here:
https://www.microsoft.com/en-us/download/details.aspx?id=54920 .
When I tried to connect I got a different error message "microsoft ace oledb 12.0 provider is not registered on the local machine."
I uninstalled Office(64) and installed 32 bit version of Office and the Microsoft Access Database Engine, now it's working.


Website  headings are rendered broken

The Website has issues with Headings
See Here

Is this being worked on? It's really annoying :sweat_smile:
If not, I can take a look...


Pneumatic valve can bypass pressure limits

https://github.com/space-wizards/space-station-14/assets/39032829/1d37804c-09b7-42e1-b5eb-6168edb11088
Description
If the control line of a pneumatic pump is over 4500kpa it will move all gas from the input into the output and back and forth.
Reproduction

Make a pneumatic pump, put over 4500kpa gas (5000/9000kpa) in the control line.
Put a pump into the input and a pipe for output.
Enjoy.

Additional context
Somewhat old issue at this point, (month or so) so might not be a issue related to any recent changes.

Also first issue on github, so if you got any info on any mistakes in formatting or such, please do correct me.

I've seen it happen with pumps as well... or, well, I was using filters but just using them as pumps.

I've seen it happen with pumps as well... or, well, I was using filters but just using them as pumps.

this is just speedup and is unrelated

pretty sure radiators had essentially the same exact issue, which i fixed


Consider - a new name for this project?

Seems like there's a company already doing drone automation software in this space -
https://twitter.com/OpenPilot
You could curate this - like by voting - perhaps a simple google form?
Problem with not renaming it is reasonably foreseeable - when people want to extend this software to actually do drone stuff - then the naming will trip people up.

We considered this, according to their docs the other OpenPIlot was discontinued and renamed to LibrePilot.
http://opwiki.readthedocs.io/en/latest/user_manual/gcs_install_op.html

Oh Come on @geohot  That is a lame excuse... some of us are still around and still make use of OpenPIlot hardware (and respect the name)... The project was not "renamed" to LibrePilot, some of the devs forked off and did their own thing.
What you referenced is NOT the OpenPilot "project docs"... it never was official project documentation.
I personally still sell OpenPIlot hardware... (including a navigation capable flight / ground vehicle controller)
https://sasquatchlabs.org/product/openpilot-oplink-mini-modem-bundle/
https://sasquatchlabs.org/product/genuine-nano-flight-controller-w-case/


chore(deps): upgrade compiler dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-compiler-dependencies-k8s-27-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3573

Questions ?
Please refer to the Backport tool documentation


Kindred Segregation

text: Reword last wording: pool cost to recruit of zero, but they have a cost (eg. Exile)
link: https://www.vekn.net/forum/rules-questions/79961-what-is-the-cost-of-an-imbued#105962

Duplicate


Tests for flask API’s

Hi @ivantha!
We should add some tests for flask API’s . It would help us in continuous integration and for easy testing of our API’s .I am working on tests for the last couple of days and will make a PR soon.
Thanks

@ivantha
I am facing a problem regarding importing module and this is my projects structure

Should i change the directory name from fact-bounty-flask to fact_bounty_flask as invalid syntax error comes when we try to import dashed file name in python?

Should i change the directory name from fact-bounty-flask to fact_bounty_flask as invalid syntax error comes when we try to import dashed file name in python?

Is there any other way to achieve this without renaming the directory?

It's done in https://github.com/scorelab/fact-Bounty/pull/135

@ivantha
The error was solved without changing the name and I had created a PR with the tests!
Thanks


Incorrect error handling for invalid input names on SavedModel export

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Any
TensorFlow installed from (source or binary): Binary
TensorFlow version (use command below): 2.3.1
Python version: 3.7.6

Describe the current behavior
import tensorflow as tf

A_FEATURE_NAME = 'a feature'
B_FEATURE_NAME = 'b_feature'

class MyModel(tf.keras.Model):

    def call(self, inputs):
        return inputs[A_FEATURE_NAME] + inputs[B_FEATURE_NAME]

inputs = {
    A_FEATURE_NAME: tf.constant([1.]),
    B_FEATURE_NAME: tf.constant([2.]),
}

model = MyModel()
outputs = model(inputs)

model.save('test')

Saving this model gives the following error message
ValueError: Got non-flat/non-unique argument names for SavedModel signature 'serving_default': more than one argument to '__inference_signature_wrapper_48' was named 'a feature'. Signatures have one Tensor per named input, so to have predictable names Python functions used to generate these signatures should avoid *args and Tensors in nested structures unless unique names are specified for each. Use tf.TensorSpec(..., name=...) to provide a name for a Tensor input.

The reason however is that A_FEATURE_NAME has an invalid name due to it containing whitespace, so it ends up not matching the placeholder name when tracing the graph for saving: https://github.com/tensorflow/tensorflow/blob/5d8b439eb41fb2373c5b5c5f33c51313cfd9e57f/tensorflow/python/saved_model/save.py#L494
Describe the expected behavior
TensorFlow checks for valid names elsewhere (such as when declaring an op), so the same logic should be used to return a more relevant error e.g., ValueError: 'a feature' is not a valid input name

I ran the code shared and i am able to replicate the error reported, please find the gist here.

@dwyatte
import tensorflow as tf

A_FEATURE_NAME = 'a feature'
B_FEATURE_NAME = 'b_feature'

class MyModel(tf.keras.Model):

    def call(self, inputs):
        return inputs['A_FEATURE_NAME'] + inputs['B_FEATURE_NAME']

inputs = {
    'A_FEATURE_NAME': tf.constant([1.]),
    'B_FEATURE_NAME': tf.constant([2.]),
}

model = MyModel()
outputs = model(inputs)

model.save('test')

@bhack thanks for the workaround, but I still think the wrong error is raised here. more than one argument to '__inference_signature_wrapper_48' was named 'a feature' suggests that I am using the same the same op name twice and in fact, the source code mentions # This should be unreachable, since concrete functions may not be generated with non-unique argument names. Instead, it might be nicer if tensorflow checked for valid op names before the aforementioned error is reached in tf.keras.Model.save and raised a more informative error (something like ValueError: 'a feature' is not a valid input name).

@dwyatte The problem with your issue is that invalid name are replaced to "

@bhack Thanks for the heads up, I was not aware there were changes in master that might handle address this. With '2.5.0-dev20201127' I do see the warning you mention, but I still receive an error which suggests the renaming is not happening:
...
WARNING:absl:Function `_wrapped_model` contains input name(s) a feature with unsupported characters which will be renamed to placeholder in the SavedModel.
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-1-bc463538c686> in <module>
     17 outputs = model(inputs)
     18
---> 19 model.save('test')

~/venv/lib/python3.7/site-packages/tensorflow/python/keras/engine/training.py in save(self, filepath, overwrite, include_optimizer, save_format, signatures, options, save_traces)
   2046     # pylint: enable=line-too-long
   2047     save.save_model(self, filepath, overwrite, include_optimizer, save_format,
-> 2048                     signatures, options, save_traces)
   2049
   2050   def save_weights(self,

~/venv/lib/python3.7/site-packages/tensorflow/python/keras/saving/save.py in save_model(model, filepath, overwrite, include_optimizer, save_format, signatures, options, save_traces)
    156   else:
    157     saved_model_save.save(model, filepath, overwrite, include_optimizer,
--> 158                           signatures, options, save_traces)
    159
    160

~/venv/lib/python3.7/site-packages/tensorflow/python/keras/saving/saved_model/save.py in save(model, filepath, overwrite, include_optimizer, signatures, options, save_traces)
     94       with utils.keras_option_scope(save_traces):
     95         saved_nodes, node_paths = save_lib.save_and_return_nodes(
---> 96             model, filepath, signatures, options)
     97
     98     # Save all metadata to a separate file in the SavedModel directory.

~/venv/lib/python3.7/site-packages/tensorflow/python/saved_model/save.py in save_and_return_nodes(obj, export_dir, signatures, options, raise_metadata_warning)
   1077   _, exported_graph, object_saver, asset_info, saved_nodes, node_paths = (
   1078       _build_meta_graph(obj, signatures, options, meta_graph_def,
-> 1079                         raise_metadata_warning))
   1080   saved_model.saved_model_schema_version = constants.SAVED_MODEL_SCHEMA_VERSION
   1081

~/venv/lib/python3.7/site-packages/tensorflow/python/saved_model/save.py in _build_meta_graph(obj, signatures, options, meta_graph_def, raise_metadata_warning)
   1263   with save_context.save_context(options):
   1264     return _build_meta_graph_impl(obj, signatures, options, meta_graph_def,
-> 1265                                   raise_metadata_warning)

~/venv/lib/python3.7/site-packages/tensorflow/python/saved_model/save.py in _build_meta_graph_impl(obj, signatures, options, meta_graph_def, raise_metadata_warning)
   1197   asset_info, exported_graph = _fill_meta_graph_def(meta_graph_def,
   1198                                                     saveable_view, signatures,
-> 1199                                                     options.namespace_whitelist)
   1200   if options.function_aliases:
   1201     function_aliases = meta_graph_def.meta_info_def.function_aliases

~/venv/lib/python3.7/site-packages/tensorflow/python/saved_model/save.py in _fill_meta_graph_def(meta_graph_def, saveable_view, signature_functions, namespace_whitelist)
    699
    700   with exported_graph.as_default():
--> 701     signatures = _generate_signatures(signature_functions, resource_map)
    702     for concrete_function in saveable_view.concrete_functions:
    703       concrete_function.add_to_graph()

~/venv/lib/python3.7/site-packages/tensorflow/python/saved_model/save.py in _generate_signatures(signature_functions, resource_map)
    563     mapped_inputs, exterior_argument_placeholders = (
    564         _map_function_arguments_to_created_inputs(argument_inputs,
--> 565                                                   signature_key, function.name))
    566     outputs = _call_function_with_mapped_captures(
    567         function, mapped_inputs, resource_map)

~/venv/lib/python3.7/site-packages/tensorflow/python/saved_model/save.py in _map_function_arguments_to_created_inputs(function_arguments, signature_key, function_name)
    504            "tf.TensorSpec(..., name=...) to provide a name for a Tensor "
    505            "input.").format(signature_key, compat.as_str_any(function_name),
--> 506                             user_input_name))
    507     arg_placeholder = array_ops.placeholder(
    508         shape=placeholder.shape,

ValueError: Got non-flat/non-unique argument names for SavedModel signature 'serving_default': more than one argument to '__inference_signature_wrapper_42' was named 'a feature'. Signatures have one Tensor per named input, so to have predictable names Python functions used to generate these signatures should avoid *args and Tensors in nested structures unless unique names are specified for each. Use tf.TensorSpec(..., name=...) to provide a name for a Tensor input.

The problem is that you are going ti mismatch between featuere a and substituted name placeholder at
https://github.com/tensorflow/tensorflow/blob/21eb35ad7c4c95343c872f68b7f298b66afc9b0e/tensorflow/python/saved_model/save.py#L494

Reading the comment above this conditional
https://github.com/tensorflow/tensorflow/blob/21eb35ad7c4c95343c872f68b7f298b66afc9b0e/tensorflow/python/saved_model/save.py#L494
makes me think it should somehow be rewritten to not capture cases where inputs are simply renamed on saving.
    # If the internal placeholders for a function have names which were
    # uniquified by TensorFlow, then a single user-specified argument name
    # must refer to multiple Tensors. The resulting signatures would be
    # confusing to call. Instead, we throw an exception telling the user to
    # specify explicit names.

If I execute
https://github.com/tensorflow/tensorflow/blob/21eb35ad7c4c95343c872f68b7f298b66afc9b0e/tensorflow/python/saved_model/save.py#L507
I get the error I would expect:
arg_placeholder = array_ops.placeholder(shape=placeholder.shape, dtype=placeholder.dtype,name="{}_{}".format(signature_key, user_input_name))
*** ValueError: 'serving_default_a feature' is not a valid scope name

You could probably open a small PR adding an and condition for Placeholder name an see if the CI will pass tests.

Still issue exists in 2.5 and Please find the gist here.Thanks!

In case this helps anyone, I had the same errors but for a different (buggy) reason. I created an InputLayer and set the name to "Input" not thinking much of it. For whatever reason, that triggered this warning during save:

WARNING:absl:Function _wrapped_model contains input name(s) a feature with unsupported characters which will be renamed to placeholder in the SavedModel.

Then when I tried loading the model I got the error:

Unable to restore custom object of type _tf_keras_metric currently. Please make sure that the layer implements get_configand from_config when saving.

Even though I DID supply the custom_objects dict and saved using the SavedModel format. "Input" doesn't have unsupported characters but it triggered this issue anyway. Removing the name (or perhaps renaming to something else entirely?) avoided the problem but it doesn't seem to be a valid problem to begin with.

Confirmed fixed. Thank you!