wasm-opt fail with [parse exception: invalid code after misc prefix: 17 (at 0:61491)]

🐛 Bug description
Our build was working up to rust 1.81.0.
With Rustc 1.82.0, I am getting an error with wasm-opt.
🤔 Expected Behavior
wasm-opt should work with rustc 1.82.0
👟 Steps to reproduce
 wasm-pack build --release --no-typescript --target nodejs --out-dir ../_build/wasm/node
[INFO]: Checking for the Wasm target...
info: downloading component 'rust-std' for 'wasm32-unknown-unknown'
info: installing component 'rust-std' for 'wasm32-unknown-unknown'
 18.7 MiB /  18.7 MiB (100 %)  13.8 MiB/s in  1s ETA:  0s
[INFO]: Compiling to Wasm...
    Finished `release` profile [optimized] target(s) in 0.08s
[INFO]: Installing wasm-bindgen...
[INFO]: found wasm-opt at "/usr/bin/wasm-opt"
[INFO]: Optimizing wasm binaries with `wasm-opt`...
[parse exception: invalid code after misc prefix: 17 (at 0:61491)]
Fatal: error parsing wasm
Error: failed to execute `wasm-opt`: exited with exit status: 1
  full command: "/usr/bin/wasm-opt" "/home/ubuntu/symbol/sdk/javascript/_build/wasm/node/symbol_crypto_wasm_bg.wasm" "-o" "/home/ubuntu/symbol/sdk/javascript/_build/wasm/node/symbol_crypto_wasm_bg.wasm-opt.wasm" "-O"
To disable `wasm-opt`, add `wasm-opt = false` to your package metadata in your `Cargo.toml`.
Caused by: failed to execute `wasm-opt`: exited with exit status: 1
  full command: "/usr/bin/wasm-opt" "/home/ubuntu/symbol/sdk/javascript/_build/wasm/node/symbol_crypto_wasm_bg.wasm" "-o" "/home/ubuntu/symbol/sdk/javascript/_build/wasm/node/symbol_crypto_wasm_bg.wasm-opt.wasm" "-O"
To disable `wasm-opt`, add `wasm-opt = false` to your package metadata in your `Cargo.toml`.

🌍 Your environment
Include the relevant details of your environment.
Ubuntu 24.04(it works on MacOs)
wasm-pack version:
wasm-pack -V
wasm-pack 0.13.1
rustc version:
rustc -V
rustc 1.82.0 (f6e511eec 2024-10-15)

The code is here - https://github.com/symbol/symbol/tree/dev/sdk/javascript/wasm

have the same problem

When wasm-bindgen 0.2.94-95 sees the wasm target-features metadata emitted by rustc 1.82.0, it enables additional webassembly features on its output (bulk memory instructions) that can cause issues with downstream tools like wasm-opt. This will be fixed in the next release of wasm-bindgen. Until then, workarounds include adding --enable-bulk-memory to the wasm-opt commandline (if your runtime is compatible with bulk-memory instructions), not using wasm-opt at all, or downgrading to rust 1.81 or to wasm-bindgen 0.2.93.
Related discussions:
https://github.com/rustwasm/wasm-bindgen/issues/4250
https://github.com/rustwasm/wasm-bindgen/issues/4211
https://github.com/rust-lang/rust/issues/132620
https://github.com/trunk-rs/trunk/issues/904


new implementatation of parameter combination

I now think that the parameter combination should work. Please check for errors

I'm not shure with "Add to all variables and functions (return-)types. Makes it much easier to read and understand." Didn't you change already the function headers? Or what do you mean?
Yes theres a mathematical approach. The number of lists should be the product over the longest value_pool lists. The amount of the longest lists in the calculation is determined by param_combi.

I'm not shure with "Add to all variables and functions (return-)types. Makes it much easier to read and understand." Didn't you change already the function headers? Or what do you mean? Yes theres a mathematical approach. The number of lists should be the product over the longest value_pool lists. The amount of the longest lists in the calculation is determined by param_combi.

I added missing types.

@jonathanheitzmann @dsiev What is the current status? Have you found any new bugs?

yes and I just fixed the bug. Should I merge?

yes and I just fixed the bug. Should I merge?

If it works, there you go!

@jonathanheitzmann Is there anything left to do? If not please delete branch.


Model reparameterization changing outcome probabilities

calling set_all_parameterizations on a full TP model to a GLND model is changing some outcome probabilities non-trivially.
To reproduce:
`from pygsti.modelpacks import smq1Q_XY as std
datagen_model = std.target_model("GLND")
#Arbitrary error where I observed the problem
error_vec = [0] * 48
error_vec[0] = .01
datagen_model.from_vector(error_vec)
design = std.create_gst_experiment_design(16)
#Circuit with the maximum difference
bad_circuit = design.all_circuits_needing_data[394]
datagen_model_copy = datagen_model.copy()
datagen_model_copy.set_all_parameterizations("full TP")
datagen_model_copy.set_all_parameterizations("GLND", ideal_model=std.target_model("GLND"))
datagen_model.probabilities(bad_circuit)['0'] - datagen_model_copy.probabilities(bad_circuit)['0']`
Expected behavior
The code above outputs a probability difference of -1.406968064276981e-08. This is a substantial difference causing issues in my current project which requires comparison of gauge-equivalent models.
Environment:

pyGSTi version <IP_ADDRESS>pyth
python version  3.10.14
macOS Sonoma 14.4.1

Additional context
After an email exchange with Riley and Corey, Riley identified the problem in the state preparation. One of the vector entries deviates by 2.7057608985464707e-08 after conversion. This makes sense considering the model only has errors in the state preparation.
I believe I have identified the issue being in pygsti/modelmembers/states/__init__py  line 269. This scipy optimization returns the exact number above "2.7057608985464707e-08" as the error. I tried changing the tolerance of the optimization, but this did not seem to change its behavior.

Update on this issue: I identified a second bug in this function. set_all_parameterization does not function properly either with errors on measurements. The error channels and provided ideal models are not used properly. The returned model always has no measurement error, and the ideal measurement is instead the noisy measurement from the original model. @coreyostrove and I are currently working on fixing these.
Going back to the original state preparation problem, it seems like changing the optimization "method" parameter to "Nelder-Mead" solves the problem, although I don't know if using a different optimization algorithm will cause other problems.


Support for OneTouch Ultra2

We've had the Ultra2 driver in the codebase for a while now, but it has always been disabled since it didn't have proper error handling and didn't adhere to the UTC bootstrapping time format.
This PR adds:

TZOUtil across-the-the-board time
units from meter, not hard-coded
BGM checklist
skipping control solution tests
annotating HI/LOs

 OneTouch Ultra 2 and OneTouch Mini

You're right, line 356-358 should not be there.

The two new commits look good to me.
From my perspective of a visual code review, this PR is ready to be merged.
Should I approve it, or leave that to someone that has actually tested it against the device?

You can approve. I will then merge it into master and create a new version of the dev uploader for Eric or Nick to test before we push to production.

@pazaan Dorian has already reviewed the driver, but I have made some additional changes in commit b03ba87 after I discovered that sometimes it only uploaded around 12 out of 20 records. Can you please review the additional changes? (I also need someone with write access to approve)


Update skafka to 11.1.0

Updates com.evolutiongaming:skafka from 11.0.0 to 11.1.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.evolutiongaming", artifactId = "skafka" } ]

labels: library-update, semver-minor

Superseded by #164.


upgrade CCIP to 1.5

Update the version of @chainlink/contracts-ccip to 1.5
Update the version of @chainlink/local to 0.2.2
Because getSupportedTokens is removed from Router contract (if tx is invalid, it will be reverted when called), supportedTokens check is removed in the tasks: ccip-toekn-transfer, ccip-token-transfer-batch, send-token-and-data
Typo fix in README
In README example 6 step 2, update the default amount of Sepolia LINK funded to the sender contract from 0.001 to 1 since 0.001 is not enough for CCIP.
In README example 7 step 3, update the default amount of AVAX funded to the sender contract from 0.1 to 1 since 0.1 is not sufficient to make the transaction successful.

EVM and compiler versions are updated to paris and 0.8.24


Update TextBlock.php

Fix Multi Line Text reverse in Arabic And Persian Text Bug If Text Is Arabic Or Persian Write Lines True

Closing due to inactivity.


Storage patch is not thread-safe

This should only happen in development environments or other cases where MinIO is used in a Docker network.
In [1]: from rgd.geodata.models import ChecksumFile
In [2]: from rgd.utility import patch_internal_presign
In [3]: f1 = ChecksumFile.objects.first()
In [4]: f2 = ChecksumFile.objects.first()
In [5]: with patch_internal_presign(f1.file):
   ...:     assert f1.file.storage.base_url != f2.file.storage.base_url
   ...:
---------------------------------------------------------------------------
AssertionError                            Traceback (most recent call last)
<ipython-input-5-37e9d3b557e8> in <module>
      1 with patch_internal_presign(f1.file):
----> 2     assert f1.file.storage.base_url != f2.file.storage.base_url
      3

@mcovalt, could you make a test case for this and mark it as xfail?

I believe the Gobal Interpreter Lock will prevent another thread seeing the modified/patched state.


Question about PubMed performance

Hi,
In your paper, GRACE achieves 86.7% in Pubmed, and DGI achieves 86% in Pubmed. However, in the DGI paper, the performance of DGI only achieves 76.8% in Pubmed. I also notice you follow the DGI setting in your experiments. How did you make an improvement of almost 10% on DGI?

The dataset spilt setting is different. DGI uses public spilt while GRACE uses 10% train, 10% valid, and 80% test random spilt.

The dataset spilt setting is different. DGI uses public spilt while GRACE uses 10% train, 10% valid, and 80% test random spilt.

GRACE  train/test split follows a 90% / 10% .  Isn't that it?

In paper,

On these citation networks, we randomly select 10% of the nodes as the training set, 10% nodes as the validation set, and leave the rest nodes as the test set.

train/valid/test split follows a 10%/10%/80%.
And in code, train/test split follows a 10%/90%.
https://github.com/CRIPAC-DIG/GRACE/blob/master/train.py#L41C4
https://github.com/CRIPAC-DIG/GRACE/blob/master/eval.py#L58

The dataset spilt setting is different. DGI uses public spilt while GRACE uses 10% train, 10% valid, and 80% test random spilt.

GRACE train/test split follows a 90% / 10% . Isn't that it?

In paper,

On these citation networks, we randomly select 10% of the nodes as the training set, 10% nodes as the validation set, and leave the rest nodes as the test set.

train/valid/test split follows a 10%/10%/80%.
And in code, train/test split follows a 10%/90%.
https://github.com/CRIPAC-DIG/GRACE/blob/master/train.py#L41C4
https://github.com/CRIPAC-DIG/GRACE/blob/master/eval.py#L58

The dataset spilt setting is different. DGI uses public spilt while GRACE uses 10% train, 10% valid, and 80% test random spilt.

GRACE train/test split follows a 90% / 10% . Isn't that it?

In paper,

On these citation networks, we randomly select 10% of the nodes as the training set, 10% nodes as the validation set, and leave the rest nodes as the test set.

train/valid/test split follows a 10%/10%/80%.
And in code, train/test split follows a 10%/90%. https://github.com/CRIPAC-DIG/GRACE/blob/master/train.py#L41C4 https://github.com/CRIPAC-DIG/GRACE/blob/master/eval.py#L58

Thanks for answering! So, no validation sets are used in the code?

The dataset spilt setting is different. DGI uses public spilt while GRACE uses 10% train, 10% valid, and 80% test random spilt.

GRACE train/test split follows a 90% / 10% . Isn't that it?

In paper,

On these citation networks, we randomly select 10% of the nodes as the training set, 10% nodes as the validation set, and leave the rest nodes as the test set.

train/valid/test split follows a 10%/10%/80%.
And in code, train/test split follows a 10%/90%. https://github.com/CRIPAC-DIG/GRACE/blob/master/train.py#L41C4 https://github.com/CRIPAC-DIG/GRACE/blob/master/eval.py#L58

Thanks for answering! So, no validation sets are used in the code?

GCA？Are validation sets used in GCA?

The dataset spilt setting is different. DGI uses public spilt while GRACE uses 10% train, 10% valid, and 80% test random spilt.

GRACE train/test split follows a 90% / 10% . Isn't that it?

In paper,

On these citation networks, we randomly select 10% of the nodes as the training set, 10% nodes as the validation set, and leave the rest nodes as the test set.

train/valid/test split follows a 10%/10%/80%.
And in code, train/test split follows a 10%/90%. https://github.com/CRIPAC-DIG/GRACE/blob/master/train.py#L41C4 https://github.com/CRIPAC-DIG/GRACE/blob/master/eval.py#L58

Thanks for answering! So, no validation sets are used in the code?

GCA？Are validation sets used in GCA?

GCA utilizes validation sets, whereas GRACE does not incorporate validation sets. You can refer to their eval.py.


Tangram integration

Good day,
I have been trying to follow the vignette Cell-type deconvolution using Tangram but have problems with the deconvolution step. As stated in https://github.com/theislab/squidpy_notebooks/issues/58, the commands do not work with Tangram v0.3.0. I downgraded it to v0.1.3 as suggested, but that downgraded scanpy to 1.6.0 and >= 1.7.0 seems to be needed to run squidpy. I tried Tangram v0.1.3 and scanpy v1.8.0.dev90+gbcfa925 and ran tg.mapping_optimizer.MapperConstrained with no issues but output, F_out = mapper.train(learning_rate=learning_rate, num_epochs=num_epochs) showed the next error:
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-26-6b53317541ec> in <module>
----> 1 output, F_out = mapper.train(learning_rate=learning_rate, num_epochs=num_epochs)

/hpc/pmc_stunnenberg/cruiz/miniconda3/envs/python_pHGG_project/lib/python3.7/site-packages/tangram/mapping_optimizer.py in train(self, num_epochs, learning_rate, print_each)
    221                 loss = self._loss_fn(verbose=False)
    222             else:
--> 223                 loss = self._loss_fn(verbose=True)
    224             optimizer.zero_grad()
    225             loss.backward()

/hpc/pmc_stunnenberg/cruiz/miniconda3/envs/python_pHGG_project/lib/python3.7/site-packages/tangram/mapping_optimizer.py in _loss_fn(self, verbose)
    175
    176         M_probs_filtered = M_probs * F_probs[:, np.newaxis]
--> 177         d_pred = torch.log(M_probs_filtered.sum(axis=0) / (F_probs.sum()))  # KL wants the log in first argument
    178         density_term = self.lambda_d * self._density_criterion(d_pred, self.d)
    179

TypeError: sum() received an invalid combination of arguments - got (axis=int, ), but expected one of:
 * ()
      didn't match because some of the keywords were incorrect: axis
 * (torch.dtype dtype)
      didn't match because some of the keywords were incorrect: axis
 * (tuple of ints dim, torch.dtype dtype)
 * (tuple of ints dim, bool keepdim, torch.dtype dtype)
 * (tuple of ints dim, bool keepdim)

Any suggestions on how to run this part of the tutorial? I am using the last version of squidpy from the dev branch
Thanks in advance!

Good day, in the meantime, I wanted to reproduce the Tangram tutorial but was not able to recreate the environment used in that notebook. Could you please tell me how I can install Squidpy v0.0.0, please?

hi @ccruizm ,
sorry for late reply. That error seems related to tangram rather than version inconstistieces between squidpy and scanpy. We are in touch with tangram developer and hope to update very soon (max 2 weeks) an updated tutorial with latest squidpy and tangram version.
In the meantime, have you tried with the solution described here? https://github.com/theislab/squidpy_notebooks/issues/58#issue-869693405
sorry again for late reply, we are aware of the issue and are working actively toward a solution.

Hello @giovp,
No worries. Looking forward to the updated tutorial.
The problem I had, was once I downgraded Tangram to v0.1.3, it also downgraded scanpy to v1.6.0 and the current version of Squidpy relies on >v1.7.0. I saw the old notebook and there was an older version of Squidpy (v.0.0.0 - first release I guess) but could not find a way to reproduce that environment to run the current Tangram integration tutorial (where the versions used are scanpy==1.6.0 anndata==0.7.4 umap==0.4.6 numpy==1.20.0 scipy==1.6.0 pandas==1.1.4 scikit-learn==0.24.1 statsmodels==0.12.0 python-igraph==0.8.3 leidenalg==0.8.2 squidpy==0.0.0).
Is there a way I could set this environment in the meantime? Could not find the old version of Squidpy to do so
Thanks in advance!

The problem I had, was once I downgraded Tangram to v0.1.3, it also downgraded scanpy to v1.6.0 and the current version of Squidpy relies on >v1.7.0

can you try to install tangram and then install squidpy? There shouldn't be any problem on tangram side for higher version of scanpy.

I tried in that way and did not also work. Since I am using 'pip install squidpy' after installing tangram, pip will upgrade scanpy and get the next: tangram-sc 0.1.3 requires scanpy==1.6.0, but you'll have scanpy 1.7.2 which is incompatible.. This is where the problem relies on. I would need to install an older version of squipy that uses scanpy 1.6.0

and get the next: tangram-sc 0.1.3 requires scanpy==1.6.0, but you'll have scanpy 1.7.2 which is incompatible.

but this is an internal check by pip you should still be able to run the notebook. Have you tried that?

Yes, when using an env set with scanpy==1.8.0.dev90+gbcfa925 anndata==0.7.5 umap==0.5.1 numpy==1.20.2 scipy==1.6.2 pandas==1.2.3 scikit-learn==0.24.1 statsmodels==0.12.2 python-igraph==0.9.1 louvain==0.7.0 leidenalg==0.8.3 pynndescent==0.5.2 squidpy==1.0.0 tangram==0.1.3 I was able to run
mapper = tg.mapping_optimizer.MapperConstrained(
    S=S,
    G=G,
    d=d,
    device=device,
    **hyperparm,
    target_count=adata_st.obs.cell_count.sum()
)

but in the next step got an error output, F_out = mapper.train(learning_rate=learning_rate, num_epochs=num_epochs)
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-26-6b53317541ec> in <module>
----> 1 output, F_out = mapper.train(learning_rate=learning_rate, num_epochs=num_epochs)

/hpc/pmc_stunnenberg/cruiz/miniconda3/envs/python_pHGG_project/lib/python3.7/site-packages/tangram/mapping_optimizer.py in train(self, num_epochs, learning_rate, print_each)
    221                 loss = self._loss_fn(verbose=False)
    222             else:
--> 223                 loss = self._loss_fn(verbose=True)
    224             optimizer.zero_grad()
    225             loss.backward()

/hpc/pmc_stunnenberg/cruiz/miniconda3/envs/python_pHGG_project/lib/python3.7/site-packages/tangram/mapping_optimizer.py in _loss_fn(self, verbose)
    175
    176         M_probs_filtered = M_probs * F_probs[:, np.newaxis]
--> 177         d_pred = torch.log(M_probs_filtered.sum(axis=0) / (F_probs.sum()))  # KL wants the log in first argument
    178         density_term = self.lambda_d * self._density_criterion(d_pred, self.d)
    179

TypeError: sum() received an invalid combination of arguments - got (axis=int, ), but expected one of:
 * ()
      didn't match because some of the keywords were incorrect: axis
 * (torch.dtype dtype)
      didn't match because some of the keywords were incorrect: axis
 * (tuple of ints dim, torch.dtype dtype)
 * (tuple of ints dim, bool keepdim, torch.dtype dtype)
 * (tuple of ints dim, bool keepdim)

I assumed it was related to the incompatibility of the scanpy versions. Maybe it is linked to something else.

ah ok intersting, not that's something from tangram side. Sorry for that, you could either open an issue there or just wait that we push an updated version of the notebook. Thank you!

No problem. I will wait for the updated version 👍🏼
Thanks for the quick response and help

Good day! I just saw there is a new Squidpy update! Super nice. Is the tangram issue fixed in this version (v.1.1.0)? Thanks!

hello @ccruizm , indeed! Here's the link to the tutorial: https://squidpy.readthedocs.io/en/latest/external_tutorials/tutorial_tangram.html
you'd have to install the development branch of Tangram
git clone https://github.com/broadinstitute/Tangram.git
cd Tangram
git switch ziq
pip install -e .

let me know if it works. We'll update the tutorial once a new Tangram release is avaialble.

hi @ccruizm ,
wehave also updated the tutorial using the latest tangram, you can find it here: https://squidpy.readthedocs.io/en/latest/external_tutorials/tutorial_tangram.html
together with the updated conda environemnt https://github.com/theislab/squidpy_notebooks/blob/dev/envs/tangram_environment.yml
I'll close this, feel free to reopen if did not work
Giovanni


FS page design harmonization

Reuse of the same components as on the home page
Addition of an image for the Bookify logo
Objective: make the design more consistent between the two pages

Great @YakeDev you have done a great job. Don't forget to star the repo so that more people will find th repo to contribute.

Great @YakeDev you have done a great job. Don't forget to star the repo so that more people will find th repo to contribute.

Thank you @Ctoic


OmitFolders requires full path(s)

Hi there,
Just wanted to point out the -OmitFolders parameter doesn't appear to work with relative paths, ie. if you CD into a location, and execute Get-FolderSize, the function assumes the current directory, but supplying a list of paths relative to the current directory to this parameter doesn't appear to work.

Not sure if you'd consider this a bug or a feature request? :)  It might work to just add a call to Resolve-Path when doing your checks on the Omit list.
Hope this helps. Thanks for sharing your code!
Edit: Confirm Resolve-Path works from the input side at least, e.g. Get-FolderSize -OmitFolders (Resolve-Path .git) | Sort-Object SizeBytes -Descending.  This, I think would also resolve wildcards, etc. so might be useful in other ways.

I appreciate the feedback, and will work on making the usage more clear and add Resolve-Path.
Thanks!


How long does it takes to confirm a block?

According to https://github.com/EOSIO/eos/issues/4965#issuecomment-410049124 , EOS doesn't use BFT consensus. So in my understanding, it should be at least 2/3 +1 (15) BPs confirming, around 90 seconds, in pure DPoS consensus of EOS. Is that right?

It's combination of DPOS and BFT. In normal situation it takes around 330 block time (165 seconds) to make a block become irreversible. (please check https://eosnetworkmonitor.io/)

@taokayan Thx for your explanation! But I'm still puzzled. I see from https://eosnetworkmonitor.io/ that the difference of current block height and irreversible block height is around 330. But in theory, 2/3+1 (15)BPs confirming could make a block irreversible, hence the max difference of current block height and irreversible block height should be 180. Why it's around 330 in reality?

the actually implementation use DPOS + BFT which result in around 320-330 block time delay.  Pure DPOS will only result in around 180 block time.

@taokayan If the BFT you use increases the confirmation time,  what's the benefit of it? And is the BFT you use same with EOS white-paper mentioned?
And by the way, I see many cryptocurrency exchanges like Huobi and Binance credit EOS deposits to a client’s account after only one confirmation. Do you know why they can do like this?
Sorry for so many questions bothering you!

The basic idea can be found from https://medium.com/eosio/dpos-bft-pipelined-byzantine-fault-tolerance-8a0634a270ba


chain-bench: 0.1.4 -> 0.1.6

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/aquasecurity/chain-bench/releases.
meta.description for chain-bench is: "An open-source tool for auditing your software supply chain stack for security compliance based on a new CIS Software Supply Chain benchmark"
meta.homepage for chain-bench is: "https://github.com/aquasecurity/chain-bench"
meta.changelog for chain-bench is: "https://github.com/aquasecurity/chain-bench/releases/tag/v0.1.6"
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.1.6 with grep in /nix/store/b2q7prh6lh5822ajl87vv33ymkiczyh4-chain-bench-0.1.6
found 0.1.6 in filename of file in /nix/store/b2q7prh6lh5822ajl87vv33ymkiczyh4-chain-bench-0.1.6

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
chain-bench

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/b2q7prh6lh5822ajl87vv33ymkiczyh4-chain-bench-0.1.6 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A chain-bench https://github.com/r-ryantm/nixpkgs/archive/4a3162312862efd50d70439cba480fc609edce6b.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/b2q7prh6lh5822ajl87vv33ymkiczyh4-chain-bench-0.1.6
ls -la /nix/store/b2q7prh6lh5822ajl87vv33ymkiczyh4-chain-bench-0.1.6/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

chain-bench

Maintainer pings
cc @06kellyjac for testing.

Result of nixpkgs-review pr 197879 run on x86_64-linux 1

1 package built:

chain-bench


p2p: Switch.OnStop uses stale and already mutated PeerSet.List() which could have data race consequences due to a leaky abstraction of using .List()

Bug Report
The code in p2p/Switch.OnStop invokes sw.peers.List() per https://github.com/cometbft/cometbft/blob/f4d73cd5a091a997d1f040850710b6937e650125/p2p/switch.go#L251
and if we look at the code for .List() https://github.com/cometbft/cometbft/blob/f4d73cd5a091a997d1f040850710b6937e650125/p2p/peer_set.go#L153-L156 which returns a copy of the slice while under a lock but after handing it over, it is no longer under a lock and is modified by .Remove() which causes .OnStop() to read stale data.
I found this problem while working on the correct fix for https://github.com/cometbft/cometbft/issues/2157 in which the correct way to shorten a slice is to ensure you insert a nil at the last element before reslicing to ensure that there are no references to it and that the backing array can be garbage collected i.e.
endItem := slice[len(slice)-1]
slice[len(slice)-1] = nil // nil it so that it'll be garbage collected

I can reliably reproduce this issue when I apply the fix for the referenced issue
Remedy
Check and ensure that the peer is non-nil.

And indeed, the data race is exposed in https://github.com/cometbft/cometbft/actions/runs/7692838534/job/20960454814?pr=2159 in which this code uses a stale version of the peerSet's list in https://github.com/cometbft/cometbft/blob/940378cf06ca1452c161a821cbe6ea1516268425/p2p/switch.go#L276-L281

Indeed, we need to return a copy of the list of peers here.
This method is actually called in several parts of the p2p package and needs to be reconsidered.


Does your code use the aug dataset?

Thank you for your code,
Please allows me ask, does your code use the aug dataset?
We ran finishly the prepare.ipynb file. At this time, we had the agument data in the "aug" folder. And in your main.py, the link data to the train folder ( label and image) not aug folder. I though, your main.py only use the original training dataset. Is it right? thanks.

Thanks.

As you can see, data_gen_arg is used in the trainGenerator function, which is defined in the data.py file. In lines 56 and 57 of the data.py file, the augmentation dictionary is used to augment the images and masks.


Results for CalculateSun off

I'm currently rewriting a service from Javascript to Go and tried this library for calculating sunrise/down and civil dawn times. The first test for our location gave me results that looked very off, so I started comparing the library's results to other sources. OpenCage and Yr are geo and weather services, meeussunmoon is the library I'm using in the old version.
Lat: 59.9130134 / Lon: 10.7541403 for the example

solar
OpenCage
meeussunmoon
Yr

dawn
01:44:49
02:55:00
02:59:27

rise
03:35:11
04:17:00
04:18:28
04:18:00

down
20:45:11
22:27:00
22:25:58
22:25:00

dusk
22:35:34
23:48:00
23:43:48

Results from solar are sometimes off by more than an hour, compared to all others.

Do you mind comparing the code with ~kennylevinsen/wlsunset? The current code has a lot of quirks to do with timezones and leap seconds that I'm not confident is really correct.


Refactor TxBuilder to improve unlock logic

Currently it is confusing how the unlocking will occur for the different
lock types.

Closing this for now as the tests are running slower with this implementation and it is not needed currently.


[KYUUBI #3298][FOLLOWUP] Fix CI failure

Why are the changes needed?

This pr is a follow-up, aims to fix CI failure due to https://github.com/apache/incubator-kyuubi/pull/3299#discussion_r953265471
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Codecov Report

Merging #3319 (d0cba33) into master (ff52fff) will decrease coverage by 0.07%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3319      +/-   ##
============================================
- Coverage     51.47%   51.40%   -0.08%
  Complexity       13       13
============================================
  Files           471      471
  Lines         26268    26268
  Branches       3633     3633
============================================
- Hits          13522    13503      -19
- Misses        11466    11482      +16
- Partials       1280     1283       +3

Impacted Files
Coverage Δ

...client/exception/RetryableKyuubiRestException.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../org/apache/kyuubi/client/RetryableRestClient.java
48.78% <0.00%> (-24.40%)
:arrow_down:

...main/java/org/apache/kyuubi/client/RestClient.java
82.75% <0.00%> (-3.45%)
:arrow_down:

...apache/kyuubi/engine/JpsApplicationOperation.scala
77.41% <0.00%> (-3.23%)
:arrow_down:

...rg/apache/kyuubi/ctl/cmd/log/LogBatchCommand.scala
78.00% <0.00%> (-2.00%)
:arrow_down:

...pache/kyuubi/engine/YarnApplicationOperation.scala
64.81% <0.00%> (-1.86%)
:arrow_down:

.../engine/spark/session/SparkSQLSessionManager.scala
78.48% <0.00%> (-1.27%)
:arrow_down:

...ain/scala/org/apache/kyuubi/engine/EngineRef.scala
74.07% <0.00%> (-0.93%)
:arrow_down:

...he/kyuubi/ha/client/etcd/EtcdDiscoveryClient.scala
67.50% <0.00%> (-0.63%)
:arrow_down:

...n/scala/org/apache/kyuubi/engine/ProcBuilder.scala
83.22% <0.00%> (+0.62%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Thanks, merging to master


Validation should take keep track of dirtyness

Was implementing this myself when I decided to do a quick check if it wasn't already done, great work.
But it's kinda ugly to show all those error messages to a user when they haven't even typed a single character yet.
So either for schema validation put a isDirty flag on the field data right next to isValid or don't even validate if model[field] === initialModel[field]

Thanks for checking it out, I appreciate your thoughts. You're totally right, however I intentionally ignored the rabbit hole of more intricate (read: correct) form validation as there too many things to consider and this was really meant as a proof of concept.
It would be fairly trivial to implement a isDirty flag on each field, which I think would be the ideal route since initialModel[field] === model[field] is not necessarily correct (initialize w/ empty password, they type something in, delete it; fields are now the same, but invalid).
I'd be open to considering improvements in this regard, though the purpose of this library is almost exclusively to supply the model + getter/setters for it from a higher order component. The rest of the functionality is demo'd to show it's possible, but expected to be more thoroughly implemented in userland.

The way I've implemented this for myself (using reformed as base) is to create a showErrors middleware. This creates a higher-order component that provides two props: getErrorClass and showValidationFeedback. getErrorClass is what I use to display an error indication on my form fields and returns a CSS class name to be fed into classnames. The trick is that it never returns a CSS class until showValidationFeedback has been called at least once. I call showValidationFeedback when the user clicks the submit button (which is always enabled for these forms) and I also use this function to display textual error messages in a ticker and to move focus to the first field with an error.
Of course this approach already makes some assumptions about what your actual form implementation looks like, so your mileage may vary, but hopefully this gives you some ideas.

Thanks for the suggestion @arendjr, that sounds like a pretty sane approach.
I'm going to close this issue just to tidy up the project some, since it's more of a discussion than anything. I'll direct people to your suggestion if this comes up again :).


Storybook doesn't appear to be working with button

Thanks for your work with ember and theming! Really cool stuff with theemo that you've done. I want to setup ember storybook and I'm having issues finding a recent working example of ember storybook.
Seeing error here:
$ yarn api:prepare && yarn api:doc && yarn api:cleanup
$ yarn lerna-run api:prepare
$ FORCE_COLOR=1 yarn lerna --concurrency 1 run --stream api:prepare
$ /Users/jackhoward/sites/hokulea/node_modules/.bin/lerna --concurrency 1 run --stream api:prepare
lerna notice cli v3.22.1
lerna info Executing command in 2 packages: "yarn run api:prepare"
$ yarn api-extractor run --local --verbose
$ /Users/jackhoward/sites/hokulea/node_modules/.bin/api-extractor run --local --verbose
@hokulea/buttons: api-extractor 7.28.2  - https://api-extractor.com/
@hokulea/buttons:
@hokulea/buttons: Using configuration from ./api-extractor.json
@hokulea/buttons: Analysis will use the bundled TypeScript version 4.6.4
@hokulea/buttons: *** The target project appears to use TypeScript 4.7.4 which is newer than the bundled compiler engine; consider upgrading API Extractor.
@hokulea/buttons: Writing: /Users/jackhoward/sites/hokulea/temp/buttons.api.json
@hokulea/buttons: Writing package typings: /Users/jackhoward/sites/hokulea/packages/buttons/dist/buttons.d.ts
@hokulea/buttons: Warning: /Users/jackhoward/sites/hokulea/node_modules/@types/ember__application/index.d.ts:16:23 - (TS2307) Cannot find module 'ember/-private/type-utils' or its corresponding type declarations.
@hokulea/buttons: Warning: /Users/jackhoward/sites/hokulea/node_modules/@types/ember__application/node_modules/@types/ember__routing/index.d.ts:14:24 - (TS2307) Cannot find module 'ember/-private/type-utils' or its corresponding type declarations.
@hokulea/buttons: Warning: /Users/jackhoward/sites/hokulea/node_modules/@types/ember__array/-private/enumerable.d.ts:5:78 - (TS2307) Cannot find module 'ember/-private/type-utils' or its corresponding type declarations.
@hokulea/buttons: Warning: /Users/jackhoward/sites/hokulea/node_modules/@types/ember__object/evented.d.ts:2:76 - (TS2307) Cannot find module 'ember/-private/type-utils' or its corresponding type declarations.
@hokulea/buttons: Warning: /Users/jackhoward/sites/hokulea/node_modules/@types/ember__runloop/-private/backburner.d.ts:1:48 - (TS2307) Cannot find module 'ember/-private/type-utils' or its corresponding type declarations.
@hokulea/buttons: Warning: /Users/jackhoward/sites/hokulea/node_modules/@types/ember__runloop/index.d.ts:9:74 - (TS2307) Cannot find module 'ember/-private/type-utils' or its corresponding type declarations.
@hokulea/buttons: Warning: /Users/jackhoward/sites/hokulea/node_modules/@types/ember__runloop/index.d.ts:19:61 - (TS2344) Type 'M' does not satisfy the constraint '(...args: any) => any'.
@hokulea/buttons: Warning: /Users/jackhoward/sites/hokulea/node_modules/@types/ember__runloop/index.d.ts:27:70 - (TS2344) Type 'M' does not satisfy the constraint '(...args: any) => any'.
@hokulea/buttons: Warning: /Users/jackhoward/sites/hokulea/node_modules/@types/ember__runloop/index.d.ts:27:86 - (TS2344) Type 'M' does not satisfy the constraint '(...args: any) => any'.
@hokulea/buttons: Warning: /Users/jackhoward/sites/hokulea/node_modules/@types/ember__runloop/index.d.ts:79:24 - (TS2344) Type 'M' does not satisfy the constraint '(...args: any) => any'.
@hokulea/buttons: Warning: /Users/jackhoward/sites/hokulea/node_modules/@types/ember__runloop/index.d.ts:131:22 - (TS2344) Type 'M' does not satisfy the constraint '(...args: any) => any'.
@hokulea/buttons: Warning: /Users/jackhoward/sites/hokulea/node_modules/@types/ember__utils/-private/types.d.ts:1:23 - (TS2307) Cannot find module 'ember/-private/type-utils' or its corresponding type declarations.
@hokulea/buttons: Warning: addon/components/button/index.ts:51:1 - (ae-forgotten-export) The symbol "ButtonArgs" needs to be exported by the entry point api.d.ts
@hokulea/buttons: Warning: addon/components/button/index.ts:51:1 - (ae-missing-release-tag) "ButtonComponent" is exported by the package, but it is missing a release tag (@alpha, @beta, @public, or @internal)
@hokulea/buttons: Error: /Users/jackhoward/sites/hokulea/packages/foundation/addon/tokens.ts:1:1 - (ae-wrong-input-file-type) Incorrect file type; API Extractor expects to analyze compiler outputs with the .d.ts file extension. Troubleshooting tips: https://api-extractor.com/link/dts-error
@hokulea/buttons: API Extractor completed with errors
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
lerna ERR! yarn run api:prepare exited 1 in '@hokulea/buttons'
lerna WARN complete Waiting for 1 child process to exit. CTRL-C to exit immediately.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

In another repo, I have been pinning to 6.5.14, but still running into issues with these deps:
    "@storybook/addon-actions": "6.5.14",
    "@storybook/addon-essentials": "6.5.14",
    "@storybook/addon-knobs": "6.4.1--canary.f87792d.0",
    "@storybook/addon-links": "6.5.14",
    "@storybook/addon-storysource": "6.5.14",
    "@storybook/ember": "6.5.14",
    "@storybook/ember-cli-storybook": "^0.4.0",
    "babel-plugin-ember-modules-api-polyfill": "^2.12.0",
    "babel-plugin-htmlbars-inline-precompile": "^3.0.0",

oh gotcha, didn't realize this. thanks @gossi! will check it out

oh gotcha, didn't realize this. thanks @gossi! will check it out


Added Athlete Running and Cycling Performance Analysis Files.

The Athlete Running and Cycling Performance Analysis contains various types of analyses, ranging from basic to advanced. It includes the examination of multiple metrics from the data to uncover insights, trends, and patterns, ultimately aimed at preparing recommendations for improving overall performance.

Hi, I've added Jupyter notebooks for the Running and Cycling performance analysis. Please let me know if you have any feedback or changes you would like me to make.

Thank you for your contribution! I have a few suggestions to help improve this PR:
File Organization & Naming

There's a typo in "Runnig_Analysis" that should be fixed to "Running_Analysis"
Consider removing the student ID from filenames to maintain consistency with other project files
Suggest following the existing naming convention in the repository

Documentation Needs

Please expand the PR description to include:

Specific types of analyses performed
Key metrics being analyzed
Main findings or insights
How these analyses complement existing work in the repository

Additional Recommendations

Could you add a brief README.md in each directory explaining:

The purpose of each notebook
Required dependencies
How to run the analyses
Sample outputs or key visualizations

Consider breaking down the large notebooks into smaller, focused analysis files for better maintainability

Since these are Jupyter notebooks, please ensure:

All cell outputs are cleared before committing
Important visualizations are documented with descriptions
Code cells include appropriate comments

Please address these points and I'll be happy to review again. Let me know if you need any clarification!


[Enhancement] Add support for discriminators

Description
Addresses FR #156

Note: This implementation of discriminators support requires that discriminator mapping be defined. The reason is because we are working with a fully dereferenced OpenAPI object which makes implicit mapping impossible

Motivation and Context
We should do our best to support all polymorphism types, including discriminators.
How Has This Been Tested?
See deploy preview Petstore API for examples.
Screenshots (if appropriate)

I reordered the description and qualifier and removed the margin-top for consistency.

I also took one more pass at ensuring there will be nested support for discriminators. Since it's basically handled in createProperties, we should support them even when they are nested inside any other schema type, e.g. allOf, anyOf, etc.

Initially, we were only handling discriminators that mapped directly to a property at the same level as the discriminator. The recent push adds support for discriminators that map to a property inside the discriminator mapping schema itself.
Example 1:
https://user-images.githubusercontent.com/9343811/181508769-01f20e7a-e5b0-495d-af5b-0d269806856f.mov
Example 2:
https://user-images.githubusercontent.com/9343811/181509011-e0a5361e-77a4-4928-a9ea-6b22c050ed88.mov


Update welcome page

Resolves: #554
This PR following changes:

Add back button at the top left of welcome page to go back to the previous step
Add cryptopower logo to the content of welcome page
Fix dropdown background not changed on dark theme

the launch screen doesn't have the logo

I think we should move "welcome" page to before "choose setup type" page

I think we should move "welcome" page to before "choose setup type" page

leave it as it is


Potential amendment required in Practice Exam 13

How many Availability Zones should compute resources be provisioned across to achieve high availability?

A. A minimum of one
B. A minimum of two
C. A minimum of three
D. A minimum of four or more
Answer
Correct Answer: B

I can see why it is B - Minimum of two but I've read other articles that technically to achieve high availability it should be minimum of three AZ's.
So is it actually B or C? @kananinirav

@StefH @pescap @alessiobennardo @kananinirav  - What are your thoughts on the above?


fix(content): revise util of correcting nested BR tags in html string

👷have to say it's a tough issue.

🦊  🦊  🦊  🦊


Enhance Test Coverage for @yamada-ui/autocomplete

related: #775
Description
Enhance test coverage for components with less than 80% coverage to meet or exceed 80%.
Problem Statement/Justification
Testing is essential to ensure quality when operating a package.

test coverage already exceed 80%.


Add positive result for STATUS_PASSWORD_MUST_CHANGE during smb login

During SMB login STATUS_PASSWORD_MUST_CHANGE means that the password supplied was valid, but currently CrackMapExec reports it as a negative result.
Submitted to fix Issue #391

It might also be interesting to add all the folowing :

[ ]     "STATUS_ACCOUNT_DISABLED"
[ ]     "STATUS_ACCOUNT_EXPIRED"
[ ]     "STATUS_ACCOUNT_RESTRICTION"
[ ]     "STATUS_INVALID_LOGON_HOURS"
[ ]     "STATUS_INVALID_WORKSTATION"
[ ]     "STATUS_LOGON_TYPE_NOT_GRANTED"
[ ]     "STATUS_PASSWORD_EXPIRED"
[X]     "STATUS_PASSWORD_MUST_CHANGE"

Done using another color instead of green https://github.com/byt3bl33d3r/CrackMapExec/issues/391
Closing this one


MAC-OS 0.0.9?

Can you PLEASE make a working version for OS X! 0.0.8 doesn't work and there isn't an OS X app on 0.0.9

If you would still like to install 0.0.9 before the official release is out, it is quick and easy to install the app from the source code, though it requires you to run a few commands from terminal as well as having node.js (https://nodejs.org/en/) installed. I used version 8.9.4 LTS.
Simply download the source archive (https://github.com/tma02/spotispy/archive/master.zip), unzip the file into a folder, and navigate to that folder in terminal. Should be as simple as
cd ~/Downloads/spotispy-master/
followed by the two commands required to install and start the application
npm install
npm start

Thank you for the reply! When will an installer version be available?

No idea my friend, I just found this software yesterday. @tma02 is the developer

Okay. Thanks for the quick reply and the detailed solution.


java.lang.NoClassDefFoundError: Could not initialize class it.auties.whatsapp.util.JacksonProvider

Exception in thread "main" java.lang.NoClassDefFoundError: Could not initialize class it.auties.whatsapp.util.JacksonProvider
        at it.auties.whatsapp.util.Preferences.readAsInternal(Preferences.java:62)
        at it.auties.whatsapp.util.Preferences.readJson(Preferences.java:57)
        at it.auties.whatsapp.controller.Store.of(Store.java:171)
        at it.auties.whatsapp.api.Whatsapp.<init>(Whatsapp.java:81)
        at it.auties.whatsapp.api.Whatsapp.newConnection(Whatsapp.java:126)
        at it.auties.whatsapp.api.Whatsapp.lastConnection(Whatsapp.java:180)
        at me.obsilabor.bootstrap.Bootstrapper.main(Bootstrapper.java:15)

This only happens when running a compiled jar, gradlew run or running through intellij works fine
Versions
Java 17 combinded with Kotlin, Gradle 7.4.2, running on Linux (doesn't work on windows too)
WhatsAppWeb4j Version: com.github.auties00:whatsappweb4j:3.0-RC13, doesn't work on newer ones too (RC17 didn't work because of #158, ill try this version in a second; Doesn't work either

If it has something to do with the issue, my listener is written in kotlin

Exception in thread "main" java.lang.NoClassDefFoundError: Could not initialize class it.auties.whatsapp.util.JacksonProvider
        at it.auties.whatsapp.util.Preferences.readAsInternal(Preferences.java:62)
        at it.auties.whatsapp.util.Preferences.readJson(Preferences.java:57)
        at it.auties.whatsapp.controller.Store.of(Store.java:171)
        at it.auties.whatsapp.api.Whatsapp.<init>(Whatsapp.java:81)
        at it.auties.whatsapp.api.Whatsapp.newConnection(Whatsapp.java:126)
        at it.auties.whatsapp.api.Whatsapp.lastConnection(Whatsapp.java:180)
        at me.obsilabor.bootstrap.Bootstrapper.main(Bootstrapper.java:15)

This only happens when running a compiled jar, gradlew run or running through intellij works fine
Versions
Java 17 combinded with Kotlin, Gradle 7.4.2, running on Linux (doesn't work on windows too) WhatsAppWeb4j Version: com.github.auties00:whatsappweb4j:3.0-RC13, doesn't work on newer ones too (RC17 didn't work because of #158, ill try this version in a second; Doesn't work either

So the problem is probably in how you are building the jar. Can you send your full build.gradle?

plugins {
    kotlin("jvm") version "1.7.10"
    kotlin("plugin.serialization") version "1.7.10"
    id("com.github.johnrengelman.shadow") version "7.1.2"
    application
}

group = "me.obsilabor"
version = "1.0.0"

repositories {
    mavenCentral()
}

application {
    mainClass.set("me.obsilabor.bootstrap.Bootstrapper")
}

dependencies {
    implementation("com.github.auties00:whatsappweb4j:3.0-RC17")
    implementation("io.ktor:ktor-client-core:2.0.3")
    implementation("io.ktor:ktor-client-cio:2.0.3")
    implementation("io.ktor:ktor-client-content-negotiation:2.0.3")
    implementation("io.ktor:ktor-serialization-kotlinx-json:2.0.3")
}

tasks {
    shadowJar {
        manifest {
            attributes(
                "Main-Class" to "me.obsilabor.bootstrap.Bootstrapper"
            )
        }
    }
    compileJava {
        options.encoding = "UTF-8"
        options.compilerArgs.plusAssign("--enable-preview")
        options.release.set(17)
    }
}

this is my build.gradle.kts

You need to build a jar with dependencies:
https://stackoverflow.com/questions/4871656/using-gradle-to-build-a-jar-with-dependencies

I'll try this soon but isn't that exactly what the shadowjar plugins does?

I'll try this soon but isn't that exactly what the shadowjar plugins does?

Yeah, but I think that you would need to run the shadowJar task in particular which is something that you may not be doing

I edited my buildscript to fit the one from your WhatsappWebRequestAnalyzer's build.gradle.kts which creates this jar if i'm not completely blind..
Still doesn't work, same error

I edited my buildscript to fit the one from your WhatsappWebRequestAnalyzer's build.gradle.kts which creates this jar if i'm not completely blind..
Still doesn't work, same error

You are probably looking at the wrong jar as multiple are generated. It should be in the target/lib i think

I don't have a target directory. The jar with the -with-dependencies suffix is in build/libs

I don't have a target directory.

Hi!
I have the exact same error as described above, even from within IntelliJ. My Jar-File contains the requested class (via "jar tf MY_JAR_FILE.jar"). Maybe some uncaught exception occurred and propagated up to the actual ClassLoader attempting to load the class?

So i decided to create a whole new project with just the initialization of WhatsappWeb4j and a basic listener, same issue.
i can't upload jar files here so here's the download: https://workupload.com/file/6hdYn5rFFrP

So i decided to create a whole new project with just the initialization of WhatsappWeb4j and a basic listener, same issue.
i can't upload jar files here so here's the download: https://workupload.com/file/6hdYn5rFFrP

I found the error, just add --enable-preview. I didn't think about that for some reason

Hi! I have the exact same error as described above, even from within IntelliJ. My Jar-File contains the requested class (via "jar tf MY_JAR_FILE.jar"). Maybe some uncaught exception occurred and propagated up to the actual ClassLoader attempting to load the class? Using RC18 and Java 17 with preview-features enabled btw!

You don't have preview features enabled probably

It works now, thanks a lot!


Add imagePullSecret to OC yamls

What does this PR do?
The imagePullSecrets were removed in this PR but removing this made all the OCP3 builds to fail.

Closing because @uCatu created a better PR - https://github.com/cyberark/kubernetes-conjur-deploy/pull/148


Add status note Concrete Constraints on Extensions

Update to include current status. Sometimes it can be hard to find the info required to know about these amazing features. A link to a JIRA and a proposed landing version are awesome when wondering what the status of these things is.
I chose to list Swift 3.1 as the probable release version that this feature would be released with. This is what is listed in the ChangeLog.md https://github.com/apple/swift/blob/master/CHANGELOG.md#swift-31

@practicalswift Can you take a look at this? It seems right in you [gardening] wheelhouse.

@DougGregor for a once-over.
@swift-ci please smoke test.

LGTM, thanks!


ng help command produces duplicate and incorrect docs

OS
Ubuntu 14.04

Versions
$ ng --version
(node:5182) fs: re-evaluating native module sources is not supported. If you are using the graceful-fs module, please update it to a more recent version.
Could not start watchman; falling back to NodeWatcher for file system events.
Visit http://ember-cli.com/user-guide/#watchman for more info.
angular-cli: 1.0.0-beta.5
node: 6.2.1
os: linux x64

Repro steps.

Running ng help produces two sets of commands.
Available commands in angular-cli:
...
Available commands in ng2:
...

There is overlap in the commands shown and the flags/description usually differ. Some commands are only in the first set (build), some only in the second (e2e).
There's no single source of truth making it hard to find the correct command documentation.
This also occurs when running ng help generate.
$ ng help generate
...
Available blueprints:
  ng2:
    ..
  angular-cli:
    ..

All the blueprints from angular-cli that I've tried don't work:
$ ng generate component-addon test --dry-run
(node:6399) fs: re-evaluating native module sources is not supported. If you are using the graceful-fs module, please update it to a more recent version.
Invalid blueprint: component-addon

But some of the ng2 blueprints don't work either:
$ ng generate service-test test --dry-run
(node:6443) fs: re-evaluating native module sources is not supported. If you are using the graceful-fs module, please update it to a more recent version.
Could not start watchman; falling back to NodeWatcher for file system events.
Visit http://ember-cli.com/user-guide/#watchman for more info.
Cannot read property 'then' of undefined
TypeError: Cannot read property 'then' of undefined
    at Class.module.exports.Task.extend.run (/home/vagrant/ng2-widgets/node_modules/angular-cli/node_modules/angular-cli/lib/tasks/generate-from-blueprint.js:61:7)
    at Class.module.exports.Command.extend.run (/home/vagrant/ng2-widgets/node_modules/angular-cli/node_modules/angular-cli/lib/commands/generate.js:65:17)
    at Class.<anonymous> (/home/vagrant/ng2-widgets/node_modules/angular-cli/node_modules/angular-cli/lib/models/command.js:152:17)
    at lib$rsvp$$internal$$tryCatch (/home/vagrant/ng2-widgets/node_modules/angular-cli/node_modules/angular-cli/node_modules/rsvp/dist/rsvp.js:1036:16)
    at lib$rsvp$$internal$$invokeCallback (/home/vagrant/ng2-widgets/node_modules/angular-cli/node_modules/angular-cli/node_modules/rsvp/dist/rsvp.js:1048:17)
    at lib$rsvp$$internal$$publish (/home/vagrant/ng2-widgets/node_modules/angular-cli/node_modules/angular-cli/node_modules/rsvp/dist/rsvp.js:1019:11)
    at lib$rsvp$asap$$flush (/home/vagrant/ng2-widgets/node_modules/angular-cli/node_modules/angular-cli/node_modules/rsvp/dist/rsvp.js:1198:9)
    at _combinedTickCallback (internal/process/next_tick.js:67:7)
    at process._tickCallback (internal/process/next_tick.js:98:9)

ng generate --help seems to display the correct blueprints however.
Thanks.

Closing this as obsolete.


Test that MRTK can run with custom pointer mediator

Overview
MRTK should run fine even if people create custom input providers, mediators, input systems, etc. It would be good to write a set of tests that ensure we do not break custom implementations.
The current task tracks writing a test to ensure that a custom pointer mediator will work with mrtk. The test should:

declare a simple custom mediator
Instantiate MRTK with a profile that uses this custom mediator
Run MRTK for a few frames, make sure there are no exceptions thrown.

I think this was somewhat added in https://github.com/microsoft/MixedRealityToolkit-Unity/pull/8258


Scala.js support

A series of patches that enable Scala.js support plus a simple backend implementation based on the fetch api. I have not yet had the chance to test with a real project.
Summary of commits:

Extract MonadAsyncError implementations
Moves implementations into their own projects which allows sharing between the JVM and JS when applicable

Move backend tests into their projects
Moves the test http server into its own project which allows JS to access it

Change HttpTest to AsyncFreeSpec
JS cannot block so all tests must be async

Scala.js support
Cross builds core, monix, cats and circe for Scala.js. Cookies are JVM only due to their dependencies on Java only classes (the fetch api cannot access the cookie headers plus will pass them in the request). Files are handled via a wrapper class which has implicit conversions from java.io.File, java.nio.files.Path and JS web File. IDN support is provided by the JS URL class which Scala.js does not seem to be able to find on nodejs 8 so at least 10 is required to run the tests.

Fetch backend for Scala.js
A simple non streaming backend using the fetch api. Timeouts are implemented via AbortController which was only added to Chrome in the latest release (66). Running the tests therefore requires you have at least Chrome 66 plus chromedriver on the path. Will look at adding a streaming implementation in the future.

I had to disable the akka client test from running in CI due to https://github.com/akka/akka-http/issues/1930 causing the build to hang. It is still present in the root aggregate project.

Thanks a lot for the changes! There's quite a lot of things in there, so just some high-level comments for now:

I'd definitely like to keep File and InputStream support as-is. They are just very common on the JVM and very useful.
as for the JS backend implementation, I saw that it relies on the fetch APIs which are marked experimental. I'm not very up-to-date with the JS ecosystem, but will that actually work on browsers/nodejs? Is there maybe another way to implement the backend using a more compatible API?
so far I've kept the MonadError implementations local to the backends and copy-pasted when needed - it's not a lot of code but simplifies the build, which already has a lot of pretty small modules
do you think there would a way to split the PR into smaller ones with smaller surface areas?

Okay. Having File support requires the library to add extra code to handle save options such as overwriting. Async streaming implementations will need to convert to the blocking InputStream. I think its better for the caller to be required to do that complex operation. I also thought that Scala.js did not support InputStream but I obviously didn't look closely.
fetch is supported in all mainstream browsers and has been for some time. There is a nodejs implementation available on npm. The browser could be implemented via XMLHttpRequest but fetch has proper streaming support (behind a flag on Firefox).
There were only two locations before but I would like to make a Monix fetch implementation in the future. With the tests being located with each backend it also avoids having to duplicate some test code as well.
It is currently in 7 commits, I think they each focus on a single area pretty well? Do you want me to make a PR with the first one to review on its own?

I don't really know if File and InputStream are supported in Scala.JS - maybe they aren't. Will the code compile at all for scalajs if there are methods accepting these types? I think they are common enough in the JVM that they need to be supported, even if it might mean a potential run-time exception when trying to use them from js
ok, thanks, that's good enough. Though according to https://developer.mozilla.org/en-US/docs/Web/API/Fetch_API IE is not supported? ;)
as for extracting monad impls to separate projects maybe that's a good thing to do but as the PR is already very big and reorganizes a lot of the project I was thinking of ways to slim it down - that might be one possibility. If you could create more focused PRs that would definitely be great, if not I think we can try to work with this one

I have reverted the removal of InputStream and File and added support in the fetch backend.
Yeah no IE, don't think that is really a problem these days.
I have created another PR with the first patch: https://github.com/softwaremill/sttp/pull/93

Now that #93 is in I suppose this one needs an update :)

2nd patch PR https://github.com/softwaremill/sttp/pull/94
This PR can be the 4th one ;)


[Bug]: slow down in markdown

Neovim version
0.9.5
Language affected
markdown
Query
No response
Strategy
No response
Description
Hello. I'm experiencing lags on big markdown files containing a lot of code blocks, when using rainbow-delimiters plugins.

minimal `init.vim`:

let g:my_plug_dir = $HOME . '/.tmp/init.vim-debug'
if !filereadable(g:my_plug_dir . '/autoload/plug.vim')
	silent execute '!curl -fLo '.g:my_plug_dir.'/autoload/plug.vim --create-dirs  https://raw.githubusercontent.com/junegunn/vim-plug/master/plug.vim'
	execute 'source ' . g:my_plug_dir . '/autoload/plug.vim'
end

call plug#begin(g:my_plug_dir . '/vim-plug')
Plug 'nvim-treesitter/nvim-treesitter', {'do': ':TSUpdateSync'}
Plug 'hiphish/rainbow-delimiters.nvim'
call plug#end()

if len(filter(values(g:plugs), '!isdirectory(v:val.dir)'))
	echom 'Some modules are missing, running :PlugInstall'
	PlugInstall
endif

lua <<EOF
	require'nvim-treesitter.configs'.setup {
		auto_install = true,
		highlight = {
			enable = true,
		},
	}
EOF

Example of markdown file in which problem occurs:
test.md
E.g. when i try to delete any line with dd operator insinde test.md noticeable delay occurs. Unfortunately, i do not know how to measure this delay in neovim.

Can you please post one of those markdown files? It could be an issue with too
many different languages or too many code blocks.

@HiPhish I did post it in the comment above

Sorry, I missed the comment.

I have pushed a major update to rainbow-delimiters, can you please try the current master or the tag v0.7.0?


fix: handle an edge case to avoid panic

replaceWithNumbers panics if the argument is an
empty string.
This case was triggered when Generate() is
called with "{person.invalid}" argument.
This results in an empty string as an argument
to replaceWithNumbers.

Great catch!


Does sls invoke local work?

Hey mate
This is sick and i've been using it as a reference for one of our projects.
Have you gotten sls invoke local to work?
From what I can tell it probably doesn't work on this package as the serverless-rollup-plugin doesn't support it as seen in this issue
It means serverless tries to find the javascript that's been transpiled from Typescript but without rollup (with the typescript plugin), it's not transpiling and loses its shit.
Hope you're well!

Good question!
Doesn't look like it would. I haven't had to use it at this point, but will add it to the list to check out at some point.

@tomtomau sorry, completely forgot to test this. Yea it doesn't seem to work. Just have to keep an eye on that issue I guess or move to Webpack.


Is it supposed to look like this?

Endeavour OS (Arch Linux based)
Kernel: 6.12.1-arch1-1
CPU: AMD Ryzen 7 6800H with Radeon Graphics (16) @ 4.785GHz
GPU: AMD ATI Radeon 680M
GPU: NVIDIA GeForce RTX 3050 Mobile

I have seen this too I changed jdk's and sometimes it's good but most of the time it crashes.

I upgraded and this solves it for now: imgui_version=1.87.7

Should be fixed now in the latest commit, thanks @jessec


I don't see "Inject Source" in the debug menu after updating to xcode 7.1

I don't see "Inject Source" in the debug menu after updating to xcode 7.1
I installed injection using alcatraz in my xcode 7.1 but I don't see it.
Does anybody have the same problem?
BTW, Alcatraz has a small bug in xcode 7.1 and if you want to install it, you have to follow these steps:
https://github.com/supermarin/Alcatraz/issues/362#issuecomment-150457101
Thanks in advance.

Now it is working, no idea about the reason.


[Grid]: Refactor prop order for clarity

To explicitly show the container and item prop next to each other. This emphasizes the nested grid demo.

I was going to propose this change :100:.

Ready to merge whenever you ar!


mount: remove new UUID generation

On generating a new UUID(using xfs_admin here), the mount_linux
re-formats the file system; due to which all
the data in the PVC-Clone and PVC-Restore
volume is removed.
To address this issue, we use the nouuid mountoption.
Adding 'nouuid' mount option to all XFS mount so that
we will be able to mount a volume and its restored snapshot on
the same node. Without the option, such a mount fails, because
XFS detects that two different volumes with the same
filesystem UUID are being mounted
On the other hand, ext4, by nature, has the nouuid enabled due to which
we don't need to generate a new UUID there too.
Referenced Issue: https://github.com/container-storage-interface/spec/issues/482
Signed-off-by: Yug Gupta<EMAIL_ADDRESS>

Other CSI drivers also handle this issue in a similar way:
Reference:

aws-ebs-csi-driver
alibaba-cloud-csi-driver
cinder-csi
vsphere-csi-driver


feat(AwsVpcPeering): updates route tables

Description
Changes proposed in this pull request:

updates Kyma and Remote VPC routing tables

Related issue(s)
Resolves #488

/retest


XDG_DATA_HOME compose env variable location?

We're implementing a version of the docker setup outlined here for our own development, and was curious if the following was intentional:
The rails_cache volume is set to /app/tmp/cache:
https://github.com/evilmartians/ruby-on-whales/blob/f6f268d8769b55d9955cac4b00be66caeaf7014a/generator/compose.yml.tt#L32
The XDG_DATA_HOME environment variable is set to /app/tmp/caches (plural):
https://github.com/evilmartians/ruby-on-whales/blob/f6f268d8769b55d9955cac4b00be66caeaf7014a/generator/compose.yml.tt#L62
Is XDG_DATA_HOME meant to be on the rails_cache volume? or a different directory with a too-close name?

It's a typo 🙂 Fixed


请问有没有python开发的交流群？

You can go to https://github.com/wechaty/python-wechaty and scan QR Code over there to join our Python WeChat group.


Dynamic removal/adding the number label for clusters

Is there an easy way to dynamic change the number label when the markers is getting clustered? Lets say I want to load a map at zoom level 7 and showing clusters without numbers inside the markers, but when I get to zoom level 15-16, then I start adding the numbers to the clusters as normal.
Or is the only way to use the createIconFunction and then override the html?
Thanks

Fixed it by using the default createIconFunction and just remove the count in the
`  iconCreateFunction: function(cluster) {
const childCount = cluster.getChildCount();
let c = ' marker-cluster-';
if (childCount < 10) {
c += 'small';
} else if (childCount < 100) {
c += 'medium';
} else {
c += 'large';
}
  return new L.DivIcon({ html: '

', className: 'marker-cluster' + c, iconSize: new L.Point(40, 40) });
}`


Complex dart-lang/gcloud data structures

Tried to make a single class that could be understood by both gcloud and rpc packages. As soon as it got complex, i.e. beyond primitives, gcloud package conflicts with rpc, throwing this error:
RPC: Failed to parse API.

/instagram/v1:
Key: type: Unsupported property type: Type

#0      ApiServer.addApi (package:rpc/src/server.dart:71:7)
#1      main (file:///app/bin/server.dart:29:14)
#2      _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:261)
#3      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:148)

The why: Using gcloud, items that go in and out of the database are required to extend Model. All of the database-storable values have to be either primitives, or other models. To link to other models, you need to provide a database Key.  Here's an example of an integer, string and Key:
@IntProperty() int someInt;
@StringProperty() String someString;
@ModelKeyProperty() Key someKey;

The Key class has a variable of type 'Type' - which is unsupported in rpc, so it fails out.
gcloud can't store a Map, so I'm stuck with this Model structure. Solutions aren't ideal either, as the Key var is just a reference to the actual data. rpc may eventually need methods by which we can omit / add fields to the parser.
Unless I'm missing something, and this is already available...

Good point. We have discussed this before. I have send out a change adding support for ignored fields in a request/response schema class. See https://codereview.chromium.org/1276843005/.

Should be supported now.


Web component to display entity graph

The goal here is to display relations between entities in a graphical way, that can be shown in a second tab of entity pages
The solution we thought of is using force simulation from d3.js.
Sources of inspiration:

https://explore.gnd.network/gnd/118561448/relations?term=Johannes Kepler&rows=25&pos=1
https://ontome.net/classes-tree
https://ontome.net/class/61/namespace/187#graph

See issue #55


[Translation] Translation for Korean

We've translated this part of guidelines to Korean on our repository with github pages.
This is URL of Korean repository : https://github.com/BangSeongJin/translation_to_korean.git

@polpolie I understand that you have translated this github page, can you please let us know the purpose of this issue in detail, so that we can take this forward.

We learned about the tensor flow that is becoming an issue these days through various media,
So we have been working on the Korean Herbal Tensor Flow Community. And
We are looking for other ways to apply open source.

@polpolie you could raise Pull Requests and merge your contributions into the master branch of TensorFlow or TF2.0. Thanks!

@polpolie Could you please let us know if you still need help on this ? if it is resolved then please feel free to move this issue to close status ? Thanks!

Closing this due to lack of recent activity. Thanks!


Cannot work with Django Rest Framework custom action in the viewset

Django v1.11
DRF: 3.7.7
class VendorQuotationViewSet(PermissionRequiredMixin, DynamicModelViewSet):
    """
    VendorQuotations API.
    """
    object_permission_required = 'vendors.send_quotation_to_vendor'
    permission_required = 'vendors.create_quotation'
    permission_classes = (IsAuthenticated,)
    queryset = VendorQuotation.objects.all()
    serializer_class = VendorQuotationSerializer

    # URL pattern: ^vendor_quotations/{pk}/change_status/$
    # URL name: 'user-change-status'
    @detail_route(methods=['put', 'patch'], permission_required='vendors.send_quotation_to_vendor')
    def change_status(self, request, pk):
        obj = self.get_object()
        serializer = VendorQuotationChangeStatusSerializer(data=request.data, context={'request': request})
        if serializer.is_valid():
            status_changed = obj.change_status(serializer.data['status'], request.user)
            return Response({'success': True, "status_changed": status_changed})
        return Response(serializer.errors, status=status.HTTP_400_BAD_REQUEST)

My predicates are as follows:

import rules
from vendors.models import VendorQuotation

@rules.predicate
def is_quotation_creator(user, vendor_quotation):
    return vendor_quotation.created_by == user

@rules.predicate
def vendor_quotation_is_draft(user, vendor_quotation):
    return vendor_quotation.status == 'draft'

@rules.predicate
def can_create_vendor_quotation(user, vendor_quotation=None):
    return True

vendor_quotation_is_deletable = is_quotation_creator & vendor_quotation_is_draft
can_send_quotation_to_vendor = vendor_quotation_is_draft

# set_rule is for shared rules.
# rules.set_rule('can_delete_quotation', vendor_quotation_is_deletable)
# add_perm is separate
rules.add_perm('vendors.delete_quotation', vendor_quotation_is_deletable)
rules.add_perm('vendors.send_quotation_to_vendor', can_send_quotation_to_vendor)
rules.add_perm('vendors.create_quotation', can_create_vendor_quotation)

I got a 'NoneType' object has no attribute 'status' error pointing to the predicate vendor_quotation_is_draft where the vendor_quotation is None.
I was wondering if there's an explicit example that works with custom actions in DRF?

Old but stumbled upon the same problem. For future reference, here's what's going on
I think your problem comes from this line :
    @detail_route(methods=['put', 'patch'], permission_required='vendors.send_quotation_to_vendor')

permission_required will add a permission check even before the request goes into your custom action as per rest_framework implementation (here)
it basically as if you had the following:
class VendorQuotationViewSet(PermissionRequiredMixin, DynamicModelViewSet):
    """
    VendorQuotations API.
    """
    object_permission_required = 'vendors.send_quotation_to_vendor'
    permission_required = 'vendors.send_quotation_to_vendor' ### <<<---- will do a permission check before calling your custom action, as described in this repo's README
    ...
    @detail_route(methods=['put', 'patch'])
    def change_status(self, request, pk):
        ...

This is why you get a NoneType object in vendor_quotation_is_draft, it's called through PermissionRequiredMixin.check_permissions, which doesn't provide any object to your predicate (see the has_perm call here)
By the look of your code, I believe you should be okay by just dropping the permission_required parameter in the detail_route decorator and both permission checks (list and object level) should be performed normally.


Tests and benchmarks failed after upgrade gRPC to 1.7.2

As described in https://github.com/pingcap/grpc-rs/pull/113#issuecomment-344896176, benchmarks failed.
And unit tests fail frequently because of call.c:2032] assertion failed: grpc_cq_begin_op(call->cq, notify_tag)
For the benchmark, it looks like client and server failed to send the last message and close the connection, detailed logs:
Client:
Nov 18 14:34:27.584 INFO listening on [::]:10110
Nov 18 14:34:27.703 INFO receive client setup: server_targets: "localhost:33821" client_type: ASYNC_CLIENT security_params {use_test_ca: true server_host_override: "foo.test.google.fr"} outstanding_rpcs_per_channel: 100 client_channels: 64 rpc_type: STREAMING load_params {closed_loop {}} payload_config {simple_params {}} histogram_params {resolution: 0.01 max_possible:<PHONE_NUMBER>0} channel_args {name: "grpc.optimization_target" str_value: "throughput"}
Nov 18 14:34:27.859 INFO receive client mark: reset: true
Nov 18 14:34:32.864 INFO receive client mark: reset: true
Nov 18 14:35:02.862 INFO receive client mark: reset: true

Server:
Nov 18 14:34:27.581 INFO listening on [::]:10100
Nov 18 14:34:27.594 INFO receive server setup: server_type: ASYNC_SERVER security_params {use_test_ca: true server_host_override: "foo.test.google.fr"} channel_args {name: "grpc.optimization_target" str_value: "throughput"}
Nov 18 14:34:32.865 INFO receive server mark: reset: true

They are both missing a log for shutting down.
https://github.com/pingcap/grpc-rs/blob/e45a84332e7e8c5e490ade91a501610f612fdce8/benchmark/src/worker.rs#L113
https://github.com/pingcap/grpc-rs/blob/e45a84332e7e8c5e490ade91a501610f612fdce8/benchmark/src/worker.rs#L77
CC #107

This is because grpc_call_start_batch is called after the completion queue is shutdown. We need to check it every time completion queue is access.


<n-dropdown trigger="focus"/>鼠标点击不会触发select事件

TuSimple/naive-ui version (版本)
2.34.3
Vue version (Vue 版本)
3.2.45
Browser and its version (浏览器及其版本)
Chrome
System and its version (系统及其版本)
Window 10
Node version (Node 版本)
Reappearance link (重现链接)
https://codesandbox.io/s/upbeat-http-e58t57?file=/src/Demo.vue
Reappearance steps (重现步骤)
点上面的minimal reproduction。
鼠标点一下文本框，然后点一下任何一个菜单项。
Expected results (期望的结果)
弹出message，表示触发的select事件。
Actual results (实际的结果)
什么都没发生，只是菜单消失了。
Remarks (补充说明)
不使用文本框，使用按钮也是一样有这个bug的。
使用键盘上下选择，回车确认是会触发select事件的。

I'm not sure this is a bug. Looking through the documentation, available triggers are "click" and "hover". If you want it explicitly on focus, you would need the following:
<n-dropdown :show="showDropdown" @select="handleSelect">
  <n-input @focus="handleFocus" />
</n-dropdown>

You will also need to import 'ref' from Vue
import { defineComponent, ref } from 'vue'
Declare a reference to showDropdown
const showDropdown = ref(false)
Include showDropdown in the return and define the following functions after your options.
handleSelect(key) {
  message.info(String(key));
  showDropdown.value = false;
}

handleFocus() {
  showDropdown.value = !showDropdown.value
}

If you wanted to use the NButton component, you could set trigger="hover" or trigger="click" or manually as above.

hello @thibodeauxjl,
In the popover document, the trigger type is 'hover' | 'click' | 'focus' | 'manual'(and same in the .d.ts file ).  So in your example you should set trigger prop to manual.
And I have already tried your example that explicitly set the show prop in the focus/blur events. The bug still occurs.

https://codesandbox.io/s/laughing-paper-ld4jei?file=/src/Demo.vue

Maybe I know what's the problem here.
When user click the menu item:

input blur .
dropdown receive the blur event, and set show = false
menu item being clicked
because the dropdown show == false, so it ignores the click event

@thibodeauxjl

That is correct, but you are currently using a Dropdown.. not a popover, so not all the props for a popover are compatible and the documentation isn't real clear on which ones are or aren't. The only one it specifies is 'placement' under the Dropdown Props documentation.

The document says For other props, for example placement, and the trigger is not in the above. So it is other props.
In your example, you doesn't handle blur event,  so the dropdown remains showing when click outside the control. And because you toggle the boolean variable in focus event, the dropdown will hide when the input focus again. This is so bad use experience.

There is a dropdown prop for on-clickoutside. I've updated the above codesandbox to handle the @clickoutside

After testing them all, the best user experience would be on-focus and on-blur. Click functionality seems to repeat every time its been clicked. on-clickoutside disappears with a double click.
<n-dropdown :show="showDropdown" :options="options" @select="handleSelect">
    <n-input @focus="handleFocus" @blur="handleBlur" />
</n-dropdown>

You'll set value of showDropdown to true on focus and false on blur.

There is a dropdown prop for on-clickoutside. I've updated the above codesandbox to handle the @clickoutside. I'm not exactly sure what you are trying to accomplish. If you need a simple dropdown when the user clicks the input or the button, then trigger="click" will do that. If you looking for some advanced functionality then please explain what exactly it is you are looking for and I'll try my best to help you out with it.

In the example, use tab navigate one input to another, the menu is not disappear.

You'll set value of showDropdown to true on focus and false on blur.

As I said above, blur event fired first, the menu item click may be ignored.

I'm seeing the dilemma now. Give me a couple minutes.

I've updated the link above again with the auto complete component which works really well with the input functionality you are looking for.
I think it would still be beneficial if they would add a on-mousedown prop for the focus trigger on the dropdown though. Possible feature request?

和这个bug无关，我想说说我的需求。
我想要一个文本框，输入的内容是从一个固定的内容中选择的。例如：输入固定的几十种颜色。
根据这个需求的描述，首选肯定是一个Select。但是，Select的缺点是，它不是文本框，它没有文本框的各种方便特性。不能在极少数的情况下输入自定义内容（fiilterable，on-create可以，但是不好，不能基于当前选中的文本修改，而要完全重新输入。可能自定义输入的内容还会追加到当前列表，这点不好）。Select最大的不好在于，不能复制粘贴内容，这其实非常不方便。
我还试过AutoComplete。一开始我以为要有内容输入才会弹出候选列表，示例里每个都是。我试过才知道，focus的时候列表就能显示出来了。它的问题是，当列表弹出来的时候，没能自动滚动和选中当前的那一条。还是一个问题是，只在focus的时候弹出列表，选择了一个之后，列表消失，要在外面点一下，在点一下文本框列表才显示出来。不过这个问题后来我想到可以在on-select事件中，直接调用blur()来解决。
最后是当前bug的方案，Input+Dropdown。我比较想使用这个方案，在我心底就希望它是个文本框并且在focus的时候弹出候选列表。但是出来有当前的bug外，还有AutoComplete不能自动选中的问题，还有就是列表太长是没有滚动条的，会把页面拉长。
我觉得我还是会选择Select，全程鼠标操作的话，Select的体验还是最好的。

I've updated the link above again with the auto complete component which works really well with the input functionality you are looking for.
I think it would still be beneficial if they would add a on-mousedown prop for the focus trigger on the dropdown though. Possible feature request?

Thanks you. You demo is perfect to solve my orginal request. But finally I choose to use Select, becuase my candidate items are so much that dropdown will enlarge the body with a scrollbar. And Select's popover itself has a scrollbar.
And I think the bug is still there, if I doesn't guess wrong in the https://github.com/tusen-ai/naive-ui/issues/4335#issuecomment-1378447803

The bug is due to the @click event taking place after mouseup in the event cycle. The @blur event takes place after mousedown in the event cycle. Because of this, the @click never happens as it occurs after the dropdown is hidden. Which is why I mentioned a possible on-mousedown prop to work with the focus trigger.

This issue does not have any recent activity. If you are still experiencing similar problems, open a new error, including a minimal copy of the problem


Oluwasijibomi Olaifa

-#Design System- Desktop Grid, Tablet Grid & Mobile Grid
-#74 Designed the mobile low-fi and hifi for the authentication library.
-#79 Designed Error Page 500 low-fi & hifi for both desktop and mobile view
https://www.figma.com/file/UzdIIpsX43I1ZgMETPwLgQ/Authentication-Library?node-id=2329%3A3605
https://www.figma.com/file/UzdIIpsX43I1ZgMETPwLgQ/Authentication-Library?node-id=1580%3A2668


Changed references of "Athena" to "UTD-Trends"

Overview
Partially resolves #39
I replaced all occurrences of "Athena" to "UTD-Trends" except for the files in the styleguide folder.
What Changed
The CONTRIBUTING.md file was updated so that new contributors won't be confused since the repo is named utd-trends while in the contribution file it's referred to as athena. I also updated the README.md, package.json and other files.
Other Notes
There were some occurences of "athena" in files in the styleguide folder but I'm not exactly sure what the files in that folder were for so I didn't modify them.

Whoops. Forgot to create a different branch for the logo link issue. Sorry about that.


refactor: upgrade crossbell.js and wagmi

Wait wagmi switched to viem.
https://github.com/wagmi-dev/wagmi/tree/next

There are 1.0.0-next.* packages (tag: next) to test now. The expected Release date: Between 2023-05-05 and 2023-05-12 per https://github.com/wagmi-dev/wagmi/pull/2235

Ready for review as well.
This PR will be merged to next branch and wait wagmi v1 to be stable

@Songkeys Ready to merge.

vercel deployment is down.
/cc @runjuu PTAL at the source code

It seems like a duplicate type issue. I've made some changes in the main branch to try to fix the deployment. Rebase and see if the issue is fixed.

Also, @sxzz, wagmi v1 was officially released a couple of days ago. Maybe we can now merge it into the main branch directly. What do you think?

@runjuu Of course! Changed. By the way, crossbell has been released v1.

Upgraded. CI passed.


CASMTRIAGE-4349: Cray-sysmgmt-health chart backport

Summary and Scope
Summarize what has changed. Explain why this PR is necessary. What is impacted? Is this a new feature, critical bug fix, etc?
CASMTRIAGE- 4349: Changed the Cray-sysmgmt-chart version to add support for multiple targets in SNMP exporter
Issues and Related PRs
List and characterize relationship to Jira/Github issues and other pull requests. Be sure to list dependencies.
https://jira-pro.its.hpecorp.net:8443/browse/CASMTRIAGE-4349

Duplicate


Enhance branch display

This implements #258
Add gitlens.gitExplorer.branches.layout config with three available option: list, tree, and mix-tree
list is the current behavior, would display all branches with full branch name:

tree organize branch name with slashes as folder:

mix-tree sort all branches alphabetically:

TODOs

[x] Unexpected ordering with tree layout
[x] Current branch within branch folder should be at first in tree layout

[x] and should be expanded by default

[x] Detach state branch name shouldn't be parse as tree

@Yukaii Awesome -- I'm starting to look at this. But first can you adapt this to target the develop branch? As that branch is what will become GitLen 8 -- which will be the next release. Thanks!

@eamodio I've merged the changes from develop branch 😸

@Yukaii I'm trying to finalize GitLens 8 for release and as I'd like to get this in, I'm planning on just getting this in and then remove the mix-tree option in a follow-on commit -- sound ok?

@eamodio Sure 👍

This has been merged into develop and as long as GitLens 8 release candidate looks good this will ship in v8.  FYI, I made a bunch of changes to the behavior here: https://github.com/eamodio/vscode-gitlens/commit/112879dbc20c90c2fd89e5cb504742b699084572
I removed the mix-tree option and changed tree sorting to be similar to what mix-tree was. Although I pulled the current branch out of any nesting. Since I think it is important to keep the current branch at the top and stable.
I removed the folder icons in favor of using a new feature in vscode 1.20 (using the File Icon Theme). Unfortunately for vscode 1.19 there will be no icons at all, but since 1.20 is supposed to ship this week, I don't think it matter much.
I also added the new layout option to the new GitLens Settings page.
Thanks again for your awesome work! This is a great feature!


Add some zips and unzips to StrictSeq

This is useful to address a TODO comment in the ledger, which ends up implementing one of these

bors merge


[BUG] Dapper wallet info displayed incorrect

Dapper wallet info displayed incorrect.
(1) The token info in dapper wallet account is COA's token info.
(2)NFTs are not display in dapper account.
(3) When switch to Dapper wallet, can not switch back to COA.
https://github.com/user-attachments/assets/28903317-c10e-4da0-94cb-275cf9edafc2

This issue already fixed in ios prod version 2.1.12(8).


The 1921 Epic

It should be noted that only FindMyPast will have images in the first instance. We will not have images available unless transcribers wish to use their own membership.
However FindMyPast is charging UKPounds 3.50 to view each image,
In reality the cost precludes volunteers using their own memberships.
In my opinion this project should be given a low priority until the situation changes.
Geoff

@geoffj-FUG I'm pretty sure that FindMyPast T&C would not allow volunteers to use the scans for anything other than personal use anyway (along with similar sites).


Add microphone key binding container

Not sure is this useful.
Not need in ruleset now.

Need to wait until android Emu and SDK downloaded.


Fail to download

When I tried to download the dataset with ego4d --datasets clips --output_directory ., I got the error:
Datasets to download: {'clips'}
Download Path: v2
Downloading Ego4D metadata json..
Traceback (most recent call last):
  File "/home/xxx/.local/bin/ego4d", line 8, in <module>
    sys.exit(main())
  File "/home/xxx/.local/share/pipx/venvs/ego4d/lib/python3.10/site-packages/ego4d/cli/cli.py", line 251, in main
    main_cfg(config)
  File "/home/xxx/.local/share/pipx/venvs/ego4d/lib/python3.10/site-packages/ego4d/cli/cli.py", line 74, in main_cfg
    metadata_path = download_metadata(
  File "/home/xxx/.local/share/pipx/venvs/ego4d/lib/python3.10/site-packages/ego4d/cli/manifest.py", line 199, in download_metadata
    _metadata_object(version, s3).download_file(str(download_path))
  File "/home/xxx/.local/share/pipx/venvs/ego4d/lib/python3.10/site-packages/boto3/s3/inject.py", line 361, in object_download_file
    return self.meta.client.download_file(
  File "/home/xxx/.local/share/pipx/venvs/ego4d/lib/python3.10/site-packages/boto3/s3/inject.py", line 192, in download_file
    return transfer.download_file(
  File "/home/xxx/.local/share/pipx/venvs/ego4d/lib/python3.10/site-packages/boto3/s3/transfer.py", line 406, in download_file
    future.result()
  File "/home/xxx/.local/share/pipx/venvs/ego4d/lib/python3.10/site-packages/s3transfer/futures.py", line 103, in result
    return self._coordinator.result()
  File "/home/xxx/.local/share/pipx/venvs/ego4d/lib/python3.10/site-packages/s3transfer/futures.py", line 266, in result
    raise self._exception
  File "/home/xxx/.local/share/pipx/venvs/ego4d/lib/python3.10/site-packages/s3transfer/tasks.py", line 269, in _main
    self._submit(transfer_future=transfer_future, **kwargs)
  File "/home/xxx/.local/share/pipx/venvs/ego4d/lib/python3.10/site-packages/s3transfer/download.py", line 354, in _submit
    response = client.head_object(
  File "/home/xxx/.local/share/pipx/venvs/ego4d/lib/python3.10/site-packages/botocore/client.py", line 569, in _api_call
    return self._make_api_call(operation_name, kwargs)
  File "/home/xxx/.local/share/pipx/venvs/ego4d/lib/python3.10/site-packages/botocore/client.py", line 1023, in _make_api_call
    raise error_class(parsed_response, operation_name)
botocore.exceptions.ClientError: An error occurred (403) when calling the HeadObject operation: Forbidden

It seems something wrong with the Amazon bucket.

me too, did you addressed this issue?

Thank you very much. I could not get my access key & secret key.
Should I wait for a few days after I sign on Dropbox?

Closing this issue for now. Please feel free to re-open it if the problem persists. Thanks!


forbid-prop-types: Checking forbidden proptypes as part of another proptype

Say for example, I forbid the any proptype. Quite common to do so.
This would be forbidden:
AwesomeComponent.propTypes = {
  foo: PropTypes.any
};

But this would still be allowed:
AwesomeComponent.propTypes = {
  foo: PropTypes.arrayOf(PropTypes.any)
};

Even though is clearly has that pesky any in there.
Can this rule please be extended to (optionally) also check for proptypes that are part of other proptypes? Call them sub-proptypes, I suppose.

I would not expect it to be allowed; I'd consider that a bug in the rule.


Expose GCP project_id / network_name for network peering

Affected Resource(s)
Please list the resources as a list, for example:

rediscloud_subscription (GCP)

It would be useful if the rediscloud_subscription resource exposed the gcp_project_id and gcp_network_name needed to make a GCP network peering.  This data seems to only be exposed on the UI when making a peering connection manually, and thus defeats the ability to generate a full subscription/peering setup within Terraform.

Hi,
Thanks for raising this issue. We recently solved a bug in our API that prevented the peering with GCP environments, has this been raised in relation to that issue? You should be able to create a peering through terraform without UI actions

I can successfully create the peering through Terraform, however I had to pull the gcp_project_id and gcp_network_name from the UI and fill them into terraform.  This issue is to streamline that process by exposing those two parameters directly to terraform.  For example, I would like to be able to do the following:
resource "rediscloud_subscription" "example" {
....
}

resource "rediscloud_subscription_peering" "example" {
   subscription_id = rediscloud_subscription.example.id
   provider = "GCP"
   gcp_project_id = rediscloud_subscription.example.gcp_project_id
   gcp_network_name = rediscloud_subscription.example.gcp_network_name
}

I did notice that networking_vpc_id is exposed (I think I missed it before), but the Project ID for GCP seems hidden from this restriction ((using Cloud Account ID = 1 implies using Redis Labs internal cloud account). Note that a GCP subscription can be created only with Redis Labs internal cloud account)

Hi @atsui-bay1 ,
Looks like we already support exposing those two fields in Cloud-API:

projectUid
networkName

After creating a GCP VPC peering, one can get those values from our GET/Subscriptions/{subscriptionId}/peering endpoint
This is an example for a response:
{
  "taskId": "e8b83db4-efcc-450c-862a-70eb6333c698",
  "commandType": "vpcPeeringGetRequest",
  "status": "processing-completed",
  "description": "Request processing completed successfully and its resources are now being provisioned / de-provisioned.",
  "timestamp": "2021-12-09T14:24:38.447548Z",
  "response": {
    "resourceId": 129138,
    "resource": {
      "peerings": [
        {
          "vpcPeeringId": 1297,
          "projectUid": "cloud-api-256112",
          "networkName": "cloud-api-vpc-peering-test",
          "redisProjectUid": "q29b7889c8a2ae6cb-tp",
          "redisNetworkName": "c298956-us-central1-mz-rlrcp",
          "cloudPeeringId": "redislabs-peering-16da2a40-a78",
          "status": "inactive"
        }
      ]
    }
  },
  "_links": {
    "self": {
      "href": "http://localhost:8081/v1/tasks/e8b83db4-efcc-450c-862a-70eb6333c698",
      "type": "GET"
    }
  }
}

If something is still not clear, fell free to reach.

Hi @danielarbiv,
The issue is that the projectId is not exposed in any Terraform resource output or attribute, and specifically not exposed early enough to use as a reference when creating the peering connection.  Being able to query the projectId after creating the peering is not helpful in this case.


Cloud sync

Problem
After pouring one's life into this app, the phone is stolen, lost, or damaged beyond recovery, and the contents are subsequently lost.
Suggestion
Make it possible to synchronise the contents with a remote server. Personally I'd prefer WebDAV support, since that would automatically mean ownCloud/Nextcloud support, but even an option to export entries automatically as flat text files somewhere in the file system would make it possible to then have another app watching for changes to that directory and act upon its contents.
I realise this may fall outside the initial scope of this otherwise excellent app, but nobody wants to lose their diary.

We currently are in the process of creating a backup feature to export all user data into a file. Synchronization would be a great feature to add, but it may take a little longer to implement at this point. WebDAV seems to be nice option :+1:
Regards, Michael


[Authorization Logic] Write realistic examples.

@conradgrobler

We have done this as a part of integrating authorization logic with raksha. I think the examples related to ambient computing qualify.


Chapter 2 code review

I made some suggestions on the code for Chapter 2.
@Bayi, I could not run Section 3 as I did not have the latest data for it. If you share it, I can also review that section as part of this PR.

@hadyyu, @valaneila, while talking to @ClementineCttn today I noticed that my review is on the wrong file, the R script, and it should be on the Rmd. I discussed the main points of my feeback with @ClementineCttn, but I will also check the Rmd file to see if I have any other suggestion. When I'm done I will leave an update here.


Open all files in script editor

This PR allows editing any file in the script editor.
Firstly, a new ResourceFormat called TextFile. The TextFile format can take any type of file (providing its utf_8).
As TextFile can take any type the resource_loader and  resource_saver methods had to be changed. They will only load TextFile formats when explicitly asked for. This does however means that we have have to wait for the initial loading call to fail before trying again. This ensures that nothing else can load it. As loader order is "random".
To make editing all files possible, the hint type passed into InspectorDock::_load_resource is now passed all the way through the system. The script editor passes type Script. This is used to indicate that if the resource is not loaded we can safely try using TextFile. Allowing editing scene files within the ScriptEditor.
Secondly, I've Generalised ScriptEditorPlugin and ScriptEditorBase to take and edit any resource.
Thirdly, introduced a new TextEditor class extending ScriptEditorBase to edit the TextFile resources. Currently this is is fairly lightweight.
Lastly, I moved some of the text manipulation into CodeTexteditor reducing duplication from the ShaderTextEditor, ScriptTextEditor and would be TextEditor.
The changes from #19166 will have to be replicated into TextEditor.
closes #1390

I'm unsure about the resource_loader changes, weather the loader should  internally check for Script or force the caller to handle the failed load?
The TextEditor class will also need a icon to use in the script list.

Does it mean we can also edit shaders there?

Renamed p_script to p_resource.
@Zylann Sort of, the shader editor grabs focus if it is open or has been opened.
Otherwise yes, though I can't recommend it, as its like working in notepad.

Moving it shouldn't be to hard if we decide that it is a nicer workflow.
But since you usually want to see your result it should stay at in the bottom panel.

I am against hacking ResourceLoader/Saver for this.  Just harcode this support into the editor plugin, even if its considerably more code.

Integrated the changes from #19166 into text_editor.
Adjusted to no longer use ResourceLoader or ResourceSaver. Instead the ScriptEditor/ScriptEditorPlugin will handle all TextFile calls internally.
To allow Opening/Save As, it uses its own EditorFileDialog and will switch based on the type of file selected. If its a known Scripttype it will be processed as normal, else will be treated as a TextFile.

I just want to share this from our PR meeting where we agreed to merge this:
reduz> oh wow, it's really well done
Akien> Paul can work magic :)

I can only agree. Huge thanks for this!