LV.DecryptedLVName and LV.DecryptedLVPath are not filled in on Scan

The LV.Scan() is not populating the DecryptedLVName or DecryptedLVPath entries for a LV.
Example (full 'demo lvm dump-vgs` output):
The dump-vgs knows that the certs logical volume is encrypted, but didn ot
fill in its decryptedLVName and decryptedLVPath even though the device exists.
$ demo lvm dump-vgs
{
  "vg_ifc0": {
    "name": "vg_ifc0",
    "size":<PHONE_NUMBER>96,
    "volumes": {
      "ThinDataLV": {
        "name": "ThinDataLV",
        "path": "",
        "size":<PHONE_NUMBER>8,
        "type": 1,
        "vgname": "vg_ifc0",
        "encrypted": false,
        "decryptedLVName": "",
        "decryptedLVPath": ""
      },
      "certs": {
        "name": "certs",
        "path": "/dev/vg_ifc0/certs",
        "size": 12582912,
        "type": 0,
        "vgname": "vg_ifc0",
        "encrypted": true,
        "decryptedLVName": "",
        "decryptedLVPath": ""
      },
      "config": {
        "name": "config",
        "path": "/dev/vg_ifc0/config",
        "size":<PHONE_NUMBER>,
        "type": 0,
        "vgname": "vg_ifc0",
        "encrypted": false,
        "decryptedLVName": "",
        "decryptedLVPath": ""
      },
      ...

# dmsetup info /dev/mapper/certs_enc /dev/mapper/vg_ifc0-certs
Name:              certs_enc
State:             ACTIVE
Read Ahead:        256
Tables present:    LIVE
Open count:        0
Event number:      0
Major, minor:      252, 6
Number of targets: 1
UUID: CRYPT-LUKS1-dcccbc53c5144aa3a0a3f42b7e75f12a-certs_enc

Name:              vg_ifc0-certs
State:             ACTIVE
Read Ahead:        256
Tables present:    LIVE
Open count:        1
Event number:      0
Major, minor:      252, 5
Number of targets: 1
UUID: LVM-VIGSiwZc4mEXJWRCpBcm0G5S0NsEA8Kf1mPlTvfabaV7jIjiPoYCqHOhHrM5pwcr

Here some info on how to map from LV to encrypted device and back.
mapping luks devices from source <-> luks
The goal of this is to answer two questions:

Given a raw block device, how can you find the luks devices that use it (if any)?
Given a path to a luks device, how can you find the raw device that is on.

Convention is to name the luks device in some way that it indicates the source.  For example, on my Ubuntu 18.04 desktop I have /dev/mapper/nvme0n1p3_crypt which is on nvme0n1p3, but conventions vary.
info
If you have block device, you can tell whether or not that is 'luks' by doing:
$ cryptsetup isLuks /dev/vg_ifc0/certs ; echo $?
0
$ cryptsetup isLuks /dev/vg_ifc0/logs ; echo $?
1
$ cryptsetup isLuks /dev/vg_ifc0/bogus; echo $?
Device /dev/vg_ifc0/bogus doesn't exist or access denied.
4

This might be a bit more obvious than checking return codes with:
# logs is not luks, certs is
$ cryptsetup luksUUID /dev/vg_ifc0/logs
$ cryptsetup luksUUID /dev/vg_ifc0/certs
af6c1825-c324-4fb1-a86a-da51d9a32b94

dmsetup info does not show you enough information.  We can see that vg_ifc0-certs has 1 open count.  certs_enc has a UUID of 'CRYPT-LUKS1--name', where the <uuid> is the fs-uuid of vg_ifc0/certs.  But no where do we see that information here. You could fish around in /dev/ to figure out which device that is.
That gets us from encrypted name to plaintext name, but there is still no way to go the other way no easy way to go the other way.
# dmsetup  info /dev/mapper/certs_enc /dev/mapper/vg_ifc0-certs
Name:              certs_enc
State:             ACTIVE
Read Ahead:        256
Tables present:    LIVE
Open count:        0
Event number:      0
Major, minor:      252, 6
Number of targets: 1
UUID: CRYPT-LUKS1-af6c1825c3244fb1a86ada51d9a32b94-certs_enc

Name:              vg_ifc0-certs
State:             ACTIVE
Read Ahead:        256
Tables present:    LIVE
Open count:        1
Event number:      0
Major, minor:      252, 5
Number of targets: 1
UUID: LVM-RbOIAqSTZ9qtce5Qz0OeQXA8FeDl3JSD3eUW9RPBPQCK6Xn9hHz33HHqZsmd7yBi

## output of stat is in hex, 'fc' is 252
# stat --dereference --format='major: %t minor: %T' /dev/disk/by-id/dm-uuid-CRYPT-LUKS1-af6c1825c3244fb1a86ada51d9a32b94-certs_enc
major: fc minor: 6

solutions via udevadm info
The only way that I can see to do this is via udevadm info.
You can dump the entire udev database with --export-db.  Below
# for n in vg_ifc0-certs certs_enc; do \
#    udevadm info --name=/dev/mapper/$n; done
P: /devices/virtual/block/dm-5
N: dm-5
S: disk/by-id/dm-name-vg_ifc0-certs
S: disk/by-id/dm-uuid-LVM-RbOIAqSTZ9qtce5Qz0OeQXA8FeDl3JSD3eUW9RPBPQCK6Xn9hHz33HHqZsmd7yBi
S: disk/by-uuid/af6c1825-c324-4fb1-a86a-da51d9a32b94
S: mapper/vg_ifc0-certs
S: vg_ifc0/certs
E: DEVLINKS=/dev/disk/by-id/dm-name-vg_ifc0-certs /dev/disk/by-id/dm-uuid-LVM-RbOIAqSTZ9qtce5Qz0OeQXA8FeDl3JSD3eUW9RPBPQCK6Xn9hHz33HHqZsmd7yBi /dev/disk/by-uuid/af6c1825-c324-4fb1-a86a-da51d9a32b94 /dev/mapper/vg_ifc0-certs /dev/vg_ifc0/certs
E: DEVNAME=/dev/dm-5
E: DEVPATH=/devices/virtual/block/dm-5
E: DEVTYPE=disk
E: DM_LV_NAME=certs
E: DM_MULTIPATH_TIMESTAMP=1585655081
E: DM_NAME=vg_ifc0-certs
E: DM_SUSPENDED=0
E: DM_UDEV_DISABLE_LIBRARY_FALLBACK_FLAG=1
E: DM_UDEV_PRIMARY_SOURCE_FLAG=1
E: DM_UDEV_RULES_VSN=2
E: DM_UUID=LVM-RbOIAqSTZ9qtce5Qz0OeQXA8FeDl3JSD3eUW9RPBPQCK6Xn9hHz33HHqZsmd7yBi
E: DM_VG_NAME=vg_ifc0
E: ID_FS_TYPE=crypto_LUKS
E: ID_FS_USAGE=crypto
E: ID_FS_UUID=af6c1825-c324-4fb1-a86a-da51d9a32b94
E: ID_FS_UUID_ENC=af6c1825-c324-4fb1-a86a-da51d9a32b94
E: ID_FS_VERSION=1
E: MAJOR=252
E: MINOR=5
E: MPATH_SBIN_PATH=/sbin
E: SUBSYSTEM=block
E: TAGS=:systemd:
E: USEC_INITIALIZED=183913953

P: /devices/virtual/block/dm-6
N: dm-6
S: disk/by-id/dm-name-certs_enc
S: disk/by-id/dm-uuid-CRYPT-LUKS1-af6c1825c3244fb1a86ada51d9a32b94-certs_enc
S: disk/by-uuid/e2660d39-f66b-4ceb-aed4-ee6f6240c830
S: mapper/certs_enc
E: DEVLINKS=/dev/disk/by-id/dm-name-certs_enc /dev/disk/by-id/dm-uuid-CRYPT-LUKS1-af6c1825c3244fb1a86ada51d9a32b94-certs_enc /dev/disk/by-uuid/e2660d39-f66b-4ceb-aed4-ee6f6240c830 /dev/mapper/certs_enc
E: DEVNAME=/dev/dm-6
E: DEVPATH=/devices/virtual/block/dm-6
E: DEVTYPE=disk
E: DM_MULTIPATH_TIMESTAMP=1585655081
E: DM_NAME=certs_enc
E: DM_SUSPENDED=0
E: DM_UDEV_DISABLE_LIBRARY_FALLBACK_FLAG=1
E: DM_UDEV_PRIMARY_SOURCE_FLAG=1
E: DM_UDEV_RULES_VSN=2
E: DM_UUID=CRYPT-LUKS1-af6c1825c3244fb1a86ada51d9a32b94-certs_enc
E: ID_FS_TYPE=ext4
E: ID_FS_USAGE=filesystem
E: ID_FS_UUID=e2660d39-f66b-4ceb-aed4-ee6f6240c830
E: ID_FS_UUID_ENC=e2660d39-f66b-4ceb-aed4-ee6f6240c830
E: ID_FS_VERSION=1.0
E: MAJOR=252
E: MINOR=6
E: MPATH_SBIN_PATH=/sbin
E: SUBSYSTEM=block
E: TAGS=:systemd:
E: USEC_INITIALIZED=190057092

solution: dmsetup table
Another way to get the information is with dmsetup table.
# time dmsetup table | grep cert
certs_enc: 0 20480 crypt aes-xts-plain64<PHONE_NUMBER>000000000000000000000000000000000000000000000000000000 0 252:5 4096
vg_ifc0-certs: 0 24576 linear 253:19 123996160

# ls -l /dev/block/252:5 /dev/block/253:1 /dev/block/253:19 /dev/mapper/vg_ifc0-certs /dev/dm-5
lrwxrwxrwx. 1 root root      7 Mar 31 15:57 /dev/block/252:5 -> ../dm-5
lrwxrwxrwx. 1 root root      7 Mar 31 15:57 /dev/block/253:1 -> ../vda1
lrwxrwxrwx. 1 root root      7 Mar 31 15:57 /dev/block/253:19 -> ../vdb3
brw-rw----. 1 root disk 252, 5 Mar 31 15:57 /dev/dm-5
lrwxrwxrwx. 1 root root      7 Mar 31 15:57 /dev/mapper/vg_ifc0-certs -> ../dm-5

# cat /run/udev/data/b252:5
S:disk/by-id/dm-name-vg_ifc0-certs
S:disk/by-id/dm-uuid-LVM-RbOIAqSTZ9qtce5Qz0OeQXA8FeDl3JSD3eUW9RPBPQCK6Xn9hHz33HHqZsmd7yBi
S:disk/by-uuid/af6c1825-c324-4fb1-a86a-da51d9a32b94
S:mapper/vg_ifc0-certs
S:vg_ifc0/certs
W:849
I:183913953
E:DM_LV_LAYER=
E:DM_LV_NAME=certs
E:DM_MULTIPATH_TIMESTAMP=1585655081
E:DM_NAME=vg_ifc0-certs
E:DM_SUSPENDED=0
E:DM_UDEV_DISABLE_LIBRARY_FALLBACK_FLAG=1
E:DM_UDEV_PRIMARY_SOURCE_FLAG=1
E:DM_UDEV_RULES_VSN=2
E:DM_UUID=LVM-RbOIAqSTZ9qtce5Qz0OeQXA8FeDl3JSD3eUW9RPBPQCK6Xn9hHz33HHqZsmd7yBi
E:DM_VG_NAME=vg_ifc0
E:ID_FS_TYPE=crypto_LUKS
E:ID_FS_USAGE=crypto
E:ID_FS_UUID=af6c1825-c324-4fb1-a86a-da51d9a32b94
E:ID_FS_UUID_ENC=af6c1825-c324-4fb1-a86a-da51d9a32b94
E:ID_FS_VERSION=1
E:MPATH_SBIN_PATH=/sbin
G:systemd

So if we wanted to see which luks devices were on vg_ifc0-certs, we would find its major/minor with stat and then parse dmsetup table to see if any crypt devices referenced it.
If you want to see which device certs_enc is on, you have to find the device that has major:minor 252:5 (here dm-5), and then find the device name with that.
dmsetup table --concise can give a bit more information than dmsetup table
# dmsetup table --showkey --concise | tr ';' '\n'
vg_ifc0-oci_repo,LVM-RbOIAqSTZ9qtce5Qz0OeQXA8FeDl3JSD0INhXip6dCqFQNIUet2eLb4NqL0T6rAT,11,rw,0 78127104 thin 252:4 1
vg_ifc0-config,LVM-RbOIAqSTZ9qtce5Qz0OeQXA8FeDl3JSDtRZhyazIqWmCxuzdfVkhT1XGIp33y13i,7,rw,0 2097152 linear 253:19 124020736
vg_ifc0-logs,LVM-RbOIAqSTZ9qtce5Qz0OeQXA8FeDl3JSDIoOdOQz0Hw7zWFO0GClP87eYdoKt9T90,9,rw,0 37748736 linear 253:19 163866624
live-base,,1,ro,0 6242304 linear 7:1 0
vg_ifc0-ThinDataLV-tpool,LVM-RbOIAqSTZ9qtce5Qz0OeQXA8FeDl3JSDIL5rifMZGsIa0tNZemYP1e4xl4Kvnev0-tpool,4,rw,0 119537664 thin-pool 252:2 252:3 128 0 0
vg_ifc0-ThinDataLV_tdata,LVM-RbOIAqSTZ9qtce5Qz0OeQXA8FeDl3JSD0WpNB8ZdH1OAUVXsZZ0Bqll8hAVVkJem-tdata,3,rw,0 119537664 linear 253:19 264192
vg_ifc0-ThinDataLV_tmeta,LVM-RbOIAqSTZ9qtce5Qz0OeQXA8FeDl3JSDybUMi1zcjpalOcsxP3LcgAppmx03S4Ks-tmeta,2,rw,0 2097152 linear 253:19 119801856
vg_ifc0-ThinDataLV,LVM-RbOIAqSTZ9qtce5Qz0OeQXA8FeDl3JSDIL5rifMZGsIa0tNZemYP1e4xl4Kvnev0-pool,10,rw,0 119537664 linear 252:4 0
certs_enc,CRYPT-LUKS1-af6c1825c3244fb1a86ada51d9a32b94-certs_enc,6,rw,0 20480 crypt aes-xts-plain64 6e256feea36da9fbea529c6bd804e8530019b75e1741da03f95d5592698ae6df 0 252:5 4096
vg_ifc0-certs,LVM-RbOIAqSTZ9qtce5Qz0OeQXA8FeDl3JSD3eUW9RPBPQCK6Xn9hHz33HHqZsmd7yBi,5,rw,0 24576 linear 253:19 123996160
live-rw,,0,rw,0 6242304 snapshot 7:1 7:2 P 8
vg_ifc0-techsupport,LVM-RbOIAqSTZ9qtce5Qz0OeQXA8FeDl3JSDvJqOI2tmeZqS3glujMdkIirv6Lvkmq1Y,8,rw,0 37748736 linear 253:19 126117888


[KAFKA-8543]Optimize fetch thread allocation

More detailed description of your change,
if necessary. The PR title and PR message become
the squashed commit message, so use a separate
comment to ping reviewers.
Summary of testing strategy (including rationale)
for the feature or bug fix. Unit and/or integration
tests are expected for any behaviour change and
system tests should be considered for larger changes.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

a good finds!


depot tools not on path when fetching

error message:

source code:

currently working on a simple fix in tools/build.py

on a test branch this dirty thingie seems to fix the problem

looks great!

/github.com/line/stellite/issues/33

https://github.com/line/stellite/pull/37

actually it is better to use os.pathsep

https://docs.python.org/3/library/os.html#os.pathsep

i'll send a pull request with pathsep later on :)


xadd & xread & xreadgroup use invariant dict typing

Along with all of the other choices made by the previous maintainers around typing, it seems the mappings passed to xadd, read, and xreadgroup are invariant and do not allow passing in values like dict[str, str].  This makes sense when you're concerned about someone setting an incompatible value into the dict or mapping.  A bit less for cases like this where we just want to read the values.  (I think?  I'm not an expert on Python typing!)
This can be seen with xreadgroup, when passing in the streams and stream IDs:
https://github.com/valkey-io/valkey-py/blob/c490780a1d7b629317e5c0d976ec13dd64de8e55/valkey/commands/core.py#L3972-L3980
If I were to try to interact with this like such:
stream = { "incoming-text": "$" }
payload = cast(Any, client.xread(stream, None, 0))

'course, things like pyright don't like this and emit an error about it.  Invariant types and all that.
Pyright: Argument of type "dict[str, str]" cannot be assigned to parameter "streams" of type "Dict[KeyT, Unknown]" in function "xread"
   "dict[str, str]" is not assignable to "Dict[KeyT, Unknown]"
     Type parameter "_KT@dict" is invariant, but "str" is not the same as "KeyT" (reportArgumentType)

There's a helpful comment about this in the typing.py file.
https://github.com/valkey-io/valkey-py/blob/c490780a1d7b629317e5c0d976ec13dd64de8e55/valkey/typing.py#L42-L46
Using this, we can apply it to StreamIdTto create a version that's covariant.
AnyStreamIdT = TypeVar("AnyStreamIdT", int, bytes, str, memoryview)

 def xreadgroup(
     self,
     groupname: str,
     consumername: str,
-     streams: Dict[KeyT, StreamIdT],
+     streams: Mapping[AnyKeyT, AnyStreamIDT],
     count: Union[int, None] = None,
     block: Union[int, None] = None,
     noack: bool = False,
 ) -> ResponseT:

It seems to work well!  I tested it with the above example with xreadgroup, at least.  I can't seem to find any issues to this approach -- and it seems to be used elsewhere even if not here.

There's an alternative but it's definitely messier.
KT_co = TypeVar("KT_co", covariant=True)
VT_co = TypeVar("VT_co", covariant=True)

class CovariantMapping(Protocol[KT_co, VT_co]):
    def __eq__(self, other: object) -> bool: ...
    def __ne__(self, other: object) -> bool: ...
    def keys(self) -> KeysView[KT_co]: ...
    def items(self) -> ItemsView[KT_co, VT_co]: ...
    def values(self) -> ValuesView[VT_co]: ...

I have a draft PR up that I'm currently testing locally.  Unfortunately, related to #84 (some really awesome effort there!) it's somewhat difficult to check types directly.

Thanks for your report and the draft PR! Yeah I was about to mention #84.
@Rafiot could you also take a look into the attached PR? Does it clash with yours?

First of all, sorry I've not been able to allocate more time to the #84.
#130 seems fine, I don't see any obvious issue with it (note that I'm not an expert in Python typing either). Once merged, I can rebase #84 and keep going (given time).
@imnotjames: if you're interesting in contributing to #84, I'd love that. There is a lot more work to do before it's mergeable.

Just FYI, I re-did the PR (https://github.com/valkey-io/valkey-py/pull/134), this time with signoff so it's somewhat closer to mergeable.

hi @Rafiot
can you write some info about the state of this pr and what you need help with?
it's a bit hard to navigate a pr of this sort

It probably makes more sense to have this discussion in the PR, here it is: https://github.com/valkey-io/valkey-py/pull/134#issuecomment-2514210933


Make jsx-no-script-url use shared settings to specify link components

Why do we have to specify Link components in react/jsx-no-script-url settings when we already have that in shared config? Can't shared config be used here, as it is in react/jsx-no-target-blank?

That's a very good point. This could possibly be a non-breaking change - to make the rule, in the absence of specific inline config, defer to the shared config. It'd be worth a PR.

I can try it this weekend if I find time


[Future Development] Adding coloured line?

Hi,
Possible to add this type of future ? Red dotted lines where to close like

Happy to hear some feedback. You can customize this in v1.0.5
"editor.matchBrackets": false,
"bracketPairColorizer.highlightActiveScope": true,
"bracketPairColorizer.activeScopeBorderStyle": "1px dotted red; opacity: 0.65",


nixos/matomo: fix phpfpm config, add test

Motivation for this change
The module was broken since (probably) a30a1e27:
The option `services.phpfpm.pools.matomo.user' is used but not defined.

This commit also switches the module to socket and settings options introduced in 62b774a7 and 400c6aac.
Test is added to avoid future breakages.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @florianjacob @aanderse @mmahut

Please collaborate with https://github.com/NixOS/nixpkgs/pull/69342 as there is overlap/conflict.


storyboard dim request

do i have to do the storyboard dim in the json file? or on the cmd
im just a kid but when i go to the storyboard section in the json file and put 0 in the dim (example( dim { 0 )
the maps wont load and im not going to put this issue in wieku's becuase he told me to "learn how to use windows" on the last issue i submitted. sorry if this is a stupid request but idk how it works

As far as I'm aware there isn't an option specifically for storyboard dim, it's tied in with the background dim. About the maps not loading, setting the dim to 0 shouldn't break anything, from what you've sent with (example( dim { 0 ), the syntax wouldn't be correct; it should be like this in the file (under Playfield.Background):
"Dim": {
	"Intro": 0,
	"Normal": 0.95,
	"Breaks": 0.5
},

where each of the different values correspond to different parts of the run, eg: Normal would be when the map is playing, Breaks during a break etc. The value you put for them is between 0 and 1, depending on how visible you want it to be.
Hope that helps.

thanks i figured it out


[jni][jnigen] Publish 0.12.0

Take interface implementation out of experimental and publish 0.12.0.

coverage: 90.865% (-0.07%) from 90.938%
when pulling 354c5102217b48d975c207567d727711511b92da on jnigen-0.12.0
into d144f81442894f669108699517d4f4a3f7a529f8 on main.


Allow to combine object shorthand notation with type assertion

Suggestion
🔍 Search Terms
object shorthand notation type assertion
✅ Viability Checklist
My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

⭐ Suggestion
Allow to combine object shorthand notation with type conversions.
📃 Motivating Example
Using external libraries that are not typed very well.
function myFunc(type: string, data: unknown) {
  if (someExternalFunction(data)) {
    // here it's known that `data` is a string
    someOtherFunction({ type, data: data as string });
  }
}

It would be nice to be able to shorten this to
    someOtherFunction({ type, data as string });

or
    someOtherFunction({ type, <string>data });

💻 Use Cases
Just to make source code more concise.

Last time this suggestion was closed due to lack of feedback and a comment to stay "well-clear of this syntactic space": https://github.com/microsoft/TypeScript/issues/13035#issuecomment-867859066

I think we want to stay well-clear of this syntactic space to keep room for future TC39.

Ok... The issue search function did not bring that thing up. (as usual when I try to use the search :) )


Support the Notion Serif Font

Is your feature request related to a problem? Please describe.
Hope for adding support for serif font in the notion page configuration. It's a very beautiful font for publishing-like page, especially for articles written in  Chinese or Japanese characters.
Describe the solution you'd like
Is possible to get the font configuration of certain notion pages and choose different fonts. Just like nobelium can use different page layout according to the notion's Full-width option.
Describe alternatives you've considered
A font type property (e.g default or serif) assigned to every pages can be an alternative but less graceful.
Additional context
an example of page using serif font

Hi, unfortunately, To set serif font for an individual page will not be added for now. During the design process, Nobelium has always kept a clean design philosophy in mind, and we will not add too many useless features unless there is a clear usage scenario.
Getting the font style for a single page is possible, but I can't find a scenario when it would be used.
In the future, perhaps an option to set the whole page to use serif or sans serif will be added. No specific timeline, though.


Option to zip output files

CLI should be expose an option to zip output files after the container runs (useful for downloads from the end user/browser, https://github.com/Computational-Plant-Science/plantit/issues/78)

Always zip (for now)

Always zip (for now)

We should impose a maximum cumulative file size when zipping (1GB seems decent?). If exceeded, zip file should not be created and user must download files individually.

We should impose a maximum cumulative file size when zipping (1GB seems decent?). If exceeded, zip file should not be created and user must download files individually.

Just kidding. Probably cleaner to separate out a zip command with an optional maximum size, then we can wrap this into #21.

Just kidding. Probably cleaner to separate out a zip command with an optional maximum size, then we can wrap this into #21.


iframe design breaking after version upgrade

Is this a bug report or a feature request?
Yes

Have you read the guidelines regarding bug report?
Yes
Have you read the documentation in its entirety?
Yes
Have you made sure that your issue hasn't already been reported/solved?
Yes, sure.
Is the bug specific to iOS or Android? Or can it be reproduced on both platforms?
For both platform, not specific.
Environment
Environment:
React: 16.6.3
React native: 0.57.8
react-native-render-html: 4.1.2
Target Platform:
Android (6.0)
iOS (10.3)

Steps to Reproduce

We have below html structure:

                 Piyalee Maiti

Front End Developer

```

Above html should render like below:

But it's breaking like below :

It's happen from version 4.0.0.

We are in the process of cleaning the issue tracker to make it manageable. Because your issue did not follow our contribution guidelines, it will be closed. We need a full reproducible example to help you out. But you can ask this question on stackoverflow. Don't forget to add the react-native-render-html tag. We are notified on this tag, and will be happy to help you here.


[wip] Add chain params pcli command

going to add the chain info command to this PR as well
Closes #759

Wondering if there's a way we could align this with a general purpose query interface like in #806?

Yeah, I was thinking a lot of this stuff would be nice to have behind some kind of RPC backed by the same metrics we send to Prometheus.


Sortable with file drag and drop, possible?

Problem
I would like to have a sortable list of images. I would also like to be able to drag files from my file system over this list and have a placeholder appear and show where this file will be placed in the list upon drop/upload.
I've been looking at the api but can't really see any obvious way to do this with the events(drag[enter|leave], drop).
Any ideas?
Thanks

We haven't written anything for the file API.
You may have to write the file API stuff yourself. Once you drop the image, you could add the image element to your list of images. Sortable should then update and know that you added a new item.


common :: added height/width to avatar image

For the problem that I mentioned at https://github.com/propelml/propel/issues/301

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

@koab Thanks for fix! I'll just land this in #303 (no need for cla for such small changes)


Firefox browserpass broken in unstable

Describe the bug
After upgrading nixpks unstable and restarting Firefox around mid Oct 2023,
the browserpass extension in firefox stopped working.
It had been a while since I last restarted firefox, which makes it hard to tell when it broke exactly.
When trying to use browser pass via Ctrl-Shift-L or by clicking the browserpass icon in the toolbar I get
Error: Error: No such native application com.github.browserpass.native

Only broken on nixpkgs unstable (via nixos-unstable-small as of 2cb9af4323c64c93e8df3cae5988a53b8687ef3f),
stable still works (nixos-23.05-small as of 0e1cff585c1a85aeab059d3109f66134a8f76935).
Steps To Reproduce
Steps to reproduce the behavior:

Set up firefox with browserpass (programs.browserpass.enable = true; environment.systemPackages = [ pkgs.firefox ];
Confirm working on nixos-23.05 or older version of unstable (eg bfb7dfec93f3b5d7274db109f2990bc889861caf from early September works)
Upgrade to latest unstable (anything that includes d3c27efa36abeb7f65cc8764d73d7805128c5386)
Restart firefox and try to use browserpass extension again

Expected behavior
Browserpass extension should work and fill out username/password fields like it used to.

Additional context
I did some debugging using older nixos generations I had

Sep 2 (generation 133) was working.

Tested via
/nix/store/7sdnrdvzim5mzfkyn5jdscan830i99sr-nixos-system-tobati-23.11.git.bfb7dfec93f3/sw/bin/firefox

Oct 2 (generation 134) was broken.

Tested via
/nix/store/1gdxnf2q3rblj7zhw5hq7ia4jrps6d10-nixos-system-tobati-23.11.git.5d861cc17287/sw/bin/firefox

That's a large range of commits unfortunately (including the upgrade from FF 117.0 to 118.0.1),
but manually looking for relevant commits, d3c27efa36abeb7f65cc8764d73d7805128c5386 stood out to me.
That commit added a new programs.firefox.wrapperConfig setting, and changed
nixpkgs.config.firefox.enableBrowserpass = true to programs.firefox.wrapperConfig.enableBrowserpass = true,
but the new setting isn't actually used anywhere.
[[[[https://github.com/NixOS/nixpkgs/blob/d3c27efa36abeb7f65cc8764d73d7805128c5386/pkgs/applications/networking/browsers/firefox/wrapper.nix#L39]]]]
still references config.firefox.
Notify maintainers

@lovesegfault @mweinelt @K900
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
ivan@tobati:~$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.5.7, NixOS, 23.11 (Tapir), 23.11.git.2cb9af4323c6`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.17.0`
 - channels(root): `"nixos-21.11pre329341.9e70e9f732f"`
 - nixpkgs: `/etc/nixos/aij/unstable`

The new option is passed in as an override to the Firefox package: https://github.com/NixOS/nixpkgs/commit/d3c27efa36abeb7f65cc8764d73d7805128c5386#diff-2517ccea239f5cb479204bf923c5a91e8fb914de25c4493f16ed21ad4a346b97R223
You want to be using programs.firefox.enable so the override is actually applied.

I'm pretty sure I tried programs.firefox.enable and it didn't make a difference. I'll try again though to be sure, since it does seem like it should work...
In the mean time, I found the workaround of setting nixpkgs.config.firefox.enableBrowserpass = true directly.

You need to also remove pkgs.firefox from your environment.systemPackages in that case, and let the module add the overridden package.

You need to also remove pkgs.firefox from your environment.systemPackages in that case, and let the module add the overridden package.

I figured as much. For reference, here is the exact change I made to my config: https://github.com/aij/aij-nixos-config/commit/5151aba60fe81a30709346c4b2ba2605d4ed336b
And I get the No such native application error again.
Comparing the two wrapper scripts:
ivan@tobati:~$ diff -u /nix/store/1wrs7p403hvx1dw0pmz3pa3x07v3wrw0-firefox-118.0.1/bin/firefox /nix/store/rvwbg8ay3y1svszclxhcjjhla19crck6-firefox-118.0.1/bin/firefox
--- /nix/store/1wrs7p403hvx1dw0pmz3pa3x07v3wrw0-firefox-118.0.1/bin/firefox     1969-12-31 18:00:01.000000000 -0600
+++ /nix/store/rvwbg8ay3y1svszclxhcjjhla19crck6-firefox-118.0.1/bin/firefox     1969-12-31 18:00:01.000000000 -0600
@@ -218,14 +218,14 @@
 PATH=${PATH%':'}
 export PATH
 PATH=${PATH:+':'$PATH':'}
-if [[ $PATH != *':''/nix/store/1wrs7p403hvx1dw0pmz3pa3x07v3wrw0-firefox-118.0.1/bin'':'* ]]; then
-    PATH=$PATH'/nix/store/1wrs7p403hvx1dw0pmz3pa3x07v3wrw0-firefox-118.0.1/bin'
+if [[ $PATH != *':''/nix/store/rvwbg8ay3y1svszclxhcjjhla19crck6-firefox-118.0.1/bin'':'* ]]; then
+    PATH=$PATH'/nix/store/rvwbg8ay3y1svszclxhcjjhla19crck6-firefox-118.0.1/bin'
 fi
 PATH=${PATH#':'}
 PATH=${PATH%':'}
 export PATH
 export MOZ_APP_LAUNCHER='firefox'
-export MOZ_SYSTEM_DIR='/nix/store/1wrs7p403hvx1dw0pmz3pa3x07v3wrw0-firefox-118.0.1/lib/mozilla'
+export MOZ_SYSTEM_DIR='/nix/store/rvwbg8ay3y1svszclxhcjjhla19crck6-firefox-118.0.1/lib/mozilla'
 export MOZ_LEGACY_PROFILES='1'
 export MOZ_ALLOW_DOWNGRADE='1'
 XDG_DATA_DIRS=${XDG_DATA_DIRS:+':'$XDG_DATA_DIRS':'}
@@ -273,4 +273,4 @@
 XDG_DATA_DIRS=${XDG_DATA_DIRS#':'}
 XDG_DATA_DIRS=${XDG_DATA_DIRS%':'}
 export XDG_DATA_DIRS
-exec -a "/nix/store/1wrs7p403hvx1dw0pmz3pa3x07v3wrw0-firefox-118.0.1/bin/.firefox-wrapped" "/nix/store/1wrs7p403hvx1dw0pmz3pa3x07v3wrw0-firefox-118.0.1/bin/.firefox-wrapped"  "$@"
+exec -a "/nix/store/rvwbg8ay3y1svszclxhcjjhla19crck6-firefox-118.0.1/bin/.firefox-wrapped" "/nix/store/rvwbg8ay3y1svszclxhcjjhla19crck6-firefox-118.0.1/bin/.firefox-wrapped"  "$@"

ivan@tobati:~$ ls -l /nix/store/1wrs7p403hvx1dw0pmz3pa3x07v3wrw0-firefox-118.0.1/lib/mozilla/native-messaging-hosts/ /nix/store/rvwbg8ay3y1svszclxhcjjhla19crck6-firefox-118.0.1/lib/mozilla/native-messaging-hosts/
/nix/store/1wrs7p403hvx1dw0pmz3pa3x07v3wrw0-firefox-118.0.1/lib/mozilla/native-messaging-hosts/:
total 1
lrwxrwxrwx 1 root root 131 Dec 31  1969 com.github.browserpass.native.json -> /nix/store/mnsaamj4grmzlvqqxmcwffzlih78y5hd-browserpass-3.1.0/lib/mozilla/native-messaging-hosts/com.github.browserpass.native.json

/nix/store/rvwbg8ay3y1svszclxhcjjhla19crck6-firefox-118.0.1/lib/mozilla/native-messaging-hosts/:
total 0
ivan@tobati:~$

Can you try with https://github.com/NixOS/nixpkgs/pull/262017 applied?

Trying again with the latest nixos-unstable-small (44881e03af1c730cbb1d72a4d41274a2c957813a) which includes that change, I get
error:
       … while calling the 'head' builtin

         at /home/ivan/nixops/aij/unstable/lib/attrsets.nix:850:11:

          849|         || pred here (elemAt values 1) (head values) then
          850|           head values
             |           ^
          851|         else

       … while evaluating the attribute 'value'

         at /home/ivan/nixops/aij/unstable/lib/modules.nix:807:9:

          806|     in warnDeprecation opt //
          807|       { value = builtins.addErrorContext "while evaluating the option `${showOption loc}':" value;
             |         ^
          808|         inherit (res.defsFinal') highestPrio;

       (stack trace truncated; use '--show-trace' to show the full trace)

       error: undefined variable 'ff2mpv'

       at /home/ivan/nixops/aij/unstable/nixos/modules/programs/firefox.nix:227:34:

          226|           old.extraNativeMessagingHosts or []
          227|           ++ optional nmh.ff2mpv ff2mpv
             |                                  ^
          228|           ++ optional nmh.euwebid web-eid-app

I fixed that with https://github.com/NixOS/nixpkgs/pull/262617
But I still get the same error from the browserpass extension unless I set nixpkgs.config.firefox.enableBrowserpass = true too.

Please test https://github.com/NixOS/nixpkgs/pull/262623


feat: subnavigation and metrics styles

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #515
Test URLs:

Before: https://main--mammotome--hlxsites.hlx.page/dj-test/en/
After: https://515-subnavigation-and-metrics-styles--mammotome--hlxsites.hlx.page/dj-test/en/

@dzehnder absolutely. fixed the doc. PTAL.


Replaced file path with package version.

Doing this:  "nativescript-bottombar": "file:///Users/rhanb/Documents/DevLove/Plugins/nativescript-bottombar"
On dependecies wont allow users to run demo app.
Instead added this:
"nativescript-bottombar": "^3.0.3"

Hi, thanks you are right ! Completely forgot about that :smile:


fix: typo in alert RFI upload

Implements [NDT-232]

[ ] Check to trigger automatic release process

@AntBush appreciate if you can take another look, I had to remove toast from applicant side


NTP on DHCPV6 (IDFGH-11569)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
Hello ,
I am facing difficulties implementing DHCPv6 stateless configuration on the ESP32 in Ethernet mode. The primary goal is to retrieve the NTP server address from a DHCPv6 server. It seems that there might be a missing definition for LWIP_DHCP6_GET_NTP_SRV. Additionally, I cannot find any debug traces related to DHCPv6.
Code snippet in the on_got_ipv6 callback:
if (ipv6_type == ESP_IP6_ADDR_IS_LINK_LOCAL)
{
    memcpy(&s_ipv6_addr_linklocal, &event->ip6_info.ip, sizeof(s_ipv6_addr_linklocal));
    if (netif == NULL) {
        ESP_LOGE(TAG, "esp-netif is NULL");
    }
    struct netif *lwip_netif = esp_netif_get_netif_impl(&s_ipv6_addr_linklocal);
    netif_set_flags(lwip_netif, NETIF_FLAG_MLD6);
    netif_set_ip6_autoconfig_enabled(lwip_netif, 1);
    #if CONFIG_LWIP_IPV6_DHCP6
    ESP_ERROR_CHECK(dhcp6_enable_stateless(lwip_netif));
    #endif
}

Missing Definition for LWIP_DHCP6_GET_NTP_SRV:

I suspect that there might be a missing definition for LWIP_DHCP6_GET_NTP_SRV. This is critical for obtaining the NTP server address from the DHCPv6 server. Could you please confirm if this definition is included in the ESP-IDF?

No Debug Traces for DHCPv6:

Despite enabling DHCPv6 stateless configuration, I am unable to find any debug traces related to DHCPv6. This makes troubleshooting and debugging the implementation challenging. Are there specific configurations or additional steps required to enable DHCPv6 debugging?
I appreciate any guidance or insights you can provide to resolve these issues. If there are specific logs or configurations I should check, please let me know.
My esp-idf version is v4.4.1 and I can't update it because it's in production..
You can see the log output :

Thank you for your assistance!

hi @ElieSKickMaker
components/lwip/lwip/src/include/lwip/opt.h
This macro LWIP_DHCP6_GET_NTP_SRV is defined in this header file, and you can open it in this file.
If you want to track dhcpv6 log, you can also open this macro DHCP6_DEBUG in the header file above.

Thanks for reporting, feel free to reopen.


Absolute dist css file path should be relative

Description
The path to the final css in the dist/index.html file is an absolute one, e.g.:
<link href="/index-d74e4f4ab3bcfbb06d0bbc88c1ad2bda.css" rel="stylesheet"></head>

this is caused by this configuration entry in the webpack-dist.conf.js file:
new ExtractTextPlugin('/index-[contenthash].css')

This is an issue because it does not permit to deploy the webapp in a different context (e.g. www.mysite.com/something/index.htm does not work) than the root one.
So the path should be relative by default; this can easily be achieved removing the "/" from the ExtractTextPlugin configuration:
new ExtractTextPlugin('index-[contenthash].css')

In addition, the paths to JS files are instead all relatives:
<script type="text/javascript" src="vendor-1ba394c679ccc175943c.js"></script>
<script type="text/javascript" src="app-1ba394c679ccc175943c.js"></script>

Config
{
  "generator-fountain-angular2": {
    "version": "0.6.0",
    "props": {
      "framework": "angular2",
      "modules": "webpack",
      "js": "typescript",
      "css": "scss",
      "resolved": "/home/ufo/Programs/node-v6/lib/node_modules/generator-fountain-webapp/node_modules/generator-fountain-angular2/generators/app/index.js",
      "namespace": "fountain-angular2",
      "argv": {
        "remain": [],
        "cooked": [],
        "original": []
      },
      "skip-cache": false,
      "skip-install": false,
      "sample": "techs",
      "router": "router"
    }
  }
}

Environment
Node.js v6.2.2
linux 4.4.0-24-generic
yo 1.8.4
npm 3.9.5

Hi @ufoscout thanks for the report and for following the template issue, it should be fixed in the next release 😉


Release v1.10.2

Dink v1.10.2 most notably expands message sources for the chat notifier, augments notification metadata, and fixes some edge cases for the loot and death notifiers.

Do we want to add this to VersionManager:
register("1.10.2", "Custom (non-discord) webhook handlers now receive world and region for each notification, unless 'Advanced > Include Location' is disabled");

Do we want to add this to VersionManager:
register("1.10.2", "Custom (non-discord) webhook handlers now receive world and region for each notification, unless 'Advanced > Include Location' is disabled");

I don't think this is needed - this changelog entry won't matter for 99.9% of users (pulled this number out of my ass)


sys.path bug

This project has a bug in it
   pprint(sys.path)
    try:
        sys.path.insert(0,
        r"c:\study\iumsds\fall2019\cloudcomputing\fa19-516-155\cloudmesh-transfer")
    except Exception as e:
        print("PYTHONPATH not updated.")

obviously you can not update sys.path this way or any other way. this must be just working with pip install ...

I have deleted the code which was updating PYTHONPATH.
It was added so that the code can be debugged in pycharm without any configuration change to pycharm.
Thank you for catching this bug.


fix(vsc): Update vsc launch.json

What this PR does 📖

Updates the vsc launch.json to match the rename from ui to uplink

Will go on and merge this since this doesnt change the underlining code


Error when running on Windows10

In minhash.py line 223
mp.set_start_method("fork", force=True)

should be
mp.set_start_method("spawn", force=True)

in Windows OS.
https://github.com/woven-planet/l5kit/issues/129

I think the answer is more involved than that.
Spawn is more expensive but fork is unavailable in windows.
Also there are sane safe defaults for each platform.
I think the answer should be what the original issue you refer to takes.
If sys.platform == Darwin
mp.set_startmethod
Like this.
I'll try to prepare a pr that also adds comments

@soonjune can you try and see if this works for you?
https://github.com/ChenghaoMou/text-dedup/pull/51

Fork behaviour is required for the copy-on-write mechanism to work on the uf object. This was discussed in https://github.com/ChenghaoMou/text-dedup/issues/26#issuecomment-1636787593. Until someone has access to a Windows machine and thoroughly tests the memory impact, allowing spawn might lead to more problems than it solves.

Considering that it isnt JUST a MacOS thing, I resorted to documenting it in a PR with various housekeeping changes.

I am closing it for now. Feel free to reopen if you have any other issues.


Existing tag structures are not maintained when copying pages.

Hi, I've been evaluating pdf-lib to see if it will work for my project and I've run into an issue I thought I'd report. When loading a PDF that has been externally "tagged" for accessibility, the tag structure is maintained if the PDF is immediately exported to file without any modifications, however, when copying pages which contain tagged structures into pdf, the tags for the copied pages are lost.

Hello @chrisjpatty!
Thanks for reporting this. The issue you're encountering has the same root cause as https://github.com/Hopding/pdf-lib/issues/218, https://github.com/Hopding/pdf-lib/issues/341, and https://github.com/Hopding/pdf-lib/issues/159.

Closed in favor of other issues that track the same root cause. I'll probably create a new parent issue to track this in the future.


[Concurrency] Fix an issue with Sendable checking for function references.

Add a test case for passing a non-Sendable argument to a nonisolated closure in an actor-isolated context. This previously did not produce a diagnostic, but now does on main.
Don't check parameter and result types of function references for Sendability. Sendability of function references depends only on captures, and applies when the function itself is passed across isolation boundaries. Parameter and result values can only cross isolation boundaries when the function is called, so they shouldn't be checked when checking a reference for Sendable violations.

Resolves rdar://104129168

@swift-ci please smoke test

@swift-ci please smoke test Linux


add missing APIs

Pull request
Issue tracker
Fixes will automatically close the related issue
Fixes #
Feature
Add APIs that are already available in one form but not exported to the other form
Unit test

[x] Done

Manual test

[x] Done

Code Review

Unit and manual test pass
GitHub Action CI pass
At least one contributor reviews and votes
Can be merged clean without conflicts
PR will be merged by rebase upstream base

Additional Info

The librime 0.9 APIs are deprecated. New API functions added to librime 1.x are not exported as you try to do in this PR.
Would RimeApi do what you want?

The librime 0.9 APIs are deprecated. New API functions added to librime 1.x are not exported as you try to do in this PR.
Would RimeApi do what you want?


Fix name for istio canonical service names.

I accidentally copied the wrong label name. The label name is "canonical-name" not "canonical-service":
https://docs.google.com/document/d/1EsKpYNW55-LqG08zdjp1tAd224T9IEG3CEnusmL-Joc/edit#heading=h.lxsagz3b8886.
I have asked about exporting the label strings from istio side. Currently it's exported on pilot, and I don't think this should have a dependency on pilot: https://github.com/istio/istio/blob/2d1b8d6fc2a30787d397770e6f6aa1eec6a46b03/pilot/pkg/model/service.go#L151

/assign @tcnghia

/assign @vagababov


Fragment shader doesn't support indexing uniform array with int variable.

Steps to reproduce
Bellow is the content of image_processor.frag:
#include <flutter/runtime_effect.glsl>

const int CURVE_SIZE = 512;

out vec4 fragColor;
uniform vec2 uSize;
uniform float curves[CURVE_SIZE * CURVES_NUM];
uniform sampler2D image;

void main() {
    vec2 iResolution = uSize;
    vec2 fragCoord = FlutterFragCoord();
    vec2 uv = fragCoord/iResolution.xy;
    vec3 rgb = texture(image, uv).xyz;

    int idx = 0;
    rgb.x = rgb.x * curves[0]; // change this line to rgb.x = rgb.x * curves[idx];

    fragColor = vec4(rgb, 1.0);
}

It works as expected. But if I change only a single:
rgb.x = rgb.x * curves[0];   -->  rgb.x = rgb.x * curves[idx];
Instead of indexing uniform array with 0, I index it with variable idx, which is equal to 0 too - it fails to load the shader with
"RuntimeEffect error" here:
 var program = await FragmentProgram.fromAsset('shaders/image_processor.frag');
So it isn't possible to index uniform array with a variable - while I think this should be possible in GLSL.
Flutter Doctor output
Doctor output
$ flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.22.0, on Microsoft Windows [Version 10.0.19045.4291], locale en-US)
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[√] Chrome - develop for the web
[√] Visual Studio - develop Windows apps (Visual Studio Enterprise 2019 16.6.2)
[!] Android Studio (version 2021.2)
    X Unable to determine bundled Java version.
[√] Android Studio (version 2023.2)
[√] VS Code (version 1.89.1)
[√] Connected device (3 available)
[√] Network resources

! Doctor found issues in 1 category.

It seems that this is a limitation in versions of GLSL before 4.0:
https://stackoverflow.com/questions/45167538/error-sampler-arrays-indexed-with-non-constant-expressions-are-forbidden-in-gl

I just wanted to say that indexing uniform array is a very useful/needed feature in shaders. Overwise you need to pack floats into texture pixels somehow - which is so hard to do correctly and requires many operations to convert float into 4 bytes -> which will be converted into 4 normalized extra low precision floats in shader -> then convert  them into one normal precision float.

Unfortunately this is a limition of SkSL, Skia's shading language which is used on non-Impeller platforms. Besides completing the transition to Impeller on more platforms (allowing us to support newer GLSL versions), there isn't anything we can do to fix this.

@jonahwilliams so if I enable Impeller for my specific platform -  will I be able to use this feature already now?


TreeView should not capture Alt+Left or Cmd+Left (browser navigate back shortcuts)

Description
When I'm on a TreeView and select a collapsable node in the tree, Cmd+Left (or Alt+Left in Windows) are captured by the TreeView to collapse the node, instead of navigating to the previous page. These are browser native shortcuts that should not be overridden.
Source issue: https://github.com/github/repos/issues/4536
Steps to reproduce

Navigate to any Repos CodeView page e.g. nodejs/node
Click on any folder in the tree
Use the keyboard LeftArrow key to collapse it (or RightArrow to expand)
Use Cmd+LeftArrow (or Alt+LeftArrow in Windows)

Expected: I navigate to previous page
Actual: node is collapsed like if no modifier (Cmd or Alt) where pressed.
Version
v35.17.0
Browser
Chrome

@colebemis to see if he can reproduce


nbclassic 0.5.3, try restoring tests

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin please rerender

notebook downstream test still failing... will issue up.


A re-train normal CNN on scaled data beats deform-conv.

First, appreciate for your work which is easy to use and read.
In scaled_mninst.py, the normal CNN model is trained on origin data and then, tested on scaled one. It shows a bad acc of 60% (In my running). Then, you fine-tune a deform-conv on the scaled data and its accuracy is much better. However, I tried to re-train this trained CNN model on scaled data and the result confuses me definitely. It gets 96% on origin test and 98% on scaled data.
Well, this experiment can not prove the effectiveness of deform-conv layers.

I meet the same problem,I think the reason is that the mnist is too easy for them to learn.I delete first two Deformable layer  and only keep the last one seems to have a better result.

@oeway Thanks for your reply.appreciate for your work which is helpful for me.


Make all ACF data available with GraphQL

Right now only some fields are included in the GraphQL data, but we need all of them. In particular, relationship and repeater fields.
Check out this issue discussion for ideas on how to solve this: https://github.com/wp-graphql/wp-graphql/issues/214

Also have a look at the WPGraphQL Docs


very big problems in pob

Hello developer I have a lot of problems with your path of building program.
this screenshot shows my version of POB https://gyazo.com/cd18442fbb96ff7f86f60145efa93731.
when I copy someone's POB like this https://gyazo.com/025a64c429f832d68fdd313fed2eca59.
this is what I get when I insert it, but it's not the main problem. Its solution I found it is necessary to remove the letter s in https.
Then I had all the rules inserted, but after a while it stopped working. here's a screenshot when I insert without https. https://gyazo.com/deaeeceaed94dfcc7df59c70099597c9
and this screen when without the letter S (http). https://gyazo.com/f202488ba5530fce45f698bbd1bd6394
2nd different errors.
also in my POB don't share code, when I click Share with Pastebin I get an error
https://gyazo.com/affa9baca8635ae4ae0a6328961e77a9
Helm me plz.

Such a problem not have one me here is the link to forum there, too, most write, but adequate solutions there.
https://www.pathofexile.com/forum/view-thread/2912966

i have been having the same issue. it says "unknown talent tree version

"


Groovy hook scripts not found

I extended the image with some customs groovy scripts to setup credentials, but I couldn't found the scripts inside the container in runtime
FROM jenkins/jenkins:2.89.2
MAINTAINER mlabouardy<EMAIL_ADDRESS>
COPY plugins.txt /usr/share/jenkins/ref/plugins.txt
RUN /usr/local/bin/install-plugins.sh < /usr/share/jenkins/ref/plugins.txt

COPY security.groovy /usr/share/jenkins/ref/init.groovy.d/security.groovy
COPY setup-credentials.groovy /usr/share/jenkins/ref/init.groovy.d/setup-credentials.groovy

Docker commands that you execute
I'm using the following compose to deploy the service:
version: "3.3"

services:
  jenkins:
    image: mlabouardy/jenkins
    ports:
      - 8080:8080
    secrets:
      - jenkins-username
      - jenkins-password
      - registry-username
      - registry-password
      - ssh-username
    configs:
      - source: private_key
        target: /var/jenkins_home/.ssh/id_rsa
        mode: 0600
        uid: '1000'
        gid: '1000'
      - source: public_key
        target: /var/jenkins_home/.ssh/id_rsa.pub
        mode: 0600
        uid: '1000'
        gid: '1000'
    deploy:
      replicas: 1
      restart_policy:
        condition: on-failure
      update_config:
        parallelism: 1
        delay: 10s
        failure_action: rollback
      placement:
        constraints:
          - node.role == manager

configs:
  private_key:
    file: /home/mlabouardy/.ssh/id_rsa
  public_key:
    file: /home/mlabouardy/.ssh/id_rsa.pub

secrets:
  jenkins-username:
    external: true
  jenkins-password:
    external: true
  registry-username:
    external: true
  registry-password:
    external: true
  ssh-username:
    external: true

Actual result
Scripts are not copied over.


Port to Django 1.11

Can You describe this issue bit more ?

I updated my django version from 1.9.7 to 1.11.7 and whole project is running smooth and fine without any issue or error.

If everything is running fine then just update the requirements.txt

Ok I will update requirements.txt


object vs list of objects in the API output

Hi,
In our project Cybele, we are trying to integrate and reuse the API in other services; however, there is one issue we are not sure how to handle.
In the endpoint: http://grlc.io/api-git/rapw3k/cybele/#/json/get_allDatasets
we have among others this specification.
"measure": {
        "id": "$qb:component/qb:measure",
        "label": "$rdfs:label",
        "range": "$rdfs:range",
        "UoM": "$sdmx-attribute:unitMeasure",
        "description": "$rdfs:comment"
      }

The output of this endpoint returns a list of objects when there are more than one resource associated, but when is only one resource associated it is returned as a simple object. Is there a way we specify that the output should always be a list of object, even if its only one result, or any work around this?
Example output with more than one "measure"
"measure": [
        {
          "UoM": "http://qudt.org/vocab/unit/DEG_C",
          "description": "The average air temperature in Degree Celsius",
          "id": "https://w3id.org/cybele/measure/averageAirTemperature",
          "label": "Average air temperature",
          "range": "http://www.w3.org/2001/XMLSchema#double"
        },
        {
          "UoM": "http://qudt.org/vocab/unit/M-PER-SEC",
          "description": "The average wind speed in meter per second",
          "id": "https://w3id.org/cybele/measure/averageWindSpeed",
          "label": "Wind speed",
          "range": "http://www.w3.org/2001/XMLSchema#double"
        }
      ]

Example output with only one "measure"
"measure": {
        "description": "Number of food recall incidents per day",
        "id": "https://w3id.org/cybele/measure/incidencesNumber",
        "label": "DOC_COUNT",
        "range": "http://www.w3.org/2001/XMLSchema#int"
      }

Hi @rapw3k -- very good question, I'm not 100% sure how to achieve this. I'm going to pass this question to @pasqLisena who is the real expert in Transformer syntax and capabilities.

Hi @rapw3k , it is not implemented now, but it is a good feature to have!
I opened a proper issue in the SPARQL transformer repository

thank you @c-martinez , @pasqLisena, looking forward for this update!

I will close this issue and we can track progress on the SPARQL Transformer
issue.

Hi, I implemented the feature. Now your query can become:
"measure": {
        "id": "$qb:component/qb:measure$list",
        "label": "$rdfs:label",
        "range": "$rdfs:range",
        "UoM": "$sdmx-attribute:unitMeasure",
        "description": "$rdfs:comment"
      }

In short, by putting $list on a property (or on the id/anchor, for targeting the whole object), the output is forced to be a list.
@c-martinez would you mind updating the SPARQLTransformer version to latest (2.1.1)?

Hi @pasqLisena -- I've updated SPARQLTransformer version in the dev version. Should be updated in the master branch during the next release (will discuss with @albertmeronyo when that will be, but probably soon).

hi @c-martinez ,thats really cool, thanks!
tested it and works fine.
do you announce when you deploy a new release?

hi @rapw3k, great, glad it works.
We usually tweet whenever there is a new release. Next one should be out in the next couple of weeks.


system: Change DRIVER_NOTExxx to DRIVERS_NOTExxx

Summary
follow the kernel side change: https://github.com/apache/nuttx/pull/8531
Impact
Minor
Testing
CI

Is it breaking because the board configs are not up to date yet?

All update, but it merge with apache/nuttx#8531 tegather to fix the break.


Android support

With the current breakthrough from the Beeware project it is now possible to have flexx run on Android on iOS since all we need is a webview and a network connection. Since beeware's breakthrough allows almost all pure python code to work, the only feature that might need work is changing the webview it runs on.
Is this desired work?

That would certainly be interesting! I don't have the time to lead such an effort right now, but contributions to make this happen are welcome :)  I'm not sure what this would look like. I imagine there should be a compilation step of some sorts, so maybe it's better to implement this as a new project (which we can include in the flexxui org)?

Looking into beeware's tutorial, they use their packaging tool, briefcase that knows how to build apps for Linux, iOS, and android. I think we could leverage it to do the whole packaging.


TSS-395/use-ecr-pull-through-cache

TSS-395
This PR switches the kafka custom action to use our ecr pull through cache, both images arou around 300mb, so on each PR we are pulling at least 600mb from the internet.

Closing this PR since we would need additional configuration on the running side.


Missing Bracket in _Wifi.ino (will not compile if wifimanager=1)

Hi
you are missing a bracket in _Wifi.ino on rule 72
if **(**  (debug >=0 ) && ( pdebug & P_MAIN )) {

Is will not compile you enable wifimanager.

 I will correct in next version.
Thanks!

Done


Add godep dependency_manager

Adds godep support for gathering licenses: when Godeps/Godeps.json is present, it will add all Deps entries, treating either the comment (usually a tag) or the revision as version. Will look for license files in $GOPATH, as this is where godep restore will put all the packages mentioned in Godeps.json.

🎉  👍  💃


fluentd v1.0 support?

Hi,
Any  plans to support Fluentd v1.0?  I would like to take advantage of the native multi-process support version >=1.x offers.  Currently this plugin only supports fluentd version <= 0.12.43.
Thanks

/usr/lib/ruby/2.3.0/rubygems/specification.rb:2289:in `raise_if_conflicts': Unable to activate fluent-plugin-udp-native-sensors-0.0.1, because fluentd-1.2.4 conflicts with fluentd (<= 0.12.43) (Gem::ConflictError)

Got around the issue with https://hub.docker.com/r/eslam/pen/
see
https://github.com/Juniper/open-nti/tree/master/plugins/input-jti#docker-compose


FloatingBlockEntities don't drop the correct item

In this example, though nothing should be dropped by this gale grass, a gale grass item is dropped anyway.
https://cdn.discordapp.com/attachments/770698788629118997/856704612198514688/Minecraft__1.17_-_Singleplayer_2021-06-21_18-17-52.mp4

Resolved by #364


TCP connections are kept open after page is loaded

<EMAIL_ADDRESS>commented:

Which version of PhantomJS are you using? Tip: run 'phantomjs --version'.

$ phantomjs --version
1.3.0
What steps will reproduce the problem?

Run this script:

for (var i = 1; i < 100; i++) {
page = new WebPage();
page.open('http://google.com')
}

Grep for the PID of the phantomjs process
Run "lsof -p <pid>"

What is the expected output? What do you see instead?

I expect that the TCP connection should be closed once the page has been loaded.
Which operating system are you using?

Fedora 15
Did you use binary PhantomJS or did you compile it from source?

Compiled
Please provide any additional information below.

Disclaimer:
This issue was migrated on 2013-03-15 from the project's former issue tracker on Google Code, Issue #253.
:star2:   3 people had starred this issue at the time of migration.

There might still be a bug here.  Yes, PhantomJS should keep some connections open in case it is about to be asked to load more pages from the same site, just as "normal" browsers do.  But it shouldn't be possible to exhaust the client-side limit on the number of open sockets that way, and there should be a limit on the number of connections held open to the same server (again, just as "normal" browsers do).
This may or may not have been addressed by the upgrade to Qt 5 in PhantomJS 2.0.


Adding Error behavior in folder not found condition

This solution can help project.yml configurations to be shared across different projects with different folder structure. With that, even when a path that doesn't exist is specified under 'source' configuration for example, the user will be able to chose what is the expected behavior:

ERROR: Error is logged and the build is stopped
WARNING: Warning is logged but build continues
HIDDEN: Nothing is logged

In the future, this feature can also implement other behaviors, not only the 'not_found' as created in this pull request.

This PR has been open for a silly amount of time. Please forgive us, @beckerzito. This is an interesting idea. Thank you for the submission and the thought that went into this. It doesn't appear that this feature garnered support from others. I am going to close this PR. I think that between Ceedling's support for globs in :paths and its new Mixins feature for merging bits of project configuration at startup variable path structures per project should be handled without the need for special levels of error handling. Generally, we prefer to be explicit in project configurations rather than overly permissive. It tends to limit larger problems of unforeseen feature interactions.


Issues/151 - Add script to generate merged static-sky coadd catalogs as HDF5 files.

Adds merged_tract_cat.py and example_merge_tract_cat.py to script directory.
Creates one HDF5 file per tract with all ~2400 columns from coadd merged reference catalogs + 6 filters.
Specific questions to reviewers:

Look in /global/projecta/projectdirs/lsst/global/in2p3/Run1.1-test2/summary on NERSC for summaries generated from the Run 1.1p process done at IN2P3 and transferred to NERSC.  Load some catalogs and try things out.
Look in examples_merge_tract_cat.example_load_and_plot for an example use and plotting.

[ ] Should the HDF5 files have a separate HDF Groups for each patch or just one Group for the entire tract?  I'm somewhat new to HDF5 -- is there a third, better way?

You can read in with key='*' in DASK read_hdf but not with Pandas read_hdf.

[ ] If separate groups per patch is there an easy way for the user who just wants everything to grab that without having to know the names of group identifier?
[ ] If separate groups per patch, should they be named coadd_4849_1,1 or coadd_4849_11.  The latter would allow the identifier to be used as an attribute because it's a valid Python identifier.
[ ] Should there be a light-weight HDF5 file with only the most useful columns?  If so, should this be separated by patch as well for consistency (if we decide to go that way), or should it be merged across patches for convenience?
[ ] example_merge_tract_cat.py presents a list of examples as functions and then has them turned on/off by booleans in the main run.  Is there a better way to present these examples?

How about using the table format when saving to hdf so that the user can load a subset of columns? Specifically,
<table>.to_pandas().to_hdf(filebase+'.hdf5', 'coadd', data_columns=True, format='table')

cf. http://pandas-docs.github.io/pandas-docs-travis/io.html#table-format

@wmwv I was trying to run your updated script to generate hdf5 files that uses table as the data format but ran into the error below. Any idea?
Here's what I ran:
python ../DC2_Repo/scripts/merge_tract_cat.py /global/projecta/projectdirs/lsst/global/in2p3/Run1.1-test2/output 4637

And here's the output and error messages
CameraMapper INFO: Loading exposure registry from /global/projecta/projectdirs/lsst/global/in2p3/Run1.1-test2/input/registry.sqlite3
Processing tract 4637, patch 0,0
No locations for get: datasetType:deepCoadd_forced_src dataId:DataId(initialdata={'tract': 4637, 'patch': '0,0', 'filter': 'r'}, tag=set())
No locations for get: datasetType:deepCoadd_forced_src dataId:DataId(initialdata={'tract': 4637, 'patch': '0,0', 'filter': 'i'}, tag=set())

Traceback (most recent call last):
  File "../DC2_Repo/scripts/merge_tract_cat.py", line 292, in <module>
    load_and_save_tract(args.repo, tract, filename, verbose=args.verbose)
  File "../DC2_Repo/scripts/merge_tract_cat.py", line 99, in load_and_save_tract
    patch_merged_cat.to_pandas().to_hdf(filename, key, format='table')
  File "/global/common/software/lsst/common/miniconda/py3-4.3.21/lib/python3.6/site-packages/pandas/core/generic.py", line 1299, in to_hdf
    return pytables.to_hdf(path_or_buf, key, self, **kwargs)
  File "/global/common/software/lsst/common/miniconda/py3-4.3.21/lib/python3.6/site-packages/pandas/io/pytables.py", line 280, in to_hdf
    f(store)
  File "/global/common/software/lsst/common/miniconda/py3-4.3.21/lib/python3.6/site-packages/pandas/io/pytables.py", line 274, in <lambda>
    f = lambda store: store.put(key, value, **kwargs)
  File "/global/common/software/lsst/common/miniconda/py3-4.3.21/lib/python3.6/site-packages/pandas/io/pytables.py", line 891, in put
    self._write_to_group(key, value, append=append, **kwargs)
  File "/global/common/software/lsst/common/miniconda/py3-4.3.21/lib/python3.6/site-packages/pandas/io/pytables.py", line 1333, in _write_to_group
    s.write(obj=value, append=append, complib=complib, **kwargs)
  File "/global/common/software/lsst/common/miniconda/py3-4.3.21/lib/python3.6/site-packages/pandas/io/pytables.py", line 3904, in write
    self.set_attrs()
  File "/global/common/software/lsst/common/miniconda/py3-4.3.21/lib/python3.6/site-packages/pandas/io/pytables.py", line 3153, in set_attrs
    self.attrs.non_index_axes = self.non_index_axes
  File "/global/common/software/lsst/common/miniconda/py3-4.3.21/lib/python3.6/site-packages/tables/attributeset.py", line 494, in __setattr__
    self._g__setattr(name, value)
  File "/global/common/software/lsst/common/miniconda/py3-4.3.21/lib/python3.6/site-packages/tables/attributeset.py", line 436, in _g__setattr
    self._g_setattr(self._v_node, name, stvalue)
  File "tables/hdf5extension.pyx", line 712, in tables.hdf5extension.AttributeSet._g_setattr
tables.exceptions.HDF5ExtError: HDF5 error back trace

  File "H5A.c", line 259, in H5Acreate2
    unable to create attribute
  File "H5Aint.c", line 280, in H5A_create
    unable to create attribute in object header
  File "H5Oattribute.c", line 347, in H5O_attr_create
    unable to create new attribute in header
  File "H5Omessage.c", line 224, in H5O_msg_append_real
    unable to create new message
  File "H5Omessage.c", line 1945, in H5O_msg_alloc
    unable to allocate space for message
  File "H5Oalloc.c", line 1142, in H5O_alloc
    object header message is too large

End of HDF5 error back trace

Can't set attribute 'non_index_axes' in node:
 /coadd_4637_00 (Group) ''.

@yymao It looks like it's fundamental to HDF and PyTables.  The table format has an effective limit on the number of columns -- the total header size is limited to 64kB.
https://github.com/pandas-dev/pandas/issues/9590
I propose we stick with the fixed format for now.

@wmwv hmm, ok. Yes, it's fine for now. But fixed format is slower to load when the user only needs a small subset of columns (since it requires loading the whole table at once), so we might still want to come up with alternatives later.

OK, #158

@yymao @cwwalter @nicolaschotard @jchiang87
Could I kindly ask for one of you to sign-off on this ticket so that I can merge it in?


What are the list of things that will be required to take this to 1.0?

I don't want to speak for Jeremy, Sylvain, or the rest of the fast.ai community, but I suspect SwiftAI might not want to go 1.0 until the underlying Swift for TensorFlow platform achieves 1.0. On that front, we have a number of key aspects we need to improve (chiefly around performance, but also distributed training / inference, and ensuring the underlying abstractions support high quality high-level APIs such as SwiftAI itself!). These are all top-of-mind for the team, but we're optimizing for high quality, instead of shortest-time-to-1.0. :-)

I agree with @saeta . In general, all discussion around the library should be on the forum so that all the community can see :)
Closing this as we keep issues for known bugs only.


Using VUART with EFR32MG21

It would be important to have the possibility to change the CLI to use VUART instead the USART so the USART can be free for other uses.
Any instructions on how to do this ?
platform: efr32

I'm afraid at the moment we don't currently support CLI via VUART on Open Thread by default.


Inventory of Zoom hostkeys for Zowe squads

Which Zowe squads are assigned which OMP provisioned zoom hostkeys?
Context: https://github.com/zowe/vscode-extension-for-zowe/issues/732

I think we got through this latest PI Planning without issues - I would like to close this issue - if there is some action needed for next PI please created a new issue with needed actions


How to tune OPENTHREAD_CONFIG to 32 child per parent

Seeking advice on our approach
We have a customer request to have a TBR or TME that can buffer 8 frames (one full MTU) per child plus 100 shared frames.  Which works out to 32 +13 = 45 MTUs (full 1280-byte IP packets)
The code offers these CONFIG options:

#define OPENTHREAD_CONFIG_MESSAGE_BUFFER_SIZE (sizeof(void *) * 32) // which is 128 bytes in a 32-bit system
#define OPENTHREAD_CONFIG_NUM_MESSAGE_BUFFERS 256 // which we bump to 512
#define OPENTHREAD_CONFIG_DEFAULT_SED_BUFFER_SIZE 1280 // which we kept as is
#define OPENTHREAD_CONFIG_DEFAULT_SED_DATAGRAM_COUNT 1 // which we change to 13

Rationale for the math
Why OPENTHREAD_CONFIG_NUM_MESSAGE_BUFFERS changed?

Each #define OPENTHREAD_CONFIG_MESSAGE_BUFFER_SIZE (sizeof(void *) * 32) means the message buffer is 4*32 = 128 bytes
to support full 1280-byte IP packets we need 10 OPENTHREAD_CONFIG_MESSAGE_BUFFER_SIZE
we need 45 MTUs (1280-bytes each) so we need 45*10 = 450 OPENTHREAD_CONFIG_NUM_MESSAGE_BUFFERS which is why we are changing to 512 (rounding up for wiggle room and to account for the 100 shared frames)

Why OPENTHREAD_CONFIG_DEFAULT_SED_DATAGRAM_COUNT changed?

The request is to buffer 8 frames (one full MTU) per child each MTU is 1280-byte and each SED datagram is 106-octet (see the info on OPENTHREAD_CONFIG_DEFAULT_SED_DATAGRAM_COUNT, to contain 1280 we need 1280/106 = 12.07, so we round up to 13.

Questions

Does the math checks out or am I missing something?
Any other CONFIG option that we should consider/tune for the goal?
Once a message is moved to the SED Buffer does it not longer consume space in the Message Buffer?

Thanks for everyone's time.

Closing issue - moving this to discussions (seems like the more appropriate channel for it) - https://github.com/orgs/openthread/discussions/9950


Improvement[VectorStore] Optimize VectorStoreRetriever

Optimize VectorStoreRetriever:

Store similarity_score in metadata of document when search_type is "similarity_score_threshold".
Improve test cases for "similarity_score".

@eycjur @ccurme Please help to review this pr.

@ZhangShenao Thanks for creating the pull request!
It appears that some implementations insert the similarity score directly into the document metadata on the vector store side. Therefore, it would be beneficial to standardize whether the score is added at the vector store level or at the VectorStoreRetriever level. Additionally, the key used for this score should be consistent—whether it’s labeled as score or similarity_score.
Example code reference: LangChain MongoDB Atlas vector store
        if additional and "similarity_score" in additional:
            for doc, score in docs_and_scores:
                doc.metadata["score"] = score
        return [doc for doc, _ in docs_and_scores]

Let me know if any further adjustments are needed!

Thank you for your comment. I haven't thought of any adjustments at the moment. If you have any suggestions, please feel free to let me know at any time.

@ZhangShenao Thank you for your comment.
Since I’m not fully familiar with the original implementation intent, I can’t say that it’s good as-is, nor can I enforce any changes. I would like to leave the decision to the owner.

Appreciate the contribution, and happy to apply this on a vectorstore-by-vectorstore basis at the moment. For now, you can get these by calling vectorstore.similarity_search_with_relevance_scores and reading the tuples.
Reason is that adding unexpected metadata to documents may break some integrations / the ways people are expecting them to behave.
We'll consider this as part of some potential vectorstore improvements in a minor bump though!

Appreciate the contribution, and happy to apply this on a vectorstore-by-vectorstore basis at the moment. For now, you can get these by calling vectorstore.similarity_search_with_relevance_scores and reading the tuples.
Reason is that adding unexpected metadata to documents may break some integrations / the ways people are expecting them to behave.
We'll consider this as part of some potential vectorstore improvements in a minor bump though!

OK, thanks!


New API for downloading LDML files from projects that allow sharing WS data

Currently this adds a "ldmlzip" command to our hg command runner.
Will return 403 Forbidden if project does not allow sharing ws data with SLDR. Will also return same 403 Forbidden error code if project does not exist, to avoid possibly leaking project codes.
If project exists and allows data sharing, command will return a zipfile containing CachedSettings/WritingSystems/*.ldml from the tip revision.
Future plans: C# code will unzip the *.ldml files into a directory by project ID, then zip up the whole thing for the API result. Then we'll create a job to delete the LDML directory four hours later, to clean up.
Ideally we'd be able to start streaming the output zip file as we receive data from each project, rather than waiting until all projects have been accessed and only then starting to stream the output zip. But we'll see.

Unfortunately it looks as though we won't be able to use ZipArchive to add files to the output stream asynchronously as they come in: we'd probably run into this issue where ZipArchive tries to write the end-of-zipfile directory synchronously when it's disposed, but Kestrel doesn't allow synchronous writes to its HTTP output stream. So we might have to extract everything, then send the zip file separately. Which means that the HTTP connection might time out, so it's possible we'll have to make two API endpoints, one to prepare the zip file and one to download it. Testing required.


Fails to build on ARM

When attempting to build fakemachine for armel and armhf architectures, it fails to build with following error:
# github.com/go-debos/fakemachine
src/github.com/go-debos/fakemachine/machine.go:285:36: cannot use u.Release[:] (type []uint8) as type []int8 in argument to charsToString

Full build log can be found at
https://buildd.debian.org/status/fetch.php?pkg=golang-github-go-debos-fakemachine&arch=armhf&ver=0.0~git20180126.e307c2f-1&stamp=1519660834&raw=0
At a guess, u.Release[:] needs to be declared as the correct type or convert the value at the call.
I do wonder if byte might work better than uint8 in this case.

On the non-amd64 architectures where it builds successfully, it won't work anyway, due to #18.

fakemachine builds properly for arm64 now; we may add further architectures later if there is a need.


Circle charts broken

I was using the most recent version which included an update to include text instead of icons below the percentage. All was well with that. All of the charts on my site broke with the recent renaming of the files. I updated the css and js cdn paths, but now be charts will not initialize.
Using:
<link href="https://cdn.rawgit.com/pguso/jquery-plugin-circliful/master/css/jquery.circliful.css" rel="stylesheet">

To test, I have an element called "test-chart".

I am initializing on document load with:
   $("#test-chart").circliful({         animationStep: 5,         foregroundBorderWidth: 15,         backgroundBorderWidth: 15,         percent: 20,         foregroundColor: "#4caf50",         text: "CPU"     });
All I see is an empty chart with a gray outer border, none of the settings above applied.

Same thing happens in jsfiddle:
[https://jsfiddle.net/k1sfd7rt/]

This is because the cdn still has the "old" version of circliful cached and the parameters where renamed so the old version doesnt recognize your params.
Visit https://rawgit.com/faq#cdn-ttl for more details.


Troubleshooting Request: Add a Troubleshooting section in the Readme file about Blazor Deployments and S3 Bucket Public Access

Is your feature request related to a problem? Please describe.
Right now, some AWS accounts might be configured by default to block public S3 buckets. Using the deployment tool to deploy a Blazor WASM app will fail with an error: "API: s3:PutBucketPolicy Access Denied"
This is due to the account blocking public S3 buckets and the CDK fails with the above error which is not very descriptive.
Describe the solution you'd like
Add a troubleshooting section in the readme file for Blazor WASM deployments explaining that you need to turn on public S3 buckets in the AWS accounts for the deployments to succeed
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

This is a :rocket: feature request

Not needed anymore since we us CloudFront now.


[AutoPR compute/resource-manager] Swagger spec for disk.json 2018-06

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3854

Codecov Report

Merging #3310 into restapi_auto_compute/resource-manager will decrease coverage by 0.62%.
The diff coverage is 27.03%.

@@                            Coverage Diff                            @@
##           restapi_auto_compute/resource-manager    #3310      +/-   ##
=========================================================================
- Coverage                                  53.81%   53.19%   -0.63%
=========================================================================
  Files                                      10346     5540    -4806
  Lines                                     222241   141520   -80721
=========================================================================
- Hits                                      119596    75276   -44320
+ Misses                                    102645    66244   -36401

Impacted Files
Coverage Δ

...te/azure/mgmt/compute/compute_management_client.py
48.36% <0%> (-0.59%)
:arrow_down:

...18_06_01/models/compute_management_client_enums.py
100% <100%> (ø)
:arrow_up:

...re/mgmt/compute/v2018_06_01/operations/__init__.py
100% <100%> (ø)
:arrow_up:

...ute/v2018_06_01/operations/snapshots_operations.py
13.69% <13.69%> (ø)

...compute/v2018_06_01/operations/disks_operations.py
13.69% <13.69%> (ø)

...pute/azure/mgmt/compute/v2018_06_01/models/disk.py
29.41% <29.41%> (ø)

...ure/mgmt/compute/v2018_06_01/models/disk_update.py
33.33% <33.33%> (ø)

.../azure/mgmt/compute/v2018_06_01/models/snapshot.py
35.71% <35.71%> (ø)

...mgmt/compute/v2018_06_01/models/snapshot_update.py
40% <40%> (ø)

...e/mgmt/compute/v2018_06_01/models/creation_data.py
45.45% <45.45%> (ø)

... and 5253 more

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 22875a3...da74a50. Read the comment docs.

(message created by the CI based on PR content)

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/2982


Added link to editor page on landing page

Should I also change the styling? I haven't because it was mentioned. Let me know if you also want to change me the styling of link on the landing page

Hi @anandromy thanks for your contribution
If you want to do styling go ahead, open to your ideas..
You can create new issues and I can assign to you and you can make the respective changes.


Correct 'STLINK_FW' handling for all boards with STM32

Fixes #10739 and extends it to all boards which use STM32.
Sets 'STLINK_FW'  to 'stlink' by default, and when set to 'jlink' provides correct arguments.
Additionally, the discovery board l475 IoT documentation regarding used µC is corrected in a separate commit.

@carlescufi - as you asked about a patch

@mbolivar the main issue has been raised when the west wasn't around, and people were having the aforementioned issue. Regarding west usage, it would be great to mention it in the stlink-v21 docs.

update: rebased and fixed the conflicts

Please update title and related doc: doc/guides/debugging/probes and host-tools.

So, in new file probes.rst, in 'Flash and debug with ST-Link/Using Segger J-Link', add a paragraph mentioning possible use of west flash.debug --runner jlink

Tested by hand, going to merge since getting this to pass CI is going to be a pain and not show anything useful.


Create IndexedSyncCommittee

Is your feature request related to a problem? Please describe.
Right now we put indexed data of a sync committee inside EpochContext:
  currSyncCommitteeIndexes: ValidatorIndex[];
  nextSyncCommitteeIndexes: ValidatorIndex[];
  currSyncComitteeValidatorIndexMap: SyncComitteeValidatorIndexMap;

Describe alternatives you've considered
My proposal is to put index data inside an IndexedSyncCommittee, similar to IndexedAttestation
export interface IndexedSyncCommittee extends SyncCommittee {
  attestingIndices: List<ValidatorIndex>;
  validatorIndexMap: SyncComitteeValidatorIndexMap;
  // add more if needed
}

type SyncComitteeValidatorIndexMap = Map<ValidatorIndex, number[]>;

Additional context

The IndexedSyncCommittee is cached inside CachedBeaconState (similar to balances, validators) with the same currentSyncCommittee, nextSyncCommittee property key
Ensure the tree backed data structure is the same
When cloning CachedBeaconState we can just share the same IndexedSyncCommittee instead of doing a real clone

What do you think @wemeetagain @dapplion ?

@tuyennhv Could you detail the pros vs cons of this approach compared to having the data in the epoch context itself?

The most benefit of this is single source of truth - we want to manage sync committee data in single place to avoid the issue in https://github.com/ChainSafe/lodestar/issues/2631#issuecomment-857321122
Cleaner EpochContext: we'll not have sync committee code in EpochContext any more
It's also consistent to the design we have with CachedBeaconState for other fields

The down side of this is we need to make change and test everything again, and not sure if we have any obstacles with that?

@tuyennhv Sounds good to me! If performance is the same sounds like a good improvement. When I added the caches I didn't considered other alternatives like yours so it sounds reasonable.


New Logo and better image responsiveness

@Szegoo Please check that everything is working properly, so we can merge it to main and deploy to mainnet.


added support for asymmetric sprite sheets as well as for borders bet…

…ween sprites on the sprite sheet
Added support for asymmetric sprite sheets. Created a sprite struct that keeps the position and size information about each sprite. Since the symmetric is just a subset of asymmetric, all the symmetric stuff produces sprites on it's own then feeds that to the asymmetric.
Also implemented symmetric sprite sheets with borders between the sprites, which helps with issues that sprites bleed into eachother.
Closes #478

Changes Unknown when pulling 217d8ac62bd1aabc3d1ebad24e5d49c19853fdfb on asymmetric-sprite-sheets into ** on master**.

Changes Unknown when pulling c8667d6cb7fa2ba246e73cd9c0ca60aa10aadfe1 on asymmetric-sprite-sheets into ** on master**.


[Do not merge] Optimize test execution

We were unnecessarily trying to get the correlation id for long running operations, combined with the fact that in tests we didn't sleep/yield at all in our polling thread.

If this improves performance, why is it DNM?

@johanste if it does speed up the tests let's rebase it and get it in.

Or, if it doesn't improve the performance, close it :smile:

I'll close it for now.


[BUG] Xamarin.Forms Cannot encode SKImage as GIF

Description
When attempting to encode an SKImage as a GIF, it returns a null SKData value
Setting SKEncodedImageFormat to Png works as expected
Code
SKImage.FromBitmap(bitmap).Encode(SKEncodedImageFormat.Gif, 100)
Expected Behavior
I should get back an SKData object that encapsulates a GIF encoded image
Actual Behavior
I get back a null value
Basic Information

Version with issue: <IP_ADDRESS>
Last known good version: Unknown
IDE:  Visual Studio 2019
Platform Target Frameworks:

iOS:  13.2
Xamarin.Forms: <IP_ADDRESS>1537

Target Devices:

iPhone 6 (iOS 12)

Ah, yeah. Unfortunately SkiaSharp only supports 3 formats (https://github.com/mono/SkiaSharp/issues/320#issuecomment-310805723):

jpeg
png
webp

This is a bit unfortunate, but you could always use a separate library to do this if you really need a gif. For example (I have not actually used) there is NGif or BumpKit or AnimatedGif.
A lot of them require GDI, so they might only work on Windows.
There was this discussion as well: https://groups.google.com/forum/#!searchin/skia-discuss/gif|sort:date/skia-discuss/R99fiu-tYNU/1G7KaxiyEAAJ
Basically, they suggest making a set of png and using ImageMagick to make a gif.

Sadly, all of those libraries rely on the Windows (System.Drawing/GDI) which just throws PlatformNotSupportedException on Xamarin.
However, drawing inspiration from NGif (and the original Java version) and replacing all the System.Drawing components with their equivalent SkiaSharp components (Image->SKImage, Color->SKColor etc.) it just... works. Almost anti-climactic after bashing my head for a couple days on it
Performance leaves something to be desired but it's tolerable

Ooh, yeah. Do you have the code somewhere? Maybe I can take a look and see if you are maybe copying things too many times. I was actually going to have a look at this, but since you got code already, maybe we can make this an actual thing? A SkiaSharp.NGif or a NGif.SkiaSharp. People will love it. And they can now play it in Xamarin.Forms 😉

I don't have public code, though it would be very simple to do. The only part of the System.Drawing to SkiaSharp port that wasn't basic 1 to 1 was in GetImagePixels
https://github.com/avianbc/NGif/blob/master/Components/AnimatedGifEncoder.cs
Where I replaced
Image temp = new Bitmap(width, height);
Graphics g = Graphics.FromImage(temp);
g.DrawImage(image, 0, 0);
image = temp;
g.Dispose();

with
var temp = new SKBitmap(width, height);
var canvas = new SKCanvas(temp);
canvas.DrawImage(image, 0, 0);
image = SKImage.FromBitmap(temp);
canvas.Dispose();

Fairly naive/simplistic, but works as proof of concept

Hmm. Looks good. The new SKBitmap is creating some memory, and then SKImage.FromBitmap is copying it. The canvas.DrawImage is also doing a copy. Is there a reason why you can't just draw the image directly?
Or, Instead of using a SKBitmap and then SKImage, just use a SKSurface or SKBitmap. In the case of GDI, the Image is a base type for Bitmap, so there is no copying going on. And, the Graphics.FromImage is just wrapping the image - just like the cheap new SKCanvas.
Hope that helps/makes sense?

That does make sense, I'll certainly do some experiments along those lines (I haven't spent much more time on it once I was convinced it was in fact possible to do)
Cheers


Title: reversing of Array

Description
Description:
Greetings!
I want to add problem of reversing  array with two methods->
1.with the help of another array  in which firstly we will copy elements of our main array  in helping array from the back side  then again copying elements of helping array in our main array with taking O(n) time-complexity.
2. with the help of two pointers first will iterate from left to right while second will iterate right to left until left<right and swapping the array[first pointer] with array[second pointer]  with  taking O(n) time-complexity.
please assign this issue to me as I am a contributor in GSSOC'22.
Domain
Data Structure
Type of Contribution
Addition
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of LearnCPP.

/assign


main.py run multiprocess fixed

python main.py microphone/path_to_wav all met the error
RuntimeError:
        An attempt has been made to start a new process before the
        current process has finished its bootstrapping phase.

        This probably means that you are not using fork to start your
        child processes and you have forgotten to use the proper idiom
        in the main module:

            if __name__ == '__main__':
                freeze_support()
                ...

        The "freeze_support()" line can be omitted if the program
        is not going to be frozen to produce an executable.

So we add to make it run successfully.

I just released pyannote.metrics 3.1 that (I think) was causing this issue.
Can you please confirm that upgrading to pyannote.metrics 3.1 solves this issue?
pip install pyannote.metrics==3.1

I'm closing this because of inactivity, feel free to reopen if needed


FCN in train.py

Hi,
where is the FCN in the train.py line=63?
from keras_fcn import FCN ?

@jeansely yep.

thanks , i see your read me file.


⚠ Quirk Club Argentina (API endpoint) has degraded performance

In a7d9a1d, Quirk Club Argentina (API endpoint) (https://us-central1-quirkclub-dev.cloudfunctions.net/api/check/api) experienced degraded performance:

HTTP code: 200
Response time: 8141 ms

Resolved: Quirk Club Argentina (API endpoint) performance has improved in 6579e4e.


How to train the generative NeMF on dog mocap dataset?

Hello, I would like to train a Generative NeMF model on dog mocap data, but only the code for training a Single-motion NeMF model is available. Could you provide the code for this, and also let me know if there is a pre-trained model available?

Hi, thanks for your interest in our project. Sorry that we don't have a pre-trained model on dog mocap data as we only use them for validation. You need to mofify our codebase to train this model, and the logic would be:

Modify animal.py to process all the dog mocap data and split them into the training/validation/testing sets (currently we process each sequence one by one and store them separately).
Modify train.py to load the processed Animal Dataset rather than AMASS, and write a corresponding config file to properly set the hyperparaemters.
Modify train_gmp.py to train it for the Animal Dataset rather than AMASS, a new config file would be required as well.
There might be other issues I overlooked, but the basic logic is here. If you run into any issues, welcome to point them out and I can help to resolve them.

Thank you.


feat(http): added request object to http err response

The request object is added to the superagent response object on non error statuscodes. On error the request object is unavailable in the error's response object. This pull request fixes that.

[x] I added new tests for the issue I fixed/built
[x] I ran npm test for the package I'm modifying
[x] I used npm run commit instead of git commit

@Steelfish it would be good if there was an automated test for this, to prevent any accidental regressions in future. I'm happy to help if you run into any issues writing a test.

I added a test based on the response$ onError test. My habit to push format document seems to have added some spaces to various functions as well. Let me know if I should make any more changes.

Generally I (as an author of PR that added request object to response) think it is not very good pattern to include request in the response, because it is already available in the metastream. Yes this simplifies a little bit things, but creates ambiguity (how request object can be accessed), so I would even remove request from response.

I am currently using the request object on the response$ to keep track of outstanding requests. If there is another way to do this, I would love to hear it.

I somewhat agree with @whitecolor, but we already have res.request = reqOptions; for non-error responses, so we could have that same also for the error case. This was actually originally your code: https://github.com/whitecolor/cycle-http-driver/commit/fef21c817622ea545ef2b56b225ec99826f0c9d0 so I'm curious what were the original ideas behind that PR https://github.com/cyclejs/http/pull/41.

@staltz Idea was just easier access to corresponding request while handling response.
Though currently I don't use it at all because http driver is kind of task driver that just gets task request performs some async task action, and reports back with resolved or rejected result. So in general case (for other kind of task based drivers) we may not embed request into response, so that is way I consider it is inappropriate to embed it in http driver response too.

It would be a breaking change to remove response.request, and one that wouldn't help people: those who are not using it won't notice, but those who are using it will need to refactor considerably. I'm favorable to merge @Steelfish's PR.

As just a legacy it is ok.
const addRequest = r$ => r$.map(r => Object.assgin(r, request: r$.request)
HTTP.select().map(addRequest)

Merged! Thanks @Steelfish !


✨ feat: added posts, langs, lebales (#679)

1. Description

1.1. Checklist

[ ] The title of my PR starts with a Conventional Commit prefix (fix:, feat:, docs: etc).
[ ] I have read the Contributor Guide and followed the process outlined for submitting PRs.
[ ] I have updated/added tests for ALL new/updated/fixed functionality.
[ ] I have updated/added relevant documentation in docs and added dartdoc comments with ///.
[ ] I have updated/added relevant examples in examples.

1.2. Breaking Change

[ ] Yes, this is a breaking change.
[ ] No, this is not a breaking change.

1.3. Related Issues

Codecov Report

Merging #682 (4ce95d7) into main (ed8ede6) will increase coverage by 5.28%.
The diff coverage is n/a.

:exclamation: Current head 4ce95d7 differs from pull request most recent head ebacfaf. Consider uploading reports for the commit ebacfaf to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main     #682      +/-   ##
==========================================
+ Coverage   82.27%   87.56%   +5.28%
==========================================
  Files           5        7       +2
  Lines          79      193     +114
==========================================
+ Hits           65      169     +104
- Misses         14       24      +10

see 8 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Need to wait for https://github.com/shamblett/cbor/issues/56


Unable to read connection string from configuration file [GetConnectionInfoFromADUConfigFile]

Hi,
I'm currently trying to get this(https://docs.microsoft.com/en-us/azure/iot-hub-device-update/device-update-raspberry-pi) Tutorial working.
Everything works to the point where the Daemon(adu-agent) has to be started. My Problem is the Daemon dies because he cant read my connection String?
This is the status-report from the systemctl status adu-agent.

/adu/ Folder. Both Files created.

This is my config File. Connection String placed where the Tutorial told me to.

Pls help

SOLVED:
Create /adu/adu-conf.txt
Daemon is able to read this.
Template:
connection_string = `HostName=<yourIoTHubName>;DeviceId=<yourDeviceId>;SharedAccessKey=<yourSharedAccessKey>`
aduc_manufacturer = <value to send through `AzureDeviceUpdateCore:4.ClientMetadata:4.deviceProperties["manufacturer"]`
aduc_model = <value to send through `AzureDeviceUpdateCore:4.ClientMetadata:4.deviceProperties["model"]`
manufacturer = <value to send through `DeviceInformation.manufacturer`
model = <value to send through `DeviceInformation.manufacturer`

Also, the du-config.json looked ok, but the logs show 0.6.0 private preview agent instead of 0.8.0, so it's probably looking for the older adu-conf.txt, not du-config.json.
It is possible in "Download the image" step, older .wic.gz was mistakenly downloaded?
To fix, you could backup /adu/.json, flash SD card with https://github.com/Azure/iot-hub-device-update/releases/download/0.8.0/adu-base-image-raspberrypi3.wic.gz, and restore /adu/.json

Nice ty that worked.
But now I got a different problem. :(
When I try to create the "ADUGroup" tag the adu agent fails loading the extension.json file
Mar 07 07:55:18 raspberrypi3 AducIotAgent[360]: {"deviceUpdate":{"__t":"c","agent":{"deviceProperties":{"manufacturer":"fabrikam","model":"vacuum","interfaceId":"dtmi:azure:iot:deviceUpdate;1","aduVer" :"DU;agent\/0.8.0-public-preview","doVer":"DU;lib\/v0.6.0+20220201.143054,DU;agent\/v0.6.0+20220201.143052"},"compatPropertyNames":"manuf2022-03-07T07:55:18.0599Z [E] Cannot open an extension registrat ion file. ('/var/lib/adu/extensions/component_enumerator/extension.json') [GetExtensionFileEntity] Mar 07 07:55:18 raspberrypi3 AducIotAgent[360]: 2022-03-07T07:55:18.0600Z [E] Failed to load extension from '/var/lib/adu/extensions/component_enumerator/extension.json'. [LoadExtensionLibrary] Mar 07 07:55:18 raspberrypi3 AducIotAgent[360]: 2022-03-07T07:55:18.0601Z [E] Cannot get components information. erc: 0x70000003 [DetectAndHandleComponentsAvailabilityChangedEvent]
When looking into the /var/lib/adu/extensions/component_enumerator/ folder nothing is in there.

Nice Ty that worked.


MatChipSet: set_SelectedChips cannot be used to preselect chips

When using MatChipSet.SelectedChips to preselect displayed chips in OnAfterRender, the comparison is done on a Hash of the MatChip, so a match will never be found.
As a result, chips cannot be preselected. Suggested fix below...
Reproduction Steps

Use the code in the sample/demo application on the 'Chip' page.

In the 'Filter ChipSet' section, replace the line
MatChip[] selectedChips = null;
with
MatChip[] selectedChips = new MatChip[] { new MatChip() { Label = "Wedges" } };

In the Razor file, add a ref value to the MatChipSet:-
 <MatChipSet @ref="filterChipset" Filter="true" @bind-SelectedChips="selectedChips">

Override the OnAfterRender method to set the selected chips:-
    protected override void OnAfterRender(bool firstRender)
     {
         if (firstRender)
             filterChipset.SelectedChips = selectedChips;
     }

Expected Result
The 'Wedges' chip should be selected when the page is shown.
Actual Result
All chips are deselected.
Suggested Fix
In MatChipSet.cs, at line 75, match on the Label value, not a hash of the MatChip object:-
Replace
var selected = new HashSet<MatChip>(value);
foreach (var chip in _chips)
    chip.IsSelected = selected.Contains(chip);

With
var selected = new HashSet<string>(value.Select(x => x.Label));
foreach (var chip in _chips)
    chip.IsSelected = selected.Contains(chip.Label);

I'd have submitted a pull request, but I'm not a registered contributor...

It is possible to solve this with using @ref on the MatChip and storing it in an array or something like it. Example:
Markup
<MatChipSet Filter="true" @bind-SelectedChips="selectedChips">
  <MatChip Label="Wedges" IsCheckable="true" @ref="allChips[0]"/>
  <MatChip Label="Circles" IsCheckable="true" @ref="allChips[1]"/>
  <MatChip Label="Squares" IsCheckable="true" @ref="allChips[2]"/>
</MatChipSet>

Code
private MatChip[] allChips = new MatChip[3];
private MatChip[] selectedChips;

protected override void OnAfterRender(bool firstRender)
{
    if (firstRender)
        selectedChips = allChips.Where(x => x.Label == "Wedges").ToArray();
}

It would still be nice with an easier way to do this, but maybe a parameter to access _chips from MatChipSet is enough.