Merge Homebrew/15c62dfd0 into Linuxbrew/master

See https://github.com/Homebrew/brew/pull/4628

Codecov Report

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

@@            Coverage Diff             @@
##           master     #777      +/-   ##
==========================================
+ Coverage   70.45%   70.47%   +0.02%
==========================================
  Files         386      384       -2
  Lines       20702    20527     -175
==========================================
- Hits        14585    14466     -119
+ Misses       6117     6061      -56

Impacted Files
Coverage Δ

extend/os/mac/unpack_strategy/zip.rb
50% <0%> (-50%)
:arrow_down:

cask/lib/hbc/url.rb
92.3% <0%> (-7.7%)
:arrow_down:

cask/lib/hbc/cli.rb
89.18% <0%> (-0.08%)
:arrow_down:

cask/lib/hbc/cli/fetch.rb
100% <0%> (ø)
:arrow_up:

cask/lib/hbc.rb
100% <0%> (ø)
:arrow_up:

rubocops/formula_desc_cop.rb
100% <0%> (ø)
:arrow_up:

cask/lib/hbc/download_strategy.rb

cask/lib/hbc/cli/internal_audit_modified_casks.rb

cask/lib/hbc/installer.rb
85.23% <0%> (+0.04%)
:arrow_up:

download_strategy.rb
67.5% <0%> (+0.16%)
:arrow_up:

... and 4 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 58c5c32...105bb3c. Read the comment docs.


Specialize HABv4 signing to i.MX6/7 in preparation for signing through U-Boot

Changes:

Add a warning message on the imx-boot recipe to let users know signing will likely not work (relevant to the Toradex verdin-imx8mm and verdin-imx8mp modules).
Since signing on i.MX8M is now supposed to be done through U-Boot (like with the i.MX6/7), here we make a small refactor by renaming scripts and templates to make them more specific to the i.MX6/7 in preparation for the additions that will in turn be specific to the i.MX8M.

By the way, the following commits in the PR are theoretically applicable also to kirkstone:

u-boot-hab: make CSF-related source files specific to i.MX6/7
u-boot-hab: make CSF creation script specific to i.MX6/7
u-boot-hab: make signing function specific to i.MX6/7
imx-boot-hab: warn users about possibly non-working HABv4 signing
imx-boot-hab: move duplicated code to common signing function

Question: do we want to cherry pick them to kirkstone?

LGTM


firebase_auth: verifyPhoneNumber should not sign in

I want to link phone number to my user's account.
But I can't as verifyPhoneNumber implicitly signs in on native side.
https://github.com/flutter/plugins/blob/6341d3a5c7b36194ab0fd50a839b363579c3eb37/packages/firebase_auth/android/src/main/java/io/flutter/plugins/firebaseauth/FirebaseAuthPlugin.java#L161-L176
To be consistent with other methods, it should instead return Future<sms code>, and report verificationId with callback as it might be expired while waiting for the code sent.
///
Future<String> verifyPhoneNumber({
  @required String phoneNumber,
  @required Duration timeout,
  int forceResendingToken,
  @required PhoneCodeSent codeSent,
  @required PhoneCodeAutoRetrievalTimeout codeAutoRetrievalTimeout,
}) async {}

Future<FirebaseUser> signInWithPhoneNumber({
  @required String verificationId,
  @required String smsCode,
}) async {}

Future<FirebaseUser> linkWithPhoneNumber({
  @required String verificationId,
  @required String smsCode,
}) async {

But this is breaking change.

@kdy1 verifyPhoneNumber does not sign in. You can easily test that. The verificationId can be accessed through the codeSent callback.
I am not sure when onVerificationCompleted is called, but definitely not in the process of receiving the sms code. You can use either linkWithPhoneNumber or signInWithPhoneNumber after sending the code.
It might be possible that this happens when you are signed out, but I do not think so. I only tested it with anonymous accounts, but either way: If you want to link the account, verifyPhoneNumber will not sign you in with a different account and if you want to use signInWithPhoneNumber, it does not matter if it would be called natively because you would want to call it anyway.
However, I am quite confused with that implementation in general. I am pretty sure that onVerificationCompleted is never called in the process (you can prove me wrong, I only tested it with anonymous accounts) because it would also not make any sense. In order for an account to be created via a phone number, it needs to be verified in the first place, which is why I do not understand when onVerificationCompleted is thought to be executed in the lifecycle.
TL;DR
I believe that the current implementation is totally fine in the regard you mentioned and that the problem does not exist.

But if you authenticate user then call verifyPhoneNumber, user account is switched.
I digged the issue, and native code calls signInWithCredential. I think that's verifySmsCode accepts onVerficationComplete(FirebaseUser user).

@kdy1 But not before verifying?
Signing in before verifying cannot work because you need both the verificationId and smsCode to sign in with a phone number and the sms code is isolated from the application.

@creativecreatorormaybenot I get the problem, @kdy1 is saying that the only way to verify the SMS code is to use signInWithPhoneNumber, and that switches with whatever account is currently signed in. IIRC, on Android, there's a separate method for just verifying the SMS code, and that would solve the problem. I don't know on iOS but I am also looking for a solution to this as it breaks the signin/signup flow for many apps.

@creativecreatorormaybenot  onVerificationCompleted(PhoneAuthCredential) might be called automatically.
https://firebase.google.com/docs/auth/android/phone-auth?hl=en#verification-callbacks
And when it's called automatically, current api does not provide any way to get the sms code. (Instead it signs in)
timeout parameter controls only the duration of automatic retrieval. You can check this by setting it larger than 120 seconds. It will result in a PlatformException.
@gildaswise I can verify it with signInWithPhoneNumber(verificationId: null, smsCode: null), but I can't send the sms code, and can't get auto-retrieved sms code.

How can you verify the SMS code if you're passing null arguments? It should throw an error. Also, you can follow this tutorial I made, it sets up the phone verification on both Android and iOS but the problem of it overriding the previous signed in account persists, that's why I'm only using signInWithGoogle after the SMS code has been verified.

@gildaswise It was just about signature. I'll edit it.
Only automatic retrieval was problematic, and I successfully used @creativecreatorormaybenot 's branch to link phone number to existing account.
Maybe also title should be change to reflect the fact that instant verification and auto retrieval are problematic.

This is a serious blocker to my project, I am letting the user sign in with google/facebook. After the AuthCredential is obtained from the 3rd party login I create a new user and  then I try to login with phone number and associate it to the current user, but the plugin automatically signs in with the new PhoneAuthCredential on this line https://github.com/flutter/plugins/blob/49c4ad9a079f2dbf79324f3957bc774a0e294970/packages/firebase_auth/android/src/main/java/io/flutter/plugins/firebaseauth/FirebaseAuthPlugin.java#L193
I want the raw PhoneAuthCredential as I would be able to sign in or link it to an existing user. Also I think it should be similar to what Google, Facebook sign-in work as they give cred object and we can use it as we like.
I would love to send a PR to fix this, please let me know if that's ok or maybe I am missing something.
Maybe @amirh can help.

I have fixed this issue and landed a PR for it here. Waiting for Googlers to review it.

The PR has been merged. I think this issue should be closed as it has been merged. We are havving 5k+ issues and plenty of them are just not valid anymore. @collinjackson

@kdy1
This issue has been moved to https://github.com/FirebaseExtended/flutterfire/issues/646. Any further collaboration will be done there.


Using the autoload-dev setting on composer.json

Changed log

In the composer.json file, the "Swop\\GitHubWebHook\\Tests\\": "tests/" should be on autoload-dev setting.

thanks @peter279k !

Available in 3.1 https://github.com/Swop/github-webhook/releases/tag/v3.1


Refactor to class

work in progress

Ok I think this is nearly ready. The openmc_plasma_source package needs updating now to make use of this plotting class


Sleep Duration Calculator

Develop a calculator that assists users in determining their recommended sleep duration based on their age and lifestyle.

Hey, @Rakesh9100 can I work on this issue?

Sure @PRASUN-SITAULA


Add necessary javadoc to util and methods

Thanks @gitcliff thanks for  the work here.
can you instead use java Documentation Comments ??

@mozzy11 kindly review

Thanks @gitcliff ,
@bashir2  was suggesting some renaming of some classes. he can give some more clarification on that

@mozzy11 any more comments on this issue ?

Closing this stale PR; please open a new one if this is still needed.


Restoring 12 words to iPad Air Gen 3 causes empty wallet

Describe the bug
Our customer reported an issue that after restoring a 12 word seed to an iPad Air Gen. 3 no hotspots and 0 HNT balance was shown while the account owned hotspots and had balance.
We could verify that the address/public-key reported by the app was correct (by scanning the QR-code and copying the address).
Restoring the same 12 word seed to an iPhone XR did allow for the hotspots and HNT balance to be shown.

is this reproducible? Wonder if it was an API issue where the balance wasn't reported in time

Not completely sure; The issue was persistent the whole time (several days) for sure (so not a one time thing). But I don’t think the customer re-installed the app so not sure long time cache or something could explain this. I will ask.

@cokes518 reproducable:
Re-installing app and restoring 12-word seed on iPad Air shows no balance or hotspots. Address in QR is correct.
Doing the same on iPhone XR does show balance and hotspots.

we can't reproduce on our end - closing this issue. please reopen if that customer is still having issues.


Project ownership

I just noticed that the people who have write access on the project are no longer at the UTC.
Recent contributors are @courentin and @dotpy3
It would be good for the guys still at the UTC to come up with a plan to ensure that the ownership is extended/transferred so that someone from the SiMDE/BDE/UVWeb asso has the merge power.
cc @pauljenny

Adding @noeamiot to the owners of the organization to transfer the project to https://github.com/orgs/simde-utc


Manual testing is too inconvenient

Current options are
PACKAGE=ciscospark npm run grunt:package -- build test --grep "name of my unit test"

or

Replace main for package under test with src isntead of dist

run package tests:
PACKAGE=ciscospark npm run test:package -- --grep "name of my unit test"

This is mostly fixed by
npm run distsrc
npm test -- --package ciscospark --grep "rooms"

but we can make it better by loading a require hook (under certain conditions) that replaces dist with src.


Q：add a slider to play  multi-frame images

Hi.  I want to add a slider on webpage. When i drag the slider, i can change the image displayed on this page. To avoid the blank when use sequence mode, i use the addtiledImage. But, if i drag the silder too fast, the screen will be flickering. I think it's because the image hasn't been fully loaded. How can i fix it?
Can the fully-loaded work for this problem?
Here is the code of slider part.
 $("#slider").slider({
        max: src.length-1,
		animate: "slow",
  slide: function( event, ui ) {
           $("#result").text(ui.value);//get the pecent
            a = event.target;//get the event
           idName = a.attributes["id"];//get the id
		   ind = eval(document.getElementById('result').textContent);
		   test1=ind;
		viewer.addTiledImage({
            tileSource:tileSources[test1],
            index:0,
              });
           viewer.world.removeItem(viewer.world.getItemAt(1));
		   }
		  });

Thanks a lot!

One more question, if I load all images first through addtiledImage , I can play these  image sequence without flicker.
I play the multi-image by setting the item.opacity=0. But, in this way, when i zoom into a new area and then drag the slider back to previous frames, the flicker and blur will be shown again. What can i do if i want to load the previous frames first ? tileImage.Preload=true; does not work.
for (i=1;i < src.length; i++)
 view2.addTiledImage({
            tileSource:tileSources[i],
            index:0,
              });
var max=src.length;
	$("#slider").slider({
	    max: src.length-1,
		animate: "fast",
       slide: function( event, ui ) {
           $("#result").text(ui.value);//get the pecent
            a = event.target;//get the event
           idName = a.attributes["id"];//get the id
		   ind = eval(document.getElementById('result').textContent);
		   //test1=ind;
		   count=max-ind;
		   counts=count-1;
		   if (count<src.length){
		    var tileImage=view2.world.getItemAt(count);
			var tileImage2=view2.world.getItemAt(counts);
			var obj=tileImage.getOpacity();
			var obj2=tileImage2.getOpacity();
			if (obj)
			{
			 tileImage.setOpacity(0);
			}
			else{
			tileImage2.setOpacity(1);
			//tileImage.Preload=true;
			}
			}

Thanks again

In either case you want to wait until the incoming image is loaded. You could make a function like this for that purpose:
function whenLoaded(tiledImage, callback) {
  if (tiledImage.getFullyLoaded()) {
    callback();
  } else {
    tiledImage.addOnceHandler('fully-loaded-change', callback);
  }
}

...and then you can call it like this:
whenLoaded(newTiledImage, function() {
  oldTiledImage.setOpacity(0);
  newTiledImage.setOpacity(1);
});

As for the preload flag, there's no such thing as tileImage.Preload. You can either set it up with addTiledImage, like so:
viewer.addTiledImage({
  tileSource: tileSource,
  preload: true
});

...or you can set it later like so:
tiledImage.setPreload(true);

Thx! I can drag the slider to play image sequence by loading all of the images into the same viewer at once but hiding one tileImage.setOpacity(0); . Of course, the method above is really helpful, i think it will work. Here my way is preloading the previous frame by setting opacity=0.01, which also can solve the flickering problem.
Here is the code, and i have tested it, it worked.
$("#slider").slider({
	    max: src.length-1,
		animate: "fast",
       slide: function( event, ui ) {
           $("#result").text(ui.value);//get the pecent
            a = event.target;//get the event
           idName = a.attributes["id"];//get the id
		   ind = eval(document.getElementById('result').textContent);
		   //test1=ind;
		   count=max-ind;
		   counts=count-1;
		   if (count<src.length){
		    tileImage=view2.world.getItemAt(count);
			tileImage2=view2.world.getItemAt(counts);
			var obj=tileImage.getOpacity();
			var obj2=tileImage2.getOpacity();
			if (obj==1)
			{
			 tileImage.setOpacity(0);
			}
			else{
			tileImage2.setOpacity(1);
			//tileImage.setOpacity(0.01);
			//tileImage.Preload=true;
			}
			}
			}
})
$("#slider").slider({
change: function(){
for (i=0;i<src.length;i++){
              if (view2.world.getItemAt(i).getOpacity()==0)
              {
                view2.world.getItemAt(i).setOpacity(0.01);
              }
               }}
})

Good to hear! :)


invalid literal for int() with base 10: '0.79;logic_name=cpg'

Description of the bug
I am running nf-core/rnaseq with the following command:
nextflow run nf-core/rnaseq
   -r 3.14.0
   -profile docker
  --input /media/unidad/Expansion/Raw_Data/RNAseq_MS/samplesheet.csv
  --outdir /media/unidad/Expansion/analysis_MS_nfcore/results/rnaseq
  --fasta /media/unidad/Expansion/analysis_MS_nfcore/database/genomes/GRCh38/genome.fasta
  --gtf /media/unidad/Expansion/analysis_MS_nfcore/database/genomes/GRCh38/genes.gtf
  --transcript_fasta /media/unidad/Expansion/analysis_MS_nfcore/database/genomes/GRCh38/transcript.fasta
  --star_index /media/unidad/Expansion/analysis_MS_nfcore/database/indexes/GRCh38/STAR
  --salmon_index /media/unidad/Expansion/analysis_MS_nfcore/database/indexes/GRCh38/SALMON   --aligner star_salmon   -w /mnt/MyBook/work
  --max_cpus 6
  --max_memory 50.GB
  --save_unaligned
  -resume`

and I obtained the following error:

`-[nf-core/rnaseq] Pipeline completed with errors-
ERROR ~ Error executing process > 'NFCORE_RNASEQ:RNASEQ:BAM_RSEQC:RSEQC_JUNCTIONSATURATION (HCMS01)'

Caused by:
  Process `NFCORE_RNASEQ:RNASEQ:BAM_RSEQC:RSEQC_JUNCTIONSATURATION (HCMS01)` terminated with an error exit status (1)

Command executed:

  junction_saturation.py \
      -i HCMS01.markdup.sorted.bam \
      -r genes.bed \
      -o HCMS01 \

  cat <<-END_VERSIONS > versions.yml
  "NFCORE_RNASEQ:RNASEQ:BAM_RSEQC:RSEQC_JUNCTIONSATURATION":
      rseqc: $(junction_saturation.py --version | sed -e "s/junction_saturation.py //g")
  END_VERSIONS

Command exit status:
  1

Command output:
  (empty)

Command error:
  Unable to find image 'quay.io/biocontainers/rseqc:5.0.3--py39hf95cd2a_0' locally
  5.0.3--py39hf95cd2a_0: Pulling from biocontainers/rseqc
  642efca944a0: Already exists
  bd9ddc54bea9: Already exists
  4778d18b334d: Pulling fs layer
  4778d18b334d: Download complete
  4778d18b334d: Pull complete
  Digest: sha256:9fc7027efc23a9dd2309ead1d285034da324eae64e6e940cb0760f4525fa28aa
  Status: Downloaded newer image for quay.io/biocontainers/rseqc:5.0.3--py39hf95cd2a_0
  WARNING: Your kernel does not support swap limit capabilities or the cgroup is not mounted. Memory limited without swap.
  [E::idx_find_and_load] Could not retrieve index file for 'HCMS01.markdup.sorted.bam'
  reading reference bed file:  genes.bed  ...  Invalid bed line (skipped): 1	0	248956422	chromosome:1	.	.	GRCh38	chromosome	.	ID=chromosome:1;Alias=CM000663.2,chr1,NC_000001.11
   Traceback (most recent call last):
    File "/usr/local/bin/junction_saturation.py", line 97, in <module>
      main()
    File "/usr/local/bin/junction_saturation.py", line 83, in main
      obj.saturation_junction(outfile=options.output_prefix, refgene=options.refgene_bed, sample_start=options.percentile_low_bound,sample_end=options.percentile_up_bound,sample_step=options.percentile_step,min_intron=options.minimum_intron_size,recur=options.minimum_splice_read, q_cut = options.map_qual)
    File "/usr/local/lib/python3.9/site-packages/qcmodule/SAM.py", line 3928, in saturation_junction
      exon_starts = list(map( int, fields[11].rstrip( ',\n' ).split( ',' ) ))
  ValueError: invalid literal for int() with base 10: '0.79;logic_name=cpg'

Work dir:
  /mnt/MyBook/work/66/dcc3242a95a1e8cabbb89d3b6c34cf

Tip: view the complete command output by changing to the process work dir and entering the command `cat .command.out`

 -- Check '.nextflow.log' file for details

the gtf file was obtained from ENSEMBL:
wget -L ftp://ftp.ensembl.org/pub/release-${latest_release}/gtf/homo_sapiens/Homo_sapiens.GRCh38.${latest_release}.gtf.gz

and the gene.bed file was created from the gtf file using GTF2BED:
process GTF2BED {
    tag "$gtf"
    label 'process_low'

    conda "conda-forge::perl=5.26.2"
    container "${ workflow.containerEngine == 'singularity' && !task.ext.singularity_pull_docker_container ?
        'https://depot.galaxyproject.org/singularity/perl:5.26.2' :
        'biocontainers/perl:5.26.2' }"

    input:
    path gtf

    output:
    path '*.bed'       , emit: bed
    path "versions.yml", emit: versions

    when:
    task.ext.when == null || task.ext.when

    script: // This script is bundled with the pipeline, in nf-core/rnaseq/bin/
    """
    gtf2bed \\
        $gtf \\
        > ${gtf.baseName}.bed

    cat <<-END_VERSIONS > versions.yml
    "${task.process}":
        perl: \$(echo \$(perl --version 2>&1) | sed 's/.*v\\(.*\\)) built.*/\\1/')
    END_VERSIONS
    """

}
Command used and terminal output
No response
Relevant files
No response
System information
No response

@ZabalaAitor I suspect an incompatibility between FASTA and GTF- could you confirm where the FASTA you're using came from please?

Yes, it was because of an incombatibility between the FASTA and GTF files.
Thanks!


Add Approximate counting in Java

We'll merge this when #913 is merged, @PaddyKe you can probably use the blueprint provided by the PR so we can merge it rapidly

#913 is merged and I commited @stormofice 'ssuggestion, so I will merge this


add support for timermarker

Hi, I need use the information of timer marker in CHAT files, so I modified two functions in this package.

Hi, sorry for not getting to this until now... Thanks for the new feature contribution! I've just merged your contribution into a non-master branch for now (see the time-marker branch) as I notice that some tests didn't pass in the Travis CI builds in your PR and I'll need to fix them before merging the new feature into master. (My guess is that the CHILDES data upstream has changed and that's what broke the tests, but I'll have to dig into it.)
As for the Reader being slow, I've opened https://github.com/pylangacq/pylangacq/issues/4 to track the issue.


tests(rest-client/zbg): don't fail on unreachable

test(rest-client/zbg): don't unwrap on connection failed
tests(rest-client/zbg): don't fail on unreachable

If you get a network error, you should try to fix your network, for example restart your router, use a HTTP proxy, etc.
We should fail fast and fail explicitly in tests.

Actually this fail is happened in CI: https://github.com/crypto-crawler/crypto-crawler-rs/runs/7625266397


Add documents on kmesh locality failover features

What would you like to be added:
Why is this needed:

@derekwin

to this pr: https://github.com/kmesh-net/website/pull/90

The implementation schematic can be viewed here: https://github.com/kmesh-net/kmesh/pull/574

done


Authorization header works in api global header but not in per-request config?

I'm trying to build a simple api client, running on iOS. The following works:
  const api = apisauce.create({
    baseURL,
    headers: {
      'Content-Type': 'application/json; charset=UTF-8',
      Accept: 'application/json; charset=UTF-8',
      'Cache-Control': 'no-cache',
      Authorization: 'Basic ' + 'BASE_64_KEY',
    },
    timeout: 5000,
  });

But when I insert the Authorization header per-request like this:
  const retrieveUser = (
    id: number,
    context: string,
    username: string,
    password: string,
  ) =>
    api.get(
      'users/' + id + (context ? '?context=' + context : ''),
      { context: 'edit' },
      {
        header: {
          Authorization: 'Basic ' + 'BASE_64_KEY',
        },
      },
    );

It doesn't work and I always get 401 error, what could be wrong here?

headers is spelled header In the 2nd block?

Oh I see, how silly I am. I also forgot to convert the proper string to base64.


[es] Span write fails if index already exists due to race condition

ES version is 5.1.1
{
"level": "error",
"ts":<PHONE_NUMBER>.7463179,
"caller": "app/span_processor.go:103",
"msg": "Failed to save span",
"error": "Failed to create index: elastic: Error 400 (Bad Request): index [jaeger-service-2017-12-12/CV-EQal-Q1KQ3vEyxG1W7g] already exists [type=index_already_exists_exception]",
"errorVerbose": "elastic: Error 400 (Bad Request): index [jaeger-service-2017-12-12/CV-EQal-Q1KQ3vEyxG1W7g] already exists [type=index_already_exists_exception]
Failed to create index
github.com/jaegertracing/jaeger/plugin/storage/es/spanstore.(*SpanWriter).logError
/home/travis/gopath/src/github.com/jaegertracing/jaeger/plugin/storage/es/spanstore/writer.go:191
github.com/jaegertracing/jaeger/plugin/storage/es/spanstore.(*SpanWriter).createIndex
/home/travis/gopath/src/github.com/jaegertracing/jaeger/plugin/storage/es/spanstore/writer.go:148
github.com/jaegertracing/jaeger/plugin/storage/es/spanstore.(*SpanWriter).WriteSpan
/home/travis/gopath/src/github.com/jaegertracing/jaeger/plugin/storage/es/spanstore/writer.go:120
github.com/jaegertracing/jaeger/cmd/collector/app.(*spanProcessor).saveSpan
/home/travis/gopath/src/github.com/jaegertracing/jaeger/cmd/collector/app/span_processor.go:102
github.com/jaegertracing/jaeger/cmd/collector/app.(*spanProcessor).(github.com/jaegertracing/jaeger/cmd/collector/app.saveSpan)-fm
/home/travis/gopath/src/github.com/jaegertracing/jaeger/cmd/collector/app/span_processor.go:89
github.com/jaegertracing/jaeger/cmd/collector/app.ChainedProcessSpan.func1
/home/travis/gopath/src/github.com/jaegertracing/jaeger/cmd/collector/app/model_consumer.go:32
github.com/jaegertracing/jaeger/cmd/collector/app.(*spanProcessor).processItemFromQueue
/home/travis/gopath/src/github.com/jaegertracing/jaeger/cmd/collector/app/span_processor.go:126
github.com/jaegertracing/jaeger/cmd/collector/app.NewSpanProcessor.func1
/home/travis/gopath/src/github.com/jaegertracing/jaeger/cmd/collector/app/span_processor.go:57
github.com/jaegertracing/jaeger/pkg/queue.(*BoundedQueue).StartConsumers.func1
/home/travis/gopath/src/github.com/jaegertracing/jaeger/pkg/queue/bounded_queue.go:65
runtime.goexit
/home/travis/.gimme/versions/go1.9.linux.amd64/src/runtime/asm_amd64.s:2337",
"stacktrace": "github.com/jaegertracing/jaeger/cmd/collector/app.(*spanProcessor).saveSpan
/home/travis/gopath/src/github.com/jaegertracing/jaeger/cmd/collector/app/span_processor.go:103
github.com/jaegertracing/jaeger/cmd/collector/app.(*spanProcessor).(github.com/jaegertracing/jaeger/cmd/collector/app.saveSpan)-fm
/home/travis/gopath/src/github.com/jaegertracing/jaeger/cmd/collector/app/span_processor.go:89
github.com/jaegertracing/jaeger/cmd/collector/app.ChainedProcessSpan.func1
/home/travis/gopath/src/github.com/jaegertracing/jaeger/cmd/collector/app/model_consumer.go:32
github.com/jaegertracing/jaeger/cmd/collector/app.(*spanProcessor).processItemFromQueue
/home/travis/gopath/src/github.com/jaegertracing/jaeger/cmd/collector/app/span_processor.go:126
github.com/jaegertracing/jaeger/cmd/collector/app.NewSpanProcessor.func1
/home/travis/gopath/src/github.com/jaegertracing/jaeger/cmd/collector/app/span_processor.go:57
github.com/jaegertracing/jaeger/pkg/queue.(*BoundedQueue).StartConsumers.func1
/home/travis/gopath/src/github.com/jaegertracing/jaeger/pkg/queue/bounded_queue.go:65"
}

There is a race condition in the span writer, and the comment in the code calls it out:
// if there are multiple collectors writing to the same elasticsearch host, if the collectors pass
// the exists check above and try to create the same index all at once, this might fail and
// drop a couple spans (~1 per collector). Creating indices ahead of time alleviates this issue.

@black-adder did we envision any mechanisms for "creating indices ahead of time"?
Another, possibly a lot simpler alternative, would be to check if the error was specifically "index already exists".

ES driver exposes the Error struct, which has Error.Details.Type string that could be checked against "index_already_exists_exception" (assuming that string value is stable).


Update by tiktoken version

Thanks! I just gave your PR a whirl in our use of this within the Drupal community and all still working as expected for us. Would be a great new release.

@scotteuser  Thank you for contributing !


Update SS Conditions cmp to avoid unnnedded update

Use status package to properly compare conditions.
Also avoid code repetition between NRO and SS as much as possible.
handing #169

/cc @fromanirh

/cherry-pick release-4.10


how to reconnect printer before print?

how we can connect printer before print for avoiding connection reset and broken pipe issue like if there is any way to calling this private EscPosPrinter getPrinter(CallbackContext callbackContext, JSONObject data) i think problem will be solved

Disconnect -> https://github.com/paystory-de/thermal-printer-cordova-plugin#disconnectprinterdata-successcallback-errorcallback

When should I disconnect the printer? If I disconnect after printing I'm unable to print again later. 😕


k_nl integral

Implementation of the below integral which computes the scale for a non-linear cut. Currently, the basic code changes have been made, but unit tests and benchmarks are missing, and not all documentation has been updated.

Pull Request Test Coverage Report for Build 4645

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
10 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.06%) to 96.344%

Files with Coverage Reduction
New Missed Lines
%

pyccl/boltzmann.py
10
92.75%

Totals

Change from base Build 4614:
-0.06%

Covered Lines:
2925

Relevant Lines:
3036

💛  - Coveralls

As far as I'm aware the only remaining issues are: 1) not being able to re-build the 0000-ccl_note in my local environment (issue with mkauthlist); 2) a similar issue with building readthedocs; 3) flake8 really hating ccllib.py.

Is it necessary to describe what the new benchmark test consists of? If so, is the wiki the best place for this?

@slosar , to answer your question "Where is EPSREL_KNL defined?", it is defined in src/ccl_core.c -- see line 53 (#define GSL_EPSREL_KNL 1E-5). The value of 10^-5 doesn't seem to be too stringent -- it allows the benchmark to pass on my machine at least. Let me know if you want any of this changed.

@beckermr The two instances of the flake8 issue which I think are causing the Travis-CI checks to fail are:
pyccl/correlation.py:141:43: E741 ambiguous variable name 'l'
pyccl/tests/test_correlation.py:102:50: E741 ambiguous variable name 'l'
I haven't touched either of these files in this PR, so I guess this would be a problem on the master branch too.

I fixed that here: https://github.com/LSSTDESC/CCL/pull/783
However, that required changing quite a lot of the python files, since flake8 has become more of a brat lately. If people are happy with that PR (@slosar?) we could merge that, and this should be fixed if you then merge master into this PR.

@billwright93 if you merge master into this branch the flake8 problems will go away!

@billwright93 if you merge master into this branch the flake8 problems will go away!

Great thanks, will give it a go!


[devbox global] Added init hook flag to global shellenv

Summary
Added --init-hook flag to devbox global shellenv
How was it tested?

./devbox global add hello
edit global devbox.json code ~/.local/share/devbox/global/default/devbox.json and add init_hook
./devbox global shellenv --init-hook
confirm the line . ~/.local/share/devbox/global/default/.devbox/gen/scripts/.hooks.sh is included in the output.
cat ~/.local/share/devbox/global/default/.devbox/gen/scripts/.hooks.sh and confirm the init_hook is included in the file.

This addresses some of the simplifying I suggested in my comment: https://github.com/jetpack-io/devbox/pull/921

Thanks @mikeland86 for making changes that simplified this PR a lot.
Per your suggestions I removed d.ensurePackagesAreInstalled from PrintEnv() and put d.generateShellFiles() with a comment on why it's added.
I also removed impl.GenerateShellEnv() and instead of checking if we are in a devbox shell or not when running devbox shellenv, I removed DEVBOX_SHELL_ENABLED=1 from the output of shellenv. Which I believe is the correct thing to do since shellenv should not care if you're in a devbox shell or not. It should just compute and print the environment.
cc: @savil


[feature] modify the reload binding that is bound to event change

[x] I have read through the manual page (man fzf)
[x] I have the latest version of fzf
[x] I have searched through the existing issues

Info

OS

[x] Linux
[ ] Mac OS X
[ ] Windows
[ ] Etc.

Shell

[x] bash
[ ] zsh
[ ] fish

Problem / Steps to reproduce
I was trying to use fzf with the flags
--bind "change:reload:SOME_COMMANDS"
--bind "alt-n:...something to change reload:SOME_COMMANDS"

I've tryed:

--bind "alt-n:change:reload:SOME_OTHER_COMMANDS
--bind "alt-n:rebind(change:reload:SOME_OTHER_COMMANDS)
--bind "alt-n:unbind(change)+rebind(change:reload:SOME_OTHER_COMMANDS)

Changing the binding is not supported. See https://github.com/junegunn/fzf/pull/2564
You can instead make your reload command react to an external state that is updated in a different binding. It's a sketchy example, but you get the idea.
echo 10 > /tmp/limit
cmd='seq $(cat /tmp/limit)'
eval "$cmd" | fzf --bind "change:reload:$cmd" --bind "alt-n:execute-silent(echo 20 > /tmp/limit)+reload($cmd)" --disabled

Changing the binding is not supported. See #2564
You can instead make your reload command react to an external state that is updated in a different binding. It's a sketchy example, but you get the idea.
cmd='seq $(cat /tmp/limit)'
echo 10 > /tmp/limit
eval "$cmd" | fzf --bind "change:reload:$cmd" --bind "alt-n:execute-silent(echo 20 > /tmp/limit)+reload($cmd)" --disabled

# or
echo 10 > /tmp/limit
eval "$cmd" | fzf --bind "change:reload:$cmd" --bind "alt-n:reload(echo 20 > /tmp/limit; $cmd)" --disabled

Extending the ripgrep 'Switching to fzf-only search mode' example
for instance
RG_PREFIX_COLOR="rg --column --line-number --no-heading --color=always --smart-case "
RG_PREFIX_NO_COLOR="rg --column --line-number --no-heading --color=never --smart-case "
INITIAL_QUERY="${*:-}"
IFS=: read -ra selected < <(
  FZF_DEFAULT_COMMAND="$RG_PREFIX $(printf %q "$INITIAL_QUERY")" \
  fzf --ansi \
      --disabled --query "$INITIAL_QUERY" \
      --bind "change:reload:sleep 0.1; $RG_PREFIX_NO_COLOR {q} || true" \
      --bind "alt-enter:unbind(change,alt-enter)+change-prompt(2. rgcolor> )+rebind("change:reload:sleep 0.1; $RG_PREFIX_COLOR {q} || true" \
)
[ -n "${selected[0]}" ] && vim "${selected[0]}" "+${selected[1]}"

In this scenario, I can't use the strategy you mention, right? This is a lousy example, but you get my point.

Why not? You can write a reload command that works differently depending on the existence of a file that is written or deleted on a key event.

When I specify the reload action on the second bind
 --bind "alt-enter:unbind(change,alt-enter)+change-prompt(2. rgcolor> )+reload($SOME_DIFFERENT_COMMAND" )

When I press alt-enter the correct reload run once and update the output all fine. But once I press a key it's trigger the change event, and runs first defined reload (change:reload, defined on the first bind)
It's this the normal behavior, or it's a bug?

Oh, I see!
I understand now what you meant earlier! Thanks!!!


Blog - Working on ESP devkits using WSL2

Description

Introduces the topic of working with Espressif's devkits using the WSL
Simple guide how to use the WSL with serial port forwarding

Related

Internal tracker IDF-4361

Testing

Things that may be worth mentioning:

In step 2, the usbipd can be seen only after opening new terminal
Step usbipd bind --busid 1-1 requires Administrator privileges

I've successfully managed to complete the tutorial. I can see, the new device using the susb command in the WSL terminal.
But when listing the ports ls /dev/tty*, there is no change. I assumed that I can use this article to communicate with the chip from wsl, but that require the port specification.
Am I missing something? Is this the point where I need to use the esp_rfc2217_server?

Thank you, @mfialaf for the review.
I have improved the description of the steps you have pointed out. Also updated the guide with the mentioned issue as we have solved this in person. Thank you for the help!

Hi @jakub-kocka, thank you for the article.
Can you please rebase? This will fix some issues and the CI will pass. Also, update the article date to a recent one.

Hi @pedrominatel, thanks for the review. However, rebase doesn't seem to help.

Don't worry, @jakub-kocka . I'll take a look.

Not sure what is wrong with the product selector link. It keeps appearing as invalid in this PR. However, as soon as I push these changes to a test PR, the checks always pass successfully and the status of checks here immediately switches to all passed.
This PR should fix such issues, hopefully, once and for all!


Handle 'test error' to deal with error response from invalid browsers.

Tested with configuration:
      'saucelabs-custom': {
        all: {
          options: {
            urls: [
              'http://<IP_ADDRESS>:3000/'
            ],
            browsers: [
              {
                browserName : 'chrome',
                platform : 'OS X 10.11',
                version : 26
              }
            ],
            statusCheckAttempts : -1
          }
        }
      }

Chrome 26 on OS X 10.11 is unsupported by sauce labs. Returns a response where result.status is "test error". Expected behavior: test fails with an error. Actual behavior: request will be re-attempted, and when statusCheckAttempts is set to -1, the test will never complete.
This PR causes the test to complete with an error.

I just noticed, this is a duplicate of #224


Fixing Contract Tests with Open GSN

refactor test cases for all contracts that had open GSN integration

/cib


Shift+Tab Behaviour Regression in 4.0

Godot version
4.0 Beta 2
System information
Windows 10
Issue description
Shift + Tab doesn't remove leftmost tab if the caret is anywhere but next to a tab which it then proceeds to remove even if it is in middle of the text. Current behaviour is exactly the same as just pressing backspace.
In 3.x you can Shift + Tab to remove the leftmost tab no matter the caret's position in the line which is much more useful.
Steps to reproduce
Copy paste the following line into the Script Editor in Godot 4.0 and put the caret at the end of the sentence.
	# then try pressing Shift+Tab

In 3.x this would have deleted the tab at the beginning of the line.
Minimal reproduction project
No response

Duplicate of https://github.com/godotengine/godot/issues/50279.


Toc generation broken

Renderer
v3
Browser
Chrome
Operating System
Windows
What happened?
Toc generation "up to..." is broken. When clicked, {{tocDepthH3}} appear but with nothing inside
Code
No response

Thank you for reporting this issue, you are completely right, this is broken. We are looking into it.

Actually, nevermind, after talking to the team, this is indeed expected behaviour and not a change since the last changes:

Only the "Table of Contents" button makes a table of contents. All the "Include..." Buttons just add markers that will influence which headers will be considered when you click "Table of Contents"

Okay, so how i'm supposed to generate the toc ?
Because right now, adding the markers {{tocDepthH3}} doesnt change the generated toc

Okay, so, i misinterpreted that PR, and have never used this feature, but:
If you add headers inside that block:
{{tocDepthH6

###### hello test test test
}}

Any header up to the desired depth inside that will be included.
The other way of doing this is with CSS, setting it like this:
.page h4 {
  --TOC: include;
}

I think this is the correct way.
Btw, even if they are included, we don't have styles prepared for 6 levels of headers, so any header below 3 is styled as 3, i'm not sure how to change that.

Ok I understand now. Is there a documentation somewhere for theses features ?
And can we exclude h3 with the css snippet you shared ? So the toc generation will just generate H1 and H2 ?

Yes, but specificity might be a problem if you use both the blocks and the css snippet, if it doesn't work at the first try, try adding !important.


feat: expose serve-static builder

This allows accessing middleware/serve-static from third-party
Use case
I run my own handler based on Cloudflare R2 to avoid the limitations of Cloudflare KV / Pages. And I found that most of the code I needed there was already built into hono.
So what I expect is like:
import { Hono } from 'hono';
import { serveStatic as makeServe } from 'hono/base-serve-static';

const app = new Hono<Env>();

const serveContent = makeServe<Env>({
  getContent: async (path, c) => {
    // retrieve
    let response = await cache.match(c.req.raw);

    // ...get R2 object
    if (!response) {
      let obj = await c.env.CONTENT.get(path);
      if (obj) {
        response = obj.body;
        c.executionCtx.waitUntil(cache.put(c.req.raw, obj.body));
      }
    }

    c.body(response);
  },
});

app.use(serveContent);

export default app;

Author should do the followings, if applicable

[ ] Add tests
[x] Run tests
[x] yarn denoify to generate files for Deno

Hi @cometkim
This is good! But please wait until I look over the details!

@yusukebe added a type definition rather than any as it would be a public interface.
And made some changes to allow a situation where users needed to create the Response object themselves. (e.g. using with Browser/Cloudflare's Cache API)

@cometkim Thanks!


@ashfurrow => Adds support for sending select events to Adjust - fixes #749

@jessekedy would you look at that!

Looks good.

So what’s going on here? Is this a partial implementation, can it be merged, what gives?

Think it just never got merged.

Okidoki :+1:


Add icon align prop

Fixes PXBLUE-1472

Changes proposed in this Pull Request:

Add iconAlign attr

Screenshots:

Not sure how big of a deal this is, but when the page is RTL and iconAlign="left", the icon is aligned to the right. This is also the case for iconAlign="right"... the icon is then left aligned for RTL.

Not sure how big of a deal this is, but when the page is RTL and iconAlign="left", the icon is aligned to the right. This is also the case for iconAlign="right"... the icon is then left aligned for RTL.

We discussed this on the React PR and agreed that it's fine. We already have things like 'leftContent' and 'rightContent' which will be backwards too. Given that the primary use case is LTR, we will orient these props based on that.


Usage

Sorry for the lack of understanding.  I was directed here from the sqlserver adapter project.  https://github.com/rails-sqlserver/activerecord-sqlserver-adapter
How do you use mini_portile with a Rails application so that FREEtds is installed and therefore the tiny_tds gem works?
Thanks

Please read tiny_tds installation instructions in the readme file.
Sorry for top posting. Sent from mobile.
On Feb 21, 2012 8:20 PM, "workmaster2n" <
<EMAIL_ADDRESS>wrote:

Sorry for the lack of understanding.  I was directed here from the
sqlserver adapter project.
https://github.com/rails-sqlserver/activerecord-sqlserver-adapter
How do you use mini_portile with a Rails application so that FREEtds is
installed and therefore the tiny_tds gem works?
Thanks

Reply to this email directly or view it on GitHub:
https://github.com/luislavena/mini_portile/issues/12

Dunno who directed you here, but mini_portile is a tool for developers.
Please read TinyTDS readme about building and installing a local gem.
Or, you can simply meet TinyTDS dependency requirements (like installing FreeTDS and iconv) and then install the gem.

You were right, I misread the tiny_tds instructions.  Awesome library though, I'll definitely try it out in my projects


Code for Macnet

Dear Maintainers,
I hope this message finds you well. Could you please guide me on where to find the code related to the paper "Scaling Large-Language-Model-based Multi-Agent Collaboration"?
Thank you for your assistance!

Thank you for reaching out to us! This technique will soon be incorporated into this repository, please stay tuned.


qualityBool should be removed : text is unlcear

The below text is ambiguous . There should be a clear directive text....either qualityBool has to be removed or not?
Enhance the rule, indicating that you also want to exclude the variable qualityBool

Hi indcoder, this is for the predictive model and as such, the dependent variable should be removed.


Modernize build infrastructure

The build system hasn't been meaningfully touched in a very long time, and uses the old-style setuptools setup.
Python has moved on from this, and Setuptools has a newer, cleaner interface that should be used instead based on PEP517.
Additionally, with the introduction of PEP-517, I have become partial to using poetry as a replacement for setuptools, and am interested in using that for Pydle's packaging solution instead.
Regardless of which tool is used, Pydle should modernize.
I do no view the build system as covered by semver, and it will not break existing installs or block them from upgrading. Switching to PEP517 will only effect source installs that use an outdated pip. Installers from pypi should be unaffected.

As you're the maintainer I’ll leave the decision on what to do up to you,
but to chime in with some of my own experiences:
Personally I’ve found elaborate use of poetry and pyproject.toml to be mostly unwieldy solutions in search of a meaningful problem for my use cases, also applicable here since pydle has very few requirements.
For personal projects, I’ve switched to converting setup.py to its declarative cousin, setup.cfg, and a simple pyproject.toml as follows:
[build-system]
requires = ["setuptools"]
build-backend = “setuptools.build_meta"

and an empty setup.py as some setuptools features won't work without it (especially in older versions):
# for legacy compatibility
from setuptools import setup

setup()

I think this generally provides a good balance, as tools like mypy and yapf also support being configured through setup.cfg. I also generally opt to remove requirements.txt and put the dependencies in extras_require in setup.cfg.
Hope this helps.

Thank you for your opinion @Shizmob .
Its currently a toss up between switching to setuptool's newer declarative interface with setup.cfg + pyproject.toml, and using poetry with pyproject.toml.
Mypy, yapf, black, and pytest all support configuration via pyproject.toml, which appears to be the new standard.
requirements.txt will be removed regardless of the choice made, as both paths move that configuration.
I am fine going either way.
In my personal projects, i find poetry is useful even when there are very little requirements, given its ability to also handle virtual environments and how painless packaging with it is.
I also recently used the newer setuptools format in a project, and found a footgun or two around package discovery leading to broken wheels.
The only real benefit of setuptools today that i see is its support for native modules, which pydle thankfully doesn't deal with.


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_2b0c81bevb/clone/recipe/meta.yaml:14: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_2b0c81bevb/clone/recipe/meta.yaml:20: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_2b0c81bevb/clone/recipe/meta.yaml:37: missing_license_family: The recipe is missing the about/license_family key.
WARNING: /tmp/abs_2b0c81bevb/clone/recipe/meta.yaml:37: missing_description: The recipe is missing a description
ERROR: /tmp/abs_2b0c81bevb/clone/recipe/meta.yaml:37: missing_dev_url: The recipe is missing a dev_url
ERROR: /tmp/abs_2b0c81bevb/clone/recipe/meta.yaml:37: missing_documentation: The recipe is missing a doc_url or doc_source_url
Errors were found


Mixtral output mismatch on multi chip nightly

https://github.com/tenstorrent/tt-metal/actions/runs/8952213400/job/24590048040
This is seen without FD2 changes. After fd2 merge this pipeline hangs on an earlier test, so not sure how this test runs with FD2

@uaydonat , can someone on your team take a look at this? c.c. @yieldthought as well.
Btw.. @tt-dma , was there a commit that this test pass before? (i double check your spreadsheet and it looks like multi-chip nightly was passing quite recently)

@uaydonat , can someone on your team take a look at this? c.c. @yieldthought as well.
Btw.. @tt-dma , was there a commit that this test pass before? (i double check your spreadsheet and it looks like multi-chip nightly was passing quite recently)

@jliangTT I believe this test is new, it wasn't present in the nightly multi-chip runs for FD2, e.g. https://github.com/tenstorrent/tt-metal/actions/runs/8941347012

Yes, mixtral was added to CI on May 1st.

On it.
On Mon, 6 May 2024 at 04:46, Utku Aydonat @.***> wrote:

Yes, mixtral was added to CI on May 1st.
—
Reply to this email directly, view it on GitHub
https://github.com/tenstorrent/tt-metal/issues/8147#issuecomment-2095104995,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHHDXMLK3DRQTKVCYYPSB3ZA3VITAVCNFSM6AAAAABHHEUAO6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJVGEYDIOJZGU
.
You are receiving this because you were mentioned.Message ID:
@.***>

@yieldthought has fix, waiting for further testing

@yieldthought can we close this?


.col-xs-1 after .col-xs-11 wraps on mobile

On an iPhone, in portrait mode, in a row with 2 columns (.col-xs-11, .col-xs-1), the .col-xs-1 is pushed onto the next line. Reproduce by viewing the document below in the iPhone simulator. I confirmed that it also happens on the iPhone hardware.
<!DOCTYPE html>

		<link rel="stylesheet" href="http://netdna.bootstrapcdn.com/bootstrap/3.1.1/css/bootstrap.min.css">
		<title>iOS Bug in Bootstrap 3</title>

		<meta content="width=device-width, initial-scale=1.0, maximum-scale=1.0, user-scalable=no" name="viewport">

So this problem isn't going to be fixed?

I've also gotten this issue when using 4 3 column blocks on a large screen e.g. (col-lg-3) x 4, and then when trying to switch to a col-sm-6 x 4 which would ideally fill out two rows they flew all over the place.

So this problem isn't going to be fixed?

It is WONTFIX for Bootstrap v3. We hope to address it in Bootstrap v4.

I've also gotten this issue when using [...]

Feel free to open a new issue for that case.


refactor(): Added "selling the product" to README.

Simply added the components from the CartoGraphX document into the VSCode README.

[ ] Does the PR description accurately reflect the changes made?
[ ] Do the commit messages accurately reflect the changes made?
[ ] Is the changelist for this pull request under ~200 lines? (Excepting automatically generated code and package installations)
[ ] Does the code follow the coding style guidelines that we have set up?
[ ] Does the coder include unit tests for his functions?
[ ] Does the QA who reviews the code include his unit tests for all the functions?
[ ] Is there a documentation in place if the functions is related to how users build, test, interact with, or release code?

Codecov Report

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

@@            Coverage Diff             @@
##           master      #64      +/-   ##
==========================================
+ Coverage   64.09%   64.54%   +0.45%
==========================================
  Files           7        7
  Lines         220      220
  Branches       30       30
==========================================
+ Hits          141      142       +1
  Misses         56       56
+ Partials       23       22       -1

Flag
Coverage Δ

#unittests
64.54% <ø> (+0.45%)
:arrow_up:

Impacted Files
Coverage Δ

src/webview.ts
88.88% <0.00%> (+5.55%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a0ce23c...dd572b5. Read the comment docs.


kubernetes-dashboard: raise memory limit to 250MB

On my cluster, I often get OOM errors that cause the dashboard to crash just by going over 50MB.
Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

cc @offlinehacker @globin


Does Rapid's .map() trigger moveend events?

Description
Currently I use "move.embed" alongside a throttle method to capture map position updates as user pans around the map. However, for simplicity and consistency I would like to use event like "moveend.embed" to receive updates only on panning end. Is there such an event available? I tried to search the code for various keywords with no luck.
    const ctx = context
        .embed(true)
        .assetPath(`static/${version}/`)
        .locale(primaryLanguage)
        .containerNode(idContainer)
        .init()

    const map = ctx.map()

    // On map move, send the new state to the parent
    // TODO: isn't there moveend event?
    map.addEventListener(
        "move.embed",
        throttle(() => {
            // Skip if in intro
            if (ctx.inIntro()) return

            const [lon, lat] = map.center()
            const zoom = Math.floor(map.zoom())
            parent.postMessage({ type: "mapState", state: { lon, lat, zoom } }, "*")
        }, 250),
    )

A big, big thank you. I couldn't dream of a better response :star_struck:.

A big, big thank you. I couldn't dream of a better response

You're welcome!  I'm also happy to help out.

I appreciate that! It will be best to do so after I prepare contributor documentation and finalize the base code. I will note to send you a nudge when that happens. I want to have a running project before other people can contribute to it. A test server will also be launched for the public access. A few things must happen before that:

Finish JavaScript migration, so code left here (not that much!)
Migrate enough HTML templates to make the website usable
Prepare the contributor documentation: a GitHub-hosted wiki


[v1.2][ISSUE-306] Add RAG operators

What changes were proposed in this pull request?

Add RAG operators
Why are the changes needed?

How was this patch tested?

UT

DocumentSplit and LangChainTextSplitter merge as one： is nltk model same?
Move splitter.py and embedding.py from document to primitives
rename document embedding to document ingestion

DocumentSplit and LangChainTextSplitter merge as one： is nltk model same?
Move splitter.py and embedding.py from document to primitives
rename document embedding to document ingestion

refactored


Disable stdout register unless we're actually outputting

Description
Altered latency tracer to only register stdout if we intend to pipe it to unreal output.
This was done because sim clients also capture all stdout, not just what unreal outputs, so they were always capturing the tracer output, which was often many MB. Can still be turned on by altering the LogSpatialLatencyTracing category, but must be done before the call to SpatialLatencyTracer::RegisterProject
Release note
Internal tool, not externally documented.
Primary reviewers
@ImprobableVlad @improbable-danny

Good stuff! I assume tests indicate that this gets rid of the spam?

Yep - results here - https://buildkite.com/improbable/unrealgdk-nfr/builds/1427


 doesn't works in Modal Component [Android]

Bug report
Summary
I'm using  component from react-native-maps 0.28.0, So I decided to draw that component into a  for get user location and so on, I'm using  component that comes from -> react-native-magnus (https://magnus-ui.com/docs/modal/) but I don't think that's the problem, because that magnus modal component it's based on react-native-modal. I try to set flex: 1 to  Component, I've checked the permissions, but it doesn't works for me.
Environment info
OS: Windows 10
Platform: Android (Real Phone Xiaomi Redmi Note 8 Pro)
React: 17.0.1
React Native: 0.64.1
React Native Maps: 0.28.0
React Native Magnus: 1.0.62
React Native Modal: 12.0.2
System:     OS: Windows 10 10.0.19041     CPU: (4) x64 Intel(R) Core(TM) i3-4130 CPU @ 3.40GHz     Memory: 2.81 GB / 7.92 GB Binaries:     Node: 14.17.3 - ~\AppData\Local\Temp\yarn--1626544554660-0.5613243802934613\node.CMD     Yarn: 1.22.5 - ~\AppData\Local\Temp\yarn--1626544554660-0.5613243802934613\yarn.CMD     npm: 6.14.13 - C:\Program Files\nodejs\npm.CMD     Watchman: Not Found SDKs:     Android SDK:       API Levels: 29, 30       Build Tools: 28.0.3, 29.0.2, 29.0.3, 30.0.2       System Images: android-29 | Intel x86 Atom_64     Android NDK: Not Found     Windows SDK: Not Found IDEs:     Android Studio: Version  <IP_ADDRESS> AI-193.69<IP_ADDRESS>26763     Visual Studio: Not Found   Languages:     Java: 1.8.0_262 - C:\Program Files\OpenJDK\openjdk-8u262-b10\bin\javac.EXE   npmPackages:     @react-native-community/cli: Not Found     react: 17.0.1 => 17.0.1     react-native: 0.64.1 => 0.64.1     react-native-windows: Not Found   npmGlobalPackages:     *react-native*: Not Found

It should be clarified that it works perfectly is a  component or within a React Navigation Tab and so on, but within a  it does not work.

I just did a quick test, the following works perfectly fine for me:
import React from 'react';
import {StyleSheet, Modal, View} from 'react-native';
import MapView from 'react-native-maps';

export default function App() {
  return (
    <View style={styles.container}>
      <Modal visible={true}>
        <MapView style={styles.mapView} />
      </Modal>
    </View>
  );
}

const styles = StyleSheet.create({
  container: {
    flex: 1,
  },
  mapView: {
    flex: 1,
  },
});

I'm going to close this for now, as I don't think it's related to react-native-maps. If you believe that this is still relevant and related to react-native-maps, please feel free to open a new issue.


Add endpoint for retrieving # of finished challenges

The number of finished challenges can be calculated by summing up the score of the user in all of the groups that they have joined.

That's getting super important.
As currently the frontend is what computes the number of finished challenges. It computes it by iterating over the user challenges. Which means that the backend needs to send all of the user challenges to the client, which slows the network requests significantly as the user challenges increase.


Add support for typeDateTim4 and typeDateTime in switch statements

I found two more column data types that weren't fully supported in the makeGoLangScanType func and associated switch statements, which result in panics.
SMALLDATETIME NOT NULL and DATETIME NOT NULL

I just noticed a partially conflicting PR in #302, so imported those related changes into this branch.
I also used the string type "BINARY" for typeBigBinary instead of "TIMESTAMP", to comply with https://msdn.microsoft.com/en-us/library/dd358341.aspx. Does that still work for your timestamp use case @brown-m?

@forfuncsake i'm in two minds with respect to the MSDN spec. to my mind, a binary data type is a general storage case with the responsibility to track how to encode / decode the stored data falling on the application developer, for instance, storing binary data for PNGs (a crap idea not recommended). whereas TIMESTAMP, whilst technically binary data, an application designer has no input as to what data gets stored in it as it's populated by the DB. with that in mind i would expect the library to handle it as a TIMESTAMP instead of a generalised binary type.
simply put, and I may be in the minority with this wish, i would expect the use of ColumnTypes() to give me comparable results to running a select statement on INFORMATION_SCHEMA which does return TIMESTAMP for the column type instead of a generic BINARY field. but one can dream...
i believe the intent and usage for the ColumnTypes() function should be clarified before deciding on a definitive course of action.

and/or @kardianos


MODE-2474 Disallows the store-name attribute when non-composite binary stores are used

Looks great.


fix postgresql typo in json data type documentation

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/d24ead68d4db421cd49e1dea7cad5e759bec13a5/
Edited pages:

v19.1/jsonb.md
v19.2/jsonb.md
v2.0/jsonb.md
v2.1/jsonb.md

@jseldess will you like to review this?


Using MinIO

Instead of AWS S3, can I use MinIO ?

This repository is going to be archived as we don't maintain it anymore. We are now doing etcd backups using https://github.com/giantswarm/etcd-backup-operator. Feel free to open an issue there or fork this repository.


Question: How do you discern which device to connect to during Buttonless DFU?

I have one device running the app and then a couple of devices in DFU mode. When the device is put in DFU mode by the Buttonless DFU functionality how do you determine which device to re-connect to? I know the device in DFU has a MAC address that is +1 of the normal MAC address but my understanding is that iOS does not expose the MAC-address.
The reason I'm asking is just to double check that there is no risk for me to do DFU on the wrong device when I have multiple devices available.

Hi, so in all SDKs before 14 there is a chance you connect to a wrong one. DfuServiceInitiator exposes the peripheralSelector which could be set to override the default behavior which is selecting the first device advertising with DFU UUID (legacy or secure, depending on what buttonless service was found). You may for example use the RSSI to select the closes, etc.
In SDK 14 the "set bootloader name" feature was added, where during jumping to the DFU mode the lib sends a random name before jump command and the bootloader will advertise with this name instead of DfuTarg. So if the device supported it the lib will then scan for this name (see here).

Link to the specs: http://infocenter.nordicsemi.com/topic/com.nordic.infocenter.sdk5.v14.0.0/service_dfu.html


python310Packages.edalize: 0.4.1 -> 0.5.0

Diff: https://github.com/olofk/edalize/compare/refs/tags/v0.4.1...v0.5.0
Changelog: https://github.com/olofk/edalize/releases/tag/v0.5.0
Description of changes
Fix build (https://hydra.nixos.org/build/217517947)

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/)
23.05 Release Notes (or backporting 22.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

[ ] Fits CONTRIBUTING.md.

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

5 packages built:

python310Packages.edalize
python310Packages.edalize.dist
python311Packages.edalize
python311Packages.edalize.dist
silice


Do not use constructor in entity hydrator

This PR allows Entitys to have constructors.

Closing this as SW will not merge this


Bug 1093210 move newrelic to a separate file

This is to unblock https://bugzilla.mozilla.org/show_bug.cgi?id=1093210

Per discussion on the bug, I don't think we should follow that approach. Instead, we should look into integrating newrelic with our js bundles. See https://bugzilla.mozilla.org/show_bug.cgi?id=1093210 comment 6 to 9.


feat(relay): persist and expose messages through grpc

adds persistence to the relay server for messages and exposes them through the grpc server.
the goal is to have messages be published through the relay server rpc which will handle persistence and broadcasting to the grpc service which will then expose a subscription stream of new messages for a specific topic and retrieval of past messages by topic
all that while using the existing defined Models from the dojo world for the payload and keys of the messages

[x] protobuf service and messages
[x] relay storage for message persistence
[x] grpc message subscription
[x] grpc messages retrieval
[x] shared model storage for messages

@Larkooo for wasm support if cainome is causing troubles, the new tag used in the merge audit PR is:
cainome = { git = "https://github.com/cartridge-gg/cainome", tag = "v0.2.4", features = ["abigen-rs"] }


Run and check examples on CI

Thanks for noting this. On a longer time scale I think examples should be part of the documentation page, and maybe something like doctest can be used for them.

Thanks for noting this. On a longer time scale I think examples should be part of the documentation page, and maybe something like doctest can be used for them.

Another option is sphinx-gallery.


boards/arm: Adding Contextual Electronics Advanced BLE Cell board to Zephyr

This commit adds supports for the nRF52840 based BLE Cell board from Contextual
Electronics. This board contains support for BG95 Modem, BQ52895 charger, SD
card etc and can be used as a PI Hat.
In this commit, this board supports UART, I2C, SPI, Modem. Support for charger,
SD card and other things will be added later.
Signed-off-by: Bilal Wasim<EMAIL_ADDRESS>

cc: @carlescufi

@carlescufi , This is a custom board based on nRF52840 SoC. Can I get a review from you and other Nordic maintainers !

Getting a strange buildkite failure which is unrelated to the pull request (https://buildkite.com/zephyr/zephyr/builds/17251#31c9b051-7f2e-4171-b824-0aed10314ab2). Seems that the server node is out of memory.

Getting a strange buildkite failure which is unrelated to the pull request (https://buildkite.com/zephyr/zephyr/builds/17251#31c9b051-7f2e-4171-b824-0aed10314ab2). Seems that the server node is out of memory.

How likely is it this board will be made available?

This would be a question for @ChrisGammell

How likely is it this board will be made available?

This would be a question for @ChrisGammell

How likely is it this board will be made available?

This would be a question for @ChrisGammell

The initial plan for the board was the educational aspect, but I have since decided to put the boards up for sale for others. I'm still working through the details of that.
I will update the documentation to reflect future availability.

How likely is it this board will be made available?

This would be a question for @ChrisGammell

The initial plan for the board was the educational aspect, but I have since decided to put the boards up for sale for others. I'm still working through the details of that.
I will update the documentation to reflect future availability.


[Feature] How to override queue timeout?

My queue worker's default timeout is at 60 seconds which is ideal for most tasks.
But I have a lot of posts to make searchable for ElasticSearch and it can take up to 300 seconds to run, so I want to set the job timeout to that amount. How can I do that?
I think I would need to somehow add public $timeout = 300; to \Matchish\ScoutElasticSearch\Jobs\QueueableJob, but I can't make changes to that class that's part of the library.

Feel free to open PR that set value for timeout from config file

What do you think about setting this configuration inside the [ImportCommand.php:52](https://github.com/matchish/laravel-scout-elasticsearch/blob/master/src/Console/Commands/ImportCommand.php#L52). We already reaching config in this command and we can put as :
        if (config('scout.queue')) {
            $job = (new QueueableJob())->chain([$job]);
            $job->timeout = config('elasticsearch.queue.timeout', 60);
        }

Looks perfect) Thank you @hkulekci

released. Thank you guys) @hkulekci @amcsi


[bug]: .env.sample refers to non-existent GPU_DRIVER

Is there an existing issue for this?

[X] I have searched the existing issues

OS
Linux
GPU
cuda
VRAM
No response
What version did you experience this issue on?
main
What happened?
The InvokeAI Docker directory contains a .env.sample. This file demonstrates using GPU_DRIVER=cuda, which I tried in my .env file. This does not work. There is no container in docker-compose.yml that ends in -cuda, which is what the run.sh script tries to run when GPU_DRIVER=cuda. Looking at the docker-compose file, it is clear that the GPU_DRIVER is supposed to be nvidia. The sample file needs to be updated to be correct.
Screenshots
No response
Additional context
No response
Contact Details
No response

Thanks for catching this. Fixed in the linked PR.


Make android-sdk state work on macOS

Yeah, I'm not sure either, if we could get Test Kitchen setup that would at least help us get some tests for debian running, making sure it also works on macOS is going to take a bit more work since it can't be containerized as easily. It's possible to spin up a macOS vagrant box as long as the host is also macOS though, but I'm not sure the effort for that is worth it.

Yeah, test kitchen would be ideal, but it's just for internal testing at this point so I'm not overly concerned. If it doesn't work, it only affects us and we'll be immediately aware of it so for now I guess it's fine