unable to build "latest" version on linux

Hi
I downloaded the new stable tar.gz file and attempted to run the build.sh script but it failed.
 ./scripts/build.sh
created virtual environment CPython3.10.8.final.0-64 in 224ms
  creator CPython3Posix(dest=/tmp/nvim-ui-stable/venv, clear=False, no_vcs_ignore=False, global=False)
  seeder FromAppData(download=False, pip=bundle, setuptools=bundle, wheel=bundle, via=copy, app_data_dir=/home/dpriedel/.local/share/virtualenv)
    added seed packages: pip==22.3, setuptools==65.5.0, wheel==0.37.1
  activators BashActivator,CShellActivator,FishActivator,NushellActivator,PowerShellActivator,PythonActivator
Cloning into 'gir2cpp'...
remote: Enumerating objects: 532, done.
remote: Counting objects: 100% (38/38), done.
remote: Compressing objects: 100% (31/31), done.
remote: Total 532 (delta 12), reused 24 (delta 7), pack-reused 494
Receiving objects: 100% (532/532), 69.12 KiB | 1.82 MiB/s, done.
Resolving deltas: 100% (358/358), done.
Note: switching to 'ac4a2770cdb03b2c41cb0683a5ace5fc56cdfc4d'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by switching back to a branch.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -c with the switch command. Example:

  git switch -c <new-branch-name>

Or undo this operation with:

  git switch -

Turn off this advice by setting config variable advice.detachedHead to false

Collecting Jinja2
  Using cached Jinja2-3.1.2-py3-none-any.whl (133 kB)
Collecting MarkupSafe>=2.0
  Using cached MarkupSafe-2.1.1-cp310-cp310-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (25 kB)
Installing collected packages: MarkupSafe, Jinja2
Successfully installed Jinja2-3.1.2 MarkupSafe-2.1.1
Unhandled {http://www.gtk.org/introspection/core/1.0}union
The Meson build system
Version: 0.63.3
Source dir: /tmp/nvim-ui-stable
Build dir: /tmp/nvim-ui-stable/BUILD
Build type: native build
Project name: nvim-ui
Project version: 0.1.0
C compiler for the host machine: cc (gcc 12.2.0 "cc (GCC) 12.2.0")
C linker for the host machine: cc ld.bfd 2.39.0
C++ compiler for the host machine: c++ (gcc 12.2.0 "c++ (GCC) 12.2.0")
C++ linker for the host machine: c++ ld.bfd 2.39.0
Host machine cpu family: x86_64
Host machine cpu: x86_64
Found pkg-config: /usr/bin/pkg-config (1.8.0)
Run-time dependency msgpack found: YES 4.0.0
Run-time dependency libuv found: YES 1.44.2
Run-time dependency spdlog found: YES 1.10.0
Run-time dependency fmt found: YES 9.1.0
Run-time dependency gtk4 found: YES 4.8.2

meson.build:22:0: ERROR: Subproject exists but has no CMakeLists.txt file

I remember removing a subproject some time ago. Could you please try cloning the repository and building from scratch?

That did the trick.
The archive seems to be missing the msgpack-c sub-project.
A few questions:

the window content is very washed-out looking compared to other
applications on my system.  Is there some setting which controls that?

the page-up, page-down, home, end, etc keys do not work. Is there some
way to enable them?

There are a lot of logging 'info' messages output. Is there a way to
turn that off?

Thanks !!
On 11/3/22 10:05 AM, Anatolii Sakhnik wrote:

I remember removing a subproject some time ago. Could you please try
cloning the repository and building from scratch?
—
Reply to this email directly, view it on GitHub
https://github.com/sakhnik/nvim-ui/issues/53#issuecomment-1302165537,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABUOHSQ65SW6VI5UP5JI5HLWGPBCLANCNFSM6AAAAAARWFE4LA.
You are receiving this because you authored the thread.Message ID:
@.***>

the window content is very washed-out looking compared to other
applications on my system.  Is there some setting which controls that?

Could you please illustrate the "washed-out" appearance? You could also toggle the Gtk inspector (Help -> Inspector) to tinker with the CSS styles. But I suspect that's the colour scheme used in Neovim that matters.

the page-up, page-down, home, end, etc keys do not work. Is there some
way to enable them?

Absolutely! Unfortunately, I only have a Chromebook keyboard at the moment. But could you please try adding some cases to the switch statement in the source code here: https://github.com/sakhnik/nvim-ui/blob/4f40c48528afd6609cc8bdca5b8ab825e91e3864/src/GGrid.cpp#L267 ? Please don't hesitate to send a pull request.

There are a lot of logging 'info' messages output. Is there a way to
turn that off?

Sure. You could control the log level with an environment variable: SPDLOG_LEVEL=off ./nvim-ui
Reference: https://github.com/sakhnik/nvim-ui/blob/4f40c48528afd6609cc8bdca5b8ab825e91e3864/src/main.cpp#L48

Attached is a screenshot showing the desktop screen from AstroNVim.
On the left is nvim-qt
On the right is nvim-ui
Both are running from the same neovim configuration.
NOTE: in the nvim-ui window on the right the first line of the desktop
menu is offset to the left.
I hope this is useful.
On 11/4/22 6:05 AM, Anatolii Sakhnik wrote:

the window content is very washed-out looking compared to other
applications on my system. Is there some setting which controls that?

Could you please illustrate the "washed-out" appearance? You could
also toggle the Gtk inspector (Help -> Inspector) to tinker with the
CSS styles. But I suspect that's the colour scheme used in Neovim that
matters.
the page-up, page-down, home, end, etc keys do not work. Is there some
way to enable them?

Absolutely! Unfortunately, I only have a Chromebook keyboard at the
moment. But could you please try adding some cases to the switch
statement in the source code here:
https://github.com/sakhnik/nvim-ui/blob/4f40c48528afd6609cc8bdca5b8ab825e91e3864/src/GGrid.cpp#L267
? Please don't hesitate to send a pull request.
There are a lot of logging 'info' messages output. Is there a way to
turn that off?

Sure. You could control the log level with an environment variable:
|SPDLOG_LEVEL=off ./nvim-ui|
Reference:
https://github.com/sakhnik/nvim-ui/blob/4f40c48528afd6609cc8bdca5b8ab825e91e3864/src/main.cpp#L48
—
Reply to this email directly, view it on GitHub
https://github.com/sakhnik/nvim-ui/issues/53#issuecomment-1303205292,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABUOHSXPWOA7LHHLUCVFU63WGTNYNANCNFSM6AAAAAARWFE4LA.
You are receiving this because you authored the thread.Message ID:
@.***>

Any chance to take a peek at the screenshot?

the window on the left is nvim-qt, on the right is nvim-ui
both running from the same neovim configuration

Thanks! The offset issue was a known one, but I didn't know how to fix it until today. I'll have to investigate the colours more carefully because I haven't noticed it yet.

Well, the colours are fine in my system. So I'd suggest checking the global Gtk4 theme. There may be some CSS influencing nvim-ui.
But I see the layout is still messed up in a different way this time.


fault detection and heater state - sounds like stuff is complicated enough to deserve a unit test

%subj%

is c2cb3b9015797042fe3ad444863e1ea8f469ce5d what you were thinking of?

@mck1117 this is very very cool!


Cover points for the riscv priv arch tests

This PR is the continuation of the closed PR of 1danish13 which can be seen here. It contains all the riscv priv arch test commits added by 1danish13 and all future commits of RISC-V privilege arch test will be added here.

@neelgala @allenjbaum @pawks kindly add suitable reviewers for this PR.


fix: iOS restartIce on 'failed' iceConnectionState

Description
On iOS devices it is possible for the iceconnectionstate state to go from checking straight to failed without a disconnected state in between.
We should restartIce in this scenario.
Fixes: #232

this looks good, i actually seen this my self


Is there a way to have the slider button start in the middle and have the fill color also start from the middle?

Ask your Question
See title. I would like to create something like this design. Is it possible with the current features offered?

Hello @itsgeraldwang, thanks for reaching out!
There is the trackImage feature which would allow you to replace the track with the image you would have to prepare according to your design.
Another option, more complicated though, would be to use StepMarker:
Prepare the step marker transparent for each step, with their layout being the height of the track's width,
calculate the difference between the middle step and the thumbs position (by using the value prop of the thumb and the StepMarker),
change the style of those steps that are in between the middle and thumb's position, so that the desired gradient is rendered.
Give it a try, or let me know if you have some more questions/issues.
Also, if you'll have some findings or conclusions, I'm happy to hear.

Thanks for the response, I'll try playing around with that and get back to you if it works or if I have any other questions.

I was able to get it to work by overlaying a track over the slider itself and adjusting the width of the overlayed track using the value prop from the slider. One more question though:
Is there any way to adjust the styling of the slider button such as size and shape? I see there is a thumb image prop but I would rather style the button through native styling instead of using an image.

Nvm, read through some of the pending issues and looks like button styling isn't a feature yet. I'll mark this as resolved since the original question has been answered, but +1 to having a way to style the thumb!


don't use global location, because it doesn't work on cloudflare workers

In a couple of places there are references to location, which on a browser reads the location from window.location. In an SSR environment, this reads from global.location which isn't a problem on node.js, because it will just be undefined, and isn't used when doing SSR anyway. However, on Cloudflare Workers there isn't any global object at all, so the code trying to read from location tries to read a global variable named location, which doesn't exist, so the code crashes due to an undefined variable. It is possible to enable a special Node.js compatibility mode, so the global will exist and location will be undefined rather than non-existent, but this compatibility mode gives the warning:
Enabling Node.js compatibility mode for built-ins and globals. This is experimental and has serious tradeoffs. Please see https://github.com/ionic-team/rollup-plugin-node-polyfills/ for more details.

So far I have been using the compatibility mode to do SSR rendering, but it would be nice to be able to turn it off! This PR adds checks with typeof location!="undefined" before trying to read the variables so it works without combatibility mode.
Also, I had problems when I was doing "multipass SSR". By "multipass SSR" I mean that I was doing external fetching with promises in the SSR, then I waited for the promises to resolve, and did SSR again. When doing this, the ssrPath parameter was not respected for the second pass. So I also changed the semantics slightly for the ssrPath parameter. Now, if the ssrPath parameter is set, it will always be used.

@limikael Thank you for reporting this.
I wonder why this problem happens, because in Node.js location is also undefined (even though it can be accessed globally). For example, the function const currentSearch = () => location.search; wouldn't work in node either, it would throw Cannot read properties of undefined .
Normally, when ssrPath is present the code that refers to location isn't invoked at all. This is done by React, the hook useSyncExternalStore accepts second parameter that is a callback that will be called when the app is rendered on the server.
My guess would be that it happens during the compile time (can you confirm that?), but I believe I need to do proper investigation.
In the meantime, I'll take a look at your PR, thank you!

Closing this as it's been fixed in v3. Thank you!


Scrcpy not working while other android emulators runnig

i play pubg on android emulator "Smartgaga" and i have connect my android device through adb over wifi device is rooted
i run this on anroid
setprop service.adb.tcp.port 5555
stop adbd
start adbd
and this on windows terminal
adb connect <IP_ADDRESS>:5555
scrcpy
i get this
INFO: scrcpy 1.14 https://github.com/Genymobile/scrcpy
adb: error: failed to get feature set: more than one device/emulator
ERROR: "adb push" returned with value 1
Press any key to continue...
it works fine when other emulators are not running and also when device is connected through usb i just have to specify device id
i don't know how to specify device id while connected with adb over wifi
adb devices
List of devices attached
<IP_ADDRESS>:5555       device
emulator-5554   device

scrcpy -s <IP_ADDRESS>:5555

https://github.com/Genymobile/scrcpy/blob/master/FAQ.md#several-devices-connected


Asset caching

Is your feature request related to a problem? Please describe.
With nexrender-action-cache it is possible to cache the project template (AE file) with a TTL when downloaded over a network. However this caching is not implemented for assets yet (as per #616).
Describe the solution you'd like
I'm moving more and more assets around my network when rendering so being able to cache assets as well instead of hitting the network all the time will be a great thing to have. We may also need to look into more cache invalidation and per asset cache bypass methods.
Describe alternatives you've considered
Since nexrender-action-cache is already there, I thought I build on it if possible.
Additional context
I'll try to look into the implementation in a bit.

Actually, that is another great idea. I have not thought of that. Will be looking forward to the implementation!

NVM, reading back the original issue I see I mentioned something like this, but already forgot about that 😄

As mentioned in #835, I will probably be trying an alternative implementation of caching. Since I'm only focusing on caching network resources, specifically HTTP sources, I'll probably build it based off of the HTTP cache standards.

Fully support using HTTP cache standards or any existing standards for that matter. 👍  I think the URI standard, for example, fits very well with this library, and I don't even want to imagine some other custom solution that might have been added instead.

I found a pretty ideal solution in the form of make-fetch-happen which is a fetch polyfill used and maintained by NPM that support HTTP caching semantics plus a bunch of really nice stuff such as automatic retry, proxy, and custom TLS certificates.
Since it is a fetch polyfill, we can just use it as a drop in replacement for node-fetch currently in use. Caching functionalities would need to be defined by passing an extra params property to the assets that is passed directly to make-fetch-happen but it also means it is completely backwards compatible.
I've tried it out in a simple test and it works perfectly without issue, given that the server has implementation for the caching related HTTP headers. If that looks good, I can go ahead with the implementation directly to nexrender-core.

Sounds good


[ENH] clarify that BIDS does not specify names of directories containing datasets

closes #1519

@yarikoptic pinging you here because the original issue was opened after a chat with you and @sappelhoff in Copenhagen last year.

close in favor of #1741


Use the copied version of external components instead of rails-assets'

Fixes #100

@ehelms FYI

Thanks @skateman, I will test this in my environment today and report back.

@ehelms there are some issues with bs-touchsping, I'm checking them right now

Basic test of applying and installing a new patternfly gem with this alleviated the issue we were seeing in #100. I did a assets precompile test of our assets as well and everything was OK.

great, I found the problem with the touchspin, fix is coming in a few mins & I'll create a maintenance release


How to obtain 2-electron integrals with PySCF

As far as my understanding goes I can calculate the 4-dimensional tensor eri_{pq,rs} via the command mol.intor("int2e"), which is related to the 2-electron integrals (pq|rs) via the formula given in the attached screenshot. I was wondering whether there is a command to directly calculate the terms (pq|rs)?

No, mol.intor('int2e') is just $(pq|rs)$.

Thanks for the reply! But how can I reconcile that with the above equation $eri_{pr, qs} = (pq|rs) - (.5 / Nelec) [\sum_q (pq|rs) + \sum_p (pq|rp)]$?

I don't know where you got that equation from, but that is not the usual meaning of the ERIs in PySCF (or anywhere else, as far as I know). In other words, your h2e expression is not the common way to write the two-body part of the Hamiltonian.

I obtained it from the PySCF docs: https://pyscf.org/pyscf_api_docs/pyscf.fci.html#pyscf.fci.direct_nosym.contract_2e
and validated it by calculating the Hartree-Fock energy with the PySCF-calculated 1- and 2-RDMs

Unless you're really interested in full CI, you can ignore that (the language is a warning that the expected argument to this FCI function is not the "usual" ERI, which would be (pq|rs), but rather a slightly different tensor). The quantity returned by mol.intor('int2e') is just the "usual" ERI tensor $(pq|rs)$ where the indices are AOs (not MOs).

I see, thank you!


doc: add getinfo synced field

Is there a way to have a state of the syncing progress ?
(I we want to have a progress bar)

Pull Request Test Coverage Report for Build 255370119

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

Totals

Change from base Build 245053445:
0.0%

Covered Lines:
291

Relevant Lines:
709

💛  - Coveralls


Problem when replacing with ag

I encounter this problem when trying to run the following command:
:Replace /fail: function \(response\) { console.log\(response\) }/asd/

The search would succeed but the replace would simply not run.
The problem seems to be the \( and )\ that are recognized by the search (ag) but not the replace.
You can find my .vimrc here. Also I am using Neovim.
I ended up using:
set grepprg=ag\ --nogroup\ --nocolor\ -Q

But this means I can no longer search by regexp.
If you can point me in the right direction I can try a PR.

I've found a workaround:
:Replace /fail: function \((response\)) { console.log\((response\)) }/asd/

I now have \(( for ( and \)) for )
It's OK to close this if you decide it's to much trouble.


README comments

The README.md has "The is different".
Also has a second part to the Examples that perhaps requires some context for the casual reader to understand what it is showing.
use futures::future::join_all;

async fn foo(i: u32) -> u32 { i }

let futures = vec![foo(1), foo(2), foo(3)];

assert_eq!(join_all(futures).await, [1, 2, 3]);

Maybe this second example was meant to have a different use statement? As in the source doc:
use futures_buffered::join_all;

So having found join_all and try_join_all because of that second example, I wonder if perhaps this first line of the description could be changed to reflect that the crate offers more than one struct?
This project provides a single future structure: FuturesUnorderedBounded.

Yeah, thanks for bringing this up. The readme is probably quite out of date from what I first implemented


Trouble with using local_certificate_path & local_key_path

Unfortunately the email client I am using is hardcoded to enable SSL (.NET SmtpClient embedded within 3rd parties software), so I must have a secure connection between the client and the proxy. However, I am having a difficult time getting this to work with the proxy and could use some guidance.
I used mkcert to create a cert for local dev & testing (mkcert -key-file key.pem -cert-file cert.pem localhost <IP_ADDRESS>). I then altered emailproxy.config accordingly:
[SMTP-1587]
server_address = smtp.office365.com
server_port = 587
starttls = True
local_certificate_path = ./config/cert.pem
local_key_path = ./config/key.pem

And when attempting to connect the mail client to the email proxy, it hangs for a bit until the MAX_SSL_HANDSHAKE_ATTEMPTS limit is reached. Here are the logs (note that I added some additional log points to get the SSL specific errors in case they were useful):
SMTP ([::1]:52606-{[::1]:1587}-smtp.office365.com:587) <-> [ Starting TLS handshake ]
SMTP ([::1]:52606-{[::1]:1587}-smtp.office365.com:587) --> [ Client connected ]
SMTP ([::1]:52606-{[::1]:1587}-smtp.office365.com:587)     <-- b'220 DS7P222CA0017.outlook.office365.com Microsoft ESMTP MAIL Service ready at Thu, 4 Jan 2024 17:48:46 +0000\r\n'
SMTP ([::1]:52606-{[::1]:1587}-smtp.office365.com:587) SMTPModifySenderPlugin : receive_from_server --> b'220 DS7P222CA0017.outlook.office365.com Microsoft ESMTP MAIL Service ready at Thu, 4 Jan 2024 17:48:46 +0000\r\n'
SMTP ([::1]:52606-{[::1]:1587}-smtp.office365.com:587) <-- b'220 DS7P222CA0017.outlook.office365.com Microsoft ESMTP MAIL Service ready at Thu, 4 Jan 2024 17:48:46 +0000\r\n'
... (below two lines repeat multiple times until `MAX_SSL_HANDSHAKE_ATTEMPTS` is reached)
SMTP ([::1]:52606-{[::1]:1587}-smtp.office365.com:587) [ TLS handshake started... ]
SMTP ([::1]:52606-{[::1]:1587}-smtp.office365.com:587) [ SSLWantReadError ] The operation did not complete (read) (_ssl.c:1000)
SMTP ([::1]:52606-{[::1]:1587}-smtp.office365.com:587) [ TLS handshake started... ]
SMTP ([::1]:52606-{[::1]:1587}-smtp.office365.com:587) [ SSLWantReadError ] The operation did not complete (read) (_ssl.c:1000)
SMTP ([::1]:52606-{[::1]:1587}-smtp.office365.com:587) SSL socket handshake failed (reached `MAX_SSL_HANDSHAKE_ATTEMPTS`)
Caught connection error in SMTP ([::1]:52606-{[::1]:1587}-smtp.office365.com:587) : <class 'ssl.SSLError'> with message: ac.robinson.email-oauth2-proxy
Is the server's `starttls` setting correct? Current value: True
You have set `local_certificate_path` and `local_key_path`: is your client using a secure connection? github.com/FiloSottile/mkcert is highly recommended for local self-signed certificates, but these may still need an exception in your client
If you encounter this error repeatedly, please check that you have correctly configured python root certificates; see: https://github.com/simonrob/email-oauth2-proxy/issues/14
SMTP ([::1]:52606-{[::1]:1587}-smtp.office365.com:587) --> [ Client disconnected ]
SMTP ([::1]:52606-{[::1]:1587}-smtp.office365.com:587) <-- [ Server disconnected ]

So it would seem the TLS handshake is failing with SSLWantReadError: The operation did not complete (read), but have not been able to figure out why exactly.
Note that I tried a few different email clients with SSL enabled (.NET's SmtpClient, powershell's Send-MailMessage, python's smtplib.SMTP, as well as openssl cmd line utility), all of which produced similar logs as shown above.
FYI, I used the following openssl cmd for testing: openssl s_client -starttls smtp -ign_eof -crlf -connect localhost:1587, which seemed to work fine if I pointed it directly to smtp.office365.com:587 (for example). This article was a good guide for how to test sending SMTP commands manually.
I'm kind of at a loss for why it's not working with these local dev certs. As a quick test, I did setup a simple python script that opened a secure socket with the same cert. and it did seem to work. But I know there's a little more involved when it comes to an SMTP connection w/ TLS and such.
Appreciate any direction you can provide.

Is your client configured to trust self-signed certificates? Many applications/clients are fine, when using mkcert but others (for example, Firefox) require an exception to be set up.
What is the output when you enable the proxy's debug mode and try the following command in a terminal? (Replacing with your own IP and port)
openssl s_client -crlf -connect <IP_ADDRESS>:1588

Were you able to make any progress with this?

I have not been able to test any further. The openssl command you suggested running (i.e. w/o starttls/smtp) I'm fairly certain I've already tested, and IIRC it just hangs since the starttls command is expected by the proxy, but never received. Would need to retest to confirm that behavior, but fairly certain that's accurate.
As for making sure the client is configured to trust self-signed certificates, the only "clients" I'm using are python, powershell, and .net scripts, as well as the openssl command, which, to my knowledge, dont need to be configured to accept selft-signed certs (correct me if I'm wrong). Since I couldn't get those other scripts to work, I focused on trying to get it to work with openssl directly, since if I can't get it to work at that level, then of course no script or client is going to work. As mentioned, I can get the openssl command to work against Office 365's smtp endpoint, just not with the proxy. I haven't moved to testing in any actual email client program, as I'd expect to get it to work with openssl command first, therefore negating the allowing the self signed certs (I believe).
Are you actively able to get this to work with mkcert and openssl command?

Please could you elaborate about your mention of STARTTLS? This could well be the issue – you should not enable STARTTLS in your client, because as explained in the example configuration file the proxy must handle this itself in order to be able to intercept the authentication commands on your behalf.
Re: certificates, just for completeness here is the full list of commands that you'd run to set this up from scratch:
brew install mkcert
mkcert -install
mkcert <IP_ADDRESS>

Use the following configuration file entry with the proxy:
[SMTP-1587]
server_address = smtp.office365.com
server_port = 587
starttls = True
local_certificate_path = /path/to/<IP_ADDRESS>.pem
local_key_path = /path/to/<IP_ADDRESS>-key.pem

Then run the proxy (python emailproxy.py) and SMTP works without issue using the following OpenSSL command:
openssl s_client -crlf -connect <IP_ADDRESS>:1587

Quick follow-up:
I started from scratch and followed your steps exactly from above, but this time I stayed within a linux environment (been using windows, but not sure that's going to make a big difference in the long run).
Only step I had to modify was with the last part when using the OpenSSL command: openssl s_client -ign_eof -crlf -connect <IP_ADDRESS>:1587. The ign_eof just helped to allow subsequent text commands from the command window with line-endings to be sent and received properly (was failing when I get to the RCPT TO part). Afterwards, I was finally able to get an email sent successfully using the OpenSSL command.
I tried my other scripting email clients (.NET's SmtpClient & powershell's Send-MailMessage) but they were still failing with the SSL socket handshake failed (reached MAX_SSL_HANDSHAKE_ATTEMPTS) error. I tried a different .NET library called MailKit, and was able to get it to work though. The third party program I will ultimately end up needing to use this with is using the .NET SmtpClient, which is hardcoded with EnableSsl=True. I have no ability to change the client that is used or whether or not the EnableSsl property is on or off (otherwise I wouldn't of brought up this issue).
My next steps are to decipher that when setting EnableSsl=True, how does that affect what commands the .NET client sends over to the proxy. I suspect that when enabled, the .NET client will start to perform the STARTTLS negotiations, which I can't disable. I will confirm when I know more.
Not sure what platform you're running on, but .NET is cross-platform now, so if you're feeling up for helping get to the bottom of it, you can test this same code anywhere (I believe). For testing, I'd recommend installing powershell core, since it will include .NET libs. Attached are three powershell scripts I've been using for testing this out. The Mailkit script is the one that does seem to work out-of-the-box, and the other two do not. IIRC, the Send-MailMessage one basically just sends as the .NET SmtpClient underneath.
SendTestEmail_Scripts.zip

hardcoded with EnableSsl=True

This is the issue. You mentioned previously that SSL was hardcoded, but that link provides the extra context: for some reason this client only supports the STARTTLS method, and does not support connections that are SSL from the start. This is not currently compatible with the proxy.
If you can find a way to disable the EnableSsl option in your client, you'll be able to use it with the proxy. Otherwise, one potential solution would be to extend the proxy to support local as well as remote STARTTLS. You'd need to provide your own certificate for this, but you're already doing that, so I'm assuming there wouldn't be an issue there.
I'm happy to look at adding this feature on a consultancy basis if it needs prioritising.

Well I ended up finding a workaround for my particular situation. Found that I could install IIS SMTP Server Relay and place that in-between my email client and the proxy. The relay properly supports the hardcoded EnableSSL=True property set by the .NET email client (once certificates are configured under the relay), which then forwards the connection appropriately to the proxy (w/o certificate configuration). So my current flow is:
.NET SmtpClient --> IIS SMTP Relay --> Email Proxy --> Email Provider (Office 365)

Ultimately, it would be nice to get the proxy configured to support bi-directional STARTTLS flows to avoid this MITM relay, but this gives me something usable now so that I can move on with my actual use case, so I'll go ahead and close this issue for now.
Thanks for hearing me out!

Thanks for following up - I'm glad you were able to find a workaround here.
I'll look into client STARTTLS at some point if I get chance.


Update query string on click

Update query string on the search bar component  instead of opening a new tab.

Hm. Aren't we already updating the location component?

You are correct, I don't know what I was doing


Hang detection

I just came up with a way this could be used to detect and isolate infinite loops and the like, or even just very slow code:

Each time a public is called, make a note of the time.
Each time break is called, make a note of the time elapsed within that callback.
When this time passes some threshold print a warning with a traceback.
When the callback ends, clear the time.

The threshold need only be a few ms, possibly configurable, default 100ms to be safe.  For recurisve public calls such as via CallLocalFunction, I'd say don't apply this recursively - the time spent in the child publics still counts against the time spent in the original callback if the whole thing is slow.

Sounds like a nice idea.
BTW for debugging infinite loops one could send Ctrl-C/SIGINT to the server process to cause crashdetect to print a stack trace (and the server to exit). But I'm not sure if this still works, it may be broken on Linux or Windows (or both...).


Updated to mac Sonoma: issue with functions watching wrong directory

After upgrading to Sonoma, when the algolia extension is enabled and I run the emulators locally, they override my functions watch directory with the cached extensions directory
functions: Watching<EMAIL_ADDRESS>for Cloud Functions...```

The extension project structure is provided by the Firebase team.  Maybe open a ticket with the Firebase team.

@dubitoergo I have just hit this error today in the firebase emulator, and in my case it was a problem with a misconfigured index file in the firebase/functions folder that exports all my functions, it had a line referring to a function that I deleted earlier, removing that line allowed the emulator to rebuild itself correctly without the error.

Thank you for providing your resolution.


GH-99: Document Aggregator Inputs and Outputs

Related to spring-cloud-stream-app-starters/app-starters-release#99

Merged.


First crack at resources.

Something I've been thinking about for a little while now.
We basically allow users to interact with Figure and Grid objects, but these are really just part of general File objects.
This is sorta what I've got in mind right now (and this is weakly implemented already in this PR):
from plotly import resources

my_file = resources.FileResource(fid='theengineear:3409')
my_file.resource.filename = 'nah, it should be named this'
my_file.put()  # if we don't have an error, things are good!
my_file.permanent_delete()  # ERROR trash first!
my_file.trash()
my_file.permanent_delete()

This isn't really ready to be merged in, but I wanted to open up the discussion with some actual code.

@chriddyp @cldougl @etpinard @BRONSOLO thoughts?
I think a super-thin wrapper around our underlying api might be pretty nice!
It would nicely solve the following within the same framework:
"From the api..."

"how do i delete my plots?"
"how can i lookup a plot by it's filename"
"how can i change the privacy of a plot"
"how can i get a list of my plots"
"how can i move a file into a folder"

This would be totally orthogonal to the idea of Figure and Grid creation. These objects would be wrappers that make it easier to handle the more general files.
The other option is to somehow build all of this into Figure and Grid objects. But I'm starting to thing that we'd be straying too far from the actual implementation? I don't think the abstraction is all that helpful?
I'd love any thoughts on this one. There are a few issues in this repo that I think this would solve cleanly.

Also, this is a general api-wrapper question. It'd be nice to have some sort of uniform way to do things like this across our wrappers, yeah?

:+1:

I think @doeg had some thoughts on this too (from the node perspective)

Cool! Yeah, (re?)writing the Plotly API as an npm module is ticketed here: https://github.com/plotly/streambed/issues/3013
We haven't been thinking about it yet, but I'm definitely interested in this discussion. The thinner and more closely coupled the wrapper is to our actual API  the better, in my opinion.
If anyone has any examples of some especially well implemented APIs, I'd love to see 'em. (I know Twitter's entire API is an antipattern of what not to do...)

part of me feels like we should just document how to do this well with requests. Read through their quick-start, as he explores the github api: http://docs.python-requests.org/en/v1.0.1/user/quickstart/
maybe the best approach is to document how to do these common tasks (with requests), and then see if there is any annoying boiler plate code that we can simplify for people.
the nice thing about keeping it tightly coupled to the API is that we'll only need to update the api.plot.ly docs

@doeg yeah the npm module spec you've got going is right inline with what I want for the Python API.
@chriddyp totally see your point about not maintaining another hunk of code. just thought I'd bring this up though. as our api gets more consistent, implementing a thin wrapper around it gets easier. that said, it's still something to maintain.

Hey! I ended up finally doing this in #640. It ended up really helping to unify our response handling! Closing this in favor of the changes in #640. It's basically a super thin wrapper around requests.request for all our api endpoints.


longStackTraces lose the context for SyntaxErrors

What version of bluebird is the issue happening on? v3.4.6

What platform and version? (For example Node.js 0.12 or Google Chrome 32)  MacOSX, node v6.9.1

It appears Bluebird loses the file/lineno info on longStackTraces when a SyntaxError is raised inside a promise.  For example ...
With longStackTraces: false, Syntax Errors [correctly] appears like this:
boss/db/Command.js:114
  }
  ^
  SyntaxError: missing ) after argument list
  at Object.exports.runInThisContext (vm.js:76:16)
  at Module._compile (module.js:542:28)
  at Object.Module._extensions..js (module.js:579:10)
(etc...)

With longStackTraces: true, that same error appears like this:
SyntaxError: missing ) after argument list
  at Object.exports.runInThisContext (vm.js:76:16)
  at Module._compile (module.js:542:28)
  at Object.Module._extensions..js (module.js:579:10)```

**Note**: the first 3 lines - the lines with the context for the SyntaxError - are missing.   As you can imagine, this makes diagnosing syntax errors difficult.  I find myself having to restart my app with the appropriate longStackTraces setting, depending on the type of error. :-/

Any ETA on when this might be merged/published?  This is a non-trivial pain point for me - any time I have a syntax error in my code I have to disable long stack traces to determine the location.  :-(

I'll try to get release today

@petkaantonov can this be closed with https://github.com/petkaantonov/bluebird/releases/tag/v3.4.7 ?

Yes.


add hooks for tweaking how downloads happen (for grist-static)

This makes three main changes:

Adds a hook to transform download links.
Adds a hook to add an externally created ActiveDoc to a DocManager.
Rejiggers XLSX export code so it can be used without streaming, which is currently tricky in a browser. Regular usage with node continues to use streaming.

With these changes, I have a POC in hand that updates grist-static to support downloading CSVs, XLSXs, and .grist files.

Posted the POC at https://github.com/gristlabs/grist-static/pull/34


[stable/rocketchat] Correct the indentation for node affinity

What this PR does / why we need it:
This character change fixes RocketChat deployment node affinity indentation. Node affinity is not otherwise respected.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[ ] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/ok-to-test

/lgtm

@philtweir please sign off your commit (git commit --signoff) to comply with the DCO.
Also the chart needs a version bump.

Thanks @srueg - will do asap.

Apologies, took a couple goes to get the flow - have updated those with signed-off commits, and merged from master as a separate commit - is that sufficient ahead of squash & merge, or is there anything I can do to tidy up on my end?

Honestly, I don't know if it's possible to squash & merge with prow... Could you please rebase/squash in this branch?

Sure - think that should be it? Many thanks :)

/lgtm

Thanks for your contribution! 👍

Thanks @srueg!


Actually use the db name

We weren't really making use of the DB_name in the api calls...
Also included that Makefile update in here as I don't think we'll merge the Jinja2 PR.

[x] I updated the README.md docs to reflect this change.
[x] This is not a breaking API change OR
[ ] This is a breaking API change and I have incremented the API version.

Will deploy this to CI nowish


Setting name and maintainer on Schema

https://github.com/weaviate/weaviate-java-client/blob/488336ff600b06dc8f0eddef88598e63cb876471/src/main/java/technology/semi/weaviate/client/v1/schema/model/Schema.java#L15-L16
How are these two values set?

Thank you @codebrain for reporting the issue.
Indeed those 2 values are not set. They seem to be leftovers of non-existent functionality.
We will remove them from clients and weaviate soon.


Implement reverse playback for animtree

For #21665 and #27215.

Since it is assumed to be a non-reverse play, find() gets the key frame just before the current time.
Also, the Animation Node was interfering with the loop and root motion.

This fix solves these issues and implements a negative Time Scale in the Animation Tree.
https://www.youtube.com/watch?v=iPIG41NEgyM
I've uploaded a sample project to be used for testing.
animation_tracks_test.zip
reverse_root_motion.zip

This should be rebased to remove all the irrelevant commits, see PR workflow.

Sorry, I've reworked the branch and PR to go along with the rebase.
#41728


fixed issue #24 where location wasn't updating

Previously, location was updating after after entering a wrong location followed by a correct location. Ideally it should be showing unknown location after entering a wrong location but instead it was showing the location of previous searched location. I fixed this issue.

Can you please assign  hacktoberfest-accepted tag to this contribution.


Update readme.md

@mastanav Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

new line

ee


Add RID-specific runtime folder load fallback

fixes #266
/cc: @stevenbrix

Thanks Scott! Could this just replace the call to Platform.LoadLibraryEx?

Thanks Scott! Could this just replace the call to Platform.LoadLibraryEx?

NativeLibrary is only available on net5, so still need LoadLibrary at least for netstandard2, and it doesn't hurt as a fallback for net5


Add is_nan and contains_nan tensor ops

Pull Request Template
Checklist

[x] Confirmed that run-checks all script has been executed.
[x] Made sure the book is up to date with changes in this PR.

Related Issues/PRs

This PR revealed this bug https://github.com/tracel-ai/burn/issues/2089

Changes

Add is_nan tensor method
Add contains tensor method

Testing
New unit tests

Currently this is failing for WGPU backend. I have filed this bug: https://github.com/tracel-ai/burn/issues/2089

@nathanielsimard I ignored the failing tests by wgpu for now. We will enable them once we fix #2089


Adding single files is too tedious

too add files to a partition you need to have steps like:
file-resource bootcode.bin {
    host-path = "${NERVES_SYSTEM}/images/rpi-firmware/bootcode.bin"
}

then in the each task you must:
on-resource bootcode.bin { fat_write(${BOOT_PART_OFFSET}, "bootcode.bin") }

This leads to a ton of duplicated lines. (especially when you have a part_a/part_b system)
It would be nice to just be able to specify a list of files you are concerned with

Agreed


[retire] Retiring on behalf of another beneficiary & receipt errors

As identified by chaz
For pool listings, retire form, if I select another beneficiary for the retirement like so:

Once the retirement is successful, the retirement receipt we link to the SEE YOUR RETIREMENT RECEIPT button is incorrect.

As you can see in screenshot above, the URL generated is for the logged in wallet and retirement 1. In this example, the user is redirected to this receipt
Again for this example, the receipt URL should be https://www.carbonmark.com/retirements/0x3d23E091Cd16FaB640e7695D9682864BD4FC7D6e/1
So the receipt number is correct, but wrong wallet.

We will probably want to do a call to kickoff this one @sprrwhwk

We will probably want to do a call to kickoff this one @sprrwhwk

@Atmosfearful @sprrwhwk sure, no problem. I'm only available Mon - Wed morning next week then on a summer holiday until the 14th.

@Atmosfearful I've scheduled one for tomorrow. We can start this then!


Bug with intervals and logical and/or

Describe the bug
While reviewing #3408, I found an additional error when trying to evaluate select i_item_desc from test where d3_date > d1_date + INTERVAL '5 days' and d2_date > d1_date + INTERVAL '3 days':
Error: NotImplemented("Unsupported interval argument. Expected string literal, got: BinaryOp { left: Value(SingleQuotedString(\"5 days\")), op: And, right: BinaryOp { left: Identifier(Ident { value: \"d2_date\", quote_style: None }), op: Gt, right: BinaryOp { left: Identifier(Ident { value: \"d1_date\", quote_style: None }), op: Plus, right: Interval { value: Value(SingleQuotedString(\"3 days\")), leading_field: None, leading_precision: None, last_field: None, fractional_seconds_precision: None } } } }")
To Reproduce
Here is the full code which should reproduce the error:
use datafusion::prelude::*;

#[tokio::main]
async fn main() -> datafusion::error::Result<()> {
  let ctx = SessionContext::new();

  let schema = datafusion::arrow::datatypes::Schema::new(vec![
    datafusion::arrow::datatypes::Field::new("i_item_desc", datafusion::arrow::datatypes::DataType::Utf8, true),
    datafusion::arrow::datatypes::Field::new("d3_date", datafusion::arrow::datatypes::DataType::Date64, true),
    datafusion::arrow::datatypes::Field::new("d1_date", datafusion::arrow::datatypes::DataType::Date64, true),
    datafusion::arrow::datatypes::Field::new("d2_date", datafusion::arrow::datatypes::DataType::Date64, true),
  ]);

  ctx.register_csv(
    "test", "data.csv", CsvReadOptions::default().schema(&schema)
  ).await?;

  let df = ctx.sql("select i_item_desc \
        from test \
        where d3_date > d1_date + INTERVAL '5 days' \
        and d2_date > d1_date + INTERVAL '3 days'").await?;

  df.show().await?;
  Ok(())
}

Expected behavior
Return the expected output with any errors.
Additional context
In my example, data.csv contains:
i_item_desc,d3_date,d1_date,d2_date
a,2022-12-12T7:7:7,2022-12-12T7:7:7,2022-12-12T7:7:7
b,2022-12-12T7:7:7,2022-12-11T7:7:7,2022-12-12T7:7:7
c,2022-12-12T7:7:7,2022-12-10T7:7:7,2022-12-12T7:7:7
d,2022-12-12T7:7:7,2022-12-9T7:7:7,2022-12-12T7:7:7
e,2022-12-12T7:7:7,2022-12-8T7:7:7,2022-12-12T7:7:7
f,2022-12-12T7:7:7,2022-12-7T7:7:7,2022-12-12T7:7:7
g,2022-12-12T7:7:7,2022-12-6T7:7:7,2022-12-12T7:7:7
h,2022-12-12T7:7:7,2022-12-5T7:7:7,2022-12-12T7:7:7

I am not sure whether this is a bug in sql-parser, cc @andygrove . This is what I get
❯ select interval '5 days' > interval '3 days';
NotImplemented("Unsupported interval argument. Expected string literal, got: BinaryOp { left: Value(SingleQuotedString(\"5 days\")), op: Gt, right: Interval { value: Value(SingleQuotedString(\"3 days\")), leading_field: None, leading_precision: None, last_field: None, fractional_seconds_precision: None } }")
❯ select (interval '5 days') > interval '3 days';
+-----------------------------------------------------------------+
| IntervalDayTime("21474836480") > IntervalDayTime("12884901888") |
+-----------------------------------------------------------------+
| true                                                            |
+-----------------------------------------------------------------+
1 row in set. Query took 0.006 seconds.

Looks very similar to
https://github.com/apache/arrow-datafusion/issues/3408

~sqlparser-rs/sqlparser-rs#705 has been merged, but now it seems like the second condition of something like where d3_date > d1_date + INTERVAL '5 days' and d2_date > d1_date + INTERVAL '3 days' isn't being checked by DataFusion. Currently looking into this.~

This turned out to just be an error with the data file I was working with. Everything seems to work as expected now.

I actually think this issue is not yet actually "fixed" in datafusion as we haven't updated the sqlparser version to 0.28 which includes the change. I will do so later today so reopening this PR

https://github.com/apache/arrow-datafusion/pull/4568

This is fixed -- I have a PR open to add some regression test coverage: https://github.com/apache/arrow-datafusion/pull/4569


Feature Request: support sparse tensor in conv2d operator

It looks like TF doesn't support conv2d as another thread mentioned (https://github.com/tensorflow/tensorflow/issues/1604). This is critical for us since we are implementing CDSSM (from microsoft https://www.microsoft.com/en-us/research/wp-content/uploads/2016/02/www2014_cdssm_p07.pdf). Miscrosoft's library supports it and the training speed is dramatically faster than using TF. So we strongly feel this feature is important.

Is this different from #1604 ?

It is essentially the same. But that was  not reported as a feature request. I am shouting out officially to request it. :)

I will retitle that one.

That works too. Thanks!


GH-182: Optional plugins

Builds on the work contributed by @anthonyalayo in GH-183 with additional changes to improve internal APIs to robustly support the change.
Fixes GH-182.

PR description will be updated once ready.

Additional changes to include:

[x] Handle file:// URLs consistently when not found - right now we silently ignore it.
[x] Rework UrlResourceFetcher to return empty Optionals if resources are not found.
[x] Remove optional flag from Maven protoc plugin descriptors as it is not used.
[x] Handle case where no plugins and no languages are enabled.
[x] Add parameter to control whether to fail if both languages and plugins are not enabled.

/rebase

@anthonyalayo please can you build this locally and verify it works in the way you expect. If I don't hear anything in the next week or so I'll merge this and promote it :-)
Thanks for being patient!

I think for now, going to merge this and do an alpha release with it again like before... please let me know if it works for what you need and I will look towards promoting a stable version!

It works great, thank you!


CrystFEL: init at 0.10.1

Motivation for this change
CrystFEL is a tool often used in serial crystallography, see its homepage here:
https://www.desy.de/~twhite/crystfel/index.html
To use this easily from our institution, I want to include it in nixpkgs.
Unfortunately, CrystFEL has a lot of dependencies, some of them are projects developed here, some are external things like mosflm or the hdf5 plugins. For now, I've decided to include them only in this derivation.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[x] 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

[x] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 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.

@OfBorg build crystfel-gui

@ofborg build crystfel-gui

@ofborg build crystfel-gui

@ofborg build crystfel-gui

@ofborg build crystfel-gui

@ofborg build crystfel-gui

If this is still a WIP would you mind changing it to a "draft PR"?

If this is still a WIP would you mind changing it to a "draft PR"?

Converted. I expect this to take a while longer, thanks for the suggestion!

@ofborg build crystfel-gui

@ofborg build crystfel-gui

@ofborg build crystfel-gui

@ofborg build crystfel-gui

@ofborg build crystfel-gui

@ofborg build crystfel-gui

@ofborg build crystfel-gui

@ofborg build crystfel-gui

@ofborg build crystfel-gui

You can edit a previous comment and for example add a space to trigger ofborg again. Also if you name your commit to fir the contributing guide eg crystfel-gui: something ofborg should start the build automatically.

@SuperSandro2000 Thanks for the review! A lot of your suggestions are due to code style/indentation. I ran the source files through nixfmt (without any parameters). Should I use something else to format the code?

I ran the source files through nixfmt (without any parameters). Should I use something else to format the code?

nixfmt does enforce 80 line length with not ideal new lines. We don't really care about line length as long as it isn't to long. I would recommend to not format files if you are just changing a few lines to avoid it all together. I can personally recommend nixpkgs-fmt which generally works better for me and does not enforce line lengths.

@ofborg build crystfel-gui

@ofborg build crystfel

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/861

Can you squash all the package commits together? Then the PR should be in a good state.

Done.

@SuperSandro2000 Any other comments & critique? I'm very thankful for your work, by the way, it has taught me a lot about Nix!


Renders behind android virtual buttons.

I'm using an Android 7.1 smart phone and if I set the height to 100%, the view renders behind the virtual buttons, making it impossible to access components at the button of the view.
Please i would like to know iif I am the only person experiencing this and what I can do.
Thanks a lot in advance.

Hey there @i-mighty,
I am trying to understand the problem. When you say virtual buttons what do you mean exactly? Emulator buttons or your app's buttons? Also you mean the problem is in the bottom, not button right?
My guess is that some part of the UI is absolute and simply hides behind.


Should all *.tpl files be treated as Smarty templates?

In my own repository I have *.tpl file which serves as template (even contains some HTML inside) but isn't a Smarty template, yet GitHub (linguist) counts it as such.
Should it work that way? What do you think - maybe some heuristics could be added?

What's the language of these .tpl files? You should check the guidelines to add support for a new language to Linguist. If it meets the requirement on in-the-wild usage, don't hesitate to open a pull request.

The thing is that it isn't anything standard, like *.txt or *.cfg. I managed to find in-the-wild usage from Mozilla (these are definetely not Smarty templates): https://github.com/search?utf8=✓&q=extension%3Atpl+user%3Amozilla&type=Code&ref=searchresults

OpenModelica uses .tpl files for Susan language  These are definitely not Smarty.
Those are text files which start with the keyword "package". C++ like comments and white space may occur before the keyword.
Can you implement this new language, please?
Filter by extension is not appropriate here, as I can imagine many people using *.tpl as simple template file for whatever they like...

@hkiel @hckr please take a look at our contributing guide for instructions on how to create a pull request for a new language. We'd love to review a PR from one of you for this. Thanks!
https://github.com/github/linguist/blob/master/CONTRIBUTING.md#adding-a-language


KAFKA-15644: Fix CVE-2023-4586 in netty:handler

MInor Update: Upgrading Netty to 4.1.100.Final to remove CVE-2023-4586
Ref: https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2023-4586

Hi @mimaison can you please review this

Backported to 3.6: df601e3


Build with Cmake: Doesn't build with python3.12 or higher: ModuleNotFoundError: No module named 'imp'

Problem description
In my projects CMakeLists.txt:
include(FetchContent)
FetchContent_Declare(
  mavsdk
  GIT_REPOSITORY https://github.com/mavlink/MAVSDK.git
  GIT_TAG        v2.2.0
  GIT_SUBMODULES_RECURSE TRUE
)
FetchContent_MakeAvailable(mavsdk)

If cmake picks Python 3.12 (or higher) the build fails:
[cmake] ModuleNotFoundError: No module named 'imp'
imp has been removed in Python 3.12
Expected behavior
Works with Python >= 3.12
Side note:
I tried with
unset(Python3_EXECUTABLE)
find_package (Python 3.10 EXACT REQUIRED COMPONENTS Interpreter Development)

in cmake. However, this seems not to apply to dependencies as fetched by FetchContent_*
Idea

Maybe adding

unset(Python3_EXECUTABLE)
find_package (Python 3.10 EXACT REQUIRED COMPONENTS Interpreter Development)

to the MavSDK CMakeLists.txt fixes the issue?

Get rid of imp

We should probably do 2. Do you know where it's required?

This was happening in future egg but was corrected in 18.4 release: https://github.com/PythonCharmers/python-future/pull/632 .

Ok, so potentially with latest future it should be ok again?

On my side at least it solved the problem.

Nice, ok!


Add Groups to New/Set-SnipeitUser.ps1

Ok looks like 4th time is the charm...
Added ability to assign a user to groups on or after creation with New/Set-SnipeitUser

Thanks! I'll merge this to develop.

@chaosjester I just published this to master. new release with you contribution will be available in few minutes.


[Bugs] UPF build error with go 1.21

Describe the bug
UPF cannot be build with go 1.21
Expected behavior
Can build with go 1.21
Screenshots
$ go version
go version go1.21.4 linux/amd64
$ make upf
Start building upf....
cd NFs/upf/cmd && \
CGO_ENABLED=0 go build -gcflags "" -ldflags "-X github.com/free5gc/util/version.VERSION=v3.3.0-22-ge48e933 -X github.com/free5gc/util/version.BUILD_TIME=2023-11-08T05:41:37Z -X github.com/free5gc/util/version.COMMIT_HASH=4fe9ca39 -X github.com/free5gc/util/version.COMMIT_TIME=2023-10-24T14:52:58Z" -o /home/nishi/free5gc-deb/free5gc/bin/upf main.go
go: updates to go.mod needed; to update it:
        go mod tidy
make: *** [Makefile:47: bin/upf] Error 1
$

Environment (please complete the following information):

free5GC Version: e48e933dde6bad3a83f6cd327003eeba6992abfa
OS: Ubuntu 20.04.6 LTS
Kernel version: 5.15.0-88-generic
go version: 1.21.4

@ShouheiNishi
Thank you for reporting this issue. However, the recommended Go version for the free5gc official website is go1.18.10. Currently, the versions in the go.mod files of all NFs except for upf are set to go 1.17. We may consider upgrading upf to version 1.17 first to synchronize it with the other NFs.

@ShouheiNishi Thanks for the PR.


Load WFS with XML query sample

The next WFS sample ready to review please @tschie, thanks!

thanks @tschie. I've added an xml file into resources to avoid using raw XML in the code. Can you take another look please?

I think you missed the <?xml version="1.0" encoding="UTF-8"?> line at the top of the file. It's usually optional but it's good practice to include it.


Chat triggers not working with Herochat Pro

The output of /version on my server is: This server is running Paper version git-Paper-1618 (MC: 1.12.2) (Implementing API version 1.12.2-R0.1-SNAPSHOT)
The output of /version denizen on my server is: Denizen version 1.1.4-SNAPSHOT (build 1712-REL)
(If relevant) A link to a paste of a script demonstrating the issue: This is the script i used to test it https://one.denizenscript.com/haste/74770
The issue is:
The plugin doesn't detect chat triggers when using the plugin Herochat Pro, no error in console, just nothing happens when you type in chat.

Support is provided on Discord
1.12.2 is no longer actively supported, nor are old Denizen builds
This is not a Denizen bug

This is because Herochat Pro cancels the event and sends it's own chat messages. You must either use bukkit event priority or remove the plugin. As Wahrheit said, not a bug.

https://discord.gg/Q6pZGSR


Unfocused workspace on another screen doesn't refresh

Rightmost screen stops refreshing content immediately when losing focus as a result of keybind being pressed. On the contrary, moving cursor to next screen, it keeps updating normally until cursor gets over a border of a window, or enters for example this form in the browser.
When updating stops:
[ERROR] [backend/drm/atomic.c:36] DP-1: Atomic commit failed (pageflip): Invalid argument
Over a border on other screen:
[sway/input/cursor.c:835] denying request to set cursor from unfocused client
Workspace being focused (probably not related):
[ERROR] [backend/drm/atomic.c:177] setting gamma LUT 4096
Output setting from my config:
Output configuration
output HDMI-A-1 resolution 3840x2160@60Hz position 0,0 disable
output DP-3 resolution 3840x2160 position 3840,0
output DP-1 resolution 2560x1440 position 7680,0 adaptive_sync on max_render_time 1
output * bg #000000 solid_color

Tried also disabling adaptive_sync, no effect.
Latest wlroots and sway.

Your bug report is missing debug logs. Please add a link to the full debug log file.
Can you obtain DRM debug logs? See https://github.com/swaywm/wlroots/wiki/DRM-Debugging

dmesg.log
sway.log

[   30.801050] [drm:dm_update_plane_state [amdgpu]] Bad cursor position -897, 1256
[   30.801138] [drm:amdgpu_dm_atomic_check [amdgpu]] Atomic check failed with err: -22
[   30.801148] [drm:drm_atomic_check_only [drm]] atomic driver check for 000000007cf7105d failed: -22

I can confirm this issue, I noticed that the freeze happens whenever I hover something that changes the cursor (I think that's also what happens when you move your mouse over a border).

Managed to narrow it down a bit:
fa5d709fc3ea96b14163cdad435caccdbdd5bbda still working
8da9d9679e4cad640b0f0a518ca1529c8cdecbe0 doesn't compile
be90062c51ba5470f513aa118d99472c304e8f36 doesn't compile
52281cb8ba7e0badd92662ade136d25929a67828 not working

Should be fixed by https://lists.freedesktop.org/archives/amd-gfx/2020-May/049825.html

Maybe drm_connector_move_cursor should toggle cursor_enabled when it is out of bounds?

Yup, that would be a good idea. However I thought we already had that logic in wlr_output directly (see the "visible" field).

I don't think that "visible" field is propagated to the hardware cursor. cursor_enabled is only set to false when setting the cursor texture fails. Commit 52281cb always tries to render the cursor if that field is true.

Should be fixed by https://lists.freedesktop.org/archives/amd-gfx/2020-May/049825.html

That does fix the issue, but slightly off topic: is there a simpler way to test this kind of patch than building the whole kernel? I tried doing only make M=drivers/gpu/drm and then using the newly built amdgpu module but for some reason that didn't include your patch (still the same issue with the same "Bad cursor position" log which obviously wouldn't be possible when running the patched version).

starting sway with WLR_NO_HARDWARE_CURSORS=1 fixes this as a quick solution until we get the amdgpu patch (as the hardware cursor seems not to receive the "visible" field as @progandy mentioned).

I don't think that "visible" field is propagated to the hardware cursor.

Ah, indeed, visible is only used for enter/leave events.
Using cursor_enabled might not be straightforward, since we don't want to show a previous cursor when an out-of-bounds disabled cursor becomes in-bounds again.

I haven't tested it, but to me it seems possible that a similar error could also occur if you modeset a smaller resolution and the cursor is then out of bounds?

modeset a smaller resolution and the cursor is then out of bounds

No, the EINVAL check in amdgpu is only about negative coords.

amdgpu patch has been merged.


Quiz app changes

Changes introduced in the Quiz-App template

[x] edit README.md file to add screenshots
[x] add HTML comments
[x] add CSS variables
[x] add JavaScript comments

this fixes issue #112

Hey, thanks for contributing. Can you add one gif of the template also?


Is this for WSL1 and/or WSL2: Windows Support -> Windows Subsystem for Linux (WSL) -> Optional - Only if volumes are required:

This page https://www.testcontainers.org/supported_docker_environment/windows/ describes how to configure WSL to support volume mapping.

Inside the WSL shell, modify the /ect/wsl.conf file to mount the Windows drives on / instead of on /mnt/. (Reboot required after this step).

However, it's not clear whether this applies to WSL1 or/and WSL2.
It would be helpful if this was clarified. There's also a typo.

Hey @cer, thanks for the pointer. This section only applied to WSL1. WSL2 should work, as documented under WSL2 backend. Would you like to provide a PR to correct this?

Does it work correctly if you use docker-compose directly?

Does it work correctly if you use docker-compose directly?

Yes.

Does it work correctly if you configure DockerComposeContainer using withLocal(true)?

Unsure. my Windows machine is running Docker Compose 2 so I've been using  withLocal(false)

It would help to triage if you can try to use it with withLocal(true), you can switch to Docker Compose 1 in the Docker Desktop settings.
But likely, this exact combination (Testcontainers with DockerComposeContainer file mounting and containerized Docker Compose on Docker Desktop with WSL2 backend from Windows process) does not work.

Sorry for the delay. It works with withLocalCompose(true).
I'm guessing the problem is with convertToUnixFilesystemPath(), which is called here: https://github.com/testcontainers/testcontainers-java/blob/66e4be2817f82aa80490c21e8df9e8f8a87fa792/core/src/main/java/org/testcontainers/containers/DockerComposeContainer.java#L623.
I believe it converts c:\foo\bar to /c/foo/bar.

You are right, this will be the problem. Especially in DockerComposeContainer, it is very hard to take care of all possible environment permutations. Would you like to look into providing a PR?

We also just stumbled upon this issue.
Has there been some progress on this? Can we do something to help? If there is a snapshot with a fix, we'll be glad to give it a try.


[Custom application]: Field topic is not displayed on review form for publication request for Admin

EPAM AI DIAL chat version
0.18.0
How to reproduce
Steps:

by user create custom application
input any values in Topics field
input required fields and save
create publication request for custom app
by admin view publication request
click "Go to a review" link

Actual result
Review form with application details is displayed
Field Topics is not displayed
Expected result
Topics  field should be displayed in publication request on review form for Admin
https://www.figma.com/design/lwL5jneveZczAj5RLcLFH2/DIAL-Chat?node-id=10912-35964&node-type=frame&t=ffhiMdKlaLsjeGnG-0
Confidential information

[X] I confirm that do not share any confidential information

verified on staging
related issue #2723


Review Basilisk (SBP Milestone 3)

Review of the Basilisk runtime and included pallets for the Substrate Builders' Program Milestone 3.
Below is a summary of what I consider the most important points:
Runtime

make sure to generate weights
what's your dust management strategy?
you seem to have copied the Hydra pallets which IMO adds maintenance burden

pallet-asset-registry

don't use unwrap in the runtime
suggestion: check the genesis config for sanity

pallet-exchange

I don't think you cover the worst case for the on_finalize weight

pallet-lbp

some confusion about naming/semantics of pausing and pool activity

pallet-nft

mint is not marked #[transactional] but should be
benchmarking does not assume worst case

pallet-transaction-multi-payment

benchmarking does not assume worst case

Review of the Basilisk runtime and included pallets for the Substrate Builders' Program Milestone 3.
Below is a summary of what I consider the most important points:
Runtime

make sure to generate weights
In progress

what's your dust management strategy?
We are developing pallet-duster https://github.com/galacticcouncil/HydraDX-node/pull/288 please feel free to comment / review on it @enthusiastmartin if ready.

you seem to have copied the Hydra pallets which IMO adds maintenance burden
We did this because basilisk is being audited and we'll be removing them from HydraDX soon.

Others 👍🏻 on it.

All review comments have been addressed and either fixed/changed or issue has been created to address it later.


GIX-1909: Update candid files sns-js

Motivation
Use the new fields for the Neurons' Fund enhancements.
In this PR, upgrade the candid files for sns canisters.
Changes

Update candid and related files for SNS canisters.
Export new type NeuronsFundParticipationConstraints that has the maximum commitment of the Neurons' Fund.

Tests

Fixed a mock data with new parameters.

@dskloetd @mstrasinskis please review

Were some or all of the changes in this PR automated by a script?
If yes, can you put the exact script invocation in the PR description?
And if there were also manual changes, can you clearly describe what was automated and what was manual?

Were some or all of the changes in this PR automated by a script? If yes, can you put the exact script invocation in the PR description? And if there were also manual changes, can you clearly describe what was automated and what was manual?

Done! Ready for review


Add flow support

Add flow dependency and babel support

Fixin issue #5

This change is

whoa...thanks @Ridermansb .. will check it


application offline, message retrieval

a few queries about messages missed when offline:

Does an application that uses tdlib get messages when it comes back online? E.g. it goes offline for a few hours, when it comes back online, will tdlib send it the messages it missed when it was offline?
1.1 does that depend on all these flags being true: use_message_database, use_chat_info_database, use_file_database

if a user is authorised with the above database flags as false, and if in next restart when we reconnect user, we enable these to true, is that an expected workflow and will it work as expected?

TDLib will send most missed messages regardless of the used databased. There is an option "ignore_background_updates", which allows to skip the updates and can be enabled if all databases are disabled.

What is the recommended config setting to receiving all messages, not missing a single one? For example, I would like my application to continue where it left off from, irrelevant of the time it was offline. It should continue to receive updates for all messages it missed, any groups user joined etc.

What is the recommended config setting/approach to receiving all messages, not missing a single one?

This is not supported by Telegram apps.

thank you for your prompt responses 👍

sorry one more question, what is the purpose of these options then:  use_message_database, use_chat_info_database, use_file_database? What additional behaviour do they allow?
The description suggests it will cache it between restarts. Is that for performance?

They allow apps to work without internet access and to avoid downloading of the same files twice.


automatically vmap reconstruction functions if given 3d fields

for geostrophy, it should be possible to re-write every thing (including operators) to include 3d fields (but is it really necessary computationally speaking?)

better to leave control of this kind of thinghs to the end-user


Added target_bulk_bytes to the docs for logstash-output plugin

Description
Undocumented but implemented feature
Issues Resolved
n/a
Version
n/a
Frontend features
n/a

Should I do anything further?

@sn

Should I do anything further?

We're all good here. I made additional edits to the page to make it more in line with the rest of the documentation.


Fix titanic example add_dataset_objective

Add permissions to ALGO, DATASET & OBJECTIVE
Fix #167

Until now the default permissions were "private", I think they should stay that way. This will avoid unwarranted leaks if the code from the example is adapted for a real-life use case.

Okay, thank you for this input, I'll update!

In fact, I am still getting a 400 error:
Adding test data samples...
100%|███████████████████████████████████| 2/2 [00:00<00:00,  6.89it/s]
Associating data samples with dataset...
Adding objective...
Requests error status 400: {"permissions":["This field may not be null."]}
Traceback (most recent call last):
  File "/home/zen/.local/lib/python3.7/site-packages/substra/sdk/rest_client.py", line 126, in __request
    r.raise_for_status()
  File "/usr/lib/python3/dist-packages/requests/models.py", line 940, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 400 Client Error: Bad Request for url: http://substra-backend.node-1.com/objective/

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "scripts/add_dataset_objective.py", line 144, in <module>
    }, exist_ok=True)['pkhash']
  File "/home/zen/.local/lib/python3.7/site-packages/substra/sdk/client.py", line 41, in wrapper
    return f(*args, **kwargs)
  File "/home/zen/.local/lib/python3.7/site-packages/substra/sdk/client.py", line 323, in add_objective
    res = self._add(assets.OBJECTIVE, data, files=files, exist_ok=exist_ok)
  File "/home/zen/.local/lib/python3.7/site-packages/substra/sdk/client.py", line 164, in _add
    **requests_kwargs)
  File "/home/zen/.local/lib/python3.7/site-packages/substra/sdk/rest_client.py", line 261, in add
    return self._add(name, exist_ok=exist_ok, **request_kwargs)
  File "/home/zen/.local/lib/python3.7/site-packages/substra/sdk/rest_client.py", line 237, in _add
    return self.request('post', name, **request_kwargs)
  File "/home/zen/.local/lib/python3.7/site-packages/substra/sdk/utils.py", line 170, in wrapper
    return f(*args, **kwargs)
  File "/home/zen/.local/lib/python3.7/site-packages/substra/sdk/rest_client.py", line 192, in request
    **request_kwargs,
  File "/home/zen/.local/lib/python3.7/site-packages/substra/sdk/rest_client.py", line 170, in _request
    return self.__request(request_name, url, **request_kwargs)
  File "/home/zen/.local/lib/python3.7/site-packages/substra/sdk/rest_client.py", line 138, in __request
    raise exceptions.InvalidRequest.from_request_exception(e)
substra.sdk.exceptions.InvalidRequest: 400 Client Error: Bad Request for url: http://substra-backend.node-1.com/objective/

I get the same error with the latest commit of this branch with the command : sudo python3 scripts/add_dataset_objective.py
Adding dataset...
Requests error status 400: {"message":[{"name":["This field may not be null."]}],"pkhash":"c1d9b42d9538f825109c38c4f599d47391347d198a0770331ba790b7ebcfaa40"}
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/dist-packages/substra/sdk/rest_client.py", line 126, in __request
    r.raise_for_status()
  File "/usr/lib/python3/dist-packages/requests/models.py", line 940, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 400 Client Error: Bad Request for url: http://substra-backend.node-1.com/data_manager/

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "scripts/add_dataset_objective.py", line 95, in <module>
    dataset_key = client.add_dataset(DATASET, exist_ok=True)['pkhash']
  File "/usr/local/lib/python3.8/dist-packages/substra/sdk/client.py", line 41, in wrapper
    return f(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/substra/sdk/client.py", line 288, in add_dataset
    res = self._add(assets.DATASET, data, files=files, exist_ok=exist_ok)
  File "/usr/local/lib/python3.8/dist-packages/substra/sdk/client.py", line 160, in _add
    return self.client.add(
  File "/usr/local/lib/python3.8/dist-packages/substra/sdk/rest_client.py", line 261, in add
    return self._add(name, exist_ok=exist_ok, **request_kwargs)
  File "/usr/local/lib/python3.8/dist-packages/substra/sdk/rest_client.py", line 237, in _add
    return self.request('post', name, **request_kwargs)
  File "/usr/local/lib/python3.8/dist-packages/substra/sdk/utils.py", line 170, in wrapper
    return f(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/substra/sdk/rest_client.py", line 189, in request
    response = self._request(
  File "/usr/local/lib/python3.8/dist-packages/substra/sdk/rest_client.py", line 170, in _request
    return self.__request(request_name, url, **request_kwargs)
  File "/usr/local/lib/python3.8/dist-packages/substra/sdk/rest_client.py", line 138, in __request
    raise exceptions.InvalidRequest.from_request_exception(e)
substra.sdk.exceptions.InvalidRequest: 400 Client Error: Bad Request for url: http://substra-backend.node-1.com/data_manager/: [{'name': ['This field may not be null.']}]

I got the same error with this command : sudo python3 scripts/add_dataset_objective.py
Adding dataset...
Requests error status 400: {"message":[{"name":["This field may not be null."]}],"pkhash":"c1d9b42d9538f825109c38c4f599d47391347d198a0770331ba790b7ebcfaa40"}
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/dist-packages/substra/sdk/rest_client.py", line 126, in __request
    r.raise_for_status()
  File "/usr/lib/python3/dist-packages/requests/models.py", line 940, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 400 Client Error: Bad Request for url: http://substra-backend.node-1.com/data_manager/

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "scripts/add_dataset_objective.py", line 95, in <module>
    dataset_key = client.add_dataset(DATASET, exist_ok=True)['pkhash']
  File "/usr/local/lib/python3.8/dist-packages/substra/sdk/client.py", line 41, in wrapper
    return f(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/substra/sdk/client.py", line 288, in add_dataset
    res = self._add(assets.DATASET, data, files=files, exist_ok=exist_ok)
  File "/usr/local/lib/python3.8/dist-packages/substra/sdk/client.py", line 160, in _add
    return self.client.add(
  File "/usr/local/lib/python3.8/dist-packages/substra/sdk/rest_client.py", line 261, in add
    return self._add(name, exist_ok=exist_ok, **request_kwargs)
  File "/usr/local/lib/python3.8/dist-packages/substra/sdk/rest_client.py", line 237, in _add
    return self.request('post', name, **request_kwargs)
  File "/usr/local/lib/python3.8/dist-packages/substra/sdk/utils.py", line 170, in wrapper
    return f(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/substra/sdk/rest_client.py", line 189, in request
    response = self._request(
  File "/usr/local/lib/python3.8/dist-packages/substra/sdk/rest_client.py", line 170, in _request
    return self.__request(request_name, url, **request_kwargs)
  File "/usr/local/lib/python3.8/dist-packages/substra/sdk/rest_client.py", line 138, in __request
    raise exceptions.InvalidRequest.from_request_exception(e)
substra.sdk.exceptions.InvalidRequest: 400 Client Error: Bad Request for url: http://substra-backend.node-1.com/data_manager/: [{'name': ['This field may not be null.']}]

I get the same error with the latest commit of this branch with the command : sudo python3 scripts/add_dataset_objective.py
Adding dataset...
Requests error status 400: {"message":[{"name":["This field may not be null."]}],"pkhash":"c1d9b42d9538f825109c38c4f599d47391347d198a0770331ba790b7ebcfaa40"}
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/dist-packages/substra/sdk/rest_client.py", line 126, in __request
    r.raise_for_status()
  File "/usr/lib/python3/dist-packages/requests/models.py", line 940, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 400 Client Error: Bad Request for url: http://substra-backend.node-1.com/data_manager/

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "scripts/add_dataset_objective.py", line 95, in <module>
    dataset_key = client.add_dataset(DATASET, exist_ok=True)['pkhash']
  File "/usr/local/lib/python3.8/dist-packages/substra/sdk/client.py", line 41, in wrapper
    return f(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/substra/sdk/client.py", line 288, in add_dataset
    res = self._add(assets.DATASET, data, files=files, exist_ok=exist_ok)
  File "/usr/local/lib/python3.8/dist-packages/substra/sdk/client.py", line 160, in _add
    return self.client.add(
  File "/usr/local/lib/python3.8/dist-packages/substra/sdk/rest_client.py", line 261, in add
    return self._add(name, exist_ok=exist_ok, **request_kwargs)
  File "/usr/local/lib/python3.8/dist-packages/substra/sdk/rest_client.py", line 237, in _add
    return self.request('post', name, **request_kwargs)
  File "/usr/local/lib/python3.8/dist-packages/substra/sdk/utils.py", line 170, in wrapper
    return f(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/substra/sdk/rest_client.py", line 189, in request
    response = self._request(
  File "/usr/local/lib/python3.8/dist-packages/substra/sdk/rest_client.py", line 170, in _request
    return self.__request(request_name, url, **request_kwargs)
  File "/usr/local/lib/python3.8/dist-packages/substra/sdk/rest_client.py", line 138, in __request
    raise exceptions.InvalidRequest.from_request_exception(e)
substra.sdk.exceptions.InvalidRequest: 400 Client Error: Bad Request for url: http://substra-backend.node-1.com/data_manager/: [{'name': ['This field may not be null.']}]

Thank you for this input @Fabien-GELUS !

Thank you guys!


feat: update ComplexFilterPushdown()

Also bump the ddb submodule version to 10ea4832d3 (version of today).

@samansmink I'm not familiar with the procedure around bumping ddb version here.. looks like it's also linked to other extensions (which require patches)... I hope the code change works.

@stephaniewang526 Current feature branch of delta should be up to date with latest main


Fields output.log_contains and input.encoded_data described in specs not working

Hello,
I am working on the ModSecurity Regression Test Set.
Found this issues while working on owasp-modsecurity/MRTS#6 & owasp-modsecurity/MRTS#12 .
I noticed that log_contains and no_log_contains checks have no effect on test results while still being described in the latest yaml schema specs. It seems that fields log.match_regex and log.no_match_regex replaced them entirely ?
I also noticed that while input.encoded_request worked fine, input.encoded_data had no effect on the generated request used in a test case. It is also still described in the specs.
Here are the files to demonstrate the issue (there is yaml tests using the fields, the rules I used, a commented pcapng showing the requests send by go-ftw):  issue-go-ftw-log-contains-encoded-data.zip
Output I get when running these tests:
🚀 Running go-ftw!
👉 executing tests in file DEMO_ENCODED_DATA.yaml
	running 100001-1: ✔ passed in 8.397749ms (RTT 52.272652ms)
	running 100001-2: 💥 100001-2 failed in 7.556905ms (RTT 51.998539ms)
	running 100001-3: 💥 100001-3 failed in 7.536512ms (RTT 52.249954ms)
👉 executing tests in file DEMO_LOG_CONTAINS.yaml
	running 100000-1: ✔ passed in 7.921806ms (RTT 52.490335ms)
	running 100000-2: 💥 100000-2 failed in 7.079077ms (RTT 51.914455ms)
➕ run 5 total tests in 38.492049ms
⏭ skipped 0 tests
👎 3 test(s) failed to run: ["100001-2" "100001-3" "100000-2"]
Error: failed 3 tests
sebitosh@lp:~$ go-ftw -v
ftw version 1.1.2
commit: 2c1ba3070af090cbdba6603a639245f8cb4b2108
built at: 2024-11-27T06:01:26Z
built by: goreleaser

Thanks for the report @Sebitosh. log_contains and no_log_contains are still supported. There are actually two replacements, one is [no_]match_regex, the other is [no_]expect_ids.
I'll check your test files in a second.
encoded_data is indeed missing. I'll fix that.

Well, you're also right about [no_]log_contains. I'll fix those too.


Display static and dynamic blocks for behavior blocks

Currently blocking curriculum team
For this task: https://codedotorg.atlassian.net/browse/STAR-289
This change allows for the behavior category in the toolbox to display both static blocks, like "{sprite} begins" and dynamic blocks, like "patrolling".

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@84d4b91). Click here to learn what that means.
The diff coverage is 0%.

@@            Coverage Diff            @@
##             master     #174   +/-   ##
=========================================
  Coverage          ?   55.71%
=========================================
  Files             ?       96
  Lines             ?    11495
  Branches          ?        0
=========================================
  Hits              ?     6404
  Misses            ?     5091
  Partials          ?        0

Impacted Files
Coverage Δ

core/ui/block_space/flyout.js
61.62% <0%> (ø)

Continue to review full report at Codecov.

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

Will this change any existing level toolboxes?

@ajpal I could look into that possibility. If any levelbuilders have put static blocks into the behaviors category previously (these blocks wouldn't have been displayed), the blocks would not be displayed. I'll check with the curriculum team on Monday morning to see if that is something that might happen.


chore(main): release 1.13.4

:robot: I have created a release beep boop
1.13.4 (2023-04-03)
Bug Fixes

add node v19 support (previous commit didn't rigger automation) (be7b368)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/quirrel-dev/quirrel/releases/tag/v1.13.4 :sunflower:


syn_i and syn_j crash when using --syn-sdc

When I use the --syn-sdc on one of my datasets, I get the following error for all the run of one out of 9 subjects (the rest finishes without errors):
Node: fmriprep_wf.single_subject_02_wf.func_preproc_task_binocularrivalry_run_01_wf.nonlinear_sdc_wf.syn_i
Working directory: /scratch/wf/fmriprep_wf/single_subject_02_wf/func_preproc_task_binocularrivalry_run_01_wf/nonlinear_sdc_wf/syn_i

Node inputs:

args = <undefined>
collapse_output_transforms = True
convergence_threshold = [1e-08, 1e-08]
convergence_window_size = [20, 10]
dimension = 3
environ = {'NSLOTS': '1'}
fixed_image = ['/scratch/wf/fmriprep_wf/single_subject_02_wf/func_preproc_task_binocularrivalry_run_01_wf/nonlinear_sdc_wf/t1_2_ref/highres001_BrainExtractionBrain_inv_trans.nii.gz']
fixed_image_mask = <undefined>
fixed_image_masks = ['NULL', '/scratch/wf/fmriprep_wf/single_subject_02_wf/func_preproc_task_binocularrivalry_run_01_wf/nonlinear_sdc_wf/threshold_atlas/fmap_atlas_trans_maths.nii.gz']
float = True
ignore_exception = False
initial_moving_transform = <undefined>
initial_moving_transform_com = <undefined>
initialize_transforms_per_stage = False
interpolation = Linear
interpolation_parameters = <undefined>
invert_initial_moving_transform = <undefined>
metric = ['Mattes', 'CC']
metric_item_trait = <undefined>
metric_stage_trait = <undefined>
metric_weight = [1.0, 1.0]
metric_weight_item_trait = <undefined>
metric_weight_stage_trait = <undefined>
moving_image = ['/scratch/wf/fmriprep_wf/single_subject_02_wf/func_preproc_task_binocularrivalry_run_01_wf/bold_hmc_wf/enhance_and_skullstrip_bold_wf/apply_mask/uni_masked.nii.gz']
moving_image_mask = <undefined>
moving_image_masks = <undefined>
num_threads = 1
number_of_iterations = [[100, 50], [20, 10]]
output_inverse_warped_image = <undefined>
output_transform_prefix = ants_susceptibility
output_warped_image = True
radius_bins_item_trait = <undefined>
radius_bins_stage_trait = <undefined>
radius_or_number_of_bins = [56, 5]
restore_state = <undefined>
restrict_deformation = [[1, 0, 0], [1, 0, 0]]
sampling_percentage = <undefined>
sampling_percentage_item_trait = <undefined>
sampling_percentage_stage_trait = <undefined>
sampling_strategy = <undefined>
sampling_strategy_item_trait = <undefined>
sampling_strategy_stage_trait = <undefined>
save_state = <undefined>
shrink_factors = [[2, 1], [1, 1]]
sigma_units = ['vox', 'vox']
smoothing_sigmas = [[1.0, 0.0], [1.0, 0.0]]
terminal_output = stream
transform_parameters = [(0.8, 2.0, 2.0), (0.8, 2.0, 2.0)]
transforms = ['SyN', 'SyN']
use_estimate_learning_rate_once = [True, True]
use_histogram_matching = [True, True]
verbose = <undefined>
winsorize_lower_quantile = 0.001
winsorize_upper_quantile = 1.0
write_composite_transform = False

Traceback (most recent call last):
  File "/usr/local/miniconda/lib/python3.6/site-packages/niworkflows/nipype/pipeline/plugins/multiproc.py", line 52, in run_node
    result['result'] = node.run(updatehash=updatehash)
  File "/usr/local/miniconda/lib/python3.6/site-packages/niworkflows/nipype/pipeline/engine/nodes.py", line 372, in run
    self._run_interface()
  File "/usr/local/miniconda/lib/python3.6/site-packages/niworkflows/nipype/pipeline/engine/nodes.py", line 482, in _run_interface
    self._result = self._run_command(execute)
  File "/usr/local/miniconda/lib/python3.6/site-packages/niworkflows/nipype/pipeline/engine/nodes.py", line 613, in _run_command
    result = self._interface.run()
  File "/usr/local/miniconda/lib/python3.6/site-packages/niworkflows/nipype/interfaces/base.py", line 1083, in run
    runtime = self._run_wrapper(runtime)
  File "/usr/local/miniconda/lib/python3.6/site-packages/niworkflows/nipype/interfaces/base.py", line 1757, in _run_wrapper
    runtime = self._run_interface(runtime)
  File "/usr/local/miniconda/lib/python3.6/site-packages/niworkflows/nipype/interfaces/base.py", line 1791, in _run_interface
    self.raise_exception(runtime)
  File "/usr/local/miniconda/lib/python3.6/site-packages/niworkflows/nipype/interfaces/base.py", line 1715, in raise_exception
    **runtime.dictcopy()))
RuntimeError: Command:
antsRegistration --collapse-output-transforms 1 --dimensionality 3 --float 1 --initialize-transforms-per-stage 0 --interpolation Linear --output [ ants_susceptibility, ants_susceptibility_Warped.nii.gz ] --transform SyN[ 0.8, 2.0, 2.0 ] --metric Mattes[ /scratch/wf/fmriprep_wf/single_subject_02_wf/func_preproc_task_binocularrivalry_run_01_wf/nonlinear_sdc_wf/t1_2_ref/highres001_BrainExtractionBrain_inv_trans.nii.gz, /scratch/wf/fmriprep_wf/single_subject_02_wf/func_preproc_task_binocularrivalry_run_01_wf/bold_hmc_wf/enhance_and_skullstrip_bold_wf/apply_mask/uni_masked.nii.gz, 1, 56 ] --convergence [ 100x50, 1e-08, 20 ] --smoothing-sigmas 1.0x0.0vox --shrink-factors 2x1 --use-estimate-learning-rate-once 1 --use-histogram-matching 1 --restrict-deformation 1x0x0 --masks [ NULL, NULL ] --transform SyN[ 0.8, 2.0, 2.0 ] --metric CC[ /scratch/wf/fmriprep_wf/single_subject_02_wf/func_preproc_task_binocularrivalry_run_01_wf/nonlinear_sdc_wf/t1_2_ref/highres001_BrainExtractionBrain_inv_trans.nii.gz, /scratch/wf/fmriprep_wf/single_subject_02_wf/func_preproc_task_binocularrivalry_run_01_wf/bold_hmc_wf/enhance_and_skullstrip_bold_wf/apply_mask/uni_masked.nii.gz, 1, 5 ] --convergence [ 20x10, 1e-08, 10 ] --smoothing-sigmas 1.0x0.0vox --shrink-factors 1x1 --use-estimate-learning-rate-once 1 --use-histogram-matching 1 --restrict-deformation 1x0x0 --masks [ /scratch/wf/fmriprep_wf/single_subject_02_wf/func_preproc_task_binocularrivalry_run_01_wf/nonlinear_sdc_wf/threshold_atlas/fmap_atlas_trans_maths.nii.gz, NULL ] --winsorize-image-intensities [ 0.001, 1.0 ]  --write-composite-transform 0
Standard output:

Standard error:
 file NULL does not exist .
 file NULL does not exist .
 file NULL does not exist .
/tmp/ANTs/build/ITKv4/Modules/ThirdParty/VNL/src/vxl/core/vnl/algo/vnl_svd.hxx: suspicious return value (2) from SVDC
/tmp/ANTs/build/ITKv4/Modules/ThirdParty/VNL/src/vxl/core/vnl/algo/vnl_svd.hxx: M is 2x2
M = [ ...
            -nan             -nan
            -nan             -nan  ]

Return code: 1
Interface Registration failed to run.

You're running in Singularity? I wonder if there's a conflicting version of ANTs in your environment. Singularity does a kind of minimal masking of the environment, so it's possible that the native installation of ANTs is being run instead of the version in the container.

I'm running under Singularity, but I have no ANTS binaries in my PATH. I sent you an email with the data, so you can maybe have a look if you have time.

@effigies is that 'NULL' in
fixed_image_masks = ['NULL', '/scratch/wf/fmriprep_wf/single_subject_02_wf/func_preproc_task_binocularrivalry_run_01_wf/nonlinear_sdc_wf/threshold_atlas/fmap_atlas_trans_maths.nii.gz']

acceptable?

Weird I never got the email from you @Gilles86

@oesteban That's expected (and actually hard-coded), but pre-2.2.0 it would have caused issues. That's why I at first thought that the wrong version of ANTs might be being called.

I was able to reproduce this issue with the provided data.


shasum error

cd project/

-bash: shasum: command not found

whoa

fix it on centos
yum install perl-Digest-SHA -y

Sounds like an easy addition to a Makefile. Make a PRs!

Perhaps fallback to md5 or even cksum :neckbeard: if none of the sha are available.

@pc10201 What OS and shell do you use?

Closing due to missing information and age of issue. sha1sum is in GNU coreutils and BusyBox, so maybe it's some issue related to old / ancient versions of CentOS or which


WebSocketClient - Unrecognized event: APPLICATION_COMMAND_PERMISSIONS_UPDATE

General Troubleshooting

[X] I have checked for similar issues.
[X] I have updated to the [4.4.0_350].
[X] I have checked the branches or the maintainers' PRs for upcoming bug fixes.

Bug Report
I an updating my SlashCommands the Permissions and I get in the DEBUG Log following Message:
[DEBUG | JDA MainWS-ReadThread] 2021-11-28 20:10:35,422 net.dv8tion.jda.internal.requests.WebSocketClient - Unrecognized event:
{"op":0,"s":8,"t":"APPLICATION_COMMAND_PERMISSIONS_UPDATE","d":{"permissions":[{"permission":true,"id":"911591572452302870","type":1}],"guild_id":"570406198927425537","id":"914586609377226762","application_id":"764947804236939305"}}

The whole log is attached:
2021-11-28 +0100 20_10_29.log
Code Example or Reproduction Steps
Here the two container to understand my method:
        //commandData holds a map where a CommandData is mapped for a Permission to handle which roles get access
        //on these commands. I am currently trying to work here with subcommands.
        //CommandData.setDefaultEnabled(false); is the method I use for this
        commandData = new HashMap<Permission, CommandData>();

        //commandDataPublic holds CommandDatas which are open for every user.
        commandDataPublic = new ArrayList<CommandData>();

Here the whole Method I am calling and where I think the problem gets queued:
public void registerCommandsOnGuild(String guildId)
    {

        Guild guild = Main.getCurrentJDA().getGuildById(guildId);
        for (int i = 0; i < commandDataPublic.size(); i++)
        {
            guild.upsertCommand(commandDataPublic.get(i)).queue();
            Main.getLog().debug("Queued upsert for SlashCommand: " + commandDataPublic.get(i).getName());
        }

        if (commandData.size() <= 0)
        {
            Main.getLog().debug("variable: commandData.size() <= 0");
            return;
        }

        for (Permission permission : commandData.keySet())
        {
            if (guild.getRoles().size() <= 0)
            {
                Main.getLog().debug("guild has no roles.");
                return;
            }

            List<Role> list = new ArrayList<Role>();
            for (Role role : guild.getRoles())
            {
                if (role.hasPermission(permission))
                {
                    list.add(role);
                }
            }

            guild.upsertCommand(commandData.get(permission)).queue(command ->
            {
                for (Role role : list)
                {
                    command.updatePrivileges(guild, CommandPrivilege.enable(role)).queue();
                }
            });
            Main.getLog().debug("Queued upsert for SlashCommand: " + commandData.get(permission).getName());
        }

        Main.getLog().info("Finished register slashCommand on guild: " + guildId);
    }

Important to know is that I call this method AFTER I build the JDA.
Code for JDABuilder or DefaultShardManagerBuilder Used
    private static JDABuilder createBuilder()
    {
        JDABuilder jdaBuilder = null;

        jdaBuilder = JDABuilder.createDefault(bot.getToken());
        jdaBuilder.setMemberCachePolicy(MemberCachePolicy.ALL);
        jdaBuilder.enableIntents(GatewayIntent.GUILD_MEMBERS);
        jdaBuilder.setAutoReconnect(true);
        jdaBuilder.setStatus(OnlineStatus.ONLINE);
        jdaBuilder.setActivity(Activity.listening("dir"));

        return jdaBuilder;
    }

This is a debug log, not a warn, I don't think this is a bug.
I have already encountered this but this has no purpose - it's not an event for bots

@freya022 I wanted to provides this to the devs, because I dont know if this is something they want to know or not. Maybe the devs wants handle this event also in the future. I just wanted to let them know

This is an undocumented event and not intended for bots. We don't implement features that are undocumented.


exit status 1 all the time?

For some reason I'm getting this when building all the examples:
acbuild/examples/apache  master ✗                                                                                30d ◒
▶ sudo ./build-apache.sh
Beginning build with an empty ACI
Setting name of ACI to example.com/apache
Adding dependency "quay.io/coreos/alpine-sh"
Running: [apk update]
run: exit status 1
Ending the build

Any idea as to why? No debug info for this :(

Using:
acbuild/examples/apache  master ✗                                                                                30d ◒  ⍉
▶ acbuild version
acbuild version v0.2.2-2-g46ad9c4
appc version 0.7.1+git

(master)

Related: https://github.com/appc/acbuild/issues/162
I'm aware of this as a problem, but have no clue what's causing it. I'll look into it soon.

@dgonyeo #162
Okay, so I think I may have found the problem.
If you build using:
acbuild --debug begin quay.io/listhub/alpine
it works fine (without adding dep)
but if you use it depending on the dep, it doesn't
acbuild --debug dep add quay.io/coreos/alpine-sh
any ideas?

That may mean that there's some issue with overlayfs, I'll see if I can investigate further.

FYI, I'm building on Debian 8 with 3.16.0-4-amd64.

I think I may have had a friend who saw a similar, if not identical issue also in Debian once. Guess it's time to spin up a Debian vm.

Figured it out: acbuild doesn't print a useful error message when the overlayfs kernel module isn't available.
At this line: https://github.com/appc/acbuild/blob/master/lib/run.go#L83 acbuild execs modprobe overlay, and if an error is encountered it's simply returned. A much better behavior would be to detect when the error is exit code 1, which implies that the module failed to load. I'll make a PR.
For specifically debian, I found this, which says that overlayfs didn't make it into Linux until 3.18. It also looks like you can get the newer kernel from experimental, so I'd suggest you give that a shot if you want to get acbuild working in debian.
Alternatively if you avoid builds with dependencies, acbuild won't need overlayfs. You can modify the apache example by removing the line:
acbuild --debug dep add quay.io/coreos/alpine-sh

and replacing the line:
acbuild --debug begin

with
acbuild --debug beginquay.io/coreos/alpine-sh

https://github.com/appc/acbuild/pull/171

Thanks :dancers: !


Add tests on Python 3.13 in GitHub Actions

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.57%. Comparing base (abbdf03) to head (80e19b6).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #119   +/-   ##
=======================================
  Coverage   99.57%   99.57%
=======================================
  Files          61       61
  Lines        1636     1636
  Branches       74       74
=======================================
  Hits         1629     1629
  Misses          5        5
  Partials        2        2

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