"Are you trying to mount a directory onto a file (or vice-versa)?" error when creating container

I'm trying to use this container for the first time and I'm running into an error after running "docker-compose up -d":

Creating unbound ... error

ERROR: for unbound  Cannot start service unbound: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: error during container init: error mounting "/home/pi/containers/pihole-new/unbound/unbound.conf" to rootfs at "/usr/local/unbound/unbound.conf": mount /home/pi/containers/pihole-new/unbound/unbound.conf:/usr/local/unbound/unbound.conf (via /proc/self/fd/6), flags: 0x5000: not a directory: unknown: Are you trying to mount a directory onto a file (or vice-versa)? Check if the specified host path exists and is the expected type
ERROR: for unbound  Cannot start service unbound: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: error during container init: error mounting "/home/pi/containers/pihole-new/unbound/unbound.conf" to rootfs at "/usr/local/unbound/unbound.conf": mount /home/pi/containers/pihole-new/unbound/unbound.conf:/usr/local/unbound/unbound.conf (via /proc/self/fd/6), flags: 0x5000: not a directory: unknown: Are you trying to mount a directory onto a file (or vice-versa)? Check if the specified host path exists and is the expected type

It looks like unbound.conf is being created as a directory instead of a file (and note that everything is using root:root instead of pi:pi):

pi@pi-eloh:~/containers/pihole-new $ ls -la unbound
total 28
drwxr-xr-x 7 root root 4096 Apr 27 22:16 .
drwxr-xr-x 5 pi   pi   4096 Apr 27 22:16 ..
drwxr-xr-x 2 root root 4096 Apr 27 22:16 conf.d
drwxr-xr-x 2 root root 4096 Apr 27 22:16 iana.d
drwxr-xr-x 3 root root 4096 Apr 27 22:16 log.d
drwxr-xr-x 2 root root 4096 Apr 27 22:16 unbound.conf
drwxr-xr-x 2 root root 4096 Apr 27 22:16 zones.d

If I manually remove unbound.conf, then touch it to create a file, I get an error for the log.d/unbound.log, which is also a directory and not a file:

Creating unbound ... error
ERROR: for unbound  Cannot start service unbound: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: error during container init: error mounting "/home/pi/containers/pihole-new/unbound/log.d/unbound.log" to rootfs at "/usr/local/unbound/log.d/unbound.log": mount /home/pi/containers/pihole-new/unbound/log.d/unbound.log:/usr/local/unbound/log.d/unbound.log (via /proc/self/fd/6), flags: 0x5000: not a directory: unknown: Are you trying to mount a directory onto a file (or vice-versa)? Check if the specified host path exists and is the expected type
ERROR: for unbound  Cannot start service unbound: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: error during container init: error mounting "/home/pi/containers/pihole-new/unbound/log.d/unbound.log" to rootfs at "/usr/local/unbound/log.d/unbound.log": mount /home/pi/containers/pihole-new/unbound/log.d/unbound.log:/usr/local/unbound/log.d/unbound.log (via /proc/self/fd/6), flags: 0x5000: not a directory: unknown: Are you trying to mount a directory onto a file (or vice-versa)? Check if the specified host path exists and is the expected type
ERROR: Encountered errors while bringing up the project.

Manually changing both of these directories to files allows the container to start.  I checked the Dockerfile and it's not obvious to me why these are being created as directories and not files.

It looks like the filesystem isn't mounted properly via fstab.
Could you provide your docker compose and fstab, please?

Here you go:

pi@pi-eloh:~/containers/pihole-new $ cat /etc/fstab
proc            /proc           proc    defaults          0       0
PARTUUID=1534a318-01  /boot           vfat    defaults          0       2
PARTUUID=1534a318-02  /               ext4    defaults,noatime  0       1
a swapfile is not a swap partition, no line here
use  dphys-swapfile swap[on|off]  for that

docker-compose.yml.txt

Can you show the output with ls -la of the unbound dir? Is the persistent volume mapped directly or via share? User and group 1000 needs permissions, too.

I believe the persistent volume in mapped directly.  How could I tell?
Here's the default directory structure on a fresh docker-compose up -d:
pi@pi-eloh:~/containers/pihole-new $ ls -laR unbound
unbound:
total 28
drwxr-xr-x 7 root root 4096 Apr 27 23:16 .
drwxr-xr-x 6 pi   pi   4096 Apr 27 23:16 ..
drwxr-xr-x 2 root root 4096 Apr 27 23:16 conf.d
drwxr-xr-x 2 root root 4096 Apr 27 23:16 iana.d
drwxr-xr-x 3 root root 4096 Apr 27 23:16 log.d
drwxr-xr-x 2 root root 4096 Apr 27 23:16 unbound.conf
drwxr-xr-x 2 root root 4096 Apr 27 23:16 zones.d

unbound/conf.d:
total 8
drwxr-xr-x 2 root root 4096 Apr 27 23:16 .
drwxr-xr-x 7 root root 4096 Apr 27 23:16 ..

unbound/iana.d:
total 8
drwxr-xr-x 2 root root 4096 Apr 27 23:16 .
drwxr-xr-x 7 root root 4096 Apr 27 23:16 ..

unbound/log.d:
total 12
drwxr-xr-x 3 root root 4096 Apr 27 23:16 .
drwxr-xr-x 7 root root 4096 Apr 27 23:16 ..
drwxr-xr-x 2 root root 4096 Apr 27 23:16 unbound.log

unbound/log.d/unbound.log:
total 8
drwxr-xr-x 2 root root 4096 Apr 27 23:16 .
drwxr-xr-x 3 root root 4096 Apr 27 23:16 ..

unbound/unbound.conf:
total 8
drwxr-xr-x 2 root root 4096 Apr 27 23:16 .
drwxr-xr-x 7 root root 4096 Apr 27 23:16 ..

unbound/zones.d:
total 8
drwxr-xr-x 2 root root 4096 Apr 27 23:16 .
drwxr-xr-x 7 root root 4096 Apr 27 23:16 ..

This is the output of my unbound directory:
total 7
drwxrwxrwx    2 1000     1000             0 Apr 25 17:02 .
drwxr-xr-x    5 root     root          4096 Feb  3 17:11 ..
drwxrwxrwx    2 1000     1000             0 Apr 24 14:00 conf.d
drwxrwxrwx    2 1000     1000             0 Apr 28 08:05 iana.d
drwxrwxrwx    2 1000     1000             0 Apr 28 07:42 log.d
-rwxrwxrwx    1 1000     1000           294 Jan  3  2022 unbound.conf
drwxrwxrwx    2 1000     1000             0 Dec 13 16:42 zones.d

Your folder permissions doesn't seem right. Can you try to fix them?

They're created as root when the unbound directory is created by running "docker-compose up".  I can manually change them, but I would have hoped that the Dockerfile would create them with the correct permissions.  Note that this doesn't help the fact that unbound.conf and log.d/unbound.conf are created as directories.
If I manually changed unbound.conf and log.d/unbound.conf to files instead of directories, the container starts without issues.
For a bit more context, a few folks are running into this using the instructions from https://www.xfelix.com/2020/09/pihole-unbound-docker-setup-on-raspberry-pi/ (see the comments section at the bottom of the page).

So it works if you set the correct permissions? The creation of the directories and setting of the permissions is done in the dockerfile, the _unbound user has the uid/gid 1000. Please add your "root" user to gid 1000 for testing.

It's quite common to fix permissions to make containers run, btw.

@roynuj, did you actually get it working. Also following that guide and just doesn’t work.

Please comment out the volume mappings in your compose file. Does the container start?

@madnuttah Yes, it starts right up with the volume mappings commented out.
@broomwalker I got the container started, but I haven't gotten unbound to serve DNS results to pihole yet.

Please use this command, uncomment your volumes and try to start the container again:
chown -r root:1000 /home/pi/unbound/

Same error starting the container.  unbound.conf is still a directory, not a file.

Please run the following command and try to start the container:
sudo wget https://raw.githubusercontent.com/madnuttah/unbound-docker/main/unbound/examples/usr/local/unbound/unbound.conf -P /home/pi/unbound/

Now log.d/unbound.log is problematic because it's still a directory.
pi@pi-eloh:~/containers/pihole-new $ mkdir unbound
pi@pi-eloh:~/containers/pihole-new $ sudo wget https://raw.githubusercontent.com/madnuttah/unbound-docker/main/unbound/root/usr/local/unbound/unbound.conf -P unbound/
--2023-04-28 19:51:50--  https://raw.githubusercontent.com/madnuttah/unbound-docker/main/unbound/root/usr/local/unbound/unbound.conf
Resolving raw.githubusercontent.com (raw.githubusercontent.com)... <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, ...
Connecting to raw.githubusercontent.com (raw.githubusercontent.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 52162 (51K) [text/plain]
Saving to: 'unbound/unbound.conf'

unbound.conf                                                                  100%[===============================================================================================================================================================================================>]  50.94K  --.-KB/s    in 0.009s

2023-04-28 19:51:50 (5.36 MB/s) - 'unbound/unbound.conf' saved [52162/52162]

pi@pi-eloh:~/containers/pihole-new $ sudo chown -R root:1000 unbound
pi@pi-eloh:~/containers/pihole-new $ ls -laR unbound
unbound:
total 60
drwxr-xr-x 2 root pi  4096 Apr 28 19:51 .
drwxr-xr-x 6 pi   pi  4096 Apr 28 19:51 ..
-rw-r--r-- 1 root pi 52162 Apr 28 19:51 unbound.conf
pi@pi-eloh:~/containers/pihole-new $ docker-compose up -d unbound
Removing unbound
Recreating c98ea9f9069d_unbound ... error

ERROR: for c98ea9f9069d_unbound  Cannot start service unbound: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: error during container init: error mounting "/home/pi/containers/pihole-new/unbound/log.d/unbound.log" to rootfs at "/usr/local/unbound/log.d/unbound.log": mount /home/pi/containers/pihole-new/unbound/log.d/unbound.log:/usr/local/unbound/log.d/unbound.log (via /proc/self/fd/6), flags: 0x5000: not a directory: unknown: Are you trying to mount a directory onto a file (or vice-versa)? Check if the specified host path exists and is the expected type

ERROR: for unbound  Cannot start service unbound: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: error during container init: error mounting "/home/pi/containers/pihole-new/unbound/log.d/unbound.log" to rootfs at "/usr/local/unbound/log.d/unbound.log": mount /home/pi/containers/pihole-new/unbound/log.d/unbound.log:/usr/local/unbound/log.d/unbound.log (via /proc/self/fd/6), flags: 0x5000: not a directory: unknown: Are you trying to mount a directory onto a file (or vice-versa)? Check if the specified host path exists and is the expected type
ERROR: Encountered errors while bringing up the project.
pi@pi-eloh:~/containers/pihole-new $ ls -laR unbound
unbound:
total 76
drwxr-xr-x 6 root pi    4096 Apr 28 19:52 .
drwxr-xr-x 6 pi   pi    4096 Apr 28 19:51 ..
drwxr-xr-x 2 root root  4096 Apr 28 19:52 conf.d
drwxr-xr-x 2 root root  4096 Apr 28 19:52 iana.d
drwxr-xr-x 3 root root  4096 Apr 28 19:52 log.d
-rw-r--r-- 1 root pi   52162 Apr 28 19:51 unbound.conf
drwxr-xr-x 2 root root  4096 Apr 28 19:52 zones.d

unbound/conf.d:
total 8
drwxr-xr-x 2 root root 4096 Apr 28 19:52 .
drwxr-xr-x 6 root pi   4096 Apr 28 19:52 ..

unbound/iana.d:
total 8
drwxr-xr-x 2 root root 4096 Apr 28 19:52 .
drwxr-xr-x 6 root pi   4096 Apr 28 19:52 ..

unbound/log.d:
total 12
drwxr-xr-x 3 root root 4096 Apr 28 19:52 .
drwxr-xr-x 6 root pi   4096 Apr 28 19:52 ..
**d**rwxr-xr-x 2 root root 4096 Apr 28 19:52 unbound.log

unbound/log.d/unbound.log:
total 8
drwxr-xr-x 2 root root 4096 Apr 28 19:52 .
drwxr-xr-x 3 root root 4096 Apr 28 19:52 ..

unbound/zones.d:
total 8
drwxr-xr-x 2 root root 4096 Apr 28 19:52 .
drwxr-xr-x 6 root pi   4096 Apr 28 19:52 ..
pi@pi-eloh:~/containers/pihole-new $

Please run sudo touch /home/pi/unbound/log.d/unbound.log
The container spins up?

If I remove the "./log.d/unbound.log/" directory created by the Dockerfile, then manually create the file and give it the correct permissions, yes, the container starts up.

I'll refine my documentation. If the issue is fixed, please close the issue.
Thanks for your patience and cooperation.

I'm here and glad to help if you need further assistance.

I can’t really say it’s resolved since it doesn’t work with the Dockerfile as is. Yes, we’ve identified a workaround. Is there a way I can help identify the root cause so a workaround isn’t needed?

That's no workaround, the files are there as they should be now.
Sure, there's more to do to make unbound resolve. When this particular issue is resolved by your side, please close it and open a new one to keep the issues clean.
Thanks. We'll get the rest running, too.

@roynuj I think the problem with the files occurs as soon as no mounted persistent volume is used, I didn't read about this file/directory problem until now as the most people use mounted volumes. I'll update my documentation. May @x-Felix will adapt his instructions, too.
I hope this fixes the problems @broomwalker had as well.

Hi @madnuttah , apologise to reply this closed issue. But I do see some potential issues with the file permissions.
When starts, it complains unbound.log permission issue. When dig some domain for testing, it complains root.key permission then keep restarting the service.
If I quote out the file path in conf file, line #4 of logging.conf, and line #2 of trust-anchor.conf , the unbound can start successfully without any issue.
Not sure whether those example conf duplicate with https://github.com/madnuttah/unbound-docker/blob/main/unbound/root/usr/local/unbound/unbound.conf that causing issue? I also have no idea why my existing running instance does not have those issue. It can be reproduced on a brand new implementation.

Compose won't copy files to persistent volumes, you have to put the files there first and fix permissions.
The default unbound.conf should always work, after this is verified, you can extract sections from the config and put it into separate confs.
How's the permissions for root.key? These should be set correctly already.
I guess you set up with the default conf first and afterwards structuring your configs.

@madnuttah I just test with the example configs. If map the iana.d volume and use the example config which pointing the root.key file path, the container will keep restarting once receive first query. And the log complains root.key-random characters cannot write to iana.d. If I quote out the root.key path from config file, the container can successfully start and create root.key to mapped iana.d without issue.

Can you download the root.key to iana.d and set permissions to test?

I did that. I created an initial root.key file in that folder with permission same as iana.d directory. Owned by pi:pi user. When container starts, after receiving first query. It complains the same error reading writing to iana.d/root.key and keeps restarting. When checking the iana.d directory, the root.key content gets updated, and permission changes to root:root.

Those permissions seem right to me. Can you put the root user in group 1000 please for testing?

I tested it again with the example config files. logging.conf and trust-anchor.conf has the log file and root.key path pointed.
The unbound/log.d is empty and unbound/iana.d only has root.zone.
I then run sudo chown -R root:1000 /home/pi/unbound/ . The file permission shows root:opc after the change.
Start the container without any issues. But notice, unbound/log.d/unbound.log and unbound/iana.d/root.key ownership is opc:opc . All other files remain ownership root:opc.

As I wrote earlier, this is a bit of a special setup as most people use persistent folders mapped from a NAS. The opc user has the gid 1000 which makes the container able to start due to correct permissions.

I tested it again with the example config files. logging.conf and trust-anchor.conf has the log file and root.key path pointed. The unbound/log.d is empty and unbound/iana.d only has root.zone. I then run sudo chown -R root:1000 /home/pi/unbound/ . The file permission shows root:opc after the change. Start the container without any issues. But notice, unbound/log.d/unbound.log and unbound/iana.d/root.key ownership is opc:opc . All other files remain ownership root:opc.

Hi everyone!
I know this issue is basically dead but recently I tried installing unbound on docker and I had the exact same problems as @x-Felix.
Example:
Permissions of “unbound/iana.d” before I ran the dig command (dig http://www.google.com @<IP_ADDRESS> -p 5335):
drwxr-xr-x 2 root pi 4096 Jan 16 02:16 .
drwxr-xr-x 6 root pi 4096 Jan 16 01:16 ..
-rw-r–r– 1 root pi 758 Jan 16 02:16 root.key
-rwxr-xr-x 1 root pi 2254606 Jan 15 19:10 root.zone
Permissions of “unbound/iana.d” after I ran the dig command:
drwxr-xr-x 2 root pi 4096 Jan 16 02:24 .
drwxr-xr-x 6 root pi 4096 Jan 16 01:16 ..
-rw-r–r– 1 root root 758 Jan 16 02:24 root.key
-rwxr-xr-x 1 root pi 2254606 Jan 15 19:10 root.zone
Is there any solution for this or still not?
Thank you in advance for your help

As this issue is closed, please open a new one, I'm glad to help.


Move example web3

@JustynaBroniszewska this move will done in another way. Because react can't have  dependencies on @universal-login/web3. So i will move example in @universal-login/web3 or just remove it.
Another option can be moving whole playground to example but i consider we should try move it to web3


x86: update mmustructs.h and x86_mmu.c to support long mode

arch/x86/include/ia32/mmustructs.h contains all the definitions for page tables. x86_mmu.c has the code for managing page tables.
Right now, this only implements page tables for PAE on 32-bit x86. Need to add support for 64-bit page tables as well. I think the following needs to be done:

X86_MMU_GET_* macros should take a pdpt pointer for PAE, and a pml4 pointer for long mode
When long mode enabled, need a definition for PDPTEs which specify a 1GB area, instead of a page directory. Need a container union similar to union x86_mmu_pde.
When long mode enabled, need a X86_MMU_GET_PDPT_ADDR() macro
Need some ifdefs for the different fields in all the page table structure when long mode is enabled
Need MMU_*_MASK macros for both types of PDPT, and PML4
Our new coding guidelines don't allow anonymous unions. Remove all the value fields in the table entry structures, and instead create some inline functions to convert table entries into u64_t values.
We have lots of APIs which take a PDPT but what they really need is the toplevel pointer to all the page tables, likely need to introduce some typedef

Also need to up-level x86_mmu.c so that it works in long mode.

@andrewboie I dunno how far along you are in this (if at all), but a shortcut here would be to assume everything we want to physically address is <= 512GB, and then use a "fixed" PML4 for each CPU and just change its first entry before "reloading" CR3.  I see little reason to expand the logic to 4 levels -- if we do want to use all 4, I'd suggest maybe we take advantage of the fact that page tables are recursive structures.

@andrewboie I dunno how far along you are in this (if at all), but a shortcut here would be to assume everything we want to physically address is <= 512GB, and then use a "fixed" PML4 for each CPU and just change its first entry before "reloading" CR3. I see little reason to expand the logic to 4 levels -- if we do want to use all 4, I'd suggest maybe we take advantage of the fact that page tables are recursive structures.

I'm just starting, but this sounds like a good idea. I'm currently adding structure definitions and ifdefs to mmustructs.h for long mode and cleaning up some cruft.

@andrewboie I dunno how far along you are in this (if at all), but a shortcut here would be to assume everything we want to physically address is <= 512GB, and then use a "fixed" PML4 for each CPU and just change its first entry before "reloading" CR3. I see little reason to expand the logic to 4 levels -- if we do want to use all 4, I'd suggest maybe we take advantage of the fact that page tables are recursive structures.

I didn't get this far. Right now we just generate a single get of page tables (or two if KPTI is turned on).
Back when I was implementing per-thread page tables for 32-bit, I considered having a master PDPT and just updating pointers to the first PDE when switching between threads. I decided not to do it at the time because it would make the switching more complex than a simple cr3 update.


add filter and pagination reset when status is changed

Description
added a reset function for status change to reset pagination and filters when the node status selected is changed
Changes

statusReset function:
- watches for change in filterOptions.status and resets pagination page and size along with filter inputs

Related Issues
#1604

not all filters reset the pagination
https://github.com/threefoldtech/tfgrid-sdk-ts/assets/41957921/cdf6ef79-64fc-4671-98d6-a5c1b975d1f0

not all filters reset the pagination
reset_pagination.mp4

the issue attached was for node status only but will also implement it for other filters. thanks


Added send to transfer list option by generalising details panel button press

Made a generic panel button function, controlled by param. Helped with the send to transfer list, keeps it DRY fixes #4

@martellaj no problem, I am the very same. I've made those changes now 👍

Thanks, @Huskie! Merged.


508 compliance fixes

Fixed 9 508 errors.

please resolve the conflicts

conflicts resolved.

@CuppaLabs can we merge the changes if we are good with the PR?.


v2.2.0

Includes a new init command, a bug fix, and various dev tooling updates.

Moving this back into draft until #83 is included

This has been updated to include all changes on main, and the changelog has been updated.

A new version of the package @metamask/auto-changelog (2.2.0) was published at 2021-05-26T15:22:53.401Z from
[redacted]. The shasum of this package was ccc5f8d2f056793e9736445c0444596d4fcd6ffd.


Handle BIP44 & P2PKH wallets

In the meantime, the tool should issue a warning if the wallet is a non-BIP45 wallet (checking the wallet#derivationStrategy attribute being either equal to 'BIP45' or undefined).

Update with this control. Thanks!

done!


Clear unused keys when receiving msg.ui_update.options, to prevent sending old keys on next submit

Description
When receiving a new message with new form elements to update in the form, delete old/unused values to prevent sending them on the next form submit
Related Issue(s)
Closes #343
Checklist

[X ] I have read the contribution guidelines
[ ] Suitable unit/system level tests have been added and they pass
[ ] Documentation has been updated

[ ] Upgrade instructions
[ ] Configuration details
[ ] Concepts

[ ] Changes flowforge.yml?

[ ] Issue/PR raised on FlowFuse/helm to update ConfigMap Template
[ ] Issue/PR raised on FlowFuse/CloudProject to update values for Staging/Production

Labels

[ ] Includes a DB migration? -> add the area:migration label

Updated PR, one error condition was missing, when updating the UI-Form Elements via Node-Red editor and not performing a Full Deploy.


CommonLib mod fails to load in certain conditions

Mod Version
2.3.0 - 2.3.7 (maybe older, too)
CommonLib Version
see above
Game Version
1.18.8-1.19.7
OS
Windows, Linux
Play mode
Multiplayer
Problem
When connecting to a server that does not have CommonLib installed, the mod fails to load on the client.
Reproduction steps:

setup vanilla server (no mods)
add CommonLib to the client
add any mod that requires CommonLib to the client (I used Extra Overlays)
connect client to the vanilla server and witness no overlays

Analysis
I believe this happens because the mod defaults to universal side with both client and server being required in the modinfo.json file.
Solution
The problem is fixed when I edit modinfo.json to tell it the server is not required. The defaults of universal side and client required are still in effect.
"requiredOnServer": false

Note: I left side as universal and client as required so that it still behaves as expected in every scenario. Servers will still tell clients it is required if it is installed on servers. This change just makes it so client can run it without it also being on the server.
Logs
Client log has these errors (not on startup, only when connecting to the server):
...
2.5.2024 15:20:49 [Client Notification] Found 5 mods (1 disabled)
2.5.2024 15:20:49 [Client Error] [extraoverlays] Could not resolve some dependencies:
2.5.2024 15:20:49 [Client Error] [extraoverlays]<EMAIL_ADDRESS>- Missing
2.5.2024 15:20:49 [Client Notification] Mods, sorted by dependency: game, creative, survival, extraoverlays
...
2.5.2024 15:20:55 [Client Warning] ===============================================================
2.5.2024 15:20:55 [Client Warning] (x_x) Captured 2 issues during startup:
2.5.2024 15:20:55 [Client Error] [extraoverlays] Could not resolve some dependencies:
2.5.2024 15:20:55 [Client Error] [extraoverlays]<EMAIL_ADDRESS>- Missing
2.5.2024 15:20:55 [Client Warning] ===============================================================
...

Mods
Extra Overlays v1.4.0

Screenshots
n/a

Will this really work now? I've already tried adding requiredOn* twice to fix this problem, but it only ended up causing more problems. For example, the lib was not downloaded for other Universal mods

I'll try this. Hopefully if it's only RequiredOnServer: false it will work correctly


Add SPM testTarget and more unit tests

Hey,
This PR add some more unit tests and also adds a testTarget to SPM.
Please let me know there are any more required changes.
Thanks

"Executed 21 tests, with 0 failures (0 unexpected) in 0.150 (0.151) seconds" 😍
Perfect! Thanks @4brunu!


Package request: niri

Project description
A scrollable-tiling Wayland compositor.
Metadata

homepage URL: https://github.com/YaLTeR/niri
source URL: https://github.com/YaLTeR/niri
license: GPL-3.0
platforms: linux, ?

I just found out about Niri today, and this looks like just what I am looking for!

Added in #288861

I will look at creating a module for the package as well. I've never made one before, so let's see how it goes :p

@IogaMaster you might wanna check out the niri-flake repo linked above for some inspiration on doing a niri module.
(@sodiboo: looks like you hadn't specified a license there yet. :see_no_evil:)

@IogaMaster you might wanna check out the niri-flake repo linked above for some inspiration on doing a niri module. (@sodiboo: looks like you hadn't specified a license there yet. 🙈)

in the works. (i've asked in matrix channel, just to be sure i don't break niri's GPL when picking one)
i'll add that some inspiration for my own module comes from the Hyprland modules (nixos and home-manager alike), and those (as well as other established compositors) might be more relevant to nixpkgs rules? dunno really. my module is also missing some things, that i haven't fixed yet:

vscode keyring? is this part of the compositor module even? currently, i just use the "weaker encryption" option because i didn't wanna deal with it anymore
polkit? what do
[unidentified issues]


Why does config/llama7.yaml not use OlmoLlamaBlock?

A sample "LLaMA-like" config is provided, but it doesn't use the OlmoLlamaBlock BlockType defined in model/olmo.py. Why is that?
https://github.com/allenai/OLMo/blob/97296e610c24dd1bb098ec64660dfcafcba62d24/configs/llama7.yaml#L21

The remaining changes to make that config more Llama-like in behavior are in https://github.com/allenai/OLMo/pull/358. I don't remember why we didn't merge that change, but the throughput was very poor due to how our implementation tried to get identical results in the attention mechanism.

I apologize for our delay in response. In order to help surface current, unresolved issues, we are closing tickets prior to February 29.
Please reopen your ticket if you are continuing to experience this issue.
Thank you!


[5.2] Fix seeJsonStructure which wasn't returning $this.

Fix seeJsonStructure in Illuminate\Foundation\Testing\Concerns\MakesHttpRequests which is missing return $this and breaks chaining.
Not sure how to add a test unit for a piece of the testing framework, but this snippet illustrates the type of test I was writing which didn't work because seeJsonStructure didn't return $this.
<?php

use Illuminate\Foundation\Testing\WithoutMiddleware;

class ExampleTest extends TestCase
{

    use WithoutMiddleware;

    public function test_seeJsonStructure()
    {
        $data = ['foo' => 'bar', 'biz' => 'baz'];
        $this->json('POST', "test/uri", $data)
            ->seeJsonStructure(['foo', 'biz'])
            ->seeJson(['foo' => 'bar']);
    }
}

Doesn't this need backporting to 5.1 too?


Added Wireshark

Related Issue
Closes #845

Description

Network protocol analyzer for network troubleshooting and analysis.
Screenshots

Checklist

[x] My code adheres to the established style guidelines of the project.
[x] I have included comments in areas that may be difficult to understand.
[x] My changes have not introduced any new warnings.
[x] I have conducted a self-review of my code.

Hello, contributors! This is Jason speaking. Due to some workflow issues, we will be closing all the PRs today. Therefore, I kindly request all of you to create new PRs tomorrow, as the currently open ones will be closed by our team. I apologize for any inconvenience caused. Also, please make sure to check the titles when creating the PRs. You can refer to the README.md for more details. Thank you.


Let crashkernel load modules

This PR fixes the inability to collect kernel dumps for the /persist formatted as ZFS and consists of 2 commits:

rootfs: move modprobe to the 000-mod-params initd script
Let's have modules load in a proper place: /etc/init.d/000-mod-params
scripts, which is responsible for loading modules and providing
parameters.
The other motivation is to have fine control over modules and for
example do not load modules (except ZFS) for the crashkernel.

000-kexec: remove 'nomodule' parameter, allows crashkernel to load modules
Patch removes 'nomodule' kernel parameter, which lets crashskernel
to load modules. This is utterly needed for interacting with the /persist
formatted as ZFS. ZFS is built separately out-of-tree as a module, so
has to be loaded explicitly by the crashkernel.
The majority of other modules are excluded from loading for the
crashkernel, see the 000-mod-params script.

This should be fine, enforced module signing should still apply to the crash kernel so nothing should be gain from for example somehow crashing the kernel for the intent of loading extra vulnerable modules to exploit and leak secrest (e.g. fscrypt key).

Just to keep a record, this is a Fedora patch to have a whitelist :
https://bugzilla.redhat.com/show_bug.cgi?id=560084

@rene regarding our offline discussion about the proper support of /etc/modules-load.d/, I heard you, and what you are suggesting is a good thing. But let's postpone your proposed changes for a while. This PR mainly tries to make crashkernel work on ZFS /persist, so intention is to fix this part.


"Multiple auth mechanisms are not allowed" error

Previously working workflows using MrOctopus/download-asset-action steps have started failing sometime in the last week with this error:
Error: <Error>AccessDenied<Message>Multiple auth mechanisms are not allowed; please use either query parameters or an Authorization header</Message><Resource>github-production-release-asset-2e65be</Resource></Error>

It is not a universal breakage though. For example, this is failing:
- uses: MrOctopus/download-asset-action@main
  with:
    repository: fireundubh/pyro
    asset: pyro

but this one is not:
- uses: MrOctopus/download-asset-action@main
  with:
    repository: microsoft/DirectXTex
    asset: texconv

That gave me an idea it might be the difference between repos owned by an organization vs individual, but the same issue also occurs for some organization repos.
It looks like @devkanro and @MrTolerant may have been working on the issue. Maybe they have some insights?

See octokit/core.js#415
I just change to use curl to download asset file.
See:  https://github.com/ButterCam/download-asset-action/blob/main/index.js#L56

The issue is no longer occurring for me so I'll close this as fixed via some external action by GitHub.
Thanks very much to @devkanro for the information you shared and to @MrOctopus for this useful action!

Hi @per1234 !
Thanks for raising this issue, I apologize for the late reply. As was mentioned earlier by devkanro, this issue arose because of changes made upstream by GitHub themselves that conflicted with how OctoKit handles authorization. However like you said, there seems to have been made additional changes recently to remedy the problem. Please tell me if you encounter any further issues and I'll look into them.
It's also worth mentioning that I've updated old npm dependencies (see 2ee1c70919341f6d6b424f4dd1bf5038dcaecc23 and 793d847765133189a57ba918e3a89bb51df67e25) and pushed those changes to the current release 1.0.


Remove the dependency on libc

Recent versions of bindgen use std::os::raw over libc, but currently
libsqlite3-sys is overriding that. std::os::raw is a subset of
libc that exports only the relevant type definitions, but not any
functions which require additional linking. This enables
libsqlite3-sys to be more easily used on targets that may not have a
libc available (presumably sqlite itself would have been compiled with
musl in that case)

It seems like this is an ongoing point of discussion (background from internals, open RFC). I don't think rusqlite really has a dog in this fight. I'm fine with this change 👍


selected: true is not working

In the options i'm setting the property 'selected: true'
someData: [                 { id: 1, text: 'opt1'},                 { id: 2, text: 'opt2', disabled: true },                 { id: 3, text: 'opt3', selected: true } ]
Its not setting the selected attribute for that option

#27

You should use v-model to bind options selected.

You should use v-model to bind options selected.

Sorry didn't reply before, but yes using v-model to bind the options worked perfectly. Thank you.


fix: fix loop index in transform circuit

What does this implement/fix? Explain your changes.
The part fixed in the transform circuit checks that the commitment values do not overflow. The fix replaces commitmentsValues[0] by commitmentsValues[i] in line 108. As this is input in a loop, it make sense that the commitment values should be checked for every iteration.
Does this close any currently open issues?
What commands can I run to test the change?
npm t
Any other comments?

Closed as this change is incorporated in #1435


Example?

What is an example configuration to change a verb option?

You can see it by hovering over the text above the input box,
It's "MenuOption:Replacement", leading and trailing whitespace is trimmed, letting the list be made more readable. Hyphens and numbers are accepted, despite the instructions (I forgot to update the text!). Pairs are separated by semicolons.
Such that:

"Eat:Consume" is the same as "        Eat   :  Consume     ;"
"Walk here:Move" isn't the same as "Walk       here:Move"
"Talk-To : Pester ; Drop : Yeet"

I'll also add this to the Readme.


Different loading mechanism for spacy models

Custom NER models do not return entities when loaded with spacy.blank(language).from_disk(model_path), but do with spacy.load(model_path)

@AIJoris spacy.blank(language) loads an empty model so it will not be able to return entities. On another hand, spacy.load(model_path) should load the model with a pre-trained ner model. So that should behave normally.


Add timestamps to logs (fix #132)

Add timestamps to the logs via the humantime feature on env_logger.
This formats the logs as such:
[2023-05-11T16:23:47Z INFO  lemurs] Main lemurs logger is running

Perfect! That was a way simpler solution than I would have done. Thank you so much.

Thank you for merging!  I was quite confused as to why it didn't show the time automatically, as the docs seem to suggest, but I finally realised that the default features were disabled :facepalm:.


Data Provider Threads configuration in the suite don't match the documentation

TestNG Version
7.8.0
Expected behavior
Data providers can run in parallel with the attribute parallel:
@DataProvider(parallel = true)
// ...
Parallel data providers running from an XML file share the same pool of threads, which has a size of 10 by default. You can modify this value in the  tag of your XML file:

...
Actual behavior
TestNG seems to startup "tests" that startup the dataprovider threads, and the data provider thread count is spun up per test.  So if there are 4 data-provider tests each with 10 instances, if you set the data-provider-thread-count to 10, instead of only 10 data-provider tests running in parallel, the 4 "original tests" will spin up 10 data-providers each.  Setting the thread-count is how many "tests" startup the data-provider threads.
https://github.com/bennid/testng-data-provider-threads
Is the issue reproducible on runner?

[ ] Shell
[ ] Maven
[x] Gradle
[ ] Ant
[ ] Eclipse
[x] IntelliJ
[ ] NetBeans

Test case sample
https://github.com/bennid/testng-data-provider-threads
Contribution guidelines
Incase you plan to raise a pull request to fix this issue, please make sure you refer our Contributing section for detailed set of steps.

After a bit of searching, it seems like this is similar / the same as https://github.com/testng-team/testng/issues/2019

The reason we need this solved, is that we only have a certain number of licenses for our remote selenium service.  In this suite we have ~12 tests that each have 7 data provider instances in addition to another 12 basic tests.  We can only run ~30 tests at a time in parallel to not utilize all our licenses.  In working on the suite file, I've setup the DP tests to run first w/ 8 threads, and 4 dp-threads (So a max of 32).  Then run the basic tests after.  However, this is still wasting time (we want to get the 90 some-odd tests running at around 25-30 at a time to speed up the result.
consurrency-graph

After a bit of searching, it seems like this is similar / the same as #2019

No. This issue is not similar to #2019. I believe that issue talks about having a global thread pool which is going to be shared by data driven tests and non data driven tests.
This issue you raised is specific to just having the data driven tests honour the thread pool value set globally.
To fix this, I have raised a PR which essentially adds a new attribute to the TestNG DTD so that you can now explicitly configure TestNG to use a shared thread pool for all your data providers or you can have it setup via a configuration, which will automatically enforce this in all the suites being run.


Add finder frontend metrics

This adds some new metrics to the grafana dashboard for finder frontend.

memcached free and memcached used - these require specifying the instance prefix, as apps can live on instances that do not share their name.
I am basing this on the assumption that the architecture for prod and int are equivalent in terms of naming processes and application location, please let me know if this is incorrect.
content store, rummager and registry request time - these three monitor how long individual calls take from finder frontend to the various backends.
They aren't as whitelabeled as I would like, but since the graphite keys are specified in a separate codebase I'm not convinced of the value of passing in more values into the manifest. If anyone else wants to use these same metrics for a different application then it might be worth refactoring how its handled.

I've only just noticed your comments in the Pull Request description. If you think that's worth including, its probably worth including in the respective commits as well.

I've only just noticed your comments in the Pull Request description. If you think that's worth including, its probably worth including in the respective commits as well.

I'm not sure it is worth including in the git log: It adds bloat which is only related to the ongoing AWS migration, and I don't think having questions of implementation details is useful information to tie to a specific commit.
With regards to whitelabeling it might be useful in the future. I'll add some comments in the code or something.

new screenshot:


ZEntityFramework on HybridCache (.net 9)

Hi.
Is not a issue, but i'd like to knows if zentityframework will support hybrid caching in .net 9, using methods FromCache/FromCacheAsync on IQuerable.
Thanks

Hello @fenomeno83 ,
Thank you for reporting.
We will look at it to see what is an HybridCache and what advantage it offers.
So at this moment, I cannot answer this question but I should be able soon
Best Regards,
Jon


x/pkgsite: package removal request for github.com/dmulholl/argo

What is the path of the package that you would like to have removed?
https://github.com/dmulholl/argo

Are you the owner of this package?
Yes.

What is the reason that you could not retract this package instead?
When I first created this package I briefly experimented with the idea of making it v3 of a predecessor library before ultimately deciding that it should be its own new v0.
While I was experimenting, the documentation for this ephemeral v3 version got automatically uploaded to pkg.go.dev and is now blocking the actual v0 documentation from showing up.
Obviously my own fault but if it's possible I'd appreciate it if you could wipe everything except the v0 documentation. Thanks!

as the repo still exists, you should use retract to fix this


Sema: Fix crash with stored property that has on-demand 'modify' accessor [5.3]

If a stored property has observers, the 'modify' coroutine body takes a
special form which directly yields the underlying storage and calls the
'didSet' and/or 'willSet' accessors.
However, if a property is 'dynamic', the 'modify' coroutine is synthesized
on-demand and it's SIL is serialized. It cannot reference the 'didSet'
or 'willSet' accessors, since they are private. Furthermore, an on-demand
accessor must also work if the property is overridden in a subclass.
So in this case, fall back to the more general 'modify' synthesis code,
which just calls the getter followed by the setter.
Fixes rdar://problem/62339808.

@swift-ci Please test


New short-readme version

Hey, I think our current README is very long. I think this change is good?
Check the short version: https://github.com/dev-protocol/.github/tree/short-readme/profile
Feel free to suggest if you have some ideas.

Super! Thanks for the great changes 😸

Thank you folks for reviewing 🎉


 Does the application crash? In my tests, I have been unable to reproduce the issue described.

Closing due to inactivity


99c: mismatched types, got *[2]*struct{...}, expected **struct{...}

@opennota Fixing #5 unearthed another error:
$ cd $GOPATH/src/github.com/google/zopfli/src/zopfli
$ 99c *.c
99c: mismatched types, got *[2]*struct{weight uint64,tail *struct{},count int32}, expected **struct{weight uint64,tail *struct{},count int32}
ZopfliLengthLimitedCodeLengths:0x14a: 	convert         	**struct{weight uint64,tail *struct{},count int32}, *[2]*struct{weight uint64,tail *struct{},count int32}	; katajainen.c:246:37
$

There's still a problem to solve, reopenning.


Armor Jump Crash

Another jump crash, this time I had a piece of armor on, Mekanism Osmium boots that were about half damaged.
http://pastebin.com/DJx4MmSz

Fixed again :\ for sure now


Lightbox

👷 Deploy Preview for effortless-choux-ed3c7f processing.

Name
Link

🔨 Latest commit
b4973d40f3ef7acf2da34737b9f50c4b0d48d37d

🔍 Latest deploy log
https://app.netlify.com/sites/effortless-choux-ed3c7f/deploys/664532c3b38698000782a13d


Store tree_hash in keys for superblock_manifest.tables

This prevents manifest_logs from racing each other when a table address is reacquired by one log whilst still present in uncompacted stale entries of another.
Fixes https://github.com/tigerbeetledb/tigerbeetle/issues/403

bors r+
Great fix!


Inetum job type missing - Full Stack Java Developer M/F (Middle-Senior)

ENVIROMENT: production
URL: https://peviitor.ro/
Browser: Chrome
Device: laptop
OS: Windows 10
STEPS TO REPRODUCE:

Open link https://www.inetum.com/en/job/full-stack-java-developer-mf-middle-senior
Check the job type

EXPECTED RESULTS:
The job should be on-site, hybrid or remote
ACTUAL RESULTS:
There is no job type specified

the job type is not present in the job description, and the job type is not required if it is not present


test: Audit implementations

TODO

[x] Write audit results into implementation's README.md
[x] express-graphql
[x] apollo-server
[x] graphql-yoga@v3
[x] mercurius
[x] graphql-helix
[ ] Other servers from apollo-federation-subgraph-compatibility

:tada: This PR is included in version 1.6.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Cladistic tree of depths of integration

Preface: Understanding package managers -- let alone comprehensively enough to begin to describe the design spaces for growing them towards decentralizating and imagining the design space future ones -- is eternally tricky.  I've been hoping to get some more stuff rolling in terms of either short checklists, or terminology that we might define to help shortcut design discussions to clarity faster, and so on.  This is another attempt, and it certainly won't be an end-all, all-inclusive, etc; but it's a shot.
This is a quick, fairly high-level outline of tactical implementation choices that one could imagine making when designing a package manager with some level of IPFS integration.

We could present this as a cladistic tree of tactical choices, one bool choice at a time:

using IPFS: NO

(effect: womp womp)

using IPFS: YES

aware of IPFS for content: NO

(effect: this is implemented by doing crass bulk snapshotting of some static http filesystem.  works, but bulky.)

aware of IPFS for content: YES

(effect: means we've got CIDs in the index metadata.)
uses IPFS for index: NO

(effect: well, okay, at least we provided a good content bucket!)
(effect: presumably this means some centralized index service is in the mix.  we don't have snapshotting over it.  we're no closer to scoring nice properties like 'reproducible resolve'.)

uses IPFS for index: YES

(effect: pkgman clients can do full offline operation!)
(effect: n.b., it's not clear at this stage whether we're fully decentralized: keep reading.)
index is just bulk files: YES

(effect: easiest to build this way.  leaves a lot of work on the client.  not necessarily very optimal -- need to get the full index as files even if you're only interested in a subset of it.)
(effect: this does not get us any closer to 'pollination'-readiness or subset-of-index features -- effectively forces centralization for updating the index file...!!!)

index is just bulk files: NO

(effect: means we've got them to implement some index features in IPLD.)
(effect: dedup for storage should be skyrocketing, since IPLD-native implementations naturally get granular copy-on-write goodness.)
(effect: depends on how well it's done...!  but this might get us towards subsets, pollination, and real decentralization.)
TODO: EXPAND THIS.  What choices are important to get us towards subsets, pollination, and real decentralization?

The "TODO" on the end is intentional :) and should be read as an open invitation for future thoughts.  I suspect there's a lot of diverse choices possible here.

Another angle for looking at this is which broad categories of operation are part of the duties typically expected of the hosting side of a package manager:

distributing (read-only) content
distributing (read-only) "metadata"/"index" data -- mapping package names to content IDs, etc
updating the "metadata"/"index" -- accepting new content, reconciling metadata changes, etc

This is a much more compact view of things, but interestingly, the cladistic tree above maps surprising closely to this: as the tree above gets deeper, we're basically moving from "content" to "distributing the index" to (at the end, in TODOspace of the cladistic tree) "distributed publishing".

These are just a couple of angles to look at things from.  Are there better ways to lay this out?  Can we resolve deeper into that tree to see what yet-more decentralized operations would look like?

I put together a quick visual tree version of this on mural: https://app.mural.co/t/protocollabs6957/m/protocollabs6957/1556717261380/7d93181e586fc3416ef88a42ba6d6df4b964c89b

Planning on expanding it with the different paths that certain categories of package manager are likely to take.

Going to be moving the content of this issue into the /docs in the repository in https://github.com/ipfs/package-managers/pull/53

Wow, TIL that Mural is incredibly lacking as far as flowcharting is concerned. Sadness. With that in mind and with apologies to @andrew, I reorganized this in more of a table structure so that we could add @warpfork's original explanatory text (and ideally make this more consumable by external audiences). Link to the Mural is here and screenshot pasted below for good measure.
@andrew -- when you get a moment, do you mind moving the Mural out of your personal room and into the main package managers directory I just added to Mural?
Everyone else -- suggestions/addenda/corrections more than welcome!

Ohwow, you managed to add nontrivial amounts of text to mural, AND make it pretty.  This is super nice! :D:D
One thing I notice now that I look at the content in this nice layout is that the last branch is kind of a pattern-breaker: all the earlier ones, "yes" is the "nice" answer.  Not so with the last one.  If anyone can think of a better way to phrase that question, I think there's deffo room for improvement there.

@warpfork Yeah, that bothered me too. See below for alternate, but is "granular" the right word to use in the blue question box? Is there a better word?

Slightly related old pile of notes I made for approaches based on different implementations: https://gist.github.com/andrew/cda6a99556152e3a65794406d093fada

I also attempted to make the tree as a mermaid diagram: https://hackmd.io/rX0G1w1oQJGXh5kwlRZovw

Works quite well, although you have to control the line breaks yourself which is slightly annoying with longer lines of text.


Fixed sorting of XXL, XXXL etc...

I found that the following list was not sorted correctly.
Input:
['S', 'XS', '2XL', 'L', 'XXXL', 'XXL', 'M', 'XL']
Expected:
['XS', 'S', 'M', 'L', 'XL', '2XL', 'XXL', 'XXXL']
Got:
['XS', 'S', 'M', 'L', 'XL', '2XL', 'XXXL', 'XXL']
I created some tests and fixed the issues.
I also changed the parseInt fallback to strip out all non number characters to support for shoe sizes like EU 46 or US 12.

Thanks Magnus! Looks good. Merging.


toggle-track could not work

Checklist

[X] I have read through the manual page (man fzf)
[X] I have searched through the existing issues

Output of fzf --version
0.39.0 (2023040)
OS

[X] Linux
[ ] macOS
[ ] Windows
[ ] Etc.

Shell

[X] bash
[ ] zsh
[ ] fish

Problem / Steps to reproduce
Hi,
I've read fzf manual and used the toggle-track in --bind. However, it always report unknown action: toggle-track. Does anyone know what happened?
What I verified is like this
[00:00:07] 2 karl@DENG-KARLL dotfile:unstable $ FZF_DEFAULT_OPTS=
[00:00:17] 0 karl@DENG-KARLL dotfile:unstable $ fzf --bind='ctrl-t:toggle-sort'
[00:00:03] 130 karl@DENG-KARLL dotfile:unstable $ fzf --bind='ctrl-t:toggle-track'
unknown action: toggle-track
[00:00:03] 2 karl@DENG-KARLL dotfile:unstable $ fzf --bind='ctrl-t:track'
unknown action: track
[00:00:04] 2 karl@DENG-KARLL dotfile:unstable $ fzf --version
0.39.0 (2023040)

The manual write this
           toggle-search                (toggle search functionality)
           toggle-sort
           toggle-track
           toggle+up                    btab    (shift-tab)
           track                        (track the current item; automatically disabled if focus changes)
           transform-border-label(...)  (transform border label using an external command)
           transform-header(...)        (transform header using an external command)

Thanks

Wow, thank you! Problem solved.


snmpget retry

using pysnmp cmd tools(https://www.pysnmp.com/snmpclitools/):
$ snmpget -v2c -c public demo.pynamp.com SNMPv2-MIB::sysDescr.0
when the demo.pynamp.com is not reachable , the request will be sent with increasing the request-id,

If the network transmission delay is greater than 1 second, the new request will sent endlessly.
Why don't we send the retry requests with the same id?
Thanks for reading!

Since Ilya passed away, we might not be able to learn why.
Note that if your goal is to have rock solid command line utilities to use for your daily work, the bits from NET-SNMP is still preferred (de facto standard). PySNMP command line tools are more like samples for your reference.

sad to hear that.
thanks  for your reply.


add support for CAA records

Domain Record Type

Blocked by: https://jira.linode.com/browse/MR-174


Feature/get most relevant jira ticket

Description
To reinforce our core concepts of code archeology and going beyond git blame to help devs understand code context, we're building this new feature. Get the most relevant jira ticket for a given line, block or file of code.
The steps are:

Get the commit hashes and most relevant PR title with what we were already doing before
Send the PR title (with stop words removed) as a search query to our API
Render the Jira ticket that has the most word matches, regardless of its ticket state.

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update
[ ] Chore: cleanup/renaming, etc
[ ] RFC

Notes

Still missing a programmatically retrieved user email
Still missing how to address the case where the user is authed but is not an actively paying Watermelon user

@EstebanDalelR I guess https://github.com/watermelontools/watermelon/pull/82/files is the answer to the second question I left on the Notes section?


Wrong version of cdk-assets resolved for 2.25.0 SST project when in a pnpm monorepo with a 2.24.28 SST project

I'm running into an issue in a monorepo using pnpm. The wrong version of cdk-assets is resolved for >= 2.25.0 SST project when running in a pnpm monorepo with a 2.24.28 SST project.
I console.log'ed the version of cdk-assets resolved at runtime for my >= 2.25.0 SST project and observed that it is 2.91.0, but that is the version used by SST 2.24.x.
It manifests as the following error after running sst dev
⠼  Bus PUBLISH_ASSETS_IN_PROGRESS
⠼  ShopsTable PUBLISH_ASSETS_IN_PROGRESS
|  ShopsTable PUBLISH_ASSETS_COMPLETE
⠼  Bus PUBLISH_ASSETS_IN_PROGRESS
[50%] fail: Unrecognized asset type: '[object Object]'
|  Bus PUBLISH_ASSETS_COMPLETE
⠋  Deploying...

✖  Errors
   ShopsTable UPDATE_FAILED
   stack: Failed to publish one or more assets. See the error messages above for more information.
   ShopTableSeed DEPENDENCY_FAILED
   ShopsEventHandlers DEPENDENCY_FAILED
   ShopsSubgraph DEPENDENCY_FAILED
   Bus UPDATE_FAILED
   stack: Failed to publish one or more assets. See the error messages above for more information.

More Details:

When I run the project outside of a monorepo (still using pnpm) it works fine.

When I remove the 2.24.28 SST project from the monorepo it works fine.

When I add "cdk-assets": "2.95.1" to my package json it works fine.

Possible root cause
SST depends on a package called sst-aws-cdk which appears to be a fork of aws-cdk. SST uses the publishAssets function from this library, which is where the error described above originates.
The version of sst-aws-cdk that the latest version of SST depends on is 2.91.0. It seems like this should be 2.95.1, in order to match the version of aws-cdk-lib that SST depends on. However, I'm not sure why this would cause issue in a monorepo, but not in a single repo.
Should the sst-aws-cdk package that the latest version of sst is using be updated to 2.95.1?

Discord Thread:
https://discord.com/channels/983865673656705025/1153846634271215676

I followed up on that thread.

I had the same issue yesterday. We have a monorepo that uses PNPM. There are two SST applications/packages inside that monorepo.
I am assuming the issue came from some mixed versions of SST or CDK being installed and hoisted to the root node_modules.
The way I solved it (a workaround) is to run an npm install inside the specific SST package that was throwing this error. Not saying you should ditch PNPM and go back to NPM commands, but in this case - it seems a combination of factors is throwing off the PNPM hoisting behavior.

We are transition this repo to https://github.com/sst/v2 and are closing this issue for now. If it is still relevant please reopen it there - sorry for the trouble


Remove dead links

Description

Related issue

Checklist

[ ] Tested against the NHS.UK frontend testing policy (Resolution, Browser & Accessibility)
[ ] Code follows the NHS.UK frontend coding standards
[ ] Version number has been increased in package.json (using SEMVER)
[ ] CHANGELOG entry
[ ] Whats new page entry
[ ] Page updated date

LGTM


[DFK27BUR0135]Trigger-mode case:445022

Dear TIS team,
The trigger-images Python example shows only a static image after the "source.set_tcam_property("Trigger Mode", True)" line. When I try to snap other picture by the space(" ") and enter nothing happen. Other examples like the properties does not show any video or capture and give an error("valve is broken"), the property_dialog shows the video live desired but when I switch to trigger mode it freeze and after certain seconds a disconnected device message box pop-up.
Please I appreciate so much any help about how to capture pictures by software using python for the DFK27BUR0135 camera.
Note: The  list_properties example needs the select_camera function implemented in list_devices.

Hello
I tested the sample on my computer and had no issues.
Do you get an image stream, if the line
source.set_tcam_property("Trigger Mode", True)
is commented out?
The sample, that uses the "valve" seems to have an issue with GStreamer 1.0... so please ignore that sample, until next update.
Stefan

Dear Stefan,
Thanks for your answer. When I comment the line that you indicates the stream is fine as expected. But the space command no cause any new snap in the original script.
Thanks for the answer.
Best regards.

Hello Gonzales
I found it..... The 27UR0135 has hardware trigger only, no software trigger. For unknown reasons the software triggering did not work.
I am very sorry for the long delay of figuring that out.
Stefan

Dear TIS-Stefan,
Thanks for the answer. I understand the response but I still confuse because on IC Capture the snap function works correctly.
Is there any way to implement the snap using Python, OpenCV and the gstreamer modules?
Best regards

Hello
In my notices to the camera it says, the DFK 27BUR0135 has no software trigger. Unfortunately I do
not have such a camera on stock, therefore, I am not able to confirm, that software trigger works.
You really enabled trigger mode in the Device Properties dialog and clicked the "Softwaretrigger"
button and got one single image?
Also, I wonder, whether you understand trigger mode correctly. Trigger mode means, the camera sends an image only, if an extern trigger pulse, by hardware or software, if supported, is sent to the camera. While there is no trigger pulse, you wont get any images streamed.
Trigger mode does not mean, you get the images automatically somewhere, where you can handle them.
I have the faint suspect, you only want to get one image from the live streamed images from the camera. Am I correct here or am I completely wrong?
Stefan

Dear TIS team,
I try the snap capacity on IC Capture - Windows. The image attached is the capture using the snap button. On Ubuntu 16.04 LTS the script of properties for Trigger Mode gives an error. My objective is snap a picture with a serial command through "COM" port. I already wrote some code  to achieve this based on your TIS.py example but I am testing it. In case of success I share the code but is only pull if a new sample is available on certain condition.
Thanks for your answer.

Aha, as expected. Snap means, IC Capture software takes an image from the live stream. The camera is not triggered. The trigger mode button is two buttons left from the "snap" button.
That means, in Windows you dot not run the camera triggered, as you do in Linux with the "trigger-image" sample. That means you do completely different things.
That also means, the "trigger-image" sample is not, what you need. You only want to snap am image form the live stream.  What you need is shown in the "live-video.py" sample!
Stefan


Uploaded file icon and/or filter

We've gotten feedback since the launch that the conference presentations without uploads aren't as useful. Two possible solutions:
Icon Approach
Upload icon (paperclip) included next to each resource in the list views (in case folks want to skip items that don't have uploaded files).

Going with this approach, it would be nice to have icons for resources with images (picture icon) and resources with links (chain link icon) as well.
Ideally, you could click on the icon to pull up a list of all resources with files/images/links, but this option wouldn't show up in the filtered list.

Filter Approach
Add filter for with/without uploaded file in addition to or instead of above.

Another option: If there is no upload or link, we could add some text (particularly for conference presentations)

A filter for Image/File/Link filter would possibly be useful for AASHE staff who might want to only view resources with images (and further refine by topic as needed)

Decision:

Filter can be placed at the bottom of the standard list of filters
Decision to add icons - very little work. Just be careful about performance.
Icons can include hover-over text "This resource includes one or more files/URLs"

Decided to bike rack in favor of a simpler fix to add help text whenever a file is not included.


test error: Couldn't read: 001304.txt of ground truth.

I just download the code, and use the prepared data files (*pickle) to do some easy trainning and evaluation work to reproduce the author's work.
First thanks to the excellent work of this project and the wonderful open source! Although I didn't go through the details of the code, I can easily train the model.
But when I do the evaluation step using command 'sh scripts/command_test_v1.sh' or 'sh scripts/command_test_v1.sh', there comes the same error -- 'ERROR: Couldn't read: 001304.txt of ground truth. Please write me an email!'. I wonder if there is something wrong with the pickle data?
I will go through the code details later, but I just want to see the evaluation result fisrt, is there any ideas about this issue?

Already solved here.


Please create a new release with an updated gosigar module

There is a new gosigar release https://github.com/cloudfoundry/gosigar/releases/tag/v1.2.0 that should correctly calculate available memory inside a container, to address #29.
Could you please bump gosigar and make a new log-cache release, so that this can be tested/used by kubecf?
Thank you!

@jandubois why don't you update and vendor and make a pr? I setup pr pipes too.

@jandubois Just to be clear. Release was cut https://github.com/cloudfoundry/log-cache-release/releases/tag/v2.10.3

Thanks for the quick release @Benjamintf1!  I was already notified on Slack 😄


Ancillary datasets created need their own STAC collection

At the moment, post job actions create STACK collection where specifically for WorldCereal, optical extractions are merged with ground truth rasterization. For consistency it makes more sense that ground truth is treated independently.
Needs to be discussed how to tackle this.

How to actually design this part will be discussed on Wednesday 17th of April. To be continued after this meeting.


External self transitions are not being dispatched correctly for parallel states

Expected Result
In the reproduction sandbox, "input updating..." should be printed twice.
Actual Result
"input updating..." is printing only once.
Reproduction
https://codesandbox.io/s/xstate-example-template-7xfg0
Additional context
Sandbox XState version: 4.6.7
The bug was introduced in v4.6.0. In v4.5.0 "input updating..." is printed twice.

This is expected and documented: https://xstate.js.org/docs/guides/transitions.html#internal-transitions
Always prefer being explicit if you want specific yet non-obvious (re-entry, in this case) behavior to occur.

Summary of internal transitions:

EVENT: '.foo' - internal transition to child
EVENT: { target: '.foo' } - internal transition to child (starts with '.')
EVENT: { target: 'same.foo', internal: true } - explicit internal transition to child
This would otherwise be an external transition
EVENT: undefined - forbidden transition
EVENT: { actions: [ ... ] } - internal self-transition
EVENT: { actions: [ ... ], internal: true } - internal self-transition, same as above
EVENT: { target: undefined, actions: [ ... ] } - internal self-transition, same as above

Thanks for the report - gonna look into fixing this over this weekend.

This is broken in 4.6.7 but fixed at least since 4.7.0-rc2. So this is not actionable right now unless we'd like to backport a fix to 4.6.x line.
There are also some other fixes in 4.7, would be great to release it sometime soon as stable. What are the current blockers @davidkpiano ?

@Andarist No current blockers. Just wanted to make sure it's stable since the core has changed quite a bit.

Anything I could help with?

I confirm it was fixed in v4.7.0-rc3. I wasn't able to use v4.7.0-rc3 yet because it's breaking some machines that were used to work in v4.6.0.

@hnordt Can you share those machines and report the bugs, if any?

@davidkpiano I'll try to open the bug report on Monday, this weekend I need to finish a project milestone. 😅


[backport] Check positive dilation in F.deconvolution_2d

Backport of #7449

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit 9e2baf5cf0dcfa5771896f19edc3c2b02b381488, target branch v6) succeeded!

LGTM!


Add return type to MockLEDBoard

I originally imported Component for the type checking, but the linter didn't like it.

Yeah, I can't see immediately why though. Maybe because LED is a subclass Component, it is in the scope of the type checker? (Wild speculation)

Looking into it I think these docs justify this behaviour. We have the import LED and the file containing LED imports Component. By following the import chain mypy knows about Component without needin an explicit import in this file 😃


Rename the Auto Login setting to ULP; move to features tab

Changes
There has been some general confusion around how to use the ULP with this plugin. This PR moves the setting to be more visible and changes the name to more clearly explain what the setting does. I left a reference from the old setting to the new one to help with UX.
References
Universal Login vs Embedded
Testing
No functionality changes so no testing needed.

Codecov Report

Merging #551 into master will decrease coverage by 0.04%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master     #551      +/-   ##
============================================
- Coverage     15.36%   15.32%   -0.05%
- Complexity     1605     1606       +1
============================================
  Files            69       69
  Lines          5473     5489      +16
============================================
  Hits            841      841
- Misses         4632     4648      +16

Impacted Files
Coverage Δ
Complexity Δ

lib/admin/WP_Auth0_Admin_Features.php
0% <0%> (ø)
40 <2> (+2)
:arrow_up:

lib/admin/WP_Auth0_Admin_Advanced.php
18.68% <0%> (+0.59%)
69 <1> (-1)
:arrow_down:

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 607d9c2...bacc506. Read the comment docs.

Codecov Report

Merging #551 into master will decrease coverage by 0.04%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master     #551      +/-   ##
============================================
- Coverage     15.36%   15.32%   -0.05%
- Complexity     1605     1606       +1
============================================
  Files            69       69
  Lines          5473     5489      +16
============================================
  Hits            841      841
- Misses         4632     4648      +16

Impacted Files
Coverage Δ
Complexity Δ

lib/admin/WP_Auth0_Admin_Features.php
0% <0%> (ø)
40 <2> (+2)
:arrow_up:

lib/admin/WP_Auth0_Admin_Advanced.php
18.68% <0%> (+0.59%)
69 <1> (-1)
:arrow_down:

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 607d9c2...1680863. Read the comment docs.


WIP: Include Internal Dashboard FQDN/IP value in the LDAP certificate

Since Dex will connect to LDAP using this FQDN/IP, make sure that the
TLS handshake will succeed by regenerating the certificate early in
the orchestration, so it includes this FQDN/IP in the SAN extensions
of the LDAP certificate.
Fixes: bsc#1069175
Backport of https://github.com/kubic-project/salt/pull/310

How I validated the change:
Internal Dashboard FQDN/IP: <IP_ADDRESS>.nip.io
Without this patch, LDAP will contain an invalid certificate when dex tries to connect to it using TLS and the internal dashboard reference: <IP_ADDRESS>.nip.io:389, as the certificate will contain all ip addresses of the machine along with all detected hostnames in the machine (<IP_ADDRESS>.nip.io is not among them).
With this patch, I can properly go to Dex, and login, downloading the kubeconfig file.

worked with caasp2 fresh installation:
> btrfs property set -ts /.snapshots/1/snapshot ro false
> mount -o remount,rw /
applied changes
executed Velum::Salt.orchestrate from velum dashboard rails console
wait 5 minutes - should be good to go


Fix the indentation problem

...I should check the build first

Codecov Report

Merging #609 (1a8739b) into develop (38b7756) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #609   +/-   ##
========================================
  Coverage    95.51%   95.51%
========================================
  Files          118      118
  Lines         7445     7445
========================================
  Hits          7111     7111
  Misses         334      334

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 38b7756...1a8739b. Read the comment docs.


change pre-commit to paddle repo.

change the pre-commit of yapf to paddle repo.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Zhong Hui seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


What's the difference between 'world' and 'stage'?

This Issue is about the difference between world and stage. I found that game.world.add(sprite) differs from game.stage.add(sprite)--------the 2nd method will not show the sprite on screen, So, what's the difference? I'm confused……

Stage is the root display container. World is the root Group and the first child of the Stage.
Both methods should display the sprite, unless the sprite is offscreen or covered by another object. Only World objects are affected by the camera position.
Use the game.add.* methods.


Bug: Players see murders that happen in the room they're going to

They should only see murders in the room they're currently in. Murders take place before actions.

(No principled reason behind this, you could have it the other way around. Do we have reasons to prefer one or another?)

Scratch that. Assume Amy and Lena are in the hallway. Amy tries to kill Lena, Lena tries to go to the bathroom. I think Amy should be able to kill Lena before Lena leaves. Allowing actions to take place before murders would make the killer's job much harder. It would also mean that, when they're alone in a room with someone, they're not safe to kill them without being directly observed -- somebody could walk in from another room before the murder occurs. So I think it makes sense to have murders occur before actions within the same round.


NodeGit.Cred.sshKeyNew: Waiting for USERAUTH response

Hey guys I am trying to fetch/clone using Cred.sshKeyNew, but I keep getting:
Failed to authenticate SSH session: Waiting for USERAUTH response
Source? https://github.com/karelia/libssh2/blob/master/src/userauth.c
Does anyone know whats going on?
I am using just like https://github.com/nodegit/nodegit/blob/master/test/tests/clone.js#L92-L114
I can get everything to work by using NodeGit.Cred.sshKeyFromAgent(userName). But in my case, I will have to make it work using a private key file.

"just like" unfortunately our tests pass, so that's not enough to go on.  What's your code look like? Can you paste as much information as possible?

Sorry @tbranyen, I get it. I did try below to paste as much as I could, this is basically the first thing I do, and it fails if I use sshKeyFromAgent.
var cloneURL =<EMAIL_ADDRESS>
var storageLocalPath = require('path').join(process.cwd(), 'app/git', "static");

var remoteCallbackOptions = {
  certificateCheck: function() {
    return 1;
  },
  credentials: function(url, userName) {
    return NodeGit.Cred.sshKeyNew(userName, path.join(process.cwd(), './git_keys/key.pub'), path.join(process.cwd(), './git_keys/key'), '');
  }
};

var cloneOptions = {
  remoteCallbacks: remoteCallbackOptions
};

NodeGit.Clone(cloneURL, storageLocalPath, cloneOptions).then(function() {
  return console.log("got it");
})["catch"](function(error) {
  return console.error(error);
});

I believe that I am calling everything correctly, so, if we assume that, I would need help to try to understand that error code, since, maybe the issue is in my keys or something else. I did test the key out using:
ssh -i path/to/file gitosis@server
I did try with another key, that I use normally to access that repository, and it WORKED. But, I dont know if the issue is related to MacOS, or something like that, since the other key that I am trying is not being used in the OS.
Thanks for the help!

So I'm able to repo that issue on #580 but only when the ssh key is not tied to any account. Did you add the key to an account before you used it?

@johnhaley81 , thats exactly my issue.
The key that worked was one tied to my account already. The one that I need to use, it is not tied to the account, and that is what I need.

@lucastschmidt so the error that comes back from libgit2 is pretty cryptic. We're not getting the underlying error from libssh2 that we ought to. I dove into the C code and saw that what's happening is that the server is rejecting authenticating the key because it has no knowledge of the key. Then it polls the same credential callback and gets the same key and gets the same error. It does this until it times out which is where we get the Waiting for USERAUTH error which is misleading.
But the first part makes sense and is correct. If you're trying to use a key that the server has no knowledge of then it's going to reject it. There's really nothing we can do on our side to remedy this. If it's a requirement that the server has no knowledge of the SSH key you're trying to use before hand then there's nothing that we can do for you.
If that isn't a requirement then I'd put the key on the server and it should work for you.

@johnhaley81 got it, in my case, the key was suppose to work, it was (supposedly) correctly added in the server, and I was able to test the key using ssh -i path/to/file gitosis@server.
The only thing that I thought may be something to consider is that the key is not associated to my user account in the OS. If I use another key, that is associated to my user account, it works fine. While using this other one, it doesnt.

@lucastschmidt if you're using ssh and it has any other keys in there that will authenticate then it will try to use other ones when you fail with the one specified in the path which is probably why it's succeeding. Try removing all other keys from the agent (e.g. renaming ~/.ssh to something else)


Audit Report - version 1

Why is this issue important?
This is a critical functionality for partner administrators to QA the Excel workbooks that are submitted by agency staff users.
Current State
No audit report exists
Expected State
Administrators are able to generate an audit report from the UI.
The audit report will include the following sheets:
Sheet 1
Sheet 2
Sheet 3
Sheet 1 will have one row for each project. The columns will be:
Project Name - This data will get pulled from Input template sheet name {{ insert sheet name }} column name {{ insert column name }}
Project ID - This data will get pulled from Input template sheet name {{ insert sheet name }} column name {{ insert column name }}
Column 1 - This data will get pulled from Input template sheet name {{ insert sheet name }} column name {{ insert column name }}
Column 2 - This data will get pulled from Input template sheet name {{ insert sheet name }} column name {{ insert column name }}
Column 3 - This data will get pulled from Input template sheet name {{ insert sheet name }} column name {{ insert column name }}
Sheet 2 will have one row for each project. The columns will be:
Project Name - This data will get pulled from Input template sheet name {{ insert sheet name }} column name {{ insert column name }}
Project ID - This data will get pulled from Input template sheet name {{ insert sheet name }} column name {{ insert column name }}
Column 1 - This data will get pulled from Input template sheet name {{ insert sheet name }} column name {{ insert column name }}
Column 2 - This data will get pulled from Input template sheet name {{ insert sheet name }} column name {{ insert column name }}
Column 3 - This data will get pulled from Input template sheet name {{ insert sheet name }} column name {{ insert column name }}
Implementation Plan
Example Input template added by @jcomeau1
Example Audit report added by @jcomeau1
Relevant Code Snippets
No response

checking if there is a subawards input template

There is no subawards template or section within the guidance for them.

closing as a duplicate of #82


Collections extension test broken as of stac-fastapi v2.5.5

https://github.com/stac-utils/stac-fastapi-elasticsearch-opensearch/blob/d2104af3255a6c0e9e16fab98915a406dfcdd6e6/stac_fastapi/tests/resources/test_collection.py#L110-L124

The extension adds a item_assets field to a collection. I can create a collection with the field and then GET that collection back and the field still exists but pushing the same collection through a PUT request erases the field.

Test passes now here #234


Add InvokeFunction Policy for API Gateway

Feature request
In the AWS environment I'm in, it would be beneficial if zappa deploy added a policy to allow the newly-created API Gateway to have InvokeFunction permissions on related Lambda function.
For us, I need to be explicit in the resources that can invoke the Lambda function via the InvokeFunction policy. Zappa adds a policy for the CloudWatch keep warm event, so that makes it work out of the box. However, I have to add a policy manually for the newly created gateway to have the InvokeFunction policy. It seems to me it would make sense to create this automatically and to be explicit.
I'd be open to exploring this myself with a PR if this is of interest. I'm not sure if this creates any issues for others out there, but I would think it wouldn't.

I'm still researching this more, but it appears that Serverless does something similar here.
https://github.com/serverless/serverless/blob/a900bffb97d35863f1b03d389c06ce1596852bd8/lib/commands/deploy_endpoint.js#L925
However, I've tried to update the policy with CloudFormation manually without success. I have to manually go in and edit the Lambda function in the API Gateway console (not change it, just edit, which asks me if I want to give permissions). Once I do this, all is well—I can even update these permissions in the future and it works just fine.

Is it possible you're not seeing the change because you're calling update rather than redeploying? Update just updates the function code, not the API deployment config.

The issue happens to me on deploy. I'll walk through my issue here:

I run zappa deploy dev and it sets everything up.
I go into the API Gateway console and try to test a call. I get Execution failed due to configuration error: Invalid permissions on Lambda function.
I run a little script I wrote or I try to update the CFN stack and the permissions are added
I try to test in the console—same error
I go into the "Integration Request" section and simple edit the region and function name (I don't change it, just edit it).
I get asked if I want to give the gateway permissions, and I say yes. I have to do this for the root method and the resource both to make all URLs work.
I try to test again—everything works

But this is where it gets strange:

I run zappa update dev
This wipes the permissions in the Lambda function and I get the "invalid permissions" again.
I run my script again that adds permissions (or CFN update, both do the same)
I test and the request works

So after that initial manual click in the console, I can update permissions programatically. Very strange.

I believe this was all related to a permission issue, and I'd move to close this for now, though I'll mention one thing at the end that may need another issue.
Just for the sake of info, I'll mention what happened. My execution role did not have the lambda:InvokeFunction policy. When I would try to call a Lambda function—even if the function had the correct permissions—it was result in the "invalid permissions" issue. If I would update the methods in the console, it would replace my limited execution role with my own user's role, which would then give permissions to the gateway to invoke the function. I wish they would have mentioned that was happening in the UI :)
Now that we've added the necessary policy, it worked, and without the need for adding permissions to the Lambda function itself.
The thing I wanted to mention was that it feels like there should be a separate role for the API gateway, where we only add the the lambda:InvokeFunction policy. This would then be the role in the integration credentials in the CFN template created by Zappa.


Crashlytics AW Sigaction crash fix #6434

This fixes the crashlytics crash on apple watch, which doesn't support sigaction. Fix for #6434

Thanks for the fix! Please fix the CI failures and add a note to https://github.com/firebase/firebase-ios-sdk/blob/master/Crashlytics/CHANGELOG.md.

@paulb777 looks like there might be an issue with execution policy for tvos? Not able to see the details of why it is failing.

@jostster Running scripts/pod_lib_lint.rb FirebaseCrashlytics.podspec --platforms=tvos --verbose in your branch shows
    Undefined symbols for architecture x86_64:
      "_FIRCLSSignalInitialize", referenced from:
          ___FIRCLSContextInitialize_block_invoke_2.29 in FirebaseCrashlytics(FIRCLSContext.o)
      "_FIRCLSSignalCheckHandlers", referenced from:
          ___52-[FIRCLSReportManager crashReportingSetupCompleted:]_block_invoke in FirebaseCrashlytics(FIRCLSReportManager.o)
    ld: symbol(s) not found for architecture x86_64
    clang: error: linker command failed with exit code 1 (use -v to see invocation)

    Test session results, code coverage, and logs:
    	/Users/paulbeusterien/Library/Developer/Xcode/DerivedData/App-etocetmlokomdlfjvqpeqfycrpbt/Logs/Test/Run-FirebaseCrashlytics-Unit-unit-2020.09.10_10-22-02--0700.xcresult

    Testing failed:
    	Undefined symbol: _FIRCLSSignalInitialize
    	Undefined symbol: _FIRCLSSignalCheckHandlers
    	Testing cancelled because the build failed.

It looks like #if adjustments are needed.

Comment added per review

Should be good to merge when you're ready @jostster 👍

@samedson I don't have write access so cannot merge.

Woops alright I got you!


Update AVERT_2021_NOx_lb_per_kwh.csv

Previously incorrect NOx emissions profiles (they were the same as SO2)

Looks good, can you add a line or two (and possibly a link) on where the NOx emissions profile came from for documentation purposes? Was there any data transformation?

added a bit of information in latest changelog commit! More info is in user manual


fixes #279 Consul heartbeat stream limit being exceeded

make a number of requests configurable in consul.yml and reset the connection if the max number is exceeded.

@NicholasAzar Thanks for the review. I have made the change accordingly and merged to develop.


Add remote branch option

Draft Implementation as proposed in https://github.com/yamadashy/repomix/issues/195

Hi, @tranquochuy645
I've made some minor tweaks, but it works perfectly!
Thank you for your contribution!
Merging this.
Really appreciate your help!
Note: I'll address the npm vulnerabilities in a separate PR.


Enhancements for the import template (from Marco)

TODO

[x] For indicators, sources of information, risk and mitigation, increase options from 3 to 5.
[x] For the Timeframe and Contacts information section, please just leave the “Operational Timeframe” field, remove all others as they are completed at latest stages and not during the use of the offline template.
[x] In all section, double the width of the “Value” Cell to 128 pixels, so double, this works better for the long text format values.
[x] Is it possible to assign also lengths or high to the rows, row by row, so we have have a visual cue of which are the long format rows by having bigger cells space.
[x] Is it possible for the excel too always be formatted to adjust to text
[x] We need to have more separation between sections, like between the actions section and the needs section as well as to have more subtitles

[x] We need to add extra headings on the form.
[ ] The borders are not set when the background is set.
[x] The template is also not consistent with the template provided. (content, color, tab color, font size, font color, etc.)

[x] In excel, when adding “-“ or “=“ at the start of a cell, it things is a formula and gives an error, in some cases user will want to have “-‘ as bullet point to list activities and that could bring an error. Not sure of there is something to avoid that.

[x] Add this as the description text of List of Activities field in Operation sheet: A list should start with an ' * ' followed by a space. There are no limits to the number of lists that can be included.
Eg: * Activity XYZ * Activity ABC (each one in new line)

[x] Error while opening the file
[x] Add all the missing fields in the modified template

[x] Date of the event should be replaced with Date when the trigger was met
[x] ICRC > Actions Needs
[x] List coordination mechanisms/platform in place at local/district and national level. Indicate the lead authorities/agencies. How the National Society is involved/positioned in this coordination. Does the NS in any lead/co-lead role? Any identified gap/overlap in the coordination (e.g., sector missing…)?
[x] Date of Submission to GVA
[x] Date of Approval

To be discussed

[ ] Currently if we set the height, the row does not auto grow. We need to look into alternative solutions.
[ ] We also need to discuss with Marco about consistent color for headings. What about heading levels more than 2-3? We currently already have headings on 5th levels. (make the list gray and alternate color use blue color only for headings).

Hey @marcovvu1,
For the feedback

For the Timeframe and Contacts information section, please just leave the “Operational Timeframe” field, remove all others as they are completed at latest stages and not during the use of the offline template.

Do you mean to,

Remove all other fields from the section Operational Timeframes and only keep the field Operation timeframe

Or,

Remove Tracking data and contacts section and only keep Operational Timeframes section.

It's a bit confusing due to the same name. Let us know which one is it.
CC: @udaynwa @tovari @anamariaescobar

Hey @marcovvu1,
For the feedback

For the Timeframe and Contacts information section, please just leave the “Operational Timeframe” field, remove all others as they are completed at latest stages and not during the use of the offline template.

Do you mean to,

Remove all other fields from the section Operational Timeframes and only keep the field Operation timeframe

Or,

Remove Tracking data and contacts section and only keep Operational Timeframes section.

It's a bit confusing due to the same name. Let us know which one is it.
CC: @udaynwa @tovari @anamariaescobar

Hi!
We can keep the section lets say, the Operation timeframe section, but leave just the on question about the amount of months of the operation.

Hey @marcovvu1,
For the feedback

For the Timeframe and Contacts information section, please just leave the “Operational Timeframe” field, remove all others as they are completed at latest stages and not during the use of the offline template.

Do you mean to,

Remove all other fields from the section Operational Timeframes and only keep the field Operation timeframe

Or,

Remove Tracking data and contacts section and only keep Operational Timeframes section.

It's a bit confusing due to the same name. Let us know which one is it.
CC: @udaynwa @tovari @anamariaescobar

Hi!
We can keep the section lets say, the Operation timeframe section, but leave just the on question about the amount of months of the operation.

@marcovvu1 And remove the Tracking data and contacts section? Or keep it as well?

Questions from Marco (20/9/24)

date fields: how can we ensure that people always put the right format to ensure no issues in the import
Column A: question on whether column A can be blocked from any editing, and if that would not impact collaborative work

cc @tovari @barshathakuri @frozenhelium @tnagorra @samshara

@marcovvu1 @tovari

date fields: how can we ensure that people always put the right format to ensure no issues in the import

Any date format will be accepted. The import template will automatically convert the entered date into the correct format. For the unconvertible date, the import template gives an error pop-up. (screenshot attached)

Column A: question on whether column A can be blocked from any editing, and if that would not impact collaborative work

Implementing the blocking is not entirely possible. Microsoft Excel can override the implementation, so there is no straightforward way.

cc @samshara @frozenhelium @barshathakuri


how work on nuxt?

hi everyone,
sombody knows how make vue-strap on nuxt?
i think i am failing in import as a plugin

i have the same problem, any solutions?
@gabrielmancini


use refactoring of BMP info headers in code

Onde issue #68  will have been completed.

Completed. Tested.


Add more tests with focus on covering happy paths

Improve test coverage with focus on testing happy paths.

Closed by PR #176.  I'll open a separate issue to add more tests.


[$250] Web - Task – App logging out when leave thread in a new non-validated account

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

Version Number: 9.0.18-7
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4836943
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Action Performed:

Go to https://staging.new.expensify.com/
Log in as a new Gmail account, do not validate it.
Create a chat
Send a message in the chat
Reply in thread and send a comment
Click on the chat header and leave the thread

Expected Result:
Main chat opens
Actual Result:
App logging out
Workaround:
Unknown
Platforms:

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

Screenshots/Videos
https://github.com/user-attachments/assets/f3d4ed2e-ade0-47ce-9469-e517242c6747

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~018c285e8464a196cb
Upwork Job ID:<PHONE_NUMBER>366705877
Last Price Increase: 2024-08-24

Issue OwnerCurrent Issue Owner: @dukenv0307

@kevinksullivan  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

Cannot recreate

unable to recreate


(IDEA) disable visitor's  auto - vip - upgrade after accepting an invite sent by exhibitor

reason ( @jprzybylski4 )
We've counted the vip zone capacity - it makes no business sense to make a potentially big group of visitors VIPs without any tool for us to check.
We're currently working on precise databases that will be dedicated to finding proper VIPs

@jprzybylski4 i cannot respond at the moment - i must do some checks in various places in the code to see if i can easily use "organizer's settings" to enable/disable this feature


Header buttons does not work if you clicked it once already

I want to report a bug, the news button on top right will not work if you are already in a news article.
Example: it does not work anymore in here: https://opensearch.org/blog/meetings/2021/05/community-meeting/
You have to click something else then the news button is working again
similar behavior on all the button in the header.
I am not sure if this is intended or not, seems like the button state is not reset after clicking once.
Thanks.

Non-clickable, see bottom left without any hyperlink

Likely Related here: https://github.com/opensearch-project/project-website/blob/44db3c93ac5d3105463ae1d0438efbd1eccac8f3/_includes/header.html#L31


Edge 128.0.2739.42 Timeout renderer

After updating Edge to version 128.0.2739.42 or 128.0.2739.54 over the weekend, selenium started failing at taking screenshots.
File "selenium\webdriver\remote\webelement.py", line 347, in screenshot
  File "selenium\webdriver\remote\webelement.py", line 326, in screenshot_as_png
  File "selenium\webdriver\remote\webelement.py", line 315, in screenshot_as_base64
  File "selenium\webdriver\remote\webelement.py", line 395, in _execute
  File "selenium\webdriver\remote\webdriver.py", line 354, in execute
  File "selenium\webdriver\remote\errorhandler.py", line 229, in check_response
selenium.common.exceptions.TimeoutException: Message: timeout: Timed out receiving message from renderer: 10.000
  (Session info: MicrosoftEdge=128.0.2739.42)
Stacktrace:
	GetHandleVerifier [0x00007FF6AFAE3A12+10594]
	Microsoft::Applications::Events::ILogConfiguration::operator* [0x00007FF6AFA80439+323321]
	(No symbol) [0x00007FF6AF89F16E]
	(No symbol) [0x00007FF6AF88ACA3]
	(No symbol) [0x00007FF6AF88AA2E]
	(No symbol) [0x00007FF6AF888C84]
	(No symbol) [0x00007FF6AF8892BF]
	(No symbol) [0x00007FF6AF8ABB90]
	(No symbol) [0x00007FF6AF8E3EDB]
	(No symbol) [0x00007FF6AF907B7A]
	(No symbol) [0x00007FF6AF8DBEDD]
	(No symbol) [0x00007FF6AF8DBDAD]
	(No symbol) [0x00007FF6AF907E60]
	(No symbol) [0x00007FF6AF8DBEDD]
	(No symbol) [0x00007FF6AF921B07]
	(No symbol) [0x00007FF6AF9077E3]
	(No symbol) [0x00007FF6AF8DB3EC]
	(No symbol) [0x00007FF6AF8DA8CD]
	(No symbol) [0x00007FF6AF8DAFB1]
	Microsoft::Applications::Events::EventProperty::empty [0x00007FF6AFC6E4B4+918756]
	Microsoft::Applications::Events::EventProperty::~EventProperty [0x00007FF6AFA0C9BF+54575]
	Microsoft::Applications::Events::EventProperty::~EventProperty [0x00007FF6AF9FF557+199]
	Microsoft::Applications::Events::EventProperty::empty [0x00007FF6AFC6D23D+914029]
	Microsoft::Applications::Events::ILogConfiguration::operator* [0x00007FF6AFA87EC1+354689]
	Microsoft::Applications::Events::ILogConfiguration::operator* [0x00007FF6AFA84374+339508]
	Microsoft::Applications::Events::ILogConfiguration::operator* [0x00007FF6AFA844A9+339817]
	Microsoft::Applications::Events::ILogConfiguration::operator* [0x00007FF6AFA7AD6C+301100]
	BaseThreadInitThunk [0x00007FFF34207374+20]
	RtlUserThreadStart [0x00007FFF351FCC91+33]

Is there a way to increase that renderer timeout?
BTW this is still also problem #124

Same code worked with selenium on version 127.0.2651.105.

Fixed by adding --headless=old default --headless was causing this issue.
Might be generic problem with chromium.


Support for jenv local

Support for configuring a local version (per directory) with jenv local

Hi. Thank you for your suggestion. Unfortunately I have no idea how to implement something like this.
It would be possible to create a command that checks the current directory and then applies the specific JEnv.
I could then modify the cd command so it applys the jenv on every cd. Also the command has to be run on every open of a new terminal.
If you have any ideas how this could be implemented than please answer.
I haven't decided yet if I will implement the above idea as it will require a lot of configuration from the user

version2 is now released and contains the jenv local feature
Thanks for your suggestion and please report any bugs


'final3dTransform' is inaccessible due to 'private' protection level

When I integrated your library at my project I faced with this Error
xxx/LNZSafariTabLayout.swift:387:25: 'final3dTransform' is inaccessible due to 'private' protection level
I think, this problem cause by (private)
private func final3dTransform(forItemOrigin attributesOrigin: CGPoint, andSize attributesSize: CGSize, enforcingAngle angle: CGFloat? = nil) -> CATransform3D
May be it has to be fileprivate?
(because, You use function final3dTransform at extension(at the and of file))

Hello, which swift version are you using and which Xcode version are you using?

swift 4.1
xcode 9.3

The project is in swift 4.1 as well. From Swift 4 if the access level is private, the var can still be used from extensions as long as the extension lives in the same file. My best guess here is that cocoa pods is not marking the pod project swift version to be 4.1


add a site-wide search to the navbar

I'd like something like flickr.com has, where when you start to search it asks what you'd like to find.
For us, it'll be musuems, collections, pixes, faq, etc.

@tacman I have push code in the Survos:RK branch.

So, not quite.  First, go to https://flickr.com/ and run a few searches from the box in the upper right.
I've tweaked our code so make it easier to see the problem.  At a minimum, we need to pass the search string to the controller.