Improved navigation, better tools docs

Updated submodules (PHP markdown conversion, tools repo)
Use the nf-core/tools readme for the tools page so that we don't have to maintain two documents
Improved the navigation: dropdowns for Usage and Developers
Added new page for documentation describing how to add a new pipeline (docs themselves still to be written properly)

Fixes #20 and #18, starts to address #42

Added more stuff sorry @apeltzer - feel free to merge when you're happy 😀


ConnectionResetError: [WinError 10054] An existing connection was forcibly closed by the remote host

loading weights/cellbit.pth
gin_channels: 256 self.spk_embed_dim: 109

2023-07-24 19:34:12 | ERROR | asyncio | Exception in callback _ProactorBasePipeTransport._call_connection_lost(None)
handle: <Handle _ProactorBasePipeTransport._call_connection_lost(None)>
Traceback (most recent call last):
File "C:\Users\WhiteC\AppData\Local\Programs\Python\Python310\lib\asyncio\events.py", line 80, in _run
self._context.run(self._callback, *self._args)
File "C:\Users\WhiteC\AppData\Local\Programs\Python\Python310\lib\asyncio\proactor_events.py", line 162, in _call_connection_lost
self._sock.shutdown(socket.SHUT_RDWR)
ConnectionResetError: [WinError 10054] An existing connection was forcibly closed by the remote host

I think you successfully get the audio after voice conversion.

I think you successfully get the audio after voice conversion.

Same issue, but I successfully get the audio. But why this error is showing?
2024-05-29 11:58:48 | INFO | infer.modules.vc.modules | Get sid: keruanV1.pth
2024-05-29 11:58:48 | INFO | infer.modules.vc.modules | Loading: assets/weights/keruanV1.pth
2024-05-29 11:58:50 | INFO | infer.modules.vc.modules | Select index: logs\keruanV1.index
2024-05-29 12:00:35 | INFO | infer.modules.vc.pipeline | Loading rmvpe model,assets/rmvpe/rmvpe.pt
2024-05-29 12:01:01 | ERROR | root | Exception in callback _ProactorBasePipeTransport._call_connection_lost(None)
handle: <Handle _ProactorBasePipeTransport._call_connection_lost(None)>
Traceback (most recent call last):
File "asyncio\events.py", line 80, in _run
File "asyncio\proactor_events.py", line 162, in _call_connection_lost
ConnectionResetError: [WinError 10054] An existing connection was forcibly closed by the remote host


Support getting response headers

Describe the feature
HTTP Location headers are being used to point to resources created by POST request as part of a http 201 response.
My use case uses an after handler to transform my internal urls to public ones, I want to be able to filter headers as well.

You could use ctx.res.getHeader(headerName) ?

Awesome I missed that there is access to the backing http response


[RAO] Pooled model of delegation

Most recent design
https://docs.google.com/document/d/15DxE9wbja3iGYqkq8zrh7x2pTj0HBpDF-f1_LYL1nu4/edit?tab=t.0
Background (The issue)
The chain triggers a subnet's emission calculation and step depending on its tempo. This means every few blocks we run a calculation to determine the correct emission for every subnet based on the current StakeMap. The calculation of emissions has been taking a long time, slowing block times.
This calculation appears to be slowed by the large number of entries in the StakeMap. To remedy this, two suggestions were made:

https://github.com/opentensor/subtensor/pull/361 - Set a minimum stake amount for each nomination entry
https://github.com/opentensor/subtensor/pull/367 - Limit the number of nominators for each entry

The accepted solution (should be live on the chain Apr 29th) is (1). This means we greatly reduce the number of delegator entries in the StakeMap at the cost of limiting participation to TAO stake holders with more than the limit of k TAO (dynamic value).
The issue
The issue with this, as mentioned, is this change reduces the participation of holders to only those that hold over a certain threshold of TAO.
What is a good solution (Acceptance Criteria)
A good solution would mean we still reduce the runtime of the calculation during each subnet tempo step, while retaining as much participation among as many stake-holders as possible.
Existing Solutions
Polkadot implements an approach called Nomination Pools[^1] which addresses a similar issue by pooling stakers into one account, reducing the size of their stake entries while maximizing participation of individual (read: small) holders.
[^1]: https://github.com/paritytech/polkadot-sdk/blob/master/substrate/frame/nomination-pools/src/lib.rs For the polkadot/FRAME impl
Regarding their implementation, some elements of their design are unclear, i.e., I am not sure why they chose to:

separate rewards from stake
rebalance shares (aka "points") on each join/leave

Though I think this may be because of Polkadot's stake bonding time requirements (Bittensor lacks this lock period).
I imagine they decided to rebalance on every change to the pool in order to maintain the 1:1 (points to funds) ratio because fractional ownerships are imprecise and can cause future issues. I think we could do the same with not much more hit to performance.
Proposed Solution
We could represent delegation as shares in the a stake pool, similar to the Polkadot[^1] solution.
Then on subnet tempo steps, we have less accounts to consider in the StakeMap.
This would change the way delegation works, i.e., delegates would operate as stake pools, where the corresponding entry in the StakeMap is the pool's total stake.

On emission to a pool, increase the pool stake and increase owner shares (or rebalance)
On add_stake, update the shares of the staker (or all the shares)
On remove_stake, update the shares of the unstaker (or all the shares)

Considerations
It is useful to consider when the shares are adjusted versus the pool's stake (i.e. the ratio of shares to stake). There are currently two ideas:

Adjust the ratio on every change to the total stake
a. i.e. maintain the 1:1 ratio of shares to TAO
Do not adjust the ratio at all    a. increase the pool balance on emission
b. i.e. commission to the pool owner increases their shares
c. (de)issue new shares on (un)stake
The example python code implements the latter (2).

There is a trade-off here. (2.) is less precise due to fractional ownership of the pool's stake, but requires less operations (e.g. rebalances) to occur. Whereas (1.) is exactly precise, but will require rebalancing of the shares on every update to the pool, which may not improve runtime at all (yet to be tested).
An example in python:
from typing import Dict

class NominationPool:
    owner: str
    shareMap: Dict[str, float] = {}
    shareSum: float = 0
    commission: float
    poolStake: float = 0

    def __init__(self, owner: str, comission: float = 0.18):
        self.owner = owner
        self.shareMap[owner] = 0.0
        self.shareSum = 0.0
        self.comission = comission

    def addNomination(self, nominator: str, amount: float):
        # Find how much this amount will increase the pool
        if self.poolStake == 0:
            increase = 1
            # Issue initial shares
            new_shares = 1
        else:
            increase = amount / self.poolStake
            # Find the amount of shares issued
            new_shares = increase * self.shareSum

        # Issue the shares
        if nominator not in self.shareMap:
            self.shareMap[nominator] = 0
        self.shareMap[nominator] += new_shares
        self.shareSum += new_shares
        # Increase the pool stake
        self.poolStake += amount

    def getNomination(self, nominator: str):
        return self.shareMap[nominator]

    def getStake(self, nominator: str):
        # Find the amount of stake the nominator has
        if self.shareMap.get(nominator, 0.0) == 0:
            return 0.0

        return (self.shareMap[nominator]/self.shareSum) * self.poolStake

    def getCommission(self):
        return self.commission

    def getPoolStake(self):
        return self.poolStake

    def getPoolSize(self):
        return len(self.shareMap)

    def getShareOfPool(self, nominator: str):
        return self.shareMap[nominator] / self.shareSum

    def emitThroughPool(self, amount: float):
        # Find the owner commission
        commission = amount * self.comission
        # Give the rest to the pool
        left_amount = amount - commission
        self.poolStake += left_amount
        # Add the commission to the owner
        self.addNomination(self.owner, commission)

    def removeStake(self, nominator: str, amount: float):
        # Find the amount of shares to remove
        decrease = amount / self.poolStake
        # Find the amount of shares to remove
        remove_shares = decrease * self.shareSum
        # Remove the shares
        if self.shareMap[nominator] < remove_shares:
            raise ValueError("The nominator does not have enough shares")

        self.shareMap[nominator] -= remove_shares
        self.shareSum -= remove_shares
        # Decrease the pool stake
        self.poolStake -= amount

Tasks

[x] Implement share pool primitive
[x] Add share pool state maps/values and implement three share pools: Alpha, Delta Alpha, and Global
[ ] Replace Alpha with Alpha share pool without breaking anything else. Remove Alpha state map.
[ ] Introduce Global share pool: Implement rebalancing function that populates Global share pool from Alpha share pool, call it on every block for now. Make get_global_for_hotkey_and_coldey_on_subnet use this Global share pool instead of Alpha share pool, maintain Global share pool updates in coinbase (e.g. when hotkey emissions happen, delegate take, childkey take, and miner emission are added to stake, etc.).
[ ] Introduce Delta Alpha: Make run_coinbase add to delta alpha instead of adding to Alpha, make all RPC use delta alpha in combination with global share pool to calculate account Alpha. Make rebalancing function use delta alpha instead of Alpha (keep it running every block for now).
[ ] Reimplement TotalColdkeyAlpha and TotalHotkeyAlpha with share pools (reuse denominator for Alpha).
[ ] Replace Stake map with GlobalShare wherever it is used. Remove Stake map
[ ] Implement rebalancing triggers and stop calling rebalancing function on every block.
[ ] Migrations

[ ] Remove Stake map and populate Alpha shares and Global shares initially
[ ] Migration to remove LastAddStakeIncrease map
[ ] Migration to remove HotkeyEmissionTempo
[ ] Add PreAlpha swapping to perform_swap_hotkey

Tests

[ ] Share Pools

[x] Basic test: Shared value: 10, coldkey1 share: 3, coldkey2 share: 2, denominator: 5 => coldkey1 value = 6
[x] Edge case: denominator: 0 (no shares) doesn't panic
[ ] Basic test: Update one record => all keys update absolute value
[ ] Basic test: Update shared value => all keys update absolute value

[ ] Integration epoch tests

[ ] Delegate take
[ ] Childkey take
[ ] Miner emissions
[ ] Subnet owner cut
[ ] Nominator rewards

Regarding their implementation, some elements of their design are unclear, i.e., I am not sure why they chose to:

separate rewards from stake
rebalance shares (aka "points") on each join/leave

Though I think this may be because of Polkadot's stake bonding time requirements (Bittensor lacks this lock period).
I imagine they decided to rebalance on every change to the pool in order to maintain the 1:1 (points to funds) ratio because fractional ownerships are imprecise and can cause future issues. I think we could do the same with not much more hit to performance.

I suppose points exist as a solution for efficient accounting? With "the points system" established, after a pool gets slashed, instead of applying the penalty to all affected pool members immediately (by reducing their balances), runtime only needs to decrease the overall balance, which is much cheaper. The slash can then be applied lazily, like when a member withdraws, with the help of ratio of member_points / overall_points. I guess that's why they need to keep the points of members proportional: member_points / overall_points = member_balance / overall_balance?
Afaik, it's not too expensive to maintain. Like when some funds is bonded, the member's points can be calculated by balance * (overall_points / overall_balance), and then increase the overall balance and points.

On hold and back to research. Current issue is that we have local shares (per each subnet) and global shares (nominator's share of stake across all subnets). Local is something people can unstake, but global is something that determines how much local grows on each emission. And that's the problem because we can't just increase the counter of how much we have in the local pool. We need to re-balance the shares too because they are not growing evenly, which is the same amount of computation.

Shelved due to complexity.
Potentially an approach using pools as a staker-type in the maps makes more sense.
Then increase the individual staker min-stake to a sufficient reduction in map size.


S82 QSO Dataset

Adds the Stripe 82 QSO dataset from http://faculty.washington.edu/ivezic/cmacleod/qso_dr7/Southern.html, with the Light Curve files converted into a source parquet file, and the DB_QSO_S82.dat file converted into an object parquet file.
Adds some basic unit tests to validate the ability to load these datasets into TAPE (lsstseries).

unit tests are blocked by lsstseries v0.1.0, new TAPE release should resolve this.


InvalidLocationConstraint error in AWS S3 handled.

Creating a bucket with the boto3 AWS S3 client may return a InvalidLocationConstraint error, if we use the default value for an AWS region ("us-east-1") in the LocationConstraint parameter. More specifically:
ClientError: An error occurred (InvalidLocationConstraint) when calling the CreateBucket operation: The specified location-constraint is not valid
This error has been previously reported:
https://github.com/boto/boto3/issues/125
https://github.com/boto/boto3/issues/811
I simply propose a quick fix that retries the operation without the LocationConstraint if the region is "us-east-1".
Developer's Certificate of Origin 1.1
   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.

Great, thanks @GEizaguirre!


Can you upload the model files?

Hi, Thanks for your uploaded great work and your effort.
As you said on README, I just followed your guide.
I'm not good at tensorflow so I didn't understand how to generate python model files for unity.
Can you upload the bytes files necessary for running video sample scene?
I really want to try your great project.

Would also love to try it out
the model will be very appreciated


Drop support for obsolete PodSecurityPolicy

What this PR does / why we need it:
Drop obsolete support for PodSecurityPolicy. PodSecurityPolicy was deprecated in Kubernetes v1.21, and removed from Kubernetes in v1.25, but in the package k8s.io/api/policy/v1beta1 the type was still available until v1.29 for backwards compatibility.
Also various dependencies have been updated.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

Drop support for obsolete PodSecurityPolicy

Could you please adapt the pull request description? I think there is an error in the kubernetes version.
Hope it is clearer now.


Implement {to,from}_primitive for generated types.

This allows converting values to and from a representation that use only
primtive types, lists, and maps. This representation is easily serializable
into YAML/JSON.
Resolves #14.

:shipit:

:ship:

(ship)


feat: In Memory File System

This PR adds in memory file system feature.
How it works
During the build process (based on config) vulcan will read all specified directories and map the files. After this the build process will inject this mapping to a global var.
The fs node module will get infos from this in memory mapping.
How to test
You can test this in a next compute project.
In your project root create vulcan.config.js file to use this feature. The file must be like this:
module.exports = {
    memoryFS: {
        injectionDirs: ['.faststore/@generated/graphql'],
        removePathPrefix: '.faststore/',
    }
};

After this, run build and dev commands.
vulcan build --preset next --mode compute
vulcan dev

:tada: This PR is included in version 2.0.0-stage.1 :tada:
The release is available on:

npm package (@stage dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Adding Declared

Type: Missing
Summary:
Adding Declared
Details:
Not sure if this should be NONE or OTHER.  There is no license info on the NuGet page.  If you download the package, there is a line that says "View license terms," but the link goes to a defunct website - https://acedollar.com/.
Resolution:
Decided on NONE, as no findable license terms for this package.
Affected definitions:

AD.System.Web.Helpers.dll 2.0.0

@ariel11 I agree with NONE.


Remove updating the version status for now, noticed annotation also u…

This is not reliably updating.  We are using get nodes status to reliably check the version anyway.  I noticed the annotation update on Create also updates status.  Don't want to have 2 updates to status during the same reconcile.

The right way to avoid two updates is to batch them. For any non-cma users, I think it is important for us to provide a mechanism to determine when an update is complete. The current method using kubectl and parsing is less than ideal...

We decided to revert.  I'm doing that in another branch


[BUG] Cloud credential deletion fails intermittently in Rancher v2.7.0

Rancher Server Setup

Rancher version: v2.7.0
Installation option (Docker install/Helm Chart): Helm chart

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc): RKE1 with Kubernetes v1.21.14 and v1.24.4

Proxy/Cert Details: N/A

Information about the Cluster

Kubernetes version: v1.21.14 and v1.24.4
Cluster Type (Local/Downstream): local

User Information

What is the role of the user logged in? Default Admin

Describe the bug
When trying to delete a cloud credential via the cluster management -> cloud credentials option, an intermittent error appears with the message
Index with name by-cloud-cred does not exist

I noticed this when trying to delete a credential created via Terraform, but was able to replicate the issue solely using the GUI as well.
To Reproduce
Via GUI

Navigate to the cloud credentials page.
Create a new cloud credential for Digital Ocean.
Wait a few seconds.
Create a second cloud credential for Digital Ocean with the same API key.
Try deleting the first credential.
This message appears:

Keep trying to delete the credential and notice that it eventually gets deleted upon retries.

Via Terraform

Use Terraform to create a cloud credential in Rancher.

Sample Terraform configuration
terraform {
  required_providers {
    rancher2 = {
      source  = "rancher/rancher2"
      version = "1.25.0"
    }
  }
}

variable "api_url" {}

variable "token_key" {}

variable "do_token" {}

provider "rancher2" {
  api_url   = var.api_url
  token_key = var.token_key
  insecure  = true
  timeout   = "60s"
}

resource "rancher2_cloud_credential" "cloud_credential" {
  name = "cloud-credential"

  digitalocean_credential_config {
    access_token = var.do_token
  }
}

Plan & apply the creation.

Wait a few seconds, then plan & apply a deletion.

Error message appears:
Error: Error removing Cloud Credential: Bad response statusCode [500]. Status [500 Internal Server Error]. Body: [baseType=error, code=ServerError, message=Index with name by-cloud-cred does not exist] from [https://my-address/v3/cloudCredentials/cattle-global-data:id]

Note: I was also able to reproduce this issue with a single cloud credential, but it is intermittent. This issue appears to be more reproducible when there are more replicas of Rancher running. I scaled down my deployment to extract logs more reliably and was unable to reproduce the issue with a single deployment replica.

Result
Credential deletion fails intermittently.
Expected Result
Credential deletion goes through.
Additional Context

The error message also appears in Rancher pod logs, but enabling debug logging did not show more information:2023/02/13 14:55:57 [ERROR] Unknown error: Index with name by-cloud-cred does not exist

Issue appears to be related to a race condition between replicas, as it does not seem to happen when Rancher is deployed on a single replica.
I did not test other credential types, but I believe this is not specific to the Digital Ocean type of credential.

I can confirm this. I wanted to open an issue in the rancher2 tf repo. :)
My setup is slightly different, rancher is v2.7.0, v1.24.9+rke2r1, 3-node cluster. I’m creating only one s3 cloud credential through rancher2 terraform provider, and getting the same error when trying to delete. I added sleep after deleting the downstream cluster that is using that credential, but didn’t help.
This is happening from the UI as well.

Same problem here. I found out that clicking "Delete" multiple times led to successful removal for me.

Same thing here at 2.6.9

This issue seems like a duplicate of https://github.com/rancher/rancher/issues/39300 which was resolved by this PR https://github.com/rancher/rancher/pull/39622 and will be included in the 2.7.2 release (disregard the use of 2.7.1 in the PR title, this issue was resolved before the security release 2.7.1 was announced / planned)

This also happens when you edit a credential and it results in a new one being created that can't be deleted.
I also noticed the disabled Name text entry is blank. The dupe that gets created doesn't have a name. Feels related.

Closing as duplicate of https://github.com/rancher/rancher/issues/39300, will be resolved in 2.7.2.


dispatch to void-returning method, can't get simple example to compile

I believe this is my fault, and that I don't understand the dispatch call.
Make a new dub project. Add "optional": "~>0.1.0" as a dependency. The single source file of this new project contains the following example code. I've tried to build this with dub (no unittest) with optional 0.1.0 from dub, and also with the github master HEAD (6acdc75). In both cases, I get the errors at the bottom.
Example code:
import std.stdio;
import optional;

class C {
    void method() { writeln("Hi."); }
}

void main()
{
    auto c = some(new C());
    c.dispatch.method();
}

I can fix this by changing the return type of the method from void to int, return a value from the method (that I ignore in main). Is dispatch designed only for value-returning methods? How to call a void-returning method then?
Another way to fix: Removing the line with dispatch and putting if (auto c2 = c.unwrap) c2.method(); in its place fixes the errors below and prints "Hi." as expected.
Error:
Performing "debug" build using /usr/bin/dmd for x86_64.
bolts 0.1.2: building configuration "library"...
optional 0.1.0: building configuration "library"...
opti ~master: building configuration "application"...
../../../.dub/packages/optional-0.1.0/optional/source/optional/dispatcher.d(35,86): Error: template optional.optional.some cannot deduce function from argument types !()(void), candidates are:
../../../.dub/packages/optional-0.1.0/optional/source/optional/optional.d(231,6):        optional.optional.some(T)(T t)
source/app.d(11,22): Error: template instance `optional.dispatcher.OptionalDispatcher!(C, cast(Flag)true).OptionalDispatcher.opDispatch!"method".opDispatch!()` error instantiating
../../../.dub/packages/optional-0.1.0/optional/source/optional/optional.d(221,54): Error: template std.conv.to!string.to cannot deduce function from argument types !()(void), candidates are:
/usr/include/dlang/dmd/std/conv.d(205,7):        std.conv.to!string.to(A...)(A args) if (A.length > 0)
/usr/include/dlang/dmd/std/conv.d(212,7):        std.conv.to!string.to(S)(ref S arg) if (isStaticArray!S)
/usr/include/dlang/dmd/std/conv.d(219,7):        std.conv.to!string.to(S)(ref S arg) if (isAggregateType!S && !isCopyable!S)
../../../.dub/packages/optional-0.1.0/optional/source/optional/dispatcher.d(14,9): Error: template instance `optional.optional.Optional!void` error instantiating
../../../.dub/packages/optional-0.1.0/optional/source/optional/dispatcher.d(35,32):        instantiated from here: OptionalDispatcher!(void, cast(Flag)false)
source/app.d(11,22):        instantiated from here: opDispatch!()
/usr/bin/dmd failed with exit code 1.

Thanks for the report! Got a fix and will release soon as well 👍


showing blank/white screen on ios

i have implement this package which is working perfect in android and making live payment also, but when apply on ios it shows blank screen. i am testing on physical device.

i have implement this package which is working perfect in android and making live payment also, but when apply on ios it shows blank screen. i am testing on physical device.

Same issue

for this i have

i have implement this package which is working perfect in android and making live payment also, but when apply on ios it shows blank screen. i am testing on physical device.

Same issue

for solution i had expo eject and use the tipsi-stripe package for stripe payment, and that is working fine. expo has very limitations.

i have implement this package which is working perfect in android and making live payment also, but when apply on ios it shows blank screen. i am testing on physical device.

If you have use it with expo can you guide me how to use it?
because i am getting lot of errors.


Split out atomic linked list behaviors into their own datatype

There are some signal issues that are related to bugs with an atomic list behavior on Linux.  Create a dedicated atomic linked list concept that allows us to build and test a reliable atomic linked list concept separate from the rest of signal.

[x] Ensure this type can be used with polymorphic types without needing reallocation

This test it's failing on all platforms, I think it is the root cause of our signal instability.  Will investigate when I get home.


Exit status 99

I'm trying to google these issues but not getting any hits...but I haven't been able to get this working on two different Macs running Big Sur.
I'm trying to install the SmartThings bitbar (xbar) plugin. I followed all the instructions including chmoding the script, the entire folder, etc. The xbar menu icon has a yellow triangle with exclamation in the middle. When I click on the xbar icon, it says fork/exec ./ST: permission denied and also Make your script executable: chmod +x ST. I've tried chmoding and sudo chmoding many times now.
When I open the Plugin Browser, it says at the top in a yellow banner Something went wrong, this: Get "https://xbarapp.com/docs/plugins/categories.json": context deadline exceeded
The St.1m.sh menu bar item says exit status 99.
Any tips on what to try?

I am no longer using SmartThings.   This ST app is no longer supported.


[Editted]  [Original = Build and install via source] [New = Disable logging]

Hi , I'm currently try to use piper on Ubports app and include piper on the clickable package , to do that i need the compiled piper not a precompile nor a whl package that can download from pip .
so only options left is to build it via source code
already done building it . but don't know how to install it to a custom site-packages folder since the source code doesn't have "setup.py" it cannot install via pip3 install --target=folder .
any thoughts on this ? thanks

okay i found a solution , i just build piper phonemize then install piper tts and onnyxruntime with --no-deps ,
but i've got another issue , ubports app is confined app , it doesn;t allowed Logger from onnxruntime , where can i find the code that can disable onnyxlogging ?


Severe: RoutObserver does not work for iOS swipe gesture.

I was very surprised to find out, that there is no way to address the iOS 'swipe back gesture' (in Flutter at all), which I assume 95% of iOS user use. Overall, I know of just not a single way in flutter to overwrite or observe the iOS swipe back gesture (besides WillPopScope to simply block it).
So for instance, didPop() of RouteObserver is not working (only triggered via back button) so how should I be able to e.g. save a value to a database, if a user pops a route via iOS swipe back (e.g. for unread messages -> timestamp saving or saving of data via back swipe etc.).
How should I be even able to push another route to a swipe back gesture?
This issue is so severe, I would highly suggest that it is implemented asap.

Related:
https://github.com/flutter/flutter/issues/29596

Let's merge into #29596 since it's the same issue.


About AppSplitter two tag.

<menu-page></menu-page>
<app-tabbar></app-tabbar>

I don't know why this two tag will auto fit to components: { AppTabbar, MenuPage }?
I can't find out these two tag documents anywhere.
Could you help?

Those two tags are Vue components created in this application. MenuPage is here and AppTabbar is here. The are imported with import AppTabbar from '...', etc.


add --order parameter to storm list command

Hi @emre
there is problem with this program, for example image somebody has its config file and into that file it has one generic config and two client config. Then he decides to use stormssh to makes his life easier.
Now he tried to add new host, Stormssh add this new host at the end of config file, So what is the problem ?
since you added to end of the file, whenever he/she tries to ssh to that new client all generic options will be used  by ssh? Why ? 'Cause order is important on ssh config file.
Thanks


Update CHANGELOG.md with changes in #89

This PR:

[X] Updates CHANGELOG.md with changes in #89.

@nake89 Is this good or do you want me to include more details?

I'm trying to think what would be good info to tell the people who read the CHANGELOG.md.
Maybe you can add a nested list. You can use your own discretion.
Suggestions. Maybe these are important?

Replaces mocha with Vitest as it has better ESM and TypeScript support.
Replaces rollup and other rollup related dev dependencies with tsup.

Not sure if this is important:

Uses tsup to output both CJS and ESM entry points.

Because I only use tiged as CLI tool. But people who import or require or whatever might be interested? Dunno :D

Sure I can add more details.

@nake89 I would love that, it seems like the link is not valid though.

Thanks for letting me know. Hopefully this works: https://discord.gg/bNHqU797ee
I'll update it to the read me soon.


Add project members when a new project is created

https://github.com/remsfal/remsfal-backend/blob/0307cc7ab59182efef0defedbbac6fbd5dcf1514/remsfal-service/src/main/java/de/remsfal/service/control/ProjectController.java#L62

Is this bug still a problem? When I tried it, my user was added to the project as a member.

first issue claimed

This issue is obsolete.


update addclass UML diagrams

Codecov Report

Merging #316 (923a69c) into master (a023abd) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #316   +/-   ##
=========================================
  Coverage     71.20%   71.20%
  Complexity      801      801
=========================================
  Files           129      129
  Lines          2608     2608
  Branches        322      322
=========================================
  Hits           1857     1857
  Misses          665      665
  Partials         86       86

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 a023abd...923a69c. Read the comment docs.


Change From implementations to TryFrom

I feel a bit bad about opening this issue as I proposed implementing From for the InfoType.
Looking at the current code, I think it could be more idiomatic Rust. When I proposed implementing From I thought that to_*() methods would be removed from InfoType as they are redundant (I'm a fan of having only a single way of doing things). Though I can understand to keep them for backwards compatibility.
Nonetheless, if we now have both, I propose replacing the From implementations with TryFrom. This way one could decide whether one wants to panic or process the error. Let me give you a code snipped to show what I mean.
#[derive(Debug)]
pub struct InfoTypeError(String);

impl TryFrom<InfoType> for cl_int {
    type Error = InfoTypeError;

    fn try_from(info_type: InfoType) -> Result<Self, Self::Error> {
        match info_type {
            InfoType::Int(a) => Ok(a),
            _ => Err(InfoTypeError("not a cl_int".to_string())),
        }
    }
}

impl InfoType {
    pub fn to_int(self) -> cl_int {
        cl_int::try_from(self).expect("not a cl_int")
    }
}

Another reason to use TryFrom is that the documentation for From states:

Note: This trait must not fail. If the conversion can fail, use TryFrom.

And I'd consider a panic also as "failing" (though, I'm not sure if it matches Rusts definition)

I'd consider a panic also as "failing" (though, I'm not sure if it matches Rusts definition)

The rust definition (from the Rust book) is here: To panic! or Not to panic! and states "You could call panic! for any error situation". E.g. a bug in the calling program due to a programmer attempting to convert an InfoType into an incorrect specific type. That is not a "recoverable error", such as a OpenCL function returning an error code. Therefore, the conversion cannot fail unless the programmer writes buggy code (in which case the panic will help him/her fix their bugs), hence From is correct, not TryFrom.

I thought that to_*() methods would be removed from InfoType as they are redundant

I'm fan of changes not breaking existing APIs. Volker, as you are well aware, the opencl3 is based upon this library and calls the to_*() methods in many places, did you expect me to rewrite them all for that change?!
I'm also a fan of explicit code vs implicit code. The to_*() methods are more explicit and easier for someone to debug, if and when they call the wrong to_*() method on an InfoType than a From/TryFrom conversion would be.

I'm a fan of having only a single way of doing things

So am I! Since the to_*() methods are more explicit than From/TryFrom and you can't decided between From and TryFrom, I propose deleting the From traits from info_type.rs. ;)

I feel a bit bad about opening this issue

And so you should!

Thanks for the detailed reply.


docs: accurate readme example

The module does not have named exports

Codecov Report

Merging #7 (f94b261) into master (334e40b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master       #7   +/-   ##
=======================================
  Coverage   99.32%   99.32%
=======================================
  Files           5        5
  Lines         148      148
=======================================
  Hits          147      147
  Misses          1        1

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 334e40b...f94b261. Read the comment docs.


Ability to have custom key mappings

A re-implementation of PR #50 with tests included, a slight difference, extend is now extendMap
Mousetrap.extendMap({
  173: 'mute',
  174: 'volumedown',
  175: 'volumeup',
  176: 'nexttrack',
  177: 'previoustrack',
  178: 'stop',
  179: 'playpause'
});

Is there something bad about this PR ?

Sorry. I don't spend a lot of time working on mousetrap these days. I started merging this a couple weeks ago, but forgot. I renamed the method from Mousetrap.extendMap to Mousetrap.addKeycodes.
It should work as of version  1.6.0.


build.modulePreload=false breaks lazy CSS injection

Expected
I want to disable the preload of JavaScript modules,
but I still want to inject CSS files when the modules are actually loaded.
For achieving this, I'm configuring the build.modulePreload=false.
Actual
The preload of JavaScript modules is disabled as expected,
but the stylesheets of lazy loaded modules are not injected.
Reproduction
https://stackblitz.com/edit/vitejs-vite-25ecud?file=vite.config.ts
Steps to reproduce
Steps to reproduce the issue can be found in the README
of the reproduction repository.
https://stackblitz.com/edit/vitejs-vite-25ecud?file=README.md
More Info
This Vite configuration breaks the injection of lazy CSS:
export {
    ...
    build: {
        modulePreload: false,
    },
}

BUT this configuration works as expected:
export {
    ...
    build: {
        modulePreload: {
            resolveDependencies(url, deps, context) {
                return []
            },
        },
    },
}

System Info
System:
    OS: Linux 5.0 undefined
    CPU: (8) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 0 Bytes / 0 Bytes
    Shell: 1.0 - /bin/jsh
  Binaries:
    Node: 16.14.2 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 7.17.0 - /usr/local/bin/npm
  npmPackages:
    @vitejs/plugin-react: ^4.0.0 => 4.0.0
    vite: ^4.3.5 => 4.3.5

Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Duplicate of #10773


add telegram icon and link to footer

based @jsigwart ideas on #bounty channel LBRY Discord Server.
My Preview Screenshot

This has to be done differently. The footer is a partial html page that is pulled in, so adding a head tag is not appropriate here. You need to modify the head tag in the correct header file.

You don't need to import font-awesome its suppose to already imported in the CSS

but in css file there no icon telegram

It could be an old version of font-awesome let me look it up

our version of font-awesome doesn't have the telegram icon

I open an issue for an update to font-awesome #589

shouldn't this go after Discord?

@jsigwart here the result . i found the correct header file on layout/basic.php

@jsigwart its ready to merge. based on your reverting font awesome to earlier version

*Should be a top menu item [JOBS]. No one is going to see it hidden in the footer.

No footer on the homepage 😮 = No "LBRY is hiring!" link!


[connectors] Slack polish

https://github.com/dust-tt/tasks/issues/1389
Description

Add action in "Thinking .." text
Header with links to dust and conversations
Add selector for switching to another assistant

Risk

Deploy Plan
deploy connectors

It feels weird that the link to the conversation is displayed twice:

Any chance we can consolidate everything in the context under the separator?


Plugin update to version 0.13.46 not possible (ENOTEMPTY)

Log Messages

OS | Raspbian GNU/Linux Bullseye (11)
Homebridge | v1.5.0 (running as service)
Node.js Version | v16.17.1
NPM Version | v8.13.2

Debug Files

This is an issue with npm, not with Homebridge Hue.  Best delete that directory .homebridge-hue-… manually.

I'm running a fresh install of the latest Homebridge version (with node upgraded to v16.17.1):
Getting the same error. Can't install<EMAIL_ADDRESS>

Guess again.  That version has had over 3,500 downloads this week.

And what? Let's not be cocky and over-defensive. And I'm guessing most of those are upgrades, not fresh installs.
If you work in software engineering I'm sure you know that some bugs only manifest themselves under very specific conditions.

If you work in software engineering, you should know better than guessing and collect facts instead:
$ sudo npm -g list homebridhe-hue
/usr/lib
└── (empty)

$ sudo npm -g i<EMAIL_ADDRESS>
added 110 packages, and audited 111 packages in 14s

51 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities

Not sure what "homebridhe" [sic] has to do with anything I posted.
 Very childish and immature behavior.
To anyone else reading this: Yeah can still reproduce. Fresh Homebridge image (latest), Raspberry Pi Zero W, installing 0.13.46 results in an error but 0.13.44 works.

My bad, that was a type.  I am far from infallible, but I had uninstalled Homebridge Hue just before.  To wit:
$ sudo npm -g list homebridge-hue
/usr/lib
└── (empty)

$ sudo npm -g i<EMAIL_ADDRESS>
added 110 packages, and audited 111 packages in 14s

51 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities

As I said above: this is an issue with npm, nothing to do with the Homebridge Hue code.  You're more than welcome to guess otherwise, but please don't waste my time with that.  If you want my help, rather than accusing me of rudeness, you could provide the actual error message and attach the npm log file.

It's not a random guess, but an educated guess (synonym: hypothesis). We do a lot of those at software companies.
It's based on:

A fresh install eliminates any bad state I myself might have induced
If 0.13.44 works but 0.13.46 doesn't (after repeated retries), then it points towards something with 0.13.46 even if there isn't a bug in the actual business logic or anything like that. But its a starting point for the investigation
From experience I know bumping dependencies can sometimes cause unexpected issues, often from the implicit bumps of the transitive dependencies further down the tree
Comparing the commit of 0.13.46 with 0.13.44 we can indeed see that a lot of transitive deps changed between the two versions, further strengthening the hypothesis<EMAIL_ADDRESS>My npm complains something diffuse about the googleapis package (which I noticed is a part of the transitive deps changeset as well)

I'll reproduce again and see what the logs said exactly. Another hypothesis I had was that perhaps some of the newer deps require a newer version of node, but I upgraded to v16.17.1 with similar results.

The Goolgle APIs are a dependency of the fakegato-history package; Homebridge Hue doesn't use that function.  What OS are you running on the Pi Zero W?  Did you update npm after installing / updating nodejs?


Theme overflow menu icon when theming the toolbar

$subject

Why not call colorToolbarOverflowIcon and pass toolbar instead of duplicating the logic?

Will extract the logic after I checked the code works from a lifecycle perspective 👍 (hence the push and waiting for jitpack to finish

@stefan-niedermann feel free to re-review :)


Add Portworx description

- What I did
Added documentation for Portworx volume plugin in docs.
- How I did it
Markdown
- How to verify it
Render check:  https://github.com/portworx/docker_docs/blob/master/docs/extend/legacy_plugins.md
- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "master" git@github.com:portworx/docker_docs.git somewhere
$ cd somewhere
$ git rebase -i HEAD~2
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

Should be done.  Please let me know if I missed (and what I should do instead).   Thanks

Thanks! LGTM

LGTM 🐸

Thanks!


[T6A1][W13-C2]

@liwanghuan @leahlim @bernardified
please note that when you add new features you need to update the user guide, help and corresponding tests.
Some comments added. Please ack & close the PR after reading comments.


Create greeting/on guild member join message

Greeting works. Need to update welcome command to allow this to be done via bot while the dashboard is being made.


fix(pongo.sh) docker-compose does not have -v flag any more

I don't exactly know when this change happened, but on my recently
installed docker,
$ docker -v
Docker version 20.10.8, build 3967b7d

When I execute docker-compose -v, it does not understand the -v
flag:
$ docker-compose -v
unknown shorthand flag: 'v' in -v

As a result this test in pongo always fails.
The fix should make the test pass as long as there is an executable
named docker-compose in the path, making this change forwards and
backwards compatible, hopefully.

Travis seems to be failing with
'docker-compose' command not found, please install docker-compose, and make it available in the path.

... which is exactly the problem that this PR is trying to fix (it fixed the problem on my dev machine). @Tieske any idea about how to fix this?

Tests seems to be failing though.

Closing this in favor of #199


Collectables getting abandoned

Collectable gathering is suddenly abandoning nodes instead of gathering.

See here : #252

That fixed it, thank you!


[Docs] Database preparation: Clarify why database client is installed on Gitea server

The database client program (mysql or psql) is used to test database
connection, but Gitea use Go's database driver to do the same.
Also, rename "server part" and "client part" to "server application" and
"client program" to disambiguate.

Codecov Report

Merging #12833 into master will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #12833      +/-   ##
==========================================
+ Coverage   43.15%   43.17%   +0.02%
==========================================
  Files         654      654
  Lines       72205    72205
==========================================
+ Hits        31159    31174      +15
+ Misses      35998    35980      -18
- Partials     5048     5051       +3

Impacted Files
Coverage Δ

modules/indexer/stats/queue.go
64.70% <0.00%> (-11.77%)
:arrow_down:

modules/indexer/stats/db.go
52.17% <0.00%> (-8.70%)
:arrow_down:

modules/queue/unique_queue_disk_channel.go
53.84% <0.00%> (-1.54%)
:arrow_down:

modules/git/repo.go
49.74% <0.00%> (-0.51%)
:arrow_down:

services/pull/pull.go
41.57% <0.00%> (-0.47%)
:arrow_down:

models/issue_comment.go
53.90% <0.00%> (+0.15%)
:arrow_up:

routers/repo/view.go
38.11% <0.00%> (+0.64%)
:arrow_up:

services/pull/check.go
48.46% <0.00%> (+0.76%)
:arrow_up:

services/mirror/mirror.go
19.93% <0.00%> (+1.35%)
:arrow_up:

models/unit.go
49.31% <0.00%> (+2.73%)
:arrow_up:

... and 4 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 53b6565...3b589ff. Read the comment docs.

Please resolve the conflicts.


Webhook support?

many other mods allow using webhooks so that the player's username and icon show as a discord user in discord. This functionality is the only thing keeping me from using this mod, so i'd love to see it implemented

I've seen this implemented in other mods but didn't know if it was a feature users cared much about. I'll look into adding webhook support.

It makes it much more clean and it's not super hard to do. https://discohook.com makes it pretty easy to make a template for your webhooks. If you're using an API wrapper for Discord, it probably has webhook support.

If u add support for webhooks, please make support for FabricTailor skins

Webhook support has been implemented in 1.1.0!
Unfortunately, adding support for FabricTailor skins was not practical. In general, while AMCDB strives to be compatible with other mods (i.e., it should behave in a sane way and not crash your server when other mods are installed), integrating with non-vanilla features of other mods is a completely different story and would quickly make this project un-maintainable.


NIFI-6126 - throttle requests to refresh the canvas to prevent too ma…

…ny server requests when holding down CTRL-R

Added lodash utility library to leverage its throttle capability (and many other useful functions in the future).
Made lodash available in all JSP's so it could be imported into nf-common (or any component for that matter).
Added a throttle function to nf-common that just wraps _.throttle

Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[X] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[X] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[X] Has your PR been rebased against the latest commit within the target branch (typically master)?

[X] Is your initial contribution a single, squashed commit?

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[x] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[x] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

Thanks @rfellows! This has been merged to master.


http error pass

Many times there are http responses that set 500 for not found errors. Clean this up generally.
Specifically we'll likely need to add some sort of error type that can return a status.

Fixed by #46


Add test coverage for shared code used by code generator.

It would be covered by client library tests indirectly but having explicit tests is always better.

Too vague and we've started doing it.


Update docs link for go client

Link was broken for the go client, point to correct link

 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.


Need a proper json reading library for package.json updates

Problem:
When package.json file contains a comma ',' at the end of the last key item in the file, the file is read as an empty {} object.

package.json should always be parseable by JSON.parse, but perhaps this is useful for the schema things. hjson looks promising.

hjson looks good, I will look into this. Thanks.


MINOR: Do not throw exception when InterBrokerSecurityProtocolProp value is empty string

When we set broker config as follows
security.inter.broker.protocol=
inter.broker.listener.name=INNER

and restart broker, the exception org.apache.kafka.common.config.ConfigException: Only one of inter.broker.listener.name and security.inter.broker.protocol should be set. will be throw and server start failed.
This pr will
(1) add a nonEmpty check for value of InterBrokerSecurityProtocolProp to make through empty value case
(2) detailed info When exception throw
Committer Checklist (excluded from commit message)

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

@guozhangwang @showuon PTAL


Recommendation

For those new to the field, I highly recommend checking out the MasonryFlashList by Shopify. This library offers a more efficient alternative to using ScrollView, which, while a quick solution, lacks performance. Although I haven't personally used MasonryFlashList, my experiences with Shopify's flashlist have been positive. I hope this suggestion proves helpful to you!

MasonryFlashList has a bug with android setup which is still not fixed yet… and it’s really upsetting.
https://github.com/Shopify/flash-list/issues/926


Remove Windows from list of supported platform

Windows is no more an official FDB Build. We should remove it

Assigning @khouari1 as he has already done the job on his fork and accepted to bring it back upstream :+1:
Thanks again!


LP-364 Refactoring of the PATCH implementation for e-mail

JIRA link
https://companieshouse.atlassian.net/browse/LP-364
Change description

New Patch DTO defined with JsonNullable fields
Config classes added to support mapping of Patch data to the LP Data DTO
Update audit fields added to the LP DAO
Service changes to map and save data

Failing tests commented out for now. New tests need to be added.
Work checklist

[ ] Bug fix
[ ] New feature
[ ] Breaking change
[ ] Infrastructure change

Pull request checklist

[ ] I have added unit tests for new code that I have added
[ ] I have added/updated functional tests where appropriate
[ ] The code follows our coding standards

An exhaustive list of peer review checks can be read here
Merge instructions
We are committed to keeping commit history clean, consistent and linear.  A BREAKING CHANGE can be part of commits of any type,
types other than fix: and feat: are allowed, for example build:, chore:, ci:, docs:, style:, refactor:, perf:, test:, and others,
footers other than BREAKING CHANGE: <description> may be provided.

CI: No security warnings found


Errors when trying to mock Java class that extends Comparable

ScalaMock Version
3.6.0
Scala Version
2.11
Runtime (JVM or JS)
Oracle JDK 1.8.0_181
Please describe the expected behavior of the issue
I expect to be able to mock such a class and do all the normal mock-related operations.
Please provide a description of what actually happens
I am trying to mock the FileStatus class (source code here) from the Hadoop common API Java library.  The following compiler errors result:
Error:(34, 24) double definition:
override def compareTo(o: Any): Int at line 34 and
override def compareTo(x$1: T): Int at line 34
have same type after erasure: (o: Object)Int
    val mockFile = stub[MockFileStatus]
Error:(34, 24) name clash between defined and inherited member:
def compareTo(x$1: T): Int in trait Comparable and
override def compareTo(o: Any): Int at line 34
have same type after erasure: (x$1: Object)Int
    val mockFile = stub[MockFileStatus]

The problem seems to be with the fact that this class extends Comparable without specifying any type parameters.
Any workarounds (however ugly) are welcomed.  Thank you.
Reproducible Test Case
Note: you will need to add a dependency on hadoop-common 2.6.5.
test("my failing test") {
  // your imports
  import org.apache.hadoop.fs.FileStatus
  // code that fails
  val mockFile = mock[FileStatus]
}

Please use Stackoverflow for seeking help. This place is for reporting bugs only

My apologies; I thought this was a similar case to #86, which had some discussion, but I guess it's not.  Will post to SO instead.


Support Recurrent Batch Normalization

See http://arxiv.org/abs/1603.09025. It can be built with existing building blocks, though the implementation may be a bit complicated (in particular for handling different BatchNormalization moving statistics for different time steps), and require much memory (since F.lstm cannot be used). So it is better to design new one.

:+1:

Here is a reference chainer implementation of recurrent batch normalization

bump


Add Status Field to Character Model

i.e. 'Alive', 'Deceased'.

Coverage remained the same at 100.0% when pulling dea06e8abcfd3c0fffe1f9fe5967382636ef94de on feature/17 into 22bc338ff423fb4179b01bd6ce01a67561ffc98d on master.


refactor: export OffTargetDetector

This should be in the offtarget namespace if OffTargetResult is

I tend to agree .. but can you also search/replace to fix the imports in the examples please?

Yep, done. (And did the same for #36 )


unusef link definitions are not present in parser output

Please consider following example:
# Test
[ref-link][link-id]
[shortcut]
[collapse][]

[link-id]:https://my.link/foo "my link"
[shortcut]:./shortcut
[collapse]:https://my.link/collapse
[obsolete]:https://my.link/obsolete-link

Last link definition (obsolete) is not present in the parser output in any way. In my use case I would still like to know about such links so I could surface those in some way. Maybe there could be an option for the parser which would emit events for link definitions when user opts-in ?
Even better (but more intrusive) alternative might be to make Tag::Links refer to it's definition. It could be either Inline, Autolink, Emailor a namedDefinition`. Although it does not appear to fit well current design.

Is this the same as #423?


Release new version

Hey, 7fd6d16514e9d is really important for us, could you please bump version and release it?

Yeah the thing is, what if you've set the pid-file in the config file. Then the phased-restart will try using the wrong pid (that's my guess at least).
Could you test to do a phased restart without the --pidfile option?

Well, why don't we just add the same conditional (if [ -e '#{puma_config}' ]) as in normal restart? Pumactl supports -F argument for config file so it should be done anyway.

Yeah but it was a bug in it that was fixed just a week ago here: https://github.com/puma/puma/issues/694.
We should bump the version to 0.3 and set puma version as well.

I guess we'll have to wait until puma releases a new version too. I'll create a pull request that fails for now and ping you when it's ready to merge.

Sounds great. thx


Add example with parameters

Replaces the PI example with an example that uses parameters to show that the key must be unique for each resource
Adds an extra line of explainer to the key option in the readme and common.ts

Nah, all good. I'll update the PR now. Thanks for the feedback

I've updated the PR. I removed the PI example and added the getUserById example. I also changed the URL to something that will work, just because people may want to copy it and run it to see t working.

awesome! thank you for the contribution 🎉


Feature: Flag Mails as read

If you open a mailbox in the web interface to see all its mails there is no indicator on which mails have already been read.
It would be nice if you could add such a feature by adding an attribute "read" that equals to false/true OR add another class as indicator. Using a class you could also add a css-rule that prints the Header of an unread message bold.

Seems feasible.  Would need to be added to smtpd.Message interface, implemented in in filestore, then plumbed into web UI.

Bumping to 1.3 since this would require a datastore wipe of existing installs.

Looking at the REST API, it doesn't support any other query uses, so I don't see a reason to add seen.  The message metadata is pretty small, and now contains a seen boolean.


import Wallet Bug

Hi,
I am testing Cryptonomic in dev and release also.
There are some issue when importing an existing wallet.
Issue
When user import an existing wallet and he/she put a passphrase,
validations can not capture all cases
following cases emit an error which users can not notice so they are likely to just wait
case1: user input invalid but more than 8 lengths of passphrase
case2: user input valid but incorret passphrase
where 'valid' means that a passphrase must include:
- at least 8 characters
- 1 upper case
- 1 lower case
- 1 speacial chracter
- 1 number
Suggestion

add a type of invalid passphrase for case1 in validations
(line 116 at app/reducers/walletInitialization.duck)

   const validations = [
      { value: walletLocation, type: 'locationFilled' },
      { value: `${walletLocation}/${walletFileName}`, type: 'validJS' },
      { value: password, type: 'notEmpty', name: 'Password' },
      { value: password, type: 'minLength8', name: 'Password' }
      #add this
      { value: password, type: 'invalid', name: 'Password' }
    ]

return wrongPassphrase and notify users of this, then wait user inputs again
(in TesozWallet.ts from conseiljs)
const keys = <KeyStore[]>JSON.parse(CryptoUtils.decryptMessage(encryptedKeys, passphrase, salt));

Thanks for the bug report. Could you please retest after #130 is done and we will assign this ticket to a dev if the issue still persists?

@mcwithimp Now that #130 is closed, do you still have this issue?


Support for Blade 2019 (i7 8750H, 2080MQ)

Enumerate output:
PS C:\Users\v_opo\Downloads> .\razer-cli.exe enumerate
RazerDevice { vid: 0x1532, pid: 0x023a, manufacturer: Razer, product: Razer Blade }
RazerDevice { vid: 0x1532, pid: 0x023a, manufacturer: Razer, product: Razer Blade }
RazerDevice { vid: 0x1532, pid: 0x023a, manufacturer: Razer, product: Razer Blade }
RazerDevice { vid: 0x1532, pid: 0x023a, manufacturer: Razer, product: Razer Blade }
RazerDevice { vid: 0x1532, pid: 0x023a, manufacturer: Razer, product: Razer Blade }
RazerDevice { vid: 0x1532, pid: 0x023a, manufacturer: Razer, product: Razer Blade }
RazerDevice { vid: 0x1532, pid: 0x023a, manufacturer: , product:  }
RazerDevice { vid: 0x1532, pid: 0x023a, manufacturer: Razer, product: Razer Blade }
Info output:
PS C:\Users\v_opo\Downloads> .\razer-cli.exe -p 0x023a info
Error: hidapi error: HidD_SetFeature: (0x00000001) Incorrect function.

Thanks for reporting the issue @RoastedHobnai1. Incorrect function was fixed in #2 but not yet released. If you can build HEAD and try it, that would be helpful.
Otherwise you can wait for release, I'll do it after #8 functionality is merged.

Oh, didn't think that you're still working on the project!
Compiled and tested - something's still broken:
PS C:\Users\Vladimir\Downloads> .\razer-cli.exe enumerate RazerDevice { pid: 0x023a, path: \\?\HID#VID_1532&PID_023A&MI_01&Col05#7&3a2dd8e&0&0004#{4d1e55b2-f16f-11cf-88cb-001111000030} } RazerDevice { pid: 0x023a, path: \\?\HID#VID_1532&PID_023A&MI_02#7&20346034&0&0000#{4d1e55b2-f16f-11cf-88cb-001111000030} } RazerDevice { pid: 0x023a, path: \\?\HID#VID_1532&PID_023A&MI_00#7&277a1b50&0&0000#{4d1e55b2-f16f-11cf-88cb-001111000030}\KBD } RazerDevice { pid: 0x023a, path: \\?\HID#VID_1532&PID_023A&MI_01&Col01#7&3a2dd8e&0&0000#{4d1e55b2-f16f-11cf-88cb-001111000030}\KBD } RazerDevice { pid: 0x023a, path: \\?\HID#VID_1532&PID_023A&MI_01&Col02#7&3a2dd8e&0&0001#{4d1e55b2-f16f-11cf-88cb-001111000030} } RazerDevice { pid: 0x023a, path: \\?\HID#VID_1532&PID_023A&MI_01&Col03#7&3a2dd8e&0&0002#{4d1e55b2-f16f-11cf-88cb-001111000030} } RazerDevice { pid: 0x023a, path: \\?\HID#VID_1532&PID_023A&MI_00&Col03#8&2658cb92&0&0000#{4d1e55b2-f16f-11cf-88cb-001111000030} } RazerDevice { pid: 0x023a, path: \\?\HID#VID_1532&PID_023A&MI_01&Col04#7&3a2dd8e&0&0003#{4d1e55b2-f16f-11cf-88cb-001111000030} } PS C:\Users\Vladimir\Downloads> .\razer-cli.exe -p 0x023a info Error: Failed to get performance mode and fan mode: Ok((Balanced, Auto)) Err(Command failed with unknown status: 01)
Let me know if there's anything else I can provide to help!

@RoastedHobnai1  I pushed big change #14 , it might unblock razer-cli functionality for your laptop. The output of razer-cli.exe manual --pid 0x023a info and razer-cli.exe enumerate would be very helpful!

@tdakhran , thank you for the update!
Here you go:
PS C:\Users\Vladimir\Downloads> .\razer-cli.exe enumerate
Model: RZ09-02888
Supported: false
PID: [
    0x023a,
]
PS C:\Users\Vladimir\Downloads> .\razer-cli.exe manual --pid 0x023a info
Device: Descriptor { model_number_prefix: "Unknown", name: "Unknown", pid: 570, features: ["battery-care", "lid-logo", "lights-always-on", "kbd-backlight", "fan", "perf"] }
battery-care: Err(Command not supported)
lid-logo: Ok(Off)
lights-always-on: Ok(Disable)
kbd-backlight: Ok(0)
Fan: Auto
Performance: Ok((Balanced, Auto))


Add a build step before running gramps CLI command

The gramps command points to the dist/dev/server.js file, but doesn't force a build. This is confusing. It should either run the build before starting GrAMPS, or point to the source files and use nodemon.
Using nodemon may be preferable, because it would solve the problem of watching for changes.

I think that this only affects those who are developing with a locally linked version of gramps. When you yarn install gramps it only comes with the /dist folder so there is nothing to build.

True. It’s also kind of irrelevant now that the CLI is being pulled to its own package. I’m going to close this one.


Low accuracy on WeakLensingTracer at low redshifts

The heuristic that determines the support of the lensing kernel is overly aggressive.
For the case of n(z) of the first tomographic bin of KiDS-1000, CCL determines a chi_min of ~200 Mpc. This causes >5% deviations in the Cls:

Forcing chi_min to a lower value by setting WL_tracer_fix._trc[0].chi_min = 5.0 after the creation of the tracer recovers the result from cosmosis. I originally suspected cosmosis to be the culprit but the cosmosis result was robust against a number of tests and various changes in accuracy parameters. The behaviour of CCL when manually changing chi_min behaves as expected, so I am inclined to believe that CCL is the issue here.
If CCL is indeed the culprit, we might need to look into the benchmarks for the Cls, since the deviation is much larger than the advertised accuracy of CCL. This was likely introduced sometime in the last ~2 years, as this was not an issue when I ran similar tests back then.
I will look into exposing CCL_FRAC_RELEVANT, which controls the support estimation, in ccl_cl_tracer_t_new and WeakLensingTracer.

That's weird. By default chi_min should go to zero for anything involving weak lensing. Let me look into it. What does the N(z) look like, and do you have a code snippet?

OK, I think I know what's happening. I bet the N(z) is poorly sampled (I guess on intervals Dz~0.05 or 0.1), right?
There are two solutions:
a) in your particular case, oversample the N(z) a bit.
b) In general, we can make sure CCL enforces chi_min = 0 for anything lensing related.

The n(z) is indeed sampled at dz=0.05, which is a common case though.
When searching for the kernel support, currently the comoving distance chi_min is set to where the kernel has a value greater than some fraction of the maximum. For discontinuous or sparsely sampled kernels, there is no guarantee that the volume of the kernel between chi_min and chi_min - Delta_chi is small.
I think an easy and general fix would be to take chi_min as the comoving distance before the kernel gets larger than the threshold value:
      // Find minimum
      for (ichi=0; ichi < n_w-1; ichi++) {
        if (fabs(w_w[ichi+1]) >= w_max) {
          tr->chi_min = chi_w[ichi];
          break;
        }
      }

      // Find maximum
      for (ichi=n_w-1; ichi >= 1; ichi--) {
        if (fabs(w_w[ichi-1]) >= w_max) {
          tr->chi_max = chi_w[ichi];
          break;
        }
      }

I can make a PR if that looks good to you.

Yep, that sounds good to me! Thanks a lot!

Fixed with #866 .


[Event Request] codeunit 1339 Cancel Posted Sales Cr. Memo - CreateCopyDocument OnBeforeSalesHeaderInsert

Could we get an event in C1339 in function CreateCopyDocument to be able to set some additional fields before actually inserting the new sales header. This would help a lot.
Event Declaration:
[IntegrationEvent(false, false)]
local procedure OnBeforeSalesHeaderInsert(var SalesHeader: Record "Sales Header"; var SalesCrMemoHeader: Record "Sales Cr.Memo Header")
begin
end;

Event Call:
local procedure CreateCopyDocument(var SalesCrMemoHeader: Record "Sales Cr.Memo Header"; var SalesHeader: Record "Sales Header")
var
    CopyDocMgt: Codeunit "Copy Document Mgt.";
begin
    Clear(SalesHeader);
    SalesHeader."No." := '';
    SalesHeader."Document Type" := SalesHeader."Document Type"::Invoice;

    // new event >>
    OnBeforeSalesHeaderInsert(SalesHeader,SalesCrMemoHeader)
    // new event <<

    SalesHeader.Insert(true);
    CopyDocMgt.SetPropertiesForInvoiceCorrection(false);
    CopyDocMgt.CopySalesDocForCrMemoCancelling(SalesCrMemoHeader."No.", SalesHeader);
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Update cudf dependency to 0.16-SNAPSHOT

This updates the cudf dependency to 0.16-SNAPSHOT.
Marking this as a draft for now, as it appears somehow the libcudf.so in the current cudf-0.16-SNAPSHOT says it requires librmm.so which fails the build because the jar does not contain librmm.so.  Investigating.

The librmm.so issue with cudf-0.16-SNAPSHOT has been resolved, this should be ready to go.

build

Waiting on https://github.com/rapidsai/cudf/pull/6215 to cleanup the rmm_log.txt files that are failing the CI build.

It might take a while to get the rmm_log.txt issue fully resolved, so for now I excluded it from the rat checks.  If this is acceptable as a short-term solution, I'll file a followup issue to have it removed when cudf is no longer emitting this file by default.

build


Provide code syntax hints/mappings for file extensions

Sort of inline with https://github.com/raphaelmansuy/code2prompt/issues/9 to add a feature to allow users to pair custom file extensions with specific code syntax highlighting.
Right now for my code base, I see code2prompt has a lot of wrapped code in unknown syntax highlighting
```unknown

For example, maybe there's a way to tell code2prompt that *.inc extension files have bash shell scripting so when it comes across *.inc extension files code it wraps it in
```bash

Wasn't sure about how that would be done so asked ChatGPT4o 🤓

Define a Command Line Argument for Mappings: Introduce a new command line argument, such as --syntax-map, which takes a list of mappings.
Specify Mappings in a Clear Format: Use a simple and clear format for specifying each mapping, for example, extension:syntax.
Handle Multiple Mappings: Allow users to specify multiple mappings separated by a comma or provide multiple --syntax-map arguments.

code2prompt --syntax-map "inc:bash,customext:python,ext2:javascript"

Added in the roadmap.

Will be released soon

I see the readme has this updated usage example but --help for 0.8.1 doesn't show it from pipx install?


Update lua version in makefiles

Hi! I am trying to build with Makefiles and it fails because the Lua version is not syncronized with external/lua-*.

Thanks a lot!


calcValue in hare beacon uses fnv

hare/eligibility/beacon.go
The func calcValue (of the beacon) uses fnv instead of our agreed (low collision hash).
To my understanding, we still haven't agreed on such hash.
We use Hash32 for blocks and we also have Hash12. We should decide what we want to use for the beacon.
@barakshani any input on that?

Will handle as part of work on self-healing

This is partially addressed in #2394 but other usages still exist:
cmd/hare/hasher.go
4:      "hash/fnv"

hare/mock_oracle.go
8:      "hash/fnv"

hare/notifytracker.go
5:      "hash/fnv"

hare/haretypes.go
8:      "hash/fnv"

eligibility/fixedoracle.go
7:      "hash/fnv"


[contract] can't pause production

'u128_sub Overflow')
happening on both of my resource productions.

@credence0x would you know where it comes from ?

this issue is killing me


4x: Starting to flesh out SSVM integration

Services offered

CommonVirtualService - Housing the shared VM instance
VirtualInvoker - Invoking method in the shared VM instance and reporting back what the return values were
VirtualOptimizer - Invoking methods with optimzer passes and reporting back an optimized version of methods where possible

Codecov Report
Attention: 241 lines in your changes are missing coverage. Please review.

Comparison is base (6a72a30) 66.75% compared to head (34f3d69) 66.08%.

Files
Patch %
Lines

.../coley/recaf/services/vm/CommonVirtualService.java
41.55%
76 Missing and 14 partials :warning:

...ley/recaf/util/io/BufferDelegatingPrintStream.java
17.00%
82 Missing and 1 partial :warning:

...tware/coley/recaf/services/vm/ArgumentBuilder.java
41.17%
56 Missing and 4 partials :warning:

.../java/software/coley/recaf/services/vm/VmTest.java
87.23%
6 Missing :warning:

...ftware/coley/recaf/services/vm/VirtualInvoker.java
90.00%
2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##               dev4     #721      +/-   ##
============================================
- Coverage     66.75%   66.08%   -0.67%
- Complexity     2240     2284      +44
============================================
  Files           305      311       +6
  Lines         13002    13453     +451
  Branches       1481     1517      +36
============================================
+ Hits           8679     8891     +212
- Misses         3644     3863     +219
- Partials        679      699      +20

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


Regenerate w/o static ResourcesFactory

creates resources with the URI like http://localhost:8803/resource/B000GWGJK2 but should be http://localhost:8803/services/resource/B000GWGJK2
the factory is initialized with new ResourcesFactory(OSLC4JUtils.getPublicURI());

Also, resolveUri does not use the configured getServletPath but the request path:

obsolete due to #153


Bare NCurses usage howto ?

Hello,
I am finding myself attempting to implement some terminal interactive (text-based) input in julia, and I finally found this NCurses.jl wrapping, that seems to work on my machine (tictactoe display something), unlike my first attempt at directly ccalling Ncurses_jll routines...
So I d like to know what would be the best way to get the very basic / lowlevel behavior of Ncurses in julia, without all the widgets added on top. something alike python's curses
Would splitting the NCurses in another repo/package wrapping make sense ?
Can I directly use NCurses from this package, but how ? Is there a tutorial/example somewhere ?
Thanks for the help. I have long experience in programming, but just getting my feet wet with Julia...
And thanks for the work on that ncurses wrapper, which could help me a lot !

Hi @asmodehn !
This package is under a huge rewrite. However, the submodule NCurses.jl is very stable.
You can do something like:
julia> using TextUserInterfaces

julia> using TextUserInterfaces.NCurses

And now you can call the NCurses functions that are registered. Take a look here:
https://github.com/ronisbr/TextUserInterfaces.jl/blob/master/src/submodules/NCurses/ncurses_functions.jl

In the package version I have (0.0.1) :
julia> using TextUserInterfaces

julia> using TextUserInterfaces.NCurses
ERROR: UndefVarError: NCurses not defined

Also it seems initscr() is available from the lower module:
using TextUserInterfaces

stdscr = initscr()

gives me:
ERROR: LoadError: ArgumentError: NULL library handle
Stacktrace:
 [1] #dlsym#1
   @ ./libdl.jl:57 [inlined]
 [2] dlsym
   @ ./libdl.jl:56 [inlined]
 [3] initscr()
   @ TextUserInterfaces ~/.julia/packages/TextUserInterfaces/k1unl/src/ncurses/ncurses_functions.jl:132
 [4] top-level scope
   @ /opt/Projects/kittyforth/tui.jl:9

Which breings me to @eval $f($(argsj...)) = @_ccalln $f($(argsc...))::$r which is not really helpful...
Any clue ?

You need to use the development version! Type:
julia>]dev TerminalUserInterfaces

I didn't have much luck with that either:
(kittyforth) pkg> dev TerminalUserInterfaces
     Cloning git-repo `https://github.com/kdheepak/TerminalUserInterfaces.jl.git`

signal (11): Segmentation fault
in expression starting at none:0
unknown function (ip: 0x7fb1493ff008)
Allocations: 5726597 (Pool: 5722308; Big: 4289); GC: 7
Segmentation fault (core dumped)

or
(kittyforth) pkg> dev TextUserInterfaces
     Cloning git-repo `https://github.com/ronisbr/TextUserInterfaces.jl.git`

signal (11): Segmentation fault
in expression starting at none:0
unknown function (ip: 0x7ff776dff008)
Allocations: 4890650 (Pool: 4886749; Big: 3901); GC: 6
Segmentation fault (core dumped)

I m thinking it might be because of the main <-> master branch naming...
So I did a manual clone in the proper place (need to be careful with that .jl suffix).
Then I found out how to use a the dev version of a package from a script.
So now I am back to:
tui.jl
import Pkg
Pkg.develop("TextUserInterfaces")

using TextUserInterfaces
using TextUserInterfaces.NCurses

stdscr = initscr()

julia --project tui.jl
   Resolving package versions...
  No Changes to `/opt/Projects/kittyforth/Project.toml`
  No Changes to `/opt/Projects/kittyforth/Manifest.toml`
ERROR: LoadError: ArgumentError: NULL library handle
Stacktrace:
 [1] #dlsym#1
   @ ./libdl.jl:57 [inlined]
 [2] dlsym
   @ ./libdl.jl:56 [inlined]
 [3] initscr()
   @ TextUserInterfaces.NCurses ~/.julia/dev/TextUserInterfaces/src/submodules/NCurses/ncurses_functions.jl:328
 [4] top-level scope
   @ /opt/Projects/kittyforth/tui.jl:8
in expression starting at /opt/Projects/kittyforth/tui.jl:8

Looking at init_tui, I guess I miss a call to Lib.dlopen, I ll try to figure it out...
I just wish using NCurses.jl would be easier for beginners.
Let me know if I can help in any way.

Ok This works for me now, (with a caveat on the default git branch used by the repo maybe ?)
import Pkg
Pkg.develop("TextUserInterfaces")

using TextUserInterfaces
using TextUserInterfaces.NCurses

load_ncurses()
stdscr = initscr()
clear()
printw("Hello World !")
refresh()
getch()
endwin()

I'll start from there, thanks for all the help.
If you ever want to extract NCurses in its own package, ping me and I ll give you a hand.
Cheers !

I am glad it worked!

This is a package under heavy development. I am lacking time however to finish everything. That's why I did not tag a new version.


GraphQL Subscriptions generate an 'Internal Server Error'

I am trying to get a GraphQL subscription working with Java/Vert.x and to have the results shown in GraphiQL. I see all the System.out.println statements in the console, but GraphiQL is not displaying any results because the server is generating an 'Internal Server Error' message. I am using vert.x 3.8.0 jars
Schema:
type Subscription {
    test: String
}

Vert.x Verticle
private RuntimeWiring getRuntimeWiring() {
  return new RuntimeWiring()
    .type("Subscription", builder -> builder
      .dataFetcher("test", getTestDataFetcher()))
    .build();
}

private VertxDataFetcher<Publisher<String>> getTestDataFetcher() {
  return new VertxDataFetcher<>((env, future) -> future.complete(doTest()));
}

private Publisher<String> doTest() {
  AtomicReference<Subscription> ar = new AtomicReference<>();
  Observable<String> obs = Observable.just("Hello");
  Publisher<String> pub = obs.toFlowable(BackpressureStrategy.BUFFER);
  pub.subscribe(new Subscriber<String>() {
    @Override
    public void onSubscribe(Subscription s) {
      System.out.println("SUBSCRIBE");
      ar.set(s);
      s.request(1);
    }
    @Override
    public void onNext(String s) {
      System.out.println("NEXT="+s);
      ar.get().request(1);
    }
    @Override
    public void onError(Throwable t) {
      System.out.println("ERROR");
    }
    @Override
    public void onComplete(){
      System.out.println("COMPLETE");
    }
  }
  return pub;
}

If I run the subscription using GraphiQL and look on my vert.x servers console, the output on the console is:
SUBSCRIBE
NEXT=Hello
COMPLETE

The GraphiQL output window says "Internal Server Error" and is sent a 500 error code from the server
If I modify the DataFetcher to exactly what is shown at the bottom of the first link, I also receive "Internal Server Error".
private DataFetcher<Publisher<String>> getTestDataFetcher() {
  return env -> doTest();
}

I do not see any stack traces for the 500 error in the vertx console.  So maybe this is a bug?
Sidenote  - If I try using a CompletionStage as shown below (based off the bottom of the 2nd link) I get an error message saying 'You data fetcher must return a publisher of events when using graphql subscriptions'
private DataFetcher<CompletionStage<String>> getTestDataFetcher() {
  Single<String> single = Single.create(emitter -> {
    new Thread(()-> {
      try {
        emitter.onSuccess("Hello");
      } catch(Exception e) {
        emitter.onError(e);
      }
    }).start();
  )};
  return environment -> single.to(SingleInterop.get());
}

I have used the following sources as references to get this far:

https://www.graphql-java.com/documentation/v9/subscriptions/
https://vertx.io/docs/vertx-web-graphql/java/

Duplicates https://github.com/vert-x3/vertx-web/issues/1334


Updates to DR workflow github-actions

Context
Need to update service workflows after updates to related github-actions
Changes proposed in this pull request
Updates to DR workflows, backup db, restores, maintenance
to be merged after github-actions pr is merged
Guidance to review
Run against updated github-actions branch using the staging env

backup: https://github.com/DFE-Digital/itt-mentor-services/actions/runs/11328445134
ptr: https://github.com/DFE-Digital/itt-mentor-services/actions/runs/11328145369
ptr backup: https://github.com/DFE-Digital/itt-mentor-services/actions/runs/11328519109

Link to Trello card
https://trello.com/c/JPtELArk/2082-template-github-action-workflows

 Template Github action workflows


Fixed: now settings are brought to front only the first time they are…

… clicked

Thanks @rtugores!


needs client install scripts

We need to create a install script to automate the process of installation for clients. I think it needs to be written in python for now. this will allow cross platform support as apposed to bash. However consideration should be made if we add C++ client support.

To do list:
make sure python is installed.
make sure pip is installed
set up virtualenv or conda?
client:
cp client.py to proper location
cp the public key to where client.py is looking for it
cp the systemd config files to /lib/systemd/system/
update systemd and enable the service
server:
cp server.py to right location
generate the RSA keys
set up systemd service


fix room-leave room_id issue

fix bug room-leave EventRoomLeavePayload with issue

For Error Consider using Python 3 style super() without arguments (super-with-arguments):
Refer to :

super in python3
Understanding Python super() with init() methods [duplicate]

I think you should change the style of calling super __init__ method to :
    def __init__(self, options: PuppetOptions, name: str = 'hostie_puppet'):
        super().__init__(options, name)

You can try it.

For Error Consider explicitly re-raising using the 'from' keyword (raise-missing-from):
Refer to:

python3 built-in-exceptions
Python “raise from” usage

So you should raise WechatyPuppetPayloadError  from origin_exception

Waiting for your new commits.

The last step : I think you should update the VERSION file to deploy the package.


Cant fetch table/view list - "Error getting db service(s)"

I have Apigility deployed on HostGator, v1.4.1.
<?php
return [
    'db' => [
        'adapters' => [
            'db1@mysql@localhost' => [
                'database' => 'db1',
                'driver' => 'PDO_Mysql',
                'hostname' => 'localhost',
                'username' => 'db1',
                'password' => 'pass',
                'dsn' => 'mysql:dbname=db1;host=localhost',
            ],
        ],
    ],
];

With given parameters, I am able to connect to database using mysql shell client. However, when trying to create DB connected service, I always get "Error getting db service(s)" error, and cant figure why.

This repository has been closed and moved to laminas-api-tools/api-tools-admin; a new issue has been opened at https://github.com/laminas-api-tools/api-tools-admin/issues/10.


Populate external labels for annotations expansion

Attempt to fix https://github.com/prometheus/prometheus/issues/3043

As noted in #3043, this is by design. In addition it'd be weird if external_labels were applied twice, and also weird if they ended up in ALERTS.

external_labels were applied twice

@brian-brazil what do you mean ?

Is there anything I can do to help push this change along?

@sylr do you think you'll work on this in the foreseeable future?
Otherwise, this is just up for grabs for whoever feels qualified to do it.

First step is really only introducing .ExternalLabels. The other points I listed don't have to be in this PR.

I'll take a look this afternoon.

@sylr you can rebase on master to pass the CI.

@beorn7 PTAL

This doesn't cover console templates.

No it does not, to be honnest I didn't even know about it.

I think the CI failure was due to not being able to download staticcheck. Re-running…

:tada: Build is green.
@brian-brazil is there anything left to improve in this PR?

I think the CI failure was due to not being able to download staticcheck. Re-running…

That happens a bit unfortunately.

I'm still not seeing console templates covered.

Can you give me pointers ?

Look for ExpandHTML.

Looks like the build is green. Can this make it's way into 2.8?

I don't see this making it into 2.8, the code is still not right.

Sorry for not following this review in the necessary depth to figure out why this is getting stale.
@sylr are you aware what makes @brian-brazil think this is not quite there yet? Or alternatively, @brian-brazil , could you explain in more detail?

@beorn7 I believe @brian-brazil does not like the config being accessed through a global var.

Do you think it's a reasonable concern? Do you know how to address it?
(Just trying to unblock you two here.)

I don't know, more experienced go developer should answer that. What I know is that it provided an easy enough solution to implement that did not require to touch the current configuration reloading system which seems complex to me.

I'm expecting the current system to be used as it's our standard way of doing things, and is better for those reusing our code.

What could be done to make it prettier is to:

unexport config.CurrentConfig and add the needed function to config to set and get the current config
move the config.CurrentConfigMutex inside the Config struct and add Public method to Lock/Unlock.

RuleManager is already hooked into config updated, extend what's there.

So reverting this: https://github.com/prometheus/prometheus/commit/bd9f7460eb6fac35a103036d8fc5cd63c42ddd63
cc @fabxc

You don't have to revert that.

Hey ya'll.
Is there anything we can do to help push this feature over the line in time for Prometheus 2.9.0?

I hoped @sylr and @brian-brazil could shepherd this into a merge on their own. I would dive in here and help, but I am very short on time right now. Next week hopefully will be better...

I finally got some time to look into this. After I found myself writing a lot of code changes myself, I decided to create #5463 on top of this. Please have a look there.

Superseded by #5463.


Charm shows as active/idle even when raft configuration is incorrect

Describe the bug
As a follow-up to #88 -- when this happens there isn't an immediately obvious way to detect that something is wrong.
The charm will remain in active/idle even when raft has reported that it is unhealthy. There should probably be some sort of warning in this case.
Furthermore, it's not obvious what triggers raft to become unhealthy. Even without the bugfix in #88, when I tested manually after the new pod rejoined raft, everything appeared to be healthy. Raft only reached an unhealthy state once I removed the leader unit.
To Reproduce
Deploy an old version of the charm, before the bugfix in 88.
juju add-model raft-cluster-bug
juju deploy vault-k8s --channel edge -n 5 --trust --revision 53

Wait for everything to be up and running. Then, configure vault to connect to your vault cluster directly.
Install vault and yq if not already installed
sudo apt install vault yq

# Set vault token
vault_initialization_secret=$(juju secrets --format=yaml | yq '. | with_entries(select(.*.label == "vault-initialization")) | keys | .[]')
export VAULT_TOKEN=$(juju show-secret ${vault_initialization_secret} --reveal | yq '.[].content.roottoken')

# Extract the vault CA cert and set the path
vault_ca_certificate_secret=$(juju secrets --format=yaml | yq '. | with_entries(select(.*.label == "vault-ca-certificate")) | keys | .[]')
juju show-secret ${vault_ca_certificate_secret} --reveal | yq '.[].content.certificate' > vault_ca.pem
export VAULT_CAPATH=$PWD/vault_ca.pem

# Set the address of vault
export VAULT_ADDR="https://$(juju status vault-k8s --format=yaml | yq '.applications.vault-k8s.address'):8200"

Check the raft health
vault operator raft autopilot state

$ vault operator raft autopilot state
Healthy:                      true
Failure Tolerance:            2
Leader:                       raft-cluster-bug-vault-k8s/0
Voters:
   raft-cluster-bug-vault-k8s/0
   raft-cluster-bug-vault-k8s/1
   raft-cluster-bug-vault-k8s/2
   raft-cluster-bug-vault-k8s/4
   raft-cluster-bug-vault-k8s/3
Servers:
   raft-cluster-bug-vault-k8s/0
      Name:            raft-cluster-bug-vault-k8s/0
      Address:         <IP_ADDRESS>:8201
      Status:          leader
      Node Status:     alive
      Healthy:         true
      Last Contact:    0s
      Last Term:       3
      Last Index:      72
   raft-cluster-bug-vault-k8s/1
      Name:            raft-cluster-bug-vault-k8s/1
      Address:         <IP_ADDRESS>:8201
      Status:          voter
      Node Status:     alive
      Healthy:         true
      Last Contact:    494.230733ms
      Last Term:       3
      Last Index:      72
   raft-cluster-bug-vault-k8s/2
      Name:            raft-cluster-bug-vault-k8s/2
      Address:         <IP_ADDRESS>:8201
      Status:          voter
      Node Status:     alive
      Healthy:         true
      Last Contact:    99.573454ms
      Last Term:       3
      Last Index:      72
   raft-cluster-bug-vault-k8s/3
      Name:            raft-cluster-bug-vault-k8s/3
      Address:         <IP_ADDRESS>:8201
      Status:          voter
      Node Status:     alive
      Healthy:         true
      Last Contact:    4.714894438s
      Last Term:       3
      Last Index:      72
   raft-cluster-bug-vault-k8s/4
      Name:            raft-cluster-bug-vault-k8s/4
      Address:         <IP_ADDRESS>:8201
      Status:          voter
      Node Status:     alive
      Healthy:         true
      Last Contact:    2.488287328s
      Last Term:       3
      Last Index:      72

Delete a pod
kubectl delete pod vault-k8s-1 -n raft-cluster-bug

Wait for everything to settle back to active/idle.

But, raft itself seems to report everything as healthy at first. It's not obvious how we can detect the health of the raft cluster.

Environment

Charm / library version (if relevant): 53 for reproducing, but the general issue applies beyond that.
Juju version (output from juju --version): 3.1.6
Cloud Environment: microk8s
Kubernetes version (output from kubectl version --short):

Client Version: v1.27.7
Kustomize Version: v5.0.1
Server Version: v1.27.7

Additional context

As already mentioned in the ticket, it isn't possible to reproduce the issue with the same steps after the fix in #89 .
The charm after the fix uses hostnames instead of IP addresses so there will be no need to update IP addresses or Raft nodes. And based on a few tickets reported we see that there is a known issue with Raft nodes updating the IP addresses if the pod crashes and comes up again with a new IP.
Example issues:

https://github.com/hashicorp/vault/issues/10702#issuecomment-769076069
https://discuss.hashicorp.com/t/understanding-node-ip-changes-for-a-raft-cluster/56854
https://discuss.hashicorp.com/t/change-vault-cluster-node-ip/14923/1
https://github.com/hashicorp/vault/issues/17014

Based on fix #89 Raft cluster should not report unhealthy because of an IP change, however we still need to get a feedback if the cluster reports unhealthy for any other reason.
In general the cluster will report unhealthy if at least one of its nodes is reporting unhealthy and a node will report unhealthy it it can't heartbeat the leader unit or it can't keep up with the data replication from leader node (reference here)
PR #117 adds logging to report any node going unhealthy.

PR #117 has been merged.


Getting "FILE LOAD ERROR"

Description
Hi. I'm experiencing an issue: the client won't load the resources. I can't interact with the buttons without getting FILE LOAD ERROR. I can't even see the monster art in the main page.
Screenshots

What I did
I downloaded the compressed folder, extracted it and moved the Build folder in the C:\Program Files (x86)\Steam\steamapps\common\Yu-Gi-Oh! Master Duel\ [TREE] directory.
I also tried following every single step of FileLoadError.md, but it didn't work at all.
Server logs
Initialized
Req System.info
Req Account.Steam.get_user_id
Unhandled act Account.Steam.get_user_id
Req Account.auth
Req Billing.product_list
Unhandled act Billing.product_list
Req User.entry
Req User.home
Req EventNotify.get_list
Req Shop.get_list
Req System.info
Req Account.Steam.get_user_id
Unhandled act Account.Steam.get_user_id
Req Account.auth
Req Billing.product_list
Unhandled act Billing.product_list
Req User.entry
Req User.home
Req EventNotify.get_list
Req System.info
Req Account.Steam.get_user_id
Unhandled act Account.Steam.get_user_id
Req Account.auth
Req Billing.product_list
Unhandled act Billing.product_list
Req User.entry
Req User.home
Req EventNotify.get_list
Req System.info
Req Account.Steam.get_user_id
Unhandled act Account.Steam.get_user_id
Req Account.auth
Req Billing.product_list
Unhandled act Billing.product_list
Req User.entry
Req User.home
Req EventNotify.get_list
Req EventNotify.get_list
Req EventNotify.get_list
Req EventNotify.get_list
Req EventNotify.get_list
Req EventNotify.get_list
Req EventNotify.get_list
Req EventNotify.get_list

The tree is correct. Assuming you've fully downloaded the game I'm not sure what's wrong unfortunately. Your Yu-Gi-Oh! Master Duel folder is roughly 5-6 GB in size, right?

Oh you have several sub folders in LocalData. The code will pick the fist one it finds which may be empty. Delete the unused LocalData folders.

It works now, thanks. It pisses me off the fact that I have to move the other account's folders away, but I can deal with it.

The code could be improved to check for a specific file and skip empty folders. But I don't have plans at the movement to implement this at the moment.
https://github.com/pixeltris/YgoMaster/blob/7b5e3ecb1a1afd9d7d2cd8a31a747816f87a32d7/YgoMasterClient/Program.cs#L917-L951

Here we are, 2024. 1.49.
Same problem, and I deleted the useless folders.
Here is my tree, of someone have an idea.
Initialized
Req System.info
Req Account.Steam.get_user_id
Unhandled act Account.Steam.get_user_id
Req Account.auth
Req Billing.product_list
Unhandled act Billing.product_list
Req User.entry
Req User.home
Req EventNotify.get_list


Add printf and fflush

Following up on https://github.com/numba/numba/issues/5679. This PR implements printf and fflush for jitted functions.
To-Do

[x] Fix %p specifier when JIT is disabled.
[x] Fix char/string print when JIT is enabled.
[x] Add tests
[x] Add docs

Obs: fflush function type is compiler/machine dependent since it depends on the FILE* stuct. But since the implementation calls it as fflush(NULL), one can bypass the LLVM typing system by just declaring it as a int32 fflush(void*)

Pull request is ready for a review. Not sure how to test it since I couldn't capture the output.
ps: flake8 is failing in files not related to this PR.

Pull request is ready for a review. Not sure how to test it since I couldn't capture the output.

For capturing the output, try using the method from https://stackoverflow.com/questions/24277488/in-python-how-to-capture-the-stdout-from-a-c-shared-library-to-a-variable

Thanks for the help @pearu

The test that I wrote hangs on CI. So, I will just comment it for now.

@guilhermeleobas I reviewed the documentation from a user point of view, and also looked at the tests. Unfortunately, I know next to nothing about @intrinsic so I cannot tell if your implementation is correct. sorry!

Thanks for the review @luk-f-a

@pearu

Could you enable these tests for the local run?
Is the hang platform-dependent?

I think the test crashes only in specific machines on CI. See:

I will try to enable this test only for macOS and Windows.

Not a single test executed.

Every single test stuck on test discovery phrase. I think this is related to changing the file descriptor at the beginning of the test file. I will try to address this in a near future.

No, it never made to Numba main branch. It was hard to get the tests working on CI.

That's a shame, it was a great addition to numba.
Thanks for the effort you put into it!


Type annotations and ligatures

Type definitions can trigger ligatures that exists in many popular code fonts:
let foo = (~a: option<array<int>>=?) => ...

Not sure if this is a huge concern, but perhaps we can format in a way that doesn't trigger the ligature and is also more pleasant on the eyes?
let foo = (~a: option<array<int>> = ?) => ...

Thanks. This is a VSCode bug. Merged into https://github.com/rescript-lang/rescript-vscode/issues/35