[service-bus] track2 - when a session is already locked the error message is uninformative

The error message in JS when a session is already locked is just not helpful.
For example, in .NET:
Unhandled exception. Microsoft.Azure.ServiceBus.SessionCannotBeLockedException: The requested session 'helloworld' cannot be accepted. It may be locked by another receiver.
In JavaScript:
(node:24832) UnhandledPromiseRejectionWarning: MessagingError: Received sessionId 'undefined' does not match the provided sessionId 'helloworld' while creating the receiver
Not nearly as descriptive.
Originally posted by @richardpark-msft in https://github.com/Azure/azure-sdk-for-js/pull/7595/review_comment/create

In #9766, we improve the error message to Failed to get a lock on the session ${this.sessionId} with error code as SessionCannotBeLockedError. What our .Net friends have is definitely better, and we can make the update

@JoshLove-msft, Do you still use the same message as above in Track 2? In Track 1 it made sense as the method used was called acceptSession or something like that. In Track 2, the user facing methods are createSessionReceiver() and createSessionProcesor. I would expect the error message to change to something generic like

Failed to create receiver for the requested session 'helloworld'. It may be locked by another receiver.

@JoshLove-msft, Do you still use the same message as above in Track 2? In Track 1 it made sense as the method used was called acceptSession or something like that. In Track 2, the user facing methods are createSessionReceiver() and createSessionProcesor. I would expect the error message to change to something generic like

Failed to create receiver for the requested session 'helloworld'. It may be locked by another receiver.

I think this message comes from the service, so yes we still use it.

Fixed with #10451


Fix: allow skipping of L1Msg tx part 2 i.e. calculate num_all_txs in tx circuit

Description
This PR aims to add constraints on tx.block_number in tx table. We group txs with same block number and then get the
num_all_txs in each group (i.e. block). Then we do lookup arg into block table in tx circuit. Since the num_txs in pi's chunk data hash preimage means num_all_txs now, we now copy it to the NumAllTxs row in block table. That is, the NumTxs and CumNumTxs in block number is not constrained by pi circuit then.
The lookup arg into block table in tx circuit are done at tag == BlockNum rows and it looks as

the input expr is (NumAllTxs, tx_table.value, num_all_txs_acc).

This is the 2nd part of the https://github.com/scroll-tech/zkevm-circuits/pull/760.
Issue Link
https://github.com/scroll-tech/zkevm-circuits/pull/759
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Close this as we have it rebased against develop branch.


Make it possible to filter repositories by topics

It is currently possible to filter repositories by language, but nothing else. It would be nice to be able to filter by topic (and show the topics).

Done.


Wrap PyVista

Wrap PyVista Meshes Directly
First off, great work with this library/package!
I'd like to improve the performance when wrapping pyvista meshes, so I've proposed the following method:
PolyMesh.from_pyvista(pvmesh)

It can actually wrap any VTK/pyvista object and does it quickly as it utializes a bunch of pyvista's fast methods for getting data from VTK.
For example:
pvmesh = examples.download_nefertiti()

def old(pvmesh):
    ugrid = pvmesh.cast_to_unstructured_grid()
    return PolyMesh.from_vtk(ugrid)

def new(pvmesh):
    return PolyMesh.from_pyvista(pvmesh)

New approach is about 20x faster (and will be even faster with https://github.com/pyvista/pyvista/pull/1228):
>>> %timeit old(pvmesh)
2.28 s ± 20.3 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

>>> %timeit new(pvmesh)
117 ms ± 3.62 ms per loop (mean ± std. dev. of 7 runs, 10 loops each)

Also, makes examples a bit cleaner:

Amazing! Thanks a ton :)


ERROR: Could not find a version that satisfies the requirement nvidia-nccl-cu11==2.14.3 (from versions: 0.0.1.dev5) ERROR: No matching distribution found for nvidia-nccl-cu11==2.14.3

Hello, when I execute
pip install -r requirements.txt

I encountered the following error:
ERROR: Could not find a version that satisfies the requirement nvidia-nccl-cu11==2.14.3 (from versions: 0.0.1.dev5)
ERROR: No matching distribution found for nvidia-nccl-cu11==2.14.3

My environment is Windows with Miniconda. Please help me, thank you!

Hi, I don't know if you already solved this problem, but I encountered the same problem today and I simply just change the version nvidia-cudnn-cu11== (number) in requirements.txt to <IP_ADDRESS> and it seemed to work, you can try this also if you still have the issue :)


demo data in CEL-Seq2

Hello,
Thank you for developing so nice software: CEL-Seq2.
Where can I download the demo data of  in Experiment Table:
lane1-R1.fastq.gz, lane1-R2.fastq.gz, lane2-R1.fastq.gz, and lane2-R2.fastq.gz
in the website:
https://github.com/yanailab/celseq2
OR in the file config_single_lib.yaml:
R1: '/ifs/home/yy1533/Lab/cel-seq-pipe/demo/data/7_S1_L001_R1_001.fastq.gz'
R2: '/ifs/home/yy1533/Lab/cel-seq-pipe/demo/data/7_S1_L001_R2_001.fastq.gz'

Thank you in advance for great help!
Best,
Yue

Hello Yue,
Sorry I did not provide the demo data at Github. It was expected to be a small CEL-Seq2 sequencing fastq file.  I was afraid I no longer work at NYU so I cannot provide you. However, here are two solutions.

If possible, you could use the original CEL-Seq2 paper's data. It seems each single cell has it own FASTQ so you need to merge them to run this celseq2 pipeline.

Alternatively, I found myself have implemented dummy CEL-seq2 sequencing reads. It just randomly generated a pair of R1 + R2 fastq.gz, a GTF and a fasta of a dummy species.

Once you install celseq2, these Bash commands are available to you.
# generate dummy species
celseq2-dummy-species --gtf a/b/c.gtf --fasta a/b/c.fasta
# generate dummy CEL-seq2 reads
celseq2-simulate --gtf a/b/c.gtf --fasta a/b/c.fasta --savetor1 R1.fq --saveto2 R2.fq

In this way you will have everything to technically run the celseq2 pipeline.
Best,
Yun

Hello, Yun,
Thank you so much for your detailed explanation!
Thank you again and really appreciated!
Best,
Yue
li@-Desktop-590-p0xxx:~/celseq2-master$ celseq2-simulate --gtf Danio_rerio.GRCz10.87.gtf --fasta Danio_rerio.GRCz11.dna.toplevel.fa --savetor1 SRR9609653_1.fastq --savetor2 SRR9609653_2.fastq
Traceback (most recent call last):
File "/home/li/.local/bin/celseq2-simulate", line 8, in
sys.exit(main())
File "/home/li/.local/lib/python3.6/site-packages/celseq2/dummy_CELSeq2_reads.py", line 264, in main
gzip=args.gzip)
File "/home/li/.local/lib/python3.6/site-packages/celseq2/dummy_CELSeq2_reads.py", line 183, in dummy_CELSeq2
min_qual=default_qual)
File "/home/li/.local/lib/python3.6/site-packages/celseq2/dummy_CELSeq2_reads.py", line 110, in dummy_readquality
assert not length is None, 'Specify length.'
AssertionError: Specify length.

li@-Desktop-590-p0xxx:~/celseq2-master$ head -n 12 SRR9609653_1.fastq
@MN00336:7:00CELSEQ2:1:78888:10964:367 1:N:0:ATGCGC
AAAAAAAGACTC
+
,@/9?C9:+86E
@MN00336:7:00CELSEQ2:1:20266:32663:7 1:N:0:ATGCGC
AAAAATAGACTC
+
.-?/D.;8,+GI
@MN00336:2:00CELSEQ2:1:42122:46947:341 1:N:0:ATGCGC
AAAAAGAGACTC
+
.1<76.0.4;;/

li@-Desktop-590-p0xxx:~/celseq2-master$ head -n 12 SRR9609653_2.fastq
@MN00336:7:00CELSEQ2:1:78888:10964:367 2:N:0:ATGCGC
CATTCTTCACAGAGTATCTGCAGGTATTGATCACCCCTGATCAGTTATTG
+
;B80ED6-::B=:<6,,HE4F;.E=G@->CF5;@0EC,=60=I<.:CHA,
@MN00336:7:00CELSEQ2:1:20266:32663:7 2:N:0:ATGCGC
GCGGAAGCCCCGCTCCAGCTAAACACCCAGTGGTTCGCTCTCAGAGTTAT
+
.AIGF5A:H@EE6A-1H7@<3=G5500D@H?H1016I19,+;>:GEH<3/
@MN00336:2:00CELSEQ2:1:42122:46947:341 2:N:0:ATGCGC
GCAGCTCCTGCTTTCATTCAATTAGTGTGATTACAGACAAACGTCATCAA
+
22-9HDC954+1I/<8AI4HB:<88D->@A/B?C>-B?01CD,0I7,9@:

No. My simulation is really naive so it is only working for the dummy species I generated. Here it seems you directly run on the zerbra fish.

Hello, Puriney,
Thank you so much for your message!

I downloaded the gtf and fasta file of C_elegans.

It only comes out Chr1, is that correct?
li@Desktop-590-p0xxx:~/celseq2-master$ celseq2-dummy-species --gtf Caenorhabditis_elegans.WBcel235.99.gtf --fasta Caenorhabditis_elegans.WBcel235.dna.toplevel.fa
End of chr1 is 4200

I downloaded the GSE78779 only the C-elegans species, and merged them together: GSE78779_C_elegans.fastq. Still has error:

li@Desktop-590-p0xxx:~/celseq2-master$ celseq2-simulate --gtf Caenorhabditis_elegans.WBcel235.99.gtf  --fasta Caenorhabditis_elegans.WBcel235.dna.toplevel.fa --savetor1 GSE78779_C_elegans.fastq
usage: celseq2-simulate [-h] [--gtf FILE] [--fasta FILE] --savetor1 FILE
                        --savetor2 FILE [--expected-alignment FILE] [--gzip]
                        [--test] [--verbose]
celseq2-simulate: error: the following arguments are required: --savetor2

But When I used following command, it only comes out one fastq file: SRR3196087_1 fastq, not two R1+R2.

li@Desktop-590-p0xxx:~/sratoolkit.2.9.6-1-ubuntu64/bin$ fastq-dump --split-files SRR3196087.sra
Read 1273706 spots for SRR3196087.sra
Written 1273706 spots for SRR3196087.sra

Thank you again and really appreciate any of your help!
Best,
Yue

The simulation is to create a fake genome but it is designed by me. So don't use any real species data.

Generate a fake species
celseq2-dummy-species --gtf fake_species.gtf --fasta fake_species.fasta

This command will generate the new GTF and FASTA files for this fake genome.

Generate dummy CEL-seq2 reads
celseq2-simulate --gtf fake_species.gtf --fasta fake_species.fasta --savetor1 R1.fq --saveto2 R2.fq

The command will generate random reads for the fake species.
From now on, consider the fake species as a human/mouse genome data. Do routine pre-processing.

Run bowtie2 / STAR to index the fake_species.fa

Follow the tutorial to setup the pipeline.

Use the R1.fq, R2.fq, fake_species.gtf and the aligner index to run the pipeline.

Hello, Puriney,
Thank you so much for your great help!
It works!
Thank you again!
Best,
Yue

Glad to hear that. Thanks for using our tool.


Hamming Lesson

Adds hamming lesson with examples.
I'll admit I'm still rather new to powershell. The example might not be the best

The example might not be the best

We don't worry too much about the example, since it's never exposed to the users. It's just used to verify that the test suite actually can be solved / makes sense.
Thanks for this! :heart:


Add shellcheck action

Fixes #411

The workflow can definitely be improved since it currently just hardcodes directories that already contain shell scripts. Also I don't know how to test this except by merging it into main. Suggestions welcome!


[#4238] fix (trino-connector) Make the Trino connector test more stable and improve the logs

What changes were proposed in this pull request?

Add more output log  for run the Trino Integration test.
Enable debug for TrinoQuery runner
Increase timeout for wait catalog to create.

Why are the changes needed?
Fix: #4238
Does this PR introduce any user-facing change?
NO
How was this patch tested?
NO

@yuqi1129 can you please help to review?


MiotNumberEntity is overriding deprecated methods on an instance of NumberEntity

custom_components.xiaomi_miot.number::MiotNumberEntity is overriding deprecated methods on an instance of NumberEntity, this is not valid and will be unsupported from Home Assistant 2022.10. Please report it to the custom component author.
custom_components.xiaomi_miot.number::MiotNumberSubEntity is overriding deprecated methods on an instance of NumberEntity, this is not valid and will be unsupported from Home Assistant 2022.10. Please report it to the custom component author.
custom_components.xiaomi_miot.number::MiotNumberActionSubEntity is overriding deprecated methods on an instance of NumberEntity, this is not valid and will be unsupported from Home Assistant 2022.10. Please report it to the custom component author.

https://github.com/home-assistant/core/pull/73233


Fix integ test gen

Fixes #30

@SimeonEhrig Since this touches your code, could you please have a look at it? Thanks! 😊


Publish very slow with large number of packages

We're using Aptly (0.9.6) to warehouse packages containing debug symbols for binaries that are evolving over time. As a result, we have a repo containing ~50k packages which vary in size from 100k to 10MB.
Doing the "publish" operation on this repo is very slow with Aptly, consuming about 10 minutes at the end of our CI builds. Most of this time is spent in the Generating metadata files and linking package files step, which appears to be single threaded, but mostly disk bound blocking on reads. So my questions are:

What is actually going on during this step? It looks like this is the code, but I can't tell if it's maybe unpacking each archive to access the metadata? If so, could some of this be cached in the DB to avoid having to do it all each time?
Is this something which could be better parallelized to take advantage of multi-core VMs, eg building up the index in memory and flushing to disk at the end?
Would it be on your radar to work on this and/or provide design guidance to an external person to work on it?

Thanks for a great tool; any thoughts greatly appreciated.

@smira I may be able to offer some time to helping with this, but I would need some implementation guidance. Do you have any thoughts on how we could make the publish step faster?

Some of the heavy stuff is parsing the package contents (if you enabled it)
My repos are over 80k packages and the snapshot switch takes extremely long time.
Right now I do some testing in replacing the go code with libarchive, initial tests show it to be faster, but there is a library dependency.
My test for trusty main:
aptly repo show test
Name: test
Comment:
Default Distribution:
Default Component: main
Number of packages: 28592

with optimizations
real    235m50.366s
user    217m59.507s
sys     3m18.563s

without:
real    289m38.725s
user    285m30.790s
sys     31m22.838s

still very long but 50 minutes faster

@sobczyk Do you have a fork somewhere with this modification, which I could examine?
What is the data which is coming out of the archives? Wouldn't it be faster to read it once on initial insertion, and then cache it somewhere, so we don't need to be reopening the archives again later?

It is caching on publish not on insert (repo import/add/include)
see https://github.com/smira/aptly/issues/276#issuecomment-126452927
a second publish of the same data is much faster compared to first time publish
real    10m46.479s
user    8m20.164s
sys     0m11.789s

fork with my modifications https://github.com/sobczyk/aptly

So the easiest way is to disable package contents generation altogether. And yes, aptly caches package contents once, so it's only one-time overhead. If you don't need contents indexes, it's fine to disable them.

The Contents index could definitely be the issue for us. Is there an existing flag or configuration option to turn that off?

It is supported via config: https://github.com/smira/aptly/blob/master/api/publish.go#L187
"skipContentsPublishing": true should do the trick

I've added the flag to my /etc/aptly.conf file, which now looks like this:
{
  "rootDir": "/var/aptly",
  "downloadConcurrency": 4,
  "downloadSpeedLimit": 0,
  "architectures": [],
  "dependencyFollowSuggests": false,
  "dependencyFollowRecommends": false,
  "dependencyFollowAllVariants": false,
  "dependencyFollowSource": false,
  "gpgDisableSign": false,
  "gpgDisableVerify": false,
  "skipContentsPublishing": true
}

I've verified that there's no ~/.aptly.conf, but the publish is still very slow (on the Generating metadata files and linking package files step), and my Contents-amd64.gz file is getting a new timestamp. I also tried giving the option explicitly on the CLI, with the same results:
aptly publish update -skip-contents=true trusty myrepo

This occurs with Aptly 0.9.6 and 0.9.7.

I believe it has no effect on commands which update published repository, only those which create it (publish repo or snapshot) should process it. #347 has details

@smira I've tried a couple times, and it seems that I get a long Generating metadata files and linking package files step on initial publish, after which successive updates are quick (and there are indeed no Contents-*.gz files generated).
Thanks!


chore(deps) Dependency Bumps [chain-chores]

Generated with chain-chores
Changes:

tendermint/tendermint v0.34.21 -> tendermint/tendermint v0.34.23
cosmos/cosmos-sdk v0.45.9 -> cosmos/cosmos-sdk v0.45.11
cosmos/iavl v0.19.3 // indirect -> cosmos/iavl v0.19.4 // indirect

Other PR fixed this. Thank you for pushing the PR


Add make test to Makefile

It should be possible to run all Proteus tests from the makefile.  This command should mirror the command used by travis ci.

I've noticed several files have not been included in the tests folder...these should be added

[ ] pressure_laplace_matrix.txt (solver_tests)
[ ] stokes_2d.py (mesh_tests)

#411 partially addresses this issue.


Bar not fitting icons fully.

When using the Toggle Game Addon the bar wont be the right size

thank you for your report :3
there are many things about this theme which are unfinished/need to be touched, and this is one of those things i plan to fix sooner than later. :D

thank you for your report :3
there are many things about this theme which are unfinished/need to be touched, and this is one of those things i plan to fix sooner than later. :D

I didnt knew i needed this theme until i saw it.

iiiiiitttttttt shhhhhhould be fixed now :3


I hope the dialog dialog box can have the function of up and down center alignment. We need to be able to center up and down in many scenes.

希望有上下居中对齐的功能


Change Unturned to Supported

Unturned works with BattlEye natively as far as I know

@Faalagorn From a cursory glance on ProtonDB, this looks to be incorrect? Under Wine or Proton, you are unable to join BattlEye protected servers. VAC protected servers appear to work just fine both ways. Perhaps I should add a * to the entry to mention it has caveats?

It has a native Linux port, just like e.g. Ark on the list.

Yup… unless we want to separate native games from Wine/Proton ones?


Content update - Dependency property security (user story 1878471)

@adegeo, this PR is ready for your review.
Includes a new code snippet to illustrate some of the concepts.
Internal preview link

@adegeo, thanks for the review. :)


Negative skillgains when under the infuelce of stats altering spells

If a player is influenced by a negative stats altering spell (Mindrot, curse, weaken, whatever)
He/She will gain negative skill-gains instead of positive should any skillgain occur during the spell is active.

 I did SPELLEFFECT s_curse, 100.0 on myself and i started attacking an npc, from 10.0 tactics and wrestling are increasing, not decreasing. Does this happen really with every skill or also for ones? In the latter case, do you have any negative or unordered values for ADV_RATE, BONUS_, STAT_ etc in the SKILL block?


Add modules

Should address #15.
There are a few shortcomings I'm aware of:

no source code location tracking within the module path data type for more accurate information in errors;
failure to resolve a path like X.Y.Z.x because there is no module Y in X results in an error message printing the whole path, which is not ideal;
pretty-printing of named types prefixed by a path is not implemented properly.

Another problem, that is not addressed in these changes is how to handle the situation when a public value has a private type. Should we care about that? Or should we raise an error or warning?


htlcswitch+routing: omit next_hop_id in final hop payload

Currently we include a short_channel_id for all hops in the path, though BOLT 4 says that it should be omitted for the final hop. This PR corrects this, and consolidates instantiation of the fields into a record package so to reduce code duplication.

@joostjager reworked the pr to omit the next hop id rather than modify the encoding, ptal


[Bug Report] For loadbalancer submodule, update to frontend IP configuration for Public IP throws error PublicIPAddressCannotBeDeleted

Describe the bug
When the frontend_ips map configuration is updated that results in new frontend Ip configuration name and/or new Public IP resource, then Terraform fails to properly create the new Public Ip, update the Load balancer, and destroy the old Public Ip resource.
This issue is well documented here: https://github.com/hashicorp/terraform-provider-azurerm/issues/17885
Here's an example of a change:
load_balancers = {
  "public" = {
    name = "public-lb"
    nsg_auto_rules_settings = {
      nsg_vnet_key = "transit"
      nsg_key      = "public"
      source_ips   = ["<IP_ADDRESS>/32"] # TODO: Whitelist public IP addresses that will be used to access LB
    }
    frontend_ips = {
      "app1" = {
        name             = "app1"
        public_ip_name   = "public-lb-app1-pip"
        create_public_ip = true
      }
    }
  }

load_balancers = {
  "public" = {
    name = "public-lb"
    nsg_auto_rules_settings = {
      nsg_vnet_key = "transit"
      nsg_key      = "public"
      source_ips   = ["<IP_ADDRESS>/32"] # TODO: Whitelist public IP addresses that will be used to access LB
    }
    frontend_ips = {
      "newApp1" = { #<--- updated this!
        name             = "newApp1"
        public_ip_name   = "public-lb-newApp1-pip"  #<--- updated this!
        create_public_ip = true
      }
    }
  }

Running terraform apply with these changes results to following error message:
╷
│ Error: deleting Public I P Address (Subscription: "<REDACTED>"
│ Resource Group Name: "rg-mygroup-001"
│ Public I P Addresses Name: "public-lb-app1-pip"): performing Delete: unexpected status 400 (400 Bad Request) with
error: PublicIPAddressCannotBeDeleted: Public IP address /subscriptions/<REDACTED>/resourceGroups/rg-mygroup-
001/providers/Microsoft.Network/publicIPAddresses/public-lb-app1-pip can not be deleted since it is still allocated to
resource /subscriptions/<REDACTED>/resourceGroups/rg-mygroup-001/providers/Microsoft.Network/loadBalancers/lb-
public/frontendIPConfigurations/app1. In order to delete the public IP, disassociate/detach the Public IP address from
the resource.  To learn how to do this, see aka.ms/deletepublicip.
│
│

Workaround is to add the following lifecycle block to azurerm_public_ip resource:
resource "azurerm_public_ip" "this" {
  for_each = { for k, v in var.frontend_ips : k => v if v.create_public_ip }

  name                = each.value.public_ip_name
  resource_group_name = var.resource_group_name
  location            = var.region
  allocation_method   = "Static"
  sku                 = "Standard"
  zones               = var.zones
  tags                = var.tags

  lifecycle {
    create_before_destroy = true
  }

}

Module Version
3.0.1
Terraform version
1.9.2
Expected behavior
No response
Current behavior
No response
Anything else to add?
No response

Hello @jinkang23, thanks for raising this. This issue will be fixed with PR #77.


Using a compressed .ply crashes my mobile browser, the .splat does not

When on desktop and loading this compressed .ply file I notice faster download times (of course the file is smaller), but nothing else is noticably different (good) as opposed to this .splat file (derived from the same original .ply file).
But on mobile, the compressed .ply will break the site (aw, snap), while the uncompressed .splat file loads fine.
Just spitballing here but could it be that, when uncompressed, the .ply file takes up more memory than the .splat file would?

I forgot to update the code for the online demo with the latest release :) Does it work for you now?

Thanks for your fast reply. I think you misunderstood my issue. The examples on your demo page work beautifully. I was talking about the links in my post. I'll try some different splats tomorrow, could well be something on my side.


Enhance Cleanup logic in QuickStart CI jobs

Cleanup temp data directory for QuickStart CI jobs
Use kill -9 when the quickstart process is not terminated in 60 seconds.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 0.00%. Comparing base (59551e4) to head (1c2f946).
Report is 1212 commits behind head on master.

:exclamation:  There is a different number of reports uploaded between BASE (59551e4) and HEAD (1c2f946). Click for more details.
HEAD has 48 uploads less than BASE

Flag
BASE (59551e4)
HEAD (1c2f946)

integration
7
2

integration2
3
2

temurin
12
2

java-21
7
2

skip-bytebuffers-true
3
1

skip-bytebuffers-false
7
1

unittests
5
0

unittests1
2
0

java-11
5
0

unittests2
3
0

integration1
2
0

custom-integration1
2
0

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #14268       +/-   ##
=============================================
- Coverage     61.75%    0.00%   -61.76%
=============================================
  Files          2436        3     -2433
  Lines        133233        6   -133227
  Branches      20636        0    -20636
=============================================
- Hits          82274        0    -82274
+ Misses        44911        6    -44905
+ Partials       6048        0     -6048

Flag
Coverage Δ

custom-integration1
?

integration
0.00% <ø> (-0.01%)
:arrow_down:

integration1
?

integration2
0.00% <ø> (ø)

java-11
?

java-21
0.00% <ø> (-61.63%)
:arrow_down:

skip-bytebuffers-false
0.00% <ø> (-61.75%)
:arrow_down:

skip-bytebuffers-true
0.00% <ø> (-27.73%)
:arrow_down:

temurin
0.00% <ø> (-61.76%)
:arrow_down:

unittests
?

unittests1
?

unittests2
?

Flags with carried forward coverage won't be shown. Click here to find out more.

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


are not displayed at the bottom of the sensor names in the topology

cut off the names

@arturmon It is by design. Name of the sensor will be shown on bottom. If we see in small screen will be looking like this. However we can drag the sensor and can get complete view. Please reopen this issue if you cannot drag.


Issue Reporter: re-enable extensions

I've tried reporting an issue via the Issue Reporter for the first time and there were two UX issues with it:
First, the blue link to disable extensions was missing the hand pointer so it was not clear what the clicking will do:

Probably worse, clicking the link disabled all my extensions and I don't know how to turn the relevant ones back on. I've restarted VSCode and all my extensions are still disabled. I have about 30 of them and about 25 were enabled, 5 disabled. Now I assume I will need to go one by one, try to remember if it was previously enabled or disabled, and do that manually which is not something I'm looking forward to :)
VS Code Info
VS Code version: Code 1.20.0 (c63189deaa8e620f650cc28792b8f5f3363f2c5b, 2018-02-07T17:10:31.920Z)
OS version: Windows_NT ia32 10.0.16299
Reproduces without extensions

Agree, the most puzzing thing for me is that I don't know what to do next after clicking "disabling all extensions...." ect.
I suggest the plug-in window can be designed like this following:

Each plug-in will show you a status (green for "enabled" and red for "disabled", maybe "enabled (workspace)" or "disabled (workspace)"), depending on which kind of state you choose by clicking the "settings" icon beside it.

And each plug-in should have a check box that we can let them multi-choosen. And when we choose some/All of them, and click "Checks Enabled"：All the "checked" plug-in will be "enabled" and on the other side, "disabled", so changed the words of the button itself.
This option isn't for "workspace enable/disable" but only for globally enable or disable plug-ins.

On the head bar of "Extensions" there should be a multi-choosen check box with which we can do multi chooses.

@borekb I've changed the behavior of disabling all extensions and reloading the window so that the extensions are only temporarily disabled. Reloading VS Code will return them to the original state they had, and the cursor now correct displays as a hand. Sorry for the inconvenience, let me know if that makes sense.

Hi @RMacfarlane , sounds good but when I tried it, the extensions stayed disabled. Steps to reproduce:

Go to issue reported, click disabling all extensions and reloading the window
The window is reloaded, all extensions disabled
Close the Issue Reporter window
Invoke the Reload Window command from Ctrl+Shift+P palette

Oh I needed to restart VSCode entirely. That enabled the extensions as you said. Before that, they were in a weird state that I haven't seen before:

Yeah, that state is expected. It's the same as running VSCode with the --disable-extensions flag, the extensions are not added to the list of runtime extensions, so they are not activated and their state can't be changed.

@RMacfarlane

Yeah, that state is expected.

Except that is not expected by anyone using the software who is not familiar with the details of the implementation (basically anyone except yourself) because nothing in the UI explains that the extensions are all disabled until you restart VSCode. This is a tragically bad UI design.

@Neutrino-Sunset Thanks for the feedback, you're right that it's not intuitive and really needs more explanation. I've filed https://github.com/Microsoft/vscode/issues/47506 to track this, please have a look.


account

I have created my account and I can't log in

Have you tried to check the log file created in the folder? You can see if the nickname and the password are the same, maybe are write in a wrong way

I have created my account and I can't log in

Hey adele123YI,
Thanks for opening an issue! I would suggest trying the following:

Make sure you are running TamoStudy.jar in the same file folder as the assets and profile folders; This is crucial for the file input and output system to work flawlessly.
If this is already the case, make sure that your profile exists inside of the profile folder. (You will see a text file with your username inside of it)
Lastly, make sure upon selecting "Load Existing Profile" on the welcome screen, ensure that you are entering your username and password correctly. It IS case sensitive so even typing something with a wrong case will cause you to be unable to login.

Kindest regards,
narlock

Hey
Thanks for your answers
-Tamostudy.jar is in the same file as the assets and profile folders
-the profile folder is empty
-I don't have any pictures (I've seen people having them on discord but I don't have them)

Hey
Thanks for your answers
-Tamostudy.jar is in the same file as the assets and profile folders
-the profile folder is empty
-I don't have any pictures (I've seen people having them on discord but I don't have them)

Hey, thanks for the response
If you could, send a screenshot of your Finder folder with your assets, profile, and TamoStudy.jar file in them. I am curious to see if anything else could be created the issue from here.
Otherwise, I will look more into this issue. I plan to release the next release of TamoStudy sometime today, so there will be a new folder and jar file to try and run. If you can't get this version of TamoStudy to work, perhaps the next version will work.
I will continue to look for a solution if this bug persists.
Thank you :)

Hey
Yes of course (but it's in French, sorry)

Hey
Yes of course (but it's in French, sorry)

Thank you again for sending me this image.
I will work to see if I can find a solution. I'll leave this ticket open until I can find one.
I also saw you joined our Discord server, so I may contact you there if it's easier.

Hey adele123YI,
I have found a temporary (and most likely the solution for now...) solution to the issue at hand.
We will solve this problem by creating a command line script. This will be similar to macOS except using the shell sh file extension.
To do this with Windows, follow the following steps:

Locate your TamoStudy source folder (the folder with the assets, profiles, and TamoStudy.jar files in them)
Copy the address of this directory. (For example, if you have this folder on your desktop, your path would look something like "C:\Users\narlock\Desktop\TamoStudy a0.5.0"
From here, create a new text file (we will save this as a .BAT file - USE THIS EXTENSION WHEN SAVING!!) preferably store this also inside of the TamoStudy folder as well.
From here, enter the following on both lines:
cd C:\Users\narlock\Desktop\TamoStudy a0.5.0
java -jar TamoStudy.jar
Note: Use your path (obviously) for the cd command.
Save your file as a .BAT file
Run the BAT file and TamoStudy will now launch with assets and read/write capabilities

From here, you will now be able to use TamoStudy. Remember, you must use the BAT file in order to launch TamoStudy every time.
I apologize that this is kind of a hassle, but I am glad that there is some solution at the moment. I will continue to look for possible fixes to this in the future that are more efficient.


add state_overrides param to estimate_gas call

pass state_overrides
made apply_state_ovverides public

yeah sorry you got sniped on this...
if you don't find anything else lmk, I'll find something appropriate


Turn logging on/off programmatically at runtime

It would be really useful if we could turn logging off/on at will.
For example in a production environment if there is a problem, you could drop into the console window and turn on logging to inspect the redux logs.

Use predicate option

Great thanks, works perfectly

@freeranger How did you achieve what you described? I'm thinking I could have a predicate function that looks for a certain variable that normally isn't defined, but that I can temporarily define in order to temporarily turn on logging. What did you do? Cheers.

Sorry I have no access to that codebase anymore to job my memory but I believe it was something similar to what you describe

@freeranger No problem. In case it's of interest, I ended up doing something like this:
const actions = [] // hardcode any actions you like to always log here, eg ['ACTION1','ACTION2']
export default createLogger({
...
predicate: (getState, action) => includeAlways(action.type) || includeTemporarily(action.type)
})
const includeAlways = actionType => actions.includes(actionType)
const includeTemporarily = actionType =>
window && window.MY_GLOBAL && window.MY_GLOBAL.log && window.MY_GLOBAL.log.enabled === true && isArr(window.MY_GLOBAL.log.actions) && window.MY_GLOBAL.log.actions.includes(actionType)
I can then add some actions to log by running something like this in the browser console:
MY_GLOBAL = { log: { enabled: true, actions: ['ACTION3', 'ACTION4'] } }
And turn them off by modifying the actions array, eg
MY_GLOBAL = { log: { enabled: false, actions: [] } }
Seems to do the job for now.
Cheers.


Fix usage of unsafe cell

UnsafeCell's usage was incorrect. Instead of using UnsafeCell to get multiple mut references to the same memory, correct usage is to keep non-mut references to the UnsafceCell and use it to get mutable references to the underlying data exceptionally, when we are sure nothing else has a reference to the underlying data (of any type).

This PR depends on #3 btw

Hehe I guess I was a bit barebones ;)


Future plans?

Cool project!
What are further plans with this project?
In particular, I am wondering if there are any plans extend the API
(and maybe support nanovg https://github.com/memononen/nanovg).

I've no plans to support drawing vector graphics at this moment, this is way complex than it sounds. Regarding future plans I consider the project finished and it already fulfills its job.
Would be cool to have a vector graphics API on top Sokol like NanoVG, I could work on that someday, who knows, but not of free will because at this moment and I don't need such feature.


[QUESTION] Tests for FastAPI applications

How can I write tests for FastAPI applications? Can you make an example application (best practices) with unit and acceptance tests? (Mock database, test env etc)

the documentation, the tests folder in the library itself are pretty good
examples.
Le ven. 12 avr. 2019 à 3:09 PM, dimuska139<EMAIL_ADDRESS>a
écrit :

Description
How can I write tests for FastAPI applications? Can you make an example
application (best practices) with unit and acceptance tests? (Mock
database, test env etc)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tiangolo/fastapi/issues/152, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABDZPstOWZttK87M_vfbzLTXO1_T7KzKks5vgIVsgaJpZM4csFp2
.

@euri10 Thank you!

Thanks @euri10 !
@dimuska139 I just added docs on testing here: https://fastapi.tiangolo.com/tutorial/testing/ :tada:

As this is now part of the docs, I'll close this issue now. But feel free to add more comments or create new issues.


rivinecg generate blockchain fails

Fetching repository: https://api.github.com/repos/threefoldtech/rivine-chain-template/tarball/master ...

Unpackaged in: pax_global_header
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/.gitignore
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/Caddyfile
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/LICENSE
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/README.md
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/babel.config.js
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/package-lock.json
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/package.json
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/public
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/public/favicon.ico
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/public/index.html
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/public/styles.css
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/App.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/assets
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/assets/logo.png
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/common
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/common/config.ts
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/common/helpers.ts
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Blocks
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Blocks/Block.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Charts
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Charts/ChartInfo.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Charts/LineChart.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Charts/PieChart.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Common
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Common/BlockStakeOutput.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Common/CoinOutput.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Common/CoinOutputCustodyVoid.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Common/GeneralStats.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Common/Input.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Common/MinerOutput.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Common/Navigation.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Common/Output.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Common/Search.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Conditions
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Conditions/AtomicSwapCondition.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Conditions/Condition.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Conditions/CustodyFeeCondition.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Conditions/MultisignatureCondition.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Conditions/UnlockhashCondition.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Fulfillments
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Fulfillments/AtomicSwapFulfillment.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Fulfillments/Fulfillment.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Fulfillments/MultiSigFulfillment.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Fulfillments/SingleSignatureFulfillment.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Outputs
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Outputs/BlockstakeOutputHash.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Outputs/CoinOutputHash.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Transactions
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Transactions/CoinCreationTransaction.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Transactions/DefaultTransaction.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Transactions/MinterDefinitionTransaction.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Transactions/Transaction.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Transactions/TransactionSummary.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Wallets
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Wallets/BlockCreatorWallet.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Wallets/CustodyVoidWallet.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Wallets/DefaultWallet.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/components/Wallets/Wallet.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/main.ts
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/plugins
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/plugins/vuetify.js
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/router.ts
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/shims-tsx.d.ts
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/shims-vue.d.ts
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/store
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/store/getters.ts
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/store/index.ts
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/store/modules
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/store/modules/explorer.ts
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/store/modules/index.ts
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/store/modules/transaction.ts
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/views
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/views/Block.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/views/Charts.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/views/Hash.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/views/Home.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/views/NotFound.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/src/views/Transactions.vue
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/tsconfig.json
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/tslint.json
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/vue.config.js
Unpackaged in: threefoldtech-rivine-chain-explorer-0d466cf/webpack.config.js
Error: lstat threefoldtech-rivine-chain-explorer-v0: no such file or directory
Usage:
  rivinecg generate blockchain [flags]

Flags:
  -c, --config string     file path of the config, ecoding is based on the file extension, can be yaml or json (default "blockchaincfg.yaml")
      --explorer string   frontend explorer to generate, options: vuets,plainjs,none (default "vuets")
      --faucet            Generate the frontend faucet, opt-out. (default true)
  -h, --help              help for blockchain
  -o, --output string     file path where the blockchain will be generated to

lstat threefoldtech-rivine-chain-explorer-v0: no such file or directory

The problem is that it tries to copy it from threefoldtech-rivine-chain-explorer-v0 while it is downloaded at threefoldtech-rivine-chain-explorer-0d466cf

Yes, because the current algorithm uses in a very error-prone way the Content-Disposition value, which since the first release now points to the release tag instead of that commit hash. Instead the filename should just be used as given by the user.

You want to look at the untar function, it could return the root folder (the first target unpacked), as probably the simplest way to know the actual dest folder.
A much better idea than to work with that Content-Disposition header value, given you'll have to a lot of parsing. Currently it returns as follows:
Content-Disposition: attachment; filename=threefoldtech-rivine-chain-explorer-v0.1.0-0-g0d466cf.tar.gz

Meaning you would have to manually filter substring(s), but that's a very error prone approach as I'm sure as you agree. Therefore returning the root dir seems to me like the sane approach.

fixed: https://github.com/threefoldtech/rivine/commit/741c0e47d191302da5ab17aff49048a2ea48f7b1


Locust with custom clients only displays maximum response time

Description of issue
Any calls to events.request_failure.fire or events.request_success.fire update the maximum response time only on the UI, not the average, median or min. This occurs when using the response_time parameter. The code in question:
Expected behavior
The UI should show the Maximum, Average, Median, and Minimum response time
Actual behavior
The UI shows only the Maximum response time, and the Average, Median, and Minimum as 0.

Environment settings

OS: Amazon Linux 2018.3
Python version: 3.7
Locust version: 0.11

Steps to reproduce (for bug reports)
start_time = time.time()
try:
    result = self.redis_client.zrevrange(key, 0, -1)
except Exception as e:
    execution_time = int((time.time() - start_time) * 1000)
    events.request_failure.fire(request_type=command, name=key, response_time=execution_time, exception=e)
else:
    execution_time = int((time.time() - start_time) * 1000)
    events.request_success.fire(request_type=command, name=key, response_time=execution_time, response_length=len(str(result)))

hi @adriano-chwy ! I could not reproduce this. Is it possible that your response times are actually < 1ms for real?

Hey @cyberw, thanks for looking into this. I could not reproduce the issue either outside of this example, and due to the redis cluster being in the same availability zone as the server I can only assume it's possible for <1ms response times. I will close this for now.


is there a workaround for when a kernel config is not present?

https://github.com/alexeadem/qbo-home/blob/e0567e78563e0f73eef1b3f98143da9055d5b044/qbo#L23
I was trying out the command in a github codespace with docker. ubuntu 20.x
uname -r
5.4.0-1090-azure

 $ ./qbo start api
  => start api
clean = 0
OS = Linux
warning: start does not exist, searching other paths for kernel config ...:
error: cannot find kernel config:
  try running this script again, specifying the kernel config::
    CONFIG=/path/to/kernel/.config ./qbo or ./qbo /path/to/kernel/.config:

You can comment  the line with run_linux_tests in the script.
https://github.com/alexeadem/qbo-home#linux
It is just checking cgroups version and limits on max_user_watches and max_user_instances You can modify these manually if needed as indicated here:
https://github.com/alexeadem/qbo-home/blob/e0567e78563e0f73eef1b3f98143da9055d5b044/qbo#L330

is it worth adding a helper to run/parse sysctl outputs? for your test?

Yes that would be useful. Those are the only two systcl values to consider. The only limitation if those values are low is that you'll be limited on the number of nodes you can run.

For more info:
https://kind.sigs.k8s.io/docs/user/known-issues/#pod-errors-due-to-too-many-open-files
The other test is to verify that you have Cgroups v1.

Let me know if that works. Closing the issue.


Progressively migrate old transaction activities

See this comment: https://github.com/opencollective/opencollective-api/blob/53cda4a23a1b35c708d5ee18b9a7218b03eaf578/migrations/20220816142946-enhance-activities-links.js#L86.
We've only migrated transactions after 2022-06-01 because this thing is super heavy and would timeout in a migration script. We need to progressively migrate the data by running the query filtered on id or createdAt.

We are dropping the transaction activities: https://github.com/opencollective/opencollective/issues/7162


PPA production playbook

todo:

[] decrypt qa vault, what's there
[] re-encrypt slightly diff

I think we're ready for a co-pilot sess on this tomorrow. Flagging review just to signal that


userdata example?

Is there any chance we could get an example showing how to create a (full) userdata containing a rust struct and correctly set up a metatable to call methods on it?
I seem to be getting a bunch of weird segfaults, probably because I'm doing something wrong.
I think @jugglerchris was doing some work with userdata, so tagging him here.

Does c982326c1f286774fa8d83ffe51a1a8e59588e8b help?

But won't the result of Point2D::new() get dropped before lua_new() returns?  I'd guess we need to do some sort of fiddling with mem::forget() and then set up a __gc metamethod to call drop() explicitly when the object gets collected later.

In the above example there's no drop() to worry about.  If there is one, you need to be more careful; I'll try to extract the code I use sometime this week.

Poor wording on my part: I was thinking about in the general case. :-)  Look forward to seeing your approach!

@tinyplasticgreyknight, that is correct. I don't think you need mem::forget as you don't have ownership of the value - Lua does. I've written an example that wraps a Vec and drops it here: https://github.com/jcmoyer/rust-lua53/blob/master/examples/userdata-with-drop/main.rs
Admittedly I am not too familiar with the super low level details of Rust so I'd like someone more experienced in that area to review it and make sure it's correct.

That looks pretty close to what I do.  Some subtleties, though:

I think calling check_integer etc. from a Rust function is potentially dangerous, since an error can unwind past a language boundary.
It's theoretically possible for an object to be resurrected after being garbage collected (from another __gc method); if all you do is drop it, then there's a possibility of accessing it again later.  I opted to store an Option<T> in the userdata; in the __gc I use ptr::swap with a None.  If anything does manage to access the userdata after that it's relatively harmless.

I've split out my in-development (badly documented and so on) interface into a github repo: https://github.com/jugglerchris/rlua in case it's of interest.
I store (roughly) an Option<Box<Rc<RefCell<T>>> in userdata, so that I can share access between Lua code and Rust, but via a Box<Any> so that I can verify the type before using it.

I've just added a test for trying to access a userdata after gc to my library after the above, and ahem fixed the bug which made it segfault rather than raise an error.

@jcmoyer Okay, I played around with your Vec example and indeed it looks like writing the object into the raw pointer prevents it from being dropped at the end of lua_new().  For some reason I didn't think that was the case.
The behaviour of raw pointers used in this way seems to be:

the value is assumed to be still alive "somewhere", so it won't be auto-dropped when all other references go out of scope
since rustc can't tell where the raw pointer goes out of scope, it won't be auto-dropped ever
we need to manually drop it when we know it's not referenced anymore (in our case, using a __gc metamethod)

Phew! :-)
@jugglerchris: I was wondering if we could manually manipulate the refcount of an Rc (in new and __gc) to take account of lua having one reference.  It looks like Rc doesn't expose that sort of thing, but anyway it looks like your Option layer already takes care of that part anyway. :+1:
Your RefCell component is to help maintain borrowing rules, and if I'm reading this correctly the Box is only necessary because of your Box<Any> checking, is that right?

I'm not sure what you mean by manually manipulating the refcount - I just store an actual Rc instance/clone.  The Option is only there to handle the access-after-__gc case sanely.
Yes, the RefCell is there to maintain borrowing rules, specifically when I want mutable references from both Lua and Rust, and the Box is for the Box<Any> dynamic type checks.  (I'm not sure how necessary that is - I think you'd have to try pretty hard to make that go wrong from Lua; but probably from debug.setmetatable()).
I think if you don't need to share objects between native Rust code and Lua (ie Lua gets full ownership), then I think you can lose all the wrapping except the Option for post-gc safety.

I think I'm talking rubbish about the refcounting above, I was thinking about some refcounting techniques I'd used in C for writing a lua module recently.

Using all the info we've put together, I've managed to get a working rust module of my own built and tested successfully!  Good work, everybody! :-D


create personal access token link was broken

i found that create  personal access token link was broken so, added correct  github docs link to it for use.

Why:
Closes https://github.com/github/docs/issues/21173

What's being changed (if available, include any code snippets, screenshots, or gifs):

Check off the following:

[x] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

Sir kindly review it.

Sir add it now.

Thanks for your contribution, @dhruv-porwal! I'll get this one merged for you.

Thanks for your contribution, @dhruv-porwal! I'll get this one merged for you.

Thank you Sir

sir kindly mark it hacktoberfest  accepted

please  sir

Done


Access DOM in Shiny Live

Hi,
I cannot access the DOM document object provided by Pyodide in a Shiny Live example. The code is here.
I created an issue in the Pyodide project asking about this. It seems the reason is that the Shiny Live Python code is running in a Web worker.
May I ask please if there is way, or a plan to create a way, to acess DOM from within ShinyPy? Such a mechansim will have the benefit of enabling a large part of the web components ecosystem for Shiny Live.
Many thanks
Best

Interesting. Where is the ShinyLive repo?


Unable to upgrade CRD with Helm

I'm trying to upgrade my Flagger installation using Helm from 0.12.0 to 0.15.0 and I'm getting this error:
Error: failed to create patch: merging an object in json but data type is not struct, instead is: map

Apparently this is an issue upgrading CRDs (see https://github.com/helm/helm/issues/5853).

Upgrade workaround:

uninstall Flagger helm delete flagger --purge (the CRD and the canary resources will be left untouched)
update the CRD with kubectl kubectl apply -f https://github.com/weaveworks/flagger/blob/master/artifacts/flagger/crd.yaml
reinstall Flagger with Helm with out the CRD helm install --set crd.create=false

This has been fixed in Helm v2.14.3, I've tested it by upgrading from Flagger 0.12.0 to 0.15.0 to 0.18.1 🎉


VSCode editor showing Could not find a Flutter SDK. Please ensure flutter is installed and in your PATH (you may need to restart).

Steps to Reproduce
I am trying to run flutter from VSCode and I have installed flutter using the installation step and I can see the Flutter SDK in my path but visual studio says Flutter SDK not found
Logs
Run your application with flutter run and attach all the log output.
Run flutter analyze and attach any output of that command also.
Flutter Doctor
Paste the output of running flutter doctor -v here.

For more information about diagnosing and reporting Flutter bugs, please see https://flutter.io/bug-reports/.

Hi @ajaybeniwal, thanks for the feedback! Can you re-file your issue over at https://github.com/Dart-Code/Dart-Code/issues? Thanks!

@ajaybeniwal This has come up quite a bit and usually is one of the following:

PATH is not updated properly (eg. on Windows sometimes it seems to require a reboot for Code launched from Explorer to get the updated PATH)
PATH being set in a way that doesn't get passed to Code (eg. some custom shells seem to handle path differently)

If you're on Windows, I suggest trying a reboot. Otherwise, try launching code from the terminal immediately after confirming flutter is on PATH.
You can set the dart.flutterSdkPath setting in Code to tell it where your SDK is if you can't fix it another way, but it'd be best to have it detected from path.
If you still have issues, please file a bug at the address Devon gave above and/or grab me in Dart Code Gitter. Thanks!

For anyone still having path issues, please try v2.10.0 Beta 1 of Dart Code and it should now allow you to easily select the SDK folder from the error message.

@DanTup Helped, thank you

@eddyekofo94 Great! You don't need to use the beta above as that functionality shipped since.
If you have any further problems, feel free to raise issues at https://github.com/Dart-Code/Dart-Code

Hi Friend  i have small confuse i m web developer i want to like become App Developer.
So i am decided to Learn flutter whether its help for my life????
Please Any one give suggestion.
Thanks

@senthilamul iOS, Android for native app development.  Flutter for something more react like.

@senthilamul you can always switch from web to mobile development anytime with flutter but I would have preferred you have prior experience with mobile programming either with Android or iOS so that your transition to Flutter can be smoother but all the same, you can go ahead in your journey

Delete Cache folder under flutter/bin

Simply open Environment Variables, add new PATH where your flutter package is.
For example: C:\dart\flutter\bin
Restart visual studio code, open CMD and type

flutter
That will check for all files and download some if missing.
Reboot your PC and try creating new project using Visual Studio Code.

Simply open Environment Variables, add new PATH where your flutter package is.
For example: C:\dart\flutter\bin
Restart visual studio code, open CMD and type

flutter

That will check for all files and download some if missing.
Reboot your PC and try creating new project using Visual Studio Code.

@ajaybeniwal This has come up quite a bit and usually is one of the following:
1. PATH is not updated properly (eg. on Windows sometimes it seems to require a reboot for Code launched from Explorer to get the updated PATH)

2. PATH being set in a way that doesn't get passed to Code (eg. some custom shells seem to handle path differently)

If you're on Windows, I suggest trying a reboot. Otherwise, try launching code from the terminal immediately after confirming flutter is on PATH.
You can set the dart.flutterSdkPath setting in Code to tell it where your SDK is if you can't fix it another way, but it'd be best to have it detected from path.
If you still have issues, please file a bug at the address Devon gave above and/or grab me in Dart Code Gitter. Thanks!

Rebooting computer helped remove that message for me

@ajaybeniwal This has come up quite a bit and usually is one of the following:
1. PATH is not updated properly (eg. on Windows sometimes it seems to require a reboot for Code launched from Explorer to get the updated PATH)

2. PATH being set in a way that doesn't get passed to Code (eg. some custom shells seem to handle path differently)

If you're on Windows, I suggest trying a reboot. Otherwise, try launching code from the terminal immediately after confirming flutter is on PATH.
You can set the dart.flutterSdkPath setting in Code to tell it where your SDK is if you can't fix it another way, but it'd be best to have it detected from path.
If you still have issues, please file a bug at the address Devon gave above and/or grab me in Dart Code Gitter. Thanks!

Rebooting computer helped remove that message for me


fs: introduce FS_O_TRUNC fs_open flag

Let's add LFS_O_TRUNC has a supported flag for fs_open.
=> aligned with the open() usage
=> most FS already support a similar translation of this flag
=> provide a proper support for fopen("path", "w") which is a common pattern to create fresh file when using posix API

This needs a test that verifies it works correctly on all in-tree file systems.
I have a recollection this was proposed before, and it was discovered that the semantics of the flag was sufficiently different for littlefs and FatFs that it couldn't be done. But I may be mixing that up with another flag.

No problem to implement additional tests to tests/subsys/fs/fat_fs_api:

open with FS_O_CREATE | FS_O_TRUNC on an existing file and check the empty content

FS_O_TRUNC without FS_O_CREATE can be manage by truncating the file after fatfs f_open. I just wonder if this unusual usage need to be fully implemented or not ?

FS_O_TRUNC without FS_O_CREATE can be manage by truncating the file after fatfs f_open. I just wonder if this unusual usage need to be fully implemented or not ?

Whatever the Zephyr API specifies must happen needs to be implemented and tested.  If the API specification is unclear, that needs to be fixed.

Whatever the Zephyr API specifies must happen needs to be implemented and tested. If the API specification is unclear, that needs to be fixed.

The question was related to the way FA_APPEND was not implemented on Fatfs. I can be implemented (ie. seeking at end on every write) but the choice was made to not implement it. I guess this is due to the large amount of code required for this flag.
Zephyr still target small embedded devices and I just wonder how to balance between "full support" and "limited features support".
Personally, I'm fine with full support when the implementation is light (which will be the case with TRUNC)

I think that FS_O_TRUC should be completely served within fs_open with no changes to file systems drivers.

I don't really agree. Drivers are here to manage what can't be factorized, and flags translation is part of it.
Here is one example:
This pull request is a first step before submitting a "semihosting_fs" which allow access of the host (PC) FS from ARM through GDB / GDB+Qemu (https://developer.arm.com/documentation/dui0471/g/semihosting/semihosting-operations).
This is very useful for dev / testing / big file recoding while little flash, etc..
Unfortunately, there is not "truncate" operation available, and consequently, open+create+truncate can only be performed at driver open level.

Whatever the Zephyr API specifies must happen needs to be implemented and tested. If the API specification is unclear, that needs to be fixed.

The question was related to the way FA_APPEND was not implemented on Fatfs. I can be implemented (ie. seeking at end on every write) but the choice was made to not implement it. I guess this is due to the large amount of code required for this flag.

IIRC it was more that it couldn't easily match the behavior of the same flag on other file systems.
I don't object to FS_O_TRUNC being implemented at the file system level, rather than in the wrappers, but only if the behavior is the same on all file systems, as verified by a test.  If it can't, then it needs to be done at the wrapper level, or not done at all.

IIRC it was more that it couldn't easily match the behavior of the same flag on other file systems.

Yes that is true, in case of ELM FAT it means that the file position will be set at the end on open but till not be reset to that position on every write.

Unfortunately, there is not "truncate" operation available, and consequently, open+create+truncate can only be performed at driver open level.

That is actually valid point.

@invoxiaamo Are you willing to add the tests?

hello @de-nordic. I was in holidays and my spare time to work on Zephyr has collapsed in the meantime. I still will to finalize this feature + tests but don't expect any progress too soon...

I'm going to mark this stale, since it had no activity for a long time until zephyrbot touched it.  If it's still going anywhere please remove the label and update it.

I'm going to mark this stale, since it had no activity for a long time until zephyrbot touched it.  If it's still going anywhere please remove the label and update it.


PBL-37865: pybluetooth socket should immediately release device in del()

The pyusb device was being claimed longer than needed, preventing a BTStack() from grabbing the device after a pyusb_bt_sockets.has_bt_adapter() call.
Devices matching both the CUSTOM_USB_DEVICE_MATCHER and default bt_adapter_matcher would appear twice in the results of find_all_bt_adapters().

Looks good, we'll need to bump the version number too to re-deploy on PyPi I guess? 👍 🚢

re-deploy on PyPi I guess
Yes, I'll do that after dinner.


cookie doesnt work + need exe

Cookie doesnt get logged + I tried to use py to exe but it doesnt worked, can u add exe compiling option

What you mean it doesnt get logged, i need a screen please, and what is the error with pytoexe (btw, this is bad, use pyinstaller it's still bad, but better)

What you mean it doesnt get logged, i need a screen please, and what is the error with pytoexe (btw, this is bad, use pyinstaller it's still bad, but better)

Okay, did you try using pyinstaller, and did you use obfuscated one, or not? and this error can happen if your wifi is very slow sadly, i can't really help with that

hello?

Cookie doesnt get logged + I tried to use py to exe but it doesnt worked, can u add exe compiling option

I bet its dualhooked skid virus lol dont trust this guy

Give me any proof ?

And like every other, they don't have any proof of "dualhook" or "virus"


[WIP] Design panel refactor

Summary
Very early WIP for design panel refactor.
Only the sizePosition panel has been implemented so far.
Very little documentation is added to the new structure, but also very little needed.
Note that the old panels still exist - they'll of course be deleted completely once this is done.
Relevant Technical Choices

To-do
A lot!
User-facing changes

Testing Instructions

Fixes #4104

This is too complex and also slightly outdated. Will re-implement this in much smaller steps in subsequent PR's


Including appendix sections

eLife articles may contain one or more appendices. These can contain their own assets:
figures
figure supplements
tables
videos
source data and source code related to the above
References in eLife appendices got into the main reference list used for the whole article.

It is also important for Érudit

@JGilbert-eLife can we break this down into separate issues with examples?

I've added individual tickets for
Figures: #1026
Tables: #1027
Videos: #1028
The XML for those individual items is the same as in the main text; what is different is it is appearing within the appendix structure and the labels take the appendix number as a prefix (e.g. Appendix 1—figure 1).
We (eLife) need to nail down an issue with appendix tagging before I can post examples of that, but hopefully will be able to do so by the end of the week.


"Animating auto" demo breaks with v9

Not sure if this is a bug but I'd like to know how to make it go away:
🐛 Bug Report
The useSpring docs contain a demo of a tree component with nested elements whose heights are animated to "auto" using a bound ref.
This demo doesn't work when react-spring is updated to the latest v9 beta, as the outer element height doesn't animate to expand when inner elements are expanded.
To Reproduce
I forked the demo and updated the react-spring dependency to the latest beta; no other changes were made.
Click on a collapsed node to see the issue.
Expected behavior
Same behavior as with current stable react-spring: outer height should animate to expand when inner node is expanded.
Environment

react-spring v9.0.0-beta.31
react v16.8.6

I figured out that the issue was not with v9, but with the check setting the height to auto:
{ height: isOpen && previous === isOpen ? 'auto' : height }

Since root element doesn't have a previous element, this check was failing in its case. Needed to put a prop on the root element and add it to the check above.
{ height: isOpen && ( previous === isOpen || isRoot ) ? 'auto' : height }

Closing with thanks.

Just wanted to add that although adding the root check works to expand elements as expected, it stops the root element itself from opening once it's been collapsed.
Still leaving this issue closed because I don't need to collapse the root node, but noting it here in case anyone wants to look into this and provide an explanation or solution.


Error on adding wallet over cel-key

Celestia Node version
v0.18.2-mocha branch
OS
ubuntu 22.04
Install tools
go version go1.23.1 linux/amd64
Others
No response
Steps to reproduce it

Clone the repository
Make a checkout to branch v0.18.2-mocha branch
Execute make build command
Move celestia and cel-key built binaries to corresponding go directories to make them executable over cli.
Execute command cel-key add bridge-wallet --node.type bridge --p2p.network mocha

Expected result
Wallet with name bridge-wallet is created successfully.
Actual result
Got an error
Error: no or invalid node type provided
Relevant log output
cel-key add bridge-wallet --node.type bridge --p2p.network mocha
Error: no or invalid node type provided
Usage:
  keys add <name> [flags]

Flags:
      --account uint32           Account number for HD derivation (less than equal<PHONE_NUMBER>)
      --algo string              Key signing algorithm to generate keys for (default "secp256k1")
      --coin-type uint32         coin type number for HD derivation (default 118)
      --dry-run                  Perform action, but don't add key to local keystore
      --hd-path string           Manual HD Path derivation (overrides BIP44 config)
  -h, --help                     help for add
      --index uint32             Address index number for HD derivation (less than equal<PHONE_NUMBER>)
  -i, --interactive              Interactively prompt user for BIP39 passphrase and mnemonic
      --ledger                   Store a local reference to a private key on a Ledger device
      --multisig strings         List of key names stored in keyring to construct a public legacy multisig key
      --multisig-threshold int   K out of N required signatures. For use in conjunction with --multisig (default 1)
      --no-backup                Don't print out seed phrase (if others are watching the terminal)
      --nosort                   Keys passed to --multisig are taken in the order they're supplied
      --pubkey string            Parse a public key in JSON format and saves key info to <name> file.
      --recover                  Provide seed phrase to recover existing key instead of creating

Global Flags:
      --home string              The application home directory (default "~")
      --keyring-backend string   Select keyring's backend (os|file|test) (default "test")
      --keyring-dir string       The client Keyring directory; if omitted, the default 'home' directory will be used
      --node.type string         Sets key utility to use the node type's directory (e.g. ~/.celestia-light-celestia if --node.type light is passed).
      --output string            Output format (text|json) (default "text")
      --p2p.network string       Sets key utility to use the node network's directory (e.g. ~/.celestia-light-mynetwork if --p2p.network MyNetwork is passed). (default "celestia")

Is the node "stuck"? Has it stopped syncing?

Notes
No response

Also see that's reproducible for v0.16.2-mocha version

Can confirm the same error

confirm

Confirm the same error


macOS support

related pull request
https://github.com/yahoojapan/SwiftyXMLParser/pull/6

This issue has already been solved in https://github.com/yahoojapan/SwiftyXMLParser/pull/14


Can't extract argument valid rule if multiple rules existed

Description
Can't extract argument valid rule if multiple rules existed
Steps to Reproduce

creat a script tmp.sh with the following content:

#!/usr/bin/env bash

source <(docker run --rm renaultdigital/bash-base)

args_parse $# "$@" destGitRepoURL

if [[  -n "${currentGitRepoURL}"  ]]; then
  args_valid_or_read destGitRepoURL '^(https|git).+\.git$' "git repository " "${currentGitRepoURL}"
else
  args_valid_or_read destGitRepoURL '^(https|git).+\.git$' "git repository "
fi

chmod +x tmp.sh
./tmp.sh -h

Expected behavior:
...
DESCRIPTION
    [-h]                help, print the usage
    [-q]                optional, Run quietly, no confirmation

    destGitRepoURL      git repository
...

Actual behavior:
...
DESCRIPTION
    [-h]                help, print the usage
    [-q]                optional, Run quietly, no confirmation

    destGitRepoURL
...

Reproduces how often:
always
Versions
all
Additional Information
None

:tada: This issue has been resolved in version 1.4.5 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release
v1.4.5

Your semantic-release bot :package::rocket:


AddValue return value policies are inconsistent

The implementation of AddValue in the case where arguments.Get(0) is a list will modify each the list by appending each element, and then return the last element added.
If arguments.Get(0) is a dictionary, then it returns a count of the size of the map after all additions have been made.
So I would think that the obvious thing to return is the first object, to facilitate chaining of operations. That is, if you just built up a list or map, then you probably want to do something with it.
Are there use cases for the current policies?

Actually, in the GIT repository it actually returns the last value if the first object is a Dictionary. I just haven't created a release of it quite yet. In fact, if I am not entirely wrong, I think this was implemented according to your wishes some time back ago.
(Not to self) - Create a new Release on Sunday, or during the weekend ... ;)


chore(dependabot): adjust conf to relax bumping flexible requirement specs

Purpose
Quiet down the number of dependabot PRs for every minor or patch version that comes out when the current range definition will automatically pull the latest dependency for the user/developer.
Rationale
This more so comes from experience that the default strategy for dependabot does not bode well for using ranges. It has a tendency to bump the lowest part of the range up until there is no more range!  That was definitely not the intention so this strategy adheres to ranges and only opens a PR when a version outside of the range is available.

@relekang , @bernardcooke53, ready for review


MVRVA Transport Beta - Capstone Project

In this Project, we:
@Kanyorok
@jkanyi-web
@vangelif
@Rifat8080
@Al-Amin-Khan-Shakil
implemented the kanban board and assigned the cards between front and back end teams and to specific team members within the teams.
Kanban Board HERE

Hi,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You have submitted a project, that does not contain a screenshot of the kanban board.
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.

Hi,
Good job so far!
There are some issues that you still need to work on to go to the next project but you are almost there!

Required Changes ♻

[ ] This card does not have an assignee, points, or a description.  Does it belong to the kanban board?  Kindly take a look. 👍

Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

this?

this?

@vangelif I forgot to paste the image.  Here it is:

Wow, you did it 🎉

Thank you for implementing the changes requested by the previous reviewer 💪 🥇 ㊗
Well done!
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
To highlight

All the cards have assignees, pointings and descriptions now✔

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


chore: run CI on latest Node.js

Part of https://github.com/netlify/team-dev/issues/19
This runs the CI on the latest Node.js version.

There seems to be some issue on Node 16. Looking into it now.

The error is just a deprecation warning due to the main field in one of the ESLint plugins being invalid.
This does not impact how ESLint runs, it only prints a warning.
More information at https://github.com/you-dont-need/You-Dont-Need-Lodash-Underscore/issues/317 and https://github.com/you-dont-need/You-Dont-Need-Lodash-Underscore/pull/315


Calculate size

option to only calculate without deletion

This works well, however if you run this on a bigger stack of versions it might take a while and showing the result in the notification might be missed. Could this be tweaked to show the result in another interface widget that replaces the first one? Ideally it would tell you the size, and user could just continue in removing calculated files.
@tokejepsen what do you think?

I've logged the size as well, so it persistent there. Dunno what the intent of the logging is for the end user?
Only other way would be to display the interface again, but not sure the action handlers are designed for this? @iLLiCiTiT

Only other way would be to display the interface again, but not sure the action handlers are designed for this?

It is possible and there are few approaches for that, all of them must be handled properly... One possible approach is in action_delete_asset.py. I can look into that when I'll get back from holiday.

Last option is to show a job, which might be good anyways to show progression. The job title could be the size.

I might take this logic into a Library Loader instead; https://github.com/pypeclub/pype/issues/396

@tokejepsen @iLLiCiTiT
I'm happy to merge this and update later on.


Is this project alive?

Looks like there is no any activity since summer 2014. =/

@schmittjoh It's sad to see this project dying, lots of people use it and there's already an ecosystem around it. If you need help with maintaining it I'm available, you can get in touch at @gmail.com.

Just a heads up that Johannes has given me access to this repo and I'll be helping with maintaining it from now on.


SSH channel - reconnect if the transport is not active

Type: new feature (improvement for SSH channel)

Implements #1780

Description
Checks whether underlying SSH transport is active before executing SSH commands or performing SFTP operations. Attempts to re-establish the connection if it is not active.

@benclifford This will help us to run long-running Parsl jobs from a remote node over the SSH channel.

@benclifford I agree. Made changes as suggested.

@benclifford do you need me to work on any other changes?

@benclifford do you need me to work on any other changes?

No, i think this is fine. I was waiting to test it out on one of my systems but for various reasons that's been delayed, so I will merge this now.


Fixed the calidate user funciton

Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

L)


Review results of bandit on master

See original issue on GitLab
In GitLab by [Gitlab user @traveltissues] on Nov 26, 2019, 15:19
now that there is a sast job in ci, at least the high risk items should be reviewed: https://gitlab.com/BuildStream/buildstream/security/dashboard/?project_id=1975139&scope=dismissed&page=1&days=90

In GitLab by [Gitlab user @traveltissues] on Dec 2, 2019, 16:06
assigned to [Gitlab user @traveltissues]

In GitLab by [Gitlab user @traveltissues] on Dec 2, 2019, 16:06
assigned to [Gitlab user @traveltissues]

In GitLab by [Gitlab user @tlater] on Dec 2, 2019, 16:42
mentioned in merge request !1743

In GitLab by [Gitlab user @tlater] on Dec 2, 2019, 16:42
mentioned in merge request !1743

In GitLab by [Gitlab user @traveltissues] on Dec 2, 2019, 16:50
assigned to [Gitlab user @tlater] and unassigned [Gitlab user @traveltissues]

In GitLab by [Gitlab user @traveltissues] on Dec 2, 2019, 16:50
assigned to [Gitlab user @tlater] and unassigned [Gitlab user @traveltissues]

In GitLab by [Gitlab user @tlater] on Dec 2, 2019, 16:58
mentioned in merge request !1744

In GitLab by [Gitlab user @tlater] on Dec 2, 2019, 16:58
mentioned in merge request !1744

In GitLab by [Gitlab user @tlater] on Dec 9, 2019, 17:49
mentioned in merge request !1763

In GitLab by [Gitlab user @tlater] on Dec 9, 2019, 17:49
mentioned in merge request !1763

In GitLab by [Gitlab user @tlater] on Feb 6, 2020, 01:37
unassigned [Gitlab user @tlater]

In GitLab by [Gitlab user @tlater] on Feb 6, 2020, 01:37
unassigned [Gitlab user @tlater]


Update README.md

will be where

Alright, I'll merge. Please bring your laptop tomorrow so that I can teach you how to use GitHub and Git. Thanks for contributing!


unclear navigation to next card

feedback:
I find the arrows to go back and forth a bit confusing, since I was not sure if the content was specific for the card I selected or if the arrows were meant for all the cards available. There is a lot of text and content. Therefore, the navigation should be even more clear so that I don't have to read all the time what is in the cards to see where I am.

at one point we discussed not having a stack in postcard view, but a card on the left and right of the current card. That could be something we take into consideration again if this is an ongoing problem

I find this issue very subjective and I don't think we would be able to solve it without re-designing the whole navigation.
Closing for now, up for discussion.


[DNM]support batch send msg from raftstore to server

This pr aim to reduce the channel cost when there are a huge number of little message. I will introduce bench result later.

can we close this now?


fix: avoid race condition on permissions

The policy and the Alexa skill are created at the same time, although the skill needs permission defined in the policy to succeed. This fix adds an explicit dependency.

It's still valid


Add a note about C++ compilers to README.md

This updates the README text to include the need for a suitable C++ compiler when installing pyodbc, and a reference to the appropriate wiki page as well.  Hopefully this will help people when installing pyodbc for the first time.

Great idea - I still field direct email questions on this all the time.


Mac DNS resolution failing CGO_ENABLED=0 for macos

What happened?
Hi I have a DNS resolution problem on my Mac (M1) with pulumi AWS provider
I keep getting error
 1 error occurred:
    	* error reading EC2 VPC (vpc-0f364d0fbd5be0015) ClassicLinkDnsSupported: RequestError: send request failed
    caused by: Post "https://ec2.eu-central-1.amazonaws.com/": dial tcp: lookup ec2.eu-central-1.amazonaws.com on <IP_ADDRESS>:53: no such host

My DNS servers are coming from DHCP server (my router):
nameserver <IP_ADDRESS>
nameserver <IP_ADDRESS>
nameserver <IP_ADDRESS>

When I set my DNS server to <IP_ADDRESS> pulumi then starts working
I suspect it is the same case as these issues
https://github.com/hashicorp/terraform/issues/3536
https://github.com/hashicorp/terraform-provider-kubernetes/issues/969
Steps to reproduce
n/a
Expected Behavior
no dns problems
Actual Behavior
dns is not resolving
Output of pulumi about
5.16.2
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

Perhaps someone can help me modify my fork to build the binary I can test https://github.com/jans-forks/pulumi-aws/actions/runs/3288534376/jobs/5418989775

Hi @1oglop1 - you should be able to build your own provider locally, running make provider.
As for the resolution failure, we're not certain it's an M1 issue specifically, but rather a result of an upstream Golang resolver issue and as such we have very limited options to address this here.

The linked upstream issue was fixed.


Repo description typo

In the GitHub repo description, it says "mermade.js" instead of "mermaid.js"

Thanks!


list of interval variants

Hello, I performed mapping about two traits on CeNDR.
Two traits are similar, so the mapping results of them overlap each other.
The results have one common peak (common peak position and region), but the peaks have different numbers of interval variants.
And the two traits also have similar peaks (different peak positions but same region). They have different numbers of interval variants, too.
I thought that the list of interval variants would be the same if the regions were the same.
I wonder what made them different.
Best,
Jiseon

Hi Jiseon,
Unless the traits are identical, then the correlations of genotype and phenotype are going to be different. This difference will go all the way to the variant level. Additionally, if the two traits have slightly different strains that were mapped, then the variants will also be different. If you would like to share your mapping reports with me (by DM), I can offer some more specific advice.
Best wishes and stay healthy,
Erik

Hello Erik,
Thank you for the quick reply.
The difference of variants seems to be due to the difference in the mapped
strains as you mentioned.
Best,
Jiseon
2020년 3월 24일 화요일, Erik Andersen<EMAIL_ADDRESS>작성:

Hi Jiseon,
Unless the traits are identical, then the correlations of genotype and
phenotype are going to be different. This difference will go all the way to
the variant level. Additionally, if the two traits have slightly different
strains that were mapped, then the variants will also be different. If you
would like to share your mapping reports with me (by DM), I can offer some
more specific advice.
Best wishes and stay healthy,
Erik
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/AndersenLab/CeNDR/issues/136#issuecomment-602959998,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANUZHDQ7GXQ7R6IX47QARSLRJAGPNANCNFSM4LRVVTIA
.

Hi Jiseon,
Makes sense. I hope that it all works out for you.
Best wishes,
Erik


Flow types - Indexable signature not found in List

Hi there,
Just highlighting a case I've found with flow, that works, but flow complains about
take this code ...
    const [first, second] = Immutable.List.of('a', 'b');
    console.log(first); // Logs 'a'
    console.log(second); // Logs 'b'

Works as expected ... but flow will complain with "Indexable signature not found in List"
Would be useful to have flow not complain, raising without a PR because I don't know enough about complex types in flow to implement a fix it quickly (hoping someone else does)

I suggest opening this bug against the Flow project - unfortunately this is not solvable from this codebase


deploy: move vol-rep crds to kubernetes-csi-addons

This PR moves the volume replication operator to kubernetes-csi-addons.
This includes copying the types.go file and use it to generate the crds
and add rbac rules
Signed-off-by: yatipadia<EMAIL_ADDRESS>

It seems a few files were modified/created, and those need to be checked in as well.

follow the steps to transition current apis and controller to multigroup folder layout https://kubebuilder.io/migration/multi-group.html

Hey, @Rakshith-R do you mean for adding controllers/ part , didn't get what is missed.

@Rakshith-R @nixpanic @Madhu-1


Ref/sat to CTE

Hi @DVAlexHiggs
I did not mean to open a PR on your repo, I wanted to do on my fork,
BUT as I sent it your way this is related to my comment in Slack from this evening.
Step 1 refactor using CTE, step 2 add filtering on the source table.
Self closing this, but I'd like your opinion on the code... :)

HI Rob. These changes make sense, however we'll soon be releasing an update to satellites with some similar changes and improvements, as well as support for a new materialisation we're developing!
Thanks for your contribution


Enhancement: Have ability to add 0 for %N to aid in sorting exported assets

Hello, I love the plugin!  One thing that would be nice is if you could specify adding a "0" when using %N for the cases of selections that are between 1 and 9.  This is because when sorting exported assets on the filesystem, "11-layername.png" with come before "2-layername.png".

You can do this by adding %NN into the input.  Hope t

Oh awesome - thanks for the tip!


Error "Incompatible API version (3.0)"

I am trying to install the extension in TFS 2015.3 and I am getting the error "Incompatible API version (3.0)"
Maybe I downloaded the extension for TFS 2017?
where can I download the extension for TFS 2015.3?
Thanks

@leonjalfon1 That is by design, the extension relies on APIs that were only introduced with the API version 3.0.


nixos/systemd-userdbd: add option to enable

Description of changes
This service is shipped as a part of upstream systemd but not currently
incldued in NixOS. This change adds a configuration option which may
be used to enable it
Things done

Built on platform(s)

[ ] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[x] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

(My motivation for this is that I have a PAM module which depends upon userdbd)

Bit off topic but pam_systemd already supports systemd-userdb. Why a new
pam module?
On Sun, 10 Jul 2022, 11:33 Erin, @.***> wrote:

(My motivation for this is that I have a PAM module
https://github.com/erincandescent/pam_sduserdb which depends upon
userdbd)
—
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/180952#issuecomment-1179692654,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEZNI2NDWNU3DLBLRV4TIDVTKKHVANCNFSM53EXLFPQ
.

It would be nice if we can add a VM test that makes sure all the userdbd stuff works.  e.g. that we can query /etc/shadow through it etc.   Maybe just some calls to userdbctl to make sure everything is alright as a smoke test? This just makes sure that we don't accidentally break it with our systemd-specific patches.  Apart from that LGTM.

Thanks gotcha. It seems the account module that systemd ships is systemd-home but that only deals with user records in the home folder; not generic user records. Bit surprising to me!

On other Linux systems this would be handled via pam_unix -> getspent(3) -> nss_systemd but shadow can't go through nscd (unless you want to do all your DNS resolutions as root 😨)

Bit tempted to ship pam_sduserdb in NixOS proper now and use NSCD just for DNS.  I think it would solve a lot of our issues we've had with nscd, dynamic users and service restarts...  (CC @flokli )

You still need nss_systemd (and hence nscd) for all apps which do traditional NSS based user lookups (getpwent, getpwnam, getpwuid, etc)

@erincandescent would you want to add a simple VM test?

@erincandescent if you can't add a test, can you describe how to test this?
Including this in a VM test is important, so we spot potential regressions, and also see how this is supposed to work.

The userdb command isn't helpful here, because if it can't talk to userdbd it falls back on its own internal implementation of it.
So I think the best bet is to pull in libvarlink and do a direct call to the service. Something like checking the following succeeds:
varlink call unix:/run/systemd/userdb/io.systemd.Multiplexer/io.systemd.UserDatabase.GetUserRecord '{"uid":0, "service":"io.systemd.Multiplexer"}'


Support testing l1_handlers

To what component is your bug related to?
Forge
Feature Request
Add support for calling #[l1_handler] from test.
While we do not yet have full l1<>l2 message testing, we should at least have the ability to directly call l1_handlers and test their output

Probably relevant to #347


[ADDITION] Hyper Anna

Category
Software to be added
Hyper Anna
Supporting Material
URL: https://www.hyperanna.com/
Description: AI-powered data insights meets interactive storytelling in real time.
Size:
HQ: Sydney
LinkedIn: https://www.linkedin.com/company/alteryx-auto-insights/
See Record on Airtable:
https://airtable.com/app0Ox7pXdrBUIn23/tblYbuZoILuVA0X3L/recdntt02WcvVC6jR

Acquired by US company, adding to extended list


What's the relationship to github.com/google/goacme?

Hi, I just wanted to ask this question to learn.
What's the relationship of the ACME library in this repository to the one in https://github.com/google/goacme? I've learned about recently from https://github.com/golang/crypto/commit/1777f3ba8c1fed80fcaec3317e3aaa4f627764d2.
Thanks!

Not related at all.

Do the two libraries do different things?

Not that I'm aware of. Google chose to reimplement the ACME spec themselves.
The only difference that I am aware of is that goacme does only implement HTTP01.

I see, thanks.
I'll close this since my question was answered.
But I do wonder, for someone looking for an ACME library, how they should choose between these two seemingly similar libraries...

We did offer to join efforts, but were never approached from anyone. It is very unfortunate that things turned out this way.


Add documentation for Automation

Add documentation for different automation we support.
How it works.
How can an admin enable automation.

#210


api: ext-proc attributes

What type of PR is this?
Following the discussion at KubeCon NA : https://docs.google.com/document/d/1PS5xLA0IDbj6McHIXXhShn51Zq37WvuistaidBPeHoE/edit?tab=t.0
Scope and Motivation
This API will allow users to define which attributes are sent to the external processor as context for requests/responses.
Attributes provide HTTP extensions with additional context (e.g. TCP, TLS and XDS attributes) that can be relevant inputs for the extension logic.
Comparison to other extension options
Many Envoy extensions are inherently capable of interaction with context attributes:

lua: stream and connection attributes are available through StreamInfo
wasm: get attributes through get_property

For out-of-process extensions like ext-proc, Envoy must be explicitly configured to allow access to attributes and and define the scope of access. With the increase in ext-proc use cases, such as the llm-instance-gateway, envoy-ai-gateway and externally-deployed WAFs, there is a greater need to provide extension with context.
Security Considerations
Most attributes are scoped to the current connection or stream by their prefix (connection.*, request.*, response.*), with the exception of xds.* attributes such as xds.listener_metadata, xds.upstream_host_metadata, xds.node and generic metadata and filter state access attributes.
Related Work

Issue: https://github.com/envoyproxy/gateway/issues/3170
Previous Attempts by various contributors:

#4496
#4160
#3247

Release Notes: Yes

curious what happens when something like connection.unknown is used ? is it a no-op ?

curious what happens when something like connection.unknown is used ? is it a no-op ?

Yes, it's just ignored and not passed. Config is still accepted by the proxy.