How to know when textures are completely loaded/applied in scene graph?

Hi,
How to know when textures are completely loaded/applied in scene graph? I see there is a load event for src parameter, but that does not take into account the textures being changed in future through scene graph.

The setSource() method is async, so if you await it, it should be ready at that point. If that's not the case, it's a bug.

I tried loggin this line
let  applyPBRTexture =  (channel) => {
  	        //console.log(event.target.value);
  	        material.pbrMetallicRoughness[channel].texture.source.setURI(baseColor);

  	        //removeLoader();
  	      }

  	      let applyPBRNormalTexture =  (channel) => {
  	        //console.log(event.target.dataset.normalmap);
  	        material[channel].texture.source.setURI(normalMap);
  	        //removeLoader();
  	      }

  	      let applyOcclusionPBRTexture =  (channel) => {
  	        //console.log(event.target.dataset.occlusion);
  	        material.pbrMetallicRoughness[channel].texture.source.setURI(occlusionMap);
			//console.log(material.pbrMetallicRoughness[channel].texture.source.setURI(occlusionMap));

  	      }

		  async function loadTextures()
		  {

      	   await applyPBRTexture('baseColorTexture');
      	   await applyPBRNormalTexture('normalTexture');
      	   await applyOcclusionPBRTexture('metallicRoughnessTexture');

		   console.log(await applyOcclusionPBRTexture('metallicRoughnessTexture'));

		     //
		  }

		  loadTextures();

that's how I have setup the code. I am actually trying to creating loading spinner and then the model is completely loaded including the textures I want to it to disappear. All the css part is there, its jut that when I put it after the console line, It does not appear at all as if it is not waiting for the function to finish. And when I log this line
	   console.log(await applyOcclusionPBRTexture('metallicRoughnessTexture'));

I do get promise status as fulfilled.

I think you may want to read up a bit more on async JS first. You're trying to await functions you've written as non-async, which doesn't do anything.

Got it. Thank you.


Symbols, not strings.

For locales. See d3/d3-shape#23.

Anyway to get the locale definition? ex: decimal, thousands


Statistics

It would be nice if the results of all played rounds were saved and shown in some kind of table on the edge. Just playing but never really knowing how you actually did is kind of disappointing...

 Juni 2023,
23:16:

It would be nice if the results of all played rounds were saved and shown
in some kind of table on the edge. Just playing but never really knowing
how you actually did is kind of disappointing...
—
Reply to this email directly, view it on GitHub
https://github.com/MOD0912/Blackjack/issues/3, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARMQYKDLTIAN46HDQJRVLV3XLN3UFANCNFSM6AAAAAAZIM65GQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


"results" folder not stored when using nextflow engine

Discussed in https://github.com/aws/amazon-genomics-cli/discussions/481

Originally posted by MrOlm June 13, 2022
Hello,
After a Nextflow run there is usually a "results" folder created, but that output is uploaded to AWS at the end of a nextflow amazon genomics cli run.
For example, the rnaseq workflow provided as an example (https://github.com/aws/amazon-genomics-cli/tree/main/examples/demo-nextflow-project/workflows/rnaseq) results in the following output on my end:
(base) mattolm@mac-nugget:$ aws s3 ls agc-698960807664-us-west-2/project/NextflowDemo/userid/mattolm4x9RhE/context/spotContext/nextflow-execution/
                           PRE logs/
                           PRE runs/

(base) mattolm@mac-nugget:$ aws s3 ls agc-698960807664-us-west-2/project/NextflowDemo/userid/mattolm4x9RhE/context/spotContext/nextflow-execution/runs/
                           PRE 23/
                           PRE 5c/
                           PRE 62/
                           PRE fd/
                           PRE stage/
                           PRE tmp/
2022-06-13 14:11:23          0

And when I ask for the workflow output I just get the run-id:
(base) mattolm@mac-nugget:$ agc workflow output aca7054d-3261-4c29-8cee-53efb5aed86a
2022-06-13T14:23:29-07:00 𝒊  Obtaining final outputs for workflow runId 'aca7054d-3261-4c29-8cee-53efb5aed86a'

OUTPUT  id      aca7054d-3261-4c29-8cee-53efb5aed86a

However, according to the actual repo (https://github.com/nextflow-io/rnaseq-nf), there should be a folder and file created called results/multiqc_report.html. When I run this same command locally, as below, the results folder is indeed created:
(base) mattolm@mac-nugget:$ nextflow run https://github.com/nextflow-io/rnaseq-nf.git -resume -params-file ../workflows/rnaseq/inputs.json
N E X T F L O W  ~  version 21.10.6
Launching `nextflow-io/rnaseq-nf` [tiny_marconi] - revision: 37c5039435 [master]
 R N A S E Q - N F   P I P E L I N E
 ===================================
 transcriptome: /home/mattolm/.nextflow/assets/nextflow-io/rnaseq-nf/data/ggal/ggal_1_48850000_49020000.Ggal71.500bpflank.fa
 reads        : s3://1000genomes/phase3/data/HG00243/sequence_read/SRR*_{1,2}.filt.fastq.gz
 outdir       : results

Uploading local `bin` scripts folder to s3://agc-698960807664-us-west-2/project/NextflowDemo/userid/mattolm4x9RhE/context/spotContext/nextflow-execution/runs/tmp/59/64a5bb5c2c0bfb58a1bb7b8ec38a4e/bin
executor >  awsbatch (4)
[46/644f98] process > RNASEQ:INDEX (ggal_1_48850000_49020000) [100%] 1 of 1 ✔
[cc/41410f] process > RNASEQ:FASTQC (FASTQC on SRR099964)     [100%] 1 of 1 ✔
[63/f9c8dd] process > RNASEQ:QUANT (SRR099964)                [100%] 1 of 1 ✔
[da/a87450] process > MULTIQC                                 [100%] 1 of 1 ✔

Done! Open the following report in your browser --> results/multiqc_report.html

Completed at: 13-Jun-2022 15:22:30
Duration    : 26m 42s
CPU hours   : 0.7
Succeeded   : 4

(base) mattolm@mac-nugget:$ ls
nextflow.config  results  work

(base) mattolm@mac-nugget:$ ls results/
fastqc_SRR099964_logs  multiqc_report.html

It's not a big deal for a small workflow like this, but when I ran a larger nf-core workflow (https://nf-co.re/mag?q=mag) there is supposed to be a beautiful "results" folder created, but I only the messy "runs" folder above is uploaded at the end.
Thank you in advance for your help,
MO

This is prohibiting regular/routine use of AGC for us as well. Publishing outputs in a specific directory to rename and organise results is a key feature for many NF pipelines and we'd love to see this handled by AGC.


pytube.exceptions.RegexMatchError: regex pattern

While the downloads work fine for many videos it fails on some like this video:
https://www.youtube.com/watch?v=tOMPGQP1G8o
the error is :
File "/usr/local/lib/python2.7/dist-packages/pytube/cipher.py", line 252, in get_signature
    tplan = get_transform_plan(js)
  File "/usr/local/lib/python2.7/dist-packages/pytube/cipher.py", line 71, in get_transform_plan
    name = re.escape(get_initial_function_name(js))
  File "/usr/local/lib/python2.7/dist-packages/pytube/cipher.py", line 47, in get_initial_function_name
    return regex_search(pattern, js, group=1)
  File "/usr/local/lib/python2.7/dist-packages/pytube/helpers.py", line 41, in regex_search
    .format(pattern=p),
pytube.exceptions.RegexMatchError: regex pattern (yt\.akamaized\.net/\)\s*\|\|\s*.*?\s*c\s*&&\s*d\.set\([^,]+\s*,\s*(?:encodeURIComponent\s*\()?(?P<sig>[a-zA-Z0-9$]+)\() had zero matches

Maybe related to this - https://github.com/jdf76/plugin.video.youtube/pull/624/commits/b7ef984b65c055ea254fe42b63a97e37a65fb056

@Elite  will your commit resolve current issue

Based on what @Elite posted, I changed cipher.py line 40 to:
r'.*?\s*c\s*&&\s*d|a\.set\([^,]+\s*,\s*(?:encodeURIComponent'
and line 43 to:
r'\bc\s*&&\s*d|a\.set\([^,]+\s*,\s*(?:encodeURIComponent'
and this resolved the problem for at least one video. No time to test properly and make a pull request right now, though.

The signature is getting generated but resulting link fails to download, let me check this.

It seems not all signatures are valid, many videos produce invalid signatures.

The video in the README produces the error

Same thing here.

I think this project is not being maintained anymore? Can anybody confirm this and if possible share an alternative.

Well ... it is maintained. Just slowly.
If you try the latest version pip install git+https://github.com/nficano/pytube.git --upgrade, your video works for me


Set up database connection and create tables

Relates to issue #143 and #154
Overview

[x] Added partial newDBLayer function which sets up the db connection and runs migrations.
[x] Added a small test case to check that the connection works and tables exist.

@rvl Please notice that the branch was rebased against rvl/147/sql-schema (which was rebased before against master) and forced pushed, so if #227 is merged then you can put it against master and it should compile and be green (made needed fixes and also cleaned imports). I started addressing PrivateKey table and stumbled into problem how to really represent  (Key 'RootK XPrv, W.Hash "encryption") . I tried to represent them as (Text, Text) via ByteString but then have problem to return from db to Key 'RootK XPrv

@paweljakubas Thanks :smile:

@rvl Please read it before you continue working :

rvl/147/sql-conn is rebased against master, compiles and should be green
I ported all tests from MVarSpec adding SQL flavor (especially to wallet state which is represented as (UTxO, PendingTx))
A number of tests does not pass - I denoted them as xit rather than it (at this moment 16 tests do not pass)
I moved shared parts between MVar and SQL to DBSpec - still properties can also be moved to DBspec and shared as they are almost the same - maybe we can do it at the end together with lens refactoring

@rvl Two things done:

ErrWalletAlreadyExists supported
Handled quite rare :

test/unit/Cardano/Wallet/DBSpec.hs:403:9:
 1) Cardano.Wallet.DB.Sqlite, put . read yields a result, Wallet Metadata
      Falsifiable (after 30 tests):
        (PrimaryKey (WalletId {getWalletId = 1d05f4517d2decf9b118b7a48e46a534b42f67a4}),WalletMetadata {name = WalletName {getWalletName = "bulbazaur"}, passphraseInfo = Just (WalletPassphraseInfo {lastUpdatedAt = 1864-04-28 23:32:45.015596992627 UTC}), status = Restoring (Quantity (Percentage {getPercentage = 100})), delegation = NotDelegating})
      expected: Just (WalletMetadata {name = WalletName {getWalletName = "bulbazaur"}, passphraseInfo = Just (WalletPassphraseInfo {lastUpdatedAt = 1864-04-28 23:32:45.015596992627 UTC}), status = Restoring (Quantity (Percentage {getPercentage = 100})), delegation = NotDelegating})
       but got: Just (WalletMetadata {name = WalletName {getWalletName = "bulbazaur"}, passphraseInfo = Just (WalletPassphraseInfo {lastUpdatedAt = 1864-04-28 23:32:45.015596992627 UTC}), status = Ready, delegation = NotDelegating})

Added unit tests exemplifying this case plus changed Gen .

bors r+


It's should be possible to start() a LanguageClient that failed previously.

My use case is the following:
I have a plugin which will read the settings to start from the configuration (i.e.: executable / arguments), so, whenever those settings are changed, the LanguageClient is restarted.
This works well enough, but if the user has some wrong setting (say, he mistyped the executable) then an error will be raised when starting the Language (the ServerOptions passed to the LanguageClient is a function which is called when the LanguageClient is started, but in this case it'll throw an error -- afterwards, trying to call LanguageClient.start() or LanguageClient.stop() will always just reraise the same error and the function passed as the ServerOptions will not be called again).
Tested with "vscode-languageclient": "^7.0.0" and "vscode-languageclient": "^8.0.0-next.7".
Note: not sure if it makes a difference, but in this scenario the LanguageClient was started successfully once, then the user made changes to make the error happen and then when fixing it to have the proper settings it won't work (because it has failed the previous time and calls to stop and start just reraise the previous error instead of calling the function passed as ServerOptions).

Can you please craft me a GitHub repository I can clone that demos this to make sure we are looking at the same problem. Thanks

Can you please craft me a GitHub repository I can clone that demos this to make sure we are looking at the same problem. Thanks

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!


Build on install

Coverage remained the same at 88.043% when pulling 9a5d50fab8bcf7d5762f6f5d5c5bdf8403887620 on newrelic-forks:build-on-install into da20c1f4b4784a849aa3183652a6f9d2987ec631 on airbnb:master.


feat: wait until the target pod (or resources) are up before marking ready

closes #36

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

29 of 57 (50.88%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+4.3%) to 35.736%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

controllers/base_request_controller.go
23
25
92.0%

controllers/builders/pod_access_builder.go
0
26
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
4.3%

Covered Lines:
575

Relevant Lines:
1609

💛  - Coveralls


Add optional override arg for behavior of generate_surrogate_key()

resolves #777
This is a:

[ ] documentation update
[ ] bug fix with no breaking changes
[x] new functionality
[ ] a breaking change

All pull requests from community contributors should target the main branch (default).
Description & motivation
See issue
Checklist

[ ] This code is associated with an Issue which has been triaged and accepted for development.
[ ] I have verified that these changes work locally on the following warehouses (Note: it's okay if you do not have access to all warehouses, this helps us understand what has been covered)

[ ] BigQuery
[ ] Postgres
[ ] Redshift
[ ] Snowflake

[ ] I followed guidelines to ensure that my changes will work on "non-core" adapters by:

[ ] dispatching any new macro(s) so non-core adapters can also use them (e.g. the star() source)
[ ] using the limit_zero() macro in place of the literal string: limit 0
[ ] using dbt.type_* macros instead of explicit datatypes (e.g. dbt.type_timestamp() instead of TIMESTAMP

[ ] I have updated the README.md (if applicable)
[ ] I have added tests & descriptions to my models (and macros if applicable)
[ ] I have added an entry to CHANGELOG.md

@joellabes upon your return from vacation, do you want to give me a 👍 / 👎 on the associated issue?
If you're onboard I'll bring the PR to completion (docs, changelog, etc)

Closed per the discussion in the linked issue.


[Bug] Fix Layouts

Think open new issues.
PR  #54 destroy the layout.
And the place need to be here:

Originally posted by @aviv1620 in https://github.com/hasadna/open-bus-map-search/issues/80#issuecomment-1751448102
We should fix the layouts of DashboardPage and GapsPatternsPage
DashboardPage:
Before:

After:

GapsPatternsPage:
Before:

After:

@NoamGaash , I close this issue, the pr was merged.

thank you!it looks so much better :clap:


3879 test acme issuer setup

This PR adds unit tests for ACME issuer's Setup function.
This follows #3877 , see the comment about testing here .
Some notes:

this test is not a particularly good unit test as such- I think that in general this function should be refactored which would allow us to test it better. However, maybe it'd be better if we merged (if we are going to at all) #3695 first. Also perhaps, already having some unit test in place will make refactoring easier

We usually unit test using test builder - which deals with mocking any requests to API server. It did not seem to make sense to use the test builder here as Setup does not have access to controller's context etc. This made it difficult to, for example, properly mock requests to create/list secrets. I think that once we refactor this method, this could be tested better.

This PR slightly refactors the Setup method and as a side-effect of that, it should fix the issue where sometimes the observed generation for ACME issuer is not updated correctly (I have observed this when hitting this code path after updating an ACME EAB field for an already valid issuer.

/kind cleanup

/release-note NONE

/release-note-none

/test pull-cert-manager-e2e-v1-20

/test pull-cert-manager-e2e-v1-20-feature-issuers-venafi-tpp

Thank you very much for the code review @wallrj !
I have tried to address all the comments, in some cases by creating new issues.
Thank you for the coverage, this is very useful. I have added one new test case and created #4006 for testing the functionality for updating ACME account email.
The other sections that aren't covered by tests are namespace checking (I think perhaps the namespace should be set on Acme in caller), removal of ACME URI from Issuer status (this seems to be an intermittent step, not sure if there is a value in testing it) and some functionality in the helper functions (I think there needs to be a refactor of Setup which may change how helpers behave, not sure if it's worth testing them in depth in this test).


externalresources(engine): implement GCCoordinator & GCRunner

What problem does this PR solve?

Issue Number: ref #5513
What is changed and how it works?

Implemented GCCoordinator.
Implemented GCRunner.

Check List
Tests

Unit test

Release note
None

/run-leak-tests


Iss6 - about us

Ready for code review.
My story demo can be viewed here:
http://www.screencast.com/t/hkdludIq0

It looks like you may need to upgrade your WebStorm or install an EditorConfig plugin if it doesn't have EditorConfig enabled already.  I noticed this because you're using a mix of 2 spaces, 4 spaces and tabs for indentation and some files lack newlines at the end.
You can fix these manually, but it's nice to allow your editor to fix this for you.  Let me know if you need help making sure EditorConfig is installed in your preferred editor.

I didn't have editorconfig formatting selected. Should all be formatted now.

I made a couple comments about some indentation that still differs.  Everything looks good to me besides those two issues.

There was some hidden indents from Sublime. I think we're good now.

Great! :+1: from me :ship:

In your integration test, it looks like you are testing links on pages other than aboutus.html. Your integration test at this point should concentrate on making sure that the outbound links (the links on aboutus.html to other pages) are pointed to functioning pages. That means you should start on aboutus.html, find a link, follow it, then navigate back to aboutus.html, find the next link, follow it, etc.

The test has been rewritten to navigate back to the about us page from each other page.


How to setup MTU for tunnel - guide in docs does not work?

How do I set the MTU? I followed the guide on [1] and I still get tunnel MTU at 1440 (larger than my wireguard MTU 1420) .
Expected Behavior
The tunnel MTU to be the one I set via ipipMTU - 1400 .
Current Behavior
When I display MTU for all interfaces on all nodes I get 1440 for tunnel.
salt-ssh  '*' -r 'ip addr' | grep mtu
        1: lo: <LOOPBACK,UP,LOWER_UP> mtu 65536 qdisc noqueue state UNKNOWN group default qlen 1
        2: eth0: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc pfifo_fast state UP group default qlen 1000
        3: wg0: <POINTOPOINT,NOARP,UP,LOWER_UP> mtu 1420 qdisc noqueue state UNKNOWN group default qlen 1
        4: docker0: <NO-CARRIER,BROADCAST,MULTICAST,UP> mtu 1500 qdisc noqueue state DOWN group default
        5: cali217c693643a@if3: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1400 qdisc noqueue state UP group default
        6: cali893367b5351@if3: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1400 qdisc noqueue state UP group default
        7: tunl0@NONE: <NOARP,UP,LOWER_UP> mtu 1440 qdisc noqueue state UNKNOWN group default qlen 1
        8: cali12d4a061371@if4: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1400 qdisc noqueue state UP group default
        1: lo: <LOOPBACK,UP,LOWER_UP> mtu 65536 qdisc noqueue state UNKNOWN group default qlen 1
        2: eth0: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc pfifo_fast state UP group default qlen 1000
        3: wg0: <POINTOPOINT,NOARP,UP,LOWER_UP> mtu 1420 qdisc noqueue state UNKNOWN group default qlen 1
        4: docker0: <NO-CARRIER,BROADCAST,MULTICAST,UP> mtu 1500 qdisc noqueue state DOWN group default
        1: lo: <LOOPBACK,UP,LOWER_UP> mtu 65536 qdisc noqueue state UNKNOWN group default qlen 1
        2: eth0: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc pfifo_fast state UP group default qlen 1000
        3: wg0: <POINTOPOINT,NOARP,UP,LOWER_UP> mtu 1420 qdisc noqueue state UNKNOWN group default qlen 1
        4: docker0: <NO-CARRIER,BROADCAST,MULTICAST,UP> mtu 1500 qdisc noqueue state DOWN group default
        5: tunl0@NONE: <NOARP,UP,LOWER_UP> mtu 1440 qdisc noqueue state UNKNOWN group default qlen 1
        6: cali3aa079d1c1a@if4: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1400 qdisc noqueue state UP group default
        7: cali91193ebe10d@if4: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1400 qdisc noqueue state UP group default
        8: calid14c9d9c505@if4: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1400 qdisc noqueue state UP group default
        9: calidcc9b422d75@if4: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1400 qdisc noqueue state UP group default
        10: cali92ed41d3a22@if4: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1400 qdisc noqueue state UP group default
        1: lo: <LOOPBACK,UP,LOWER_UP> mtu 65536 qdisc noqueue state UNKNOWN group default qlen 1
        2: eth0: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc pfifo_fast state UP group default qlen 1000
        3: wg0: <POINTOPOINT,NOARP,UP,LOWER_UP> mtu 1420 qdisc noqueue state UNKNOWN group default qlen 1
        4: docker0: <NO-CARRIER,BROADCAST,MULTICAST,UP> mtu 1500 qdisc noqueue state DOWN group default
        5: cali66c0e5aef8d@if3: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1400 qdisc noqueue state UP group default
        6: cali71d64841ae9@if3: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1400 qdisc noqueue state UP group default
        7: tunl0@NONE: <NOARP,UP,LOWER_UP> mtu 1440 qdisc noqueue state UNKNOWN group default qlen 1
        8: cali9cafa0a893e@if4: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1400 qdisc noqueue state UP group default
        9: caliad17b2e6582@if4: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1400 qdisc noqueue state UP group default
        1: lo: <LOOPBACK,UP,LOWER_UP> mtu 65536 qdisc noqueue state UNKNOWN group default qlen 1
        2: eth0: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc pfifo_fast state UP group default qlen 1000
        3: wg0: <POINTOPOINT,NOARP,UP,LOWER_UP> mtu 1420 qdisc noqueue state UNKNOWN group default qlen 1
        4: docker0: <NO-CARRIER,BROADCAST,MULTICAST,UP> mtu 1500 qdisc noqueue state DOWN group default
        5: calicbf4a3084d5@if3: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1400 qdisc noqueue state UP group default
        6: calicaf8383f0e4@if3: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1400 qdisc noqueue state UP group default
        7: tunl0@NONE: <NOARP,UP,LOWER_UP> mtu 1440 qdisc noqueue state UNKNOWN group default qlen 1

Possible Solution
??
Steps to Reproduce (for bugs)
Steps I took:

updated calico config mtu to 1400 with kubectl -n kube-system edit configmap calico-config
added felix ipipMTU with 1400 via calicoctl replace -f felix.yaml
restared all cluster nodes at once with salt-ssh

I'm trying to get tunnel to match wg0 MTU size of 1420 - my example shows with 1400, it's not a typo, this is what I tried in my second approach.
[1] https://docs.projectcalico.org/v3.1/usage/configuration/mtu
Context
I'm trying to fix issue described here https://github.com/projectcalico/calico/issues/1709#issuecomment-405060902 . In summary: responses larger than 1868 bytes time out in my ingress-nginx running with host networking.
I confirm it's caused by MTU - I have tcpdump captures and I've reproduced the issue + fixed it by lowering the MTU to 1400.
I wish to align the MTU's according to the guide, just to be ok.
Your Environment

Calico version
calicoctl version
Client Version:    v3.1.3
Build date:        2018-05-30T17:15:59+0000
Git commit:        231083c2
Cluster Version:   v3.1.3
Cluster Type:      k8s,bgp,kdd

Orchestrator version (e.g. kubernetes, mesos, rkt):
kubeadm 1+3 node cluster of k8s 1.11

Operating System and version:
NAME="Debian GNU/Linux"
VERSION_ID="9"
VERSION="9 (stretch)"

Linux m01 4.9.0-6-amd64 #1 SMP Debian 4.9.88-1+deb9u1 (2018-05-07) x86_64 GNU/Linux

Could you take a look at the start of the felix logs (the calico-node container)? I believe it logs the MTU values it reads in for configuration, maybe that will shed some light on why it isn't behaving as expected.


iPad Scribble flicker and crash

Previously, dragging to select with an Apple Pencil on an iPad (Scribble) caused the context menu to rapidly hide and show. Sometimes this even caused an assertion error when using SystemContextMenu due to showing two context menus in one frame. After this PR, the flicker and crash are gone.
The flicker happened on both the Flutter-rendered context menu and SystemContextMenu, but the error only happened with SystemContextMenu due to a safeguard that prevents two from showing at the same time.
The flickering is likely a regression caused by https://github.com/flutter/flutter/pull/142463.

Before this PR
After this PR

Flutter's behavior isn't perfect compared to native (below), but it's a major improvement. If we want to match native, I think we might have to mess with the engine and see why it's calling showToolbar so much. I checked and scribbleInProgress is false during this selection gesture, so we can't use that.

Scribble native video
https://github.com/user-attachments/assets/207e208a-ac36-4c9e-a8ed-9e90e6ef9e3a

Fixes https://github.com/flutter/flutter/issues/159259

FYI @fbcouch


NW6 | Fathi kahin | wireframe | Week1

Learners, PR Template
Self checklist

[ ] I have committed my files one by one, on purpose, and for a reason
[ ] I have titled my PR with COHORT_NAME | FIRST_NAME LAST_NAME | REPO_NAME | WEEK
[ ] I have tested my changes
[ ] My changes follow the style guide
[ ] My changes meet the requirements of this task

Changelist
Briefly explain your PR.
Questions
Ask any questions you have for your reviewer.

@fhkahin please complete the PR template!


[ADD BACKEND] : Want to add Auth system in your backend

Frontend Summary 📝
I want to add authentication system in your project backend.
Development Approach 🚀
I am trying to add 2 FA  in your project backend.
Deployment Status
None
Select the open source program you are contributing to
Other
Code of Conduct

[X] I adhere to the CONTRIBUTING GUIDELINES of this project.

Hey @Pujan-sarkar👋! Thanks for raising this issue and demonstrating your interest in contributing to our project!🚀
We value your engagement and eagerly anticipate your contributions. Should you wish to get involved, please use the /assignme command to self-assign the issue.
This issue provides an excellent opportunity for us to enhance our project, and we're excited to have you join us! Please take a moment to review our CONTRIBUTING GUIDELINES before getting started.
If you have any questions or need assistance, feel free to reach out to us on LinkedIn.❤

/assign

/assignme

Hello @Pujan-sarkar! We'd like to inform you that the issue has been resolved and is now closed. We appreciate your understanding and look forward to your continued involvement with our repository!🤗
If you encounter any further issues, please don't hesitate to reach out to us on LinkedIn. Thank you for your support!❣
Happy coding!✨ We hope to see you again soon!❣


Place epm rpm and deb in the fixed path /usr/local/bin/epm

In epm/dist/rpm/epm.spec and epm/dist/deb/debian/rules, epm will be installed to /usr/bin/epm rather than /usr/local/bin/epm , so you also need to revise epm/dist/deb and epm/dist/rpm files.


"cannot cast type geometry to geography" when aggregating by provinces/countries

STR:

Create a map from a points table (I used some points in Spain)
In STYLE, select the "administrative regions" aggregation
Choose admin. level "provinces" or "countries", operation "count"

The map doesn't get updated and the Maps API returns the following error:
errors_with_context:
[{type: "unknown", message: "cannot cast type geometry to geography", name: "error", length: 93,…}]
0:{type: "unknown", message: "cannot cast type geometry to geography", name: "error", length: 93,…}
code:"P0001"
file:"pl_exec.c"
length:93
line:"3216"
message:"cannot cast type geometry to geography"
name:"error"
routine:"exec_stmt_raise"
severity:"ERROR"
type:"unknown"

The data I'm using is available here -- a tweets dataset.

Sorry for the vague description. I'm trying to reproduce the error further. I was able to make it work in another account with another table. I'll keep updating the issue.

Running a
SELECT distinct(st_geometrytype(the_geom)) FROM twitter_catalansvote9n25 limit 10

is returning ST_Point and null.

Hey @javisantana any clue?

no idea, this is the query: https://github.com/CartoDB/cartodb/blob/master/lib/assets/javascripts/cartodb3/editor/style/style-converter.js#L436


How to make a custom module that is a subclass of RTCScrollView?

I need to make a custom React Native ScrollView component, that will show next & prev page previews (like here), but otherwise behaves like a default React Native ScrollView - accepts same props, styling, etc.
In the docs I only found on how to subclass RTCViewManager, but is it possible in the same manner to subclass from RTCScrollViewManager? Will I have all the default ScrollView props in this case?
In the docs there is also written that:

Note: Do not attempt to set the frame or backgroundColor properties on the UIView instance that you vend through the -view method. React Native will overwrite the values set by your custom class in order to match your JavaScript component's layout props. If you need this granularity of control it might be better to wrap the UIView instance you want to style in another UIView and return the wrapper UIView instead. See Issue 2948 for more context.

But seeing examples (ex 1 and ex 2) on how to accomplish my task - show next & prev pages peek preview I should use setClipsToBound and maybe also make scrollView a subview of a View.
I've googled a lot, but can't get my head around.. where to start, are there any posts about how to write custom UI components that are a subclass of RTCScrollViewManager? Do you have any thoughts? Any help would be much appreciated.

I need to use a native paging for the best/desired swipe 'feeling', because of that I can't accomplish this without touching obj-c. Solution with snapToInterval just don't give the desired swipe 'feeling', because even with decelerationRate={'fast'} I can easily scroll 2-4 slides at once...swipe is too sensitive: example

We're cutting down on the number of outstanding issues, in order to allow us to focus. I'm closing this issue because it has been open for over 60 days with no activity. If you think it should still be opened let us know why. PRs are always welcome.


Host link in APIRule in UI broken

When you add an APIRule that has only a short "host" defined, e.g. host: httpbin the host in the APIRule UI is broken.
The host is incorrect in the list and details view.
image
image
Expected result
The host link in the APIRule list and details is valid and doesn't contain null.
Actual result
The APIRule host link is broken.
Steps to reproduce
Apply APIRule without full host name
kubectl apply -f - <<EOF
apiVersion: gateway.kyma-project.io/v1beta1
kind: APIRule
metadata:
name: httpbin-rule
namespace: default
labels:
app: httpbin
spec:
gateway: kyma-system/kyma-gateway
host: httpbin
service:
name: httpbin
port: 8000
rules:
- path: /anything
methods: [ "GET", "POST"]
mutators: [ ]
accessStrategies:
- handler: no_auth
EOF
Check APIRules list and details of the APIRule
This issue was reported here: https://github.com/kyma-project/api-gateway/issues/870 and still persists

Hey @gopalanand333 I managed to reproduce your issue, it happens when APIRule has gateway configured in ns/name format. We will be providing a fix for that.

With today's upgrade of api-gateway in regular channel the issue should be resolved. @gopalanand333 please check if it looks OK now


Update bnb.html

Changed ACC to ACCMOD for weapons
Changes / Comments
Changed ACC to ACCMOD for weapons
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[ ] Does the pull request title have the sheet name(s)? Include each sheet name.
[X] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.


koord-scheduler: optimize ElasticQuotaPluginUT

Signed-off-by: xulinfei.xlf<EMAIL_ADDRESS>Ⅰ. Describe what this PR does

Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist

[ ] I have written necessary docs and comments
[ ] I have added necessary unit tests and integration tests
[ ] All checks passed in make test

/lgtm


Fetch with request body

Is this a bug report
May be

Have you read the Contributing Guidelines?
Yes
Environment

OS:  Windows 10
Node: v8.1.3
Yarn:  1.2.0
npm:  5.0.3
Watchman: Not Found
Xcode:  Not Found
Android Studio:  2.3.3

Packages: (wanted => installed)
    react-native: 0.49.3 => 0.49.3
    react: 16.0.0-alpha.12 => 16.0.0-alpha.12

Target Platform: Android 7

Steps to Reproduce

I don't know it's a bug or my mistake . How can I send a http request with binary body?
As both XMLHttpRequest and fetch is using NetworkingModule::sendRequest , so the following codes
fetch('https://mywebsite.com/endpoint/', {
  method: 'POST',
  body: new Uint8Array([1,2,3,4,5,6,7,8,9])
})

var request = new XMLHttpRequest();
request.onreadystatechange = (e) => {
  if (request.readyState !== 4) {
    return;
  }

  if (request.status === 200) {
    console.log('success', request.responseText);
  } else {
    console.warn('error');
  }
};

request.open('GET', 'https://mywebsite.com/endpoint/');
request.send(new Uint8Array([1,2,3,4,5,6,7,8,9]));

is correct according to
https://github.com/facebook/react-native/blob/master/Libraries/Network/XMLHttpRequest.js#L508
Type of data is RequestBody and can be
type RequestBody =
    string
  | FormData
  | {uri: string}
  | ArrayBuffer
  | $ArrayBufferView
;

But not work's. Fetch don't send request
Here https://docs.postman-echo.com we have an  echo test server
Expected Behavior

Send http request with binary body
Actual Behavior

No http request is send
Reproducible Demo

(Paste the link to an example project and exact instructions to reproduce the issue.)

Unfortunately, there is no out-of-box way to send binary data in React Native. You may track PR progress here: https://github.com/facebook/react-native/pull/11573

@facebook-github-bot answered

@Kureev So why RequestBody accept ArrayBufferView?

@Kureev I don't want send Blob just I want put an array buffer in http request body

I believe that here is android implementation
https://github.com/facebook/react-native/blob/master/ReactAndroid/src/main/java/com/facebook/react/modules/network/NetworkingModule.java#L274


[STORM-2676] Error class name for log in JsonRecordHiveMapper.java.

https://issues.apache.org/jira/browse/STORM-2676
The class in "LoggerFactory.getLogger()" should be JsonRecordHiveMapper.class.

@HeartSaVioR
Could you have a look if you have some time?

+1

@HeartSaVioR
Thanks for your support.Could you help me merge it if you have some time?

+1

@liu-zhaokun we have a policy that we need to wait 24 hours before merging a change, even a trivial one like this to give all devs time to review and comment.

@revans2
OK,thanks for your reply.


Support a data distribution strategy

Currently the data is replicated across the replicas if the number of replicas provided is >1. To support sharding the following can be implemented:

Introduce a filterPattern in the options of the CR Spec which includes a glob pattern along with the number of replicas which should have the data corresponding to that pattern. The pattern can be of the form:

  - {'1*.png': 2, '2*.png': 1, '3*.png': 1}

Which would mean all the files corresponding to 1*.png would be placed on 2 nodes and depending on the strategy they will either be replicated or distributed.

Introduce an option to provide the replication strategy. The possible strategies are:
-- Replicate (Implemented first)
-- Distribute (Implemented later)

implemented by #58


Add atomic arch instructions

This PR adds nearly all of the SPIR-V atomic instructions, the only ones not implemented are the ones around atomic flags as we don't have a type for them at the moment.
Depends on #519

I thought we had discussed this already in https://github.com/EmbarkStudios/rust-gpu/issues/442 and the plan was to expose these through the rust stdlib, instead of custom instructions?

To be clear, if I recall correctly, the entire purpose of the arch module is intended to let us prototype high level abstractions over SPIR-V concepts without getting into compiler work. There is already a high level abstraction in place for atomics (sync::atomic), no prototyping is necessary, and so I don't want to waste work to add instructions to the arch module for sake of adding instructions to the arch module. I think our effort would be better directed towards things like fixing #442.

I thought we had discussed this already in #442 and the plan was to expose these through the rust stdlib, instead of custom instructions?

Well there's no discussion in that thread, so if we did discuss it , it wasn't there and I don't remember it beyond "yes we should have them". I don't have a strong opinion on how we do it, but as I've read the spec more I'm not sure stdlib is complete abstraction for SPIR-V's atomics. For example there are way more memory semantics in SPIR-V than just Relaxed, Release, Acquire, AcqRel, and SeqCst. It also doesn't have any concept of Scope, and there's no AtomicBool equivalent in SPIR-V. So to me it seems like we'd be better providing our own abstraction.

(Re-opened temporarily to force push a rebase - the presence of the atomics branch, with a massive merge anomaly, was getting in the way of using jj on this repo, so I felt I had to untangle it)

I can delete the branch if that helps.

@XAMPPRocky Only if you think it's been obsoleted by e.g.:

#877

(even if you delete the branch now, I still thing my cleanup of it was worth it, because the PR doesn't look extremely broken - idk how much that persists after deleting the branch, but better safe than sorry)


WI using tokenRequests

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

2 of 61 (3.28%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-5.4%) to 56.173%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/types/types.go
0
2
0.0%

internal/service/secret_client_factory.go
0
3
0.0%

internal/server/server.go
2
56
3.57%

Totals

Change from base Build<PHONE_NUMBER>:
-5.4%

Covered Lines:
364

Relevant Lines:
648

💛  - Coveralls


DDP fails on multinode if each node has a different number of GPUS

Bug description
Hi there,
I'm launching a PyTorch-Lightning script in a multinode environment.
In order to do so, I have followed the suggestions at this link:
https://pytorch-lightning.readthedocs.io/en/stable/clouds/cluster_intermediate_1.html
I have configured a bash script that gets the number of nodes assigned and the number of GPUS on each node, as well as the MASTER NODE, MASTER ADDRESS, etc...
In the bash script I launch the mpirun command on every node, like:
mpirun -np 1 -H $NODE $EXEC $SCRIPT &
where $NODE is the current node, $EXEC is the python3 command, and $SCRIPT is my python script.
In the python script (which is launched N nodes times), I correctly assign the variables:
os.environ["MASTER_ADDR"] = args.master
os.environ["MASTER_PORT"] = args.port
os.environ["WORLD_SIZE"] = args.world
os.environ["NODE_RANK"] = args.rank

Then, in the trainer:
trainer = pl.Trainer(strategy=DDPStrategy(find_unused_parameters=False), accelerator='gpu', devices=gpus, num_nodes=world_size)

Everything works fine if the Scheduler assigns the same number of GPUS at each Node.
(2 GPUS on 4 NODES in the following example, with 8 gpus in total)
# Note: some prints are from my bash script
assigned nodes: gnode10 gnode16 gnode31 gnode32
assigned gpus per node: 2 2 2 2
WORLD_SIZE 4
MASTER gnode10
PORT 11551
bash script here! Waiting for all jobs to finish...
#############################################################################
# Here start the 4 scripts launched by mpirun (in this case WORLD SIZE = 4)
GPU available: True, used: True
TPU available: False, using: 0 TPU cores
IPU available: False, using: 0 IPUs
HPU available: False, using: 0 HPUs
Initializing distributed: GLOBAL_RANK: 0, MEMBER: 1/8
Initializing distributed: GLOBAL_RANK: 4, MEMBER: 5/8
Initializing distributed: GLOBAL_RANK: 6, MEMBER: 7/8
Initializing distributed: GLOBAL_RANK: 2, MEMBER: 3/8
Initializing distributed: GLOBAL_RANK: 1, MEMBER: 2/8
Initializing distributed: GLOBAL_RANK: 5, MEMBER: 6/8
Initializing distributed: GLOBAL_RANK: 7, MEMBER: 8/8
Initializing distributed: GLOBAL_RANK: 3, MEMBER: 4/8
----------------------------------------------------------------------------------------------------
distributed_backend=nccl
All distributed processes registered. Starting with 8 processes
----------------------------------------------------------------------------------------------------

LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [GPU-e2d7ffc6-fe40-bbff-cd8b-77571d64b1bc,GPU-ec7f5880-fe70-668a-2fe9-6b51118ff2ab]
LOCAL_RANK: 1 - CUDA_VISIBLE_DEVICES: [GPU-e2d7ffc6-fe40-bbff-cd8b-77571d64b1bc,GPU-ec7f5880-fe70-668a-2fe9-6b51118ff2ab]
LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [GPU-80c32807-6dbf-8b68-4740-086cece4b91a,GPU-0148835f-d43f-b272-72d2-38a6dea989d2]
LOCAL_RANK: 1 - CUDA_VISIBLE_DEVICES: [GPU-80c32807-6dbf-8b68-4740-086cece4b91a,GPU-0148835f-d43f-b272-72d2-38a6dea989d2]
LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [GPU-7c411134-8399-b08e-2f07-d45fdbff38c5,GPU-ce3d447b-109b-8294-c7a3-d4b1b36ede2e]
LOCAL_RANK: 1 - CUDA_VISIBLE_DEVICES: [GPU-d7ff4b7a-730d-7ada-6fec-40ba709c649a,GPU-2774936a-d980-022c-69a5-ea4a6e081e6f]
LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [GPU-d7ff4b7a-730d-7ada-6fec-40ba709c649a,GPU-2774936a-d980-022c-69a5-ea4a6e081e6f]
LOCAL_RANK: 1 - CUDA_VISIBLE_DEVICES: [GPU-7c411134-8399-b08e-2f07-d45fdbff38c5,GPU-ce3d447b-109b-8294-c7a3-d4b1b36ede2e]

# Then Training starts correctly.

The problem is when the Scheduler assigns a different number of GPUS on each node, which may of course happen very often since I am not the only person using the cluster.
In the following example, 4 GPUS are assigned to 3 NODES (1, 2, 1).
# Note: some prints are from my bash script
assigned nodes: gnode41 gnode54 gnode60
assigned gpus per node: 1 2 1
WORLD_SIZE 3
MASTER gnode41
PORT 25212
bash script here! Waiting for all jobs to finish...
##############################################################################
GPU available: True, used: True
TPU available: False, using: 0 TPU cores
IPU available: False, using: 0 IPUs
HPU available: False, using: 0 HPUs
Initializing distributed: GLOBAL_RANK: 2, MEMBER: 3/3
Initializing distributed: GLOBAL_RANK: 0, MEMBER: 1/3
Initializing distributed: GLOBAL_RANK: 2, MEMBER: 3/6
Initializing distributed: GLOBAL_RANK: 3, MEMBER: 4/6
----------------------------------------------------------------------------------------------------
distributed_backend=nccl
All distributed processes registered. Starting with 3 processes
----------------------------------------------------------------------------------------------------
# just hanging forever...

As you can see, the GLOBAL_RANK and MEMBER assigned are wrong.
In particular, for MEMBER the node with 2 gpus assumes a total number of processes of 6 (2 times WORLD SIZE), while the nodes with one GPU assigned assume a total number of processes of 3 (1 times WORLD_SIZE).
Additional Note: My cluster uses PBS jobs scheduler, so I do not know if the same happens with e.g. SLURM.
Thank you
How to reproduce the bug
No response
Error messages and logs
There are no error messages, since the process hangs forever.
Environment

#- PyTorch Lightning Version (e.g., 1.5.0): 1.7.7
#- PyTorch Version (e.g., 1.10): 1.11.0
#- Python version (e.g., 3.9):  3.9
#- OS (e.g., Linux): Linux
#- CUDA/cuDNN version: 11.7
#- GPU models and configuration: nvidia A100
#- How you installed Lightning(`conda`, `pip`, source): pip

More info
No response

I believe if you define the lowes number of devices from all your nodes, it shall work...
cc: @awaelchli

@SerezD
Here:
trainer = pl.Trainer(strategy=DDPStrategy(find_unused_parameters=False), accelerator='gpu', devices=gpus, num_nodes=world_size)

This seems wrong to me. num_nodes should no be set to world size, but the explicit number of nodes involved, and devices should be set to the number of GPUs per node.
It is currently not possible in Lightning to run in a distributed setup with different number of GPUs per node. However, a proposal to support that in the future is here: https://github.com/Lightning-AI/lightning/issues/14078

Hi,
Thanks @awaelchli, I will then wait for future support.
Anyway, your comment is not very clear to me.
In the following command:
trainer = pl.Trainer(strategy=DDPStrategy(find_unused_parameters=False), accelerator='gpu', devices=gpus, num_nodes=world_size)
I set the variable gpus as the number of selected gpus on that node. For instance, in the first example provided above gpus=2 on each node, while in the second example it is 1 or 2 depending on the node (which now I know won't work).
What I do not understand is what I should set in num_nodes according to your answer.
In the first example I have 4 nodes involved, and world_size=4, while in the second case I have 3 nodes involved, and world_size=3. What should the values of num_nodes be in these scenarios?
Thank you

The world size is defined as the total number of processes: (number of nodes) * (number of gpus per node), or in general, the sum of all gpus across all machines.
Example 1:
Node 0 has 2 GPUs
Node 1 has 3 GPUs
Then:
num_nodes = 2
world_size = 2+3 = 5
Example 2:
Node 0 has 4 GPUs
Node 1 has 4 GPUs
Trainer(devices=4, num_nodes=2)  # implies that world size = 4 + 4 = 8

@awaelchli

The world size is defined as the total number of processes: (number of nodes) * (number of gpus per node), or in general, the sum of all gpus across all machines.

Just a note about this.
In the guide: https://pytorch-lightning.readthedocs.io/en/stable/clouds/cluster_intermediate_1.html
It is written:

WORLD_SIZE - required; how many nodes are in the cluster

This is the reason why I assigned num_nodes=world_size, since I thought that the two were the same thing. I, therefore, think that removing the num_nodes argument will be very helpful in terms of clarity.

Thanks for pointing that out. The docs there are not correct, we need to update the description there :)


accept integer for float field

Now, insert an integer to a float field will throw an "already exists as type float" error
because there isn't precision float in Javascript, e.x:  1.0, 2.0... 10.0 all will be converted to integer automatically.
And as described in the 3032, now all numbers in influxdb are float, so could this be fixed?
Or could add an auto parse func from String?

How about using record_transformer filter to convert number type?
<filter test.**>
  @type record_transformer
  enable_ruby true
  <record>
    foo ${record["foo"].to_f}
  </record>
</filter>

Sure, that's what I did.  but It's better to run without other tricky operations. at least fitting the influxdb‘s protocol. so why not transform all number to float？


[gnome] gnome.mutter338: Fix build with separate sysprof

Description of changes
ninja: error: '/nix/store/m5zqagkrl1m3mpms1nvl2p9z7ws566si-libsysprof-capture-3.46.0/share/dbus-1/interfaces/org.gnome.Sysprof3.Profiler.xml', needed by 'src/meta-dbus-sysprof3-profiler.h', missing and no known rule to make it

Use squash merge or rebase merge.
P.s. I was aware of the large number of broken Pantheon packages on the branch (https://github.com/NixOS/nixpkgs/projects/21#column-19099710).
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] 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
[ ] 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
[ ] (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.

@ofborg build gnome.mutter338

Thank you.


Unsuccessful retrieval of node.js messages

Hi, thank you  for this library.
I have an index.js callback being rendered as follows
var io = require('socket.io').listen(app.listen(port));

io.sockets.on("connection", function (socket) {
    socket.on("send", function (data) {
        io.sockets.emit("message", data);
    });
});

where the front-end is
    socket.on('message', function (data) {
        if(data.message) {
            messages.push(data);         var html = '';
            for(var i=0; i<messages.length; i++) {
                html += '<b>' + (messages[i].username ? messages[i].username : 'Zeus') + ': </b>';
                html += messages[i].message + '<br />';
            }
            goal.innerHTML = html;
        }
    });

These work okay. I am trying to use your library to retrieve the js sockets message in c++ by following the console example/chat demo but so far, I have not been successful.
My  bind_events look something like this:
socket::ptr current_socket;           //this is a global variable
void bind_events(socket::ptr &socket)
{
    current_socket->on("send", sio::socket::event_listener_aux([&](string const& name, message::ptr const& data, bool isAck,message::list &ack_resp)
                        {
                            std::cout << "in bind send "   << std::endl;
                            _lock.lock();
                            string user = data->get_map()["username"]->get_string();
                            string message = data->get_map()["message"]->get_string();
                            std::cout << user << ":" << message << std::endl;
                            _lock.unlock();
                        }));
}

while in my main, I am listening for new messages like so:
    sio::client h;
    connection_listener l(h);
    h.set_open_listener(std::bind(&connection_listener::on_connected, &l));
    h.set_close_listener(std::bind(&connection_listener::on_close, &l,std::placeholders::_1));
    h.set_fail_listener(std::bind(&connection_listener::on_fail, &l));
    h.connect("http://<IP_ADDRESS>:8000");
    _lock.lock();
    if(!connect_finish)                //_lock is an std::mutex while connect_finish is a bool
    {
        _cond.wait(_lock);                //cond is an std::condition variable
    }
    _lock.unlock();
    current_socket = h.socket();
    bind_events(current_socket);
    std::cout << "Closing..." << std::endl;
    h.sync_close();
    h.clear_con_listeners();

When I run the code, my output looks like
[2016-08-05 11:02:36] [connect] Successful connection
[2016-08-05 11:02:36] [connect] WebSocket Connection <IP_ADDRESS>:8000 v-2 "WebSocket++/0.7.0" /socket.io/?EIO=4&transport=websocket&t=1470409356 101
socket io connected
leaving bind
Closing...
[2016-08-05 11:02:36] [warning] got non-close frame while closing
[2016-08-05 11:02:36] [warning] got non-close frame while closing
[2016-08-05 11:02:36] [warning] got non-close frame while closing
socket io closed

The bind events current_socket->on callback listeners are skipped entirely. Am I wrong in the way I am setting up the callback?
Thank you for your anticipated response!

did u ever figure this out ?

Can't remember what I did but I somehow got it to work, I think.

I got the same thing, no idea what todo...


atomics

Consider using atomics for simple locks (such as the source port lock)

[x] sport lock
[x] connection table lock

I completely removed the source port lock by pre-assigning source ports to each connection. I think it's pretty clever, because it actually took a O(n * m) search down to O(1).
For this to work, we assume that number of connections is lower than the number of port numbers available for outgoing (typically 48). There is a compile time check for this.

I added C11 atomics for the connection table lock using the:
atomic_compare_exchange
Function. The compiler typically knows what to do. But in the case of Cortex-M0 it's not supported, and it makes an external function call to a user implementation. I therefore added an example in 'contrib/atomics/atomics_freertos_gcc.c' to show how the user may implement such thing on Cortex-M0 and FreeRTOS. It's typically just deactivating interrrupts while performing the compare and exchange.
On more advanced ARM systems such as cortex M3, M4, M7, A9... and up, as well as X86, and ARM64, the compiler has assembler intrinsics that performs this using dedicated instructions.
The removal of the sport and conn locks has reduced a M0 image size by 300 bytes .text and 100 bytes .bss.


Terms of service

Add configurable ToS

template is customizable
filter enforces acceptance
per - realm configuration flags

closed with #397


Still Maintaining ??

I can see this is very old lib and has not been updated for a long time. so is this useful to use with react-native > 0.64 .
Can we consider this lib is dead!

Hi @vijaychouhan-rails, have you found any solution for this?

@shubham-s3 please consider alternate package like: react-native-thermal-printer , react-native-thermal-receipt-printer its working fine with RN-0.73.2


All plots: Better loading indicator

The current loading indicator that is used is small and inconspicuous. We probably want to use something that overlays the plot area.

Best done after #104

Here's what MUI has
https://material-ui.com/components/progress/
But we should be aware of what React's <Suspense> does (for lazy-loaded components only, at the moment, AFAIK)

Examined and tested this a bit. Perhaps not a complete solution but it works in some degree. Hope this would be helpful for one who would work for this issue. Please note that below was only tested for viz and there may be a side effect by changing Promise.tsx (see below), e.g., Subsetting, though I didn't notice much difference at the Subsetting.
At src/lib/core/components/Promise.tsx (loading icon around top middle area):
change from
{state.pending && <Loading />}
to
{state.pending}
At a viz component, e.g., src/lib/core/components/visualizations/implementations/ScatterplotVisualization.tsx
(another small loading icon around top left area):
First, import circular progress icon (assumed to use it among others)
import CircularProgress from '@material-ui/core/CircularProgress';
then, change from
     {data.pending && (
        <Loading style={{ position: 'absolute', top: '-1.5em' }} radius={2} />
      )}

to
      {data.pending && fullscreen && (
        <CircularProgress color={'secondary'} size={50} thickness={5} style={{ position: 'absolute', marginLeft: '420px', marginTop: '350px', zIndex: 10 }} />
      )}

This will show a red circling progress icon. Certainly size/thickness and pixel (or percent [%]) values can be adjusted per viz.

There still exists unnoticeable small text of Loading... at the top left of plot area, but it seems that it comes from node_modules library (e.g., wdk-client, etc.). Also, I didn't think of thumbnail/grid view much as it shows Loading... message in the center.
As an example, a screenshot for the case of scatterplot viz is attached in the following:

Thanks for doing this research, @moontrip. I was thinking we might want to do something in https://github.com/VEuPathDB/web-components so that we can have a consistent loading indicator anywhere we use plots.
The PromiseResult component is only used in one place, currently, so I'm okay with making changes. I think the entire line can be removed:
-       {state.pending && <Loading />}

@dmfalke thank you for sharing your thoughts. I think that you want to use React <Suspense>. One of my curiosity showing small text of "Loading..." at top left of plot area might be coming from PlotlyPlot component's <Suspense> .

<Suspense> only works for loading components, presently. I think we should use a similar loading indicator there, that we show above. We can use the provided height and width props to determine the placement of the loading indicator.

Linked the appropriate PR in web-components 👍


SendKeys hangs and can't find input field that is clearly there

What versions are you running?

$ go list -m github.com/chromedp/chromedp
github.com/chromedp/chromedp v0.5.2
$ google-chrome --version
Google Chrome 78.0.3904.108
$ go version
go version go1.12.5 darwin/amd64

What did you do? Include clear steps.
The main search page on duckduckgo.com features a text input field named "q":
<form id="search_form_homepage_top" class="search  search--home  js-search-form-top" name="x" method="POST" action="/html">
			<input class="search__input  js-search-input" type="text" autocomplete="off" name="q" tabindex="1" value="">
			<input class="search__button  js-search-button" type="submit" tabindex="2" value="S" />
			<input class="search__clear  empty  js-search-clear" type="button" tabindex="3" value="X" />

		</form>

so let's try to use SendKeys() to paste text into it:
package main

import (
  "context"
  "github.com/chromedp/chromedp"
  "log"
)

func main() {
  ctx, cancel := chromedp.NewContext(
      context.Background(), chromedp.WithDebugf(log.Printf))
  defer cancel()

  sel := `//input[@name="q"]`
  tasks := chromedp.Tasks{
    chromedp.Navigate(
      "https://duckduckgo.com"),
    chromedp.WaitReady("body", chromedp.ByQuery),
    chromedp.SendKeys(sel, "waaah\n", chromedp.ByQuery),
    chromedp.WaitReady("body", chromedp.ByQuery),
  }

  err := chromedp.Run(ctx, tasks)
  if err != nil {
    panic(err)
  }
}

What did you expect to see?
The selector //input[@name="q"] should find the item.
What did you see instead?
Debug output indicates it's searching forever without finding it:
2020/01/01 12:58:48 -> {"id":143,"sessionId":"CF5A3F56D987CA6FEA082B76D810F060","method":"DOM.querySelector","params":{"nodeId":14,"selector":"//input[@name=\"q\"]"}}
2020/01/01 12:58:48 <- {"error":{"code":-32000,"message":"DOM Error while querying"},"id":143,"sessionId":"CF5A3F56D987CA6FEA082B76D810F060"}

This seems to be related to https://github.com/chromedp/chromedp/issues/540, hopefully this easily reproducable example helps to find the root cause.

DOM Error while querying should be giving you a clue here. Your syntax seems wrong for ByQuery, which only accepts CSS selectors.
input[name="q"] works for me. In your browser's console, you can see how document.querySelector('input[name="q"]') works, but it doesn't when it's fed your query selector.
Alternatively, if you really want to use the XPath syntax, you need to use BySearch, which is the default. There, I can see via the debug logs that //input[@name="q"] returns two nodes; I'm not sure why that's the case. The program gets stuck there. I haven't investigated further, since ByQuery works fine, but you can debug where the problem is if you want.
This is all documented in https://godoc.org/github.com/chromedp/chromedp#hdr-By_Options.
Admittedly, we could do a better job at not looping forever in the clear cases where a query has invalid syntax. It wouldn't make your original code work, but at least it would give you a somewhat useful error.

Thanks for the response! Looks like we have a two cases now:

input[name="q"] + ByQuery: works
//input[@name="q"] + BySearch: does not work (you're saying it finds two nodes, but it shouldn't, and I can't really make sense of the debug logs)

From the docs:

BySearch is an element query option to select elements by the DOM.performSearch command. Works with both CSS and XPath queries.

It really just sends your selector to https://godoc.org/github.com/chromedp/cdproto/dom#PerformSearch. So if it's doing something weird, you could poke at that. I'd also imagine that other Chrome devtools software like Puppeteer would get the same result.
You can also use https://godoc.org/github.com/chromedp/chromedp#Nodes to actually inspect the results the selector gets. SendKeys picks the first result, so my guess is that the node where one could actually send keys is the second result instead of the first.

Hi,
I have the same issue in one of my Selenium scripts. But the error is not happening all the time. Some runs are OK and then the next is failing with the error "{"code":-32000,"message":"DOM Error while querying"}"
I have to use Selenium IDE, because I need the SIDE file to integrate with another monitoring tool. So I cannot use Java or JavaScript :|
Attached a screenshot from my script.
Command 16 opens a new window in which I then try to enter the text and the ENTER key in the search field identified by the full xpath. It finds the search field in the "wait for element editable", but in the next step when I try to use sendKeys it fails.

I'm quite new to Selenium and try to figure out how to debug this. But I'm a little bit out of ideas what to do with this.
Thanks!
Andrea

@anschuh11 It seems that you came to the wrong place. Maybe you wanted to go to https://github.com/SeleniumHQ/selenium-ide.
The OP's issue should have been addressed. Closing.


Incomplete Data Recieved

Checklist

[X] I'm reporting a bug unrelated to a specific site
[X] I've verified that I'm running yt-dlp version 2<IP_ADDRESS>. (update instructions)
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Description
--get-comments has a incomplete data received error for 1 specific video. I know its a duplicate of #507 and it says its solved but that was a few months ago.
Verbose log
[debug] Command-line config: ['--write-description', '--write-info-json', '--write-annotations', '--write-sub', '--sub-langs', 'en', '--write-thumbnail', '--write-comments', '--no-playlist', '--verbose', '--download-archive', 'C:/users/seong/archive/public/videos/downloadedarchive.txt', '-f', 'mp4', '-o', 'C:/users/seong/archive/public/videos/%(upload_date)s-%(title)s-%(id)s.%(ext)s', 'https://www.youtube.com/c/PomplamooseMusic']
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, err utf-8, pref cp1252
[debug] yt-dlp version 2<IP_ADDRESS> (win_exe)
[debug] Python version 3.8.10 (CPython 64bit) - Windows-10-10.0.19042-SP0
[debug] exe versions: none
[debug] Optional libraries: Cryptodome, mutagen, sqlite, websockets
[debug] Proxy map: {}
[debug] Loading archive file 'C:/users/seong/archive/public/videos/downloadedarchive.txt'
[debug] [youtube:tab] Extracting URL: https://www.youtube.com/c/PomplamooseMusic
.
.
.
[youtube]     Downloading comment API JSON reply thread 1 (961/2044)
[youtube]     Downloading comment API JSON reply thread 2 (963/2044)
[youtube]     Downloading comment API JSON reply thread 3 (970/2044)
[youtube]     Downloading comment API JSON reply thread 4 (973/2044)
[youtube]     Downloading comment API JSON reply thread 5 (977/2044)
[youtube] Downloading comment API JSON page 42 (986/2044)
[youtube]     Downloading comment API JSON reply thread 1 (994/2044)
WARNING: [youtube] Incomplete data received. Retrying ...
[youtube]     Downloading comment API JSON reply thread 1 (994/2044) (retry #1)
WARNING: [youtube] Incomplete data received. Retrying ...
[youtube]     Downloading comment API JSON reply thread 1 (994/2044) (retry #2)
WARNING: [youtube] Incomplete data received. Retrying ...
[youtube]     Downloading comment API JSON reply thread 1 (994/2044) (retry #3)
ERROR: Incomplete data received; please report this issue on  https://github.com/yt-dlp/yt-dlp . Make sure you are using the latest version; type  yt-dlp -U  to update. Be sure to call yt-dlp with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "yt_dlp\YoutubeDL.py", line 1311, in wrapper
  File "yt_dlp\YoutubeDL.py", line 1349, in __extract_info
  File "yt_dlp\YoutubeDL.py", line 1399, in process_ie_result
  File "yt_dlp\YoutubeDL.py", line 2430, in process_video_result
  File "yt_dlp\YoutubeDL.py", line 2599, in process_info
  File "yt_dlp\YoutubeDL.py", line 3063, in post_extract
  File "yt_dlp\YoutubeDL.py", line 3055, in actual_post_extract
  File "yt_dlp\extractor\common.py", line 3533, in extractor
  File "yt_dlp\extractor\youtube.py", line 2279, in _real_comment_extract
  File "yt_dlp\extractor\youtube.py", line 2223, in _comment_entries
  File "yt_dlp\extractor\youtube.py", line 2153, in extract_thread
  File "yt_dlp\extractor\youtube.py", line 2193, in _comment_entries
  File "yt_dlp\extractor\youtube.py", line 726, in _extract_response
yt_dlp.utils.ExtractorError: Incomplete data received; please report this issue on  https://github.com/yt-dlp/yt-dlp . Make sure you are using the latest version; type  yt-dlp -U  to update. Be sure to call yt-dlp with the --verbose flag and include its complete output.
.
.
.

Would you be able to provide the video id/link of the video this is occurring on? Appears to have been cut out of your logs

[debug] [youtube] Extracting URL: https://www.youtube.com/watch?v=wZNYDzNGB-Q

Can reproduce, but looks to be a server side issue. Though is rare, I have seen similar thing happen before (but not with comments). It happens in top sorting too.
Best is to wait it out and hopefully YouTube fixes it, or if you still want the comments up to that point you can download that video separately with --extractor-args youtube:max_comments=990

@coletdjnz We could make it so that the extracted comments are returned if user passes --ignore-errors even if there was some error. What do u think?

Another one: https://www.youtube.com/watch?v=QpPfabmu_Yk
[youtube] Downloading comment API JSON page 21 (433/1193)
[youtube]     Downloading comment API JSON reply thread 1 (437/1193)
WARNING: [youtube] Incomplete data received. Retrying ...
[youtube]     Downloading comment API JSON reply thread 1 (437/1193) (retry #1)
WARNING: [youtube] Incomplete data received. Retrying ...
[youtube]     Downloading comment API JSON reply thread 1 (437/1193) (retry #2)
WARNING: [youtube] Incomplete data received. Retrying ...
[youtube]     Downloading comment API JSON reply thread 1 (437/1193) (retry #3)
ERROR: Incomplete data received; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U

Latest master

Sorry to say, I've also found another one on version 2022.07.18: https://www.youtube.com/watch?v=0NKUpo_xKyQ
[youtube] Downloading comment API JSON page 1148 (35082/85112)
[youtube]     Downloading comment API JSON reply thread 1 (35092/85112)
WARNING: [youtube] Incomplete data received. Retrying ...
[youtube]     Downloading comment API JSON reply thread 1 (35092/85112) (retry #1)
WARNING: [youtube] Incomplete data received. Retrying ...
[youtube]     Downloading comment API JSON reply thread 1 (35092/85112) (retry #2)
WARNING: [youtube] Incomplete data received. Retrying ...
[youtube]     Downloading comment API JSON reply thread 1 (35092/85112) (retry #3)
ERROR: Incomplete data received; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
Traceback (most recent call last):
  File "yt_dlp\YoutubeDL.py", line 1431, in wrapper
  File "yt_dlp\YoutubeDL.py", line 1521, in __extract_info
  File "yt_dlp\YoutubeDL.py", line 1578, in process_ie_result
  File "yt_dlp\YoutubeDL.py", line 2569, in process_video_result
  File "yt_dlp\YoutubeDL.py", line 3360, in post_extract
  File "yt_dlp\YoutubeDL.py", line 3358, in actual_post_extract
  File "yt_dlp\extractor\common.py", line 3699, in extractor
  File "yt_dlp\extractor\youtube.py", line 2980, in _real_comment_extract
  File "yt_dlp\extractor\youtube.py", line 2954, in _comment_entries
  File "yt_dlp\extractor\youtube.py", line 2880, in extract_thread
  File "yt_dlp\extractor\youtube.py", line 2933, in _comment_entries
  File "yt_dlp\extractor\youtube.py", line 828, in _extract_response
yt_dlp.utils.ExtractorError: Incomplete data received; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U```

The issue is from Youtube's side as mentioned already. If you want to ignore this error, use -i. Providing more examples isn't helping anyone


[node-pod] Create A Kit At Scale Post

Background
With blog posts, we always seek to drive engagement and showcase excellence.
The goal for this post is to demonstrate to people how to utilize a starter.dev kit to build a production scale application that can grow with a business. starter.dev is similar to TodoMVC in that people can quickly use the scaffolded repo to build a quick example app. Individuals new to a technology tend to struggle with understanding how an application would scale as new features are added. This is a problem you will have needed to solve with the showcase application.
This blog post should not tackle every aspect of the kit as some topics are their own blog post, e.g. managing redux at scale. Topics like module management, route management, component tree structure, shared component libraries, etc. are topics for consideration.
This post should demonstrate a better-than-just-reading-the-docs understanding of this framework and could be passed around to other developers or a prospective client.
Acceptance

[ ] Create the idea for the blog post you want to create
[ ] Get your idea approved by your pod leader
[ ] Write a blog post that accomplishes the above
[ ] Have your pod technically review your post for accuracy
[ ] Send your blog to pod lead for review
[ ] Submit to the This Dot blog Contentful for posting
[ ] Write an accompanying Tweet to promote your blog to be submitted to the marketing team

Timebox: 8 hours

https://github.com/thisdot/blog-content/pull/16 <--- This is what I was able to come up with. Nobody was around for the idea approvals and stuff, so I went rogue. :)


Created a new Blog

I have added a new Blog on The Fascinating World of Youngsters' Hobbies in html. Please check and merge my pull request.

My PR will be merged or not?

My PR will be merged or not?

Akshita will merge it after reviewing

Ok. Thank You approving the changes


test: Cypress - Flaky test fix

Description
Fixed below flaky tests

GitImport_Spec.js
Entity_Explorer_Widgets_Copy_Paste_Delete_Undo_Keyboard_Event_spec.js

Type of change

Flaky test fix

How Has This Been Tested?

Cypress test runs

Checklist:
QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/ok-to-test sha=bd5b71a


Create phucdh0110.localplayer.dev.json

Requirements

[x] You have completed your website.
[x] The website is reachable.
[x] The CNAME record doesn't contain https:// or /.
[x] There is sufficient information at the owner field.
[x] There is no NS Records (Enforced as of Sepetember 4th, 2024)

Description
I use my domain for my own project + website + my own email for personal uses.
Link to Website
phucdh0110.vercel.app

I have read the CLA Document and I hereby sign the CLA

Add mail.phucdh0110.localplayer.dev for Microsoft 365 Domain Verification and add email domain


Formatting issue with summaries

The title, author list and summary are all contained within the same field rather than being given separate columns

The issue remains, but I would like to get the badge for closing an issue within 5 mins of opening it.


fails to install golang extension

Issue Type: Bug
brand new vs code install
I enter  ctrl + p
then paste
ext install golang.Go
to install golang extension as per your page
https://marketplace.visualstudio.com/items?itemName=golang.go
vs code fails with error
connect ENOENT /run/user/1000/vscode-b9b3c598-1.46.0-shared.sock
which is a totally useless error  ... get same error after uninstall then reinstall
installed on ubuntu 20.04 using
sudo dpkg -i code_1.46.0-1591780013_amd64.deb
please fix
VS Code version: Code 1.46.0 (a5d1cc28bb5da32ec67e86cc50f84c67cc690321, 2020-06-10T08:59:04.923Z)
OS version: Linux x64 5.4.0-38-generic

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz (8 x 3422)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: disabled_softwareskia_renderer: disabled_off_okvideo_decode: unavailable_offviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
0, 0, 0

Memory (System)
11.59GB (1.17GB free)

Process Argv
--no-sandbox --unity-launch

Screen Reader
no

VM
0%

DESKTOP_SESSION
ubuntu

XDG_CURRENT_DESKTOP
Unity

XDG_SESSION_DESKTOP
ubuntu

XDG_SESSION_TYPE
x11

Extensions: none

Does the /run/user/1000/vscode-b9b3c598-1.46.0-shared.sock file exist? Does the /run/user/1000 folder exist?

file
/run/user/1000/vscode-b9b3c598-1.46.0-shared.sock

does not exist however dir /run/user/1000/ does exist

Is there any /run/user/1000/*.sock file while Code is running?

yes see my above comment where I list directory  /run/user/1000/

Oh sorry about that. I know this sounds dumb... but have you tried restarting? Also, are you sure you don't have two versions of Code installed?


add autodoc plugin for mkdocs.

Add the autodoc plugin to mkdocs so that we don't have to duplicate work

The documentation will be built using sphinx in the 2021 version


If launched for the first time, we get the purple rectangle, then after a few secs, the screen to offer to connect.

Then, when I click "Use Ferdi without an account", we get the same behaviour as above.
This problem does not happen with 5.6.1-nightly.46. This does happen with 5.6.1-nightly.53 (and 5.6.1-nightly.54, and 5.6.1-nightly.55), so the bugs must have been introduced between 47 and 53 (inclusive)
Environment:

Operating System: Windows 10 21H1

Ferdi Version: see above

Server: Using without an account

Debug information:
Menu not available AFAICS

This happens to me with latest v5.6.1, on Windows 10. Ferdi launches and I get only the purple rectangle.

I have the same issue with Ferdi 5.6.1 running on Linux Mint 20.2 x86-64 (installed from deb package). I tried to reset configuration by deleting ~/.config/Ferdi/ but it still shows the purple rectangle after clicking "use ferdi without an account".

I have the same problem in Ferdi 5.6.1 on Linux Ubuntu 21.04

Same problem on Ubuntu 20.04 LTS

/cc @vraravam.

Seems to be fixed in 5.6.2 🙂

Seems to be fixed in 5.6.2 slightly_smiling_face

I can confirm this!

Closing this ticket - Thanks @janhohner , @jaimiewashere


Set IDS field when merging references with different citation keys

BibLaTeX has the ability to have multiple citation keys refer to the same bibliography entry by setting the IDS field in the .bib file to a comma separated list of the additional keys you want to refer to the entry. It would be great if BBT would automatically remember the non-primary ids of merged items, allow editing them, and write them to the IDS field when exporting BibLaTex. One use case for this is when merging two documents with different .bib files which have large overlap in the papers cited, but use different citation keys. Without this, one has to go through one of the documents manually to change all the keys to match the keys used in the other document.

This is an enhancement request. There is no error report or log.

You mean merged as in merged in Zotero? Because you can already specify an ids field by just adding bibtex[ids=key1,key2,key3,...] in the extra field.

I have to merge approximately 2,000 duplicate pairs with potentially different citation keys from several BibTeX files. For each of the 2,000 pairs, my process for merging them manually would be this:

Remember the title of the first publication displayed on the merge tab. (It is not possible to copy from the merge tab.)
Click on the library tab.
Select the search box.
Type the publication name I remembered into the search box.
Click on each paper with the same name to check whether the citation keys are different. If they are all the same, skip to step 14.
Click on the publication I am merging to and type "bibtex[ids=]" to the notes field.
Click on each other publication with the same name in the search results.
Select the citation key with my mouse.
Copy the citation key.
Click back to the original publication.
Click the "extra" field.
Paste the citation key into the into the extra field.
Click back to the merge tab.
Click merge.

I am slow at this, and this process takes me 30 seconds per duplicate pair. That times 2,000 is 60,000 seconds or about 16 hours. It also requires me to maintain concentration to remember the paper title when search and the citation key when checking if they are the same.
However, I am not picky about how the rest of the fields get merged. If the citation keys could be merged by just clicking "merge," doing that 2000 times takes maybe 10 minutes and requires no concentration.
I would be okay with writing a script to do the merge without using Zotero, but I do not know of any libraries that offer high quality and easily accessible duplicate detection heuristics. All the ones I know of mainly detect exact matches between fields. Do you know of any libraries or other software that might make this process easier? I would also be interested if there is a way to get Zotero to export a list of duplicates it has found within a collection with metadata saying which entries are duplicates. While I can export the full contents of the duplicates tab, this does not say which pairs of items should be merged.
Longer term, the kind of functionality I am looking for is anything that would make it easier to merge in bib/aux files that have large overlap with the existing Zotero library and do incremental updates as coauthors change them. This kind of functionality would make it much easier to use Zotero on multi-author projects with other authors who do not use it.
An example of a workflow for this could be something like the following.

Import a bib/aux file from my coauthors into my Zotero library.
Merge the duplicates in the Zotero or BBT UI.
BBT automatically generates the ids field when I do the merge.
BBT remembers what I previously imported so that if I ever try to import an exact duplicate of a previously imported publication, it gets excluded from the import.

BBT doesn't have an UI and is unlikely to grow one -- I really hate doing UI work. Everything UI you see is me hijacking existing Zotero UI.
I'm OK with thinking about a way to generate an ids= field from a merge, but it's not easy.
Zotero doesn't inform plugins that a merge has happened; technically, one item gets updated (the merge target) and one is deleted (the merge "victim"), and all I see is a modification and a deletion, as separate events, and while there is currently an order in which I receive them, this order is not guaranteed.
This is complicated by the fact that I delete the citation key for the "victim" item of the merge from my internal database, and I can't know up front that I'm supposed to hold on to this key for a little bit to see if a merge has happened, which as far as I can tell right now is registered after I get the deletion event, and I don't get any notification of this.
Detection and timing is going to be tricky on this thing. But I'll see what I can do.
Point 4 cannot be done realistically. Zotero importers can't look into the DB so I cannot assess whether an item already exists -- all they can do is take a stream of text, turn those into Zotero objects, and offer them to Zotero -- Zotero does the actual saving, and importers are not involved. I can't even see what file is being imported as importers only get handed a text stream to read from so I can't even say "I've seen this file name before".

:robot: this is your friendly neighborhood build bot announcing test build <IP_ADDRESS>95 ("forEach -> for")
Install in Zotero by downloading test build <IP_ADDRESS>95, opening the Zotero "Tools" menu, selecting "Add-ons", open the gear menu in the top right, and select "Install Add-on From File...".

:robot: this is your friendly neighborhood build bot announcing test build <IP_ADDRESS>96 ("fixed #1221")
Install in Zotero by downloading test build <IP_ADDRESS>96, opening the Zotero "Tools" menu, selecting "Add-ons", open the gear menu in the top right, and select "Install Add-on From File...".

Give 4296 a go. I've foregone the formal notification plumbing of Zotero and just attach to the actual action of merging (aka monkey-patching) -- it seems to work in my tests. It adds the aliases in the extra field with Citation Key Alias: in front -- you can edit this as you like, what's in the extra field is what is actually used on export. Mind that adding a bibtex[ids=...] will override, not add to this.

(monkey-patching is a little more fragile because Zotero is not required to keep the internals the same -- if they change, this breaks and I have to fix it)

This is an enhancement request. There is no error report or log. Is there a way to tag this as an enhancement request instead of a bug report?

I'd usually prefer a debug log nonetheless because it gives me a copy of your references and it makes it easier to reason about the output you want to see; I also use these to add test cases to my test suite. I've constructed a testcase myself in this instance, but testcases are really important because they prevent regressions.

:robot: this is your friendly neighborhood build bot announcing test build <IP_ADDRESS>97 ("testcase for #1221")
Install in Zotero by downloading test build <IP_ADDRESS>97, opening the Zotero "Tools" menu, selecting "Add-ons", open the gear menu in the top right, and select "Install Add-on From File...".

I really need to know of this does what you want

hello?

:robot: this is your friendly neighborhood build bot announcing test build <IP_ADDRESS>43 ("Merge branch 'master' into gh-1221")
Install in Zotero by downloading test build <IP_ADDRESS>43, opening the Zotero "Tools" menu, selecting "Add-ons", open the gear menu in the top right, and select "Install Add-on From File...".

Thank you very much for implementing this. It will be a few days and to weeks until I do the merge and can verify the new functionality. I did not think you would be this fast with adding something :-)

Can you do a test anyhow to see if it appears to do what you want? I prefer not to keep issues open for too long.

:robot: this is your friendly neighborhood build bot announcing test build <IP_ADDRESS>52 ("Merge branch 'master' into gh-1221")
Install in Zotero by downloading test build <IP_ADDRESS>52, opening the Zotero "Tools" menu, selecting "Add-ons", open the gear menu in the top right, and select "Install Add-on From File...".

:robot: this is your friendly neighborhood build bot announcing test build <IP_ADDRESS>18 ("Merge branch 'master' into gh-1221")
Install in Zotero by downloading test build <IP_ADDRESS>18, opening the Zotero "Tools" menu, selecting "Add-ons", open the gear menu in the top right, and select "Install Add-on From File...".


dcmtk: 3.6.7 -> 3.6.8

Description of changes
Fixes CVE-2022-43272.
Changes:
https://github.com/DCMTK/dcmtk/compare/DCMTK-3.6.7...DCMTK-3.6.8

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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

[x] 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/)
24.05 Release Notes (or backporting 23.05 and 23.11 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
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review run on x86_64-linux 1

1 package built:

dcmtk

Add a :+1: reaction to pull requests you find important.

Hmm tests failing on aarch64-linux?
@ofborg build dcmtk


Issues generating SBOM for Python environment

Using cdxgen 9.3.1, the following issues exist with a Python project (in a python virtual environment)

No dependencies are shown in the CycloneDX file
Installed modules are not reported
Some modules are reported twice representing the current and previously installed versions
Some modules included in SBOM are not installed in the environment

Installed cdxgen
Move to a Python virtual environment.pip list shows installed modules 31.
cdxgen -t python -o file.out
Output file contains 18 modules but several modules reported twice (looks like installed version and previous installed version). Ignoring duplicates, only 10 modules reported but 2 modules not installed according the the pip list
pip freeze shows 29 items (pip and setuptools missing from list produced by pip list)
Note it makes no difference if cdxgen is run within the Python virtual environment or just in the directory.
Note some of the installed modules have been installed from a local whl file rather than from pypi
Note Tried a clean Python virtual environment and reinstalled all of the modules from pypi. This time NO installed modules are included in the generated SBOM.

@anthonyharrison are there any package manifest files such as requirements txt or setup.py. If not this sounds like an enhancement since this scenario to generate sbom only for a venv is not tested yet.

@prabhu The modules are just 'pip installed' so no setup.py or requirements.txt in the directory
I have run cdxgen in a source directory (it takes a long time!) and it does generate an SBOM with dependencies. It still reports modules repeatedly (maybe all of the different versions of the module that has been installed?)


Abbreviation in GetLogLevelString()

In ConsoleLogger.GetLogLevelString(), level Trace outputs as trce, missing letter a; level debug outputs as dbug, missing letter e, etc.
Why does it need to use abbreviation? At the first time I saw the log output, I thought it's a typo.
I blame the file and found the code was submitted in 9e6e1674523649430dfe07bbbb74592c1c009b87, changing fully-spelled word to the current abbreviation form, but didn't give explanation.
If alignment is needed, we can use spaces. The current "dbug", "trce" look awkward, like a typo, and contrast to .NET framework's naming convention of not using abbreviation. (though I understand .NET framework's naming convention only applies to identifiers in the code)

The main reason was alignment, I believe. I don't expect we would change this after 3 years though. Spaces could work, but some folks' would likely get broken if we make a change like this at this point.
Is your main concern that it looks like a typo?

Somewhat yes. When I was trying this library, I only use log.debug(...), and it outputs  "dbug". I didn't realize the omission is for alignment until I use other log levels. If the method name is log.dbug(), it outputing "dbug" will not supprise me so much.
Is there a possibility to change GetLogLevelString() to protected method? This way supports customization and localization.

We periodically close 'discussion' issues that have not been updated in a long period of time.
We apologize if this causes any inconvenience. We ask that if you are still encountering an issue, please log a new issue with updated information and we will investigate.


Create constants.php For Manual Install

For Manual using the constant 'FACEBOOK_SDK_SRC_DIR' point to facebook directory

I do not understand what the need it for this constant? It also points to the library file and not the src of the Facebook SDK.
I also do not want to add the constants.php config file in this library as it is a default config file included in CI.


Add evaluation to configuration

This PR adds time-varying evaluation, updates README, and refactors test file.
Evaluation is conducted in the same timeframe as projections. It supposes to evaluate the metrics sequentially as forecast start date moving forward by a given interval, ending one interval before forecast end date. option is added to enable the choice between producing projections or evaluation.
More details are added to Getting Started in README.
Evaluation test is simplified to one test function.
The file changed in PR#81 is copied here due to rebase, but no need to review. These changes have been organized in the commit "rebase #81", only the commits before need review. Thanks!

@eschrom @Fuhan-Yang My top line point in all of this is that we would need to loop over a lot of things:

Data set
Season (although I'm confused if "data set" and "a season of data for a particular vaccine" are different things)
Model
Forecast date
Score function (aka evaluation metric)

See one example of what that would look like, using loops, in #83
I think the way you want to do this instead is:

Preprocessing

Clean the raw data
Separate the raw data into whatever chunks you'll want to run independently (i.e., a season of data for a particular vaccine) -- this could still be one data frame, but with some column that allows you to pick out the particular chunk you want to use
Save this somewhere

Model runs

For every "chunk" and forecasting date and model, do the forecasts
I think it's way easier to think of this, having a forecast() function somewhere in there, rather than computation nested inside 3 levels of looping!
Save every result

Evaluation

For every chunk, model, forecast date, and score function, do the scores
Save every result

Unfortunately Makefiles don't make it easy to say "run every model on every forecast date." (Something like Snakemake could, but let's leave that aside for now.) So I don't mind sticking with Python for now.
Unless this whole pipeline runs in less than 5 seconds, I would want preprocessing, doing the forecasts, and scoring the forecasts to be in separate scripts. Otherwise the time required to debug and iterate is going to be too long!

And then see #84 for how I would (start to) adapt those loops into those separate scripts


Target Android 9, to allow Android 10 devices to execute binaries from app home dir

Android 10 won't allow apps targeting Android 10 to excute binaries from its app home directory. Currently this causes ABCore on Android 10 to fail launching tor and bitcoind and crash soon afterwards.
This PR rolls back ABCore to target Android 9 (API level 28) in order to fix this temporarily. See issue #97 for more details and discussion of future options

utack 1d36f69a2901c58cf1c48d79b73f5d50d1cf6027


Extract TrackNo from ytmusic

Checklist

[X] I'm reporting a feature request
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2022.05.18 (update instructions) or later (specify commit)
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Description

https://music.youtube.com/watch?v=AUfALRGInd4&list=RDAMVMAUfALRGInd4
Go to This Is Acting (Deluxe Version) https://music.youtube.com/watch?v=AUfALRGInd4&list=RDAMVMAUfALRGInd4#:~:text=This Is Acting (Deluxe Version)
Lookup songs title
It is 6th Track https://music.youtube.com/playlist?list=OLAK5uy_lrmHcevmuzfs6xgfVerncDBqYJLNS-0K4#:~:text=6-,Cheap Thrills,-3%3A32

Verbose log
[debug] Command-line config: ['--config-location', 'D:\\\Path\\youtube-dl\\youtubedl/config/music.conf', 'https://www.youtube.com/watch?v=RvBrsX4xMJg']
[debug] | Config "D:\Path\youtube-dl\youtubedl/config/music.conf": ['-f', '251', '--audio-format', 'opus', '-x', '--output', 'Downloads/Music/%(title)s.%(ext)s', '--write-sub', '--write-auto-sub', '--sub-lang', 'en', '--convert-subs', 'lrc', '--embed-sub', '--embed-thumbnail', '--add-metadata', '--embed-metadata', '--xattrs', '--embed-chapters', '--geo-bypass-country', 'US', '--compat-options', 'abort-on-error', '--verbose', '--print', 'playlist_index']
[debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.05.18 [b14d523] (win_exe)
[debug] Compatibility options: abort-on-error
[debug] Python version 3.8.10 (CPython 64bit) - Windows-10-10.0.19043-SP0
[debug] Checking exe version: ffprobe -bsfs
[debug] Checking exe version: ffmpeg -bsfs
[debug] exe versions: ffmpeg 4.4-full_build-www.gyan.dev (setts), ffprobe 4.4-full_build-www.gyan.dev
[debug] Optional libraries: Cryptodome-3.14.1, brotli-1.0.9, certifi-2021.10.08, mutagen-1.45.1, sqlite3-2.6.0, websockets-10.3
[debug] Proxy map: {}
[debug] Using fake IP <IP_ADDRESS> (US) as X-Forwarded-For
[debug] [youtube] Extracting URL: https://www.youtube.com/watch?v=RvBrsX4xMJg
[youtube] RvBrsX4xMJg: Downloading webpage
[youtube] RvBrsX4xMJg: Downloading android player API JSON
[debug] Sort order given by extractor: quality, res, fps, hdr:12, source, codec:vp9.2, lang, proto
[debug] Formats sorted by: hasvid, ie_pref, quality, res, fps, hdr:12(7), source, vcodec:vp9.2(10), acodec, lang, proto, filesize, fs_approx, tbr, vbr, abr, asr, vext, aext, hasaud, id
[info] RvBrsX4xMJg: Downloading 1 format(s): 251
NA

@gamer191 not valid

Why not?

Also, did you try --print playlist_index

@gamer191 --print playlist_index tried it now it doesn't download at all ends at NA verbose in description

https://music.youtube.com/watch?v=AUfALRGInd4&list=RDAMVMAUfALRGInd4

Go to This Is Acting (Deluxe Version) https://music.youtube.com/watch?v=AUfALRGInd4&list=RDAMVMAUfALRGInd4#:~:text=This Is Acting (Deluxe Version)

Lookup songs title

It is 6th Track https://music.youtube.com/playlist?list=OLAK5uy_lrmHcevmuzfs6xgfVerncDBqYJLNS-0K4#:~:text=6-,Cheap Thrills,-3%3A32

FYI, the album url can be obtained through the GUI by clicking the 3 dots and clicking "go to album"

@pukkandan

The video metadata does not contain any "track number" information. So this is not possible

I know --print playlist_index doesn't work I was replying to @gamer191, I am suggesting a new feature here.

As I said already, the URL that you are passing https://www.youtube.com/watch?v=RvBrsX4xMJg does not contain the track number!! yt-dlp is not a program to compile metadata from other sources. You should look at other programs like ytmdl for it

@pukkandan not other sources, youtube music has the data


feat(packer): support for template directories

Description
Fixes #1271
With this PR you can use directories for Packer templates
Release Notes (draft)
Added support for Packer template directories

Hello,
Adding tests would ensure the ongoing functionality of these changes. Please consider including them. Thank you! 👍

I'm not really sure how to add the tests, since this function calls the actual packer binary

Hi,
can be added example for packer and respective tests
References:
https://github.com/gruntwork-io/terratest/blob/master/test/terraform_packer_example_test.go
https://github.com/gruntwork-io/terratest/tree/master/examples/packer-docker-example

is this still ongoing? we need this because our packer template use multiple files

We use this in production and works, however I didn't come around to write the tests

@NefixEstrada do you mind if I take a look and try to create the test?

@walbertus
Sure, go ahead!
Thanks,
Néfix

any input @denis256 ?
I'm thinking of adding a mock here to check the binary call but can't find a way to do it.
Do you have test recommendations in mind?


fix install groovy unzip bug

Fix install groovy unzip bug

Thanks for working on it.
Could you please file a Jira ticket for Bigtop first? Then, rename the PR title to BIGTOP-xxxx: Fix .....
It would be appreciated if you could provide instructions on how to reproduce the issue on the relevant Jira.
@jasonchrion