[Tracking] Active navigational issues

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

I have started to notice a couple of navigation issues. Creating this issue to help us better manage those and track their solutions.
List of issues

[ ] https://github.com/Expensify/App/issues/6908

Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number:
Reproducible in staging?:
Reproducible in production?:
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
Issue reported by:
Slack conversation:
View all open jobs on GitHub

Triggered auto assignment to @MitchExpensify (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

Thanks @parasharrajat, tracking issue looks good. I don't think any further triage action is required of me but let me know if you think otherwise

made progress on the list.

This issue has not been updated in over 14 days.  eroding to Weekly issue.

This issue has not been updated in over 14 days.  eroding to Weekly issue.

This issue has not been updated in over 14 days.  eroding to Weekly issue.

This issue has not been updated in over 14 days.  eroding to Weekly issue.

This issue has not been updated in over 14 days.  eroding to Weekly issue.

This issue has not been updated in over 14 days.  eroding to Weekly issue.

This issue has not been updated in over 14 days.  eroding to Weekly issue.

Steadily making progress on issues one by one....

Assigned Rajat

Moving up...

Yeah that would be great. Thanks @mountiny

Just curious, how many of these fixes have happened in the react-navigation repo? How many would we say are "workarounds"? And if we worked around something was there some justification as to why documented somewhere?

@parasharrajat I'm going to close this and move all related issues into a GitHub project for now if you see any issues related to navigation please grab me.

Thank you for collaborating on these. This was much needed.


⚠ SceneTime has degraded performance

In 47335d7, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1269 ms

Resolved: SceneTime performance has improved in d7cb8c1 after 6 minutes.


Support for RFC 3986 URIs

A request made for http://localhost:6767/asdf;zxcv  comes out of mongrel2 as
34f9ceee-cd52-4b7f-b197-88bf2f0ec378 26 /asdf 562:{"PATH":"/asdf","x-forwarded-for":"<IP_ADDRESS>","accept-language":"en-US,en;q=0.8","accept-encoding":"gzip,deflate,sdch","connection":"keep-alive","accept-charset":"ISO-8859-1,utf-8;q=0.7,;q=0.3","accept":"application/xml,application/xhtml+xml,text/html;q=0.9,text/plain;q=0.8,image/png,/*;q=0.5","user-agent":"Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/534.23 (KHTML, like Gecko) Chrome/11.0.686.3 Safari/534.23","host":"localhost:6767","cookie":"csrftoken=3688f60fec4b032fafc9d320a2e19456","METHOD":"GET","VERSION":"HTTP/1.1","URI":"/asdf;zxcv","PATTERN":"/"},0:,
The URI header is right.
The route looks like
brubeck_host = Host(
name="localhost",
routes={
'/media/': media_dir,
'/': brubeck_handler})

Technically a semi-colon is allowed in the path part of the URI, so this is a valid complaint.  I'll let Zed comment on it though, as he has purposefully left out uncommonly used parts of the standard for security reasons.

It's a pretty specific use-case: I have an api handler and i'd like to be able to support multiple ids in a single http request. Using ; is the recommended way of doing that according to the o'reilly restful web services book.

Looking at the code, it just seems that the URI parser is based off of RFC 2396 rather than the newer 3986.  Does the book mention any alternatives (especially since 2396  is more widely supported than 3986).

',' works so I'm happy  - Since ; is a legal char according to the spec I'm going to leave this open. If it's disallowed for good reason then close away

I renamed this and will leave it open for now.  Note that RFC 3986 is much newer than the HTTP 1.1 standard.

  The ; is reserved for Path parameters and is NOT part of a URL.  It's been that way since the dawn of HTTP 1.1 and the only reason it's even allowed is because Apache is file-centric and lets people do it if there's a file with a ;
In the context of an app server, you're supposed to get these as path parameters in the header, and if you want them to be part of the URL scheme then the app server backend should deal with it or just not allow it.
So what's the state on this?  I saw the commit and it looks fine but what I'm not sure of is how this is fixed.

Here's the reference in the current RFC that's actually being used by HTTP 1.1:
http://tools.ietf.org/html/rfc2396
Section 3.3:
"Each path segment may include a sequence of parameters, indicated by the semicolon ";" character. The parameters are not significant to the parsing of relative references."
So, it is not part of the path, it's parameters after that.  Now to find out how this has impacted the server's behavior.


Add yarbs v0.1.7

Thank you for the contribution. It was merged directly as 415cb1db5b5c90a2520fa96534c6dfb8e98e9ba3 to keep commit history cleaner. Your contribution is still credited to you.


Use iptables with nginx-proxy

Hello,
First all, thank to jwilder for this perfect project that I use since one month ! :)
Now, I have a litte question about iptables.
Actually, I installed Fail2Ban in a docker and f2b analyses logs of nginx-proxy to ban ip.
After X bad tryings to connect to my admin website, the ip address is banned for 15 minutes thanks to iptables.
Unfortunately, for an unknown reason, it does not work with ports 80 and 443 (linked to nginx-proxy).
So, if I do these commands (probably not necessary to do all), I'll be suppose to be completely blocked.

iptables -A INPUT -s public_address_ipv4 -j REJECT
iptables -A DOCKER -s public_address_ipv4 -j REJECT
iptables -A DOCKER-ISOLATION -s public_address_ipv4 -j REJECT

Obviously, my SSH access is blocked but I always can access to my website...
How can I prevent an ip to access to access to my website linked to nginx-proxy ?
If someone has an idea, it would be great ! :)
PS : Sorry for my English, it's not my mother tonge

My guess is that fail2ban default only rejects ssh traffic (port 22) on ssh login fails. There seems to be something called "allports" for fail2ban but i can't really figure how to use it.

You're right, there's an action called allports which block all ports for the address.
But normally if I do iptables -A INPUT -s public_address_ipv4 -j REJECT, the IP should be blocked.
It is for ssh but not for web.

Finally I found a solution to use iptables and fail2ban with nginx-proxy :)
You can find Fail2Ban for Docker on my GitHub here : https://github.com/SuperITMan/docker-fail2ban
Here is how to use iptables to prevent access to docker web host
iptables -N PRE-DOCKER
              iptables -I FORWARD -o docker0 -j PRE-DOCKER
              iptables -A PRE-DOCKER -o docker0 -m conntrack --ctstate RELATED,ESTABLISHED -j ACCEPT
              iptables -A PRE-DOCKER -i docker0 ! -o docker0 -j ACCEPT
              iptables -A PRE-DOCKER -m state --state RELATED -j ACCEPT
              iptables -A PRE-DOCKER -i docker0 -o docker0 -j ACCEPT

Thanks to Rudi Starcevic for his solution https://forums.docker.com/t/restricting-external-container-access-with-iptables/2225/3

Even if the post is already a bit older, I have a question:
How did you get to the logs? The file access.log is actually not available at all.
	&& ln -sf /dev/stdout /var/log/nginx/access.log \
	&& ln -sf /dev/stderr /var/log/nginx/error.log

Please?


[Question] When will the version 5 code be pushed?

When will the version 5 code be pushed?

@namilkimfree Its available now from the develop branch: https://github.com/extent-framework/extentreports-csharp/tree/develop


speed.exe has stopped working

Hello! All good? My name is Waldecy and I have a YouTube channel called Banko Games. I'm using "Xenon Effects" in NFSMW 2005 and also the last update of "Extra Options". When I press the "Backspace" key the game stops with the following message box "speed.exe has stopped working". Can you help me?

https://github.com/ExOptsTeam/NFSMWExOpts/releases/tag/v<IP_ADDRESS>7
You're not running the latest build, then. Use this build and it'll work.

Thanks.

It worked here. Thanks. Is there a way to return the "ShowMoreCustomizationOptions" function in NFSMWExtraOptionsSettings? I thank.


Add CannotInsertFlag to indicate that this table cannot be inserted

What problem does this PR solve?
Include no_zero_date in sqlmode, these scenarios should fail to add column:
create table t (id int);
insert into t values(1);
alter table t add column a date not null;
alter table t add column a date not null;

Considering that data may be written during the DDL state change phase, need a flag to prevent data insert during this period. So add this flag.
What is changed and how it works?
Check List
Tests

No code

Code changes

Has exported variable/fields change

Side effects
Related changes

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@bb7133  PTAL


400 error when downloading multiple plots

The tool was working fine downloading just one plot, but once another was published I get the following output:
Exception occurred:
Error while getting response for order/plot ID=Y*****d: http_status=400, response={"id":["This field is required."]}
Will wait for 18 s and then retry.

It appears the download is still actually processing, as I can see the files growing, but the progress file doesn't get updated and the console just continually shows the above.
Here is the full trace from the log:
2021-06-23 19:24:38 GMT Summer Time [__init__:184 - INFO]: Reporting the plot ID=Y*****d download state via API.
2021-06-23 19:24:38 GMT Summer Time [__init__:265 - WARNING]: Non-200 status code while getting plots for order ID=Y*****d: 400.
2021-06-23 19:24:38 GMT Summer Time [__init__:267 - DEBUG]: {"id":["This field is required."]}
2021-06-23 19:24:38 GMT Summer Time [decorator:34 - ERROR]: Error while executing report_downloading
Traceback (most recent call last):
  File "C:\Users\USERNAME\Documents\GitHub\plotorder\config\decorator.py", line 24, in wrapper
    return method(*args, **kwargs)
  File "C:\Users\USERNAME\Documents\GitHub\plotorder\client\__init__.py", line 191, in report_downloading
    self._check_response(response, plot.plot_id)
  File "C:\Users\USERNAME\Documents\GitHub\plotorder\client\__init__.py", line 268, in _check_response
    raise ConnectionError(f'Error while getting response for order/plot ID={request_id}: '
ConnectionError: Error while getting response for order/plot ID=Y*****d: http_status=400, response={"id":["This field is required."]}

Here is my config file:
[API]
api_url=http://chiafactory.com/api/v1/
api_key=6***********************************6

[LOCAL]
plot_dir=X:\
progress_file=C:\Users\USERNAME\Documents\GitHub\plotorder\progress.log

EDIT:
As mentioned, the download does continue in the background despite the errors... however there is a retry limit which will then just stops the process entirely.

sorry for the late reply, this is, unfortunately, a very silly bug. You need to remove the trailing slash in api_url.
So instead of api_url=http://chiafactory.com/api/v1/ it needs to be api_url=https://chiafactory.com/api/v1 (I also added https)
We'll be publishing a new Go version soon.

New Go version is published.


MAKE-LINE behaving oddly with small values of stroke weight

The following sketch is setup to do 2 things:

Change the coordinate system to place (0, 0) in the center of the window and range from -1 to 1 in both coordinates
Plot 0.3*sin(7x)

(defsketch make-line-bug ((title "MAKE-LINE bug")
                          (width 600)
                          (height 400)
                          (ratio (/ width height)))
  (scale (/ width 2 ratio) (/ height 2))
  (translate ratio 1)
  (background +black+)
  (let ((points (loop for x from (- ratio) upto ratio by (/ 1 1000)
                      collect (cons x (* (sin (* 7 x)) 0.3)))))
    (with-pen (make-pen :stroke +red+ :weight (/ 1 height))
      (loop for (x1 . y1) in points for (x2 . y2) in (cdr points) do
            (line x1 y1 x2 y2)))))

If you run it, you should be presented the following which hardly resemble what one might expect when trying to plot sin(x):

After a little bit of fiddling with skecth's source, I realized MAKE-LINE was using FLOOR to figure out the triangle strip to draw, and when I replaced FLOOR with / things immediately started to look better:
diff --git a/src/shapes.lisp b/src/shapes.lisp
index 19d786e..415191d 100644
--- a/src/shapes.lisp
+++ b/src/shapes.lisp
@@ -18,9 +18,9 @@
          (w (/ (or (pen-weight (env-pen *env*)) 1) 2))
          (dx (* 2 (sin a) w))
          (dy (* 2 (cos a) w))
-         (dx+ (floor dx 2))
+         (dx+ (/ dx 2))
          (dx- (- dx dx+))
-         (dy+ (floor dy 2))
+         (dy+ (/ dy 2))
          (dy- (- dy dy+)))
     (lambda ()
       (draw-shape

I am sure there is a reason why FLOOR was chosen instead of /, but it clearly does not work as expected when PEN-WEIGHT returns avalue smaller than 1 (1/400 in this example).

It indeed seems like an exact division would suit better here.
Another example:
(defsketch make-line-bug ()
  (translate 100 100)
  (scale 10)
  (with-pen (make-pen :weight 3 :stroke +black+ :fill +white+)
    (destructuring-bind (x y x* y*) (list 1 2 1 9)
      (line x y x* y*)
      (circle x y 1/10)
      (circle x* y* 1/10))))

The result:

(You can see that the line is not properly centered)
After applying the fix:

Thank you both!


Bugfix: anyOf and array schema mapping

This PR makes two important changes to the way array values are mapped in schemas:

The property array response mapper first checks if a value is already a JSON string before attempting to stringify it for use in code input components
Schemas attempt to parse a value when deciding which union property to display

There's still a bug here where validators for fieldsets in the union will apply to the incorrect member, but I'm working on that

Also found the source of the non-reactive field-level validators @pleek91 so those are fixed in this as well


cd ( change directory in remote )

remote.sudo('cd ~/one',{user:username});
remote.sudo('pwd',{user:username});
showing the result home directory of the user (~)

You should use transport#with(). Docs: https://github.com/pstadler/flightplan#transportwithcommandoptions-options-fn


standard deployment support

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
unRAID support would be ideal. Deploying custom containers derived from your workspace with custom dockerfile/permissions/privelaged changes is still causing every error in the book it seems....
Describe the solution you'd like
A clear and concise description of what you want to happen.
unRAID/TrueNAS support with easy configuration for all apps including VPN/wireguard. An IBRACORP walkthrough after that is polished would also be nice.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
N/A
Additional context
Add any other context or screenshots about the feature request here.
N/A

I have zero experience with unRAID and TrueNAS, so I would like it if someone else could have a look at this. I might get around to it at some point, but it doesn't have much priority for me right now.


Add back WidevineCDM (Netflix)

Ummmm what happened to Netflix? 🤔

#71 As I mentioned, at the moment, I removed support WidevineCDM plugin which is required to run Netflix. This is because the technology is not open-source and it takes a lot of time to keep updating the plugin manually.
Removing it allows the plugin allows me to keep development forward. But I'll keep this issue open because as soon as I figure out a way to automate the process, I'll bring it back.

webcatalog/electron-widevinecdm I'm working to bring it back :)


Installation

I am not sure where to install the mod. I put the jar in my plugins folder, but nothing happened when I started the server. Does this work for Paper servers?

No, this is for forge


Fluent Validator

Add fluent validator on API request to validate state of request.

Issue closed by #9


1⃣5⃣: Implement skeleton for per-wallet page.

The sections' contents are stubbed out for now but the structure is in place.
The intention is to have a more complete explanation of the wallet ratings, why the attributes are what they are and why they matter, as well as a full view of the wallet information on record.
This is PR 1⃣5⃣.
Previous PRs this one sits on top of:

1⃣4⃣: #51
1⃣3⃣: #50
1⃣2⃣: #49
1⃣1⃣: #48
🔟: #46
9⃣: #45
8⃣: #44
7⃣: #37
6⃣: #43
5⃣: #42
4⃣: #41
3⃣: #40
2⃣: #39
1⃣: #38

hey @polymutex - this is the error Vercel is throwing:
  Types of property 'params' are incompatible.
    Type 'WalletUrlParams' is missing the following properties from type 'Promise<any>': then, catch, finally, [Symbol.toStringTag]


tailsitter  fly forward and fall suddenly

In position mode,tailsiter fly forward , but fall sudenly.
RC alt chanel(rc_channels_0.channels[3]) no action，Why vehicle_local_position_setpoint decrised.
https://logs.px4.io/plot_app?log=e395b143-1e01-4195-b385-77ca6ab75ef7

Sample problem as in #9032


Fix potential github action smells

Fixes
Hey! 🙂
I want to contribute the following changes to your workflow:

Avoid running CI related actions when no source code has changed
Avoid executing  scheduled workflows on forks
Avoid jobs without timeouts

(These changes are part of a research Study at TU Delft looking at GitHub Action Smells. Find out more)
Proposed Changes/Todos
Pull Request checklist:

[ ] I've named my PR in the form of "Fixes #issue. Terse description."
[ ] My code follows the style guidelines of Terminal.Gui - if you use Visual Studio, hit CTRL-K-D to automatically reformat your files before committing.
[ ] My code follows the Terminal.Gui library design guidelines
[ ] I ran dotnet test before commit
[ ] I have made corresponding changes to the API documentation (using /// style comments)
[ ] My changes generate no new warnings
[ ] I have checked my code and corrected any poor grammar or misspellings
[ ] I conducted basic QA to assure all features are working

Also, a thought for future consideration for us before it disappears from my head again:
Especially with regards to Roslyn analyzers/generators, we may want to consider caching, so that they don't have to get built every time, unless one of the analyzer projects is actually modified. Would speed up the build actions.

Is this ready for me to merge?

Hey @tig :
Closely related to this, and also might influence what gets added to the gh marketplace action I'm working on:
What do you have the "Actions permissions" and "Workflow Permissions" set to for this repository, under settings > actions > general? I wanna be sure this repo is covered from the start, since it was the original catalyst for making me do it now rather than [undefined future time range] in the first place. :)
And do you have any environments configured? (that's not important - just surveying what's in use here).
And any defined rulesets or branch/tag protections?
Also, any difficulties or wish list items vis-a-vis the actions/workflows/deployment/builds/etc?
Aside from splitting the analyzers - that is dependent on these other tasks I've got outstanding.


Add tests

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.85%. Comparing base (ea703c8) to head (6fe0a43).
Report is 1 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##               main     #253      +/-   ##
============================================
+ Coverage     99.15%   99.85%   +0.70%
+ Complexity      688      686       -2
============================================
  Files            62       62
  Lines          2135     2131       -4
============================================
+ Hits           2117     2128      +11
+ Misses           18        3      -15

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


autocmd may change cwd after :tcd and :lcd

Problem:  Autocommand may change currect directory after :tcd and :lcd.
Solution: Also clear tp_localdir and w_localdir when using aucmd_win.

thanks!


accuracy not high

Hello sir
Nice work! However, I trained with the hparams as you suggested, and it shows that the validation accuracy is about 0.7, while the paper reported their accuracy is 98.7%. So I was wondering if I did something wrong or you got the same result?

Hi, I haven't trained it for long as the original study. However, yesterday I fixed a bug that improved convergence behavior. Can you install version 0.3 and train it again? Also, in the original study, image augmentation is employed but it is missing in this repo, so it might be another reason for the difference. I'll will implement image augmentation soon.

Thanks for your reply.
version 0.3 performs better. The model converges after about 100 epochs, validation accuracy reaches 0.85. Looking forward to your follow-up work.

Cool, thanks for sharing it. I have added image augmentation for MNIST as in the original study. It pads each image to size of 40 and randomly shifts in vertical and horizontal direction. You can try it now.

Hello sir
It seems that something is wrong about reconstruction likelihood? The rec_ll_loss stays high during the whole training.

Hello sir
It seems that something is wrong about reconstruction likelihood? The rec_ll_loss stays high during the whole training progress. Also the rec_ll_loss is much bigger than the other loss component. Did I get it wrong?

Hi,
With default hyperparameters, what I observed is that reconstruction loss decreases very slowly. But, it seems same with the original implementation. Somehow, it takes model many steps to learn templates. How long did you train it and with what hyperparameters?

Hi,
I tried the default hyperparameters, which is same as the tensorflow version (batchsize=128, learning rate=3e-5) .But the result is worse than before(batchsize=32, learning rate=1e-4).
I alse tried tensorflow version, and it seems that the loss curves are different.
tensorflow version:

pytorch lightning version:

Hi, thanks for sharing those. The result might be worse than previous version because it augments images on default. Also, I noticed that I did not turn learning rate scheduler on default, which is used in the original implementation. Changed it now. But, this is probably not the root cause. There might be a bug somewhere. I will check it when I have time.

I got the low accuracy as well, which configs could get more precise?

Thanks for your reply.
version 0.3 performs better. The model converges after about 100 epochs, validation accuracy reaches 0.85. Looking forward to your follow-up work.

Hi guys, could you tell me where to find version 0.3?
Thanks in advance.

Thanks for your reply.
version 0.3 performs better. The model converges after about 100 epochs, validation accuracy reaches 0.85. Looking forward to your follow-up work.

Hi guys, could you tell me where to find version 0.3?
Thanks in advance.

Hi cxiang26,
I did not create releases in this project, sorry for my inexperience. I guess you can check out to the commit cefabf93b8d65fb21cd14e3ff57efb473e055231 to get version 0.3.

We get a stable acc. over 80% if using grad_clip=2 or lesser.
Hope this is helpful.

We get a stable acc. over 80% if using grad_clip=2 or lesser.
Hope this is helpful.

I think that the key bug of this code is the initialization problem for the templates, I have got excellent results if I set the value of initialization templates to [0, 0.1] rather than [0, 1]. adding a line code q = 0.1 * q after line 67  of part_decoder.py.

@bdsaglam
Hi guys, I find this code could have below bugs compared to reimplemented experiments by myself.
BUG1: I think that the key bug of this code is the initialization problem for the templates. I got excellent results if I set the value of initialization templates to [0, 0.1] rather than [0, 1]. solution: adding a line code q = 0.1 * q after line 67  of part_decoder.py.
BUG2: Don't use Relu activation in the final layer of the caps_mlps network, which could lead to the value of cpr_dynamic larger than 0,  that makes the training is unstable prettily. Solution:  set 'activate_final=False' in object_decoder.py line 105.

@bdsaglam
Hi guys, I find this code could have below bugs compared to reimplemented experiments by myself.
BUG1: I think that the key bug of this code is the initialization problem for the templates. I got excellent results if I set the value of initialization templates to [0, 0.1] rather than [0, 1]. solution: adding a line code q = 0.1 * q after line 67 of part_decoder.py.
BUG2: Don't use Relu activation in the final layer of the caps_mlps network, which could lead to the value of cpr_dynamic larger than 0 always, that makes the training is unstable prettily. Solution: set 'activate_final=False' in object_decoder.py line 105.

Hello, I run version 0.3, the accuracy has been around 0.1, can you share your version？？？

We get a stable acc. over 80% if using grad_clip=2 or lesser.
Hope this is helpful.

@bdsaglam Hi guys, I find this code could have below bugs compared to reimplemented experiments by myself. BUG1: I think that the key bug of this code is the initialization problem for the templates. I got excellent results if I set the value of initialization templates to [0, 0.1] rather than [0, 1]. solution: adding a line code q = 0.1 * q after line 67 of part_decoder.py. BUG2: Don't use Relu activation in the final layer of the caps_mlps network, which could lead to the value of cpr_dynamic larger than 0 always, that makes the training is unstable prettily. Solution: set 'activate_final=False' in object_decoder.py line 105.

你好，请问一下你这个图片上的结果怎么得到的啊，我试了你说的方法，但是正确率还是在60%多，可不可以分享一下你得出这个实验结果的代码啊，求求了。

We get a stable acc. over 80% if using grad_clip=2 or lesser.
Hope this is helpful.

@cxiang26 你好，请问一下你这个图片上的结果怎么得到的啊，我试了你说的方法，但是正确率还是在60%多，可不可以分享一下你得出这个实验结果的代码啊，求求了。

@tuwangwenx 很久没有弄这个了， 你试试我的项目，应该可以的 https://github.com/cxiang26/stacked_capsule_autoencode_pl

Thanks for your reply. version 0.3 performs better. The model converges after about 100 epochs, validation accuracy reaches 0.85. Looking forward to your follow-up work.

could you share the version you have traied?  I tried many versions and only got the val_acc of about 13%.


Provide minified version

I couldn't find a minified version of this project. I think this might be a main blocker for people trying to use it in theirs project.

Thanks for your suggestion @pbartoszek.
I provide minified versions for some of my other JavaScript projects, but I'm not sure I want to maintain minified versions for jQuery File Upload.
One reason for this is the modular nature of the components:

Should a minified version include all components, including the UI version?
Should a minified version include the jQuery UI Widget Library?
Should a minified version include the Iframe Transport?
Should a minified version include the PostMessage CORS library?
Should a minified version include the XDomainRequest Transport, which is included for cross-domain file deletion for IE 8 and IE 9 only?
Should there be multiple minified versions?

Another reason is that committing minified versions is not ideal for source control, since every change invalidates the whole minified file.
I think providing a minified bundle still makes sense to quickly include it for use in demos and such, but this practice also encourages folks to include the library directly from Github in their production projects, which is not a recommendable practice.


Remove selectAssetsNoOutputs workaround

Issue Number
ADP-1136 (and others)
Summary
This PR removes the selectAssetsNoOutputs workaround.

bors try

bors try

bors r+


SIT Reg Organiser: Sort Order of Participant List

I find the default sort order of the participant list in the SIT Reg Organiser app to be not very useful (it sorts alphabetically by first name). Personally I always want to go in there to see who has most recently registered, so for me it would make sense that the list was ordered by "ID" so I can see who has registered first/last.
Of course this may not suit everyone, so perhaps we could change the default sort order and also add the ability to sort on any of the columns in the list?

Please check the commit https://github.com/sapmentors/SITregOrganizer/commit/af40085e4cdb4b1d4b5cbe37b23832e67f99f1e4 where I've added sorting & grouping. The change should be also visible in the app at https://sitregorganizer-a5a504e08.dispatcher.hana.ondemand.com/ . Please give it a try.


Headers containing consecutive commas fails to parse

From @Haplois on June 19, 2016 0:52
I've just upgraded to RC2. I'm using custom built SCRAM for authentication, in RC2 following HTTP request fails:
POST http://localhost:5001/services/authorization HTTP/1.1
SCRAM-SHA-256: n,,n=1,r=mhtzCDgCf3ZBKUpiYNFctfnFaXYf2MlSuZgc1Oc3DUM=
Host: localhost:5001
Content-Length: 0
Connection: Keep-Alive
Pragma: no-cache

A repro can be found here: https://github.com/Haplois/HeaderParseErrorRepro
It's because of ",," in "SCRAM-SHA-256" header, if I remove it request hits the action without an exception.
Steps to reproduce

Download repro, and run it.
Open Fiddler, and send this request for error message

POST http://localhost:3744/services/authorization HTTP/1.1
SCRAM-SHA-256: n,n=1,,r=mhtzCDgCf3ZBKUpiYNFctfnFaXYf2MlSuZgc1Oc3DUM=
Host: localhost:3744
Content-Length: 0
Connection: Keep-Alive
Pragma: no-cache

Send this request for successful header parsing

POST http://localhost:3744/services/authorization HTTP/1.1
SCRAM-SHA-256: n,n=1,r=mhtzCDgCf3ZBKUpiYNFctfnFaXYf2MlSuZgc1Oc3DUM=
Host: localhost:3744
Content-Length: 0
Connection: Keep-Alive
Pragma: no-cache

You'll get following exception
System.ArgumentOutOfRangeException: Specified argument was out of the range of valid values.
Parameter name: offset
  at Microsoft.Extensions.Primitives.StringSegment..ctor(String buffer, Int32 offset, Int32 length)
  at Microsoft.AspNetCore.Http.Internal.HeaderSegmentCollection.Enumerator.get_Current()
  at Microsoft.AspNetCore.Http.Internal.ParsingHelpers.<GetHeaderSplitImplementation>d__2.MoveNext()
  at System.Linq.Buffer`1..ctor(IEnumerable`1 source)
  at System.Linq.Enumerable.ToArray[TSource](IEnumerable`1 source)
  at Microsoft.AspNetCore.Http.Internal.ParsingHelpers.GetHeaderSplit(IHeaderDictionary headers, String key)
  at Microsoft.AspNetCore.Mvc.ModelBinding.Binders.HeaderModelBinder.BindModelAsync(ModelBindingContext bindingContext)
  at Microsoft.AspNetCore.Mvc.Internal.ControllerArgumentBinder.<BindModelAsync>d__7.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
  at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
  at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
  at Microsoft.AspNetCore.Mvc.Internal.ControllerArgumentBinder.<PopulateArgumentsAsync>d__10.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
  at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
  at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
  at Microsoft.AspNetCore.Mvc.Internal.FilterActionInvoker.<InvokeAllActionFiltersAsync>d__40.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
  at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
  at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
  at Microsoft.AspNetCore.Mvc.Internal.FilterActionInvoker.<InvokeExceptionFilterAsync>d__39.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
  at Microsoft.AspNetCore.Mvc.Internal.FilterActionInvoker.<InvokeAsync>d__32.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
  at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
  at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
  at Microsoft.AspNetCore.Mvc.Internal.MvcRouteHandler.<InvokeActionAsync>d__8.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
  at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
  at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
  at Microsoft.AspNetCore.Builder.RouterMiddleware.<Invoke>d__4.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
  at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
  at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
  at Microsoft.AspNetCore.Diagnostics.StatusCodePagesMiddleware.<Invoke>d__3.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
  at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
  at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
  at Microsoft.AspNetCore.Diagnostics.DeveloperExceptionPageMiddleware.<Invoke>d__7.MoveNext()

Copied from original issue: aspnet/Mvc#4883

From @rynowak on June 19, 2016 0:57
Thanks for the thorough report. What happens if you use IEnumerable<string> as your parameter type?
/cc @Tratcher
The ArgumentOutOfRangeException leads me to believe this is just a bug with zero-length segments.
SCRAM-SHA-256: n,n=1,,r=mhtzCDgCf3ZBKUpiYNFctfnFaXYf2MlSuZgc1Oc3DUM= fails
SCRAM-SHA-256: n,n=1,r=mhtzCDgCf3ZBKUpiYNFctfnFaXYf2MlSuZgc1Oc3DUM= does not

From @Haplois on June 19, 2016 0:59
@rynowak I just tried that, I get same exception.

https://github.com/aspnet/HttpAbstractions/pull/658


Redux Toolkit Query 2.0 Beta Uncaught Error: could not find react-redux context value

What version of React, ReactDOM/React Native, Redux, and React Redux are you using?

React: 18.2.0
ReactDOM/React Native: 18.2.0
Redux: 5.0.0-beta.0
Redux Toolkit: 2.0.0-beta.0
React Redux: 8.0.7

What is the current behavior?
When upgrading from react-redux 8.0.5 to 8.0.7, along side of Redux Toolkit 2.0.0-beta.0 + Redux 5.0.0-beta.0, my application code throws an error when using an RTK Query hook:
Uncaught Error: could not find react-redux context value; please ensure the component is wrapped in a <Provider>
    at useReduxContext (useReduxContext.js:24:11)
    at useStore2 (useStore.js:17:9)
    at useDispatch2 (useDispatch.js:14:19)
    at useQuerySubscription (buildHooks.ts:689:24)
    at useQuery (buildHooks.ts:964:42)
    at RequireAuthentication (RequireAuthentication.tsx:34:7)

The application code is wrapped in a Provider and has been functioning normally for months with the Redux Toolkit alphas, and is also functioning normally with the just-released Redux Toolkit beta. It is only when I upgrade to react-redux 8.0.7 that the error is thrown. When I downgrade to react-redux 8.0.5, without modifying any other dependencies, the error is not thrown.
Interestingly, useSelector(), which I would assume is accessing the same exact context from the provider, works fine. But useGet*Query() from RTK Query throws. Final note, I am using Yarn PnP and Yarn Workspaces.
What is the expected behavior?
No error should be thrown, as the component is wrapped in a provider.
Which browser and OS are affected by this issue?
Chrome 113 on macOS Ventura
Did this work in previous versions of React Redux?

[X] Yes

My assumption is that you're using a custom context, and providing the hooks to the react API module.
One of the breaking changes made in RTK's alphas/betas (I'm unsure which one) is that the signature of reactHooksModule changed: https://github.com/reduxjs/redux-toolkit/pull/3400
reactHooksModule({
    hooks: {
      useDispatch: createDispatchHook(MyContext),
      useSelector: createSelectorHook(MyContext),
      useStore: createStoreHook(MyContext),
    },
  })

This is likely the main thing that's causing your issue.

Thanks for the reply @EskiMojo14. I'm not using custom context, just the standard createApi() which is added to the reducer object in a standard configureStore(). The weird part here is that my application code works totally fine with all versions of the RTK alphas/betas. It only breaks when upgrading from react-redux 8.0.5 to 8.0.7 (along side of either RTK 2.0.0-alpha.5 or 2.0.0-beta.0 - I haven't tested with other versions). And there don't seem to be many actual material changes in those react-redux updates between 8.0.5 and 8.0.7 - mostly documentation. One exception is a rather large update to the lockfile, which could possibly be impacting dependencies?

it's possible you have multiple versions of react-redux - try running npm ls react-redux

I'm using yarn/workspaces, but the following is equivalent. I also verified I have single versions of React, ReactDOM, Redux, and Redux Toolkit.

interesting, that's really odd - would you be able to make a recreation in codesandbox?

@jonaldinger : the lockfile updates were just me updating the website folder from Docusaurus 2.0-beta to 2.4, no runtime or devdep changes.
I would be very curious to see a repro of this.

Thanks so much for the responsiveness @EskiMojo14 and @markerikson. I'm about to sign off for the weekend, but will try to create something useful to help debug next week. FYI this isn't actively blocking, everything works 100% if I stay on react-redux 8.0.5 even when I update everything else to latest including alpha and beta versions. I opened the issue just as a proactive measure to sniff out a potential issue with the beta.

You actually have four different instances of the same version of react-redux installed here - see that hash in the square bracket after the version number. Those four versions will not be able to interact with each other - you'll have to play around with that to get them to the same instance.

@phryneas great spotting. That seems very likely to be the issue, as the working state with 8.0.5 has a common hash across all workspaces:

I'm not sure what's driving that change with the upgrade, but let's assume it's on my end. I'll do some tinkering. Thank you!

I'm actually running into this same bug with redux-toolkit 1.9.5 and I don't think there is an easy workaround.
I could be wrong, but my guess is that adding redux-toolkit 2.0-beta to react-redux peer dependencies has broken yarn pnp due to the circular peer dependency between the two packages.  This forces yarn to create virtual instances of both packages in every workspace because it cannot guarantee the dependency tree will be the same across workspaces.  Those screenshots are showing virtual packages, not duplicate packages.  If react-redux requires a singleton package, this is going to be a problem for anyone using yarn workspaces.

That... sounds horrifying.
I have no idea what to do about that.

Perhaps someone more knowledgeable about yarn can chime in to make sure I'm not talking out of my backside, but I made a simple repro here
https://github.com/shermify/react-redux-repro
It's a common pattern for workspaces to share a package that exports RTK slices and apis. Works with react-redux 8.0.5 but not 8.1.0 due to the above issue.

We can probably get rid of that singleton going forward, but that will only fix future versions.

@shermify thank you for the info and repro. I did some more local testing over the past few days and am seeing the exact same thing as you (single installed dependencies, but multiple virtual instances of both RTK and react-redux). I tried tweaking lots of things, but always got the same outcome. I think your theory sounds correct:

react-redux 8.0.5: Working without issue. No peer dependency specified for RTK.
react-redux 8.0.6/8.0.7: Added net new peer dependency for RTK. Issue started. https://github.com/reduxjs/react-redux/commit/9ee1df55e643df152ee63f1926b7677b2dccfaa9#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519R43

I also have the same feature use case - a monorepo (Yarn Workspaces) that has multiple apps that connect to common APIs. The core workspace defines the shared slices, which are then imported and consumed by multiple other app workspaces. This setup has been working great up until now (and I've been using the RTK alpha for many months).
It feels reasonable that the peer dependency would be declared in either RTK or react-redux, but not both. And I'm not an expert here, but it actually seems to make more sense to remove the peer dependency from RTK instead, since that's the "larger/core/foundational" library (it's my understanding that you can technically use RTK without react-redux, but you can't use react-redux without react and redux or RTK). For example:

react does not declare react-dom as a peer dependency (but react-dom does declare react as a peer dependancy)
redux does not declare react-redux or @reduxjs/toolkit as peer dependencies
react-redux declares 6 optional peer dependencies. 5 of those optional peer dependencies do not list react-redux as their peer dependency. @reduxjs/toolkit is the only peer dependency in that list to have a circular peer dependency declared.

@markerikson @phryneas again appreciate the responsiveness and discussion!

the issue is that there are entry points (@reduxjs/toolkit/query/react, @reduxjs/toolkit/react added in 2.0) that directly use RTK.
on the other hand, there is no code in react-redux that uses RTK. it depends on redux core types, but nothing from RTK itself.
as such i'd argue it'd make more sense for RTK to have the peer dep and react-redux not to

Yeah, I can try removing it from React-Redux

Ah ok, thanks for the context. At that point, if RTK 2.0 is now directly importing code from react-redux internally, would it make more sense for react-redux to become an actual dependency of RTK rather than an optional peerDependency? (feel free to say absolutely not - I don't know the larger considerations/implications there 😄).

it's an optional peer dependency because it's entirely possible to use RTK without react-redux - it's just the react specific entry points that would need it, basically

FYI, if you list react-redux and redux-toolkit as peer dependencies in your shared library and remove them as direct deps, it should fix the issue because then it ensures the shared library is using the correct react-redux instance provided by each app.  This is probably the right way to do it so long as react-redux requires a singleton package.  It's definitely not obvious at first and will trip a lot of people up though.
I'm not sure if removing redux-toolkit from react-redux peer dependencies alone will solve the issue because there were other changes to both package's dependencies that could cause branching.  I think the reality is this configuration will be fragile as long as react-redux requires a singleton.

@shermify going forward, we should be able to solve that "singleton context item" problem with #2039.

I'll see if I can get out 8.1.1 tonight with the context fix and the peerDep removal

okay, 8.1.1 is out:
https://github.com/reduxjs/react-redux/releases/tag/v8.1.1
Please let us know if that fixes things!

@markerikson, the dependency hashes with v8.1.1, they're beautiful 🥲 Fixed for both RTK and react-redux. Oh and also all of the behavior is seeming back to normal. I clicked around a bunch in different apps and saw no occurrences of missing context. Thanks all! I even learned some new things through this.

Works for me.  Thanks for being on top of this so quickly!

Everything remains rock solid, so closing. Thanks again!


Implement GetProperties pids for V2

Signed-off-by: Justin Terry (VM<EMAIL_ADDRESS>

@jhowardmsft - Hold Fire. Need to verify first.

@jterry75 Cool, I'll try this out.


Schema change error in a new environment

Describe the bug
Getting schema change error in the new environment
Amplify CLI Version
4.40.0
To Reproduce

Change key schema in schema.graphql
Add new environment

Do you want to use an existing environment? No
? Enter a name for the environment testa
Using default provider  awscloudformation

Push environment

amplify push --y

An error is reported:
Attempting to edit the key schema of the InvitationTable table in the Invitation stack.
An error occurred during the push operation: Attempting to edit the key schema of the InvitationTable table in the Invitation stack.

Expected behavior
User should be able to push any schema in new environment
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Ubuntu
Node Version. v10.19.0

Additional context
Add any other context about the problem here.

Error exists even if I did not change anything
In 4.29.8 as well

@ayeganyan do you have an env with name testa already in the project? If there is an existing environment with the same name in <project-root>/amplify/team-provider-info.json, the CLI does not create an new environment. If you want to recreate the environment please run amplify env remove <env-name> and then add it back

That was completely new environment...
I tried with new environments several environment, and finally reverted all my changes I have made after last successful deployment, and removed auto-generated "#current-cloud-backend", amplify-meta.json, awscloudformation folders, so it seems issue occurred  because of environment info did not cleaned up from previous environment.

I am facing issue again, as you can see graphqlapi is in Create state:
amplify push --y
✔ Successfully pulled backend environment develop from the cloud.

Current Environment: develop

| Category | Resource name                  | Operation | Provider plugin   |
| -------- | ------------------------------ | --------- | ----------------- |
| Auth     | authentication                 | Create    | awscloudformation |
| Auth     | userPoolGroups                 | Create    |                   |
| Api      | graphqlapi                     | Create    | awscloudformation |
....
| Function | invitationQueueTrigger         | Create    | awscloudformation |
| Storage  | storage                        | Create    | awscloudformation |
| Queue    | postConfirmationQueue          | Create    | awscloudformation |
| Queue    | invitationQueue                | Create    | awscloudformation |

The following types do not have '@auth' enabled. Consider using @auth with @model
         - Policy
Learn more about @auth here: https://docs.amplify.aws/cli/graphql-transformer/directives#auth

✖ An error occurred when pushing the resources to the cloud

Attempting to edit the key schema of the TenantTable table in the Tenant stack.
An error occurred during the push operation: Attempting to edit the key schema of the TenantTable table in the Tenant stack.

It worked fine after I remove #current-cloud-backend folder

Sanity check needs to be run only if the resource is being updated, irrespective of the content that is in the #current-cloud-backend. I am marking this as bug


HA Model

Hello!
As it stated in the documentation, two components (controller and init container) are used. There is also a mention about Raft based HA model.
As I understand, the HA model applies to controller only?
Can the production setup have several init and controller containers in HA mode?
Please advise.
Thanks!

Hello!
Yes, you're correct. HA mode is for the controller only. In production, you can run multiple controllers in odd numbers for a quorum, for example, 1, 3, 5, 7, etc. However, for performance reasons when appending the raft logs , you should not have too many controllers. The FAQ for etcd should also be applicable in this case: https://github.com/coreos/etcd/blob/master/Documentation/faq.md#what-is-maximum-cluster-size
For the init containers, there is no point having HA for them. However, if for some reason you need to have multiple Vault tokens for a pod, then you can chain the init containers, 1 after another.
Hope that helps! 😄


data not does not save while dropzone is empty

Expected behavior
Say, I want to use drop zone for Customer avatar. So it may be empty or user may put images. it should save to database without any images from dropzone .
I have two BLOB column in a table, then how should I insert them into database. As I have seen in Apex collection there is only one BLOB Column. How can I use it for two blob column. Otherwise while I go to save through custom insert statement I got the error: ORA-01465: invalid hex number
Actual behavior
However, while trying  to insert record without putting any images to dropzone  no data is inserting into database.
Steps to reproduce the issue
try to save without putting images in dropzone with two blob column in a table.
APEX version (4.2.6 / 5.0.3)
5.1
Used web server / version and platform (ORDS 3.0.3 / Tomcat 7 / Apache 2.4 / Linux x64)
Used web browser / version and platform (Chrome 48 Mac / Firefox 44 Windows)

HI @Dani3lSun Could you please shade light on my issue.


feat: download soil data only if unexistant

Closes #24

please look at #26 before merging

please look at #26 before merging

awesome ! thanks for the enhancement ✨ @akouete-kpakpo


Adds unused dependencies check to CI

Just like https://github.com/hrzndhrn/recode/pull/33, this PR adds an unused dependencies check to CI.

Pull Request Test Coverage Report for Build 5c01e8e5e6a554060edfb56ea15212076b9d3e83-PR-3

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 97.761%

Totals

Change from base Build bdce0d609b7c95aa216d7a0207a91cd6da227790:
0.0%

Covered Lines:
262

Relevant Lines:
268

💛  - Coveralls


sklearn sgdregressor

Implement a predictor for redshift using SGDRegressor
http://scikit-learn.org/stable/modules/generated/sklearn.linear_model.SGDRegressor.html
with huber loss

See pull request #136. The R^2 value is (how??) -5.550428722967837e+25.

Revisit after you have checked that R^2 is the same in your implementation and sklearn.

Progress made in #154. R^2 is now 0.5639171254698758. This is more reasonable but still not as good as GP regression’s 0.9113252622189996.


Possibility to move node in static graph

Is your feature request related to a problem? Please describe.
I made a tree graph with your graph library.

To do that, i set config with static property to true to can set x & y postion of my nodes. But I can't move node like graph with no static property., because when it is false, you set state for graph render ...
Describe the solution you'd like
I want to move nodes and have my own positions setted before
Describe your use case
I've got lot of link and i wanna made a modeler, so, i need to move nodes.
Describe alternatives you've considered
No alternatives but to go to a new library
Additional context
I need to keep this visual of tree graph, it's convention
If you need more information, I am at your disposal.

I'm really sorry, I didn't see the "staticGraphWithDragAndDrop" property for config ....  You can delete this issue ....

Hey, nice use case you have there, the tree looks pretty cool. The use case you described can definitely be achieved through staticGraphWithDragAndDrop, cheers!

Hi @danielcaldas ,
Haha thx dude ^^
Yes, the staticGraphWithDragAndDrop work for what I need ;)
Cheers ;)


Fix mdast update

mdast’s CLI switched to async callback mode from a previously sync
interface, which resulted in errors when integrating.
Closes GH-10.

Could someone review the CoffeeScript? Haven’t written that in a while :anguished:

Looks fine visually, assuming it works locally you should be good to release.

It worked! Away from computer, could you merge? Sorry :/

testing locally.  will merge if good!

Works great.  Thanks for the fix!


After trying to loging, I get a Workspace Error

After I login with a user/pass on my macbook, I get the error below
Is there some credential it needs? I also deleted/disabled browser cache. Any idea what it could be?
Error: relation "public.profiles" does not exist

Source
db/profile.ts (12:10) @ getProfileByUserId

  10 |
  11 | if (!profile) {
> 12 |   throw new Error(error.message)
     |        ^
  13 | }
  14 |
  15 | return profile

sir, no need to open issue with same topics. #1377
Vercel will never work with current release. you can use another method, i have given solution. Also, you need help with servers/hosting of this, please let me know.
You can close the issue. 😄


Clear wallet data confirmation

This PR request user confirmation before proceeding to delete wallet data, and also shutdown and reinitialize multiwallet after wallet data deletion.

The app does not let me in if I choose No, is this the expected behavior?


just correcting some script in README

Thanks, but it was solved by another pull request.
#1

Duplicate of #1


RSocket support

RSocket is modern Streaming message protocol with Reactive Extension/Stream semantics
Its support in Helidon is really nice to have.

This feature must start with a one-pager, to describe what it means.
RSocket is fully dependent on a third party reactive library, which is implicitly a problem for Helidon - it must be clearly understood how this would be integrated.
Also please describe the expectations for client side and/or server side - whatever is expected to be implemented as part of this feature.
If server side is expected, describe how it would fit into Helidon design (routing/messaging etc.)

Added proposal
https://github.com/dalexandrov/helidon/tree/rsocket


Proxmox Container with Ubuntu 22.04 LTS image - No website appears on installation completion

Hi, Is this script still actively maintained?
I wonder if there is any active preparation necessary to make it work.... (Other than
$> apt update && apt upgrade.     )
For example:

Is it okay to run the script as root ? Perhaps I should create a user like "erpnext" first, and run install.sh from there ?
Should I be pre-installing LEMP or LAMP features?  I notice that, once completed, there is NO listed Webserver like NGINX or Apache in the list of installed ... for example if I type:
$>  systemctl
No webserver appears either installed OR active in the list!

The script is actively maintained and you typically do not need to do anything except run update and upgrade and reboot (if you're using Ubuntu 22.04).
You SHOULD NOT run the script as root. just follow the instructions on the readme.
The script installs all the necessary prerequisites and dependencies for ERPNext to run so you do not need any package installed before running the script.
ERPNext uses NGINX web server and this is installed as part of Frappe's code execution process.


JustWatch Bridge misses some titles

Describe the bug
After some hours that a title is posted in justwatch website, the title is not posted in justwatch bridge rss.
To Reproduce
Steps to reproduce the behavior:

Go to JustWatch Bridge
Click on Country:"Italy" and Type:"All"
Go to JustWatch Wesbsite
Find a missing title between the latest titles

Expected behavior
All titles of justwatch website should be posted in justwatch rss
Screenshots

Desktop (please complete the following information):

OS: Archlinux
Browser: Brave
Version: 1.62.156 Chromium: 121.0.6167.139 (Official Build) (a 64 bit)

Additional context
n/a

@Bockiii

I just checked and there seems to be some size or amount limitations. The problem is that I didn't add a filter for all the providers (because I didnt bother to add another 70 selection options to the filter and also I think the list is very fluctuating and could produce unwanted behavior if you select a country/provider combination that doesnt exist (because BBC isnt available in guana or something like that).
This leads to the problem that even the "Today" field contains hundreds and hundreds of entries.
Second problem seems to be the "scroll to right" function on providers. So if amazon releases 140 entries on a day, I only get 10 because for the others to load, we would have to scroll.
@dvikan do you know of a way how to deal with this in php?
At this point, I don't really know how to fix this because of the limitations above.

Example for the side scrolling problem. Page says 18 new titles, only displays 10 and thus, only 10 are in source.

Maybe it could be useful a filter with just the 4-5 major providers: netflix, prime, disney, apple, paramount, crunchyroll. That are what ~99% of users need.

Alternative would be a free text field with ab explanation of what to put in there. You can define the providers and then copy paste them from the url.
Full alternative would be to just change the whole bridge to just picking up a provided link and then just scraping from there. Although that would also not help with the people that just pastel the link to the full "new" page.
Not sure on how to proceed

I found that justwatch has a graphQL api but its not really documented. I found these sources for some information, if someone wants to take a swing:
https://www.reddit.com/r/webscraping/comments/wacb8a/how_to_scrape_graphql_endpoint_with_requests/

I've been looking at this again and getting "all" just doesn't make any sense. Just today there were these additions:
"Amazon Vault History Channel": 341
"History Vault Apple TV": 84
"Acorn TV Apple TV": 95
"Microsoft Store": 99
"Eventive": 170
Those 5 alone mean 789 rss feed entries, just for today.
How about this: I'll add the top 10/top 15 of providers and if someone actually requests "Eventive", we can check this again?

Ah forget it, even that is annoying af. Paramount+ is available as "Paramount+", "Amazon Channels Paramount+", "Apple TV Channels Paramount+" and so on. And that's just for the US page, so for italy, there could be either none of those or a fourth "Channel" type etc.
This page just doesn't really fit the "select from a list" type of bridge. Maybe a "paste the link you want" would be better or maybe even removing it completely and replacing it by an xpath bridge how-to or so...

unclear to me the solution here.


tensorboard charms offline deployment fails "local charm missing OCI images. .."

Same as https://github.com/canonical/kfp-operators/issues/351
Affecting both

tensorboards-web-app
tensorboard-controller

$ juju deploy --trust --debug ./tensorboard-controller_840554c.charm --resource oci-image=$AWS_ECR_URL/kubeflownotebookswg/tensorboard-controller:v1.7.0

22:20:45 INFO  juju.cmd supercommand.go:56 running juju [2.9.45 afb8ee760af71d0bca8c3e4e0dc28af2dabc9b1d gc go1.20.8]
22:20:45 DEBUG juju.cmd supercommand.go:57   args: []string{"juju", "deploy", "--trust", "--debug", "./tensorboard-controller_840554c.charm", "--resource", "oci-image=701143232170.dkr.ecr.eu-west-1.amazonaws.com/kubeflownotebookswg/tensorboard-controller:v1.7.0"}
22:20:45 DEBUG juju.jujuclient proxy.go:65 unmarshalled proxy config for "kubernetes-port-forward"
22:20:45 INFO  juju.juju api.go:86 connecting to API addresses: [<IP_ADDRESS>:17070]
22:20:45 DEBUG juju.api apiclient.go:624 starting proxier for connection
22:20:45 DEBUG juju.api apiclient.go:628 tunnel proxy in use at localhost on port 34245
22:20:45 DEBUG juju.api apiclient.go:1151 successfully dialed "wss://localhost:34245/api"
22:20:45 INFO  juju.api apiclient.go:686 connection established to "wss://localhost:34245/api"
22:20:45 DEBUG juju.jujuclient proxy.go:65 unmarshalled proxy config for "kubernetes-port-forward"
22:20:45 INFO  juju.juju api.go:86 connecting to API addresses: [<IP_ADDRESS>:17070]
22:20:45 DEBUG juju.api apiclient.go:624 starting proxier for connection
22:20:45 DEBUG juju.api apiclient.go:628 tunnel proxy in use at localhost on port 39319
22:20:45 DEBUG juju.api apiclient.go:1151 successfully dialed "wss://localhost:39319/model/79be7fe7-68f5-488b-8a9e-231e6a0fafbb/api"
22:20:45 INFO  juju.api apiclient.go:686 connection established to "wss://localhost:39319/model/79be7fe7-68f5-488b-8a9e-231e6a0fafbb/api"
22:20:45 DEBUG juju.core.charm computedseries.go:27 series "focal" for charm "tensorboard-controller" with format 2, Kubernetes true
22:20:45 DEBUG juju.core.charm computedseries.go:27 series "focal" for charm "tensorboard-controller" with format 2, Kubernetes true
22:20:45 DEBUG juju.api monitor.go:35 RPC connection died
22:20:45 DEBUG juju.api monitor.go:35 RPC connection died
ERROR local charm missing OCI images for: tensorboard-controller-image
22:20:45 DEBUG cmd supercommand.go:537 error stack:
github.com/juju/juju/cmd/juju/application/deployer.(*factory).validateResourcesNeededForLocalDeploy:677: local charm missing OCI images for: tensorboard-controller-image
github.com/juju/juju/cmd/juju/application/deployer.(*factory).maybeReadLocalCharm:406:
github.com/juju/juju/cmd/juju/application/deployer.(*factory).GetDeployer:71:
github.com/juju/juju/cmd/juju/application.(*DeployCommand).Run:909:

juju deploy --trust --debug ./tensorboards-web-app_d2c0149.charm --resource oci-image=$AWS_ECR_URL/kubeflownotebookswg/tensorboards-web-app:v1.7.0

22:23:35 INFO  juju.cmd supercommand.go:56 running juju [2.9.45 afb8ee760af71d0bca8c3e4e0dc28af2dabc9b1d gc go1.20.8]
22:23:35 DEBUG juju.cmd supercommand.go:57   args: []string{"juju", "deploy", "--trust", "--debug", "./tensorboards-web-app_d2c0149.charm", "--resource", "oci-image=701143232170.dkr.ecr.eu-west-1.amazonaws.com/kubeflownotebookswg/tensorboards-web-app:v1.7.0"}
22:23:35 DEBUG juju.jujuclient proxy.go:65 unmarshalled proxy config for "kubernetes-port-forward"
22:23:35 INFO  juju.juju api.go:86 connecting to API addresses: [<IP_ADDRESS>:17070]
22:23:35 DEBUG juju.api apiclient.go:624 starting proxier for connection
22:23:35 DEBUG juju.api apiclient.go:628 tunnel proxy in use at localhost on port 33553
22:23:35 DEBUG juju.api apiclient.go:1151 successfully dialed "wss://localhost:33553/api"
22:23:35 INFO  juju.api apiclient.go:686 connection established to "wss://localhost:33553/api"
22:23:35 DEBUG juju.jujuclient proxy.go:65 unmarshalled proxy config for "kubernetes-port-forward"
22:23:35 INFO  juju.juju api.go:86 connecting to API addresses: [<IP_ADDRESS>:17070]
22:23:35 DEBUG juju.api apiclient.go:624 starting proxier for connection
22:23:35 DEBUG juju.api apiclient.go:628 tunnel proxy in use at localhost on port 38889
22:23:35 DEBUG juju.api apiclient.go:1151 successfully dialed "wss://localhost:38889/model/79be7fe7-68f5-488b-8a9e-231e6a0fafbb/api"
22:23:35 INFO  juju.api apiclient.go:686 connection established to "wss://localhost:38889/model/79be7fe7-68f5-488b-8a9e-231e6a0fafbb/api"
22:23:35 DEBUG juju.core.charm computedseries.go:27 series "focal" for charm "tensorboards-web-app" with format 2, Kubernetes true
22:23:35 DEBUG juju.core.charm computedseries.go:27 series "focal" for charm "tensorboards-web-app" with format 2, Kubernetes true
22:23:35 DEBUG juju.api monitor.go:35 RPC connection died
22:23:35 DEBUG juju.api monitor.go:35 RPC connection died
ERROR local charm missing OCI images for: tensorboards-web-app-image
22:23:35 DEBUG cmd supercommand.go:537 error stack:
github.com/juju/juju/cmd/juju/application/deployer.(*factory).validateResourcesNeededForLocalDeploy:677: local charm missing OCI images for: tensorboards-web-app-image
github.com/juju/juju/cmd/juju/application/deployer.(*factory).maybeReadLocalCharm:406:
github.com/juju/juju/cmd/juju/application/deployer.(*factory).GetDeployer:71:
github.com/juju/juju/cmd/juju/application.(*DeployCommand).Run:909:

As commented in https://github.com/canonical/kfp-operators/issues/351#issuecomment-1759072323 too, it looks like you are using charms from latest/edge for tensorboards-operators too and those charms have changed the name of the resource they require (although it's the same resource). You can see that in the error you 've posted that they look for resources with their new names (web app and controller in main). Please make sure you are using charms for the bundle you would like to deploy/

@orfeas-k
Thanks for the highlight, it seems the local charms I was provided came from another channel
I just tested with --resource tensorboards-web-app-image= and --resource tensorboard-controller-image= and they work


auto register service

项目采用组件化方案时，涉及到路由、组件统一注册时，往往需要手动在初始化是统一注册，这对各业务开发耦合度比较高，auto register就是解决这种耦合场景，让框架可以更紧凑。

@galaxybruce
Please resolve the conflicts first.
Besides, I've refactored the AspectD design so it can act as a dill transformer container, Can you try to build your own transformer above that?

nice!!! I will take time to read the latest code

I have refactor auto register feature  based latest code,  you see if it's necessary to merge


[Node.js] - develop branch - module not found @iota/wallet

Bug description
When updating the instructions on how to get started with node.js for the develop branch, I keep getting Error: Cannot find module '@iota/wallet' when trying to run any example.
Rust version

Rust version: rustc 1.67.1 (d5a82bbd2 2023-02-07)

Version
develop
Hardware specification

Operating system: Ubuntu 22.04
RAM: 16
Cores: 12
Device: HP HP Pavilion Gaming Laptop 16-a0xxx

Steps To reproduce the bug

git clone git@github.com:iotaledger/wallet.rs.git
cd wallet.rs/bindings/nodejs/
npm i
npm run install
node examples/1-create-account.js

or simply : git clone git@github.com:iotaledger/wallet.rs.git && cd wallet.rs/bindings/nodejs/  && npm i && npm run install && node examples/1-create-account.js
Expected behaviour
The example should run,  and create an account.
Actual behaviour
The example throws a MODULE_NOT_FOUND error  for @iota/wallet
Errors
node:internal/modules/cjs/loader:958
  throw err;
  ^

Error: Cannot find module '@iota/wallet'
Require stack:
- /home/lucas/Documents/IOTA/tests/buildWalletNode/wallet.rs/bindings/nodejs/examples/1-create-account.js
    at Module._resolveFilename (node:internal/modules/cjs/loader:955:15)
    at Module._load (node:internal/modules/cjs/loader:803:27)
    at Module.require (node:internal/modules/cjs/loader:1021:19)
    at require (node:internal/modules/cjs/helpers:103:18)
    at Object.<anonymous> (/home/lucas/Documents/IOTA/tests/buildWalletNode/wallet.rs/bindings/nodejs/examples/1-create-account.js:6:38)
    at Module._compile (node:internal/modules/cjs/loader:1119:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1173:10)
    at Module.load (node:internal/modules/cjs/loader:997:32)
    at Module._load (node:internal/modules/cjs/loader:838:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:81:12) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [
    '/home/lucas/Documents/IOTA/tests/buildWalletNode/wallet.rs/bindings/nodejs/examples/1-create-account.js'
  ]
}

Node.js v18.8.0

Perhaps this is related to https://github.com/iotaledger/wallet.rs/issues/1861

I think this is if the installation isn't correctly found in the examples
Not sure what's the best way there, if we should import it from the parent folder instead of @iota/wallet or if there is another good way
Personally I linked it with npm ln, but never sure what's the best way

npm run install does nothing the first time 🤔
But npm run build and then npm i in the examples folder works for me, the .env file also needs to be created to run it successfully


Surveys & Report Grid: add link to jexl syntax page

Description
Power users would benefit from the complete docs, not just the curated subset we show
https://commons.apache.org/proper/commons-jexl/reference/syntax.html
Resourcing
We intend to contribute this feature


Fix/xemm order size taker volume factor

Before submitting this PR, please make sure:

[ ] Your code builds clean without any errors or warnings
[ ] You are using approved title ("feat/", "fix/", "docs/", "refactor/")

A description of the changes proposed in the pull request:
Changed order_size_taker_balance to an unused parameter called order_size_taker_volume
There is no parameter that helps to limit the ratio of available quote asset on taker when calculating market size and this parameter is unused in the main code. I am not entirely sure of the purpose of this parameter.
But to me, it seems that the intention of this parameter is to limit the taker quote volume that can be used in the strategy. Hence I am proposing this change to use this parameter instead to limit the taker quote volume so that there is a separation of quote asset allocation and base asset allocation.
Please reject this proposal if this is not the intent.
Tests performed by the developer:
Tips for QA testing:

https://snapshot.org/#/hbot-prp.eth/proposal/0x56cfab895789d4b1065bfae934dd81baab2f02b58e1d141321a01b18d4d17108

PRP have not been approved because it didnt reached the minimum quorum. (https://snapshot.org/#/hbot-prp.eth/proposal/0x56cfab895789d4b1065bfae934dd81baab2f02b58e1d141321a01b18d4d17108)
If still interested in adding this PR to the main code base, please review it, and create a new PRP.
Thank you for the contribution!


doc: reorganize docs, navigation, consolidate cli reference to one page

Let's keep all URLs lowercased.


problem with dask partitions and creating dask graph

Hello, I am using a tool in a singularity container and it uses dask. it used to work so wll and run on 39 paritions  which speeds up the process but suddenly the maximum is 1-5 paritions.  I have ample amount if ram and threads. not sure what went wrong. usually if i ask for a higher number of threads it gets stuck at creating dask graph.
2024-10-07 21:14:51,822 - distributed.worker - INFO -       Start worker at:   tcp://<IP_ADDRESS>:41117
2024-10-07 21:14:51,822 - distributed.worker - INFO -          Listening to:   tcp://<IP_ADDRESS>:41117
2024-10-07 21:14:51,822 - distributed.worker - INFO -          dashboard at:         <IP_ADDRESS>:41551
2024-10-07 21:14:51,822 - distributed.worker - INFO - Waiting to connect to:    tcp://<IP_ADDRESS>:8786
2024-10-07 21:14:51,822 - distributed.worker - INFO - -------------------------------------------------
2024-10-07 21:14:51,822 - distributed.worker - INFO -               Threads:                         35
2024-10-07 21:14:51,822 - distributed.worker - INFO -                Memory:                   1.64 TiB
2024-10-07 21:14:51,822 - distributed.worker - INFO -       Local Directory: /tmp/dask-worker-space/worker-lgrp83cl
2024-10-07 21:14:51,822 - distributed.worker - INFO - -------------------------------------------------
2024-10-07 21:14:51,826 - distributed.scheduler - INFO - Register worker <WorkerState 'tcp://<IP_ADDRESS>:41117', status: init, memory: 0, processing: 0>
2024-10-07 21:14:51,828 - distributed.scheduler - INFO - Starting worker compute stream, tcp://<IP_ADDRESS>:41117
2024-10-07 21:14:51,829 - distributed.core - INFO - Starting established connection to tcp://<IP_ADDRESS>:56430
2024-10-07 21:14:51,829 - distributed.worker - INFO -         Registered to:    tcp://<IP_ADDRESS>:8786
2024-10-07 21:14:51,829 - distributed.worker - INFO - -------------------------------------------------
2024-10-07 21:14:51,830 - distributed.core - INFO - Starting established connection to tcp://<IP_ADDRESS>:8786
Singularity> pyscenic grn \

--num_workers 10 \
--seed 777 \
--client_or_address tcp://<IP_ADDRESS>:8786 \
-o /data/adj_merged_94k_1.tsv \
/data/merged.loom \
/data/allTFs.txt

2024-10-07 21:15:29,687 - pyscenic.cli.pyscenic - INFO - Loading expression matrix.
2024-10-07 21:16:05,190 - pyscenic.cli.pyscenic - INFO - Inferring regulatory networks.
2024-10-07 21:16:05,198 - distributed.scheduler - INFO - Receive client connection: Client-37923334-84d8-11ef-a2bd-ac1f6bcfd966
2024-10-07 21:16:05,198 - distributed.core - INFO - Starting established connection to tcp://<IP_ADDRESS>:35270
preparing dask client
parsing input
creating dask graph
1 partitions
computing dask graph

Can you provide an example that is reproducible? I.e. some code that we can look at?

  #this was the full code for the part i pasted above ( i used chatgpt because of my little knowledge about dask)

  singularity shell -B /ibex/user/mohamesk/new_GRN:/data \
/path/aertslab-pyscenic-scanpy-0.12.1-1.9.1.sif

  # Start Dask scheduler
dask scheduler &

# Start Dask workers
dask-worker tcp://<IP_ADDRESS>:8786 --nthreads 40 --memory-limit 2000GB &

# Run
pyscenic grn \
    --num_workers 10 \
    --seed 777 \
    --client_or_address tcp://<IP_ADDRESS>:8786 \
    -o /data/adj_merged_94k_1.tsv \
    /data/merged.loom \
    /data/allTFs_hg38.txt

    ######################################
#but this how i used it before and it worked. In 1 day  the slurm job finished  and now suddenly it took 14 days and never finished (stuck at creating dask step) althought the requested slurm resources are 40 cores and over 2000G RAM!  it can process to the partition step if less number of workers are specified in the code below such as 1 or 5 maximum.

 #First build the image both failed and previously worked
# pySCENIC CLI version.
singularity build aertslab-pyscenic-0.12.1.sif docker://aertslab/pyscenic:0.12.1
# pySCENIC CLI version + ipython kernel + scanpy.
singularity build aertslab-pyscenic-scanpy-0.12.1-1.9.1.sif docker://aertslab/pyscenic_scanpy:0.12.1_1.9.1

module load singularity/3.9.7

custom_tmp_dir='/tmp/HelloWorld'
mkdir -p "${custom_tmp_dir}"

# Run pySCENIC with a custom TMPDIR.
TMPDIR="${custom_tmp_dir}" singularity run -B /path:/data \
    /path/aertslab-pyscenic-scanpy-0.12.1-1.9.1.sif \
    pyscenic grn \
        --num_workers 10  \
        -o /data/adj_merged_94k_2.tsv \
        /data/merged94k.loom \
        /data/allTFs_hg38.txt

 #################################
   there is also a conda package for it it is also stuck at the creating dask graph part
 conda create -n test_env
 conda install lalit.shaktawat::pyscenic

pyscenic grn
    --expression_mtx_fname merged94k.loom \
    --tfs_fname allTFs_hg38.txt \
    --method grnboost2 \
    --output adj_merged_94k_conda1.tsv \
    --num_workers 39

    ###########################

honestly i am not sure how to provide you with a reproducible error so kindly ask what I need to run from my end even if it is another tool that relies on dask. it is just unfortunate that suddenly it stopped working .

We will need the code that you ran on the cluster, we have an explanation about minimal examples here: https://matthewrocklin.com/minimal-bug-reports

Generate dummy data
import numpy as np
import loompy

# Set parameters
n_cells = 106000  # Number of cells
n_genes = 24000   # Number of genes
mean_expression = 10.0  # Increase mean expression level
zero_fraction = 0.1  # 10% of the values will be zero

# Step 1: Generate expression matrix
X = np.random.binomial(n=1, p=(1 - zero_fraction), size=(n_cells, n_genes)) * np.random.negative_binomial(1, 1/(1 + mean_expression), size=(n_cells, n_genes))

# Step 2: Create cell and gene identifiers
cell_ids = [f"cell_{i}" for i in range(n_cells)]
gene_ids = [f"gene_{i}" for i in range(n_genes)]

# Step 3: Compute nGene and nUMI
nGene = np.sum(X > 0, axis=1)  # number of detected genes per cell
nUMI = np.sum(X, axis=1)  # total UMI counts per cell

# Step 4: Define row and column attributes
row_attributes = {
    "Gene": np.array(gene_ids)  # Gene names as row attributes
}
col_attributes = {
    "CellID": np.array(cell_ids),  # Cell IDs as column attributes
    "nGene": nGene,  # Number of detected genes per cell
    "nUMI": nUMI  # Total UMI counts per cell
}

# Step 5: Create the Loom file
loom_path = "test_data.loom"
loompy.create(loom_path, X.T, row_attributes, col_attributes)

# Optional: Check the generated file
ds = loompy.connect(loom_path)
print("First 10 rows (genes):", ds.ra['Gene'][:10])
print("First 10 columns (cells):", ds.ca['CellID'][:10])
print("Shape of the matrix:", ds.shape)
print("Example values for the first 10 cells:", ds[:, :10])  # Show values for the first 10 cells
ds.close()

# Generate a text file named 'TF.txt' containing gene names from "gene_0" to "gene_999"
tf_gene_names = [f"gene_{i}" for i in range(1000)]  # First 1000 genes (gene_0 to gene_999)

# Specify the file path
tf_file_path = "TF.txt"

# Write the gene names to TF.txt
with open(tf_file_path, "w") as f:
    for gene in tf_gene_names:
        f.write(gene + "\n")

print(f"TF.txt has been created with gene names from gene_0 to gene_999 at '{tf_file_path}'")

#Build the image
module load singularity/3.9.7
singularity build aertslab-pyscenic-0.12.1.sif docker://aertslab/pyscenic:0.12.1
#!/bin/bash
#SBATCH --job-name=GRN_singu_step1
#SBATCH --output=grn_step_%j.out
#SBATCH --error=grn_step_%j.err
#SBATCH --time=2-00:00:00
#SBATCH --ntasks=1
#SBATCH --cpus-per-task=40
#SBATCH --mem=1000G

# load singularity module

module load singularity/3.9.7

singularity run -B /path/folder:/data \
    /path/folder/aertslab-pyscenic-0.12.1.sif \
    pyscenic grn \
        --num_workers 39 \
        -o /data/adj.tsv \
        /data/test_data.loom \
        /data/TF.txt

How the out file looked like
preparing dask client
parsing input
creating dask graph
39 partitions
computing dask graph
not shutting down client, client was created externally
finished

This isn't minimal at all, can you provide something that we can run locally on our machines without building any images?

Hi @Flu09 - It looks like you are having problems running "pySCENIC" (which is apparently using Dask).
I think we'd love to help you here, but are very unlikely to reproduce your problem locally. None of us are using slurm or singularity on our local machines, and so it would take significant time and effort to reproduce. It may make sense to file an issue in the pySCENIC repo, but they may say something similar.

will do thanks.


Why is it deprecated?

Did you find a better way to solve this kind of problem or you just decided there's no point in implementing Send for MutexGuard?
I'm having a similar problem:
while ... {
    ...
    let sender = sender.lock().unwrap();
    sender.send(event).await.unwrap();
}

which outputs the error: future cannot be sent between threads safely

Hey @flaviolc18, thanks for the interest! I set async_mutex to be deprecated because it is unsound in some cases, which I didn't have time to solve. Those cases are mostly related to drops during strange times.
My current suggestion is to use a more maintained piece of code, like this one:
https://docs.rs/async-std/1.5.0/async_std/sync/struct.Mutex.html

Yes, I found about it, there's also https://tokio-rs.github.io/tokio/doc/tokio/sync/struct.Mutex.html. But thanks for answering anyway.


Lint: use ignore-this on generated files

Make golint not complain about generated file - it's a bit of cheating,
but I think that is OK.

Codecov Report

Merging #596 into master will increase coverage by 0.15%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #596      +/-   ##
==========================================
+ Coverage   57.73%   57.89%   +0.15%
==========================================
  Files          36       37       +1
  Lines        9976    10097     +121
==========================================
+ Hits         5760     5846      +86
- Misses       3168     3189      +21
- Partials     1048     1062      +14

Impacted Files
Coverage Δ

zmsg.go
50.51% <ø> (ø)
:arrow_up:

zcompress.go
22.22% <ø> (ø)
:arrow_up:

ztypes.go
41.83% <0%> (ø)
:arrow_up:

udp.go
88.23% <0%> (-1.77%)
:arrow_down:

udp_linux.go
70% <0%> (ø)

Continue to review full report at Codecov.

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

Seems tkey has a change here as well?

Merging this; probably means TKEY is now broken-ish


Sanity check FASTA for ASCII text

added sanity checking code that FASTA files contain only ASCII text (the FASTA format predates Unicode)

Fixes EncoderFallbackException to give a more helpful error message
This was something I did a couple weeks ago but just now took the time to give it its own PR.

Currently, we seem to use UTF8 whenever we parse FASTA files. It wasn't necessarily a conscious choice, but it's what StreamReader gives us by default.
Even if we are allowing UTF8, we can still be pretty effective at detecting binary files, just by looking for character 0. Most binary files have lots of zeroes in them, and text files never do.

The most common case may be someone picking a .fasta.gz file and in that case yeah, there's tons of 0s. I'm trying to also catch rarer cases where there might be a small amount of garbage somewhere in the file and it won't necessarily have 0s but if it's random garbage each byte only has a 96/256 chance to be valid.

Ping @nickshulman - any reason this shouldn't go in?


Can the Interactive button be set to default true or false?

Please describe the feature that you want to propose
I want to set ReactFlow to read-only for display purposes only, and all elements cannot be edited. What should I do?

For this you should set
<ReactFlow
  nodesDraggable={false}
  nodesConnectable={false}
  elementsSelectable={false}
  nodesFocusable={false}
  edgesFocusable={false}
/>

or you could use CSS:
.react-flow {
  pointer-events: none;
}


Adding DCCP support

I would like to add DCCP support. It is currently only available on Linux. It would basically only involve exposing the Type, Protocol and some socket options listed here. Would this be something of interest?
Currently I'm proposing to just add something like this
    #[cfg(target_os = "linux")]
    /// Type corresponding to `SOCK_DCCP`
    ///
    /// Used for the DCCP protocol
    pub const DCCP: Type = Type(sys::SOCK_DCCP);

and
#[cfg(target_os = "linux")]
    /// Protocol corresponding to `DCCP`
    pub const DCCP: Protocol = Protocol(sys::IPPROTO_DCCP);

In addition I'd propose adding the socket options prefixed with dccp_. So for example the socket option DCCP_SOCKOPT_SERVICE could be dccp_set_service() and dccp_service()

Sounds good 👍

@onestay can this be closed after https://github.com/rust-lang/socket2/pull/359 was merged?

Oh, yeah of course. Sorry, totally forgot.


fix: plugin legacy - module output doesn't support Safari 10

Description
When using the plugin-legacy the modern bundle of the app includes the dynamic import, and in safari 10 it throws a syntax error that blocks the parsing and the execution of the entire application.
safari 10.1 Syntax Error Unexpected Keyword import
Since this import is a thing used for a specific purpose I just implemented a parameter that let me prevent the plugin from using that forced dynamic import and keep the application running also in safari 10.
closes #10099

Additional context
With this plugin, we just inject the Safari 10 noModule fix, but when the 'esm' version of the application is parsed inside the Safari 10 browser and read the 'import' keyword the Syntax Error is thrown.

What is the purpose of this pull request?

[X] Bug fix
[X] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[X] Read the Contributing Guidelines.
[X] Read the Pull Request Guidelines and follow the Commit Convention.
[X] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[X] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[X] Ideally, include relevant tests that fail without this PR but pass with it.

If you don't like the parameter question, then we should check if it is possible to block the execution of the "modern" output in Safari 10 and favor the execution of the legacy version. How about?

I think it is intended to run the legacy chunk for Safari 10 because Safari 10 does not support dynamic import and import.meta. If the legacy chunk does not run for Safari 10, that is a bug.
related: https://github.com/vitejs/vite/pull/3885

I think it is intended to run the legacy chunk for Safari 10 because Safari 10 does not support dynamic import and import.meta. If the legacy chunk does not run for Safari 10, that is a bug.
related: #3885

next time I will check better, sorry


[App]: Downloading csv from the web app generates incoherent column names

Current Behavior
I generated a plot in a report from grouping line plots from multiple runs. I download then the csv associated to this plot to work on it. The plot looks like this:

When I download the csv and extract the column names I get this:
Index(['Training step', 'loss.beta_on_loss: true - _step',
       'loss.beta_on_loss: true - _step__MIN',
       'loss.beta_on_loss: true - _step__MAX',
       'loss.beta_on_loss: true - Variance Loss CVL over Variance Training Loss',
       'loss.beta_on_loss: true - Variance Loss CVL over Variance Training Loss__MIN',
       'loss.beta_on_loss: true - Variance Loss CVL over Variance Training Loss__MAX',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:45:31 - _step',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:45:31 - _step__MIN',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:45:31 - _step__MAX',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:45:31 - Variance Loss CVL over Variance Training Loss',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:45:31 - Variance Loss CVL over Variance Training Loss__MIN',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:45:31 - Variance Loss CVL over Variance Training Loss__MAX',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:43:43 - _step',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:43:43 - _step__MIN',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:43:43 - _step__MAX',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:43:43 - Variance Loss CVL over Variance Training Loss',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:43:43 - Variance Loss CVL over Variance Training Loss__MIN',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:43:43 - Variance Loss CVL over Variance Training Loss__MAX',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:41:11 - _step',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:41:11 - _step__MIN',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:41:11 - _step__MAX',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:41:11 - Variance Loss CVL over Variance Training Loss',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:41:11 - Variance Loss CVL over Variance Training Loss__MIN',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:41:11 - Variance Loss CVL over Variance Training Loss__MAX',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:39:29 - _step',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:39:29 - _step__MIN',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:39:29 - _step__MAX',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:39:29 - Variance Loss CVL over Variance Training Loss',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:39:29 - Variance Loss CVL over Variance Training Loss__MIN',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:39:29 - Variance Loss CVL over Variance Training Loss__MAX',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:34:26 - _step',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:34:26 - _step__MIN',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:34:26 - _step__MAX',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:34:26 - Variance Loss CVL over Variance Training Loss',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:34:26 - Variance Loss CVL over Variance Training Loss__MIN',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:34:26 - Variance Loss CVL over Variance Training Loss__MAX',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:32:29 - _step',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:32:29 - _step__MIN',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:32:29 - _step__MAX',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:32:29 - Variance Loss CVL over Variance Training Loss',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:32:29 - Variance Loss CVL over Variance Training Loss__MIN',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:32:29 - Variance Loss CVL over Variance Training Loss__MAX',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:31:11 - _step',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:31:11 - _step__MIN',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:31:11 - _step__MAX',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:31:11 - Variance Loss CVL over Variance Training Loss',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:31:11 - Variance Loss CVL over Variance Training Loss__MIN',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:31:11 - Variance Loss CVL over Variance Training Loss__MAX',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:29:51 - _step',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:29:51 - _step__MIN',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:29:51 - _step__MAX',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:29:51 - Variance Loss CVL over Variance Training Loss',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:29:51 - Variance Loss CVL over Variance Training Loss__MIN',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:29:51 - Variance Loss CVL over Variance Training Loss__MAX',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:25:19 - _step',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:25:19 - _step__MIN',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:25:19 - _step__MAX',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:25:19 - Variance Loss CVL over Variance Training Loss',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:25:19 - Variance Loss CVL over Variance Training Loss__MIN',
       'loss.beta_on_loss: 2gaussians_dsm_cvl_2023-10-19 10:25:19 - Variance Loss CVL over Variance Training Loss__MAX',
       'loss.beta_on_loss: 2gaussians_dsm_cvl - _step',
       'loss.beta_on_loss: 2gaussians_dsm_cvl - _step__MIN',
       'loss.beta_on_loss: 2gaussians_dsm_cvl - _step__MAX',
       'loss.beta_on_loss: 2gaussians_dsm_cvl - Variance Loss CVL over Variance Training Loss',
       'loss.beta_on_loss: 2gaussians_dsm_cvl - Variance Loss CVL over Variance Training Loss__MIN',
       'loss.beta_on_loss: 2gaussians_dsm_cvl - Variance Loss CVL over Variance Training Loss__MAX'],
      dtype='object')

The naming is incoherent. I do have a parameter loss.beta_on_loss but I don't understand why some columns have loss.beta_on_loss: true and others nothing. Note that I am grouping wrt to two parameters beta_on_loss and sigma. Also in the column names I do recognise the names I gave to my runs and that are grouped, but then a lot of the runs are missing. In total I used 30 runs to create this plot.
Expected Behavior
I expect the column names to be the one of the legends.
Steps To Reproduce
No response
Screenshots
No response
Environment
OS:
Mac OSX
Browsers:
Safari & Firefox
Version:
0.14.0
Additional Context
Thank you for any help

Hello @pablo2909 ,
I am glad to assist your concern, may I please request for the actual link of your project workspace please?

Hi, thank you for your help, please find it here

WandB Internal User commented:
pablo2909 commented:
Hi, thank you for your help, please find it here

Thank you @pablo2909 for sharing your project workspace, we'll check this out and get back to you.

WandB Internal User commented:
JoanaMarieL commented:
Thank you @pablo2909 for sharing your project workspace, we'll check this out and get back to you.

Any news ? :)

@sydholl @JoanaMarieL Anything ? :)

hey @pablo2909 - still looking into this - I was able to reproduce the behavior you're seeing and I have a few follow up questions:

are you performing this grouping within the runs sidebar? or within the chart itself? any details about the steps you're taking to perform this grouping would be super helpful
were you able to see grouped runs exported as a csv in the past? any details about this would help a ton!

thank you for your patience!

Yes the grouping is happening within the runs sidebar, the only thing I touch in the chart is changing the legend name and specifying that the spread computed in the grouping should be a standard error
It's my first time experimenting with csv on wandb so I have no prior experience

Thank you for looking into it

hey @pablo2909! few updates:

we currently do not support exporting grouped run metrics (which is why you're seeing a bunch of column names, reflective of individual runs, not the actual values of the grouped metrics). i will make a feature request for this and let you know of any plans there are to work on this & update you with any work
were you able to see more of your runs in the csv once you toggled the visibility settings in the runs table? (the eye icon next to the runs) or is the csv download not responsive to those changes?
for now I recommend exporting the csv without run grouping, then performing the grouping and data manipulation using pandas or another library

please let me know if you have any questions about this, and i will be sure to update this github thread if any progress arises on the feature request

Thank you, that would be a really cool feature imo and I hope it can happen :)
Yes toggling the eye icon impacts the number of runs exported in the csv
I'll do that !

Thanks again for the help, I'll leave the issue open then to follow up on this feature request


throw errors when invalid values passed to dash(...)

The dash(...) function now throws errors when passed invalid values

Fixes #949. Current behavior is that dash(...) accepts values such as 0 for length, which breaks rendering in some viewers. There is a demo in the linked issue of the impact of the problem.

Checklist:

[x] Tests (preference for unit tests)
[x] Documentation
[x] Update CHANGELOG.md
[x] Ready to be merged

@danthegoodman feedback appreciated 😄

@danthegoodman I think it's ready for another look.

I'm afraid this will break rendering since does not handle space option.
Will do some tests.

@blikblum You're right. I missed that in my second review. Good catch.

I'll get it squared in a couple days unless someone beats me to it.
(misspelled from my mobile phone)
On Fri, May 3, 2019, 10:51 PM Danny Kirchmeier<EMAIL_ADDRESS>wrote:

@blikblum https://github.com/blikblum You're right. I missed that in my
second review. Good catch.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/foliojs/pdfkit/pull/969#issuecomment-489290872, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAMIYNA3WAOIOWHVIGHW3J3PTUB5FANCNFSM4HKUFARA
.

I added dash unit tests to ensure no regressions are made with https://github.com/foliojs/pdfkit/commit/f486d11efe25105bf423d0f6e197bfa4d06e8545
You should rebase and add the throw tests in vector.spec.js. As long as unit tests are running all is fine

@blikblum thanks for those unit tests for dash(...) - that really illustrated how badly I had broken things 😅
I moved the new tests to vector.spec.js and fixed the issues that your unit tests revealed. I went ahead and rewrote the PR as a single commit as well.

@blikblum one more look please?

Many thanks

Any guidance on when this fix will be available in a published release?


WIP: Add JSON schemas for all OpenAI requests

Added JSON Schemas for

If a component should be generated
If a component shouldn't be generated, the message to display
If a component should be generated, the details required

Modified API call to require these schemas be included

Running into bit of an error here and I believe it is because of this option: https://github.com/Antel0pe/hydraai/blob/e5ffbef100f2de372c56132c91f90427a36b4ddc/package/src/hydra-ai/ai-service.ts#L31
When it's not included, no error. When it IS included I get the following error: "open ai Internal error: Error: 400 Invalid schema for response_format 'generateSpecifiedComponentSchema': In context=('properties', 'props'), 'additionalProperties' is required to be supplied and to be false."
The .passthrough option says that you can include any additional properties you need with this schema and that is being used by chatgpt to supply the component props values.
Since before my commits, this schema+option was given in a string along with the request, it didn't cause a problem. Now that I'm using Structured Output and demanding the output be given with this schema option, it's giving an error that I can't ask it to specify additional properties in the response.
The problem I'm currently in is dynamically defining the zod schema according to the component prop definitions so that we don't need the passthrough option and can avoid the error... which looks pretty close to this: https://github.com/Antel0pe/hydraai/blob/e5ffbef100f2de372c56132c91f90427a36b4ddc/package/src/hydra-ai/ai-service.ts#L319

hi antelope can you help me understand what this is for? also mind rebasing with latest commits?


Implement Swipe to CheckIn/Out in Attendee List

Feature Request
Feature Description

Implement Swipe to check in/out in Attendee List layout.
Change the Attendee List layout item UI to make it similar to EventBrite.

Would you like to work on the issue?
Yes

I will like to work on it

This is a just a small UI enhancement with lesser priority. Hence can be closed.


How to translate all the labels?

Research on how to translate all the labels automatically in flask-admin?

Should enable multi language on the project first, then consider how to translate all the labels.


Possible to use with a local LLM?

As titled, would prefer to use a local LLM instead of OpenAI's GPT. I arrived here via this tutorial/introduction to RAG;
https://simonevellei.com/blog/posts/leveraging-go-and-redis-for-efficient-retrieval-augmented-generation/

I suggest to use localAI and use a custom LLM. Then connect LinGoose to LocalAI using a custom openai client (WithClient( )) with local endpoint

@henomis Can you comment on why localAI and not Ollama?

@airtonix I will check this project and the possibility of integrating it into Lingoose. Thanks for the suggestion.

Ollama will be supported in the next lingoose version.