Cannot read property 'unsubscribe' of undefined!!!

I use authorization component which checks if user is authorize to see protected page where I have and SimpleNotificationsComponent. In case of unauthorized attempt, the authorization component uses Router and navigates to login page. And in that case I got:
Unhandled promise rejection TypeError: Cannot read property 'unsubscribe' of undefined(…)
which leads to NotificationComponent and:
ngOnDestroy() { this.listener.unsubscribe() }
It is probably because of component not being initializes in ngOnInit.

The listener that is trying to unregister doesn't appear to be part of the toaster component. Can you please create a small plunkr that demonstrates this behavior?

I made the Pull Request to fix this problem.
Please accept it. I really like this library but this problem does not allow her to use.

This has been published as 0.3.1-rc.1 via https://github.com/Stabzs/Angular2-Toaster/commit/b88eb5a0db0382cb9bf032d19c5b10f8d1eba7d8.
Please note that while coveralls reported test coverage remained at 100%, branch coverage dropped and had to be corrected in the above commit.  Going forward, if you contribute, please make sure to add appropriate tests and regenerate test coverage to ensure proper coverage of all code paths and branches.


Fixes engine crash by suffix timestamp to cython module name

What do these changes do?

Blocking by Github Runner
Related issue number

Fixes #1140
Fixes #1136

The job abandoned has been fixed, try to rerun the ci.


Merge master to live

Docs Build status updates of commit 2fe83b8:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

Line 136: [Error] Restore nuget package W:\tasd-s\.optemp\dependentPackages\Microsoft.OpenPublishing.CommonPlugins\packages.config in folder:W:\tasd-s\.optemp\packages failed, possibly due to transient errors from nuget servers. Please retry building your content. If the issue still happens, open a ticket in https://SiteHelp and include the build report and the following error message for our team to troubleshoot: Errors in packages.config projects     Unable to find version '1.0.224' of package 'Microsoft.OpenPublishing.CommonPlugins'.       C:\Users\ContainerAdministrator\.nuget\packages\: Package 'Microsoft.OpenPublishing.CommonPlugins.1.0.224' is not found on source 'C:\Users\ContainerAdministrator\.nuget\packages\'.       https://docfx.pkgs.visualstudio.com/docfx/_packaging/docs-build-v2-prod/nuget/v2: Package 'Microsoft.OpenPublishing.CommonPlugins.1.0.224' is not found on source 'https://docfx.pkgs.visualstudio.com/docfx/_packaging/docs-build-v2-prod/nuget/v2'.
Line 136: [Error] Restore nuget package W:\tasd-s\.optemp\dependentPackages\Microsoft.DocAsCode.ECMA2Yaml\packages.config in folder:W:\tasd-s\.optemp\packages failed, possibly due to transient errors from nuget servers. Please retry building your content. If the issue still happens, open a ticket in https://SiteHelp and include the build report and the following error message for our team to troubleshoot: Errors in packages.config projects     Unable to find version '1.0.808' of package 'Microsoft.DocAsCode.ECMA2Yaml'.       C:\Users\ContainerAdministrator\.nuget\packages\: Package 'Microsoft.DocAsCode.ECMA2Yaml.1.0.808' is not found on source 'C:\Users\ContainerAdministrator\.nuget\packages\'.       https://docfx.pkgs.visualstudio.com/docfx/_packaging/docs-build-v2-prod/nuget/v2: Package 'Microsoft.DocAsCode.ECMA2Yaml.1.0.808' is not found on source 'https://docfx.pkgs.visualstudio.com/docfx/_packaging/docs-build-v2-prod/nuget/v2'.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Block Store race condition

The block store will sometimes fall behind.
Because of this code in consensus
                this.chainState.HighestValidatedPoW = this.Tip;
                if (this.Chain.Tip.HashBlock == item.ChainedBlock?.HashBlock)
                    this.FlushAsync().GetAwaiter().GetResult();

                this.signals.SignalBlock(item.Block);

The problem is store uses this condition to store a block
                if (this.StoreTip.Height >= this.ChainState.HighestValidatedPoW?.Height)
                    break;

A very rare race condition can happen where HighestValidatedPoW is updated and store is trying to look for the next block in pending storage collection but signals.SignalBlock did not push the block yet to that collection.
Then the node will try to download the block.

@dangershony after investigating this one for a while I came to a conclusion that StoreBlockPuller and loopsteps related to it are redundant. Basically the only thing that BlockStore does is getting a block from signaler, reorging the repo if needed and saving blocks to the repository. And every time it actually downloads a block (once in a few thousands if we're lucky to catch it)- it's a wrong download and this block was already downloaded by the LookaheadBlockPuller which is the only block puller that does all the downloading (even though in summary it is said that LookaheadBlockPuller is used only for IBD but actually it is used always. And it's used for the light node where we can't use block store).
The only blocks that are being downloaded by StoreBlockPuller are ones that has been already downloaded (2nd downloads caused by that race condition).
So if we remove those unused steps we can get rid of the race condition, occasional high CPU usage (discussed in PR 1125) and make code more simple.
I'll test this thoroughly.

Based on PR #1132 we still need to fix the small CPU boost even on the current code base.


Async privacy request

Purpose
Process privacy requests asynchronously
Changes

Submissions to privacy requests push request processing onto a separate thread.
Separate thread requires a separate db connection
Reworking tests that require privacy responses

Ticket
#43
Additional Comments:
This issue was originally intended to move execution to asynchronous dask workers. Because of the number of changes involved, this ticket now only moves privacy request execution to a fire-and-forget model. Asynchronous dask execution should be the logical next step to this work.
Async:

PrivacyRequest is now run using a coroutine (python async) and fire-and-forget semantics. This requires some changes to the way db sessions are handled, since their existing implementation is not set to autocommit.
This is because
a) we need to communicate with the db in the co-routine using a separate db session
b) you cannot work with a sqlalchemy object in a session that's different than the one that it is part of
c) trying to access an object that's in an uncommited state from the db results in objects that are out of sync.

There's a separate PR open to have our sessions set to autocommit=True

To minimize coupling the PrivacyRequest runner run method only takes the privacy request id and reconstructs everything it needs directly from a dedicated db connection.

Tests:

tests on fire-and-forget tasks should use a returned future rather than polling the db for a completion condition. This means that :
Tests on privacy request responses that require an actual running of the request have been moved from the dntpoint test to a test that uses the future directly and waits for its completion.
There was some discussion of removing finished_processing_at from the PrivacyRequestResponse object in BulkPostPrivacyRequest but left it in, since it's defined as an optional and it's still useful for completed responses.
There are additionaly a number of changes in our tests to remove their hardcoded dependency on our docker setup, so they can be run against local resources. These are all driven by the .toml files, which can be customized.

You have to put "Closes #43" in the description, remember 😉

^ I can't really tell that this helped much @stevenbenjamin, pulled this latest commit down, and I'm seeing the same thing. it's too easy to create too many connections in local testing.

I agree this seems slightly improved, but this doesn't feel mergeable to me.  When creating three privacy requests at a time, the number of connections are quickly exceeded through a series of sequential requests:  20 -> 31 -> 55 -> 75 -> maxed out.  can you weigh in @seanpreston?

@stevenbenjamin can you also get this up to date with main, there's a new migration on there, will make it easier to switch back and forth

Re: increasing the size of the pool, I'm a bit puzzled since I was not able to generate any kind of load like this. I think we should have the pool size as a configurable option, possibly along with some other pool parameters

@stevenbenjamin did you try creating more than one privacy request at a time? I usually do 3-6.

Thanks for the reviews @pattisdr — I've created a follow up issue here https://github.com/ethyca/fidesops/issues/115 to capture the further work required to fix the connection limitation in the application DB.


Too few results from intersect in version v2.26.0-125-g52db654

Hi there,
This might pertain to both bedtools and pybedtools so I cc @daler.
I'm trying to select variants in a vcf file that are covered by reads in a bam file. The files I need to process in the end are quite large and I have split them by chromosome so I can use multiprocessing.
Using some test data I have encountered the following error message:
mce@:~/mini-example$ bedtools -version
bedtools v2.26.0
mce@laptop:~/mini-example$ python
Python 2.7.13 |Continuum Analytics, Inc.| (default, Dec 20 2016, 23:09:15)
>>> import pybedtools
>>> variants = pybedtools.BedTool('chromosome_vcf_files/variants_chr1.vcf.gz')
>>> reads = pybedtools.BedTool('chromosome_bam_files/reads_chr1.bam')
>>> covered_variants = variants.intersect(reads, split=True, u=True)
>>> for variant in covered_variants: print variant
...
chr1	216	rs01	T	G	.	.	RS=1;RPOS=216

chr1	576	rs02	C	A	.	.	RS=2;RPOS=576

chr1	836	rs04	A	C	.	.	RS=4;RPOS=836

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "pybedtools/cbedtools.pyx", line 787, in pybedtools.cbedtools.IntervalIterator.__next__ (pybedtools/cbedtools.cxx:11123)
  File "pybedtools/cbedtools.pyx", line 652, in pybedtools.cbedtools.create_interval_from_list (pybedtools/cbedtools.cxx:9208)
IndexError: list index out of range

Note that the intersect command itself apparently works fine; only when iterating over the results the error is raised.
Interestingly, with the test data, it appears this error can be bypassed by calling the stream=True option in pybedtools.
>>> covered_variants = variants.intersect(reads, split=True, u=True, stream=True)
>>> for variant in covered_variants: print variant
...
chr1	216	rs01	T	G	.	.	RS=1;RPOS=216

chr1	576	rs02	C	A	.	.	RS=2;RPOS=576

chr1	836	rs04	A	C	.	.	RS=4;RPOS=836

Using the sorted option (as I would like later on with the real data) does not make any difference.
>>> covered_variants = variants.intersect(reads, split=True, u=True, sorted=True)
>>> for variant in covered_variants: print variant
...
chr1	216	rs01	T	G	.	.	RS=1;RPOS=216

chr1	576	rs02	C	A	.	.	RS=2;RPOS=576

chr1	836	rs04	A	C	.	.	RS=4;RPOS=836

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "pybedtools/cbedtools.pyx", line 787, in pybedtools.cbedtools.IntervalIterator.__next__ (pybedtools/cbedtools.cxx:11123)
  File "pybedtools/cbedtools.pyx", line 652, in pybedtools.cbedtools.create_interval_from_list (pybedtools/cbedtools.cxx:9208)
IndexError: list index out of range

As the error message read similar to the one reported here https://github.com/daler/pybedtools/issues/207 (although I cannot see how groupby is involved in my case) I tried upgrading to the latetst developmental version of bedtools. Indeed, that solved the problem, and, as far as my test example is concerned, everything worked fine now.
mce@laptop:~/mini-example$ bedtools -version
bedtools v2.26.0-125-g52db654
mce@laptop:~/mini-example$ python
Python 2.7.13 |Continuum Analytics, Inc.| (default, Dec 20 2016, 23:09:15)
>>> import pybedtools
>>> variants = pybedtools.BedTool('chromosome_vcf_files/variants_chr1.vcf.gz')
>>> reads = pybedtools.BedTool('chromosome_bam_files/reads_chr1.bam')
>>> covered_variants = variants.intersect(reads, split=True, u=True)
>>> for variant in covered_variants: print variant
...
chr1	216	rs01	T	G	.	.	RS=1;RPOS=216

chr1	576	rs02	C	A	.	.	RS=2;RPOS=576

chr1	836	rs04	A	C	.	.	RS=4;RPOS=836

>>> covered_variants = variants.intersect(reads, split=True, u=True, sorted=True)
>>> for variant in covered_variants: print variant
...
chr1	216	rs01	T	G	.	.	RS=1;RPOS=216

chr1	576	rs02	C	A	.	.	RS=2;RPOS=576

chr1	836	rs04	A	C	.	.	RS=4;RPOS=836

However, when I used the above developmental version on my real data, intersect gave me far less results than expected.
Using v2.26.0:
[mce@znx-cn03 test]$ bedtools -version
bedtools v2.26.0
[mce@znx-cn03 test]$ bedtools intersect -a chromosome_vcf_files/variants_1.vcf.gz -b chromosome_bam_files/reads_1.bam -u -split -sorted | wc -l
723917

Upon upgrading to v2.26.0-125-g52db654:
[mce@znx-cn03 test]$ bedtools -version
bedtools v2.26.0-125-g52db654
[mce@znx-cn03 test]$ bedtools intersect -a chromosome_vcf_files/variants_1.vcf.gz -b chromosome_bam_files/reads_1.bam -u -split -sorted | wc -l
0

For some of the chromosome subfiles some variants are reported with v2.26.0-125-g52db654 but the number is always far lower than what I expect. When I look at files with multiple chromosomes, typically all the hits come from one or a few chromosomes with no overlapping reads/variants reported for the others.
As a work around I tried to use v2.26.0 with the stream=True option I mentioned above, however, for these very large files this does not work but I get the same error as before:
[mce@znx-cn03 test]$ python test.py > tmp.txt
 Traceback (most recent call last):
  File "basic.py", line 880, in <module>
    heterozygous_snps_by_chromosome = pool.map(get_heterozygous_snps_as_bedfile, reads_SQ_lines)
  File "/home/mengeholm/anaconda2/lib/python2.7/multiprocessing/pool.py", line 251, in map
    return self.map_async(func, iterable, chunksize).get()
  File "/home/mengeholm/anaconda2/lib/python2.7/multiprocessing/pool.py", line 567, in get
    raise self._value
IndexError: list index out of range

So, in summary, I suspect there is a problem with intersect in version v2.26.0-125-g52db654 (which is not present in v2.26.0). I am not sure what the IndexError means when iterating over the results of intersect in v2.26.0, specifically, if that is due to a problem with bedtools or pybedtools and if it can be fixed on the pybedtools side maybe.
I would be really grateful if (one of) you could have a look.
Many thanks in advance,
mce

@mce1 the index error is certainly from pybedtools but the error could be caused from a malformed line from input or intersect output, or some other issue with pybedtools.  Would you please provide example files that reproduce these errors?

@daler please find the two files attached
variants_chr1.vcf.gz
reads_chr1.sam.txt
(The bam file I have converted to .sam.txt because otherwise it would not upload.)
Thanks, mce

@daler can you reproduce the IndexError with the example files above?

@mce1 sorry was on vacation and just got back. Here is my setup, I'm unable to reproduce the IndexError. I do see 2.26 outputting zero lines, though it appears you saw dev outputting zero lines. Do you see it using the following setup?
Contents of ex.py:
import pybedtools
import subprocess as sp
print('pbt:', pybedtools.__version__)
print(
    sp.check_output(['which', 'bedtools'], universal_newlines=True).strip(),
    '(',
    sp.check_output(['bedtools', '--version'], universal_newlines=True).strip(),
    ') says:'
)
a = pybedtools.BedTool('variants_chr1.vcf.gz')
b = pybedtools.BedTool('reads_chr1.bam')
c = a.intersect(b, u=True, split=True, sorted=True)
for i in c:
    print(i, end='')
print()

Contents of test.sh:
#!/bin/bash

HERE=$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd )

if [[ ! -e bedtools-v2.26/bin/bedtools ]]; then
    git clone https://github.com/arq5x/bedtools2.git bedtools-v2.26
    cd bedtools-v2.26
    git checkout v2.26.0
    make
fi

if [[ ! -e bedtools-dev/bin/bedtools ]]; then
    git clone https://github.com/arq5x/bedtools2.git bedtools-dev
    cd bedtools-dev
    make
fi

if [[ ! -e reads_chr1.bam ]]; then
    wget https://github.com/arq5x/bedtools2/files/1191844/variants_chr1.vcf.gz
    wget -O- https://github.com/arq5x/bedtools2/files/1191857/reads_chr1.sam.txt > reads_chr1.sam
    samtools view -Sb reads_chr1.sam > reads_chr1.bam
fi

PATH="$HERE/bedtools-dev/bin:$PATH" python ex.py
PATH="$HERE/bedtools-v2.26/bin:$PATH" python ex.py

Output:
pbt: 0.7.10
/tmp/bedtools-dev/bin/bedtools ( bedtools v2.26.0-125-g52db6549 ) says:
chr1    216     rs01    T       G       .       .       RS=1;RPOS=216
chr1    576     rs02    C       A       .       .       RS=2;RPOS=576
chr1    836     rs04    A       C       .       .       RS=4;RPOS=836

pbt: 0.7.10
/tmp/bedtools-v2.26/bin/bedtools ( bedtools v2.26.0 ) says:

@daler: thanks a lot for the reply!
I can reproduce this: using bedtools 2.26 I now see an empty list instead of the IndexError. I think this is most likely due to the different versions of python, i.e. python3 now vs. python2 before. Can you reproduce the IndexError with python 2.7?
The bedtools dev version gives the same results as before, but as I said before there seems to be another problem with this version which does not become apparent with the test files here but only with larger data sets where intersect produces far fewer results than in version 2.26. This has nothing to do with pybedtools of course.

@mce1 I've updated the testing scripts to check py2 and py3, and also run the same commands outside the context of pybedtools, see this gist:
https://gist.github.com/daler/9623fad6b3cfa1c599591b0538e9f4dd
I was able to reproduce the IndexError in py2 you originally reported, thanks for pointing that out. It appears to be a difference in behavior between py2 and py3 on malformed files.
As you can see from the files created, the v2.26 version has binary information at the end. Looks like it's related to https://github.com/arq5x/bedtools2/issues/497. As far as line counts, it is possible that when you checked the line count on your real data on v2.26 results, wc -l was picking up newline characters in the binary section, and so reported an inaccurate value. You could try grep "^chr" output.vcf | wc -l (assuming all chroms start with "chr") for a potentially more accurate count in that bedtools version.
I'm afraid there's not much I can do on the pybedtools side, I need to pass it off to @arq5x.

@daler and @arq5x,
thanks a lot for your help with this so far. Below, I have attached another example which yields unexpected results with intersect in the dev version, this time when trying to get variants from a vcf that overlap transcripts in a bed file. I'm not entirely sure that this reflects the same problem I had originally (when trying to extract variants that overlap reads in bam format) but maybe it can be useful for debugging.
contents of test.py:
from __future__ import print_function
import pybedtools

import subprocess as sp
print('pbt:', pybedtools.__version__)
pybedtools.debug_mode(True)
print(
    sp.check_output(['which', 'bedtools'], universal_newlines=True).strip(),
    '(',
    sp.check_output(['bedtools', '--version'], universal_newlines=True).strip(),
    ') says:'
)

def gene_transcript_names_dictionary_from_table(filename):
    table = open(filename, 'r')
    dictionary = {}
    for line in table:
        if line[0] != "#":
            if line.split('\t')[5].rstrip() + '_' + line.split('\t')[1].rstrip() in dictionary:
                dictionary[line.split('\t')[5].rstrip() + '_' + line.split('\t')[1].rstrip()].append(line.split('\t')[0])
            else:
                dictionary[line.split('\t')[5].rstrip() + '_' + line.split('\t')[1].rstrip()] = [line.split('\t')[0]]
    return dictionary

def rename_chromosomes(feature):
    if "chr" in feature.chrom: feature.chrom = feature.chrom.replace("chr", "")
    else: feature.chrom = "chr" + feature.chrom
    return feature

def get_BedTool_transcripts_by_gene(gene):
    gene_transcript_names = gene_transcript_names_dictionary[gene]
    return [Interval for Interval in BedTool_transcripts_bed_all if Interval.name in gene_transcript_names]

def get_BedTool_variants_by_gene(gene):
    chromosome = gene.split('_chr')[1]
    chromosome_variants = pybedtools.BedTool('chr%s.vcf.gz' % chromosome)
    gene_variants_vcf = chromosome_variants.intersect(gene_BedTool_transcripts_bed_dictionary[gene], split=True, u=True)
    print(gene + ':')
    print(len(gene_variants_vcf))
    for variant in gene_variants_vcf:
        print(variant)
    return

gene_transcript_names_dictionary = gene_transcript_names_dictionary_from_table('app_snca.gene_table.txt')

all_genes = gene_transcript_names_dictionary.keys()

BedTool_transcripts_bed_all = pybedtools.BedTool('app_snca.bed.gz').each(rename_chromosomes).saveas()

gene_BedTool_transcripts_bed_dictionary = dict(zip(all_genes,
                                                   map(get_BedTool_transcripts_by_gene, all_genes)))

gene_BedTool_variants_dictionary = dict(zip(all_genes,
                                                   map(get_BedTool_variants_by_gene, all_genes)))

contents of test.sh:
#!/bin/bash

HERE=$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd )

# build python 2 and python 3 environments
if [[ ! -e py2env ]]; then
    conda create -y --prefix py2env pybedtools python=2 -c conda-forge -c bioconda
fi
if [[ ! -e py3env ]]; then
    conda create -y --prefix py3env pybedtools python=3 -c conda-forge -c bioconda
fi

if [[ ! -e bedtools-v2.26/bin/bedtools ]]; then
    git clone https://github.com/arq5x/bedtools2.git bedtools-v2.26
    (cd bedtools-v2.26 && git checkout v2.26.0 && make)
fi

if [[ ! -e bedtools-dev/bin/bedtools ]]; then
    git clone https://github.com/arq5x/bedtools2.git bedtools-dev
    (cd bedtools-dev && make)
fi

for py in 2 3; do
    source activate py${py}env
    for bt in dev v2.26; do
        echo
        echo "==========================="
        echo "Python = py$py"
        echo "BEDTools = $bt"
        PATH="$HERE/bedtools-$bt/bin:$PATH" python test.py
    done
    source deactivate
done

Test files:
app_snca.gene_table.txt
app_snca.bed.gz
chr6.vcf.gz
chr16.vcf.gz
From what I can see, python2 and python3 give the same results here. In each case, v26.0 yields 17 exonic variants for one of the two genes (app on chromosome 16) and 15 for the other (snca on chromosome 6) as expected. The dev version, on the other hand, returns all the variants in the test vcf (2706) for the first gene but not a single one for the second gene.
Would be great if you could have a look at this as well as the original problem described above.
Many thanks in advance,
mce

I will try to dig into this today!

Thanks for reporting this - the bug will require some substantial refactoring and testing and may take awhile to resolve. My apologies.

Thanks to help from @nkindlon this is now fixed in master.


Do we need a stacks command?

$ cf stacks --help
NAME:
   stacks - List all stacks (a stack is a pre-built file system, including an operating system, that can run apps)

USAGE:
   cf stacks

CNCF buildpacks combine buildpacks and stacks a bit more than CF does, however we could still list the associated stacks that the builders have.
This is a bit forward looking for https://github.com/poy/kf/issues/132#issuecomment-501802937

I think we should have a rough concept of stacks. Probably by changing the BuilderImage in Space to be an array of the following:
// SpaceSpecBuildpackBuild gets a field "default" to resolve a specific stack

type Stack struct {
  Image   string `json:"image"` // image of the builder
  Name    string `json:"name"`  // short name e.g. cflinuxfs
}

Then a stack is a set of buildpacks paired with a base image. This would solve a few different problems for us:

Supporting multiple base-images (e.g. Ubuntu, alpine)
Trying out different types of buildpacks (Heroku, CF, Cloud Native)
(future looking) supporting FaaS workloads that need highly paired down environments.
(future looking) supporting rollouts of new buildpacks, customers could have cflinuxfs and cflinuxfs-next for rolling releases.

WDYT?

per discussion with @josephlewis42:
We are going to implement stacks within kf to emulate stacks in cf. Specifically, a developer can specify the stack (and buildpack) while pushing.
If the stack+buildpack combo don't exist together, we'll return an error (much like if a developer specified a non-existent buildpack).


Add windows build

Checklist

[X] Used a personal fork of the feedstock to propose changes
[X] Bumped the build number (if the version is unchanged)
[X] Reset the build number to 0 (if the version changed)
[X] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[X] Ensured the license file is being packaged.

Second try to add windows build. Updated to newest changes and version. Supersedes #12

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

weird. does someone have to allow CI? Also smithy is behaving very faulty. It only picks up the windows CI files once I added them manually.

No it's just an issue with Azure: https://conda-forge.org/status/

Oh! Haha thanks. I'll try again tomorrow.

@tkralphs so, it kind of works now, BUT I had to disable tests (Note: they were disabled before already, probably because of this)
What makes me a bit worried is that before I disabled the tests, on Windows, it was complaining about missing symbols (looked like BLAS/LAPACK symbols). I am therefore wondering if for linking together a test executable I would need to specify those libs explicitly on the make command line.

Thanks for pushing on this again. I'll take a look soon.

Maybe someone can just test the windows build artifacts manually. I assume they will be working when linking correctly to all dependencies.

@tkralphs Have you already had the chance to look at it?
To be honest, it's not worse than before. We have an explicit Fortran compiler now for Unix, and a building windows build.
Tests were probably never working.


Update TSM, remove old version, add config

This ended up being a huge one. I removed the old TSM stuff. I also cleaned up the locking logic on writes since I needed to know the last write time to the WAL for the WAL compact logic to get updated. I also added the configuration options I could think of and updated TSM to use those config options instead of constants. Finally, the compaction planning logic has been updated to force full compactions after cold and to avoid planning on shards that have been fully compacted.
cc @jwilder, @otoolep

remove rolloverTSMFileSize constant that is no longer used
remove the maxGenerationFileCount since it is no longer a limitation that's necessary with the new compaction scheme. We no longer read WAL segments as part of the compaction so memory is only used as we read in each individual key
remove minFileCount and switch to a user configurable variable
remove the mutex from WALSegmentWriter. There's never more than one open in the WAL at one time and it's not exported through any function so the lock on the WAL should be used. This simplified keeping track of the last write time and removed a bunch of unnecessary locks.
update WALSegmentWriter.Write to take the compressed bytes so that encoding and compression can occur before the call to write (while we don't hold the WAL lock)
remove a bunch of unnecessary locking in WAL.writeToLog
Add check for TSM file magic number and vesion
Remove old tsm, log, and unused cursor code
Remove references to tsm1dev everywhere except in the inspector
Clean up config options for compaction and snapshotting
Remove old TSM configuration options
Update the config.sample.toml with TSM options
Update WAL compact to force if it has been cold for writes for a configurable period of time (1h by default)
Update Plan to do a full compaction if cold for writes
Remove MaxFileSize as a config variable from Compactor. Should be a set constant
Update Plan to keep track of if the last check was fully compacted so we can skip future planning calls
Update compact min file count to 3 so that compactions run more frequently
Change all TSM file sizes to uint32 (was causing build errors on 32 bit containers when using a max size in 2GB)
Change all data files to have tsm extension
Update cache loader to delete entries from cache
Add cache.Delete()
Update delete to look at keys in the Cache in addition to the FileStore
Update cache compaction to never happen if the cache is empty
Move TSM file extension to be a constant

"Change all data files to have tsm extension"
Yeah, it would be great if this string was defined explicitly somewhere IMHO.

Two small items, but otherwise looks good.

+1 from me, as long a points from @jwilder have been addressed to his satisfaction.

:+1:

Merge it! :-D


Explanation of algorithm?

Hi, thanks for this tool :-D
It looks like a black box to me. Are there any methods to print the model summary?

And output the model config?

Sorry for the shameless plug ... however, as I see that this repo is unmaintained maybe I will help someone.
I'm working on an open-source python AutoML package mljar-supervised. This package is producing markdown reports for each model. Additionally, it computes model explanations: coefficients for linear models, decision tree visualization, feature importance and SHAP plots. Take a look at AutoML report: https://github.com/mljar/mljar-examples/tree/master/House_price_regression/AutoML_1#automl-leaderboard and example model summary: https://github.com/mljar/mljar-examples/blob/master/House_price_regression/AutoML_1/model_37/README.md


Breaking Aspect Ratio?

Breaking Aspect Ratio?
I know mathematicians love these nice mathematical solutions that demonstrate their geniuses LOL
BUT as a front end developer, it just limits what you can do
Can we have an option to break aspect ratio, PLEASE !! PRETTY please...

Jssor Slider will always keep aspect ratio while scaling.


Portfolio: deploy application

This is a project that involves deploying my portfolio webpage with Github pages
This project involved making my portfolio responsive and deploying it online for users to access
NB:- The page cannot be found online because I have not merged the branches

The page cannot be deployed because i have not merged the branch but


Catch up to last API changes

Currently (I'm using version 56.1) PDF generation doesn't work because Pdf.php sets the default option 'format' => 'pdf' which has been deprecated as a cli parameter, see weasyprint changelog from a year ago:
https://doc.courtbouillon.org/weasyprint/stable/changelog.html#version-53-0
Version 53.0
API changes:
--format and --resolution options have been deprecated, PDF is the only output format supported.

This is first of all documentation of the problem, I might try to create a fix myself.
To improve searchability of this issue, here is the error message I first encountered:
usage: weasyprint [-h] [--version] [-i] [-e ENCODING] [-s STYLESHEET] [-m MEDIA_TYPE] [-u BASE_URL] [-a ATTACHMENT] [-p] [-O {images,fonts,all,none}] [-v] [-d] [-q] input output weasyprint: error: unrecognized arguments: --format

Ah sorry, I guess this can be closed as the ticket exists here: https://github.com/pontedilana/php-weasyprint/issues/2

Would it be possible to publish a v1.0.1 that uses pontedilana/php-weasyprint v0.10.1 ?
I'm currently stuck on Symfony 4.

Hi @BitScout I can look at it and prepare a release next weekend.

@BitScout I've added a release tag on commit https://github.com/pontedilana/WeasyPrintBundle/commit/d14a64880b0663ced42ecdb97c82ba2419af2895, this should be sufficient to work with Symfony 4.4 and php-weasyprint v0.10.1


[FLINK-20396][checkpointing] Add a 'subtaskReset()' method to the OperatorCoordinator.

This is based upon PR #14255, which represent the first of the three commits here.
What is the purpose of the change
This extends the OperatorCoordinator with a subtaskReset() method.

The subtaskReset() method is the notification of a local failover (pipelined region) where the OperatorCoordinator is not reset as a whole, but only resets the subtask-dependent state.
That method complements the method resetToCheckpoint() which is called on a global-failover and resets the entire state of the OperatorCoordinator.

The behavior is transparent across batch and for streaming execution:

All notifications come on all global- and local failover, both in batch and streaming
For streaming execution, the methods are invoked with the ID of the restored checkpoint.
For streaming execution, if there is not yet any completed checkpoint, then the methods are invoked with a checkpoint ID of -1. That represents a "virtual empty checkpoint" that marks the "beginning of the execution". -1 is before any other checkpoint ID, so it fits transparently into the contract of monotonous checkpoint IDs and "later-checkpoints-subsume-earlier-checkpoints".
Batch recovery behaves exactly like a streaming recovery before the first checkpoint: There is only ever the virtual empty checkpoint that represents the beginning of the execution, no additional checkpoint is taken. All recoveries refer to that empty checkpoint with no state. That way batch is strictly a special case of streaming.

This logic will allow us to make the consistency of the SplitEnumerator for sources easier (future PR).
Instead of reacting to failure notifications, we react to restore notifications.

Restore notifications are well ordered with checkpoints, we eliminate the races of FLINK-20290
Restore notifications generalize well across local and global failovers.
All restores are relative to checkpoints (including the virtual empty checkpoint at the beginning). That makes split assignment tracking transparent in the sources: When a task is restored, all splits that came after the restored checkpoint have to be re-assigned. In batch, the restored checkpoint is the initial empty checkpoint, so all splits every assigned to the task have to be re-assigned.

Brief change log

Commit 9f760fa improves the resetToCheckpoint() method by passing in the ID of the restored checkpoint.
Commit cf30b0e adds the subtaskReset() method, the scheduler code to call that method, and the tests.

Verifying this change
This PR adds a series of test scenarios. The class OperatorCoordinatorSchedulerTest covers the full contract of the OperatorCoordinator calls and notifications from the scheduler.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: yes
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

I reported the e2e failure in https://issues.apache.org/jira/browse/FLINK-20410.

Thank you for the fact review. Will address the formatting issue and merge this...


<EMAIL_ADDRESS>SyntaxError

Version
4.0.0-rc.3
Reproduction link
https://github.com/ManfredHu/vue-jest-nightwatch
Environment info
Chrome Version: 版本 76.0.3809.132（正式版本） （64 位）
System: macOS Mojave 10.14.5

Steps to reproduce
The project created by vue-cli3 executes npm run test: e2e cannot run.
npm i<EMAIL_ADDRESS>--save-dev
npm i chromedriver geckodriver<EMAIL_ADDRESS>--save-dev
use<EMAIL_ADDRESS>to replace vue-cli3 created project e2e folder by template.
What is expected?
no error
What is actually happening?
[Test With Pageobjects] Test Suite
Running:  e2e tests using page objects
✔ Element <#app> was visible after 22 milliseconds.
✔ Testing if element  has count: 1  - 7 ms.
✔ Expected element <Section [name=app],Element [name=@welcome]> to be visible - condition was met in 27ms
✔ Expected element <Section [name=app],Element [name=@headline]> text to match: "/^Welcome to Your Vue.js (.*)App$/" - condition was met in 29ms
OK. 4 assertions passed. (425ms)
Running:  verify if string "e2e-nightwatch" is within the cli plugin links
✔ Expected element <Section [name=app],Section [name=welcome],Element [name=@cliPluginLinks[0]]> text to contain: "e2e-nightwatch" - condition was met in 42ms
OK. 1 assertions passed. (1.109s)

TEST FAILURE: 1 error during execution 0 assertions failed, 5 passed. 2.57s
SyntaxError: missing ) after argument list
.assert.containsText('h1', 'Welcome to Your Vue.js <%- hasTS ? '  TypeScript ' : '' %>App')
^^^^^^^^^^
SyntaxError: missing ) after argument list
at createScript (vm.js:80:10)
at Object.runInThisContext (vm.js:139:10)
at Module._compile (module.js:617:28)
at Object.Module._extensions..js (module.js:664:10)
at Module.load (module.js:566:32)
at tryModuleLoad (module.js:506:12)
at Function.Module._load (module.js:498:3)
at Module.require (module.js:597:17)
at require (internal/module.js:11:18)
ERROR  Error: Command failed: /Users/pphu/Documents/vue-jest-nightwatch/node_modules/nightwatch/bin/nightwatch --config<EMAIL_ADDRESS>--env chrome
Error: Command failed: /Users/pphu/Documents/vue-jest-nightwatch/node_modules/nightwatch/bin/nightwatch --config<EMAIL_ADDRESS>--env chrome
at makeError (/Users/pphu/Documents/vue-jest-nightwatch/node_modules/execa/index.js:174:9)
at Promise.all.then.arr (/Users/pphu/Documents/vue-jest-nightwatch/node_modules/execa/index.js:278:16)
at
at process._tickCallback (internal/process/next_tick.js:189:7)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>test:e2e: vue-cli-service test:e2e
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>test:e2e script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      /Users/pphu/.npm/_logs/2019-09-11T06_45_42_392Z-debug.log

use<EMAIL_ADDRESS>to replace vue-cli3 created project e2e folder by template.

Users are not supposed to directly copy template folders to the project.
Run vue invoke e2e-nightwatch instead.


feat: replace wagmi's useClient with useMetaStreetQueryClient

Current dependencies on/for this PR:

main

PR #170

PR #171

PR #172

PR #173

PR #175   👈

This comment was auto-generated by Graphite.

Graphite Merge Job
Current status: ⏳ Queued to merge
This pull request is currently queued to be merged as part of a stack.
This comment was auto-generated by Graphite.

Job Reference: Il3EKAhxbQsIJwI9q34e


NoUselessElseFixer - bug: fixer brace logic

Hi there, great project!
I recently ran the fixer on a large code base, and wound up with some subtle bugs. I tracked it down to this original code from my project (I've simplified it to the point where it's nonsensical, but still demonstrates the bug):
function create_joins($value)
{
	$ret = '';

	if (stripos($value,'JOIN ') === false)
	{
		if (count($value))
		{
			$ret = "foo";
		}
		else
			throw new Exception("Value should not be empty");
	}
	else
		$ret .= $value;

	return $ret;
}

After running the fixer, I get:
<?php

function create_joins($value)
{
    $ret = '';

    if (stripos($value, 'JOIN ') === false) {
        if (count($value)) {
            $ret = 'foo';
        } else {
            throw new Exception('Value should not be empty');
        }
    }

    $ret .= $value;

    return $ret;
}

You can see that the logic has changed; $ret .= $value now executes on every call of the create_joins function, whereas before it only executed when the if condition failed.
This is with:
PHP: 7.0.19
PHP-CS-Fixer: 2.3.1

And with the following .php_cs.dist:
$finder = new PhpCsFixer\Finder();
$config = new PhpCsFixer\Config('json-schema', 'json-schema style guide');
$finder->in(__DIR__);
/* Based on ^2.1 of php-cs-fixer */
$config
    ->setRules(array(
        // default
        '@PSR2' => true,
        '@Symfony' => true,
        // additionally
        'array_syntax' => array('syntax' => 'short'),
        'binary_operator_spaces' => false,
        'concat_space' => array('spacing' => 'one'),
        'no_unused_imports' => false,
        'no_useless_else' => true,
        'no_useless_return' => true,
        'ordered_imports' => true,
        'phpdoc_no_package' => false,
        'phpdoc_order' => true,
        'phpdoc_summary' => false,
        'pre_increment' => false,
        'trailing_comma_in_multiline_array' => false,
        'simplified_null_return' => false,
    ))
    ->setFinder($finder)
;
return $config;

Please advise.

Hi, thanks for reporting!
By looking at the samples and digging in my mind I'm pretty sure this is done by the NoUselessElseFixer and it is a bug. Till the bug is fixed I advice to switch off the no_useless_else` rule in your configuration.

Thanks, will do. By the way, I noticed that if I replace the Throw line with anything else it seems to work, so perhaps that's a clue.

very nice case to be fixed

@keradus I think that the bug probably is for this line since the else is cleared always when the do/while is finished, and in this case the last else don't should be cleared since there are a meaningful token available like the line  $ret .= $value;


Driver for the Wolfson WM8904 audio codec

This PR brings a driver for the Wolfson/Cirrus WM8904 audio codec, based on original work of @hakehuang (recently closed PR: https://github.com/zephyrproject-rtos/zephyr/pull/66609#event-13467395343). The driver was developed and tested on the mimxrt595_evk and mimxrt685_evk boards. Following improvements were made to the driver itself:

decoupled from hal_nxp
input routing (multiplexing)
input mute and volume control
output mute and volume control
omission of dead code

Following modifications were needed to implement it:

modified clock_control_mcux_syscon.c to provide frequency for the MCLK signal
extended the codec API to handle input and output routing, extra sample rates and configuration for stream directions (playback, capture, duplex)

I added @dbaluta and @iuliana-prodan as reviewers. I didn't realize this went over the 15 reviewers limit. When this happens, Github normally warns about this limit and blocks the addition. But that time Github dropped random other reviewers instead! Temporary glitch? You have been warned.
I tried to cleanup that mess, hopefully the list is back to where it was.

CI board testing PASS.

@VitekST  looks good to merge! We tested it on imxrt 685 and works fine.

@anangl Is everything up to your standards?


Query data not fetched initially when refetchOnMount is false in 3.8.3

Describe the bug
This seems to still be an issue: https://github.com/tannerlinsley/react-query/issues/896
As the title suggests, when refetchOnMount is false, data is not initially fetched.
To Reproduce

Open this codesandbox: https://codesandbox.io/s/reverent-cohen-97pgh?file=/src/App.js
You can see that the username is not showing up. The service call is not fired.
If you now refocus the window it fetches the data as expected.

Expected behavior
the query should fetch data when on first execution.
Browser
OS: macOS Catalina 10.15.7
Browser: Chrome Version 88.0.4324.192 (Official Build) (x86_64)

From the looks of it, you're still using v2.7.0 in your codesandbox example. Maybe try updating to the latest v2 (if you don't want a breaking change), or better yet, bump up to v3?

Upgrading the codesandbox to 2.26.4, which is the latest 2.x version, solves the issue:
https://codesandbox.io/s/friendly-kapitsa-k6kd8?file=/src/App.js


issue #262: add a test and fix the issue in the HTTP server with empt…

This PR addresses GitHub issue: #262
Briefly describe the changes proposed in this PR:

Use of String.lastIndexOf to find the index of the last slash "/" in the request path, in order to determine where the prefix starts

As the original code, the proposed modification accepts URLs that are not strictly conforming to the specified request format: e.g. any number of intermediate components are silently ignored.
Make sure you've followed the Contributor Guidelines. In particular (please tick to indicate you've taken care of it):

[x] RDF4J code formatting has been applied
[x] tests are included
[x] all tests succeed (in fact, because of a problem with locale-dependent tests, I was not able to run all tests, but I've executed the tests for the affected module)

Approved from me. @manuelfiorelli I will merge this PR. If you want to do further improvements wrt CSV parsing you can open a new PR for that.

@jeenbroekstra I don't know if I will make a PR for the "icing". However, I am interested to known what should I do. Should I create a new issue and branch?

No, that wouldn't be necessary. Essentially it's still part of the same issue, so even after this PR is merged, it's fine to keep working on the existing issue, and using the same branch. You will just need to put up a new PR for those further changes.
Btw I merged your PR because I wanted to make sure your original fix was included for the 2.0 release. If you work on the csv parsing bit like @ansell proposed there's always a chance your further fixes break something. By merging now I made sure that we can go ahead with release with the fix included, regardless of whether you improve the csv parsing.


[Question]: Segmentation fault.

There is a version conflict between SentenceTransformer and infinity-sdk 0.3.0.dev5, resulting in a Segmentation fault.

Would you please provide more information? OS and distribution, SentenceTransformer version?

I cannot reproduce this error.
I use Python 3.12.4
both
import sentence_transformers
import infinity

and
import infinity
import sentence_transformers

gives no error.

Also got the same error, that is "fixed" when changing the import order.
Working:
import infinity
from sentence_transformers import SentenceTransformer

Segmentation Fault:
from sentence_transformers import SentenceTransformer
import infinity

Ubuntu 22.04.3 LTS
Python 3.10.14
sentence-transformers 3.0.1
infinity-sdk 0.3.0.dev8

It might be caused by the lower version of python. Two solutions:

Upgrade python to python 3.12 or later.
Infinity will separate the embedded version from the standalone server. At that time, there are two python sdks, and if you choose python sdks for standalone server, it will not have such problems because it does not have dependencies on libc

In my machine (Ubuntu 24.04, Conda Python 3.12.4), Segmentation Fault will happen when the libstdcxx version is 12.4.0
If you use conda, you can try the following command:
conda install -c conda-forge libstdcxx-ng

It will install libstdcxx 14.1.0 in the conda environment, and solve the SIGSEGV error.


Please add missing test db SQL

It's pretty hard to hack and run the tests with db-sync, when I don't have the test data, making it difficult to contribute.
Could you please commit a SQL dump with the dbsynctest tables?
I'm trying to fix an issue with a horrible db I have inherit where a lot of DATE fields has 0000-00-00 values, and I get the following error: SQLSTATE[22007]: Invalid datetime format: 1292 Incorrect date value: '0000-00-00' for column 'hascancelleddate' at row 1. I would like to fix it and create a PR but still know that the tests will pass.
 I've just begun to poke around the source but line 80 in sync is my starting point.

The table schemas are in here: https://github.com/mrjgreen/db-sync/blob/v3/tests/integration/FullSyncTest.php
Its not ideal - I'd like to move this out to an SQL file and utilise docker to help with the testing.
As for your issue, it seems like your target table most likely has a different sql-mode defined in the my.cnf, or is not defined and using the default sql-mode. Docs
Specifically NO_ZERO_IN_DATE and NO_ZERO_DATE modes. There are implicitly included in the STRICT_TRANS_TABLES or STRICT_ALL_TABLES mode, and in the TRADITIONAL mode.
So most likely, your my.cnf on the target is missing  the sql-mode setting or has one of these:
sql-mode="NO_ZERO_IN_DATE,NO_ZERO_DATE"
sql-mode="STRICT_TRANS_TABLES"
sql-mode="STRICT_ALL_TABLES"
sql-mode="TRADITIONAL"

Thanks! With some small changes to tests/integration/config.php, I'm able to run composer test now.

I changed mysql.cnf but I still get the same error message.

[mysql]
sql-mode="ALLOW_INVALID_DATES"

I've tracked the cut-off message culprit to be line 40 in ExceptionHandler.php. On line 104 in SyncCommand.php you set $this->output->setVerbosity(OutputInterface::VERBOSITY_DEBUG);. I was thinking that if the user uses the -v flag then we could lift $maxQueryLength to something bigger. In my case I need it to be well over 5000. But I'm trouble finding the instance creation of ExceptionHandler. What do you think?

closing because the original issue is resolved

Fixed number 2 with the answer here: http://stackoverflow.com/a/41781184/205696
I tried to set the config in mysql.cnf to:
SET SQL_MODE='ONLY_FULL_GROUP_BY,STRICT_TRANS_TABLES,ERROR_FOR_DIVISION_BY_ZERO,NO_AUTO_CREATE_USER,NO_ENGINE_SUBSTITUTION,ALLOW_INVALID_DATES'

and a dozen other similar variables names but got mysql: [ERROR] unknown variable every time. The SO answer solved it though.

As I see it, there is only 3 options to set the $maxQueryLength variable.

Set visibility of $exceptionHandler; in Database\Connection to public
Create a new class that inherit from Database\Connection and make that class available in SyncCommand and implement a setMaxQueryLength method that use $exceptionHandler->setMaxQueryLength according to verbosity level ($this->output->getVerbosity() === OutputInterface::VERBOSITY_DEBUG).
Create a new class that implements ExceptionHandlerInterface. Change the ConnectionFactory make signature to accept an ExceptionHandler - all the way to createSingleConnection(array $config, $lazy) or use the $config object. Then in createSingleConnection we can add the new ExceptionHandler via $connection->setExceptionHandler.

The first option might violate the open/closed principle.
I think the second option is my favorite as it also allows for future customization without changing Database. For now, everything would be delegated to Database\Connection but with more granular control and possibility for db-sync extensions in the future.

@mrjgreen What do you think? Is it worth creating a PR or should I just monkey patch my local db-sync?
For now I've just did no 1. Which works fine for the moment. Just wondering what your thoughts are about this. If you accept, I could put in the time to create a proper PR.

I think the correct solution may be to do something like this here, before returning the connection:
https://github.com/mrjgreen/db-sync/blob/v3/src/Command/SyncCommand.php#L219
$connection = (new ConnectionFactory())->make([
 //...
]);
$handler = new DatabaseExceptionHandler();
$handler->setMaxQueryLength(200000); // Or set from an option
$connection->setExceptionHandler($handler);
return $connection;

You are right about option 3 though - the connection factory on the database package really should support a custom error handler.
I'd be happy to accept a PR for either option 3, or the revised option in my comment above, if you feel like contributing! :)

Nice. I'll look into it in a few days, after I have finished a little CLI program (using both Database and DbSync).


remove redundant bcm43455 firmware package

This package already exists in linux-firmware_20190815.bb and upstream.
Change-type: patch
Signed-off-by: Joseph Kogut<EMAIL_ADDRESS>

ncdu output from /lib/firmware/brcm:
  480.0 KiB [#################]  brcmfmac43455-sdio.bin
  388.0 KiB [#############    ]  brcmfmac43143.bin
  380.0 KiB [#############    ]  brcmfmac43143-sdio.bin
    4.0 KiB [                 ]  brcmfmac43455-sdio.txt
    4.0 KiB [                 ]  brcmfmac43455-sdio.raspberrypi,3-model-b-plus.txt

Fixes building w/ https://github.com/balena-os/meta-balena/pull/2240
Otherwise, all the firmware is installed, breaking the build. Not quite sure why this fixes the problem yet.

@balena-ci I self certify!

The great @klutchell figured this out. It turns out that the bcm43455 firmware includes some symlinks, and removing the cypress files targeted by the symlinks caused the poky package_fixsymlinks to add linux-firmware as a dependency.

Related to: https://github.com/balena-os/meta-balena/pull/2240


[BUG] File Upload leads to "Exactly one of payloads, localPaths and streams must be provided"

System info

Playwright Version: 1.40.0 - 1.41.1
Operating System: Windows
Browser: Chromium
Other info:

Source code
We are using the following code for a file upload:
final FileChooser fileChooser = page.waitForFileChooser( ( ) -> page.locator( "..." ).click( ) );
fileChooser.setFiles( new FilePayload( ... );

This works up to 1.39.0, but since 1.40.0 following error occurs:
com.microsoft.playwright.PlaywrightException: Error {
  message='Exactly one of payloads, localPaths and streams must be provided
  name='Error
  stack='Error: Exactly one of payloads, localPaths and streams must be provided
    at prepareFilesForUpload (<REDACTED>\playwright-java-13984471466917529150\package\lib\server\fileUploadUtils.js:41:75)
    at ElementHandle.setInputFiles (<REDACTED>\playwright-java-13984471466917529150\package\lib\server\dom.js:526:77)
    at ElementHandleDispatcher.setInputFiles (<REDACTED>\playwright-java-13984471466917529150\package\lib\server\dispatchers\elementHandlerDispatcher.js:148:38)
    at ElementHandleDispatcher._handleCommand (<REDACTED>\playwright-java-13984471466917529150\package\lib\server\dispatchers\dispatcher.js:84:40)
    at DispatcherConnection.dispatch (<REDACTED>\playwright-java-13984471466917529150\package\lib\server\dispatchers\dispatcher.js:351:39)
}

	at com.microsoft.playwright.impl.WaitableResult.get(WaitableResult.java:54)

We are using the docker image "yukinying/chrome-headless-browser-stable:120.0.6099.224" within Testcontainers for our tests, so this should be compatible with 1.41.1.
I will try and provide a self contained example for this, but I would still ask you to check whether you can confirm this issue.
Thank you and best regards
Nils

I'll cherry pick it into 1.41.2


Add getTokenStatus method

Fixes #787
Description
As requested in the mentioned issue, the method getTokenStatus was added to the BotFrameworkAdapter class, using the logic that already existed in the userToken class.
Specific Changes

Add TokenStatus to the index.ts file
Port GetTokenStatusAsync from C# to TypeScript (as getTokenStatus).
Add tests for the new method

Testing

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: Aliandi sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Pull Request Test Coverage Report for Build #2014

8 of 8 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.6%) to 88.061%

Totals

Change from base Build #2012:
0.6%

Covered Lines:
3031

Relevant Lines:
3313

💛  - Coveralls

Hi @Aliandi,
Fork merges are disabled that's why the build is failing. Chris has grant you permissions to the repo now.
Could you please push your changes in a new branch from this repo?
Thanks!

Replaced by #838


Missing step in example

The example misses the New-MSOLDomain command as a step.
[Enter feedback here]

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: cd934d58-08f8-a3ca-2686-0006b0b8f74a
Version Independent ID: 84d9cab3-09b7-95ba-83ea-435ca0ccbb47
Content: Admin takeover of an unmanaged directory - Azure AD
Content Source: articles/active-directory/enterprise-users/domains-admin-takeover.md
Service: active-directory
Sub-service: enterprise-users
GitHub Login: @curtand
Microsoft Alias: curtand

#reassign:@barclayn

@SuperTekBoy Thank you for the feedback. #please-close


mainCalculator: divide by zero!

Simple Calculator
1. Addition
2. Subtraction
3. Multiplication
4. Division
5. Exit
Enter your choice: 4
Enter the first number: 69
Enter the second number: 0
thread 'main' panicked at 'attempt to divide by zero', mainCalculator.rs:24:59
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

whoop

Latest update handles the divide by zero exception as well as an overflow exception. Closing issue.


Clearing a KeyboardDatepicker value opening the calender unlike KeyboardDatePicker.

I have added a clear button in both DatePicker and KeyboardDatePicker. The ideal behavior should be, the values should be cleared which is happening correctly in KeyboardDatePicker, but in the case of DatePicker, the calendar is opening on clicking the clear button.

[x] I have searched the issues of this repository and believe that this is not a duplicate.
[x] The issue is present in the latest release.

Current Behavior 😯
The calendar is opening on clicking the clear button in DatePickerComponent.
Expected Behavior 🤔
Just the text field should be cleared, as happening in the case of KeyboardDatePicker.
I have reproduced the issue in the following code sandbox.
https://codesandbox.io/s/material-ui-pickers-usage-demo-8gu6h?file=/package.json

Thanks, @oliviertassinari  for v4 example. But there's no separate KeyboardDatepicker and DatePicker. Only DatePicker is imported, which is behaving as KeyboardDatePicker by default. And as there's no documentation for this too, I tried trial and error methods and type definition to make it work like DatePicker without Keyboard input, but I couldn't find it.
In addition, as there's no KeyboardDatePicker in the v4, so the issue is directly for v3. Also, v4 doesn't seem stable and no documentation as of now, so I believe no one will move to v4 right now.
So what solution do we have in that case?

@sirajalam049 I think that you can wait for the release of v4, we will provide a high-level upgrade guide.

Just found a way of doing this, adding here in case someone looking for it.
Add  e.stopPropagation(); on the click handler of the close icon button, it won't' pass the click event to the calendar and it won't open.


Package graph doesn't resolve for latest release/Xcode 13.2

I'm not sure how to debug this because I'm not super familiar with SwiftPM, but I just cloned this project and wasn't able to build at all due to this error:

Might be an instance of that broken vision of Xcode Apple released.

Or it could be that the project specifies the AudioKitUI dependency as main instead of 5.2.0 🤷 I'm checking now, but it's taking a while due to very slow internet.

That wasn't it, because AudioKitUI doesn't publish release versions. Giving up on this for now.

Resolves fine for me with 13.2.1.

Also worked for me. Thanks for your patience and quick response.


WS-QVH buffer format and help

Hi Can you help me, how ws-qvh works and what format, it is buffering its raw data. so i can use AVFOUNDATION ios library to mimic same.

Hi.
ws-qvh sends raw h264 NAL-units with additionally four bytes at the beginning ([]byte{00, 00, 00, 01}) over Web Socket. One NALU per message.
That's all.

I have pure swift version using AVFoundation, I ll see if I can stream to websocket.
how port is managed? random?

https://github.com/NetrisTV/ws-qvh/blob/dde948424463b4255c27e2897fa7c70eae691cc7/main.go#L23-L30


osd: expose udev into the container

In order to be able to retrieve udev information, we must expose its
socket. As per, https://github.com/ceph/ceph/pull/25201 ceph-volume will
start consuming udev output.
Signed-off-by: Sébastien Han<EMAIL_ADDRESS>

jenkins test pipeline


Getting unicode error when enabling flask-compress on my application

Hello!
I am running my app on Heroku and was hoping to add gzip compression. I tried whitenoise first and although it didn't give any errors, the gzip compression was not working. Then I found your extension and was happy to see it is so simple. Thanks for making this. Unfortunately I am running into the following error:
UnicodeDecodeError: 'utf8' codec can't decode byte 0x8b in position 1: invalid start byte
If a comment the line that add compress, then the app works again.

Fixed it. It was this problem: https://github.com/mgood/flask-debugtoolbar/issues/83

Thank you for keeping this Issue updated. Sorry I wasn't able to reply until now.


Remove Trump reference from README

A lot of people do not appreciate and are not happy about seeing "Make America Great Again" references. Replace it with something other than a racist's slogan.

Codecov Report

Merging #31 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #31   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines          78     78
=====================================
  Hits           78     78

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 a8bf80d...638fe7c. Read the comment docs.

👍


Support concurrent volume.tier.upload

What problem are we solving?
volume.tier.upload actually is a network-intensive task. We should allow it using more network bandwidth.
How are we solving the problem?
Adding a -concurrency option like other commands
How is the PR tested?
This PR does not introduce any additional logic.
Checks

[x] I have added unit tests if possible.
[x] I will add related wiki document changes and link to this PR after merging.

need to have a way to ensure limit the number of concurrent volume.tier.upload jobs per server.

I believe the uploading files is not a very fast operation even provided with 10Gbps network. Is it really needed to limit concurrency for each single server?

I believe the uploading files is not a very fast operation even provided with 10Gbps network. Is it really needed to limit concurrency for each single server?

The network resource could be limited.


Improve parsing for git URLs when adding third party CUE modules

What are you trying to do?
The current design for fetching repos is very limiting. We're parsing a URL into repo and path, but current design assumes 2 parts for the repo and rest for the path.
For example, the URL gitlab.com/dagger/dagger/stdlib/test/test gets parsed into:

Repository: github.com/dagger/dagger
Path: /stdlib/test/test

I think it's more flexible to require a .git separator in the URL to get the path after it, otherwise making the full URL the repository.
Examples:

URL
Repository
Path

gitlab.com/dagger/dagger/stdlib/test/test
gitlab.com/dagger/dagger/stdlib/test/test

gitlab.com/dagger/dagger/stdlib/test/test.git
gitlab.com/dagger/dagger/stdlib/test/test

gitlab.com/dagger/dagger/stdlib/test.git/test
gitlab.com/dagger/dagger/stdlib/test
/test

Why is this important to you?
GitLab is well adopted and you can create arbitrary depth on the repo path because of sub-groups.
There's an issue already for general improvements on this but we need to expedite this specific fix because there's users blocked by the current design.

#2511

How are you currently working around this?
There's no workaround apart from managing this manually, skipping writing to dagger.mod and dagger.sum. It's possible to use a git submodule instead, inside cue.mod/pkg.

I'm closing this issue since we are no longer working on CUE modules


ENV for enabling preview missing

The SDK should be able to be configured via an ENV to allow / disable the preview function like in the react starter project for example

@ViolanteCodes this was fixed by https://github.com/ButterCMS/buttercms-java/commit/a74bcf183ba090a9bb25a3ea59b354e8ddfa639b


Add git pre-push hook

Adding hooks into .git/hooks is not reflected
in a commit history. Creating git_hooks folder
and configuring git to use hooks from there using:
git config core.hooksPath path_to_git_hooks
This PR has:

[ ] Appropriate testing (molecule, python unit tests)
[ ] Appropriate documentation (README in the role, main README is up-to-date)

/lgtm


x/build/cmd/gomote: add ssh access to builders

People have long been requesting ssh access to gomote-creatable builders.
The idea being it'd be something like:
you@laptop$ gomote create <instance-type>
user-bradfitz-inst-type0
you@laptop$ gomote ssh user-bradfitz-inst-type0

And then you're in bash or whatever (let's ignore Windows and Plan 9 for version 1):
gopher@user-bradfitz-inst-type0$

Architecturally, the connections would be like:
    [ Client ]  <-----> [ farmer.golang.org ] <-----> [ ephemeral buildlets ]

We'd modify the x/build/cmd/buildlet code to have auth-less ssh support (they're network-isolated), and then modify the x/build/cmd/coordinator (farmer.golang.org) to have an ssh server (well, an additional one on a different port) that did the authentication and proxied to the buildlet's ssh.
I haven't used the x/crypto/ssh package much. I imagine on the buildlet it needs to create a pty (?) or something and run bash.
On the coordinator side, I'm not exactly sure what I need to do in that ssh server after authenticating the client. I'm only vaguely aware of the ssh protocol. At some level I proxy it to the buildlet.
Help wanted. What I'd love to see is:

Go ssh server on localhost:2200 that has no auth and drops into bash.
Go ssh server on localhost:2201 with auth, proxying to the localhost:2200 ssh server.

(no openssh server; just Go is most deployable in our current environments where we can update the buildlet binaries quickly but the OS images are often more baked)
Alternative I considered: only one ssh server (on the buildlet) with a random password, and have the coordinator only be a TCP proxy. But then I'd still know how to route incoming TCP connections to different backends, implying some protocol there anyway, and I think it can just be ssh.
/cc @hanwen @aclements @ianlancetaylor

there is no standard SSH server in crypto/ssh, because it would have to interact with all the nasty platform bits (process control, ptys, etc.)
I think the fastest solution would be: run openssh on the buildlet (configure it for authless login), and use the coordinator as proxy at the channel level: the proxy passes on all OpenChannel requests. The x/cprypto/ssh/test show how you can start opensshd (but you'll probbably want to run it in daemon mode).
The proxying takes the input from NewClientConn and passes it into the NewServerConn output and vice versa.

@hanwen, if I wanted to take advantage of the machines' existing ssh servers, could I do that?
Can the OpenChannel dialer do the ssh client auth (mix of fixed passwords & keys), establish the channel, and then stitch them together?

Yes, that should work.
I would avoid passwords if you can. Public keys are easier to manage: you have to protect the secret in just one place, and they're immune to MITM attacks.

A lot of our build VM images have fixed passwords. e.g. gopher/gopher:
https://github.com/golang/build/blob/9412838986dd9e33dcc936f994012a6941c55bbf/env/openbsd-amd64/make.bash#L99
We're not exactly secret about those. They're one-off VMs and run on isolated networks.
But good to hear using the existing ssh server should work. I'll play.


Expand firmware example selection

Splitting issue #12 into this issue, which is adding additional firmware (rather than just having ANY firmware)
Bluegiga BLE112
Bluegiga BLE113
Bluegiga BLE121LR
SiLabs BGM111
SiLabs BGM113
Cypress EZ-BLE
Nordic 52 series
Anaren BLE (Broadcom WICED stack, or Anaren stack)

Probably not going to do this - lots of work, minimal return.


Add metrics info to FAQ

Update FAQ to include information about source and registry errors exposed by ExternalDNS

CLA signed

/assign justinsb
@justinsb

@zachyam it looks like something is wrong with the CLA sign, can you double check?

I signed it


Prepare documentation for v1 release

The documentation needs a review before an official v1 release.
Current TODO:

[ ] basic example usage for each command
[ ] more docker image usage examples
[ ] expand documentation for installing runtime dependencies for Ubuntu
[ ] add documentation for installing runtime dependencies for Mac

Feel free to comment below if you think anything else is missing or needs improvement.

@coderpatros Any updates on getting some docs around the Docker usage? I'd liked to use a local BOM file as input to the container but volumes and binding doesn't seem to  work.

What command and parameters are you using @coobr01?

@coderpatros the one that ended up working was
docker run --rm -v "%cd%:/tmp" cyclonedx/cyclonedx-cli convert --input-file /tmp/bom-nodejs.xml --output-format CSV --output-file /tmp/coobr01.csv
If I remove the output-file param and just look to receive console output I get the  System.Byte[] as a return.

@coderpatros, any update on this ticket?
I have spent several hours trying to get the latest binary to run on both mac and linux, but the setup documentation is missing.  This is a bit frustrating, IMO, as there is no basic "Getting Started" section, nor a script to do whatever is necessary with the binaries to set up this tool.
You would probably have a higher adoption rate if a "Getting Started" section which describes how to set up the tool from was added.

@dmuse89 you should be able to just download the binary for your platform, mark the file as executable, and run it. You might have issues on Mac as we don't currently sign the binary with an Apple developer certificate. In that case it might be easier to use the Docker image.


.NET Core 2

Anyway to make this compatible with .net core 2 applications?

Everything here is .NET Core 2 compatible already.


Add link checking to docs

The current documentation might produce dangling links. Astro and starlight have integrations to automatically check the sanity of links. We should implement this so that we have a systematic way to prevent 404s.

Always a good idea!
I noticed that the Starlight links validator is only for internal links. I'm guessing you want better coverage than that since there are plenty of external links, too. Also, relative links aren't supported in the plugin, though there is an option for ignoring them.

Perhaps adding in Lychee is more fitting? I'm just not sure how best to add that given you can use it in multiple ways.

@garikAsplund Thanks for the analysis!
Lychee indeed seems like a compelling option and matches well with our strong desire to use rust-based tooling wherever possible.
The challenge here is probably that we can't easily run it via e.g. pre-commit hooks. It would be trivial to add it to the pre-commit.nix file, but the docs tracked in this repository don't contain all contents. Some of it is generated on the fly during deployment. Maybe it would make sense to add it somewhere around the check script here:
https://github.com/TraceMachina/nativelink/blob/c3f648ecaad4861983bce1a5dc67781685bd1e80/docs/package.json#L11
The build script first generates the files that are not tracked in the repo and then runs the checks. So if Lychee ran in that step I'd expect it to have full coverage and make deployments fail early if links are broken.
cc @blakehatch @SchahinRohani thoughs?

FYI, I already caught 404 issues many times while I was navigating the NativeLink Documentation Page.
They were all caused when we click the relative links in pages.
For instance, in this page -  https://docs.nativelink.com/contribute/guidelines/
there's a wrong link  for flake.nix.
But the correct link should be this.
The reason above is because these pages include relative links from *.md files, which are then converted to *.mdx files.
Since it's affecting actual user experience, my opinion is to fix the relative URLs first ( which are actually producing 404 errors ) and then think about adding CIs for checking 404s later on.
I can make a PR for fixing relative links in *.md files right now if you permit.
Thanks.
cc: @MarcusSorealheis, @allada , @aaronmondal

@aleksdmladenovic would be incredible if you want to make a PR for this! Will promptly review it once it's open :)

FYI, I think the Lychee solution is a great idea for ensuring link integrity. I agree that the current 404 errors are an issue and need to be addressed. However, I believe the relative paths should be retained in the documentation, as they are essential during development.
By properly configuring the relative paths, we can avoid the need to constantly switch between the development docs and the "hardcoded" production docs. This would significantly improve workflow and ensure consistency during development.
Therefore, I would suggest only adding hardcoded links for those that point to actual GitHub files, while keeping the relative paths for the actual Astro documentation.
cc: @aleksdmladenovic @aaronmondal @blakehatch


Ubuntu 20.04 CT install not completing - langchain-huggingface error 1

LocalAI version:
2.2
Environment, CPU architecture, OS, and Version:
Ubuntu 20.04
Proxmox virtual environment. Container.
x86 CPU
Describe the bug
On install, this failure occurs:
mkdir -p backend-assets/grpc
go build -ldflags " -X "github.com/go-skynet/LocalAI/internal.Version=v2.2.0-10-g6597881" -X "github.com/go-skynet/LocalAI/internal.Commit=65978818546a50e0628190005efdf9119d34beb7"" -tags "tts" -o backend-assets/grpc/langchain-huggingface ./backend/go/llm/langchain/
build github.com/go-skynet/LocalAI/backend/go/llm/langchain: cannot load io/fs: malformed module path "io/fs": missing dot in first path element
make: *** [Makefile:519: backend-assets/grpc/langchain-huggingface] Error 1

To Reproduce
git clone https://github.com/go-skynet/LocalAI
cd LocalAI
make GO_TAGS=stablediffusion GO_TAGS=tts BUILD_TYPE=openblas build

Expected behavior
Install and run.
Logs
N/A
Additional context
N/A

SOLVED (and marked accordingly)
For reference:
Go has to be updated from Ubuntu's standard golang package. Follow these instructions:
https://go.dev/doc/install
Run 'go mod tidy'
Installation worked perfectly.

SOLVED (and marked accordingly)
For reference:
Go has to be updated from Ubuntu's standard golang package. Follow these instructions:
https://go.dev/doc/install
Run 'go mod tidy'
Installation worked perfectly.


fixfix compatibility issue for s3 endpoint when use incompatible client

Proposed changes
there are some incompatible issues when use s3 endpoint but use oss,obs,cos location, fix them.
fix https://github.com/apache/doris/pull/22784
When s3 endpoint in catalog properties or load properties:
"s3.endpoint=cos.mycloud.com"
we can se  its value is to access cos.
And then,  when the query location is "s3://bucket/key" , we should use s3 client, and  fit it with the s3 properties.
For another,  when the query location is "cosn://bucket/key" , we should use cos client, and  fit it with the cos properties.
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 44.91 seconds
stream load tsv:          542 seconds loaded<PHONE_NUMBER>9 Bytes, about 131 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.3 seconds inserted 10000000 Rows, about 341K ops/s
storage size:<PHONE_NUMBER>0 Bytes

run fe ut


Fix CI builds

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@bc4d655). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main       #2   +/-   ##
=======================================
  Coverage        ?   62.76%
=======================================
  Files           ?        9
  Lines           ?      478
  Branches        ?      267
=======================================
  Hits            ?      300
  Misses          ?      166
  Partials        ?       12

Flag
Coverage Δ

Linux
24.16% <0.00%> (?)

Windows
66.50% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


chore: standardize bitflags

This standardizes the way bitflags are defined (bit shift & no #[non_exhaustive]).
I also removed some bitflag tests since they just restate the original definition and were not kept up to date anyway.

I believe testing the bitflag values provides as much value as testing twilight_http::API_VERSION == 9; it's just a redefinition.
Could you state why you'd like for these tests to exist?

Requiring the test to be written would make the author double-check the value with the documentation, but I can also see how it is unnecessary. We'll have to get other input on this change.

Requiring the test to be written would make the author double-check the value with the documentation

This is imo handled by PRs requiring 2 approvals

Requiring the test to be written would make the author double-check the value with the documentation

This is imo handled by PRs requiring 2 approvals

That sounds like a good enough reason. I'm fine with this then


Question about type_description

I'm trying to filter out RWStructuredBuffers counters from reflection. I can see that the type name has something like counter.var but I cannot see other fields with which to identify them. Is there a more reliable way to do it? Many thanks

Can you provide a simple HLSL/SPIR-V example of what information you are trying to find... I guess not quite sure what is not reliable about the current way

Metadata.spv.txt
I'm attaching the spv file so you know what I mean. The original hlsl has a RWStructuredBuffer, and dxc produces two resources from it, the main buffer and the counter. The counters have a type of type_ACSBuffer_counter.
During reflection, I check for (binding.resource_type == SPV_REFLECT_RESOURCE_FLAG_UAV) and both of those resources appear in the list (the buffer and the counter), so currently I have to resort to string comparing the name to find the string counter.var that is what spirv-reflect ends up returning for the binding name.
Hopefully it's clearer now

@spencer-lunarg Did that help at all? Do you need any more details?

Sorry, just lost track of this issue.

The original hlsl has a RWStructuredBuffer

Can you provide be with just that snippet of HLSL you are using? A more simplified / minimal HLSL snippet (preferably via compiler explorer link)

So looking at the SPIR-V only, I see there is a
%type.ACSBuffer.counter = OpTypeStruct %int
%ptr = OpTypePointer Uniform %type.ACSBuffer.counter

from there, I see 4 variables use it like %var = OpVariable %136 Uniform (which are the counter.var.ExampleRWStructur* handles)
... I guess I still haven't fully figured what you want and if it is something SPIRV-Reflect is missing or just DXC has lost the information when generating SPIR-V so we can't detect it

I apologize for not being clear, here's the hlsl. https://godbolt.org/z/fvaqG4b6z
If I put that through dxc with the -Od option, this is what I get. However I can see now that in Godbolt this doesn't appear. Could it be that either Godbolt or myself have older dxc? I could try with a newer one.
; SPIR-V
; Version: 1.0
; Generator: Google spiregg; 0
; Bound: 24
; Schema: 0
               OpCapability Shader
               OpMemoryModel Logical GLSL450
               OpEntryPoint Fragment %metadata "metadata" %out_var_SV_Target0
               OpExecutionMode %metadata OriginUpperLeft
          %3 = OpString "C:/Users/redorav/OneDrive/corsair-engine/Source/Rendering/Shaders/Metadata.hlsl"
               OpSource HLSL 600 %3 "

RWStructuredBuffer<float4> g_colorBuffer;

float4 metadata() : SV_Target0

{

    return float4(0.0, 0.0, 0.0, 0.0);

}

"
               OpName %type_RWStructuredBuffer_v4float "type.RWStructuredBuffer.v4float"
               OpName %g_colorBuffer "g_colorBuffer"
               OpName %type_ACSBuffer_counter "type.ACSBuffer.counter"
               OpMemberName %type_ACSBuffer_counter 0 "counter"
               OpName %counter_var_g_colorBuffer "counter.var.g_colorBuffer"
               OpName %out_var_SV_Target0 "out.var.SV_Target0"
               OpName %metadata "metadata"
               OpName %src_metadata "src.metadata"
               OpName %bb_entry "bb.entry"
               OpDecorate %out_var_SV_Target0 Location 0
               OpDecorate %g_colorBuffer DescriptorSet 0
               OpDecorate %g_colorBuffer Binding 0
               OpDecorate %counter_var_g_colorBuffer DescriptorSet 0
               OpDecorate %counter_var_g_colorBuffer Binding 1
               OpDecorate %_runtimearr_v4float ArrayStride 16
               OpMemberDecorate %type_RWStructuredBuffer_v4float 0 Offset 0
               OpDecorate %type_RWStructuredBuffer_v4float BufferBlock
               OpMemberDecorate %type_ACSBuffer_counter 0 Offset 0
               OpDecorate %type_ACSBuffer_counter BufferBlock
      %float = OpTypeFloat 32
    %float_0 = OpConstant %float 0
    %v4float = OpTypeVector %float 4
          %9 = OpConstantComposite %v4float %float_0 %float_0 %float_0 %float_0
%_runtimearr_v4float = OpTypeRuntimeArray %v4float
%type_RWStructuredBuffer_v4float = OpTypeStruct %_runtimearr_v4float
%_ptr_Uniform_type_RWStructuredBuffer_v4float = OpTypePointer Uniform %type_RWStructuredBuffer_v4float
        %int = OpTypeInt 32 1
%type_ACSBuffer_counter = OpTypeStruct %int
%_ptr_Uniform_type_ACSBuffer_counter = OpTypePointer Uniform %type_ACSBuffer_counter
%_ptr_Output_v4float = OpTypePointer Output %v4float
       %void = OpTypeVoid
         %18 = OpTypeFunction %void
         %22 = OpTypeFunction %v4float
%g_colorBuffer = OpVariable %_ptr_Uniform_type_RWStructuredBuffer_v4float Uniform
%counter_var_g_colorBuffer = OpVariable %_ptr_Uniform_type_ACSBuffer_counter Uniform
%out_var_SV_Target0 = OpVariable %_ptr_Output_v4float Output
   %metadata = OpFunction %void None %18
         %19 = OpLabel
         %20 = OpFunctionCall %v4float %src_metadata
               OpStore %out_var_SV_Target0 %20
               OpLine %3 7 1
               OpReturn
               OpFunctionEnd
               OpLine %3 4 1
%src_metadata = OpFunction %v4float None %22
               OpNoLine
   %bb_entry = OpLabel
               OpLine %3 6 5
               OpReturnValue %9
               OpFunctionEnd

As you can see, the g_colorBuffer has a counter that is implicit in RWStructuredBuffers according to HLSL. I want to be able to recognize these and discard them without resorting to comparing names, if possible, because otherwise they appear as though they were regular buffers.
Hopefully that makes more sense.

@spencer-lunarg Sorry to have wasted your time a bit here. I have updated DXC and it no longer emits these counters in the latest versions, so I don't need to exclude them by name anymore. Thanks for your help

I have updated DXC and it no longer emits these counters in the latest versions

Ok that is good to hear, also don't worry, I am just glad things are working and nothing is broken (to my knowledge) in SPIRV-Reflect


Build Core DNS Containers via gitlab-ci

Seems we should push the containers with an 8 character SHA hash, so we can reuse them per commit.
/cc @zilman @dlx ?

https://gitlab.cncf.ci/coredns/coredns/commit/c737c741fec773f07f6a9b56e6937f02e3eff69d
https://gitlab.cncf.ci/coredns/coredns/builds/2715
$ docker pull registry.cncf.ci/coredns/coredns:c737c741
Error response from daemon: Get https://registry.cncf.ci/v2/coredns/coredns/manifests/c737c741: Get https://gitlab.cncf.ci/jwt/auth?scope=repository%3Acoredns%2Fcoredns%3Apull&service=container_registry: net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers)

Looks like we have a container now.
Just need to be able to pull it externally.

Opened cncf/cross-cloud#9 to address not being able to pull this image.
We could also push to dockerhub, but I want to spend a bit of time trying to get this to work.


🔐 Auth functionality

Define the authKey as prop on every widget
auth.js contains the reference (authKey + ENV)
In your personal .env file you can set the password

@chrishelgert Ready for review


[BUG] IntelliJ Plugin (3.0.0+jre21): No error message on mouse-over

Describe the bug.
Errors are underlined in red in editor, but there is no actual error message presented
Expected behavior
Error message (shown in progressbar on the right) should also be shown on mouse-over
Screenshots

How to Reproduce

Create version 2 json
Provoke error (like invalid reference)
Move mouse over error

🥦 Browser
None
👀 Have you checked for similar open issues?

[X] I checked and didn't find similar issue

🏢 Have you read the Contributing Guidelines?

[X] I have read the Contributing Guidelines

Are you willing to work on this issue ?
None

@rupert-jung-mw hi!
I'm not sure that I understand what's went wrong
Here is a video with latest plugin version where cmd + z is working also as show error on mouse-over
https://github.com/user-attachments/assets/706a39c9-b4f7-4a61-aa16-ea01ba00f4fb


No support for creating 0 satoshi invoices

0 sat invoices are useful because it allows the payer of the invoice to set the amount of satoshi they will send to the invoice creator

Sadly this specific type of invoice has vulnerabilities (https://github.com/breez/breezmobile/issues/110), so won't implement.

Actually it turns out there is no vulnerability if implemented properly.

That's great! I rely on 0 sat invoices heavily for www.lightning.gifts

After some further research it turned out vulnerability is there, the details are at https://github.com/ottosuess/ottosuess.github.io/blob/master/zero-amount-payments.md.
Amount-less invoice functionality will be replaced by spontaneous payments in a future, for now I'd advise not to use them as given a history of previously found LN exploits it's just a matter of time before someone starts abusing this.

@btcontract thanks for the follow up. I'm removing it too


fix docs

Summary
docs died
Reproduction Steps
https://revolt-voltage.readthedocs.io/en/latest/autoapi/voltage/user/index.html
Minimal Reproducible Code
No response
Expected Results
see the docs
Actual Results
nothing shows on navbar but if you type the link manually it shows the docs, its annoying pls fix
Checklist

[X] I have searched the open issues for duplicates.
[X] I have shown the entire traceback, if possible.
[X] I have removed my token from display, if visible.

Additional Context
No response

Can you not open issues for obvious things I suppose? It has already been pointed out multiple times on the server and I've been trying to fix it I suppose. There's also the badge on the README I suppose.


Connecting issue

Please help me out.
I've deployed the Baby Buddy on Heroku, but when I try to integrate it in Home Assistant it asks for a host and API.
What do I enter as the host?
It keeps saying that it can't connect when I enter the URL.
Thanks in advance!

Go to your user settings within BB. Copy and paste the entire URL here.
Also, the API key will be at the bottom of this same screen.

Thanks, but still get invalid autentication.
So the URL is ..../users/settings?

https://babybuddy-hms.herokuapp.com/user/settings/
This should be the URL?

No, try babybuddy-hms.herokuapp.com as host and 443 as port.

Thank you so much!!

I've tried babybuddy-hms.herokuapp.com and https://babybuddy-hms.herokuapp.com with port 443 and my API key.
Now I get error 'failed to connect'

Did you ever get it to work? Im currently running into the same issues. Ive tried every possible combination of the host / port / api key


[feature request] Automate etcd backup to Azure blob storage

My company uses Azure as our cloud provider which is not ideal. It would be neat if y'all supported etcd backups to Azure blob storage targets. Targetting NFS/some storage class targets would also be sweet, but I'm not sure how hard that'd be.
gz#12744
gz#7389

+1

Hello
Could you say me the method to impement Automate etcd backup to Azure blob storage ???
Best regards.

Keep this open - there is an active RFE from a licensed customer (me)
Also, where are we with this?

Any traction on this?


Add option to filter out generated endpoints and dtos

What are the steps to reproduce this issue?

Have legacy system with hundreds of endpoints
Generate schemas

What happens?
Hundreds/Thousands of files are generated.
What were you expecting to happen?
Generate only a subset of endpoints and dtos. Maybe by OpenAPI tags? Configuration could look like:
input: {
      target: 'xxx',
      filter: {
            containsTags: ['abc']
      }
},

What versions are you using?
Package Version: 6.11.1

Closing as this has been implemented.


Add failing testcase for #5699

As per https://github.com/doctrine/doctrine2/issues/5699#issuecomment-347038010

@flack thanks, I'll be looking at this ASAP.


feat: surfaces, cards, styling fixes, storybook improvements

resolves #338
resolves #517
resolves #537

Pull Request Test Coverage Report for Build 223480229

128 of 139 (92.09%)  changed or added relevant lines in 32 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.9%) to 73.133%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/components/src/components/Header/Cover.js
1
2
50.0%

packages/components/src/components/AspectRatio.js
7
9
77.78%

packages/components/src/components/Pullquote.js
6
8
75.0%

packages/react/src/components/Campaign/Campaign.js
6
8
75.0%

packages/components/src/components/Card/index.js
18
22
81.82%

Totals

Change from base Build 199327401:
0.9%

Covered Lines:
1582

Relevant Lines:
1965

💛  - Coveralls

This is a large one 😬 Hopefully the last large one for a while.


dblab: 0.26.0 -> 0.29.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/danvergara/dblab/releases.
meta.description for dblab is: Database client every command line junkie deserves
meta.homepage for dblab is: https://github.com/danvergara/dblab
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.29.0 with grep in /nix/store/njp858ws2dnqavrmwifc9g0d5kf55698-dblab-0.29.0
found 0.29.0 in filename of file in /nix/store/njp858ws2dnqavrmwifc9g0d5kf55698-dblab-0.29.0

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

dblab

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/7380e2e0ba0e447ef2f443f2a8245a2b9e06eb87#dblab

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

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

x86_64-linux

:white_check_mark: 1 package built:

dblab

Maintainer pings
cc @dit7ya for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 362842

aarch64-darwin

:white_check_mark: 1 package built:

dblab


Google Maps support for iOS

Any plans to implement the Google Maps SDK for iOS so we don't have to use the default Apple Maps?

@EricPKerr no plans at this point. Is there a reason why Google maps should be preferred over MapKit?

@lelandrichardson Matter of preference. There aren't  any iOS ports of Google Maps for React Native yet, but the API for this library (markers/callouts as children, etc.) seems like the right approach.

This would also be useful when using Google Maps Places Search or similar. The terms of service say that you can only show the search results on a Google Map.

This is an interesting topic and I'm not sure where to go with it.
On the one hand using the same maps platform across iOS and Android would certainly be nice and able to provide a more consistent cross-platform experience...
On the other hand, mapkit handles the types of things I want this library to do (ie, arbitrary views inside of  callouts and markers). Google Maps is significantly underpowered relative to mapkit in this regard, the other maps on the android platform just don't provide anything better.
The limitation regarding places search is one I hadn't considered though.

Google map for consistency on IOS nd Android, especially we use a lot of google map API

Apple Maps don't have information about places in my region. Would definitely love using Google Maps on iOS.

Wandering what the effort would be to implement support for Google Maps SDK for iOS as an alternative to Apple Maps. Perhaps as a config flag for react-native-maps to decide on which one to build/link with.
Would be keen to hear others thoughts on this and potentially explore this further.

+1
use Google Map or AMap or other Map provider .

+1 Google Maps on iOS
since google maps has richer information in my region....

+1 for Google Maps os iOS

+1 Being able to use the Google Places API on both and consistency in the user experience would be a big plus.

+1 for Google Maps os iOS

+1 for Google Maps

+1 for Google Maps for iOS

+1 for Google Maps for iOS

+1 google maps for ios

@lelandrichardson following up on this - seems to be quite a bit of interest both for map region coverage as well as the ability to integrate the Google Places API

@EricPKerr it turns out I will most likely implement this. I'm not exactly sure how I will do it while maintaining MapKit support though. No timeline at the moment, but if anyone else has started work on this themselves, I'd love to collaborate.

+1

+1 for Google Maps for iOS

+1

How is the issue going on?

+100

Update: We're actively working on this at Airbnb.

I'm so excited to finally close this issue!
Google Maps support has been added https://github.com/airbnb/react-native-maps/pull/548
Though we are still not at feature parity, we are still actively working on it.


Mouse Wheel Addition

          Oh, I should also add in way to play with the mouse wheel.

Originally posted by @aod6060 in https://github.com/aod6060/dsge/issues/12#issuecomment-2480371222

This will be a quick issue that I should be able to get into the game engine in a few minutes.

Here is the function interface for the mouse wheel.
glm::vec2 getMouseWheel();

Its going to be pretty basic

Alright added in the mouse wheel. Don't know why I didn't add it earlier.


Link in "Note" section redirects to same page

In the "Note" section at the top of the page, there is this paragraph:

Azure Stack users should continue to use AzureRM for now. The latest version of AzureRM which supports Azure Stack is AzureRM 2.3.0.

However, clicking on the link to AzureRM 2.3.0 doesn't work, it redirects to the same page with this error message showing up at the top:

The requested page is not available for Azure RM PowerShell 2.3.0. You have been redirected to the newest product version this page is available for.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: ab81ac50-5213-1282-d66c-e0c70794d2eb
Version Independent ID: 86324274-e59e-d272-0554-f51a63e7ec5c
Content: Overview of Azure PowerShell
Content Source: docs-conceptual/azurermps-6.13.0/overview.md
Service: azure-powershell
GitHub Login: @sptramer
Microsoft Alias: sttramer

@srs81 Thank you for bringing this to our attention!

The referenced page was previously updated and the section with the link problem no longer exists.


Start app with empty graph

Hello, is there any way I can begin the app without a default node in the graph? I want the user to have an empty screen and then begin to compose his graph from the scratch.
If I don't use a default node, I get an error that the library can't render an empty graph.
Thank you in advance!

Hello @kwnkouts thanks for reporting, but please use the issues template (feature/bug) so that we have a more contextualized overview on what exactly your problem or use case is. Thanks.


undetects are not identified as missing value

closed by commit https://github.com/adokter/baltrad/commit/95ef29a817e31ca38b896b10e4be72f4a7fe261e