Notify HA of updates on fetch_data()

Need to tell listeners when new data is fetched - found when restarting the socket and calling fetch_data() HA was not notified of updated entities until an event message was received.

I am not sure I understand the need for this change. fetch_data() is only called on startup when HA calls connect(), so there is no need to call the attr changed callback there.

It is also called when the socket reconnects:
https://github.com/abmantis/whirlpool-sixth-sense/blob/f217ca550a275e1a04fd567a8efd092ad2968bce/whirlpool/appliance.py#L192C8-L192C8
which is mapped to this:
https://github.com/abmantis/whirlpool-sixth-sense/blob/f217ca550a275e1a04fd567a8efd092ad2968bce/whirlpool/eventsocket.py#L138
When the event socket reconnects, we don't "know" the states for sure, so we do a fetch to make sure everything is up to date.  Unfortunately, we don't tell HA that there is new data to update when this happens.
This will also allow us to drive 1 timed update function and still trigger all other entities (The bandaid for a crappy event socket)

It is also called when the socket reconnects: https://github.com/abmantis/whirlpool-sixth-sense/blob/f217ca550a275e1a04fd567a8efd092ad2968bce/whirlpool/appliance.py#L192C8-L192C8 which is mapped to this:
https://github.com/abmantis/whirlpool-sixth-sense/blob/f217ca550a275e1a04fd567a8efd092ad2968bce/whirlpool/eventsocket.py#L138
When the event socket reconnects, we don't "know" the states for sure, so we do a fetch to make sure everything is up to date. Unfortunately, we don't tell HA that there is new data to update when this happens.
This will also allow us to drive 1 timed update function and still trigger all other entities (The bandaid for a crappy event socket)

Ah I see! Makes sense!

Thanks!

Can we get a release in pypi for this? Looking to activate the polling feature in HA and this is necessary for HA to update all entities.

@mkmer sure! I just released it as v0.18.4.


Correct 'last doc' logic.

Purpose
Apparently you can't use the key property provided to components. Bummer. Also, off by one!!

Coverage remained the same at 46.417% when pulling 3be621fc28ffe77ee2f7535cbaad82b4940ce4b4 on rjstires:doc-fix into ac88eaa2407c3a77fcb48451257f2937980b612c on linode:voldemort.


[Rendering] iOS App crashing when rendering Adaptive Card with "lang" : "en" field

Target Platforms
iOS
SDK Version
2.8.10
Application Name
Microsoft Teams
Problem Description
The app is crashing due a parsing issue for "lang" field value "en". It appears at the moment of initialize an app from a JSON string (see sample code). Looks like the class causing the issue is ACOAdaptiveCard, on the method fromJson. Last log on console is:
libc++abi: terminating with uncaught exception of type std::runtime_error: collate_byname::collate_byname failed to construct for en
terminating with uncaught exception of type std::runtime_error: collate_byname::collate_byname failed to construct for en
Sending the card on desktop from developer portal is noticed that the "lang" field issue is marked (check image) but is only crashing on iOS, other platforms like Android and desktop show the card without a problem.
Screenshots

Card JSON
{"body":[{"type":"TextBlock","text":"Hello World!","wrap":true},{"height":"auto","type":"Image","url":"https:\/\/us-prod.asyncgw.teams.microsoft.com\/urlp\/v1\/url\/content?url=https%3a%2f%2fcataas.com%2fcat","spacing":"none"}],"lang":"en","type":"AdaptiveCard","version":"1.2"}

Sample Code Language
Objective-C
Sample Code
ACOAdaptiveCardParseResult *cardParseResult = [ACOAdaptiveCard fromJson:cardJsonString];

this is fixed.


Implement AsRawFd for Stdin, Stdout, Stderr, and RawFd

Is there a reason this is not the case already, or is it just that it is unnecessary for the typical use case and hasn't been added? Would something like this require an RFC, or would a PR suffice?

This seems fine for Stdin/Stdout/Stderr, but RawFd is just a typedef for c_int, and it seems a bit weird to implement that trait for a number.

Yeah, I'd also like this for Stdin/Stdout/Stderr. I don't think you'd need an RFC for it.

My use case is for a function with the signature <T: AsRawFd>(stream: &T), which is why I suggest implementing it for RawFd as well. But the Std* types are the relevant ones really.

On RawFd, see https://github.com/rust-lang/rust/pull/41035

The implementation for Std* is now merged; but not for RawFd, due to the concern mentioned above.


provide basic tutorials

There are should be three tutorials:

On using core with standard sklearn transformers

On using custom transformers for brain-ml on graphs

Combination of 1 and 2

Done.


Why shape Render method is sealed

public sealed override void Render(DrawingContext context);

please remove the sealed

That's by design. If you want to override OnRender, please inherit Control.
See: https://github.com/AvaloniaUI/Avalonia.Samples/tree/main/src%2FAvalonia.Samples%2FCustomControls%2FSnowflakesControlSample

[!TIP]
For questions please use Telegram or GitHub discussions. If you open bug reports or feature requests please provide as much details as possible, for example why exactly you need this or that API.


Version of db2tcl used by HammerDB including added Win build

This is the current version of db2tcl that gets downloaded from www.hammerdb.com when you do a HammerDB build from source (https://www.hammerdb.com/blog/uncategorized/how-to-build-hammerdb-from-source/). This builds without intervention on both Linux and Windows.
Includes recent additions of Db2 apis to create and drop databases and do force applications.

closes issue #6


chore: update docs for repo standards

few little bits in here. lmk what you think

Closing this cuz I guess it's old


Fix undefined variable: debug

Fix for

PHP Notice: Undefined variable: debug in Minify_MinifiedFileRequestHandler.php on line 552

thanks! i have replicate this fix in main repo https://github.com/szepeviktor/fix-w3tc .

@cesco69 , do you mind trying out this patch and see if it works? My aim is to link $debug to the Config settings (minify.debug)
https://github.com/charlesLF/fix-w3tc/commit/bbca666b3705834172b5943bf0525195f1cbb435


question about cos-similarity

hi, I've been using hnsw for a while, it's a nice project, good work! however, recently I have a question about it. I've been using cos, l1, l2 for distance calculation, that reminds me of sklearn cosine similarity. in sklearn, you can control the distance(I forgot the parameter's name), that means you can filter your result when you are using cos-similarity to find items with shorter distance in between. However, in hnsw, it seems we can only set 'cos' to use cosine similarity and get the default results. Just wanna know am I right here? is there any way we can control the cos distance in hnsw? thank you for helping.

Hi @652994331, sorry for a late reply, I've missed this issue for some reason.
I am not sure I understood it. The cosine similarity that I know does not have parameters. Can you provide a link to sklearn?


Add credential offer - OID4VCI draft 13

credential-offer - https://openid.net/specs/openid-4-verifiable-credential-issuance-1_0.html#name-credential-offer

Closed. Added to the public repo.


Release the code?

Hello, I am very interested in your work. Could you tell me when the code will be released?

Hello, thank you for your interest in our work. We will release our ADCA code when it is published in the journal.
------------------ 原始邮件 ------------------
发件人:                                                                                "yangbincv/ADCA"                                                                                @.>;
发送时间: 2022年10月23日(星期天) 下午2:16
@.>;
@.***>;
主题: [yangbincv/ADCA] Release the code? (Issue #1)
Hello, I am very interested in your work. Could you tell me when the code will be released?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Hello, I am very interested in your work. Could you tell me when the code will be released?

Thank you for your interest in our work, we will release the code within two months.
------------------ 原始邮件 ------------------
发件人:                                                                                "yangbincv/ADCA"                                                                                @.>;
发送时间: 2023年2月5日(星期天) 上午10:19
@.>;
<EMAIL_ADDRESS>主题: Re: [yangbincv/ADCA] Release the code? (Issue #1)
Hello, I am very interested in your work. Could you tell me when the code will be released?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>


Why Writeindent is protected? Why WriteComment not have a "newline with indent" override?

I'm try to write some custom comment by custom JsonConverter for some members.
But i cannot find a direct way to write indent width of current level.
And WriteComment not have a "start with newline and indent" override.
Expected output
{
  "MySubObj": {
    /*The SubProperty1 is for balabalabala...*/
    "SubProperty1": [],
    /*The Dictionary is for balabalabala...*/
    "Dictionary": {
      /*The key1 is for balabalabala...*/
      "key1": "value1",
      /*The key2 is for balabalabala...*/
      "key2": "value2",
      /*The key3 is for balabalabala...*/
      "key3": "value3"
    }
  }
}

Expected behavior
Make WriteIndent public or make WriteComment have a override can output in newline with indent.
Actual behavior
WriteIndent is protected, and WriteComment only append text only, cannot write a indented comment of current indent level directly.

My current implementation:
public class JsonWriterEx
{
    static void WriteIndent(this JsonWriter jw)
    {
        typeof(JsonWriter).GetMethod("WriteIndent", System.Reflection.BindingFlags.NonPublic | System.Reflection.BindingFlags.Instance).Invoke(jw, null);
    }
}


WIP added onCheckStatusChange

I decided  create separate branch regarding to this comment. Just to not swamp original PR, there are enough other comments)

Sorry, I tried, but couldn't got a full picture how it should work. Hence, without understanding I can't made that changes.  Instead, I've created a separate PR with another possible implementation.
Probably, my brain get used to work in another way. If you implemented your version there, I would glad to see it.

See https://github.com/Mirantis/lens-extension-cc/pull/673


Open Folder: Persist opened folder on launch

If the user has opened a folder, and then launches Oni again from an install directory, we should persist that and default to opening from their previous folder.
This fits in with persisting other editor settings - like window size (#452), open files, etc.

This should be a setting...
I don't like this in vscode/sublime I hated whenever I closed it it would reopen what I was doing last.

Just making the "Open Folder" menu item start from the current open folder would be good.


mCSD Changes plus more

This was the changes I did to add in mCSD and mACM support which are built in STU3.  Also added in FHIR.js for XML conversions.

@rcrichton @lduncan-intrah Wanted to see if we are stuck on something here. I don't see commits to the master branch since mid-Feb.


refactor: store protobuf messages in corresponding tables

This PR stores protobom message data in the associated ent schema tables. It also generates deterministic UUIDs from a hash of the wire format message to use as primary keys.
All changes in this PR are auto-generated except the files in backends/ent/ and internal/backends/ent/schema/.

Oh, thank I was just thinking ahead, not necessarily for now, but it's cool you pushed them :)

@ashearin @pkwiatkowski1 @lmphil
Can you guys take a look at the changes in internal/backends/ent/schema/?
Looking to see if the new fields make sense (i.e. storing the proto messages as an additional field alongside their corresponding ent schemas), or if there might be a better way to do it.

Can you guys take a look at the changes in internal/backends/ent/schema/?
Looking to see if the new fields make sense (i.e. storing the proto messages as an additional field alongside their corresponding ent schemas), or if there might be a better way to do it.

@jhoward-lm  From what I understand (and looked up just now) about ent schemas and mixins, what you're doing makes sense.


installation: integrate Webpack support

Closes #79.

Waiting for Invenio release so Pipfile.lock can be updated.


Visualizations: Add units to axes when available

Units are currently a static property of a variable's metadata. If available, the unit should be added in parenthesis for plot axes. A simple rule for this is that we add this to the label properties of plots.

@moontrip the following studies should have units available for some variables:

India ICEMR Fever Surveillance
GEMS1 Case Control

Let me know if you are not seeing units.

@dmfalke Thanks alot 👍 I wondered what variable would have units as I usually used GEMS1 which seems not to have units haha.
By the way, may I ask one more thing to clarify? I checked some variables (for checking metadata) and study.ts and realized that parameters that have unit are only two: binUnits for DateVariable and Units for NumberVariable. I wonder if we also need to use binUnits (for DateVariable) to be displayed in the plot viz? Or simply check Units only?

I would just focus on NumberVariable for the time being. I need to look more into DateVariable.

I just checked qa.clinepidb.org and "weight" does not have units for the GEMS study.

@danicahelb Thank you for your information. It is working fine at my local VM for visualizations so my guess is that this work is not updated at qa.feature site yet: @dmfalke usually update the site so he can confirm this.
Anyway, please note that this unit work did not apply to histogram filter at Subset (like your initial screenshot but you might have removed it later) as I thought it was only for visualizations. If you think it should also be done in there, then it can probably be done easily, I suppose. Please just let me know about your thought.

I just checked qa.clinepidb.org and "weight" does not have units for the GEMS study.

I will be updating the site today, and changes will show up tomorrow. Sorry for the delay.

units are now showing up on the axes of visualizations. Still need to add units to other places on the site, but that is covered in  https://github.com/VEuPathDB/EdaNewIssues/issues/26


The ''Normal' execution method locks tables for MySQL dumps

System information:

Operating system (distribution) and version RHEL 8.5
DBeaver version 22.0.0 UE
Additional extensions

Connection specification:

Database name and version MariaDB 10.3.23
Driver name
Do you use tunnels or proxies (SSH, SOCKS, etc)? SSH

Steps to reproduce, if exist:

Create a dump task
Choose the 'Normal (no locks)' Execution Method
Execute the task. Observe that all tables are locked.

Fixed.

verified


Width for individual columns

How do i set width for individual columns?
and also how do i set alternating row color?

Setting th width in CSS should be enough.
Do you mean striped table ? Same, i did it with CSS it works
.datatable tbody tr:nth-child(even) {
  background-color: #f2f2f2;
}


Docs: Notes on mutable assets & determining asset owners

Based on some questions from @MiracleB on Gitter, I added some more documentation about assets:

How to determine the current owner(s) of an asset.
Mutable and immutable assets, and how to change a mutable asset.

[Current coverage][cc-pull] is 89.82%

Merging [#430][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master       #430   diff @@
==========================================
  Files            18         18
  Lines          1258       1258
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1130       1130
  Misses          128        128
  Partials          0          0

[][cc-pull]

Powered by Codecov. Last updated by [fc8faca...9428230][cc-compare]
[cc-base-branch]: https://codecov.io/gh/bigchaindb/bigchaindb/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/bigchaindb/bigchaindb/compare/fc8facaa6153bf4ef099e09e38e6b0bbe4128474...94282301013189ba955a1d3c28a4781b06908531?src=pr
[cc-pull]: https://codecov.io/gh/bigchaindb/bigchaindb/pull/430?src=pr

:+1:


Regression: Fix iOS video not displaying after switching source

Bug
I've got the same wrong behavior as described in: https://github.com/react-native-video/react-native-video/pull/1395
Also there are no errors in the console. I just hear the new source audio playing while I see the image of the old source.
If I set controls={false} it works just fine as described in one of the comments of the mentioned PR (the maxBitRate suggestion from there did not work).
The thing is that I need the controls to on because of https://github.com/react-native-video/react-native-video/issues/2808. The workaround I got for this is to set controls to false in the onLoadStart callback and set it back to true in the onLoad callback.
It is also important to mention this is an issue only in iOS, not Android.
Platform

iOS

Environment info

React native info output:
info Fetching system and libraries information...
System:
    OS: macOS 12.3.1
    CPU: (4) x64 Intel(R) Core(TM) i5-7360U CPU @ 2.30GHz
    Memory: 29.54 MB / 8.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 16.15.1 - ~/.nvm/versions/node/v16.15.1/bin/node
    Yarn: 1.22.19 - ~/.nvm/versions/node/v16.15.1/bin/yarn
    npm: 8.11.0 - ~/.nvm/versions/node/v16.15.1/bin/npm
    Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
  Managers:
    CocoaPods: 1.11.2 - /usr/local/bin/pod
  SDKs:
    iOS SDK:
      Platforms: DriverKit 21.0.1, iOS 15.0, macOS 12.0, tvOS 15.0, watchOS 8.0
    Android SDK:
      API Levels: 31
      Build Tools: 31.0.0
      Android NDK: Not Found
  IDEs:
    Android Studio: Not Found
    Xcode: 13.1/13A1030d - /usr/bin/xcodebuild
  Languages:
    Java: Not Found
  npmPackages:
    @react-native-community/cli: Not Found
    react: 17.0.2 => 17.0.2
    react-native: 0.68.2 => 0.68.2
    react-native-macos: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

Library version: 6.0.0.alpha1 (also tested in the top of master commit https://github.com/react-native-video/react-native-video/commit/f32da092b17b912f013da611358302addcea310d)
Steps To Reproduce

Have a video playing with a source X
Change the source to Y

Expected behavior

Have both video and audio from source X
When changing to source Y also have its respective video and audio

Reproducible sample code
import React, {useState} from 'react';
import Video from 'react-native-video';

export default ({}) => {
  const [source, setSource] = useState('http://commondatastorage.googleapis.com/gtv-videos-bucket/sample/BigBuckBunny.mp4');

  setTimeout(() => { setSource('http://commondatastorage.googleapis.com/gtv-videos-bucket/sample/ElephantsDream.mp4') }, 10000);

  return (<Video controls={true} source={source} />);
}

Faced the same issue. Any updates?

Can this be the solution?
https://developer.apple.com/forums/thread/106875

@rafamanzo and @zavvdev
https://github.com/react-native-video/react-native-video/blob/8b09cb58dfd55d2a6351965be0de33ced994a136/ios/Video/RCTVideo.swift#L283
I changed this line to
self._player = self._player ?? AVPlayer()

It works. Can you confirm?

I am having same problem with 0.71.1 and react-native-video: 5.2.1. Does someone has a solution. Is it a use_framework problem ?

This issue shall be fixed in 6.0.0-RC.2.
I will close the issue, please let me know if reproduced


Use std::lock_guard instead of manual lock/unlock calls

I didn't test the Unix port, but I think it should compile/work.

Thank you again, I'll look through all of your changes Tomorrow hopefully and merge it then.

You're welcome. There are other changes, but I need these first so I can continue sending you new PRs. Tell me if you find something wrong. :+1:


MmCopyVirtualMemory - KPROCESSOR_MODE ?

Stupid question here... but I cannot find any documentation on MmCopyVirtualMemory. Inside BlackBone the MmCopyVirtualMemory  is set to use KernelMode.
From my testing, writing to a user process with MmCopyVirtualMemory and KernelMode/UserMode result in the same thing.
What is the difference if I were to use KernelMode or UserMode as "previous" mode ?
Appreciate any explanation, thanks.

In UserMode function does additional validation of UM addresses.

Interesting, thanks. So that would be all ?
The main reason I'm asking is because I'm writing in another process memory, and I don't want to have any "leaks" that can be detected.

That's all


Update the pyca-cryptography version

Commit 9bfeeef made some function parameters const. This actually broke
the pyca-cryptography tests. The discussion in #3360 considers this to
actually be a problem with pyca-cryptography not an OpenSSL issue (they
replicate some of our header file contents which then causes function
prototype mismatches). This commit updates the pyca-cryptography version
to pull in their fix for this issue and make our external tests pass again.

Checklist

[x] tests are added or updated

Pushed. Thanks.


WIP: nrop: cleanup dangling objects removal

WIP: TBD

unit test failures are expected atm, no time to look into them and fix the problem

the failure here is reproducible, will debug later


Validate lockfile (rather than re-resolve) in uv lock

Summary
Historically, in order to "resolve from a lockfile", we've taken the lockfile, used it to pre-populate the in-memory metadata index, then run a resolution. If the resolution didn't match our existing resolution, we re-resolved from scratch.
This was an appealing approach because (in theory) it didn't require any dedicated logic beyond pre-populating the index. However, it's proven to be really hard to get right, because it's a stricter requirement than we need. We just need the current lockfile to satisfy the requirements provided by the user. We don't actually need a second resolution to produce the exact same result. And it's not uncommon that this second resolution differs, because we seed it with preferences, which fundamentally changes its course. We've worked hard to minimize those "instabilities", but they're still present.
The approach here is intended to be much simpler. Instead of resolving from the lockfile, we just check if the current resolution satisfies the state of the workspace. Specifically, we check if the lockfile (1) contains all the relevant members, and (2) matches the metadata for all dependencies, recursively. (We skip registry dependencies, assuming that they're immutable.)
This may actually be too conservative, since we can have resolutions that satisfy the requirements, even if the requirements have changed slightly. But we want to bias towards correctness for now.
My hope is that this scheme will be more performant, simpler, and more robust.
Closes https://github.com/astral-sh/uv/issues/6063.

I think the code changes here might be roughly a wash? But I added a lot of tests.

https://github.com/astral-sh/uv/issues/6063 no longer occurs.

It's possible that we could still support that... But the framing would have to be a little different. Like, our goal would be to use metadata from the lockfile in the normal resolve, not the noop resolve.


use kubeedge-pi-counter-app.yaml problem?

environment：
kubeedge-v1.13.0
add imagePullSecrets
cat kubeedge-pi-counter-app.yaml
apiVersion: apps/v1
kind: Deployment
metadata:
  labels:
    k8s-app: kubeedge-pi-counter
  name: kubeedge-pi-counter
  namespace: default
spec:
  selector:
    matchLabels:
      k8s-app: kubeedge-pi-counter
  template:
    metadata:
      labels:
        k8s-app: kubeedge-pi-counter
    spec:
      nodeName: edgenode-u39j
      nodeSelector:
        node-role.kubernetes.io/master: ""
      hostNetwork: true
      tolerations:
      - key: node-role.kubernetes.io/edge
        operator: Exists
        effect: NoSchedule
      containers:
      - name: kubeedge-pi-counter
        image: <IP_ADDRESS>:30006/dev-test/kubeedge/kubeedge-counter-app:v1.0.0
        imagePullPolicy: IfNotPresent
      imagePullSecrets:
      - name: regcred
      nodeSelector:
        node-role.kubernetes.io/edge: ""
      restartPolicy: Always
```
`

kubectl logs kubeedge-pi-counter-6757859b6c-7wpw4
`W0920 11:11:11.899378       1 client_config.go:608] Neither --kubeconfig nor --master was specified.  Using the inClusterConfig.  This might not work.
W0920 11:11:11.899610       1 client_config.go:613] error creating inClusterConfig, falling back to default config: unable to load in-cluster configuration, KUBERNETES_SERVICE_HOST and KUBERNETES_SERVICE_PORT must be defined
2023/09/20 11:11:11 Failed to create KubeConfig, error : invalid configuration: no configuration has been provided, try setting KUBERNETES_MASTER environment variable
`

looking for help

Hi, you used a wrong image.

image: <IP_ADDRESS>:30006/dev-test/kubeedge/kubeedge-counter-app:v1.0.0
It shoule be ended with kubeedge-pi-counter.
Please check your deployment yaml :)

@Lazy-Guys  built it in the harbor. That has nothing to do with it。error: invalid configuration: no configuration has been provided, try setting KUBERNETES_MASTER environment variable


Lavalink 3 is incompatbile with Java version <10

When I attempt to run Lavalink using Java 9, I get the following:
P:\Mateusz\Developer\Discord\Lavalink>java -jar Lavalink.jar
Exception in thread "main" java.lang.UnsupportedClassVersionError: lavalink/server/Launcher has been compiled by a more recent version of the Java Runtime (class file version 54.0), this version of the Java Runtime only recognizes class file versions up to 53.0
        at java.base/java.lang.ClassLoader.defineClass1(Native Method)
        at java.base/java.lang.ClassLoader.defineClass(Unknown Source)
        at java.base/java.security.SecureClassLoader.defineClass(Unknown Source)
        at java.base/java.net.URLClassLoader.defineClass(Unknown Source)
        at java.base/java.net.URLClassLoader.access$100(Unknown Source)
        at java.base/java.net.URLClassLoader$1.run(Unknown Source)
        at java.base/java.net.URLClassLoader$1.run(Unknown Source)
        at java.base/java.security.AccessController.doPrivileged(Native Method)
        at java.base/java.net.URLClassLoader.findClass(Unknown Source)
        at java.base/java.lang.ClassLoader.loadClass(Unknown Source)
        at org.springframework.boot.loader.LaunchedURLClassLoader.loadClass(LaunchedURLClassLoader.java:93)
        at java.base/java.lang.ClassLoader.loadClass(Unknown Source)
        at org.springframework.boot.loader.MainMethodRunner.run(MainMethodRunner.java:46)
        at org.springframework.boot.loader.Launcher.launch(Launcher.java:87)
        at org.springframework.boot.loader.Launcher.launch(Launcher.java:50)
        at org.springframework.boot.loader.JarLauncher.main(JarLauncher.java:51)

Here's my Java version:
P:\Mateusz\Developer\Discord\Lavalink>java -version
java version "9.0.4"
Java(TM) SE Runtime Environment (build 9.0.4+11)
Java HotSpot(TM) 64-Bit Server VM (build 9.0.4+11, mixed mode)

Presumably this means that my Java version (9) is older than required (presumably 10).
Considering that large number of servers running Lavalink will not have anything above Java 8 (since a lot of stuff is based on Debian stable or oldstable, and these go up to Java 8, maybe 9 only), this effectively renders Lavalink 3 useless on most server machines.

Nevermind, seems Oracle revised their support plans for all Java versions now.
Way to go...


How to enable compression support?

  xt::xzarr_file_system_store store(this->m_FileName); // version 2 file
  auto h = xt::get_zarr_hierarchy(store);
  //std::cout << "h.get_nodes " << h.get_nodes().dump(); // attempts to read non-existent file
  //std::cout << "h.get_children " << h.get_children("/").dump(); // attempts to read non-existent file
  std::cout << "h.get_array " << h.get_array("spectra"); // Unkown compressor type: blosc

Stepping through the code using debugger, I noticed that instance().m_builders only contains one entry: binary. As I had to specify the paths to zlib and blosc, I went back to CMake configurations, and noticed that xtensor-io has a few have settings so I enabled HAVE_Blosc, HAVE_GDAL and HAVE_ZLIB. After an incremental build of xtensor-io and xtensor-zarr, now  instance().m_builders contains no entries. What am I doing wrong?

You need to register the compressor first, like so:
https://github.com/xtensor-stack/xtensor-zarr/blob/90f0acfbf74a4e3664b0861920409b6143600d67/test/test_gcs.cpp#L22.

When I add that, I run into a link error: itkIOOMEZarrNGFF-5.3.lib(itkOMEZarrNGFFImageIO.obj) : error LNK2019: unresolved external symbol "void __cdecl xt::xzarr_register_compressor<class xt::xzarr_file_system_store,struct xt::xio_blosc_config>(void)" (??$xzarr_register_compressor@Vxzarr_file_system_store@xt@@Uxio_blosc_config@2@@xt@@YAXXZ) referenced in function "protected: __cdecl itk::OMEZarrNGFFImageIO::OMEZarrNGFFImageIO(void)" (??0OMEZarrNGFFImageIO@itk@@IEAA@XZ). This should not happen, as that procedure is templated. Do you have any suggestion? If it helps, my current code is here.

I don't know, maybe @JohanMabille has some ideas?

We have some precompiled instantiations of xzarr_register_compressor, maybe they prevent the ocmpiler from instantiating a new one?
Can you try adding this in your code before calling the register function:
namespace xt
{
    template void xzarr_register_compressor<xzarr_file_system_store, xio_blosc_config>();
}

Yes, that did it. Perhaps it should be added to documentation? See #57.

@dzenanz yes definitely. Would you like to open a PR?

I just added a new commit to #54.


Allow deactivation of automatic color styling

Closes #36

I don't think that it is the right place to do it at all.
These methods start returning values of various types, sometimes undefined, etc.
I would rather put a simple check in here:
https://github.com/euvl/vue-js-toggle-button/blob/master/src/Button.vue#L103
Something like:
backgroundColor: this.cssColors ? null :  this.colorCurrent,

Pretty much thats the only change that should be done to old code.

You're right. The previous commit was a left over of the approach in the first attempt + programming before coffee this morning. This is much cleaner now. Thanks!

Nice, great job 👍

Very cool, thanks! And take your time with releasing this.

@euvl before you deploy a new version, you need to run npm run build and commit the changes because my two PRs were both in relation to the master branch, and thus they exclude each other in their state of the dist folder. A better idea may be to define npm run build automatically as the prepare script and exclude the comited versions in dist through .gitignore, see https://docs.npmjs.com/misc/scripts

Oh, dont worry, i will rebuild it anyways. I had a case on another project i was working on, where someone tried to commit a change with bitcoin miner in dist file 😄
Some people use plugin directly from Github repo, for example, in china npm sometimes is not an option (as far as i was told), i don't remember other cases, but having dist in main repo is a good thing for plugin 😄

Btw I got my laptop yesterday night so will be able to publish this one tonight.

@euvl friendly reminder to not forget to publish this. And I hope the new 💻 is good!

@euvl any chance to publish a new version? Sorry for keep bugging you...

Wow, i completely forgot about it, will do tonight. THanks 👍  😄

👋

@euvl another reminder that this still hasn't been taken care of. Could you please release a new version? Thanks.


Add Support for Humidifier DR-HHM001S

Is your feature request related to a problem? Please describe:
Not a problem, just getting the error:
[01/12/2024, 17:16:28] [Dreo Platform] Adding new accessory: Humidifier
[01/12/2024, 17:16:28] [Dreo Platform] Humidifier not yet supported
Describe the solution you'd like:
I'd love to see the addition of the humidifier to the list of supported accessories.
Describe alternatives you've considered:
No other plugins for DREO seem to exist in the same capacity that this one does.

Duplicate of #28


Does negative_sampling properly work after train_test_split_edges?

I would like to check that negative_sampling method always works properly after splitting edges by train_test_split_edges.
Let's say, we have positive edges: row=[0, 1, 2, 3, 4, 5] and col=[1, 2, 3, 4, 5, 6].
train_test_split_edges possibly splits the positive edges into 3 pieces:

train_pos_edge_index: [[2, 3, 4, 5], [3, 4, 5, 6]]
val_pos_edge_index: [[0], [1]]
test_pos_edge_index: [[1], [2]]

Next, we call negative_sampling(train_pos_edge_index) and the following lines samples negative edges.
# line 76 in negative_sampling.py
perm = sample(size, int(alpha * num_neg_samples))
mask = torch.from_numpy(np.isin(perm, idx.to('cpu'))).to(torch.bool)
perm = perm[~mask][:num_neg_samples].to(edge_index.device)

It samples arbitrary edges from all edges not included in train_pos_edge_index. I think sometimes it can sample the positive edges belong to val_pos_edge_index or test_pos_edge_index as negative samples, and it will cause incorrect estimation of a model while training.
Am I missing something at this point?

Yes, this is intended. Given the standard machine learning scenario of hold-out validation and test sets, it might well happen that positive validation edges and test edges appear as negative samples during training. We are not allowed to acknowledge their existence during training to prevent any data leakage. Although this seems to be counter-intuitive at first glance, it is to be expected that such a noise in the learning signal is averaged out during optimization.

@rusty1s Thanks for your answer. I will close this issue.


gormlite: In-Memory DB does not work properly

I switched from github.com/glebarez/sqlite to this sqlite implementation, because it supports the Backup API.
Unfortunately now the In-Memory DB does not work anymore. No other code was changed, all the tests worked before.
I'm using it like this:
db, err := gorm.Open(gormlite.Open("file::memory:?cache=shared"), &gorm.Config{})

The initial migrations work without an error, but afterwards it is not possible to use this database as no tables seem to exist anymore. If I use a database file, everything works fine.
I'll try to create a succinct test case tomorrow and add it to this issue.

See: https://github.com/ncruces/go-sqlite3/discussions/97


ENH: Add OpenXRRemoting external project

Download v2.9.1 of Microsoft.Holographic.Remoting.OpenXR
References:

https://www.nuget.org/packages/Microsoft.Holographic.Remoting.OpenXr/2.9.1

Running the emulator requires special permission:

Notes on installing Visual Studio component to support building the "player" application referenced in the VTK - OpenXR Holographic Remoting documentation^1.
See https://discourse.slicer.org/t/windows-preview-build-delayed-visual-studio-2022-updates/29045

Step by step

Starts the emulator

Open the solution associate with the player application. After it has been built (see below), it reports the following in the Visual Studio output tab.
ConfigurationParser/Constructor/EnumerateEmulators: START
    NumberOfDevicesFound = 1
    ConfigurationParser/Constructor/EnumerateEmulators/Emulator
	    Id = 32b69331-0d59-2dec-951b-9b4e72ae9a2a
	    Name = HoloLens 2 Emulator 10.0.20348.1535
	    Type = Emulator
	    Version = 10.0.20348.1535
	    UwpVersion = 10.0.20348.1535
ConfigurationParser/Constructor/EnumerateEmulators: END (Success, 0:00:00.146)
ConfigurationParser/Constructor: END (Success, 0:00:00.218)
IsEmulatorSupported: START
    Is64BitOperatingSystem = True
DetectSlat: START
    isHyperVRunning = True
DetectSlat: END (Success, 0:00:00.105)
    SlatState = SlatAndHWVEnabled
    IsEmulatorSupported = True
IsEmulatorSupported: END (Success, 0:00:00.109)

Building the player app
References

https://github.com/Kitware/VTK/blob/master/Rendering/OpenXRRemoting/README.md
https://github.com/microsoft/MixedReality-HolographicRemoting-Samples
https://learn.microsoft.com/en-us/windows/mixed-reality/develop/native/holographic-remoting-version-history

Prequisites
Update Visual Studio 2022 installation (Tools -> Get Tools and Features):

MSVC v143 - VS 2022 C++ ARM64 build tools (Latest)
MSVC v143 - VS 2022 C++ ARM64 Spectre-mitigated libs (Latest)
C++ Universal Windows Platform support for v143 build tools (ARM64)

To address the Error messsage: The message "Sorry, something went wrong", see these instructions.
Building
On Windows, build the "player application"
git clone https://github.com/microsoft/MixedReality-HolographicRemoting-Samples

Open player/sample/SamplePlayer.sln with right-click and select Open With -> Visual Studio 2022
On first use, right-click the solution and select Restore NuGet Packages
Select Release

Anticipating the publication of the package PyPI, a trusted published has been registered
Copied from https://pypi.org/manage/account/publishing/:

Trusted Publisher Management
OpenID Connect (OIDC) provides a flexible, credential-free mechanism for delegating publishing authority for a PyPI package to a trusted third party service, like GitHub Actions.
PyPI users and projects can use trusted publishers to automate their release processes, without needing to use API tokens or passwords.
You can read more about trusted publishers and how to use them here.

Pull requests implementing required changes:

https://github.com/Slicer/Slicer/pull/7190
https://github.com/Slicer/Slicer/pull/7191

Pull request implementing useful changes:

https://github.com/Slicer/Slicer/pull/7193

Pull request implementing required changed:

https://github.com/Slicer/Slicer/pull/7195

Especially the backport of MR-10452: OpenXR: Consistently name manifest install sub-directory

The extension can now be built on Windows (and even packaged & installed) without requiring the developer to copy files and libraries.
Next step we will need to refactor SlicerVirtualReality extension so that relevant functionalities can be reused while minimizing duplication:

https://github.com/KitwareMedical/SlicerVirtualReality/issues/121

We will also have to improve the OpenXR integration so that we can retrieve the position of each controllers & trackers:

https://github.com/KitwareMedical/SlicerVirtualReality/issues/122

cc: @LucasGandel @lassoan @cpinter @adamrankin


How is a handler that handles multiple requirements invoked?

This page clearly states how to create a handler that handles multiple requirements, but it does not say how such a handler is registered or invoked. For single-requirement handlers, you add the requirement to a policy, and the framework will resolve the handler based on the requirement. How do you add a multiple-requirement handler to a policy (perhaps through the authentication scheme)? Also, it's not clear in what order requirements will be resolved if you mix single-requirement and multiple-requirement handlers together, which could be relevant when dealing with the PendingRequirements property. More information would be helpful.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: b1141d92-adeb-6841-229f-914c808f9a2b
Version Independent ID: 7089b376-949f-c1c6-c37a-5d8cc7ecf104
Content: Policy-based authorization in ASP.NET Core
Content Source: aspnetcore/security/authorization/policies.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Moved to #16191


Compact Solars

Please add compact solars (like the IC2 compact solars)
I would be very happy if you added this although some people may consider it to be overpowered

This is somewhat implemented under #32, and I think that's as far as I'm going.


[#20] feat: add build.yml for github action

Description
[#20] feat: add build.yml for github action

Related Issue

close: #20


When testing on the expanded matrix, run tests on M1 Mac (via custom runner) as well.

Description

Provide details of the change, and generalize the change in the PR title above.

If running tests against the expanded matrix, it will build an ARM64 version of the integration tests, and run it on an M1 Mac GitHub runner that we host ourselves.
For security purposes, this job does not use any third-party marketplace actions, nor does it pass in secrets on the command line.
The test_desktop_custom_runner step is written as generically as possible to support other custom runners we may add in the future.

Testing

Describe how you've tested these changes. Link any manually triggered Integration tests or CPP binary SDK Packaging Github Action workflows, if applicable.

Run integration-tests workflow on the expanded matrix. This works both on prebuilt SDK and from source.

Type of Change
Place an x the applicable box:

[ ] Bug fix. Add the issue # below if applicable.
[ ] New feature. A non-breaking change which adds functionality.
[X] Other, such as a build process or documentation change.

Notes

Bug fixes and feature changes require an update to the Release Notes section of release_build_files/readme.md.
Read the contribution guidelines CONTRIBUTING.md.
Changes to the public API require an internal API review. If you'd like to help us make Firebase APIs better, please propose your change in a feature request so that we can discuss it together.

@sunmou99 I'm still considering this a draft change because I want to make sure that the runner is stable before merging this, but could you take a look at the changes I've made in this PR and give me your comments?
In particular I am wondering if it would be better to separate out architecture in the summarize script rather than the hack I've done. I'm intending to add x86 integration tests soon, so maybe that would be a better time to make that change.


Key store support

@uschi2000 @markelliot
Sorry, this took a bit longer than I thought -- actual code is pretty straight-forward, but getting the certificate extensions for intermediate CA CRLs to work correctly for DropWizard tests took a bit. Core parts of the change are separated as independent commits.
Main things in this change:

KeyStore supports the "PEM" store type -- can point it to a single PEM file that contains private key and certificate chain, or directory containing multiple such files

Had to import BouncyCastle PKIX library to read the PKCS#1 RSA key, but that library only has PKIX operation utilities (not an entire security provider) so it's a fairly small import (~600k)

Added "Puppet" store type which can be used for trust or keys (assumes Puppet SSL directory structure)
Bash script for cert generation refactored and adds generation/test cases for keys signed by an intermediate CA

This should make it much easier to get an SSL HTTP connection up and running with minimal command line operations in most cases.

Have uncertain connectivity today, may not get a chance to look until
tomorrow.
On Sun, Mar 27, 2016 at 5:34 PM nmiyake<EMAIL_ADDRESS>wrote:

@uschi2000 https://github.com/uschi2000 @markelliot
https://github.com/markelliot
Sorry, this took a bit longer than I thought -- actual code is pretty
straight-forward, but getting the certificate extensions for intermediate
CA CRLs to work correctly for DropWizard tests took a bit. Core parts of
the change are separated as independent commits.
Main things in this change:

KeyStore supports the "PEM" store type -- can point it to a single
PEM file that contains private key and certificate chain, or directory
containing multiple such files

Had to import BouncyCastle PKIX library to read the PKCS#1 RSA
key, but that library only has PKIX operation utilities (not an entire
security provider) so it's a fairly small import (~600k)

Added "Puppet" store type which can be used for trust or keys
(assumes Puppet SSL directory structure)
Bash script for cert generation refactored and adds generation/test
cases for keys signed by an intermediate CA

This should make it much easier to get an SSL HTTP connection up and
running with minimal command line operations in most cases.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/palantir/http-remoting/pull/92#issuecomment-202176841

Ping -- this isn't urgent (I'm duplicating the code I need in my client project, which is good enough for now), but if either of you get a chance to look over this please let me know so I can iterate on any feedback.

@markelliot @uschi2000
Ready for review. Adds ability to read PKCS#1 keys from PEM files. Uses BouncyCastle as a compile-only dependency. If it's present on the classpath, it will read using those libraries. If it's not, it will fall back to an implementation that uses sun.security.util classes to read the key. Although these classes are Sun proprietary, they exist in both Open JDK and Oracle JDK 7 and 8. If the Sun classes are not present, then a descriptive exception is thrown.
gradle-docker-test-runner is used to verify the behavior in all of these scenarios -- the standard tests run with BouncyCastle as a test runtime dependency, and there are Docker environments for no BouncyCastle with Sun (OpenJDK) and no BouncyCastle or Sun (IBM JDK).

Generally looks very cool, thanks Nick! I didn't look at all tests in all detail. Besides a few nits above, I have one ask: add the canonical minimal examples for working with different keystores and key types to the readme.

@uschi2000
Addressed PR feedback

I am happy with this. @markelliot ?

2 notes, can take or leave them. Think it's probably worth taking this but then refactoring support for PCKS1 to use ServiceLoader if urgency, otherwise might delay.

@markelliot @uschi2000 ready for review -- incorporated Mark's feedback and implemented using ServiceLoader instead of using previous approach, and makes everything much cleaner. Also makes it possible to test without using gradle-docker-test-runner since environments can be controlled by whether or not projects are included rather than having to depend on JVM environment.

Thanks for continued iteration here --- I think the ServiceLoader stuff makes this way simpler and easier/safer to lean on the appropriate impls

Addressed PR feedback

lgtm. Squash?

Squashed into 2 main commits

Cheers, Nick!

Thanks for the help on this! Depending on what else we want in it, can we cut a 0.10.0-rc2 with this in it so I can use it in a client project?

yep, waiting for#96
On Wed, Apr 13, 2016 at 2:00 PM Nick Miyake<EMAIL_ADDRESS>wrote:

Thanks for the help on this! Depending on what else we want in it, can we
cut a 0.10.0-rc2 with this in it so I can use it in a client project?
—
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub
https://github.com/palantir/http-remoting/pull/92#issuecomment-209645680


Profession visible in display name for PSC users

[ ] Get the reference table
[ ] Implement in the user mapping

Hey team! Please add your planning poker estimate with Zenhub @heroinedor @JDixmier


ament_cmake_catch2: 1.2.1-1 in 'humble/distribution.yaml' [bloom]

Increasing version of package(s) in repository ament_cmake_catch2 to 1.2.1-1:

upstream repository: https://github.com/open-rmf/ament_cmake_catch2.git
release repository: https://github.com/ros2-gbp/ament_cmake_catch2-release.git
distro file: humble/distribution.yaml
bloom version: 0.11.2
previous version for package: 1.2.0-3

ament_cmake_catch2

No changes

PR was opened by bloom but manually pushed a commit to switch versions for doc and source to humble which is in line with https://github.com/ros2-gbp/ament_cmake_catch2-release/pull/1


Error with ignoreMinBufferTime --> Cannot read property 'variants' of null

Have you read the FAQ and checked for duplicate open issues? yes
What version of Shaka Player are you using? 2.5.2
Can you reproduce the issue with our latest release version? Yes
Can you reproduce the issue with the latest code from master? Yes
Are you using the demo app or your own custom app? Both
If custom app, can you reproduce the issue using our demo app? Yes
What browser and OS are you using?
Chrome 75 over macOS Mojave
For embedded devices (smart TVs, etc.), what model and firmware version are you using?
What are the manifest and license server URIs?

https://livesim.dashif.org/livesim/segtimeline_1/utc_head/testpic_6s/Manifest.mpd
What did you do?

Load the next page: https://shaka-player-demo.appspot.com/demo/#audiolang=es-ES;textlang=es-ES;ignoreMinBufferTime=true;uilang=es-ES;asset=https://livesim.dashif.org/livesim/segtimeline_1/utc_head/testpic_6s/Manifest.mpd;panel=SEARCH;build=uncompiled
Stream url: https://livesim.dashif.org/livesim/segtimeline_1/utc_head/testpic_6s/Manifest.mpd
Enable ignoreMinBufferTime param (custom config)
What did you expect to happen?
The video start without any problem
What actually happened?

There are a error: "Cannot read property 'variants' of null"

Ok, after looking over it, I found a bug in the DASH parser where it can end up setting the presentationDelay to NaN in some situations. Now that I see it, it's obvious enough that I briefly wondered if that was somehow intended behavior. I'll get a CL out soon. Thanks for the bug report!


Update doc to use export default where relevant

I may be wrong as I did a search/replace on the whole doc by filtering some files where a module.exports is needed.

Closing this as part of cleaning because it's old and may have to be done later again.


Run scripts on published modules and fail publishing depending on outcome

Would love it if we could configure PS scripts to be run on every script that was uploaded / published and publish or return an error based on the result of the script.
Some examples:

Run psscriptanalyzer and refuse to publish if script does not meet threshold.
Check for and run pester tests, refuse to publish if tests fail or are nonexistent.
push to git repo on upload
Sign scripts / modules with internal signature

@Tiberriver256 Good ideas..
We have a similar system for the public repository - PowerShellGallery.com where contents are ScriptAnalyzer scanned/tagged regularly.
In fact, ScriptAnalyzer has rules to validate the quality of a DSC resource (Tests present etc). It would be a good idea to run them and ensure the Check-in tests pass 100%
These must be DSC resources themselves - and integrated in Publish-Module pipeline


Questions on Utilizing COBRA for Megaphage and Plasmid Analysis

Thank you very much for providing these tools. I have two questions I would like to consult with you. My research focuses on the ecological functions and evolution of bacteriophages.
1.	Can the high-quality megaphage sequences recovered using COBRA be analyzed together with non-megaphage contigs (including processes like binning and functional annotation)? I am uncertain whether combining these extended high-quality viral contigs with other contigs for joint analysis might introduce systematic biases.
2.	Is it possible to use COBRA to recover or extend plasmid fragments?
I would greatly appreciate it if you could provide answers to these questions. Thank you once again for your kind assistance!

Hi, thank you for your questions.

COBRA does not check if a query comes from a megaphage or a non-megaphage.
Yes, it is doable.
Please let me know if you have more questions.

Please keep in mind all queries (-q) and contigs/scaffolds (-f) should be original from the assembly without cutting or any other modifications.

Thank you for your response. Regarding the first question, my earlier phrasing may not have been clear. Can the high-quality viral contigs recovered using COBRA be analyzed downstream together with the remaining contigs that could not be recovered? Would such an approach introduce systematic biases?

Yes, I understand that all queries (-q) and contigs/scaffolds (-f) should originate from the original assembly.

good question.
it is fine. COBRA will generate a new fasta file, which includes all the unique joined sequences and other original sequences that have not been joined, for subsequent analyses.

Got it, thank you so much—I truly appreciate your help!

if your queries are predicted viral contigs/scaffolds, and you want to focus on them for the subsequent analyses, you could directly combine the five fasta files of the five categories.

I understand.

Hello Professor Chen, another question: You mentioned that all queries (-q) and contigs/scaffolds (-f) should be original from the assembly without cutting or any other modifications. However, If I have generated scaffolds using MetaSPAdes and produced some viral scaffolds with MetaVirusSPAdes, is it appropriate to merge these datasets and then use a viral classifier to select those sequences identified as viral, subsequently using them as input for COBRA?

Hi, I have not tested MetaVirusSPAdes, but please remember that -q and -f contigs/scaffolds should come from the same assembly. I hope this helps.


docs: add ibc channel info for testnet 62 iapetus

WIP: started this PR after release of 62, but wasn't able to figure out the right steps. In particular, the --timeout-height default arg will not work.

The error I'm seeing on the timeout-height is: error: Invalid value "0-0" for '--timeout-height <TIMEOUT_HEIGHT>': attempted to parse an invalid zero height.
If I override the default value with a custom one, such as "5-100000":
Error: status: Internal, message: "could not broadcast transaction: Error submitting transaction: code 1, log: timeout height 5-100000 is less than the latest height 5-3153474", details: [], metadata: MetadataMap { headers: {"content-type": "application/grpc"} }

What structure should this arg take? What is a sane default value?

@hdevalence clarified in chat that the timeout-height applies to the counterparty chain, and additional is a tuple of two values: chain revision and block height. I can't think of a sane default value for this, so I've updated the docs to be explicit about declaring it. Relevant docs:  https://rustdoc.penumbra.zone/main/penumbra_ibc/struct.Ics20Withdrawal.html

LGTM


Remove warning on run serve with one worker

Currently, -w1 show warning "number of workers must be larger than 1"

Yes. One worker is alright.


Implementation suggestion for Nord Vlei Extension

In polaris-web, there are three different methods with a slightly different implementation as per the use cases of signify-polaris-web.
Namely:

authorize
authorizeAid
authorizeCred

Signify Extension
If authorize call is made user can select both credential or aid
If authorizeAid call is made user can only select aid
If authorizeCred call is made user can only select a credential

However, this behavior is a bit different in Nord Vlei. NordVlei extension only listens to authorize method call and based on that displays a list of credentials to be selected.
State of NordVlei extension when authorize call is made from polaris-web

We want to make Reg Pilot Web APP compatible with both NordVlei and Signify Extension.
If we use authorize: In Signify Extension, it would allow user to pick any of Credential or AID, we dont want that.
If we use authorizeCred: it is not implemented in NordVlei and it would not be listened.
Suggestion:
Our suggestion is to replace authorize method in NordVlei with authorizeCred. That way it would be compatible for both of the extensions.
@2byrds

@HunnySajid thank you for the details. @lenkan and @daviddm lets discuss when you return.

Thanks for the detailed description @HunnySajid ! We have added support for authorizeCred in our extension now, we just need to wait for approval before publishing the updated extension.

Thanks for the detailed description @HunnySajid ! We have added support for authorizeCred in our extension now, we just need to wait for approval before publishing the updated extension.

That's great, go team!

It is now published in version 0.0.5 of the extension in the chrome store.

It is now published in version 0.0.5 of the extension in the chrome store.

much appreciated 👏