how can i run tensorflow in second gpu

I am tired to make TensorFlow 2.1 to work in the second , third, fourth GPU
but every time TensorFlow uses the first GPU
I have 4 GPU and I want to train 4 model each model in different GPU
System information

windows 10 x64
python 3.6
TensorFlow  2.1
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2019 NVIDIA Corporation
Built on Fri_Feb__8_19:08:26_Pacific_Standard_Time_2019
Cuda compilation tools, release 10.1, V10.1.105
GPU : Tesla K20Xm

I tired before run jupyter notebook write this command
set CUDA_VISIBLE_DEVICES=0 & jupyter notebook
set CUDA_VISIBLE_DEVICES=1 & jupyter notebook
set CUDA_VISIBLE_DEVICES=2 & jupyter notebook
set CUDA_VISIBLE_DEVICES=3 & jupyter notebook
also inside jupyter
import os
os.environ["CUDA_VISIBLE_DEVICES"]="1,2,3"
what is solution please ??

Hi @abdoelsayed2016, in order to use multiple GPUs with Tensorflow please take a look at the Distributed training with TensorFlow guide. This guide walks through the distribution strategies available and shows how to use them.

hi @nikitamaia  , thank for your replay
I want to run each jupyter in different gpu ,
in this link  , i need to change all code in jupyter to add it
i need a solution to make TensorFlow use not default gpu:0

I'm not sure I understand your question? The solution to make TF not use default gpu:0 is what is explained in that link. You have to set it explicitly. Let me know if I'm misunderstanding what you are asking about.

When i tired the tf.device
I see the operation in the command line still using first gpu , and only training using second gpu

If u see in the photo most of operations use the gpu:0

Ahh I see, looks like those VarHandleOp are being executed on GPU 0. It's difficult to say much more without a minimal reproducible example, but something else you might want to try is using tf.config.set_visible_devices to specify which devices are visible to the runtime. This will allow you to make only 1 of your 4 GPUs available for memory allocation and operations.

@nikitamaia
thank you for your solution last link work with me

Glad to hear it! Closing this issue now since a solution was found.


Support online reload tls in ticdc

Is your feature request related to a problem?
Usually, there will be an expiration of the TLS certificates, we have to support online reload with new certificates instead of just restart the process.
Describe the feature you'd like
no
Describe alternatives you've considered
No response
Teachability, Documentation, Adoption, Migration Strategy
No response

/label area/cdc

/area ticdc

/assign @CharlesCheung96

/severity critical


Improve checks

Actions:

Add new compiler names
Add new gui app names.
Move missing_wheel class to the place near uses_setuptools, because mainly both are used in host.
Add .idea to .gitignore

I'm fine moving these around if everyone else is! Thanks for organizing/adding this, @skupr-anaconda!


Implement http basic account manager

This PR includes two commits:

The first commit creates a BasicAccountManager which follows the HTTP Basic Authentication to get the username from a HTTP request.
The second commit labels the owner of the created docker container based on the user from BasicAccountManager.

BTW, I added write permission on this github repo to @denniscy1993 for merging PR, doing presubmit checks, and changing reviewers freely. FYI, @ser-io & @jemoreira

What happens if user info is not included in http request? and can we specific user/pw in cvdr for now?

If using the new http basic account manager and there's no user info the request should fail. We can continue using the unix account manager in a lab environment for now. And yes, for cvdr the next step would be to extend the authentication options with http basic credentials: similar to this: https://github.com/google/cloud-android-orchestration/pull/264/commits/0a09977043d30086e81205320e5a0fd0c268701e

With this PR, user would receive http.StatusBadRequest if the user info is not included in the http request when using HTTPBasicAccountManager. Thanks @ser-io for the reference, I'll continue to work on adding http basic credentials to cvdr.


4i dataset

...
Hi, a subset of 4i dataset is available in https://squidpy.readthedocs.io/en/stable/api/squidpy.datasets.four_i.html#squidpy.datasets.four_i.
I am wondering how to load/find the rest part of the 4i data reported in original science paper?

Hi @yuanzhiyuan
We received this subset of the 4i dataset directly from the authors of the original science paper. Unfortunately I am not aware of any public location of the entire dataset. I'd suggest to contact the authors directly for full access to this dataset.
As a side note: if you are interested in how we processed the data to the ImageContainer format, you can have a look at this preprocessing notebook here: https://github.com/theislab/squidpy_reproducibility/blob/master/notebooks/prepare_data/4i.ipynb

@yuanzhiyuan hope this solves your issue, I'll close this but feel free to re-open in case not addressed.


Wrapping an sc_array only works by manually allocating a new array?

Follow-up issue for the discussion started at https://github.com/trixi-framework/Trixi.jl/pull/637#discussion_r656862002:
Comment 1 by @efaulhaber:

https://github.com/trixi-framework/Trixi.jl/blob/3883fd3990c1d4550883ff69f7f12aec73f1201e/src/auxiliary/p4est.jl#L34
Why does this work, but
unsafe_wrap(Array, reinterpret(Ptr{p4est_tree_t}, mesh.p4est.trees.array), 1)

crashes with an access violation?

Comment 2 by @ranocha:

MWE
julia> using Trixi

julia> redirect_stdout(devnull) do
           include("examples/2d/elixir_advection_basic_p4est.jl")
       end

julia> unsafe_load(Ptr{Trixi.p4est_tree}(mesh.p4est.trees.array), 1)
P4est.LibP4est.p4est_tree(quadrants=P4est.LibP4est.sc_array(elem_size=0x0000000000000018, elem_count=0x0000000000000004, byte_alloc=128, array=Ptr{Int8} @0x000000000435f040), first_desc=P4est.LibP4est.p4est_quadrant(x=0, y=0, level=29, pad8=-1, pad16=-1, p=P4est.LibP4est.p4est_quadrant_data(user_data=Ptr{Nothing} @0xffffffffffffffff, user_long=-1, user_int=-1, which_tree=-1, piggy1=P4est.LibP4est.var"Cstruct_anonymous|native|which_tree::(p4est_topidx_t)|owner_rank::(𝐣𝐥.Cint)"(which_tree=-1, owner_rank=-1), piggy2=P4est.LibP4est.var"Cstruct_anonymous|native|which_tree::(p4est_topidx_t)|from_tree::(p4est_topidx_t)"(which_tree=-1, from_tree=-1), piggy3=P4est.LibP4est.var"Cstruct_anonymous|native|which_tree::(p4est_topidx_t)|local_num::(p4est_locidx_t)"(which_tree=-1, local_num=-1))), last_desc=P4est.LibP4est.p4est_quadrant(x=1073741822, y=1073741822, level=29, pad8=-1, pad16=-1, p=P4est.LibP4est.p4est_quadrant_data(user_data=Ptr{Nothing} @0xffffffffffffffff, user_long=-1, user_int=-1, which_tree=-1, piggy1=P4est.LibP4est.var"Cstruct_anonymous|native|which_tree::(p4est_topidx_t)|owner_rank::(𝐣𝐥.Cint)"(which_tree=-1, owner_rank=-1), piggy2=P4est.LibP4est.var"Cstruct_anonymous|native|which_tree::(p4est_topidx_t)|from_tree::(p4est_topidx_t)"(which_tree=-1, from_tree=-1), piggy3=P4est.LibP4est.var"Cstruct_anonymous|native|which_tree::(p4est_topidx_t)|local_num::(p4est_locidx_t)"(which_tree=-1, local_num=-1))), quadrants_offset=0, quadrants_per_level=Int32[0, 4, 0, 0, 0, 0, 0, 0, 0, 0  …  0, 0, 0, 0, 0, 0, 0, 0, 0, -1], maxlevel=1)

julia> unsafe_wrap(Array, Ptr{Trixi.p4est_tree}(mesh.p4est.trees.array), 1)

signal (11): Segmentation fault
in expression starting at none:0
[...]

Weird...

Comment 3 by @ranocha:

Note that this call to unsafe_load is also overwritten by CBinding.jl at https://github.com/analytech-solutions/CBinding.jl/blob/575150b055725a9107e5c39d9bdebb2e5acfce35/src/cetc.jl#L11 but unsafe_wrap comes from Base. My impression is that this issue is somehow caused by CBinding.jl, which I don't know very well. @sloede?

Comment 4 by @sloede:

Sorry, I do not know any of the internals of CBinding.jl, this is all voodoo to me. If we cannot figure this out but it is important, we can consider pinging at-krrutkow and/or opening an issue at https://github.com/analytech-solutions/CBinding.jl. We'd might need a more mininum MWE (or we could try with this and see if he recognizes the issue immediately).

@JoshuaLampert Could you please check whether this is still present/an issue? I suppose it shouldn't after switching to the new version of P4est.jl.

Yes, the MWE from above works now (with some adaptions to the new version of P4est.jl):
julia> using Trixi

julia> redirect_stdout(devnull) do
           include("examples/p4est_2d_dgsem/elixir_advection_basic.jl")
       end

julia> unsafe_load(Ptr{Trixi.p4est_tree}(unsafe_load(unsafe_load(mesh.p4est).trees).array), 1)
P4est.LibP4est.p4est_tree(P4est.LibP4est.sc_array(0x0000000000000018, 0x0000000000000004, 128, Ptr{Int8} @0x00000000062f16d0), P4est.LibP4est.p4est_quadrant((0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x1d, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff)), P4est.LibP4est.p4est_quadrant((0xfe, 0xff, 0xff, 0x3f, 0xfe, 0xff, 0xff, 0x3f, 0x1d, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff)), 0, (0, 4, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, -1), 1)

julia> unsafe_wrap(Array, Ptr{Trixi.p4est_tree}(unsafe_load(unsafe_load(mesh.p4est).trees).array), 1)
1-element Vector{P4est.LibP4est.p4est_tree}:
 P4est.LibP4est.p4est_tree(P4est.LibP4est.sc_array(0x0000000000000018, 0x0000000000000004, 128, Ptr{Int8} @0x00000000062f16d0), P4est.LibP4est.p4est_quadrant((0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x1d, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff)), P4est.LibP4est.p4est_quadrant((0xfe, 0xff, 0xff, 0x3f, 0xfe, 0xff, 0xff, 0x3f, 0x1d, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff)), 0, (0, 4, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, -1), 1)

Would this also work with PW instead of the many unsafe loads, possible with a call to the pointer function for the unsafe wrap case?

I'm just interested if this is a use case where PW can help as well, or whether they are not well designed for it.

I would expect that this works with the PW. Let's keep this open until the PW are merged in P4est.jl and then I'll test it again with PW instead of the unsafe_loads or the unsafe_wrap.

The following works with P4est.jl v0.4.1:
julia> using Trixi, P4est

julia> redirect_stdout(devnull) do
           include("examples/p4est_2d_dgsem/elixir_advection_basic.jl")
       end

julia> PointerWrapper(mesh.p4est).trees.array
PointerWrapper{Int8}(Ptr{Int8} @0x000000000f0a95b0)

julia> PointerWrapper(mesh.p4est).trees.array[]
24

julia> unsafe_wrap(Array, pointer(PointerWrapper(mesh.p4est).trees.array), 1)
1-element Vector{Int8}:
 24

julia> unsafe_load(PointerWrapper(mesh.p4est).trees.array)
24

Thanks!


chore(sentry): configure

Current dependencies on/for this PR:

main

PR #2796   👈

This comment was auto-generated by Graphite.

@wittjosiah queued this pull request to merge with Graphite.

@wittjosiah merged this pull request with Graphite.


Running in ROS Melodic Ubuntu 18.04

Hi,
Is there a way we can run this repo on ROS Melodic Ubuntu 18.04 setup? or anyone had tried please let me know. Thanks

Good day,
This is an older version of a thesis project. Here you can find only the original draft of what we intended to do, in the end we had to cut down on the final product due to time constraints.
I have the final version set to private on my account, but i don't think is what you are looking for.
We had to tamper with the drone's controller in order for the python program to move it, and the tracking was done with a webcam throughout image processing.
I will ask @asmateus to set this project to private too in a couple of weeks given that the readme.md doesn't match with the content of the repo.
Here is a link of a summary that we did back in the day for the university.
https://manglar.uninorte.edu.co/handle/10584/7798


[FEATURE] Add support for inline diffs on pull requests

Is your feature request related to a problem? Please describe.
I would like to show the additions and deletions on a pull request to help gauge the size of the pull request.
Describe the solution you'd like
I would like to show additions and diffs in a similar way to Github by rendering additions in green and deletions in red e.g. +10 -5
Additional context
pull_request.addtions will return an interger for additions and pull_request.deletions will return an interger for deletions

Implemented in #123

closed by #123


Can someone show me the config settings for 1000 or 2500 nfts

If i generate 100 nfts it uses all folders and layers. If i generate more it only uses certain ones. So can someone show me what i'm doing wrong by showing the weights, editionsize and percentagelayer? Thanks in advance!

The outer limit of fully randomised nft's seems to be 100


Add to the ShowInstrumentInfo() method ability to output information to the file define by key --output

Add to the ShowInstrumentInfo() method ability to output information to the file define by key --output.
Also, change the example for --info key in READMEs.

Implemented in #9. Now you can use the --output key and define the file name where to save the received information. By default, the results are stored in info.md.
Example:
$ tksbrokerapi -v 10 --ticker IBM --info --output IBM-info.md

TKSBrokerAPI.py     L:3992 DEBUG   [2022-10-28 01:47:52,384] >>> TKSBrokerAPI module started at: [2022-10-27 22:47:52] UTC, it is [2022-10-28 01:47:52] local time
TKSBrokerAPI.py     L:4006 DEBUG   [2022-10-28 01:47:52,385] TKSBrokerAPI major.minor.build version used: [1.3.dev77]
TKSBrokerAPI.py     L:4007 DEBUG   [2022-10-28 01:47:52,385] Host CPU count: [8]
TKSBrokerAPI.py     L:210  DEBUG   [2022-10-28 01:47:52,385] Bearer token for Tinkoff OpenAPI set up from environment variable `TKS_API_TOKEN`. See https://tinkoff.github.io/investAPI/token/
TKSBrokerAPI.py     L:223  DEBUG   [2022-10-28 01:47:52,385] Main account ID [**********] set up from environment variable `TKS_ACCOUNT_ID`
TKSBrokerAPI.py     L:272  DEBUG   [2022-10-28 01:47:52,385] Broker API server: https://invest-public-api.tinkoff.ru/rest
TKSBrokerAPI.py     L:408  DEBUG   [2022-10-28 01:47:52,407] Local cache with raw instruments data is used: [/Users/tim/projects/TKSBrokerAPI/tksbrokerapi/dump.json]
TKSBrokerAPI.py     L:409  DEBUG   [2022-10-28 01:47:52,407] Dump file was last modified [2022-10-27 20:31:18] UTC
TKSBrokerAPI.py     L:1047 DEBUG   [2022-10-28 01:47:52,407] Requesting current prices for instrument with ticker [IBM] and FIGI [BBG000BLNNH6]. Wait, please...
TKSBrokerAPI.py     L:1366 DEBUG   [2022-10-28 01:47:52,533] Requesting current trading status, FIGI: [BBG000BLNNH6]. Wait, please...
TKSBrokerAPI.py     L:1372 DEBUG   [2022-10-28 01:47:52,628] Records about current trading status successfully received
TKSBrokerAPI.py     L:843  INFO    [2022-10-28 01:47:52,628] # Main information: ticker [IBM], FIGI [BBG000BLNNH6]

* Actual at: [2022-10-27 22:47] (UTC)

| Parameters                                              | Values                                                  |
|---------------------------------------------------------|---------------------------------------------------------|
| Ticker:                                                 | IBM                                                     |
| Full name:                                              | IBM                                                     |
| Sector:                                                 | it                                                      |
| Country of instrument:                                  | (US) Соединенные Штаты Америки                          |
|                                                         |                                                         |
| FIGI (Financial Instrument Global Identifier):          | BBG000BLNNH6                                            |
| Exchange:                                               | SPB_MORNING                                             |
| ISIN (International Securities Identification Number):  | US4592001014                                            |
| Class Code:                                             | SPBXM                                                   |
|                                                         |                                                         |
| Current broker security trading status:                 | Normal trading                                          |
|                                                         |                                                         |
| Buy operations allowed:                                 | Yes                                                     |
| Sale operations allowed:                                | Yes                                                     |
| Short positions allowed:                                | No                                                      |
|                                                         |                                                         |
| Limit orders allowed:                                   | No                                                      |
| Market orders allowed:                                  | No                                                      |
| API trade allowed:                                      | Yes                                                     |
|                                                         |                                                         |
| Type of the instrument:                                 | Shares                                                  |
| IPO date:                                               | 1915-11-11 00:00:00                                     |
| Payment currency:                                       | usd                                                     |
|                                                         |                                                         |
| Previous close price of the instrument:                 | 134.77 usd                                              |
| Last deal price of the instrument:                      | 134.5 usd                                               |
| Changes between last deal price and last close  %       | -0.20                                                   |
| Current limit price, min / max:                         | 132.87 usd / 136.13 usd                                 |
| Actual price, sell / buy:                               | N/A usd / N/A usd                                       |
| Minimum lot to buy:                                     | 1                                                       |
| Minimum price increment (step):                         | 0.01                                                    |

TKSBrokerAPI.py     L:852  INFO    [2022-10-28 01:47:52,629] Info about instrument with ticker [IBM] and FIGI [BBG000BLNNH6] was saved to file: [IBM-info.md]
TKSBrokerAPI.py     L:4329 DEBUG   [2022-10-28 01:47:52,629] All operations were finished success (summary code is 0).
TKSBrokerAPI.py     L:4336 DEBUG   [2022-10-28 01:47:52,629] >>> TKSBrokerAPI module work duration: [0:00:00.245039]
TKSBrokerAPI.py     L:4337 DEBUG   [2022-10-28 01:47:52,629] >>> TKSBrokerAPI module finished: [2022-10-27 22:47:52 UTC], it is [2022-10-28 01:47:52] local time


what is the Chinese pretrained-transformers used in the paper?

Thanks for the great work. Just wonder for the pre-trained Roberta-gen (Chinese version), which one do you use in the experiments for math23k.

Roberta for Chinese datasets, uer/chinese_roberta_L-12_H-768 https://huggingface.co/uer/chinese_roberta_L-12_H-768.
Bert for Chinese datasets, bert-base-chinese https://huggingface.co/bert-base-chinese.
GPT2 for Chinese datasets, uer/gpt2-base-chinese-cluecorpussmall https://huggingface.co/uer/gpt2-chinese-cluecorpussmall.
Thanks for your support!


Remove stale branches in git repo

Hello, while browsing the branches I found that a number of branches have been left in the repo.
It appears many are multiple years old, and some have already been merged. Would you consider removing the stale and/or merged branches as you have time?
Thank you so much for considering this request!

Absolutely. I'll try to give it a few cycles soon, I thought most open branches were due to old PRs that got ported and which are waiting to get merged ... but cleaning the backlog is definitely becoming increasingly necessary.

Looks like the stale branches have decreased. Closing now. Thanks @Anteru!


Supplement missing vi translations

I've just translated missing vi items from latest en.json

Thank you!


Aliased field in results pane gets data type of field of same name in the table

System information:

Windows
dbeaver <IP_ADDRESS>101032114

Connection specification:

PostgreSQL 11.6

Describe the problem you're observing:
This is admittedly a subtle edge case. If you do a SELECT query against a table and alias a field in the query to the same name as another field in the underlying table, the results pane viewer/editor UI seems to assume the field is of the data type of the underlying actual field (not the aliased field). This is noticeable, for example, when the actual underlying named field is an integer type but you are aliasing a varchar field to its name. Thus, the varchar field assumes the integer type in the editor which causes the value to display incorrectly and not be editable in the results pane grid.
Steps to reproduce, if exist:
Sample query:
CREATE TABLE my_table (
id int4 not null,
foo varchar not null
);
INSERT INTO my_table ( id, foo ) VALUES ( 1, 'test' );
SELECT foo as id
FROM my_table;

thanks for the report
duplicates #11143


Resurrect old galaxy autoencoder architecture

This PR resurrects the old galaxy autoencoder architecture before I added the changes in #323. Specifically, we use the architecture in the commit 382fccfd51f617150d9cfd595272d86b48c88d89.
There are a few reasons for exploring this change. It's clear that there is a performance regression for the regular autoencoder on the real SDSS data. When I implemented #323, I hadn't yet run the full pipeline, so these changes weren't optimized for real-data performance. For @ismael-mendoza's paper, I think it is prudent to restore the architecture to where it was before. It seems to be more robust to moving to real-data.
Second, I want to see how well the VAE does with minimal changes to the original architecture. I've got some promising initial results just by doubling the dimension of the encoder and applying F.softmax to the second-half. I'd like to see how well this does on real data.
This doesn't mean that the new architecture is going away. Rather, I'd like to add back the new architecture improvements one-by-one and only keep them if they improve performance on both real data and single galaxies.

Here's the results on the autoencoder (not the VAE):

On the AE, we see that there's poor performance on galaxies with magnitudes between 19.5 and 21.5. I think this is because many galaxies are being mis-classfied as stars. Likely caused by #435, because nothing else should have changed in the estimation procedure.

Actually, the AE results are better than I thought. On #435 I just used a constant background value (865.0) in predict.py since we were intending on removing that code. After passing the real SDSS background values, detections are pretty good now!

The VAE is still having the same issue as before:

These images are from using MAP estimates for galaxy parameters instead of sampling

Hi @dereklhansen - I really appreciate you making this PR! This will allow me to get back to producing results a lot faster :)
I have attached the old metrics below. It seems like the recall improved somewhat (except for a dip around 21.5) and the precision is mostly the same except for that dip around the values you mentioned. Do you understand that dip? Is it because of the background change? Interestingly there was no dip in precision in your first set of plots

@dereklhansen - feel free to merge this PR sooner rather than later if this is the only issue remaining and I can help investigate that dip as well.

It seems like the recall improved somewhat (except for a dip around 21.5) and the precision is mostly the same except for that dip around the values you mentioned. Do you understand that dip? Is it because of the background change? Interestingly there was no dip in precision in your first set of plots for the AE

I think the dip in precision is because the LocationEncoder is finding more "objects" that are not detected by SDSS. If you look at the VAE plots, which have the locations plotted, there are several locations where there isn't an SDSS object but we label it as one. You can also see this happening if you look closely at the AE plots; there are a few reconstructions from very faint / invisible objects.

I think I finally solved the main problem with the VAE. While padded tiles without galaxies do not factor into reconstruction loss, I was evaluating the divergence loss (q(z | x) - p(z)) at every padded tile. This was encouraging q(z|x) to be close to the prior p(z) whenever x was an empty image. This is why the decoder got so unstable when the galaxies were so faint.

This fixes the VAE - the only trouble spots are where the LocationEncoder + BinaryEncoder identified blends where there aren't any.

@dereklhansen - feel free to merge this PR sooner rather than later if this is the only issue remaining and I can help investigate that dip as well.
[EDIT] Although I see you are investigating things with the VAE so please take your time.

I think at this point the only issue is the dip, so I'll merge this.


Move concurrent activation information from consul to controller

Currently, the controller writes out the number of completed activations (per use) and so does every invoker.  This creates a lot of continuous traffic on consul and I believe can cause latency issues.  Now that active ack is in place, the invoker can stream back completion as they occur instead of writing a large piece of information for the state.  This has the benefit of reducing traffic, making consul healthier, and centralizing information in the load balancer.
@jeremiaswerner As discussed before, we want to reduce consul traffic.  The proposal that @rabbah and I arrived at is the following.   @ChrBi @markusthoemmes  Any thoughts?

The controller extends the outstanding activation map to include non-blocking as well as blocking calls.  Both types are already part of the active ack from the invoker so no work needed there.
A secondary map based on user id can then be maintained in the controller giving the number of concurrent activations.  This also optimizes the responsiveness of the throttle because the load balancer immediately has information to this rather than enduring consul latency.
The N invokers no longer writes any such information to consul - thus greatly decreasing the load.  The controller only writes the digested information (queued activations per user) to consul.  This also improves monitoring.
As a future option, centralizing the notion in the controller also allows the creation of an admin endpoint that would allow manual un-wedging of a user.
RAS:  This is the hard part.  What if the invoker's active ack fails?  From the controller's perspective, an activation was issued but it did not hear back.  This will be rare but we must be robust against this.  Also we must consider the case when the invoker is restarted.

Initial idea.  Use the global action timeout (or some factor thereof) as a proxy for when the invocation has already been completed.   This is no worse than the existing system in that we could fail to write to consul and have the same issue.  However, repeated lost acks can cause unbounded queue growth so we don't like this.

New idea.  Use a revised version of the original protocol.  We extend the active ack with a count for actions completed for that user as well as an indication of invoker restart by adding the timestamp for when the invoker started.  Since there is a count reported, a few active ack will not affect the cumulative count.  If the timestamp varies indicating a reboot, we reset the count for that user+invoker pair.

Additional benefit:  The monitoring service does not have to fetch a huge volume of data repeatedly which probably also hurts consul.  Since the controller will reflect its view of concurrent activity, this can be used by the UI directly and we can avoid logic getting out-of-sync.
If we do this, I think it should proceed in 3 phases:

This first step should immediately make things better.  Decrease the frequency of consul updates by the invoker so we have some breathing room.  Since we don't want extra connection and also don't want health updates to be slower, we can simply do the per user count writes on every other iteration of consul posting while doing health updates on every iteration.
This creates the appropriate abstraction for the monitoring service.  Have the controller put into consul its view of concurrent activity.  The monitoring service should look only at these values and not at the old controller values and not anything from the invoker.
Intermediate step to confirm that adding 2 extra (small) fields will not impact performance  Extend the active ack with the additional fields as outlined in "New idea" above.
Cutover to new logic and decrease consul traffic  Change the controller to use active ack and maintain the new maps and no longer poll consul for invoker information.   Delete the posting of per counts by the invokers.

Steps 1 and 2 are completed.   Steps 3 and 4 will probably be done in parallel so we can start reducing consul load as there is evidence that occasional lags and slow performance in the invoker and loadbalancer is resulting from consul overload.

@perryibm wrt. monitoring, I'am wondering if we might want to write log's for the queue length by using markers, i.e. if the controller calculates the overall queue length for the throttles it could just write a log. We could then directly gather the monitoring information out of the logs, rather than querying consul from the monitoring service. @cbickel what do you think?

Either way is fine with me once we complete the existing items.

We'd need to add statsd-gaiges to the markers. But I can have a look a it in the next days.

The invoker no longer writes activation counts (global or per subject) nor does the controller read this of course.   Instead, the controller maintains the state and publishes the digested information for the monitor to read by using the activation maps it maintains.  This hugely reduces consul traffic volume taking it down by a factor of however many users have ever run an activation since deployment.


Add support for automatic distribution to local file system

Add support to automatically distribute generated files to different local file locations.

Implemented in https://github.com/monstermichl/ninja-bear-distributor-fs


✨ browsr refactor

This PR performs a substantial refactor on browsr.
The app is now broken out into more explicit pieces and makes use of messaging between components instead of funky  references through the app attribute.
This PR should make it easier to expand browsr's functionality down the road.
Additionally, this PR implements some new features:

c binding to copy file path
r binding to refresh the file
double click functionality on the directory tree

:tada: This PR is included in version 1.19.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Pipeline default/approval-pipeline can't be Run; it contains Tasks that don't exist: Couldn't retrieve Task "approvaltask": tasks.tekton.dev "approvaltask" not found

I have followed the documentation and video but after creating pipeline when I try to run the pipeline run I am facing the below error.
Please help me resolving the issue. As per my understanding I dont require to create approval task help me understand more.
I am using latest code
Error:
Pipeline default/approval-pipeline can't be Run; it contains Tasks that don't exist: Couldn't retrieve Task "approvaltask": tasks.tekton.dev "approvaltask" not found

looks like you haven't registered the CRD of ApprovalTask - https://github.com/automatiko-io/automatiko-approval-task#install-approval-tasks-crd
Note that this requires cluster admin rights.
In addition, please provide versions of:

Tekton
which custom task api version you use - v1alpha1 or v1beta1
automatiko approval task version

Hello @mswiderski ,
I have created CRD of approval task, Do I need to modify anything in the CRD of approval task
approvaltasks.tekton.automatiko.io                    2023-04-10T08:46:28Z

Tekton versions:

Dashboard:  v0.34.0
Pipelines: v0.46.0
Triggers:  v0.22.1

Do I need to configure custom task ?
Can you please provide me with the Yaml file.

But i have tried creating custom task for approval task but its asking for output reference to the approved task.
Not sure whats wrong, For Approval task reference do we need to create approval task or CRD will create it one for you?

automatiko approval task version I am using the code from latest which is tag v0.7.0

Can you share the pipeline file you used?

Hello @mswiderski
Please find the pipeline file
apiVersion: tekton.dev/v1beta1 kind: Pipeline metadata:   name: testpipeline2 spec:   tasks:     - name: deploy-dev-approval-task       taskRef:         apiVersion: tekton.automatiko.io/v1beta1         kind: ApprovalTask         name: approvaltask       params:         - name: pipeline           value: "$(context.pipelineRun.name)"         - name: description           value: "Approval from pipeline $(context.pipeline.name) for development environment"         - name: approvers           value:             -<EMAIL_ADDRESS>   - name: deploy-dev-approved-task       when:         - input: $(tasks.deploy-dev-approval-task.results.decision)           operator: in           values: [ "true" ]       taskRef:         name: print-decision       runAfter:         - deploy-dev-approval-task       params:         - name: decision           value: "APPROVED"         - name: comment           value: $(tasks.deploy-dev-approval-task.results.comment)     - name: deploy-dev-rejected-task       when:         - input: $(tasks.deploy-dev-approval-task.results.decision)           operator: in           values: [ "false" ]       taskRef:         name: print-decision       runAfter:         - deploy-dev-approval-task       params:         - name: decision           value: "REJECTED"         - name: comment           value: $(tasks.deploy-dev-approval-task.results.comment)

ok, so this looks like the default test pipeline. Could it be you are creating it in different namespace? The CRD is namespaces, so you need to make sure where you create pipeline is where you created approval task CRD.
Then the same goes for deployment of the controller which will then watch only selected namespaces, by default it watches default namespace.
https://github.com/automatiko-io/automatiko-approval-task/blob/main/k8s/v1beta1/kubernetes-basic.yml#L171

can you also look of custom task are enabled, see the config map https://github.com/automatiko-io/automatiko-approval-task#configure-tekton
approval task are based on Tekton's custom task so it is Tekton that transforms it to custom run and then approval task controller is triggered for the custom run.

Yea its enabled its value as #custom-task-version: v1beta1
as per the latest code
Please edit the object below. Lines beginning with a '#' will be ignored,
and an empty file will abort the edit. If an error occurs while saving this file will be
reopened with the relevant failures.

apiVersion: v1
data:
await-sidecar-readiness: "true"
custom-task-version: v1beta1
disable-affinity-assistant: "true"
disable-creds-init: "false"
enable-api-fields: stable
enable-provenance-in-status: "false"
enable-tekton-oci-bundles: "false"
enforce-nonfalsifiablity: none
require-git-ssh-secret-known-hosts: "false"
resource-verification-mode: skip
running-in-environment-with-injected-sidecars: "true"
send-cloudevents-for-runs: "false"
kind: ConfigMap
metadata:
annotations:
kubectl.kubernetes.io/last-applied-configuration: |
{"apiVersion":"v1","data":{"await-sidecar-readiness":"true","custom-task-version":"v1beta1","disable-affinity-assistant":"false","disable-creds-init":"false","enable-api-fields":"stable","enable-provenance-in-status":"false","enable-tekton-oci-bundles":"false","enforce-nonfalsifiablity":"none","require-git-ssh-secret-known-hosts":"false","resource-verification-mode":"skip","running-in-environment-with-injected-sidecars":"true","send-cloudevents-for-runs":"false"},"kind":"ConfigMap","metadata":{"annotations":{},"labels":{"app.kubernetes.io/instance":"default","app.kubernetes.io/part-of":"tekton-pipelines"},"name":"feature-flags","namespace":"tekton-pipelines"}}
creationTimestamp: "2022-11-29T11:09:09Z"
labels:
app.kubernetes.io/instance: default
app.kubernetes.io/part-of: tekton-pipelines
managedFields:

apiVersion: v1
fieldsType: FieldsV1
fieldsV1:
f:data:
.: {}
f:await-sidecar-readiness: {}
f:disable-creds-init: {}
f:enable-api-fields: {}
f:enable-provenance-in-status: {}
f:enable-tekton-oci-bundles: {}
f:require-git-ssh-secret-known-hosts: {}
f:resource-verification-mode: {}
f:running-in-environment-with-injected-sidecars: {}
f:send-cloudevents-for-runs: {}
f:metadata:
f:annotations: {}
f:labels:
.: {}
f:app.kubernetes.io/instance: {}
f:app.kubernetes.io/part-of: {}
manager: kubectl-client-side-apply
operation: Update
time: "2022-11-29T11:09:09Z"
apiVersion: v1
fieldsType: FieldsV1
fieldsV1:
f:data:
f:enforce-nonfalsifiablity: {}
f:metadata:
f:annotations:
f:kubectl.kubernetes.io/last-applied-configuration: {}
manager: kubectl
operation: Update
time: "2023-04-04T13:42:50Z"
apiVersion: v1
fieldsType: FieldsV1
fieldsV1:
f:data:
f:custom-task-version: {}
f:disable-affinity-assistant: {}
manager: kubectl-edit
operation: Update
time: "2023-04-10T08:04:47Z"
name: feature-flags
namespace: tekton-pipelines
resourceVersion: "93329540"
uid: 0b4f0de0-61e8-4c7c-803a-3d4fac010c68

Are you available for meeting ?
Can you provide your email address and available time ?

glad to hear things moved forward, thanks for the update.
In general tasks are looked up based on approvers, meaning if you have your pipeline setting with approver or group then you need to provide this via the form or query parameters when using the direct link to task.
https://github.com/automatiko-io/automatiko-approval-task#approval-forms
So in general, using the form you need:

provide the pipeline run name
task name - from pipeline
user name/email if you used approver in pipeline
group name if you used groups in the pipeline

Additionally, if you look at the approval task created in Kubernetes you will find there is approverUrl which gives you direct link to the task. then you need to append following query param:

?user=username/email  if you used approver in pipeline
?group=groupname  if you used groups in the pipeline

When you integrate this with emails the link will be fully functional and will not need to add the query param.

Great!
After appending the user i am able to approve the task

But I need guidance on Email approval
I have provided all the parameters
Below parameters are mandatory, Do I need to create a configmap using /template folder or only approval-email file ?
What is the below parameters mailer password and instance end ?
- name: QUARKUS_MAILER_PASSWORD
value: password
- name: QUARKUS_AUTOMATIKO_ON_INSTANCE_END
value: "keep"
- name: QUARKUS_AUTOMATIKO_TEMPLATES_FOLDER
value: "/templates"
Lastly Can you tell me what all the procedure I have to follow for Email intergration ?

Have a look at readme for configuring emails: https://github.com/automatiko-io/automatiko-approval-task#configure-email-server
It has also section about templates but you only need it if you want to change the default templates used for email or forms.
instance end is configuration to keep completed approvals in the system for as long as associated pipeline run is in the system. So this one should stay like that.

@rajesh-carelon have you managed to get this working?

closing due to no response from reporter


Update code to use mixin.detect

Use mixin.detect instead of what we currently do in https://github.com/softlayer/sl-ember-components/blob/c79f24b99c8850eba970938c669f592af1e4ad1b/tests/unit/mixins/sl-input-based-test.js#L11 and https://github.com/softlayer/sl-ember-components/blob/c79f24b99c8850eba970938c669f592af1e4ad1b/tests/unit/mixins/sl-tooltip-enabled-test.js#L37.
Mixin.detect is a more accurate test. Mixin.detect performs checks to see if the mixin is present within the object.

@notmessenger closing this. For testing if mixin can be mixed in, we should check if object creation was successful after extending mixin. For checking if mixin is present on component we should use mixin.detect. The cases above we are not checking if components have the mixin.


showPopup: false - no match in Android 6.x

Option showPopup: false causes errors in Android 6.
First use is OK - I get list of matches, but the second and every next try - it fails.
There is an immediate response: No match (it's not even waiting for me to stop talking).
Few times there was also RecognitionService busy info.
With showPopup: true everything works just fine.
On Android 5 everything works fine in both cases. I was trying en-US and pl-PL.
I use the newest version of the plugin and Cordova.
Android 6 - Samsug Galaxy A5 model 2016 - Android 6.0.1
Android 5 - Samsung Galaxy S4 - Android 5.0.1

Hi all, I have the same issue!
Android 6 - Samsug Galaxy S5 - Android 6.0.1
Please help, I'm trying to debug the problem

Found the issue (in my scenario).
Google App old version.
Just updated the Google App with a last version and all good.


Select the first line automatically

Can I automatically select the first line of the typeahead, when an user press the return or tab buttons?

Looking for that as well.

Please use that:
$('#address').bind("typeahead:autocompleted", addressPicker.updateMap);


bigquery: Add option for disabling legacy SQL

Fixes #281

I signed it!
On Fri, Jul 29, 2016 at 9:06 PM, googlebot<EMAIL_ADDRESS>wrote:

Thanks for your pull request. It looks like this may be your first
contribution to a Google open source project. Before we can look at your
pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/
https://cla.developers.google.com/ to sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll
verify. Thanks.

If you've already signed a CLA, it's possible we don't have your
GitHub username or you're using a different email address. Check your
existing CLA data https://cla.developers.google.com/clas and verify
that your email is set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.
If you signed the CLA as a corporation, please let us know the
company's name.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/gcloud-golang/pull/304#issuecomment-236329162,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGpFivjsSNmSMm1hcc_LFs0SZyPKZ-Eks5qaqOKgaJpZM4JYr8Z
.

We are using Gerrit for code reviews, could you go through the instructions at https://github.com/GoogleCloudPlatform/gcloud-golang/blob/master/CONTRIBUTING.md and contribute the change?


Mock the Active Bridges Set

Create a mock of the Active Bridges Set in the WBI solidity contract.
This set is used to represent the number of actives nodes acting as Bridges between Ethereum and Witnet.

Closed because of #26


Added AutoResolve for SqsQueueTriggerAttribute

Hi, I've added the attributes! If it's ok with you, could you please merge it and publish a new nuget version?
Closes #35

Thank you for your interest in this project. Let me go over them...

Looks good to me.


Add shuffle and sample

Reference implementations:
https://github.com/jashkenas/underscore/blob/master/underscore.js#L324-347

Can random/sample be used now in ramda?

Can random/sample be used now in ramda?

I don't believe there is any random number generating library associated with ramda. I've used http://chancejs.com/ before but don't really know much about it.

@kedashoe not exactly number generator, but pick random element from array. Shuffle is too not in the ramda

Hi @idchlife
I would like ramda to pick up an random element from array too.
So I created a small Math library to me https://github.com/angeloocana/ptz-math
After reading this whole issue, I think It needs an randomWith like ramda has memoizeWith.

@angeloocana the funny thing, pure function can be used even for random number/element of an array. There are implementations with passing randomizer object or even "random" number (timestamp) as second argument, making function pure by definition.
Same randomizer passed - same result will be in return. It's for developer to provide randomizer, of course. But function in ramda will stay pure.

https://github.com/Risto-Stevcev/pure-random might be of interest here. Stumbled across it at #1677

hmm, interesting reading around this subject :)
here's a simple Knuth Fisher-Yates shuffle function, adapted from https://bost.ocks.org/mike/shuffle/
It requires a random number generator of your choice, default is Math.random
Realised I have no idea how to go about testing whether a function produces a random output!
const shuffle = function (random, list) {
	// default random function
	random = R.defaultTo(Math.random, random);

	let m = list.length, t, i,
		// work on clone of the list
		listout = R.clone(list);

	// work from end to beginning of the list
	// 'shuffled' elements start filling from the back
	while (m) {
		// select index of a non-shuffled element
		i = Math.floor(random() * m--);
		// shuffle the elements by swapping them
		t = listout[m];
		listout[m] = listout[i];
		listout[i] = t;
	}
	return listout;
}

// demo
let aLetters = ["a", "b", "c", "d", "e"],
shuffleLetters = () => console.log(shuffle(null, aLetters));
// repeat the output
R.times(shuffleLetters, 10)

example output
["d","e","b","c","a"]
["b","d","e","c","a"]
["a","d","b","e","c"]
["c","d","a","b","e"]
["d","c","a","b","e"]
["b","a","c","d","e"]
["e","c","a","d","b"]
["b","e","a","d","c"]
["c","e","d","a","b"]
["e","d","b","a","c"]

ahah, just come across the random shuffle extension that was removed at https://github.com/ramda/ramda/commit/992aff2c0ad3fdb57a8289616c839acafcfb3222
const shuffler = R.curry(function(random, list) {
         var idx = -1;
         var len = list.length;
         var position;
         var result = [];
         while (++idx < len) {
             position = Math.floor((idx + 1) * random());
             result[idx] = result[position];
             result[position] = list[idx];
         }
         return result;
     }),
shuffle = shuffler(Math.random);

shuffle(R.range(0, 5))


Ridiculously small typo fix

Many thanks for all your efforts.
Cheers

Thanks.  Since I have sync'ed (no pun intended) the code base of psd and asd, a little one like this was bound to slip in.


[Bug?]: Docs Build is Broken

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
I get the error:

[vite-server-references] Invalid left-hand side in prefix operation. (1:2)
file: /Users/ryancarniato/Development/solid-start/docs/routes/api/GET.md?pick=getFrontMatter:1:2
ERROR  Invalid left-hand side in prefix operation. (1:2)                                                             4:15:01 PM
at constructor (node_modules/.pnpm/@babel+parser@7.23.9/node_modules/@babel/parser/lib/index.js:353:19)

Expected behavior 🤔
That the docs build
Steps to reproduce 🕹
pnpm docs:build
works fine in dev
Context 🔦
As far as I can tell this began when I started doing single flight. But only seems to effect the docs transformation.
Your environment 🌎
No response

I'd try removing the twoslash from the code block on the GET.md file

fixed by f73de3a


chore(brew): Add a retry to the installation script

I notice the installation script fails sometimes with:
[09:24:58 g.i/s/wayblue-hyprland-main-userns-hardened:br-staging-40] => error: RPC failed; curl 92 HTTP/2 stream 5 was not closed cleanly: CANCEL (err 8)
[09:24:58 g.i/s/wayblue-hyprland-main-userns-hardened:br-staging-40] => error: 6412 bytes of body are still expected
[09:24:58 g.i/s/wayblue-hyprland-main-userns-hardened:br-staging-40] => fetch-pack: unexpected disconnect while reading sideband packet
[09:24:58 g.i/s/wayblue-hyprland-main-userns-hardened:br-staging-40] => fatal: early EOF
[09:24:58 g.i/s/wayblue-hyprland-main-userns-hardened:br-staging-40] => fatal: fetch-pack: invalid index-pack output
[09:24:58 g.i/s/wayblue-hyprland-main-userns-hardened:br-staging-40] => Failed during: /usr/bin/git fetch --quiet --force origin

a retry could improve reliability

The problem happens somewhere inside this homebrew install script. I think that the issue needs to be upstreamed to them or a patch to the installer would have to be added to BlueBuild.

@KernelFreeze
I made the upstream issue:
https://github.com/Homebrew/install/issues/890


blender live link

any chance for a blender 3d live link

Can you describe what functionalities it would have?

sorry for late replay.  I was thinking something like select a 3d mesh in blender with uv(or if possible without uv)  and one click send the model to material maker, do the painting part and send the mesh back to blender with baked maps.

I'm not sure if this is what you mean, but here is one - although I'd argue it might make sense as a feature of material maker
https://blenderartists.org/t/material-maker-linker/1443394


Bundle genesis files for public networks

Currently, we need to configure genesis file paths. But for default networks, we can bundle these genesis files with yaci store binary.

Implemented as part of #105

When a custom genesis file path is set with classpath prefix, it doesn't work.


Fix CI

Use Travis for CI and fix the tests (hopefully :pray:).

Huh, so the failure is because it can't sign the test app. I guess we'll have to add the cert to the repo? Or something?

Does it need to be codesigned?

IIRC part of the testing is signature verification so without that we're cobbling the tests.

:metal: It's at least building and running tests now.

Well, all the tests are passing except that some will randomly fail:

"The application “TestApplication” could not be launched because a miscellaneous error occurred (OSStatus -10810)."

I haven't seen this locally so I kinda think it's a Travis-ism. I'll keep poking around, but if anyone has any ideas throw the out there :pray:

Well, I :doughnut: know what's going on.
Unless anyone else has ideas, my current plan is to detect when we're running tests on Travis and disable those tests :cry:

This is ready, but


Send text to the connected terminal from kakoune

Would it be possible to send text from kakoune to the connected terminal ?
My initial idea is to use it as an interactive REPL to send the current selection + 'return' to ipython for e.g. But I think it can have a lot of other uses.

https://github.com/mawww/kakoune/tree/master/rc/windowing/repl

Wow didn't know this feature, that awsome thx !


markdown link to file highlight fails for paths with spaces

VSCode Version: 1.50.1-1
OS Version: Manjaro 20.1.1 Mikah

Steps to Reproduce:

open a new markdown file;
write something such as  [text](<path with spaces>) *;
notice how that is highlight differently compared to other links in the form [text](path_without_spaces).

(*angle brackets should allow for links with spaces)
Similar issue: https://github.com/Microsoft/vscode/issues/24763#issue-221783835
Except there the syntax was not complying with the CommonMark spec, while this should.
Again, links work in the PDF visualization, this is a text highlight issue, not a rendering one.

Does this issue occur when all extensions are disabled?: Yes

I am unsure exactly how to tackle this one but I think editing https://github.com/microsoft/vscode-markdown-tm-grammar/blob/master/markdown.tmLanguage.base.yaml#L424
          (<?)((?<url>(?>[^\s()]+)|\(\g<url>*\))*)(>?)      # The url

and changing it to:
          (<?)(.*?)(>?)      # The url

might fix the issue (and updating the corresponding captures)? If this is sufficient I can open an MR tackling this issue too.
FYI using https://github.com/textmate/markdown.tmbundle/blob/master/Syntaxes/Markdown.tmLanguage#L241 as inspiration

Fixed by 2f9bda284d391f97982f7932ba49eb89b2ac83b1


Include pretrained weights in repo[examples]

@mzouink We accomplished this, right?

yes we have cosem finetune


tuic listen to only ipv6

in my configuration even that i have disabled ipv6 but it only listen to ipv6
tuic-server:
  enable: true
  listen: <IP_ADDRESS>:3478
  token:
    - mytok
  certificate: ../ssl/a.crt
  private-key: ../ssl/a.key
  congestion-controller: bbr
  max-idle-time: 15000
  authentication-timeout: 1000
  alpn:
    - h3
  max-udp-relay-packet-size: 1500

INFO[0000] Tuic proxy listening at: [::]:3478
lsof -i:3479
COMMAND      PID USER   FD   TYPE  DEVICE SIZE/OFF NODE NAME
clashmeta 561194 root    8u  IPv6 2923253      0t0  UDP *:3479

@hiddify1 Go will listen on dual stacks by default

yes but as you can see it "only" listen to ipv6 in clash i do not know why
i can not use ipv4 in it :(

yes but as you can see it "only" listen to ipv6 in clash i do not know why
i can not use ipv4 in it :(

It doesn't affect your use of ipv4.

You are right.
It was my misconfiguration in the client side


fix: Use rosetta2 instead

Fixed packages:

https://github.com/b4b4r07/gist
https://github.com/b4b4r07/git-bump
https://github.com/b4b4r07/github-labeler
https://github.com/b4b4r07/gomi
https://github.com/b4b4r07/iap_curl
https://github.com/b4b4r07/stein

Thank you for your contribution!

Released v1.6.0 🎉


Lower minimum version for Faraday to increase compatibility

Faraday is required with a minimum version of 0.13
Some other gems (such as oauth)  in our projects require the version to be <= 0.10, so we couldn't start working with this gem.
I forked the project and used a requirement of Faraday 0.10 and there don't seem to be any negative consequences.

I released a new version which requires Faraday >= 0.10.0


Pin Dask to avoid 0.18.2 bug with masked arrays.

Basically, hacking various things to get the tests passing again.
It starts by pinning dask, but that uncovered multiple other problems which have crept in since we merged stuff with failing tests ...

@lbdreyer I hope this fixes all the issues raised.
I :trumpet: have now tracked down the key problem + raised it against Dask here .

#3134  is in, can you rebase the 2d_coords branch?

#3134 is in, can you rebase the 2d_coords branch?

We shouldn't rebase a feature branch if we can avoid it.
The correct thing to do is rebase this, then merge it.
I'll see whether that works out ok ...

Ok @lbdreyer now rebased to bring in the recent tests fix from #3134
Also addressed your most recent comments (I hope).
Please re-review.

We shouldn't rebase a feature branch if we can avoid it.
The correct thing to do is rebase this, then merge it.

No, it is right to rebase the feature branch.
Otherwise it won't merge back cleanly in future.
Will come back here after doing that ...
The current state here is a mistake.

Ok, now rebased onto the new 2d_coords branch, lately updated from master to include #3134
Please re-check @lbdreyer

Okay looks all fine :+1:


CNPHI Export Header Issues - sequencing centre, primary specimen id, #version

Issues with DataHarmonizer -> CNPHI Export Headers:

Does not include "Sequencing Centre" field which is mandatory for CNPHI upload.
CNPHI will not accept the field name "primary specimen identification number", it is expecting "primary specimen id".
CNPHI will not accept uploads that have the DataHarmonizer version information, e.g. "#validated using data harmonizer version 0.13.1". At least, not on the header row.

Can put a hold on this until we meet with CNPHI on Friday 2020-Oct-16.
Will update then.

Updates - we need to implement to following changes:

Include a "Sequencing Centre" field.
Change "Primary Specimen Identification Number" to "Primary Specimen ID".
Remove the "#validated using data harmonizer version 0.13.1" etc. output from the header row. Lev is going to send me a field name that they use for additional comments that we can output that information into for each observation.

@cmrn-rhi will add a "Sequencing Centre" picklist to the "DataHarmonizer Templates" document that will be used to populate the new field.

I did that already!
And will make change to code to handle the "#validated using ... " message better. We need a way to control, for each template or export format, whether it is added, and under what column label.

Ah, though I see you mention "picklist" so setting up some values for the field.  Its a select field then.

Yes! I just made it a select field and added the list from the CNPHI template to the DataHarmonizer templates.

At moment test data doesn't validate on the new Sequencing Centre selections. But I have all the field changes working except waiting for new comments field for version.

I've set it up so that on any template, we can specify a field of datatype = provenance which at moment then receives a "DataHarmonizer vX.Y.Z" value in first row of that column's data - but only if you press "Validate".  So we can talk about what kind of functionality tweaks we should have for this, i.e. do we

want dataharmonizer version to be included anytime someone saves the file, or just updated when someone presses Validate?
want the version info to be written to every row of output data?
One can actually put other content in the provenance field.  Version will always be inserted before that content.
The specification is shown in the CanCoGen Covid19 tab, on line 120.
This is demonstrated on the multi-template DataHarmonizer website.

Resolved that version info is written to each row of output data.

Having an issue with primary specimen id now as there appear to be some leading characters that make it unrecognizable to CNPHI. I can see these chars when view with UTF-7 encoding.

leading characters = ï»¿

Can you send me the test file containing this?  Is it in header row?  Not seeing it in some saved Laser dataset.

Sent via SLACK.
I'm also seeing those leading chars in files I saved from DataHarmonizer yesterday. Looking at some previous DH1 testing I did (October 6th, 2020) and neither the saved files nor CNPHI export had these leading characters in them.

Ok, so in the current javascript generated export file, there is a BOM header that is often included with UTF-8 to indicate that the file is encoded in UTF-8.  Unpleasant that CNPHI objects to the presence of this on import because these same characters enable Excel to recognize a comma delimited file as such. I'm just checking that out and determining whether those characters can be safely dropped or if we have to tailor something just for the CNPHI export.

Technical note: I’m thinking of switching file export output for ".csv" and ".txt" files to UTF-16 with no BOM.  Looks like the easiest way to do that is have “txt” as the booktype rather than “csv” (while keeping the delimiters appropriate for csv and tsv).

Solution was to create a special .csv output format encoded as UTF-8 but without the BOM.  Awaiting test on whether that works with character sets going into CNPHI. Alternately we force output format to ASCII.

CNPHI now accepts the import; however, they don't appear to accept symbols in UTF-8 (at least not _greater than or equa_l to, and degree)
CNPHI Upload Error:

Have emailed NML/CNPHI liaison outlining these issues and asked:

Does CNPHI require ASCII text?
Are there other acceptable characters for mathematical logic?
Would it be possible for CNPHI to accept the BOM?

This might be fixed now with a .csv (ASCII) format save. Awaiting testing.

.csv (ASCII) format has been tested and there have been no further issues.


feat(fields): read/set all the fields from the given struct

Summary by Sourcery
Implement a new feature to iterate over struct fields with options for setting and getting values, including recursive traversal and type conversion. Refactor existing code to improve struct value handling. Add new tests and a dependency for testing.
New Features:

Introduce a new functionality to iterate over struct fields, allowing setting and getting values with options for recursive traversal, type conversion, and handling of nil structs.

Enhancements:

Refactor the Set function to utilize a new helper function reducedStructValueOf for cleaner handling of struct value extraction and kind chain reduction.

Build:

Add a new dependency on github.com/davecgh/go-spew for testing purposes.

Tests:

Add comprehensive tests for the new iterate functionality, covering various scenarios including setting values, handling unexported fields, and recursive struct traversal.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

180 of 226 (79.65%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-5.0%) to 92.317%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

fields/iterate.go
103
105
98.1%

fields/path.go
9
27
33.33%

internal/reflect/iterate.go
46
72
63.89%

Totals

Change from base Build<PHONE_NUMBER>4:
-5.0%

Covered Lines:
745

Relevant Lines:
807

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

180 of 226 (79.65%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-5.0%) to 92.317%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

fields/iterate.go
103
105
98.1%

fields/path.go
9
27
33.33%

internal/reflect/iterate.go
46
72
63.89%

Totals

Change from base Build<PHONE_NUMBER>4:
-5.0%

Covered Lines:
745

Relevant Lines:
807

💛  - Coveralls

@sourcery-ai review

@sourcery-ai review


Pin Scala LTS version for Steward

@mkurz Do you know, why Steward haven't created update to 3.4 yet? 🤔

@mkurz Do you know, why Steward haven't created update to 3.4 yet? 🤔

hmm.. maybe because in #751 it upgraded to from 3.3.1 to 3.3.3 and maybe on next visit it would then upgrade to 3.4.
I think scala-steward first tries to upgrade to latest patch version, then it upgrades to latest minor version and so on. But not 100% sure.

I have the same assumption. Anyway this pin will be useful for us 😄

@Mergifyio backport 1.6.x

@mkurz Do you know, why Steward haven't created update to 3.4 yet? 🤔

Here is the solution: https://github.com/scala-steward-org/scala-steward/blob/ddfdff8d6a9f587f4bc452a6cf10cb76734fbaae/modules/core/src/main/resources/default.scala-steward.conf#L33-L36
updates.ignore = [
   ...

  // No upgrades to non LTS versions
  { groupId = "org.scala-lang", artifactId = "scala3-compiler",     version = { prefix = "3.4." } },
  { groupId = "org.scala-lang", artifactId = "scala3-library",      version = { prefix = "3.4." } },
  { groupId = "org.scala-lang", artifactId = "scala3-library_sjs1", version = { prefix = "3.4." } },

  ...

Cooool! 😄 Don't need to pin LTS in all repos 🥳

We can keep this two pull requests because one day the main branch will become 2.0.x and there we will not want scala to be upgraded anymore. But yes we do not need to add the pins everywhere.


Unable to call an archiver's openWrite function using PHYSFS_openWrite

I'm trying to write an archiver with streaming write support (a simple format covered by the unpacked template). I implemented an openWrite() function I wanted to test, but PHYSFS_openWrite() never reaches it. It fails in verifyPath() because it uses the directory archiver's stat function, determines it doesn't exist, and returns an error.
It calls DIR_stat() here, and here's an example of my source code:

const char packname[] = "Armor_012_Upper.pack";

int main(int argc, char** argv) {
    PHYSFS_init(argv[0]);
    PHYSFS_mount(PHYSFS_getBaseDir(), NULL, true);
    PHYSFS_setWriteDir(PHYSFS_getBaseDir());

    PHYSFS_registerArchiver(&archiver_sarc_default);

    printf("Mounting %s...\n\n", packname);
    PHYSFS_mount(packname, NULL, true);

    PHYSFS_file* test_write = PHYSFS_openWrite("/ActorLink/Armor_012_Upper.bxml");
    printf("%p\n", test_write);
}

I'm able to read the file into a buffer with PHYSFS_openRead() and PHYSFS_readBytes() and write it to disk (I omitted the code for this to keep the code snippet short). So the file definitely exists, and the archiver is registered correctly.
TLDR: Writing has very different behaviour from reading which seems to prevent me from even reaching the writing code in my archiver. I could try to re-implement PHYSFS_openWrite() myself, but I would prefer not to if possible.

Update: I fixed this by making the function search the search path for an appropriate handle instead of using the write directory in a fork. I'm still running into a ton of roadblocks trying to get write support working for a simple format, though.
I need to copy files into their own buffers for streaming writes then rebuild it into an archive when writing is done, for it to be even a little bit efficient. I could make it atomic, but I really don't want to rebuild the archive file and write it to disk on every single write. If there's some better way to implement this that doesn't require changing a lot of the PhysicsFS code, that would be great. As it stands, I think I need to edit PHYSFS_flush() and maybe PHYSFS_close() to make this work the way I want it to.


Ensure Python processes are cleaned up when the simulation is reset

A report on the forum indicates that sometimes Webots fails to clean up the controller processes.
Possibly related, I've seen times when Webots notes that it's going to force-kill the controller because it didn't exit within one second. The docs imply that when Webots tells you (via return from the step method) that the simulation is going to end, that's a courtesy rather than a demand, yet the warning seems to disagree.
I'm not sure if these two are related, however it seems plausible that Webots is actually expecting the controller processes to quit ASAP after it informs them the simulation is going to end (rather than waiting to be killed).

This is quite noticeable when running multiple robots. The processes add up quickly

Looking briefly into how this works:

Signals aren't sent to the sr_controller.py process before hard terminate
Signals aren't sent to the robot.py process before hard terminate

Here's how the hard terminate works: https://github.com/cyberbotics/webots/blob/56bab11b4491c03fd6e9dee8fc64f2ee60b6a701/src/webots/control/WbController.cpp#L157
Also interestingly, the response from webot.step doesn't appear to change when the controller is told to stop. This might be because of the non-standard way we're running step in a background thread.

I wonder if we should exec rather than fork the robot.py process?

An alternative approach than #143 might be to inline execute the competitors' python code within the main controller process.
I'm fairly sure we could do that relatively easily. It would involve:

setting the working directory to the new value (we already have this to hand anyway)
reading in their robot.py
calling exec on that code (passing empty state)

I would hope that this would avoid the proliferation of child processes and may even help clarify error messages from the child process as it would remove the indirection.


All my files were deleted!

Issue Type: Bug
When I opened a folder, all the files I had in the previously opened folder were deleted. They are nowhere to be found!
Extra information (most likely relevant): The files where under the Extensions Path of the Visual Studio Code
This is outrageous!!
 Can you create a test folder with some files and open it from the command line using code.cmd .

@AngelinCalu  can you also please give more details about

Extra information (most likely relevant): The files where under the Extensions Path of the Visual Studio Code

Thanks!

@AngelinCalu can you also please give more details about

Extra information (most likely relevant): The files where under the Extensions Path of the Visual Studio Code (vscode)

Thanks!

I started working on a vscode  extension a couple of days ago. At first I was working in some folder and copying the extension folder to the %LocalAppData%\Programs\Microsoft VS Code\resources\app\extensions\ to check how the changes are looking. Then I realised that I could just make the changes directly to the files from that the before-mentioned folder and just restart vscode to see the changes in actions. That was fine until one time when I wanted to open a folder (interactively using File -> Open Folder ...  from the menu. When opening a folder in vscode all opened files are getting closed and vscode seems to be restarted. At that point all my (extension) files were removed. I tried recovering them using a few 3rd party softwares without any luck.
I have tried replicating the issue in a different folder than the extensions path and I didn't had any problems.

@AngelinCalu just to be clear, you added files to a location where VSCode itself is installed? That is not a good idea because during installation this directory will be replaced. You should never put files in there.
Can you explain what you are trying to do?

@AngelinCalu just to be clear, you added files to a location where VSCode itself is installed? That is not a good idea because during installation this directory will be replaced. You should never put files in there.
Can you explain what you are trying to do?

I'm not sure what else I could explain more than I already have.
I wanted to work on a new VS Code extension in the %LocalAppData%\Programs\Microsoft VS Code\resources\app\extensions\my-extension-name folder. Those files "didn't make it" to a new vscode extension, so there is no question about something installed over that folder.
Could it be something related to the fact that in version 1.31 there is a new feature which automatically load newly installed extensions (without being needed to reload VS Code) ?

@AngelinCalu can I ask why you choose to create that extension in %LocalAppData%\Programs\Microsoft VS Code\resources\app\extensions\my-extension-name? That is not the location where installed extensions come from (only built in ones). The extension directory is %USERPROFILE%\.vscode\extensions

@bpasero Can't seem to find the link where it was written that that's the way to go when starting a new extension development.
However that might be, it worked (development and debugging) until my files were deleted and I'm not sure that deleting files is in any way the expected or desired behavior in such case.

Ok please check https://code.visualstudio.com/api/get-started/your-first-extension
Closing for now.

The extension I was working on was a "Programming Languages" one, used to syntax highlight some new file format. There is no real need to compile using F5 I guess, isn't it?
You can, of course, close it, but the problem still exist.


Multi selection - alternative internal structure to remember order

I think it would be nice to have the ability to enable something like a sorted selection mode where the selection is saved in a LinkedHashSet instead of a  HashSet.
I need this because I'm selecting items that are pushed into a group. And the items can be sorted in their group so the first order should be the one in which the items were selected...

@MFlisar which count of selected items do you expect in general? if the count is not to much you could simple sort them on your side?

I need the order of selected items. I can't just sort the set as the data for sorting is missing (the order in which the user selected the items)

the order in which the user selected the items. You get this, by using your SelectionListener

You're right, the SelectionListener could be used for this as soon as it's working correctly

I think about following change:
private Set<Integer> mSelections;

public FastAdapter() {
    this(false);
}

public FastAdapter(boolean rememberSelectionOrder) {
    setHasStableIds(true);
   if (rememberSelectionOrder)
       mSelections = new TreeSet();
   else
       mSelection = new LinkedHashSet();
}

What does actually speak against this from your point of view?
Changes
mSelections is a TreeSet or a LinkedHashSet (currently it's always a TreeSet).

@MFlisar I am not 100% sure. What speaks against a TreeSet in as your perspective? Also I assume in general that selections (the count of those) is quite minimal in comparision to the amount of items displayed in a list. So having the access as fast as possible is more imprtant than probably some overhead in storing this information

Do you think a small reusable helper class for this functions make sense?
A custom selection listener that is able to save/restore it's state in a bundle and that internally just remembers the selection order?
Or do you think this is just a rare case, that it should stay a (my) user class?

As of the current moment it seems to be a really rare usecase. There were no other issues opened for this problem. theoretically it would be ok to allow any list which extends Set during creation. Would this already solve your requirement?

That was my suggestion anyways in the beginning... I now did following:
public class RememberSelectionOrderSelectionListener<T extends IItem> implements ISelectionListener<T>
{
	private static final String BUNDLE_SELECTIONS = "bundle_ordered_selections";

	private LinkedHashSet<Long> mSelectedIds;

	public RememberSelectionOrderSelectionListener()
	{
		mSelectedIds = new LinkedHashSet<>();
	}

	@Override
	public void onSelectionChanged(T item, boolean selected)
	{
		if (selected)
			mSelectedIds.add(item.getIdentifier());
		else
			mSelectedIds.remove(item.getIdentifier());
	}

	public Set<Long> getSelection()
	{
		return mSelectedIds;
	}

	public Bundle saveInstanceState(Bundle savedInstanceState) {
		return saveInstanceState(savedInstanceState, "");
	}

	public Bundle saveInstanceState(Bundle savedInstanceState, String prefix) {
		if (savedInstanceState != null)
			savedInstanceState.putSerializable(BUNDLE_SELECTIONS + prefix, mSelectedIds);
		return savedInstanceState;
	}

	public RememberSelectionOrderSelectionListener<T> withSavedInstanceState(Bundle savedInstanceState) {
		return withSavedInstanceState(savedInstanceState, "");
	}

	public RememberSelectionOrderSelectionListener<T> withSavedInstanceState(Bundle savedInstanceState, String prefix) {
		if (savedInstanceState != null && savedInstanceState.containsKey(BUNDLE_SELECTIONS + prefix))
			mSelectedIds = (LinkedHashSet<Long>)savedInstanceState.getSerializable(BUNDLE_SELECTIONS + prefix);
		return this;
	}
}

And just append this listener and use it...
But I would still be happy with the ability to decide myself, if I take the cost of having a LinkedHashSet directly in your adapter...


Cifar10 experiments

Hi,
Thank you very much for realizing the code. I would like to know if you intend to release the Cifar10 code of your paper.
Best,
Kilian

Hi, I am currently traveling but can attend to this by Monday. This code was intended to be included, sorry for any inconvenience.

Hi,
No problem. Thank you very much for realizing the code.
Best,
Kilian

I found the relevant code - will add it this to the repo by tomorrow, though I want to type up a few notes so that it is easily reproducible before pushing. But the details are simple enough if you would like to go ahead yourself - as mentioned in B.4 of our appendix, we used the control GAN of Balaji et al. (https://github.com/yogeshbalaji/robustOT) without robustification weights enabled and 5% contamination. (At a glance, I think running the command python main.py --base_cfg_path configs/experiments/CIFAR10_MNIST.json --cfg_path configs/unconditional/WGAN.json from the GAN folder should do the trick, but I can provide full instructions tomorrow). For the robust GAN, we made the following adjustment

to their trainer.py (https://github.com/yogeshbalaji/robustOT/blob/main/GAN/trainer.py).

Thank you. I prefer waiting for you to release the code as I want to reproduce your results.

See latest update - hope that is helpful!


Test and Fix: subset and superset for empty character set

1- add tests for subset and superset for empty character set
2- fix runtime crash of
CharacterSet(charactersIn: "ab").isSuperset(of: CharacterSet())

3- fix return value of
CharacterSet().isSuperset(of: CharacterSet())

expected value is true, currently returned value is false

@swift-ci Please smoke test

Good catch. Can you add tests for the two false comparisons? (emptySet.isSuperset(of: immutableSet) and immutableSet.isSubset(of: emptySet))

Thank you for your kind words.

@swift-ci Please smoke test

@swift-ci Please smoke test

I'm not sure this is the right fix. Why would just this one API be broken? cc @phausler

I would guess this is not the only one broken here; most likely there are other cases that will fail too - my guess is actually it has to do with the secondary character set being passed across. So formUnionWithCharacterSet: or formIntersectionWithCharacterSet: might be broken too

The new test is also failing (runtime segfault) on the resilience bot. Did it uncover a resilience bug?

New test also segfaulted on watchOS simulator. I assume we have unresolved CharacterSet bugs somewhere. I'll revert this change and file bug reports for now.

Filed rdar://29474937. We'll revisit this once the bug is fixed.

Filed https://bugs.swift.org/browse/SR-3311 for the incorrect empty set behavior that this change was trying to fix. @jrose-apple and @phausler, if you suspect other empty set problems then please follow up there.

I added tests for the resilience bug, so as to force the bug to appear in every testing causing runtime error. I also added tests for 3 other functions : isStrictSubset() and isStrictSuperset() and equality operator . Current file is at : https://github.com/MuhammadAnnaqeeb/swift/blob/master/test/stdlib/TestCharacterSet.swift   Please  reopen this Pull Request just for enough period to run tests. I only want to see if the tests would now constantly fails.

I don't think we can re-open a PR that was merged and then reverted. Can you make a new one?

OK, I will open another PR.


Keyless IDistributedPub/Sub ?

Maybe(/probably) I'm missing something but I'm not quite sure why IDistributedPub/Sub forces me to specify a key?
I'm using this with named pipes for direct IPC between a windowless backgound service application and a desktop agent, both running on the same machine. Currently, I'm just passing the pipe name as the key but I really don't need it for anything...
Could a keyless IDistributedPub/Sub<T> be added or are there any arguments against it?

I implemented it for network(especially redis) at first.
Therefore, it was only natural that a KEY was needed.
I feel that keyless is difficult as a common interface.


Upgrade: Upgrade doc for v2.1.0

This PR updates the upgrade document from v2.0.x to v2.1.0
Signed-off-by: Madhu Rajanna<EMAIL_ADDRESS>

@nixpanic done PTAL

@MergifyIO refresh


Problem with Menu - Active style

For any page i.e settings, profile, .... always the "Home" is active , there is a bug , kindly resolve it

Hi @Gardens18 ! This has been solved with our latest version!
Thanks,
Alin


inna9Z : add my bio

Checklists
General Checks

[x] the branch is up to date with main/master
[x] the code works when pulled and run locally
[x] All CI checks pass
[x] all conflicts are resolved (if any)
[x] PR has a descriptive title
[x] PR has appropriate labels and milestones for easy identification
[x] PR it is assigned to the owner
[x] reviewers are assigned
[x] the PR contributes only one focused change
[x] It is in the appropriate column in the project board (if necessary)
[x] has short and clear description
[x] is linked to an issue (if it is related)
[x] feedback is addressed (if any and if it is appropriate feedback.)

Markdown

[x] the markdown source is formatted
[x] spelling and grammar is correct in all text
[x] The markdown looks correct when you preview the file
[x] all links and images work
[x] any embedded code snippets are formatted

Well done, always double check the text you write, I used Grammarly in the past, very handy. In markdown try to avoid HTML tags.

@bermarte Thank you for all your comments. Its really important remarks i will notice all of them. And i already downloaded Grammarly.


[gh-509] improve rooch init to ECDSA and Schnorr.

Resolves #509

@wow-sven I have added command args for init.rs, create.rs and import.rs according to #509
However, there are several implementational errors, for instance, I can successfully command rooch account create with Ed25519 while others resulting errors:
rooch account create --scheme 3
0x661b491fa5bb7d6c287c2039647c481e97b3d30b46f6f5a74041771c77d27c74
Generated new keypair for address with scheme "schnorr" [0x661b491fa5bb7d6c287c2039647c481e97b3d30b46f6f5a74041771c77d27c74]
Secret Recovery Phrase : [census sudden knife document sick disease flush brand sport head genius warrior]
Sign message error: Cannot find key for address:[0x661b491fa5bb7d6c287c2039647c481e97b3d30b46f6f5a74041771c77d27c74]

rooch account create --scheme 2
0x616aa7030c13095f8966c23c54d1445397672f703eb1b83fea1d23f572f40190
Generated new keypair for address with scheme "ecdsa" [0x616aa7030c13095f8966c23c54d1445397672f703eb1b83fea1d23f572f40190]
Secret Recovery Phrase : [arrive scheme chunk mutual hip twenty tunnel correct fee sign bag high]
Sign message error: Cannot find key for address: [0x616aa7030c13095f8966c23c54d1445397672f703eb1b83fea1d23f572f40190]

rooch account create --scheme 1
thread 'main' panicked at 'not yet implemented', crates/rooch-key/src/key_derive.rs:113:13
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

rooch account create --scheme 0
0xf19440a886870fbbc37d3294b80d8270497a8a69795eea26f0db6864fd06aab1
Generated new keypair for address with scheme "ed25519" [0xf19440a886870fbbc37d3294b80d8270497a8a69795eea26f0db6864fd06aab1]
Secret Recovery Phrase : [onion armor erase curtain obvious wear virtual danger used usual leopard hurt]
{
  "sequence_info": {
    "tx_order": "2",
    "tx_order_signature": {
      "scheme": "3",
      "payload": "0x0305da1cd2bfe328183f68231fea70a3b01ce4f80e0804623eff5ff179d889b3c66bb32351a1fedbceb488fe2d35dae29e61d97c09912bf8c40290a9f1afaddd10e81833ca50327fc46910231bf58f2adfac9c3373f4b02b60dbad530b1373dcf2"
    },
    "tx_accumulator_root": "0x046ff90a9897cbbf0fe32119b99275291e67642b1ca526a5733859774f3c8c9d"
  },
  "execution_info": {
    "tx_hash": "0xb75c7ba9db709f23b14e57a4573f8f6f5543876d868da573b8a6fe48a6a0d57a",
    "state_root": "0x6df0ba561d675f171d4c345c791d67852fe003849ce6c17b1b54085f6372762d",
    "event_root": "0x414343554d554c41544f525f504c414345484f4c4445525f4841534800000000",
    "gas_used": 0,
    "status": {
      "type": "moveabort",
      "location": "0000000000000000000000000000000000000000000000000000000000000002::account_storage",
      "abort_code": "1"
    }
  },
  "output": {
    "status": {
      "type": "moveabort",
      "location": "0000000000000000000000000000000000000000000000000000000000000002::account_storage",
      "abort_code": "1"
    },
    "table_changeset": {
      "new_tables": {},
      "removed_tables": [],
      "changes": {
        "0x0000000000000000000000000000000000000000000000000000000000000000": {
          "entries": {
            "0xf19440a886870fbbc37d3294b80d8270497a8a69795eea26f0db6864fd06aab1": {
              "type": "new",
              "value": "0xf19440a886870fbbc37d3294b80d8270497a8a69795eea26f0db6864fd06aab1f19440a886870fbbc37d3294b80d8270497a8a69795eea26f0db6864fd06aab14aaece3a8934b49892dc3d46591dfd24c2c423760f45328ff9acf7040346169a9d728fd8828095c63dd49afd8947c646f0a749b0fc4257f7cbec74df75bc8f7f",
              "value_type": "0x2::object::Object<0x2::account_storage::AccountStorage>"
            }
          }
        },
        "0x4aaece3a8934b49892dc3d46591dfd24c2c423760f45328ff9acf7040346169a": {
          "entries": {
            "0x52303030303030303030303030303030303030303030303030303030303030303030303030303030303030303030303030303030303030303030303030303030333a3a6163636f756e743a3a4163636f756e74": {
              "type": "new",
              "value": "0x0100000000000000",
              "value_type": "0x3::account::Account"
            }
          }
        }
      }
    },
    "events": [],
    "gas_used": 0
  }
}

Regardless of successfully creating key pair with schnorr and ecdsa schemes in the above examples, when I issue command rooch account list, there is only ed25519 key scheme in the key store:
rooch account list
                          Rooch Address                            |              Public Key (Base64)              | Scheme  | Active
--------------------------------------------------------------------------------------------------------------------------------------
0x18152bf5a89d4cb01fb47b9b2731ddc6034ada02ff72b03d34bb2e32b0de23f3 | AFXbLkeEPQSVE3dX58BScy92ioHYCPVno+Msje1HmtVq  | ed25519 |
0x324ea18c103aa1b0d7d1631db18ee33b5b928e19fbce2d5475c8db1d012adc53 | AGvGjvPmtZSw+Zze29iYKVof9WLgYnci58pIu6oOhWsU  | ed25519 |
0x363f6d7738c39a73014639659562ecd2bc931ecc9a260c74bf922d4d5488eb17 | AGoLToYqEFOv725/zMLhrssLZpssMO89FDZ5fO546ifN  | ed25519 |
0x42797f90b083f6c0e95d734651c0016683a6e4654bc578a407ebbb536d406bd5 | AH/h8G8Nj1OE7mhBUhMhZSqhLRvZSoM/LsYsCrTuln9y  | ed25519 |
0x677f0bf7cbb5e09045d4c0ccd419205fd671179ce9dc193a24d9cafa26007682 | AP+dVrhGdbfn1IXCJaRoTKCRfZdfXbKB9WCMnpOsNEma  | ed25519 |
0xc634422187c519504826604ace5b44323b0c9a64501ccf505d05033e3747bf9f | AOK2m78MBrq7NWb03f5m5BICsuYCVpDGVoBz7IRQUeWS  | ed25519 |
0xf19440a886870fbbc37d3294b80d8270497a8a69795eea26f0db6864fd06aab1 | ADc6VrQvmZ3tHFptABx8TRpI5hZUuTNsRhOmdeYXv4LO  | ed25519 |

Please help review and possibly give me some hints. @wow-sven @jolestar

Still struggling with the VM validation error for ECDSA:
Transaction error: RPC call failed: ErrorObject { code: ServerError(-32000), message: "validate failed: VMError with status ABORTED with sub status 1 at location Module ModuleId { address:<PHONE_NUMBER>000000000000000000000000000000000000000000000000000003, name: Identifier(\"ecdsa_k1\") } at code offset 0 in function definition 3", data: None }

But now it can init, create, although with VM errors, and import with ECDSA key pairs.

Init:

rooch init --scheme 2
Creating config file ["/Users/system/.rooch/rooch_config/rooch.yaml"] with custom server and Ecdsa crypto scheme.
Rooch server URL: http://<IP_ADDRESS>:50051
Environment alias for [http://<IP_ADDRESS>:50051] : de
Generated new keypair for address with scheme "ecdsa" [0xc1741c186205427b8114217fd97d4c3d788644d12bed2584749899ffc53a1c5e]
Secret Recovery Phrase : [fly reduce south sheriff repair round reflect recall purpose benefit family rotate]
"Rooch config file generated at /Users/system/.rooch/rooch_config/rooch.yaml"

rooch account list

                          Rooch Address                            |              Public Key (Base64)              | Scheme  | Active
--------------------------------------------------------------------------------------------------------------------------------------
0xc1741c186205427b8114217fd97d4c3d788644d12bed2584749899ffc53a1c5e | AgPfxgvzDhu4+g+lWzQXjTIOqDt29fNRXC2o/lHEuszMAg== | ecdsa  |  True

Create:

rooch account create --scheme 2
0x667c272f6fbefc1905416f36529a8b1df759adcb20fcf24e69751fc0091856a3
Generated new keypair for address with scheme "ecdsa" [0x667c272f6fbefc1905416f36529a8b1df759adcb20fcf24e69751fc0091856a3]
Secret Recovery Phrase : [beauty lonely black garbage soup help guess alone major engine swear admit]
[102, 124, 39, 47, 111, 190, 252, 25, 5, 65, 111, 54, 82, 154, 139, 29, 247, 89, 173, 203, 32, 252, 242, 78, 105, 117, 31, 192, 9, 24, 86, 163]
0x667c272f6fbefc1905416f36529a8b1df759adcb20fcf24e69751fc0091856a3
Transaction error: RPC call failed: ErrorObject { code: ServerError(-32000), message: "validate failed: VMError with status ABORTED with sub status 1 at location Module ModuleId { address:<PHONE_NUMBER>000000000000000000000000000000000000000000000000000003, name: Identifier(\"ecdsa_k1\") } at code offset 0 in function definition 3", data: None }

rooch account list

                          Rooch Address                            |              Public Key (Base64)              | Scheme  | Active
--------------------------------------------------------------------------------------------------------------------------------------
0x667c272f6fbefc1905416f36529a8b1df759adcb20fcf24e69751fc0091856a3 | AgNggb4JyvUuPy49xD7EYeGlix1rfPFyFYNqypu3j07E0g== | ecdsa  |
0xc1741c186205427b8114217fd97d4c3d788644d12bed2584749899ffc53a1c5e | AgPfxgvzDhu4+g+lWzQXjTIOqDt29fNRXC2o/lHEuszMAg== | ecdsa  |  True

Import:

rooch account import --scheme 2 "rib remove bar public eager glow image push call surge over alcohol"
"rib remove bar public eager glow image push call surge over alcohol"
Key imported for address [0x50111db264b6de16cd6016417f5fc37452bfad1d4aceb949bf3a3ba9a708b103]

rooch account list

                          Rooch Address                            |              Public Key (Base64)              | Scheme  | Active
--------------------------------------------------------------------------------------------------------------------------------------
0x50111db264b6de16cd6016417f5fc37452bfad1d4aceb949bf3a3ba9a708b103 | AgNo7sat+5yIf9Fprzz8NPsamG+CdT7u8uYw4QAPyg6qyA== | ecdsa  |
0x667c272f6fbefc1905416f36529a8b1df759adcb20fcf24e69751fc0091856a3 | AgNggb4JyvUuPy49xD7EYeGlix1rfPFyFYNqypu3j07E0g== | ecdsa  |
0xc1741c186205427b8114217fd97d4c3d788644d12bed2584749899ffc53a1c5e | AgPfxgvzDhu4+g+lWzQXjTIOqDt29fNRXC2o/lHEuszMAg== | ecdsa  |  True

You can just implement and test the import command. The account create command need to do some refactor to auth validator.

Ah sad. I'm on a half way implementing account create with correct VM validation. Other commands, such as import and init, are supported for Ed25519, ECDSA and Schnorr.

skip the account create command test, and merge this PR.

I made it ready for review.

Unbound module error. Need to rerun the CI and merge.


Liquid Exception: exit in _layouts/post.html

I get a error with your theme: (oh btw. thank you for your awesome work :heart:  )
Invalid Date: '' is not a valid datetime.
Liquid Exception: exit in _layouts/post.html

I guess the error is this part: (I have no Idea how you can call in ruby a method with a pipe but probably it's possible)
<time datetime="{{ page.date | date_to_xmlschema }}">

Because this work fine for me:
<time datetime="{{ date_to_xmlschema(page.date) }}">

$ jekyll --version
jekyll 2.1.1

You can move the posts to an "example" directory without problems. You just have to add "example" to the exclude list in _config.yml.

Thanks everyone! I finally caught the one left .md file which caused the problem!


Raise better warning when function isn't hydrated

Without this you get a really confusing NoneType has no object stub warning. Easy to accidentally get if you import a second Modal function inside the first one, but not in global scope.

It's possible https://github.com/modal-labs/modal-client/pull/975 might cover this case too. Let me add a unit test for it


How to use JFXHighlighter

I tried to copy the code that is in github but it did not work I do not know if I did something wrong

it's as simple as creating a new instance of JFXHiglighter and calling the method highlight
JFXHighlighter highlighter = new JFXHighlighter();
highlighter.highlight(Parent to highlight, simple search query string )

the highlighter instance can be re-used to highlight any other parent, so there is no need to create another instance. Also, it can be cleared by calling the method clear()

Oh i see thank you, now it its imposible to use with a textarea? i tried but it didnt work. it works perfectly with Text.

yes you can:
Platform.runLater(()-> highlighter.highlight((Parent) textArea.lookup(".content"), "query"));

When i try using like that i have the next error

incompatible types: Parent cannot be converted to Pane
Platform.runLater(()-> highlighter.highlight((Parent)textarea.lookup(".content"), "word"));

and if i cast it as Pane i have other error

"JavaFX Application Thread" java.lang.ClassCastException: com.sun.javafx.scene.control.skin.TextAreaSkin$ContentView cannot be cast to javafx.scene.layout.Pane

Yeah I totally forgot, my bad. I updated JFXHighlighter recently to support Parent instead of Pane. however it's not released yet, it will be included in the next release hopefully in the next few days.
Regards,

Ok thx, i hope it you will update the .jar too cause i dont use maven. Well i have to start using that


Component: Change the [Card Group] design specs to include use case with 4 cards

User story

As a [user role below]:
developer on the DDS Squad

I need to:
have the Card Group design specs

so that I can:
correctly change the Card Group component

Additional Information

Carbon for ibm.com epic (#5406 )
Where there are 4 cards in a card group, Card grouping with card group default and content block cards default, create a variation where if there are 4 cards, they can be viewed as 2 columns with 2 rows and 2 cards per column (4 by 4 matrix)

Acceptance Criteria

[ ] A Sketch file is created or updated with the Card Group design and specs
[ ] Create a Box folder for the Card Group (if not already there)
[ ] The design specs and pdf version are added to the Box folder
[ ] Update the Carbon for ibm.com epic with the spec link
[ ] Add Box folder link to the website content if it exists


CalculateAverage semotpan attempt 1

System: Apple Intel Core i9, 8 cores, 16GB RAM
openjdk version "21.0.1"
Check List:

[ ] Tests pass (./test.sh semotpan)
[ ] All formatting changes by the build are committed
[ ] Output matches that of calculate_average_baseline.sh

Execution time:
real    0m24.179s
user   2m11.723s
sys     0m50.460s

00:22.210, thx for participating!


Update CODEOWNERS for chemistry

Per @tclune, Arlindo was concerned that @adarmenov might not see some changes to GOCART. So a proposed set of changes to the CODEOWNERS file.

I asked @adarmenov which ones he'd like knowledge over (though this should become an "aerosol-team" or something).
A few areas are ones both @adarmenov and @GEOS-ESM/chemistry-gatekeepers both edit.
And I also made @christophkeller a CODEOWNER

Anything other than these or CMake fall back to the @GEOS-ESM/chemistry-gatekeepers

Please clarify the difference between code-owning and gate-keeping.
Do code-owners approve changes, or merely get alerts when certain files change?

@mmanyin A CODEOWNER file lists files and directories on a repository that someone "owns". That is, when a pull request is made with changes to a file, the code owner is one of the "automatic" reviewers. This doesn't stop someone from adding more reviewers, but the codeowner is automatically assigned. Gatekeepers are more of a GEOS concept wherein they are
For example, if any CMakeLists.txt file in Chem is edited, when a PR is made the @GEOS-ESM/cmake-team is automatically added to the reviews for that PR. Before an edit to any other file triggered a review from @GEOS-ESM/chemistry-gatekeepers.
Now let's look at the new file

Thanks for the explanation Matt. I don't have any issue with @adarmenov and @christophkeller being added as codeowners. Are @amolod and Pete Colarco aware of this proposed change?

As GEOS-ESM is currently set up, can a code-owner submit a PR, and then approve it?
Can a gate-keeper submit a PR, and then approve it?

In re the gate keepers, as long as there are more than one (as is the case with @GEOS-ESM/chemistry-gatekeepers) @gmao-esherman can make a PR and then you (@mmanyin) can approve it.
I think that if @adarmenov makes a commit and an PR, that should satisfy the CODEOWNERS logic as, well, the code owner made the change. I suppose @tclune can test this in a toy repo, or, if preferred, I can add @GEOS-ESM/chemistry-gatekeepers as co-CODEOWNERS for the Anton and Christoph folders.
The "correct" thing to do would be to have an @aerosol-team and @geoschem-team with two+ members. If @adarmenov and @christophkeller have suggestions for "deputies" for their two bits of code, I can create the teams and edit the file to refer to those. If nothing else, I could be a member of the team as the creator, but I'd be pretty rubbish at understanding any scientific changes made to the codes.

I like the idea of an @geoschemchem-team, and we should include at least one person from the GEOS-Chem support team (I suggest @lizziel).
Eventually, we will want to directly link GEOSCHEMchem to the GEOS-Chem main repo anyways, in which case it makes a lot of sense to have someone from the GCST reviewing & commenting code changes.

I like the idea of an @geoschemchem-team, and we should include at least one person from the GEOS-Chem support team (I suggest @lizziel).
Eventually, we will want to directly link GEOSCHEMchem to the GEOS-Chem main repo anyways, in which case it makes a lot of sense to have someone from the GCST reviewing & commenting code changes.

Christoph brings up a good point; it would be best to have a single version of the GEOS-Chem code that works both in the Harvard CTM and the NASA CCM.  If a geoschemchem team of Christoph and Lizzie can test changes in both contexts, that would be very helpful!