libssl.so.1.1 missing

I just rebuilt my win11 machine / WSL / Ubuntu 22.04LTS.
I pulled down the current build for momument but it failed with missing library.   libssl.slo.1.1
This seems to be known issue, you can fix it with
wget http://nz2.archive.ubuntu.com/ubuntu/pool/main/o/openssl/libssl1.1_1.1.1f-1ubuntu2.19_amd64.deb
sudo dpkg -i libssl1.1_1.1.1f-1ubuntu2.19_amd64.deb
Might be worth adding to the installation instructions.

Huh, interesting - thanks for flagging!  If others are likely to run into this, it'd definitely be good to add it to the install instructions.
So, to clarify, this is a problem when downloading the latest release (from the GitHub releases page) rather than building from source (i.e. using the cargo command)?

Yes the issue came up with the downloading the latest release from GitHub.  However I  know in the past I built from source and I think it was the same problem as well that the library was missing.  Seems a common Ubuntu 22.04 issue.  I have not tested on Debian yet.  If i get a chance i will
Paul

From: Ben White-Horne @.>
Sent: 14 August 2023 00:11
To: kneasle/ringing @.>
Cc: Butler, Paul @.>; Author @.>
Subject: Re: [kneasle/ringing] libssl.so.1.1 missing (Issue #300)
Huh, interesting - thanks for flagging! If others are likely to run into this, it'd definitely be good to add it to the install instructions. So, to clarify, this is a problem when downloading the latest release (from the GitHub releases page)
ZjQcmQRYFpfptBannerStart
This Message Is From an External Sender
This message came from outside your organization.
ZjQcmQRYFpfptBannerEnd
Huh, interesting - thanks for flagging! If others are likely to run into this, it'd definitely be good to add it to the install instructions.
So, to clarify, this is a problem when downloading the latest release (from the GitHub releases page) rather than building from source (i.e. using the cargo command)?
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https://github.com/kneasle/ringing/issues/300*issuecomment-1676476230__;Iw!!N3hqHg43uw!qsbOJSmRYJOPcQ1INESAZZ5JBC8ksJBtovHVbEWt28ChO104folKe2zycXoO3W3MBucTYm1tTS0TfU1QD73TIGt6GbZG$, or unsubscribehttps://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AOFFMREXGQFCQEZD6223RULXVFGKTANCNFSM6AAAAAA3MR27LU__;!!N3hqHg43uw!qsbOJSmRYJOPcQ1INESAZZ5JBC8ksJBtovHVbEWt28ChO104folKe2zycXoO3W3MBucTYm1tTS0TfU1QD73TIPpv6k2n$.
You are receiving this because you authored the thread.Message ID: @.***>

OK, nice - I'd be surprised if a fresh build behaved differently since the release builds are built using cargo, just as when installing.
I've made a pull request to add install instructions to the guide.  Can you quickly check if what I've written is helpful before I merge it and make it public:
https://github.com/kneasle/ringing/blob/libssl-ubuntu/monument/cli/guide.md#library-libsslso11-not-found
Let me know any improvements and I'll fix and merge :)

Hi Ben
text looks fine
cheers
Paul
PS will you implementing Grandsire/Stedman features ?

From: Ben White-Horne @.>
Sent: 16 August 2023 00:09
To: kneasle/ringing @.>
Cc: Butler, Paul @.>; Author @.>
Subject: Re: [kneasle/ringing] libssl.so.1.1 missing (Issue #300)
OK, nice - I'd be surprised if a fresh build behaved differently since the release builds are built using cargo, just as when installing. I've made a pull request to add install instructions to the guide. Can you quickly check if what I've
ZjQcmQRYFpfptBannerStart
This Message Is From an External Sender
This message came from outside your organization.
ZjQcmQRYFpfptBannerEnd
OK, nice - I'd be surprised if a fresh build behaved differently since the release builds are built using cargo, just as when installing.
I've made a pull request to add install instructions to the guide. Can you quickly check if what I've written is helpful before I merge it and make it public:
https://github.com/kneasle/ringing/blob/libssl-ubuntu/monument/cli/guide.md#library-libsslso11-not-foundhttps://urldefense.com/v3/__https://github.com/kneasle/ringing/blob/libssl-ubuntu/monument/cli/guide.md*library-libsslso11-not-found__;Iw!!N3hqHg43uw!o-qCEUkQRjwhCKeTuSGujYqcc0ftVdTH455SXroXeizqXF9p22_RNa3qRJDjkCQxpaTiSZRqCS6VsERkZOf2U0e-zqev$
Let me know any improvements and I'll fix and merge :)
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https://github.com/kneasle/ringing/issues/300*issuecomment-1679686292__;Iw!!N3hqHg43uw!o-qCEUkQRjwhCKeTuSGujYqcc0ftVdTH455SXroXeizqXF9p22_RNa3qRJDjkCQxpaTiSZRqCS6VsERkZOf2U8d0HtLa$, or unsubscribehttps://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AOFFMREWGPXWVMQF2RBZVJTXVPXQZANCNFSM6AAAAAA3MR27LU__;!!N3hqHg43uw!o-qCEUkQRjwhCKeTuSGujYqcc0ftVdTH455SXroXeizqXF9p22_RNa3qRJDjkCQxpaTiSZRqCS6VsERkZOf2U1RQ6eAQ$.
You are receiving this because you authored the thread.Message ID: @.***>

text looks fine

Nice, I've merged the PR so it's now public.

PS will you implementing Grandsire/Stedman features ?

So, this is a little tricky for two reasons.
Firstly, Grandsire singles will be tricky to implement because they affect two changes (whereas every other call in common use only affects one).  Currently, Monument assumes that calls only overwrite one piece place notation, and it would require a lot of work to relax the assumption.
Secondly, Monument's goal isn't just to generate compositions, it wants to generate human-friendly compositions.  For plain-bob-like methods, there aren't a lot of high-level structure conventions - enforcing tenors together with relatively few calls generally produces very human compositions.  But both Grandsire and Stedman have no such constraints, instead opting for mixing 'padding' courses (which keep some bells fixed to generate more of the same music) and 'turning' courses (which switch between these different types of music).  So, a human-friendly composition generator must be aware of this overall structure.
In summary, this may be possible (though very tricky) to achieve with Monument's current search algorithm, but to do this well you would really want a generator which designed specifically for Grandsire and/or Stedman.


cutorch does not load

I have tried deleting and installing torch, cutorch, cunn multiple times. However, I am just not able to make require 'cutorch' work.
I removed cutorch using:
luarocks remove cutorch --force
And then re-installed using:
luarocks install cutorch
At the end of installation I get:
cutorch scm-1 is now built and installed in /afs/csail.mit.edu/u/s/smadan/torch/install/ (license: BSD)
But the error still persists.
Error when I try luajit -l cutorch is:
luajit: .../u/s/smadan/torch/install/share/lua/5.1/cutorch/init.lua:2: cannot load '/afs/csail.mit.edu/u/s/smadan/torch/install/lib/lua/5.1/libcutorch.so'
stack traceback:
[C]: in function 'require'
.../u/s/smadan/torch/install/share/lua/5.1/cutorch/init.lua:2: in main chunk
[C]: at 0x0046b7d0
[C]: at 0x004065d0
Error when using require 'cutorch' in interactive th session:
...du/u/s/smadan/torch/install/share/lua/5.1/trepl/init.lua:389: .../u/s/smadan/torch/install/share/lua/5.1/cutorch/init.lua:2: cannot load '/afs/csail.mit.edu/u/s/smadan/torch/install/lib/lua/5.1/libcutorch.so'
stack traceback:
[C]: in function 'error'
...du/u/s/smadan/torch/install/share/lua/5.1/trepl/init.lua:389: in function 'require'
[string "_RESULT={require 'cutorch'}"]:1: in main chunk
[C]: in function 'xpcall'
...du/u/s/smadan/torch/install/share/lua/5.1/trepl/init.lua:661: in function 'repl'
...adan/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:204: in main chunk
[C]: at 0x004065d0
I also get the error:
luajit -l libcutorch
luajit: error loading module 'libcutorch' from file '/afs/csail.mit.edu/u/s/smadan/torch/install/lib/lua/5.1/libcutorch.so':
/usr/local/MATLAB/MATLAB_Runtime/v901/sys/os/glnxa64/libstdc++.so.6: version `GLIBCXX_3.4.18' not found (required by /afs/csail.mit.edu/u/s/smadan/torch/install/lib/libTHC.so.0)
stack traceback:
[C]: at 0x0047b020
[C]: at 0x0046b7d0
[C]: at 0x004065d0
CUDA version: 8.0.

Oops, I met the same problem, did you solve the problem?


No component is showing up.

Describe the bug
Hi there, I am trying to use WPF UI in my project with .NET 7.0 normal WPF, whatever I do, nothing shows up, there are no errors too. I followed the tutorial, nothing happens
To Reproduce

Download
Setup the XAML
Setup up CS
Run

Expected behavior
The controls to appear
Screenshots

OS version
Windows 11
.NET version
.NET 7.0
WPF-UI NuGet version
2.1
Additional context
No response

UPDATE I think the error is because of this:

weird i get the error but i can still add the controls and stuff in my vs 2022 project so i don't think its that did you remember to add the references to the app.xmal and the xmal file you are trying to get the controls in? also are controls appearing in the toolbox?

weird i get the error but i can still add the controls and stuff in my vs 2022 project so i don't think its that did you remember to add the references to the app.xmal and the xmal file you are trying to get the controls in? also are controls appearing in the toolbox?
(oh yeah for infobars make sure isopen is true)

Yes, all of that is finished, I used the extension now it works!


Long delay before rdpClientConProcessMsgClientInfo

I'm using xrdp 0.9.19-2 on arch with kde and it works. When connecting I have a 10-12 second period where I only see a black screen, this is when resuming a session or starting a new one. In my .xorgxrdp.10.log I see
[  1185.024] rdpkeybChangeKeyboardControl: autoRepeat off
<long delay here>
[  1201.082] rdpClientConProcessMsgClientInfo:
[  1201.082]   got client info bytes 7072
[  1201.082]   jpeg support 0
[  1201.082]   offscreen support 0
[  1201.082]   offscreen size 0
[  1201.082]   offscreen entries 0
[  1201.082]   client can not do offscreen to offscreen blits
[  1201.082]   client can do new(color) cursor
[  1201.082]   client can not do multimon

I had been using manjaro+kde+xrdp for a while on the same hardware before switching to arch and connections were immediate. I'm connecting with the mac microsoft remote desktop client. Any idea what might be causing my issue?

Hi @jdart
Could you post your xrdp.log and xrdp-sesman.log files from just after trying to connect until the 10-12 second period is over?
Also, and this is a pure guess, check that name resolution of the client IP isn't hanging on the target machine. Log in to the target and check that dig -x <client-ip> comes back reasonably swiftly. If you're running dual-stack, check both the IPv4 and IPv6 addresses.

@matt335672 Ty for the help! Dig responded instantly, I only do ipv4. Here are the logs:
[20221119-13:16:26] [DEBUG] xrdp_wm_log_msg: connecting to sesman on <IP_ADDRESS>:3350
[20221119-13:16:26] [INFO ] connecting to sesman on <IP_ADDRESS>:3350
[20221119-13:16:26] [INFO ] xrdp_wm_log_msg: sesman connect ok
[20221119-13:16:26] [INFO ] sesman connect ok
/var/log/xrdp.log
[20221119-13:16:26] [DEBUG] xrdp_wm_log_msg: sending login info to session manager. Please wait...
[20221119-13:16:26] [INFO ] sending login info to session manager. Please wait...
[20221119-13:16:26] [DEBUG] xrdp_wm_login_mode_changed: login_mode is 3
[20221119-13:16:26] [INFO ] xrdp_wm_log_msg: login successful for user jon on display 10
[20221119-13:16:26] [INFO ] login successful for user jon on display 10
[20221119-13:16:26] [INFO ] loaded module 'libxup.so' ok, interface size 10296, version 4
[20221119-13:16:26] [DEBUG] xrdp_wm_log_msg: started connecting
[20221119-13:16:26] [INFO ] started connecting
[20221119-13:16:26] [INFO ] lib_mod_connect: connecting via UNIX socket
[20221119-13:16:26] [INFO ] lib_mod_log_peer: xrdp_pid=365410 connected to X11rdp_pid=319839 X11rdp_uid=1000 X11rdp_gid=1000 client_ip=::ffff:<IP_ADDRESS> client_port=55892
[20221119-13:16:26] [DEBUG] xrdp_wm_log_msg: connected ok
[20221119-13:16:26] [INFO ] connected ok
[20221119-13:16:30] [WARN ] xrdp_mm_chansrv_connect: connect failed trying again...
[20221119-13:16:30] [DEBUG] Closed socket 20 (AF_UNIX)
[20221119-13:16:34] [WARN ] xrdp_mm_chansrv_connect: connect failed trying again...
[20221119-13:16:34] [DEBUG] Closed socket 20 (AF_UNIX)
[20221119-13:16:38] [WARN ] xrdp_mm_chansrv_connect: connect failed trying again...
[20221119-13:16:38] [DEBUG] Closed socket 20 (AF_UNIX)
[20221119-13:16:42] [WARN ] xrdp_mm_chansrv_connect: connect failed trying again...
[20221119-13:16:42] [ERROR] xrdp_mm_chansrv_connect: error in trans_connect chan
[20221119-13:16:42] [DEBUG] status from xrdp_mm_connect() : 0
[20221119-13:16:42] [DEBUG] Login state change request WMLS_CONNECT_IN_PROGRESS -> WMLS_CLEANUP
[20221119-13:16:42] [DEBUG] Closed socket 18 (AF_INET6 ::1 port 55280)
[20221119-13:16:42] [DEBUG] xrdp_wm_login_mode_changed: login_mode is 4
[20221119-13:16:42] [DEBUG] Login state change request WMLS_CLEANUP -> WMLS_INACTIVE
[20221119-13:16:42] [DEBUG] xrdp_wm_login_mode_changed: login_mode is 5

/var/log/xrdp-sesman.log
[20221119-13:16:26] [INFO ] Socket 12: AF_INET6 connection received from ::1 port 55280
[20221119-13:16:26] [INFO ] ++ reconnected session: username jon, display :10.0, session_pid 319837, ip ::ffff:<IP_ADDRESS>:55892 - socket: 12
[20221119-13:16:26] [ERROR] sesman_data_in: scp_process_msg failed
[20221119-13:16:26] [INFO ] Starting session reconnection script on display 10: /etc/xrdp/reconnectwm.sh
[20221119-13:16:26] [ERROR] sesman_main_loop: trans_check_wait_objs failed, removing trans
[20221119-13:16:26] [INFO ] Process 365411 has exited

This looks significant:-
[20221119-13:16:30] [WARN ] xrdp_mm_chansrv_connect: connect failed trying again...
[20221119-13:16:30] [DEBUG] Closed socket 20 (AF_UNIX)
[20221119-13:16:34] [WARN ] xrdp_mm_chansrv_connect: connect failed trying again...
[20221119-13:16:34] [DEBUG] Closed socket 20 (AF_UNIX)
[20221119-13:16:38] [WARN ] xrdp_mm_chansrv_connect: connect failed trying again...
[20221119-13:16:38] [DEBUG] Closed socket 20 (AF_UNIX)
[20221119-13:16:42] [WARN ] xrdp_mm_chansrv_connect: connect failed trying again...
[20221119-13:16:42] [ERROR] xrdp_mm_chansrv_connect: error in trans_connect chan

xrdp-chansrv appears to be failing on your system. xrdp tries to connect 4 times with a time out of 3 seconds each time.
Since you're running 0.9.12, it could be #1512. Could you look in ~/.local/share/xrdp/xrdp-chansrv.10.log and see if there's anything useful there?

I tried removing the aur xrdp package, building/installing from source, had some problems, did a make uninstall, then re-installed the xrdp package from aur and now I connect immediately... so no idea what my issue was. Closing...


REST documentation out of date

It looks like the Swagger-based REST documentation is out of date:

I'm not sure what memberships-library does, but I think members should be changed to members-library?
https://github.com/oaeproject/Hilary/blob/master/node_modules/oae-search/lib/rest.js#L56

Assigning to @mrvisser for fix

Assigning to @sathomas for review

@mrvisser the fix looks fine to merge, but I'm not sure that the Travis CI should be ignored.

@sathomas yes that one I believe is a known intermittent failure and unrelated to this fix. Those tests are being investigated as they occur on master builds, where we have access to the log files.

Merged


"Cut the line" shortcut

Add shortcut to cut the line. Here i mean not only spaces but text as well. Option copy to clipboard would make it even more useful.
Ctrl+Shift+Delete.
Thank you!

Do you have a set character limit that you want?  You could make your own shortcut to do this.

There is already Shift+Alt+K for removing line. You can change shortcut to one that suits you better using >Preferences: Open Keyboard Shortcuts or in keybindings.json:
{
    "key": "shift+alt+delete",
    "command": "editor.action.deleteLines",
    "when": "textInputFocus && !editorReadonly"
}

For cutting line into clipboard you can just use Ctrl+L to select whole line and then regular Ctrl+X


Wrong width calculation of empty search input

Now search input width calculate as parent innerWidth():
    if (this.$search.attr('placeholder') !== '') {
      width = this.$selection.find('.select2-selection__rendered').innerWidth();

This is wrong, because ul.select2-selection__rendered has left&right padding.
So search input makes select2 container grows up.
To fix that it just need to calculate width by width(), that not count paddings.
JS fiddle: https://jsfiddle.net/karakum/2o6xzupe/
Empty select:

Not empty:

It's not entirely clear to me how switching this from innerWidth to width would fix the issue (since width should return a larger value than innerWidth), but I can confirm that this bug exists.

@kevin-brown, This pictures from jQuery's documentation:

innerWidth include node paddings, width - does not.

That's pretty convincing, looks like I was getting the two mixed up. :+1:


Adding Data Lasso build files

This changes two things:

New /build directory, where Data Lasso script will be built to. /build/datalasso.js is now what you will get when you require('data-lasso')
Using sassr to include up styles. There is no other good way to include stylesheet from an npm module, and with little styles that Data Lasso has, sassr is a good option.

Because of that, stylesheet build stuff is now gone too - it is all just single js file now.

@heylookltsme @kmck

:+1:

:+1:


Consider moving the K8x manifest examples to the top folder

There are some example Kubernetes manifests:
https://github.com/openshift-knative/showcase/tree/86e484eee57c45301a9d78ab8a0aea7402789318/quarkus/src/main/kubernetes
Maybe they should be moved to the top level, to be easier to find. Also, a bit of documentation about them will be helpful.

/kind cleanup

I would like to work on this.


Vertical buttons incorrectly aligned on 4k with scaling

System information:

Operating system (distribution) and version Win10
DBeaver version 21.2.2

Steps to reproduce, if exist:
Open data editor on 4k resolution with scaling (e.g. 175% or 200%)

Verified


Automerge option

Hi, it would be great if meeseeksbot could be configured to automerge PRs. I trust my CI system and the fact that it was merged cleanly enough to want to do that.
https://github.com/scientific-python/MeeseeksDev/blob/bfd704fcbb4cfe58fa851e1fde740e4b71cafb1c/meeseeksdev/meeseeksbox/commands.py#L1011-L1027

Isnt' that covered by github merge queues now (https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/configuring-pull-request-merges/managing-a-merge-queue) – which I haven't tried so I might be wrong.

I want to make meeseeksbot fully automatic: on merge, it creates the backport PR and “hits the button” for it to get merged.
I think a merge queue wouldn’t change this, instead of hitting automerge, someone would have to hit autoqueue.

I think a merge queue wouldn’t change this, instead of hitting automerge, someone would have to hit autoqueue.

👍


Move operator instances to own module

See Issue #5 for the rationale behind these changes.
BinaryOp.PLUS becomes binary.plus
binary.plus is still an instance of BinaryOp, but we make instances available in their own module rather than bolting them on to the class.

LGTM


Interaction between clickOutsideDeactivates and returnFocusOnDeactivate

My use case is that I have a modal, but the background is scrollable while the modal is open.
I am using both clickOutsideDeactivates and returnFocusOnDeactivate.
If the page is scrolled and you click outside in a non-focusable area, the page will jump back upwards to the last focused element. What I would like to to have an option where clicking outside is treated differently from keyboard deactivation. For KB deactivation, return focus, for click, do not. This of seems like it conflicts with https://github.com/davidtheclark/focus-trap/issues/60, so another option would probably be needed for this. Do you have any thoughts on this?
Another option might be to return focus to the nearest on-screen element, but determining that sounds costly...
If we can come up with a solution that makes sense, I'm happy to submit a PR for it.

Closing this because we found a good way around it using mousedown events and preventDefault. Seems like Firefox already has this behaviour by default as well.


JDK upgrade

As a user I want to use features from JDK 11. So, please support it.

I believe that as a user, you import the carina project to your test infrastructure where you can use JDK 11.
I believe that forcing JDK on carina infrastructure may cause many working projects to stop working as mostly, JDK 11 is not supported in most of the companies (yet).
While you can use JDK 11 with backward compatibility to JDK 8, it cannot be vise versa.
Please correct me if i am wrong

moving even carina-grid onto the latest JDK produced a lot of troubles. And in carina-grid we control everything including deployment in scope of the container. Moving carina-core can be too risky

whole our zebrunner infrastructure where carina is widely used is already on jdk 11 so we switched to this version carina core as well in upcoming release: https://github.com/qaprosoft/carina/commit/2157100624c6d08f01a5f6aa2f12ab4496fb91c2


Comparison for int 0 not work

Package version eg. v9, v10:
v10
Issue, Question or Enhancement:
I'm using gin which uses validator/v10 for query/uriParams validation.
But I found that comparison like min or gt for int 0 not work
Code sample, to showcase or reproduce:
query := struct {
	ID     int    `form:"id" binding:"required,min=1"` // or gt=0
}{}
err := c.ShouldBindQuery(&query)

The test result is err is always nil when param id euqals to 0.

I think you're using incorrect tag name. Try form or query.

You're using incorrect tag name. Should be form instead of binding.

@MatheusZabin form indicates the key of query isn't it? The rule is defined by the binding tag. Am I missing or mistaking something?

You're using incorrect tag name. Should be form instead of binding.

@MatheusZabin form indicates the key of query isn't it? The rule is defined by the binding tag. Am I missing or mistaking something?

Sorry, I missread it.
Try to fill a gin issue, could be more appropriated.

query := struct { 	ID     *int    form:"id" binding:"required,min=1" // or gt=0 }{}

@hellowords
any idea why that happens? Is 0 somehow transformet to null or something else in (i guess) gin unmarshaling?

any issue here, any updates?


Please don't ignore election safety invariant

Hi, I was reading through the code to see what optimizations were provided around determining the actual matching log index of a follower that may have more uncommitted log entries than a leader in a manner similar to etcd's implementation when I stumbled across this code.
This code ignores the fact that there are two leaders for the same term which violates the critical election safety property shown in figure 3 of the raft paper. If this occurs it means that 2 different leaders received a quorum of votes and that those two leaders could potentially commit different log entries to the same log index at different followers. While I have no reason to suspect that this bug actually exists in this code base, it seems to me that allowing the protocol to silently continue and possibly corrupt data indefinitely is worse than crashing and making the system unavailable. Personally as a user/operator I'd much rather have an opportunity to see and fix a serious bug while the system was down, then work with corrupted data and potentially perform incorrect operations unwittingly.
I'd strongly suggest panicing here, even in production. Ideally and most likely, this bug doesn't actually exist, and it won't ever be hit in testing or production. But if it is ever seen, someone will at least know it happened.
Thank you,
Andrew

Thanks for bringing this issue. You're right, this can be a critical issue if it comes from a member in the same group, and it would be important for the application to have the option to be notified.
The reason why we (eBay) has ignored this is because of our deployment. We operate tens of thousands of Raft groups concurrently, and under certain conditions, a malfunctioning server might send heartbeats to members of a different Raft group. In such a case, what that Raft group should do is to ignore the message rather than to panic, as there is essentially no real issue.
Therefore, rather than mandating a panic, I will introduce a callback mechanism to allow the application to determine the appropriate action in these situations.

Thanks for bringing this issue. You're right, this can be a critical issue if it comes from a member in the same group, and it would be important for the application to have the option to be notified.
The reason why we (eBay) has ignored this is because of our deployment. We operate tens of thousands of Raft groups concurrently, and under certain conditions, a malfunctioning server might send heartbeats to members of a different Raft group. In such a case, what that Raft group should do is to ignore the message rather than to panic, as there is essentially no real issue.

Gotcha. That makes sense. The way I've handled this in the past is to add a separate "cluster id" that gets sent with each message so that these can get discarded. That does add a touch of overhead though, and your solution works.

Therefore, rather than mandating a panic, I will introduce a callback mechanism to allow the application to determine the appropriate action in these situations.

Thank you! I think this will be sufficient. I'm wondering if instead of a callback this could maybe be a configuration instead though, since it should be a one time setting only.

I prefer callback, as it will be more flexible and extensible than config. Callback can do things differently according to the contents of the request.
The fix has been merged: https://github.com/eBay/NuRaft/pull/537

Thank you for adding the callback.


Config: underscores in numeric literals

Doesn't look like Bootique can parse numeric literals in configs that are using Java 8 style with underscores. E.g.:
scheduler:
  threadPoolSize: 2
  triggers:
    - fixedDelayMs: 1_800_000
      initialDelayMs: 10_000

causes
com.fasterxml.jackson.databind.exc.InvalidFormatException: Can not construct instance of long from String value '1_800_000': not a valid Long value
An alternative solution may be to map each number using a specialized value object... LIke the above is a time interval that can be expressed in second or minutes

As I can see, the deserealization for numerics in yaml files is covered inside JsonDeserealizer, and all of that deserealizer methods are marked as final, so we cannot override that logic. In Jackson 2.10.0 is stated removing final's process for some methods, so possibly it will be done for JsonParser too.


After most recent update, all icons gone

After updating Icon folder all the folder icons are gone, the settings for the plugin as well as the right click menu for it

Please be more descriptive. What specific update version? Is there any error in the developer console of obsidian? Do you have a minimal reproducible example?

I've also had this problem. Updating to 1.3.5 today on Windows 10, and to 1.4.6 on iOS, the icons don't appear, and the plugin options in the right click context file explorer menu and in the settings all disappear.

The error showing on Obsidian in Windows 10 is:

Yeah, basically what @carissas086 has added is what I'm experiencing after updating

I debugged this a little, because it happened to me too. This might be helpful for the developer(s) as well as you users: This happens when you move files, as the filepath in the data.json can't be found in your vault anymore.
An easy fix for you now would be to fix the filepath(s) in that data.json: Just check all of them, they live in <your obsidian config folder, usually .obsidian>/plugins/obsidian-icon-folder/data.json. You will basically see a mapping from filepaths to icon names there, and I suppose that at least one of the filepaths will be wrong.
The other way to fix it would be to delete data.json, of course, but then you'd have to set everything up again.

I also have missing icons with the same TypeError, but i still have the plugin settings and context menu; icons disappear every time I close and reopen Obsidian. This started after updating to Icon Folder 2.0.2, and persists in 2.1.0 (Windows 11, Obsidian 1.3.5)

context menu still has options to change and remove icon, even though no folder icon is visible
re-installing Icon Folder does not fix this issue (as suggested in #175 for a similar, but different issue)

I am unable to recreate the disappearing icons bug in a fresh vault, but similar TypeErrors show up once Bookmarks core plugin is involved. 1 folder with 1 file, set icon to the folder, add the note into a bookmark group = Uncaught TypeError. The error does not happen when in Restricted Mode, and leaving Restricted Mode (re-enabling Icon Folder at the same time) also raises an error.

I wasn't really able to replicate this behavior. However, I've pushed a small fix and will release the version today.
For now, I'll close the issue. If this issue still persists, please reopen it with a reproducible vault.

fix worked, error's gone from console and the settings have reappeared along w the icons. thank you!


Webapp: journal specific data views

https://github.com/greenelab/scihub/commit/e1fa9453251f3d02c7be53fdda88f68769c97fc3 created the journal-specific data we'll need for the webapp. Specifically, we've computed:

scihub-log-journal-averages.tsv.xz: summary of request traffic per journal
scihub-log-journal-quantiles.tsv.xz: distribution of request traffic per journal
scihub-log-journal-top-articles.tsv.xz: top 100 visited articles per journal

These datasets still need to split into files for each journal, which will create many small files. But @arielsvn, I wanted to get you the dataset structure, so we can start prototyping journal specific views.

Nice @dhimmel! Last time we talked about having these two elements in the Journal specific page:

a bar chart of coverage by year
a table of the top 100 most downloaded papers

What data should I use to create the bar graph with the coverage by year??
Also it would be good to have test files for one journal, to make the graphs work with that data initially.

Here are three TSVs for journal 12001 (Journal of the Experimental Analysis of Behavior):

jounal-year-coverage-12001.txt
scihub-log-journal-top-articles-12001.txt
scihub-log-journal-quantiles-12001.txt

We can use these to prototype the journal specific pages before actually uploading all the files. Now is the time to determine the best format for these datasets and whether we need to make any changes.

Todo: In scihub-log-journal-top-articles-12001.txt, denote whether article is in Sci-Hub / LibGen.

@dhimmel that's the place where you intended to put the files?? Just to let you know that won't work:

XMLHttpRequest cannot load https://github.com/greenelab/scihub/files/1058944/jounal-year-coverage-12001.txt. Redirect from 'https://github.com/greenelab/scihub/files/1058944/jounal-year-coverage-12001.txt' to 'https://github-production-repository-file-5c1aeb.s3.amazonaws.com/85594549/…lename%3Djounal-year-coverage-12001.txt&response-content-type=text%2Fplain' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://localhost:9001' is therefore not allowed access.

It gives an error, that prevents AJAX requests to these urls. Maybe there's a configuration option on github to enable cross-origin requests for these files? If not, we'll have to find another solution to store them. We discussed some other options here https://github.com/greenelab/scihub/issues/10#issuecomment-300313572

that's the place where you intended to put the files??

The URLs above were automatically created by GitHub when I uploaded those files to this issues. They are not the permanent location for these files.

Maybe there's a configuration option on github to enable cross-origin requests for these files?

I think cross-origin requests will only work on GitHub for raw.githubusercontent.com/* URLs.
Can you workaround this for now, until we do the bulk upload?

The URLs above were automatically created by GitHub when I uploaded those files to this issues. They are not the permanent location for these files.

Perfect! I was confused with those urls. On raw.githubusercontent.com/* they work fine.

Can you workaround this for now, until we do the bulk upload?

Yes, that's what I'm doing with the sample files. I got Vega-Lite working on the app, with a standalone configuration file. I really like this setup! Will show you soon.


Why the fork?

I'm wondering if there's a reason for the fork, and upon what basis I should choose to use memmap v1 versus v2?

https://github.com/RazrFalcon/memmap2-rs/blob/master/CHANGELOG.md
Original memmap is abandoned.

https://github.com/RazrFalcon/memmap2-rs/blob/master/CHANGELOG.md
Original memmap is abandoned.

@RazrFalcon Thank you! That makes sense.


Downgrading from 1.6.0 to 1.5.1

Describe the bug
I removed 1.6.0 Canary 1 via brew, then installed 1.5.1 with brew, now I get this error:
start VM: VM exited unexpectedly; logs:
panic: last-used version 3 is newer than current version 2. Downgrades are not supported; please update OrbStack [recovered]
	panic: last-used version 3 is newer than current version 2. Downgrades are not supported; please update OrbStack [recovered]
	panic: last-used version 3 is newer than current version 2. Downgrades are not supported; please update OrbStack

This implies to me that the brew remove doesn't delete some state, and the removal script could be improved?
If this isn't fixable it'd be great to know what I should manually delete to get up and running again
To Reproduce

brew install orbstack (currently getting 1.5.1)
(possibly needed?) open OrbStack and use it for a while
Switch to canary channel and update in-app
brew remove orbstack
brew install orbstack
Run a command like orb

Expected behavior
I'd expect the brew remove to clean the system completely and allow a clean reinstall of 1.5.1
Diagnostic report (REQUIRED)
OrbStack info:
Version: 1.5.1
Commit: 4cfac15e1080617c70eb163966e1cb2009dac1c2 (v1.5.1)
System info:
macOS: 14.0 (23A344)
CPU: arm64, 8 cores
CPU model: Apple M1
Model: MacBookPro17,1
Memory: 16 GiB
Screenshots and additional context (optional)
No response

Thanks, deleting those files fixed the issue for me
I was only downgrading for benchmarking purposes :)

@kdrag0n I don't want to extend this issue given that it is already closed, but, I also had to downgrade to 1.5.1 because Webpack stopped working.
When the step for running yarn build I noticed that it was taking a lot of time so I had to abort the build. After downgrading to 1.5.1, the build worked normally.
Not sure if it is something related to Webpack, npm, but this what I can share from my dockerfile:
# Preparing Go Application Builder
FROM golang:1.21-alpine as builder

# Installing nodejs and other dependecies
RUN apk add --update nodejs-current python3 curl bash build-base

# Installing Yarn
RUN curl -o- -L https://yarnpkg.com/install.sh | bash -s --
ENV PATH="$PATH:/root/.yarn/bin:/root/.config/yarn/global/node_modules"
RUN yarn set version berry

ADD package.json .
ADD yarn.lock .
RUN yarn install
RUN yarn build

...

@dmuriel Please share a diagnostic report from Help > Upload Diagnostics, as well as your docker build command, in a new issue.
Also, is this --platform linux/amd64 on Apple Silicon by any chance?

Yes @kdrag0n, you are correct. I created a new issue #1186, I've put more information there.


How to allow combination of elements without repetition?

e.g. I want to allow any combination of the characters 'abcd' (but at least one of those characters) before a final character 'END'
So valid:
abcdEND
dbaEND
cEND
dcEND

Invalid:
END
fooEND
bcdeEND

At the moment I'm handling this by rejecting matches with js, but I'm wondering if there's a way of encoding this without writing three macros that expand to 4!, 3!, 2! elements (respectively).

You could do:
"a":? "b":? "c":? "d":? "END" {% //check for at least one in JS %}

You still have to have a reject call in JS in case there is none of "abcd" at all but it seems simpler to what you are describing.

That's pretty much what I am doing :)
My actual problem is how to parse vim-style bracket expressions, e.g:
<C-a> means Control and a
<A-a> means Alt and a
<CA-Enter> means Control Alt and enter
<Enter> means Enter without modifiers
<Delete> ...

My grammar:
BracketExpr -> "<" Modifier ModKey ">" {% bexpr=>bexpr.slice(1,-1) %}
             | "<" Key ">" {% bexpr=>bexpr.slice(1,-1) %}
Modifier -> [ACMS] ([ACMS] ([ACMS] ([ACMS]):?):?):? "-" {%
    /** For each modifier present,
        add its long name as an attribute set to true to an object */
    (mods, _, reject) => {
        const flatten = (ary) =>
            ary.reduce((a, b) => a.concat(Array.isArray(b) ? flatten(b) : b), [])
        mods = flatten(mods)

        const longNames = new Map([
            ["A", "altKey"],
            ["C", "ctrlKey"],
            ["M", "metaKey"],
            ["S", "shiftKey"],
        ])

        let modifiersObj = {}
        for (let mod of mods) {
            let longName = longNames.get(mod)
            if (longName) {
                // Reject if the same name is used twice.
                if (longName in modifiersObj) return reject
                else modifiersObj[longName] = true
            }
        }
        return modifiersObj
    }
    %}
# <>- permitted with modifiers, but not otherwise.
ModKey -> "<" | ">" | "-" | Key {% id %}
Key -> [^\s<>-]:+ {% (key)=>key[0].join("") %}

The code is a bit simpler if I do :?, which is how I was doing it before:
Modifier -> "a"i:? "c"i:? "m"i:? "s"i:? "-" {%
    /** For each modifier present,
        add its long name as an attribute set to true to an object */
    (mods, _, reject) => {
        const longNames = new Map([
            ["A", "altKey"],
            ["C", "ctrlKey"],
            ["M", "metaKey"],
            ["S", "shiftKey"],
        ])

        let modifiersObj = {}
        for (let mod of mods) {
            if (mod === null || mod === "-") continue
            let longName = longNames.get(mod.toUpperCase())
            if (longName) {
                // Reject if the same name is used twice.
                if (longName in modifiersObj) return reject
                else modifiersObj[longName] = true
            }
        }
        return modifiersObj
    }
    %}

Closing since it's been a while. :)


fix: hummock manager list version deltas by memory

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
Previously, when compute node call rpc ListVersionDeltas to fetch version updates, HummockManager in meta node reads meta store (e.g. use etcd if enabled) but not its memory data structures. This PR fix the problem.
Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

Seems this rpc is no longer used without test. I'll try to remove it.


feature: Specification - Contact Details [NP-717]

This is a really small spec as it's static and almost completely configurable.
Only assertions from story / Panos' input was that the top item should be bold and there should be at least 1 paragraph at all times.

Rightio. I've got that on every single component. So they need removing then, no problems.


simplify _ContainerApp

Making the initializer a global function instead (and cleaned up some tangentially related stuff)
Once this is merged, I can remove the synchronization of _ContainerApp and _LocalApp and turn them into simple dataclasses

thanks @mwaskom

👍


Feature/events separate page

Description
added functioning remove and open event buttons on modal. button on top right of event in both dashboard and calendar components. remove produces a modal warning popup. added some styling to single event page to add the side button and gray background.
Type of change

[x] New feature (non-breaking change which adds functionality)

Change Status

[x] Complete, tested, ready to review and merge

How Has This Been Tested?

[x] basic unit test created along with new component.  tested locally using local server and app instance.

Checklist

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

Oops, 2 more conflicts, @kyle-richardson.


New Component: HTML5 Notifications

Here are some common use cases:
https://web-push-book.gauntface.com/demos/notification-examples/
https://developer.mozilla.org/en-US/docs/Web/API/Notifications_API/Using_the_Notifications_API

Closing this ticket as it looks like BalusC is going to tackle it with OmniFaces and he already ran into some roadblocks he is thinking through and found browser oddness implementing. See: https://github.com/omnifaces/omnifaces/issues/508


Sporadic crash when running tests

When running cargo test --all-features I occasionally experience a crash. So far I've seen a few different errors when this happens:
running 6 tests
test chunk_20_default_params_16k ... FAILED
error: test failed, to rerun pass `--test snapshot`

Caused by:
  process didn't exit successfully: `/Users/robert/other/silero-rs/target/debug/deps/snapshot-922dfd7ba42f0d16` (signal: 11, SIGSEGV: invalid memory reference)

---- chunk_30_default_params_16k stdout ----
thread 'chunk_30_default_params_16k' panicked at /Users/robert/.cargo/registry/src/index.crates.io-6f17d22bba15001f/ort-2.0.0-rc.4/src/environment.rs:51:9:
assertion `left != right` failed
  left: 0x0
 right: 0x0
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

failures:
    chunk_30_default_params_16k

snapshot-922dfd7ba42f0d16(48386,0x700009d8f000) malloc: Heap corruption detected, free list is damaged at 0x600000f90600
*** Incorrect guard value:<PHONE_NUMBER>2143
snapshot-922dfd7ba42f0d16(48386,0x700009d8f000) malloc: *** set a breakpoint in malloc_error_break to debug
error: test failed, to rerun pass `--test snapshot`

I can reproduce the problem with a script which runs the tests in a loop:
#!/bin/sh

for i in {1..100}
do
    echo "Running test iteration $i"
    cargo test --all-features
done

I'm using rustc v1.80.0 on an Intel Mac running macOS 14.4.1. Tested with commit 00fccb247c18eb0274655e7acb593885d7362048.
There is no unsafe code in this crate so the problem must be something that the ort crate is doing that is somehow being triggered by this crate. Looking at ort-2.0.0-rc.4/src/environment.rs:51(from the log above) I suspect this might be an issue with concurrent initialization of the library by multiple threads in cargo's test runner.

So I think you're right on the cause I've raised an issue in ort and I'll see what they say. Might figure out if there's a nice way we can initialise the env in a thread safe way in the VadSession :thinking:


Add deconstructors for relevent types

The option to deconstruct types exist since C# 7, so it was about time to add the support to SVO's that benefit from it:

[x] Date
[x] DateSpan
[x] Fraction
[x] Money
[x] MonthSpan
[x] WeekDate
[x] YearMonth

It also has been added to FormattingAgurments, an helper to combine the arguments provided to apply .ToString(string format, IFormatProvider formatProvider), defined by IFormattable.

Not sure if useful information for you right now, but it is also possible to have multiple deconstructors for the same type, in order to allow for deconstructing in pattern matching. e.g.:
var x = ym switch
{
    YearMonth("2024-01") => "foo",
    YearMonth(string str) => str,
    YearMonth(2024, 1) => "bar",
    YearMonth(int y, int m) => "baz",
    _ => string.Empty,
};

But this might cause ambiguity conflicts when deconstructing in the way you are currently doing in the tests when the number of arguments is the same.

Not sure if useful information for you right now, but it is also possible to have multiple deconstructors for the same type, in order to allow for deconstructing in pattern matching. e.g. (..)

I'd tried to stick with the same guidelines as MS did in .NET. Do you have a concrete suggestion of a deconstruct you want add?

I was initially thinking to be able to match on the string version, but it gets finicky with different cultures, so perhaps it's for the better to exclude it


'afterValidated' event?

Hi @RobinHerbots, is there some way to hook up a function after the validation has been run by the Inputmask?
I've referenced the issue here, https://github.com/RobinHerbots/Inputmask/issues/1146
Lets say you have this configuration.
{
  min: 1,
  max: 40
}

I'm wondering on how to properly implement this kind of behavior. Thanks.

It would be nice

@adummy832 , @mfrischbutter ,
onBeforeWrite: null, //executes before writing to the masked element.
postValidation: null, //hook to postValidate the result from isValid.


[BUG] Mouse cursor disappearing when pressing ALT + ENTER

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?
the mouse cursor is disappearing whenever I press alt + enter for FSE which is useful for me and also when you're running on vulcan.

This only occurs for some people, and that's sort of just how FSE is. There isn't really anything that can be done to fix this, sorry.


[Enhancement] Generate code between functions is tabbed up to tab level

I like the autogenerated code emitted by omnisharp for vscode.
I do not like how when they get generated empty lines don't match the current tab level, as seen in the image.
People seem to really like their code prettyfiers but I don't use any, mostly for the fact it eliminates the tabs. My tabs are sacred to me and its purpose for being there is so that I don't have to tab to make a function or deal with the existence of empty lines since the editor when pressing enter will give you one. You also will get tabs emitted when you select a function's closing brace and press enter to create said function (the only way to do it besides clicking into the 'pit' to tab it up then to create a function).

As I mentioned to on discord, the OmniSharp project has no control over this. You'll need to open a feature request at https://github.com/dotnet/roslyn.

Reference: https://github.com/dotnet/roslyn/issues/66424


SQL. add support for and

This PR adds support for the And not. In our current prototype, this can only occur in selections over lists of conditions, something like:
table.filter([table.a == 5 & table.b == 7])

However, this is equivalent to writing:
table.filter([table.a == 5, table.b == 7])

With this change, we lower to the latter form, if the former is encountered.
This might be a bit hacky, but it works for the TPC-H queries nad it closes the second not-so-nice thing about our representations.

Anticipating your reply: if you want to do a hacky thing now given the tight timeline, please mark it as such in the code and document briefly what a better solution could look like.

I agree again with this being an optimization and not a lowering. Again, I took the route of least resistance for this prototype, as this is the last thing I want to add and this really only being something that is added to make one plot a bit nicer and delete one sentece claiming a difference.
I feel like I would have the time to implement this differently, i.e., modeling the And and then rewriting it during lowering or doing it as an optimization pass, or even a step further, modeling And in rel_alg and getting a proper design this way. I don't really mind which way we go here.
Do you feel this is necessary? Or do you think explaining the optimized designs in a todo would suffice?

I will do some other things that I feel are more important atm, like getting the tables in shape, and then I will revisit this closer to the deadline again.

Do you feel this is necessary? Or do you think explaining the optimized designs in a todo would suffice?

No, as I said, marking it as a hack with a brief description of how you imagine it being done correctly is enough.


Can't get the example code to work.

It's most likely my stupidity
this is my error on python 3.5.2
Traceback (most recent call last):
File "C:/Users/Administrator/Desktop/test.py", line 1, in
from chatterbot import ChatBot
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python35\lib\site-packages\chatterbot_init_.py", line 4, in
from .chatterbot import ChatBot
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python35\lib\site-packages\chatterbot\chatterbot.py", line 4, in
from .logic import MultiLogicAdapter
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python35\lib\site-packages\chatterbot\logic_init_.py", line 1, in
from .logic_adapter import LogicAdapter
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python35\lib\site-packages\chatterbot\logic\logic_adapter.py", line 3, in
from chatterbot.utils import import_module
ImportError: cannot import name 'import_module'

Which example are you trying to run?

Hi @tryzombie501 See my response here for how to fix: https://github.com/gunthercox/ChatterBot/issues/443

Thank you, I've been dying to play around with this.

Sorry to bother a closed issue but i was woundering if the chatbot can learn from speaking with the user for example through terminal

Yes, check out the examples, also the docs!

but from what i seen in the example terminal was that it only talks to the user, Or does it learn without needing .train called?

@tryzombie501 That is correct, the chat bot will learn by speaking to the user. You can prevent the chat bot from learning by setting read_only=True, but by default the chat bot will continuously learn based on how the user responds to it.


[catch2] Fix wrong lib Catch2WithMain dir

Fix https://github.com/microsoft/vcpkg/issues/31774

Note: The function of manual-link  is to solve the conflict problem of the library, and the Catch2WithMain does not conflict with other libraries, so it does not need to be put into the manual-link.

[x] Changes comply with the maintainer guide
[ ] SHA512s are updated for each updated download
[ ] The "supports" clause reflects platforms that may be fixed by this new version
[ ] Any fixed CI baseline entries are removed from that file.
[ ] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

Note: The function of manual-link is to solve the conflict problem of the library, and the Catch2WithMain does not conflict with other libraries, so it does not need to be put into the manual-link.

Wrong. The included main() part is the conflicting one which is why you cannot/shouldn't auto-link it in msbuild.

@Neumann-A So, you mean that we should install the lib Catch2WithMain to ${CMAKE_INSTALL_PREFIX}/lib/manual-link?

So, you mean that we should install the lib Catch2WithMain to ${CMAKE_INSTALL_PREFIX}/lib/manual-link?

Yes. https://github.com/microsoft/vcpkg/issues/31774 only speaks about correcting the generated *.pc file.

Ok, I will fix this patch.

I don't understand the changes. Isn't it enough to do vcpkg_replace_string on the installed pc files in the portfile?

I don't understand the changes. Isn't it enough to do vcpkg_replace_string on the installed pc files in the portfile?

Can you explain in detail vcpkg_replace_string?

#32047
It is concerning if the community needs to explain manual-link and vcpkg_replace_string.

Duplicate of https://github.com/microsoft/vcpkg/pull/32047


stopPropagation on MenuItem doesn't work as expected

Based on this example, I'm trying to set stopPropagation on the event object to true. However, it doesn't seem to work.
Repro:

CodeSandbox
Click "Open menu"
Click "Delete"

Observed behavior: "click proparated to the parent div" appears in the console
Expected behavior: it shouldn't appear
What am I missing here?

Hi @moroshko
The boolean stopPropagation prevents onItemClick on root menu component from firing.
onItemClick is a custom event on Menu component.
If you want to stop the native onClick  event from propagating to ancestor elements, you may call e.syntheticEvent.stopPropagation() method.
Here is an example: https://codesandbox.io/s/react-menu-issue-389-fvyf5

Awesome, thanks a lot @szhsin ❤

@szhsin How would you stop propagation on disabled items?
CodeSandbox

@moroshko sorry for the late reply.
It looks like the native html tags (<button/> for example) automatically call stopPropagation and preventDefault when disabled. Therefore this is indeed a bug.

@szhsin Any workarounds for now, or an idea when this might be fixed?

It should be a quick fix, I will do it today.

Issue fixed in v2.3.4.

Thanks a lot @szhsin !!


BKEX.com is stolen my money

Dear Devs,
I need help.
bkex.com said that they havent received my coins.
I have sent them all ts details. They said no. we dont have them:)
Sender: 3c1c2ed8dee16be3fc1f55d9ac8a5545846904710835bf6db17c90950df88c401fb
Receiver: 111019d09a98f34c115c7a27f95e97f30fb38e3063adcd41057918a50b70b632d2
Amount: 50 UFO
Kernel ID: 21b5f3b5b4ee090043bbd9ef2a648be982ecb9abc058a9230a8c0f73a02caa87
UFO is anonymous coin and it allows them to stole money.
I am ready to send you my wallet.dat and seeds/password/etc.
Help me, please.

Sorry to hear that, but we can do nothing for helping as developers.
You'd better to communicate to BKEX and submit your deposit evidences.
Best wishes.


Esp32s2 support

A WIP PR for esp32s2 support.

Removes the DATE_REG detection in favour of the CHIP_DETECT magic instead.
Adds the esp32s2 module for s2 support.

I can't actually flash anything at the moment, the bootloader returns InvalidMessage:
Command: (10, DATA, 0)
Detected Esp32s2
Command: (13, DATA, 0)
Command: (10, DATA, 0)
Command: (10, DATA, 0)
Command: (9, DATA, 0)
Command: (9, DATA, 0)
Command: (9, DATA, 0)
Command: (9, DATA, 0)
Command: (9, DATA, 0)
Command: (10, DATA, 0)
Command: (10, DATA, 0)
Command: (9, DATA, 0)
Command: (9, DATA, 0)
Command: (13, DATA, 0)
Erase size: 12288
Command: (2, DATA, 0)
Error: bootloader returned an error: InvalidMessage

Any ideas?
Once I figure out whats going on, I think we can probably merge the esp32 & esp32s2 modules by expanding the Chip trait.
Builds on top of #29 & #30

Refer to the following lines in esptool.py:
params = struct.pack('<IIII', erase_size, num_blocks, self.FLASH_WRITE_SIZE, offset)
if isinstance(self, (ESP32S2ROM, ESP32S3BETA2ROM, ESP32S3ROM, ESP32C3ROM, ESP32C6BETAROM, ESP32H2ROM)) and not self.IS_STUB:
     params += struct.pack('<I', 1 if begin_rom_encrypted else 0)

Chips newer than the ESP32 must have an additional field in BeginParams denoting whether or not ROM encryption is being used. The command is being rejected by the bootloader because its length is 4 bytes too short.

added the rom encryption parameter in a very hacky way for now, the flashing looks to go fine now.
It now crashes in the bootloader with an "invalid partition table error"
I (24) boot: ESP-IDF v4.4-dev-1404-gc13afea635-dirty 2nd stage bootloader
I (24) boot: compile time 20:23:13
I (25) boot: chip revision: 0
I (29) boot.esp32s2: SPI Speed      : 80MHz
I (33) boot.esp32s2: SPI Mode       : DIO
I (38) boot.esp32s2: SPI Flash Size : 2MB
I (43) boot: Enabling RNG early entropy source...
E (48) flash_parts: partition 0 invalid - offset 0x10000 size 0x3f0000 exceeds flash chip size 0x200000
E (58) boot: Failed to verify partition table
E (63) boot: load partition table error!


I would like to ask why only the npy file for image was obtained without a mask in the end

I would like to ask why only the npy file for image was obtained without a mask in the end

Sorry. using CodeForMulitpleAnnotations/get_second_mask.py can solve the problem.


I would name the context key workspace.platform not workspacePlatform to make it more consistent with the other keys

Testing #131467
I see that we have camel case keys as well but the majority of the keys seem to take . to build a qualified name

Going to keep as-is due to workspaceFolderCount being camel already, I think it'd be more odd to have workspaceFolderCount//workspace.platform than workspaceFolderCount//workspacePlatform.

But if you look at all the keys we have most of them are '.' qualified names so IMO we should start using those independent whether we have old once that are still camcel cased.


Add support for field.* validation attributes

Hi,
Assuming that we have
<input type="text" name="field[]" />
<input type="text" name="field[]" />
<input type="text" name="field[]" />

We should be able to do one of these in the language file validation.php
'attributes' => [
    'field.*' => 'field label'
]

or
'attributes' => [
    'field' => [
        '*' => 'field label'
    ]
]

Unfortunately both options doesn't work

With this PR merged https://github.com/laravel/framework/pull/12578, you'll be able to apply what you need. You only have to wait until 5.2.23 is released :)


Add missing item components

Add item components that got deleted in the HCF removal but is actually a valid component

Looks good to me, thanks!


Add (linked) load tests for PR 1976 and 4606

Adds a couple of load tests for the XRef parsing part of the code-base, see PR #1976 and #4606.

/botio test

Good, thank you!


Fix typo bug where object "b" is not being checked

The typo caused the calendar to not be updated when an event is deleted from the data source

Thank you !

I just pushed the version 1.0.2 with the fix


SR:Function: Formula for outmap2 is different than description

The descripton for G3 (outmap3) says "In calculations, if field inspection found an outlet, use F40 indicator score". But the formula uses 1 if outlet found.

THE LATTER IS CORRECT. FIXED DESCRIPTION IN COLUMN H.
edited to remove the text in script


Error when Importing

Sample
https://github.com/pnp/powerautomate-samples/tree/main/samples/convert-sharepoint-library-doc-file-to-pdf
Author(s)
@RapidCircle
What happened?
The solution file is invalid. The compressed file must contain the following files at its root: solution.xml, customizations.xml, and [Content_Types].xml. Customization files exported from previous versions of Microsoft Dynamics 365 are not supported.
Steps to reproduce
Open Power Platform
2. Solutions
3. Import
4.Point to file click open
Expected behavior
I expect the solution to import.

My mistake

Yes anoying for us but imagine a normal user to give PnP samples a try ... adoption down the drain.
Refer my idea to use proper naming conventions.


added sloth souls

Please note: This repository is being rebuilt to accept the new volume of token additions and modifications. PR merges will be delayed.

I agree to not ping anybody on Discord/Twitter/email about this pull request. Instead I will inquire by posting a new comment in the pull request if needed.

PRs are reviewed in bulk and and can take up to two weeks to be merged.
This repository is managed using an auto merge action. Please ensure your PR has no deleted lines, and it will be merged.
Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have

Token Address: GoQDVRK1vso9gSGJ2D8hfeGGYnqjarEVqCoFv888guG6
Token Name: Sloth Souls
Token Symbol: SLOTH
Logo: GoQDVRK1vso9gSGJ2D8hfeGGYnqjarEVqCoFv888guG6/logo.png
Link to the official homepage of token: https://www.theslowpatrol.com/
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_): not available

Auto merge requirements
Your pull request will be automatically merged if the following conditions are met:

Your pull request only adds new tokens to the list. Any modification to existing
tokens will require manual review to prevent unwanted modifications.

Your pull request does not touch unrelated code. In particular, reformatting changes to unrelated
code will cause the auto merge to reject your PR.

Any asset files added correspond to the token address you are adding. Asset files
must be PNG, JPG or SVG files.

Your change is valid JSON and conforms to the schema. If your change failed validation,
read the error message carefully and update your PR accordingly.

No other tokens shares the same name, symbol or address.

For example, this change would be rejected due to unrelated changes:

The bot runs every 60 minutes and bulk-merges all open pull requests to prevent conflicts.
This means that you need to wait up to 60 minutes for your pull request to be merged or reprocessed.

will create a new PR without the label


Add urijs Bower dependency to blueprint

@paulstraw this is what I figured was needed :) works for me!


Admin screen: show last n' users and their last interaction times + group mailto: button

Purpose is for a quick n' easy way to notify users of emergency shutdowns.

@kamransaleem


Figure out and document how to hook up components in WP

Polyfills only on old browsers
Can we use <script type="module"> on modern on what?
What happens with webpack and lazy loading etc?

adapted from the stencil documentation and tested in sage:
import { applyPolyfills, defineCustomElements } from 'genero-design-system/loader';

applyPolyfills().then(() => {
  defineCustomElements();
});

Could you add this together with react usage to the readme?


itch: init at 25.5.1

Motivation for this change
Fixes #90233
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

However while itch gui run normally, trying to play a windows game results in an error even with wine installed.
I have posted a bug report here:
https://dale.itch.ovh/iK14K85TvgUh5BKFrGSZLSQx3lsnNGNRP9ItBYQh71awuHG1ebiXJT1LApe9FVQg

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/714

Result of nixpkgs-review pr 157921 run on x86_64-linux 1

2 packages built:

itch
itch-setup

@pasqui23 I've found itch crashes if I attempt to add another install location (Preferences -> Manage install locations -> Add location); the logs don't show anything revealing but I'm going to guess that electron might be missing some dynamically-loaded dependency that is needed for the file picker.

I will try to look into it when possible,meanwhile please open a bug report so that also other cann look into it and keep track of it

I had similar issues (dll import errors) with another game running with wine (v rising server), removing .cache .config .local .steam .wine in the game directory and then starting it again resolved my issues, I will try to narrow it down (probably .wine is enough), I did not find a bug report about that hence my comment here, the link to the bug report on the dale.itch.io website seems broken as well. I assume this is a wine issue but did not find much at all..


[WIP] Run osbuild-composer in separate "target" VM in CI

See the commit message

The code itself seems to be coming along well, but now this requires creating 2x the amount of VMs and could get us deadlocked if we have a LOT of instances running at once and we hit our quota limit. Also, I worry what happens if we're overloaded in PSI and Jenkins is forced to spawn a worker in AWS instead.

The code itself seems to be coming along well, but now this requires creating 2x the amount of VMs and could get us deadlocked if we have a LOT of instances running at once and we hit our quota limit. Also, I worry what happens if we're overloaded in PSI and Jenkins is forced to spawn a worker in AWS instead.

That does not sound great. Do you think running the target VMs inside the OpenStack VMs for stages (e.g. using qemu) would be a better solution than creating a separate OpenStack VMs?

@msehnout That could be possible. That would be a little slower in AWS since we don't have nested virtualization.

This does not look like a promising direction. I'll soon submit alternative version using AWS.


Update the project process page

Add an FAQ
https://github.com/open-austin/open-austin.github.io/issues/105
https://github.com/open-austin/iced-coffee/issues/163

👍 👍 👍

:bee:


Set injection language for doc comments /// to use markdown

Later, this fact should also be used for LSP-guided highlighting of code comments.
Also, consider getting rid of @see and @since tags in doc comments and instead use pure markdown everywhere. That is, to go from this:
/// ...
/// @see https://docs.tact-lang.org
/// @since Tact 1.X.X

To this:
/// ...
/// * See https://docs.tact-lang.org
/// * Since Tact 1.X.X

Or this:
/// ...
/// See:
/// * https://docs.tact-lang.org
/// * https://docs.tact-lang.org/book/exit-codes
///
/// Available since Tact 1.X.X

P.S.: The info on availability can be moved to the top.
P.P.S.: Of course, this should not be detrimental to the parsing speed.

After researching this for a while, I think that it's not practical and barely feasible (only in Neovim) to do so purely in Tree-sitter injection queries.
That's because only Neovim has #offset! directive, and even with it there would be issues with indentation or blank lines in-between sequences of /// comments. And with all that hassle, only Neovim users would get this enhancement.
Therefore, it only makes sense to alter the highlighting of code comments via the LSP. That said, this issue should be closed.
P.S.: Also, the following would probably become the preferred style for documentation comments to move away from any @ tags in them and use markdown all the way through:
/// ...
/// See:
/// * https://docs.tact-lang.org
/// * https://docs.tact-lang.org/book/exit-codes
///
/// Available since Tact 1.X.X


Combo bug4

does open-cluster-management/backlog#13002
does open-cluster-management/backlog#12231

/lgtm


optimize string decoding

Optimized the decoding process for string. Reduced memory copy overhead by reading contiguous memory. Use branch-less method to determine the length of utf-8 strings.
Since a standard benchmark is missing, I can't determine the exact performance improvement.
However, from what I've seen in practice, it has reduced the string parsing overhead significantly.

Some features of c++17 are used. Maybe we need to update the requested compiler version in README.


GlowBrush Effect Layout Bug

<controls:MetroWindow
x:Class="MainWindow"
...
BorderBrush="{DynamicResource AccentColorBrush}"
BorderThickness="1"
GlowBrush="{DynamicResource AccentColorBrush}"
...
I use Windows 10, .Net 4.5, MahApps.Metro <IP_ADDRESS>, and Wpf Toolkit with AvalonDock <IP_ADDRESS>.
As we can see in the attachment, if the Window is moved then there is some strange outline (border is moving about 1 pixel and makes background is drawn between border and Window). In this attachment, the 1 pixel gab is above Window.

@yoyokits you should test it with the latest alpha, cause this should be fixed.


API router does not respect WAGTAIL_APPEND_SLASH setting

Issue Summary
The API router will append slashes whether the WAGTAIL_APPEND_SLASH is set to True or False. The behaviour is hardcoded in wagtail.api.v2.router.py.
Changing the behaviour in the router might qualify as a breaking change, if the slash is just removed in case WAGTAIL_APPEND_SLASH is set to False. Right now clients are redirected from endpoint_without_slash to endpoint_with_slash with WAGTAIL_APPEND_SLASH, but that won’t happen automatically the other way around.
DRF has the trailing_slash argument for its router implementations so this might be a compromise to make the behaviour configurable while maintaining backwards compatibility until a breaking change is possible in the next (major?) version.
I’ll gladly submit a PR for a fix once we settled for a solution.
Thank you for your time!
Steps to Reproduce

Follow the instructions in the documentation to setup the API
Set WAGTAIL_APPEND_SLASH to False in settings
Start the dev server
Goto /api/v2/pages (which is immediately redirected to /api/v2/pages/.

I have confirmed that this issue can be reproduced as described on a fresh Wagtail project: yes

Technical details

Python version: 3.8.2
Django version: 3.0.5
Wagtail version: 2.8.1
Browser version: Chromium 81

Unfortunately it’s not only the router that generates URLs with trailing slashes. When overriding the get_urlpatterns method of the Wagtail router I noticed that detail URLs will contain the object id as part of the basename so they started looking like /api/images4/. This is because the get_urlpatterns method of the BaseAPIViewSet class also hardcodes slashes at fixed positions.

Though unrelated to the REST API router: another documented interface that does not respect the WAGTAIL_APPEND_SLASH setting is the RoutablePageMixin. Both the get_resolver and route functions are based on the assumption that the slash is appended to URLs and ignore the WAGTAIL_APPEND_SLASH setting.

I would really like to be able to configure slashes as well. Commenting for traction

Hey @kmohrf, @shipskin,
If you have the time, a PR would be highly appreciated and help us push this forward.

created a PR https://github.com/wagtail/wagtail/pull/7921

Unless there is a good reason for it, I'm against us making this change.

WAGTAIL_APPEND_SLASH is only supposed to apply to page URLs as these are what appear in a users browser and may be hand typed.
It doubles the amount of URLs we might have to cache and invalidate from a cache
It makes Wagtail implementations slightly different from one another making it more difficult to write a single client library that works across many Wagtail sites

This was separately proposed and rejected in #7421 - sorry for not catching this earlier.


Flash Messages

There is no message when Registering for an Event.

Steps for solving=>

Create a global storage mechanism to store a flash message.


Updated Basic examples to rn 0.36.0 + other misc. changes

I've updated the dependencies to latest stable versions (react 15.3.2 & react-native 0.36.0), and while I was at it thought I'd make some changes to the examples themselves.
Nothing too complex, just some switches/segments to change blurType & hide/show the components.
Also used a new image for the background (courtesy of unsplash) - as previous one pixelated on large displays.

Thoughts?

Soooo niiiiice ❤


root as lca?

discussion point: do we want to report root as the lowest common ancestor with unipept commands like pept2lca or taxa2lca

records that are not mentioned on stdout are records that have no matches in the Unipept database (pept2lca)
records that are reported with root as their lowest common ancestor, are records that have representatives in at least two of the kingdoms: Archaea, Bacteria, Eukaryotes

Original issue by @pdawyndt on Fri May 02 2014 at 20:14.
Closed by @pdawyndt on Thu May 08 2014 at 14:03.

According to my opinion, it does make sense to report root as the lowest common ancestor (LCA). Only I would call the root node organism instead of root if the biodiversity taxonomy is used.
Original comment by @pdawyndt on Sat May 03 2014 at 10:16.


Possible to Dim/Brighten

is it possible to somehow dim/brighten a light with this?  Ultimately, I'd like to work like it does regularly with the light I am controlling with this (Meross wifi light).

You could do this using automations, but it would be a bit of work...
The way homebridge works, this provides a device to HomeKit, but cannot control other devices in HomeKit.


Way to change prioritisation

Hello, I'm having the following issue with a local-geocode search. My query is how to go about the following.
Input:
Toledo, Spain
Expected Output:
Toledo, Spain [documented on geonames here here]
Actual Output:
Toledo, Ohio [documented on geonames here here]
I am expecting that this is something to do with the prioritisation in that both Toledo's were discovered, but that the Ohio one is being prioritised due to a larger population? If so, would be great to have a mechanism to fallback to some other type of prioritisation, or even to filter on the country by a parameter in the decode function. Thanks!

Hi - right yeah, I was often surprised by some results. Let's take a look at this specific case to see why this is.
Let's first pull in the raw data. Please note that if you do this it will pull several GBs of data from geonames, extract it and load it all into a pandas dataframe. Be sure to have enough RAM/disk space. Maybe best to run it on a Google Colab.
from geocode.geocode import Geocode

gc = Geocode()
df = gc.get_geonames_data()

Now let's look for all places in the US and Spain with at least 30k population (which is the default threshold)
>>> df[(df.name == 'Toledo') & (df.country_code.isin(['US', 'ES'])) & (df.population > 30_000)]
         geoname_id    name                                     alternatenames  latitude  longitude feature_code country_code  population
3029006     2510409  Toledo  Taleda,Toledas,Tolede,Toledo,Toledo i Spania,T...  39.85810   -4.02263         PPLA           ES       84282
3076342     6361828  Toledo                                              45168  39.86765   -4.00988         ADM3           ES       84019
11015695    5174035  Toledo  Fort Industry,Port Lawrence,TOL,Talida,Toledo,...  41.66394  -83.55521        PPLA2           US      279789

From this I can see that Toledo, Ohio has actually a huge population of 279k, whereas Toledo, Spain has 84k.
Unfortunately, at this time the Spanish Toledo gets "swallowed" by the US one because of the population difference, and the library only outputs a single entry for "Toledo". I think the reason I decided to do this was because of the huge number of "collisions". There's a surprisingly large number of Toledos in the world (e.g. apparently there's an admin area Toledo in Brazil with 142k inhabitants):
         geoname_id    name                                     alternatenames  latitude  longitude feature_code country_code  population
231525      3834251  Toledo                                             Toledo -31.55378  -64.00742          PPL           AR        3046
810719      3446370  Toledo                                  TOW,Toledo,Толедо -24.71361  -53.74306          PPL           BR      119313
857867      6321886  Toledo                                                NaN -22.70569  -46.39041         ADM2           BR        5761
859000      6323019  Toledo                                                NaN -24.73518  -53.82059         ADM2           BR      142645
2442867     3666959  Toledo                                             Toledo   7.30984  -72.48295        PPLA2           CO        5911
2442869     3666961  Toledo                         Municipio de Toledo,Toledo   7.23217  -72.31127         ADM2           CO       17272
2442870     3666962  Toledo                         Municipio de Toledo,Toledo   7.02349  -75.71487         ADM2           CO        5697
3029006     2510409  Toledo  Taleda,Toledas,Tolede,Toledo,Toledo i Spania,T...  39.85810   -4.02263         PPLA           ES       84282
3076342     6361828  Toledo                                              45168  39.86765   -4.00988         ADM3           ES       84019
6459331     3981414  Toledo                                             Toledo  24.80061 -104.46510          PPL           MX         247
6605347     8885405  Toledo                                             Toledo  18.92681  -98.37497          PPL           MX          73
6621485     8901548  Toledo                                             Toledo  22.94361 -101.32222          PPL           MX          25
6659939     8940027  Toledo                                             Toledo  16.73667  -93.21861          PPL           MX           5
8130440     1681602  Toledo  Ciudad ti Toledo,Dakbayan sa Talisay,Dakbayan ...  10.37730  123.63860          PPL           PH      207314
8545351     3372516  Toledo                                             Toledo  38.70000  -28.15000          PPL           PT          60
10105657    4251330  Toledo          Majority Point,Prairie City,Toledo,Толедо  39.27365  -88.24365        PPLA2           US        1221
10721872    4878703  Toledo  Toledo,Tolehdo,Tolido,twldw  aywwa,twlydw,Толе...  41.99555  -92.57686        PPLA2           US        2202
11015695    5174035  Toledo  Fort Industry,Port Lawrence,TOL,Talida,Toledo,...  41.66394  -83.55521        PPLA2           US      279789
11596146    5757007  Toledo                                      Toledo,Толедо  44.62151 -123.93845          PPL           US        3511
11652493    5813681  Toledo  Cowlitz Landing,Plomondon's Landing,TDO,Toledo...  46.43983 -122.84678          PPL           US         727
12141864    3439838  Toledo                                             Toledo -34.73807  -56.09469          PPL           UY        4397

so if I wanted to show some Toledos and not others, there would need to be some sort of general rule of which ones to include and in which order. Let me know if you have any suggestions to overcome this issue.

Makes sense and thank you for the thorough explanation.
I think there's two possible solutions here.
Kwargs for decode that can be used for filtering
I'd imagine having these on the decode function so it's defined something like
def decode(self, input_text, feature_code_in=[], country_code_in=[], population_between=(0, max)):
matches = self.kp.extract_keywords(input_text)
# Either doing this in the extract_keywords call or here:
if feature_code_in:
    matches[
        (matches.feature_code.isin(feature_code_in))
    ]
if country_code_in:
    matches[
        (matches.country_code.isin(country_code_in))
    ]
...

Any thoughts on this @mar-muel?

Hi - The prioritiziation/filtering cannot be changed at docode time. This is because both prioritization and filtering affect the pickle files.
These arguments would have to be passed to the __init__ at runtime and then when running gc.load() the first time it would trigger recomputation of the pickle files.
Currently we have these 3 arguments:
min_population_cutoff=30000, large_city_population_cutoff=200000, location_types=None

Happy to add country_code_in (or similar) and a way to provide priorities. Though not sure about changing priorities globally/arbitrarily. Currently this is how priorities are assigned:
        # Priorities
        # 1) Large cities (population size > large_city_population_cutoff)
        # 2) States/provinces (admin_level == 1)
        # 3) Countries (admin_level = 0)
        # 4) Places
        # 5) counties (admin_level > 1)
        # 6) continents
        # 7) regions
        # (within each group we will sort according to population size)
        # Assigning priorities
        df['priority'] = np.nan
        df.loc[(df.feature_code == 'RGN'), 'priority'] = 7
        df.loc[(df.feature_code == 'CONT'), 'priority'] = 6
        df.loc[(df.feature_code_class == 'A') & (df.admin_level > 1), 'priority'] = 5
        df.loc[df.feature_code_class == 'P', 'priority'] = 4
        df.loc[(df.feature_code_class == 'A') & (df.admin_level == 0), 'priority'] = 3
        df.loc[(df.feature_code_class == 'A') & (df.admin_level == 1), 'priority'] = 2
        df.loc[(df.population > self.large_city_population_cutoff) & (df.feature_code_class == 'P') & (~df.is_altname), 'priority'] = 1

That makes sense @mar-muel, think the country_code_in is a suitable solution, since it may just be that a user is interested in only North American countries

Created a PR for this now


Generic Type Inference Not Bounded Properly

TypeScript Version:  2.5.3
Code
interface Cls<T = any> {
  new(...args: any[]): T;
  __id: string;
}

class Entity {
  static from<T>(this: Cls<T>, data: Partial<T>): T {
    return new this();
  }

  duration: number;
}

class Service {
  save<T>(cls: Cls<T>, obj: T): T {
    return obj;
  }
}

let res = new Service().save(Entity, Entity.from({ duration: 5 }))

Expected behavior:
res should be a type of Entity
Actual behavior:
res is of type { "duration": number }
If I remove __id: string; from the Cls interface, the type resolution properly resolve to Entity

Note in your interface __id: string is required in Entity. Adding a static property solves the problem.

I'm trying to get a better minimal scenario.  In my local code I extend the Function typing to include __id as I add it to the class via transformer.
Realistically, if Entity isn't a valid Cls, the test should fail.
Will update the code in a bit.

Automatically closing this issue for housekeeping purposes. The issue labels indicate that it is unactionable at the moment or has already been addressed.


Integration of Fraunhofer SAMG as an alternative to PyAMG

We integrated Fraunhofer SAMG as an alternative to PyAMG as the preconditioner used in LOBPCG.
We tested this on the swiss roll as an artificial dataset and on the data from word2vec used in previous benchmarks of megaman. In both cases using SAMG proved to be significantly faster than using PyAMG. For example in our tests using SAMG on the data from word2vec we were able to achieve a speedup of the embedding process by a factor between 2 and 6.

Note though that Fraunhofer SAMG is a commercial software and one needs a license to use it. For more information on SAMG see here and for licensing (including test or educational licenses) contact<EMAIL_ADDRESS>

How do you install PySAMG? I looked it up and couldn't find it anywhere. Thanks!

How do you install PySAMG? I looked it up and couldn't find it anywhere. Thanks!

Do you mean SAMG or PyAMG? PyAMG can be found here https://github.com/pyamg/pyamg for SAMG, please contact the Fraunhofer team as mentioned above.

How do you install PySAMG? I looked it up and couldn't find it anywhere. Thanks!

Do you mean SAMG or PyAMG? PyAMG can be found here https://github.com/pyamg/pyamg for SAMG, please contact the Fraunhofer team as mentioned above.

No, I mean PySAMG (line 29 of utils/eigendecomp.py). I have access to an SAMG license, which was acquired by my group a long time ago, but I can only call it through C++ or FORTRAN. I was wondering if PySAMG was a development of @jrsassen or if Fraunhofer SCAI had released a Python package.

I developed PySAMG (a Python interface for SAMG) during my time at Fraunhofer SCAI. It was never publicly released as SAMG itself is not publicly available, but (I hope) if you contact the SAMG team they should still have the code.

Thanks! I contacted them.


Relax http response parsing?

I'm currently trying to replace a libsoup based http client implementation with boost beast.
However,  beast's  http response parser is (deliberately) very strict when it comes to RFC compliance.
For example,  (in contrast to libsoup) it only accepts:
status-line = HTTP-version SP status-code SP reason-phrase CRLF
Unfortunately, various "real world" servers  don't strictly play by the rules and e.g. return status lines that are only terminated by a LF.  While it would theoretically be the best to fix these servers, in practice this is often not really possible.
I realize that this (or a similar) topic has already been discussed in the past:

https://github.com/boostorg/beast/issues/1761
https://github.com/boostorg/beast/issues/1138
https://github.com/boostorg/beast/issues/2344

And of course, one could always just patch the parser code as needed.  Nevertheless, I'd like to know:

Are there any plans to provide a more relaxed way of parsing? E.g. maybe via some opt-in basic_parser::relaxed()  function? Or would something like this be strictly against the spirit of the library?
If adjusting parsing is not an option, what would be the best/easiest way to preprocess/filter incoming data first?  To me, the approach outlined in the icy_stream.hpp example seems overly complicated but so far I haven't found a simpler/more obvious approach.

No, and the icy_stream example is the way.

I see. That's unfortunate and makes switching to beast more complex.
Out of curiosity: is there a particular (technical) reason for not following the suggestions from RFC-2616/section-19.3:

Clients SHOULD be tolerant in parsing the Status-Line and servers
tolerant when parsing the Request-Line. In particular, they SHOULD
accept any amount of SP or HT characters between fields, even though
only a single SP is required.

The line terminator for message-header fields is the sequence CRLF.
However, we recommend that applications, when parsing such headers,
recognize a single LF as a line terminator and ignore the leading CR.

Out of curiosity: is there a particular (technical) reason for not following the suggestions from RFC-2616/section-19.3:

Yes. That guidance was changed in rfc7230 which obsoletes rfc2616. Implementations still "may" be tolerant but it is noted that the practice exposes programs to vulnerabilities (see https://www.ietf.org/rfc/rfc7230.html#section-9.5).
Also note that rfc9110 and rfc9112 obsolete rfc7230, but these have not been exhaustively applied to Beast. I'm willing to consider the possibility of allowing more tolerant parsing if the newer RFCs say something on the matter.

Interesting, I didn't know that :+1:  I don't quite understand how rfc7230/section-9.5 (and 3.3.3) invalidates the more lenient approach to start-line and header field parsing mentioned in both rfc7230 (section-3.5) and rfc9112 (section-2.2), though.
Anyhow, thanks for your quick response. I will then stick to patching the parser as I failed to get the icy_stream approach working with a boost::beast::ssl_stream in a reasonable amount of time.

@klemens-morgenstern if you want to have a go at putting together a branch that has the more lenient parsing, I'd be open to it. We don't have any parse_options yet (a-la URL or JSON) but we could in theory add it.

Are you sure that's a good idea? That sounds to me like something that's more an area that http-proto would be customized on.

The reported issue correctly notes that clients may use relaxed parsing. This basically means we have to implement it if we want full compatibility.


How to use mixins with Arg group?

Currently if i try to use @Mixins with @Arg groups, i get the error:@Mixins are not supported on @ArgGroups

Can you give more details on what you’re trying to achieve? Maybe with some example code?

Just as i asked this question i found out that i can use nested Arg groups.So this issue can be closed.

In my case, I have  options which are used in multiple commands.In some commands they are used independently and in other cases they are used mutually exclusive with other options.

Glad to hear that you’ve been able to resolve the issue!
Enjoy picocli!


converting a few info into debug message

Changes
A taskRun/pipelineRun is watched as soon as created. Until the run is complete and ready to proceed further, the chains controller is logging that the taskRun/pipelineRun is still running at the info level. This looks good but generates a lot of info in the controller logs and exponentially increases with the number of taskRuns and pipelineRuns. Converting such message to be reported at the debug log level.
Formatting a few error message such that the namespace/name is enclosed with parenthesis so that its easy to read.

Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[ ] Has Docs included if any changes are user facing
[x] Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[ ] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
NONE

/retest

@pritidesai are you still interested in driving this PR forward?


[debug] Console triggers bad/blurry Chrome antialising

Open attached file and zoom in a lot
Might be reproducible only on Windows
Input from @alexandrudima :

check why are you getting a new gpu layer
we already had this issue in other trees and Joao discovered it was happening due to scroll bars
another reason for this might be the positioning (non integer coordinates) of the repl or the background. Though this one is unlikely

:zap:

Probably will not fix since we will move the repl to use the editor not the tree.


Truncate synonyms on the disease profile page

For some diseases and phenotypes, they have synonyms that are very long, causing page layout issues on smaller screens.
For example, for congenital anomalies of kidney and urinary tract syndrome with or without hearing loss, abnormal ears, or developmental delay, one of the synonyms is congenital anomalies of kidney and urinary tract syndrome with or without hearing loss, abnormal ears, or developmental delay; CAKUTHED.

On screens smaller than approx 1770px wide, the length of the text in the pill causes overflow issues and the horizontal scrolling becomes active as the page width exceeds the browser window.
As a temporary fix, can we please truncate the text in the synonym pill to the first 75 characters, followed by an ellipsis? And on-hover, can we please display the full text?
So for example, the above-mentioned synonym would become the following:
congenital anomalies of kidney and urinary tract syndrome with or without h…

The synonyms use the same component (header) as the page header - see issue #1069


[Bug]: 404 page on opening apps

What happened?
I get a 404 page whenever I open some apps
To reproduce the behavior?
fix the store
What browser are you seeing the problem on?
Chrome

Hey buhron, I am unable to reproduce the stated issue could you please share something like a screenrecording.

Yash I get this error too. This error is caused by you trying to open some apps as websites and as you said, it might say that the websites don't let their site get displayed on another site. I'm not able to take a screenshot of this issue because for some reason when I hover my mouse over the capture button on chromebook the error disappears. But the bassically the site refuses to connect, if you can I recommend using this site "https://minecraftforfreex.com/eaglercraft/"


Adding plugins to php version

Is there a way to target the php version when adding plugins?

Of course you can. You can check out more at here.
docker build --build-arg \
    plugins=filemanager,git,linode \
    github.com/abiosoft/caddy-docker.git#:php


Doc update: Example tailwind.config.js and Dioxus.toml

Hi
Thanks for all the hard work with this. I am loveing this DaisyUi integrations just a couple of minor things to help people along:
While not an ideal regex and something borrowed from houseabsolute I have crafted a script that searches all of the rust files.
So my tailwind.config.js looks like (could probably do away with the index.html):
/** @type {import('tailwindcss').Config} */
module.exports = {
    content: {
        files: ["index.html", "**/*.rs"],
        extract: {
            rs: (content) => {
                const rs_to_tw = (rs) => {
                    return rs
                        .replaceAll("(", ":")
                        .replaceAll(")", "")
                        .replaceAll("_", "-");
                };
                let one_class_re = "[a-z0-9_]+(\\([a-z0-9_]+\\))*";
                let class_re = new RegExp(one_class_re , "g");

                let classes = [];
                classes.push(
                    ...[...content.matchAll(class_re)].map((m) => {
                    return rs_to_tw(m[0]);
                  })
                );
                if (classes.length > 0) {
                    console.log(classes);
                }
                return classes;
            },
        },
    },
    theme: {
      extend: {},
    },
    plugins: [
      require("daisyui")
    ],
  }

I also manually installed tailwindcss into the dioxus-cli directories ~/.local/shared/dioxus/tools/tailwindcss/tailwindcss (due to it not fully supporting the tools directory yet) and updated my Dioxus.toml to include this:
[application.tools]
tailwindcss = {input = "src/tailwind/tailwind_input.css", config="src/tailwind/tailwind.config.js" }

and removed the style setting.
This means apart from having to install daisyui via npm (which would be nice to avoid), dioxus takes care of all of the heavy lifting and generates a css file which only contains classes I use. It looks like during a release it also does a minify on the css (extra bonus).
Let me know if you need more details for a readme.

Thanks a lot for the comment.
Last time I checked dioxus-cli, the tool subcommand was commented out in the source code, so I didn't dig into the tailwindcss tool setting, if this works, it's pretty neat.
Thanks for the sample tailwind.config.js, there are some corner cases that current regex not covering.

for the 0.5, 1.5, 2.5, 3.5 part in CSS, I define the constants as _half, 1_half, 2_half, 3_half for a clearer meaning.
for 1/2, 1/3 ..., I define as 1__2, 1__3 use double underscore to make it different from normal -
for the 2xl: modifier, since 2xl is not a valid rust function name, I declare both _2xl and xxl as a hack (also add a max_xxl).
also, for arbitrary values, I'm also use functions, e.g. w_any, h_any (using any! macro)

(will try to add these special cases into documents later)
This config will still be very helpful for development, but might cause some issues for production usage, not sure about how to cover these corner cases with it. I did checked the tailwindcss source code, seems the core logic been rewritten in rust already, maybe it's possible to do some tweak there for a much cleaner fix.
The static crate way is a bit more tedious, but more flexible, it's quite nice for some sites actually need a ssr output, but quite clumsy if not. I did think a bit for a better solution, not found one yet.
Anyway, thank you for trying this and giving feedback. If you have time, any improvements are welcomed, feel free to send pull request.
I'll try to setup the dioxus tools and do some experiments when got some time later.

About the arbitrary value, here is an example from https://tailwindcss.com/docs/width#arbitrary-values
class: class!(w_any("32rem"))

will be converted to
class="w-[32rem]"

Just update the crates with Dioxus 0.5.0, also using a new way to generate class, all class! macro calls are gathered by using side effects in proc_macro, I think it's much better than a text base approach.
Check BUILD.md for more details


API v3 Server 500 Errors

These urls responded with server 500 errors:
Error /api/v3/packages/?limit=20&offset=980
Error /api/v3/packages/?limit=20&offset=1600
Error /api/v3/packages/?limit=20&offset=1680
Error /api/v3/packages/?limit=20&offset=2320
Error /api/v3/packages/?limit=20&offset=2380

Just tested all 5 of these limits/offsets today and received data back for each response without error.

Works for me. Closing.
Please reopen if this issue remains.

They've changed:
In [10]: for offset in offsets:
    resp = requests.get(url + offset)
    print url + offset, resp.status_code
   ....:
https://djangopackages.org/api/v3/packages/?limit=20&offset=1000 500
https://djangopackages.org/api/v3/packages/?limit=20&offset=1620 500
https://djangopackages.org/api/v3/packages/?limit=20&offset=1700 500
https://djangopackages.org/api/v3/packages/?limit=20&offset=2340 500
https://djangopackages.org/api/v3/packages/?limit=20&offset=2400 500

In [11]: print offsets
['1000', '1620', '1700', '2340', '2400']


added steps for cloning or extracting project when using oc CLI. Fixe…

…s RHOARDOC-23

merged: b19ed8c6c22c6df38a6f16ae34c1259cd4ba58cd


reverted uri encoding

Reverted https://github.com/ParsePlatform/parse-server/pull/148

Sounds good to me, having replicasets working is probably more important than passwords with an '@' in them as its easier to change your password than your replicaset architecture :)