index with -f for multiple summary files

Hi,
I'm trying to use -f to pass in a .txt file containing the path for each of many sequencing_summary.txt files during nanopolish -index.
I keep getting the error "Could not find filename column in the header of filename" (or whatever the first line of my file is). I can get other programs (e.g. R) to understand my first line as header, but not here. Should I be carefully formatting this file in some way?
First few lines:
filename
mnt/f/path/albacore-2.2.5-FLO-PRO001-SQK-LSK109-by_dir/1/sequencing_summary.txt
mnt/f/path/albacore-2.2.5-FLO-PRO001-SQK-LSK109-by_dir/2/sequencing_summary.txt
mnt/f/path/albacore-2.2.5-FLO-PRO001-SQK-LSK109-by_dir/3/sequencing_summary.txt

Additionally, I want to make sure I am indexing things correctly. I have:
/path/fast5/ containing 216 folders (numbered 0 : 215) each with many .fast5
/path/albacore-2.2.5-FLO-PRO001-SQK-LSK109-by_dir/ containing 215 folders (numbered 1 : 215), each containing a sequencing_summary.txt and /workspace/0 with .fast5s
I am using something strucured like:
nanopolish index -d /path/fast5 -f all_summaries.txt (pointed at those .txts in /path/albacore...) reads.porechopped.fastq
Thank you!

The -f file should not contain a header, I think if you remove the first line it should work.
jared

Ah, sorry should have been more clear, if I remove "filename" the error reports the first line of the file. e.g.:
"Could not find filename column in the header of mnt/f/path/albacore-2.2.5-FLO-PRO001-SQK-LSK109-by_dir/1/sequencing_summary.txt"

What does head mnt/f/path/albacore-2.2.5-FLO-PRO001-SQK-LSK109-by_dir/1/sequencing_summary.txt look like?

filename        read_id run_id  channel start_time      duration        num_events      passes_filtering        template_start  num_events_template     template_duration       num_called_template     sequence_length_template        mean_qscore_template    strand_score_te
mplate  calibration_strand_genome_template      calibration_strand_identity_template    calibration_strand_accuracy_template    aligned_speed_bps_template
PCA0045_20180517_0004A30B0020430F_PH_p_103_00_sequencing_run_Brachmann_CFL_Pippin_21764_read_100_ch_1633_strand.fast5   9f61ad33-99ff-4a16-a6d4-ffdac9051ccd    c0f03a85ac9a5b4dfbdfa46510ea47d1d67baabe        1633    277.198 13.36125        10689   False   0.0     10689
13.36125        10689   5335    5.377   -0.0004 filtered_out    -1.0    -1.0    0.0
PCA0045_20180517_0004A30B0020430F_PH_p_103_00_sequencing_run_Brachmann_CFL_Pippin_21764_read_100_ch_1920_strand.fast5   da47a154-c429-4c96-b2d8-19bdcd0b3220    c0f03a85ac9a5b4dfbdfa46510ea47d1d67baabe        1920    224.002 4.0     3200    False   0.0     3200    4.0
3200    1901    4.552   -0.0004 filtered_out    -1.0    -1.0    0.0
PCA0045_20180517_0004A30B0020430F_PH_p_103_00_sequencing_run_Brachmann_CFL_Pippin_21764_read_100_ch_1812_strand.fast5   fb097417-af70-4767-afcd-8684d3f9f189    c0f03a85ac9a5b4dfbdfa46510ea47d1d67baabe        1812    258.6105        2.873   2298    True    0.0     2298
2.873   2298    1263    10.319  -0.0008 filtered_out    -1.0    -1.0    0.0
PCA0045_20180517_0004A30B0020430F_PH_p_103_00_sequencing_run_Brachmann_CFL_Pippin_21764_read_101_ch_1812_strand.fast5   05159a17-71b2-4ac0-9b3e-ed6e77991365    c0f03a85ac9a5b4dfbdfa46510ea47d1d67baabe        1812    261.4835        12.12925        9703    True    0.0
9703    12.12925        9703    5449    10.162  -0.0002 filtered_out    -1.0    -1.0    0.0
PCA0045_20180517_0004A30B0020430F_PH_p_103_00_sequencing_run_Brachmann_CFL_Pippin_21764_read_101_ch_849_strand.fast5    2e88b4b3-a7d6-4079-9101-14ecc6fcdbeb    c0f03a85ac9a5b4dfbdfa46510ea47d1d67baabe<PHONE_NUMBER>        11.01525        8763    False   0.06125
8763    10.954  8763    4155    4.892   -0.0002 filtered_out    -1.0    -1.0    0.0
PCA0045_20180517_0004A30B0020430F_PH_p_103_00_sequencing_run_Brachmann_CFL_Pippin_21764_read_100_ch_320_strand.fast5    4ac31aa1-b514-4518-a954-1cc03a21cf8f    c0f03a85ac9a5b4dfbdfa46510ea47d1d67baabe<PHONE_NUMBER>5       11.22775        8982    False   0.0
8982    11.22775        8982    3439    4.826   -0.0002 *       -1.0    -1.0    0.0
PCA0045_20180517_0004A30B0020430F_PH_p_103_00_sequencing_run_Brachmann_CFL_Pippin_21764_read_101_ch_1953_strand.fast5   6cf7ec2e-1016-466b-a795-d2eafa1b73d1    c0f03a85ac9a5b4dfbdfa46510ea47d1d67baabe        1953    258.81025       33.09075        26015   True    0.57125
26015   32.5195 26015   14344   9.642   -0.0002 filtered_out    -1.0    -1.0    0.0
PCA0045_20180517_0004A30B0020430F_PH_p_103_00_sequencing_run_Brachmann_CFL_Pippin_21764_read_102_ch_1706_strand.fast5   3d27b70c-b6f7-4697-a7f3-930d6930f47f    c0f03a85ac9a5b4dfbdfa46510ea47d1d67baabe        1706    201.25725       14.317  11286   True    0.20875 11286
14.10825        11286   6211    10.133  -0.0002 filtered_out    -1.0    -1.0    0.0
PCA0045_20180517_0004A30B0020430F_PH_p_103_00_sequencing_run_Brachmann_CFL_Pippin_21764_read_102_ch_1507_strand.fast5   4c882967-a824-4a56-bc42-efa0e67c7d4c    c0f03a85ac9a5b4dfbdfa46510ea47d1d67baabe        1507    269.412 5.02175 3961    True    0.0695  3961    4.95225
3961    2104    9.876   -0.0004 filtered_out    -1.0    -1.0    0.0

Yes, looks like it:
Could not find filename column in the header of filename        read_id run_id  channel start_time      duration        num_events      passes_filtering        template_start  num_events_template     template_duration       num_called_template     sequence_length_templat
e       mean_qscore_template    strand_score_template   calibration_strand_genome_template      calibration_strand_identity_template    calibration_strand_accuracy_template    aligned_speed_bps_template

Although I'm curious about this issue, no stress, as I was running the slow version without -s or -f while we worked here, and that has finished. Almost suspiciously quickly, seeing how I have like 100+GB of fast5 files (about 2-3 hours, I had read one can expect closer to 8hrs? My computer is fine, but not like a beast or anything).
The last lines of the index output to the console were:
[readdb] indexing /mnt/f/Ian/Genome/nanopore_lmu/fast5/20180517_1534_Brachmann_CFL_Pippin/fast5/98
[readdb] indexing /mnt/f/Ian/Genome/nanopore_lmu/fast5/20180517_1534_Brachmann_CFL_Pippin/fast5/99
[readdb] num reads: 862981, num reads with path to fast5: 862870

Are the "missing" reads without a path to fast5 strike you as odd?
Thanks for all the quick responses!

Are you sure you ran with -s instead of -f there? Could you send me that summary file so I can try it locally?
The runtime of index depends a lot on your filesystem, if the number of files it indexed is similar to the  number of reads in your dataset, you're probably fine.
Jared

My mistake, mixed things up while copy-pasting, I did run -f again. Now with -s this does not report an error. But, it is just kind of hanging for some time now, I see only the new *.fastq.index has been created, but the file size = 0kb.
(Off-topic question, I can open a new issue if you like) When running makerange.py with parallel:
python "/mnt/c/Users/ia643055/nanopolish/scripts/nanopolish_makerange.py" "/mnt/f/igwill/Genome/nanopore_lmu/big/big.contigs.fasta" | parallel --results nanopolish.results -P 8 \
    "/mnt/c/Users/ia643055/nanopolish/nanopolish" variants --consensus -o polished.{1}.vcf -w {1} -r "/mnt/f/igwill/Genome/nanopore_lmu/Brachmann_CFL.porechop.fastq" -b bignano.sorted.bam -g "/home/igwill/hybrid/nanopore_lmu/big/big.contigs.fasta" -t 4 --min-candidate-frequency 0.1

I receive an error related to parallel: mkdir nanopolish.results/1/tig00000002:0-50200: Invalid argument at /usr/local/bin/parallel line 9833.
I have Nanopolish 0.10.2.  GNU parallel 20180922 (should be the most recent).
Thank you

Running makerange without parallel seems OK, getting:
tig00000002:0-50200
tig00000002:50000-73160
tig00000003:0-50200

...many tigs in between...
tig00000743:3500000-3550200
tig00000743:3550000-3561158
tig00007633:0-18431
tig00007634:0-12940
tig00007635:0-34668
tig00007636:0-24852
tig00007637:0-4219
tig00007638:0-4231
tig00007639:0-5885
tig00007640:0-5883
tig00007641:0-7940
tig00007642:0-7940
tig00007643:0-13698
tig00007644:0-13654

Ok, it isn't a problem with the sequencing summary itself then. It must not like the .fofn file. Index will be slow because you only gave one index file in that test.
I'm not sure why parallel is giving that error. Try this test:
python "/mnt/c/Users/ia643055/nanopolish/scripts/nanopolish_makerange.py" "/mnt/f/igwill/Genome/nanopore_lmu/big/big.contigs.fasta" | parallel --results nanopolish.results -P 8 echo "test" {1}

Hi,
running python "/mnt/c/Users/ia643055/nanopolish/scripts/nanopolish_makerange.py" "/mnt/f/Ian/Genome/nanopore_lmu/big/big.contigs.fasta" | parallel --results nanopolish.results -P 8 echo "test" {1}
gives the same error
mkdir nanopolish.results/1/tig00000002:0-50200: Invalid argument at /usr/local/bin/parallel line 9833.
I also tried submitting to our computing cluster, which is running GNU parallel 20141022 and a slightly different error: mkdir nanopolish.results: File exists at /usr/bin/parallel line 4990.

Hm, in that case it appears to be a problem with parallel, not nanopolish. It is not able to make the directory for the log files it generates. Maybe there is a permissions error somewhere?

Since it might have been a permission thing, I slapped a sudo in front of the test command you gave me and at least the error has changed.
Now I get:
File "/mnt/c/Users/ia643055/nanopolish/scripts/nanopolish_makerange.py", line 5, in <module>
   from Bio.SeqIO.FastaIO import SimpleFastaParser
ImportError: No module named Bio.SeqIO.FastaIO

However, if I try:
python
import Bio.SeqIO.FastaIO

as a way to test if I have that module, I don't get any error. So I would think the module is there. Does it look like a problem with my biopython setup to you?

Nothing changed, it's all a little bit baffling. To double check I just ran:

The ...echo "test" {1} command you gave, and got the same error. I do see that this command successfully made new directories, /nanopolish.results/1/ which is empty.

The sudo ... version of that command, where I still get that Bio.SeqIO.Fasta error I just reported, and no new directory is made.

nanopolish_makerange.py only, which seems to work fine, printing many lines with tig*.

sudo nanopolish_makerange.py only, which gets me the Bio.SeqIO.Fasta error

If you think this still boils down to a permissions problem, I will contact our department IT folks, since this is a work computer. Thanks for sticking with this.

Although maybe slow, is there a way to run this without parallel? Even it takes me x8 times longer presumably, at least I'll get my results.

Ah, glad to hear it!

 However, the head of my slurm.out seems to be reporting an error and is oddly big for an *.out (630MB):
HDF5-DIAG: Error detected in HDF5 (1.8.14) thread<PHONE_NUMBER>03872:
  #000: H5F.c line 604 in H5Fopen(): unable to open file
    major: File accessibilty
    minor: Unable to open file
  #001: H5Fint.c line 990 in H5F_open(): unable to open file: time = Mon Oct 22 17:50:30 2018
, name = '/mnt/f/Ian/Genome/nanopore_lmu/fast5/20180517_1534_Brachmann_CFL_Pippin/fast5/170/PCA0045_20180517_0004A30B0020430F_PH_p_103_00_sequencing_run_Brachmann_CFL_Pippin_21764_read_11408_ch_1818_strand.fast5', tent_flags = 0
    major: File accessibilty
    minor: Unable to open file
  #002: H5FD.c line 992 in H5FD_open(): open failed
    major: Virtual File Layer

I neglected to consider my 1TB of fast5s are stored on my machine and were index locally. Which I think is what this error is telling me. I can't upload them to my little cluster niche. Which then leaves me still sorting out how to get running on my computer.
I tried uninstalling & installing the exact version of parallel that the cluster had that worked, 20141022, but got fundamentally the same error. Only the referenced line has changed. mkdir nanopolish.results/1/tig00000002:0-50200: Invalid argument at /usr/local/bin/parallel line 4990. A fresh install of everything on a different machine, using a slightly older apt-get install version of parallel leads to the same thing (but on line number 3757 or so).
Any thoughts? There must be something off about the environments of the desktops I have access to?

Is there a nanopolish.results file or directory? If so try deleting it before running the command.

That was the holdup on the cluster, but I have deleted any nanopolish.results files and dirs when trying locally. I even just tried moving to a new directory to run the command from, so there should be no chance of lingering nanopolish.results files. No dice, same error along with the creation of dir /nanopolish.results/1.

nanopolish.results/1 made in the whichever directory I run the command from.

I tried the makerange.py with echo "test" {1} and had no problems with my personal laptop. I think you must be right that some weird permission issue on the work machines prevent the making of the tig*/ directories, but not the initial nanopolish.results/1 creation.
example result: test tig00000743:2850000-2900200 and I get nanopolish.results/1/various tig dirs created.
I will try the full analysis once I can access my data files again. Will let you know how that goes, but am hopeful, so no need to waste your brain power on this issue for now.

It appears to be running on my laptop now, painfully slow, but chugging along. I'll need to get on a better rig to do this in a reasonable time. But the issue with using my lab computers probably is something that needs hashed out with our IT department, I guess.
I am getting some warnings with the run however:
igwill@LAPTOP-D6T3ATUK:/mnt/f/Ian/Genome/nanopore_lmu/nanopolished/laptoprun$ python "/mnt/f/Ian/Genome/nanopore_lmu/nanopolished/nanopolish_makerange.py" "/mnt/f/Ian/Genome/nanopore_lmu/big/big.contigs.fasta" | parallel --results nanopolish.results -P 2 "/mnt/c/Users/igw87/nanopolish/nanopolish" variants --consensus -o polished.{1}.vcf -w {1} -r "/mnt/f/Ian/Genome/nanopore_lmu/Brachmann_CFL.porechop.fastq" -b "/mnt/f/Ian/Genome/nanopore_lmu/nanopolished/bignano.sorted.bam" -g "/mnt/f/Ian/Genome/nanopore_lmu/big/big.contigs.fasta" -t 4 --min-candidate-frequency 0.1
Warning: The index file is older than the data file: /mnt/f/Ian/Genome/nanopore_lmu/nanopolished/bignano.sorted.bam.bai
Number of variants in span (14) would exceed max-haplotypes. Variants may be missed. Consider running with a higher value of max-haplotypes!
Number of variants in span (17) would exceed max-haplotypes. Variants may be missed. Consider running with a higher value of max-haplotypes!
Number of variants in span (14) would exceed max-haplotypes. Variants may be missed. Consider running with a higher value of max-haplotypes!
Number of variants in span (15) would exceed max-haplotypes. Variants may be missed. Consider running with a higher value of max-haplotypes!
Number of variants in span (10) would exceed max-haplotypes. Variants may be missed. Consider running with a higher value of max-haplotypes!
Warning: 72 variants in span, region not called [68606 68823]
Number of variants in span (14) would exceed max-haplotypes. Variants may be missed. Consider running with a higher value of max-haplotypes!
Number of variants in span (34) would exceed max-haplotypes. Variants may be missed. Consider running with a higher value of max-haplotypes!
Number of variants in span (27) would exceed max-haplotypes. Variants may be missed. Consider running with a higher value of max-haplotypes!
Number of variants in span (19) would exceed max-haplotypes. Variants may be missed. Consider running with a higher value of max-haplotypes!
Number of variants in span (13) would exceed max-haplotypes. Variants may be missed. Consider running with a higher value of max-haplotypes!
Number of variants in span (18) would exceed max-haplotypes. Variants may be missed. Consider running with a higher value of max-haplotypes!
Warning: 110 variants in span, region not called [69250 69472]
Number of variants in span (18) would exceed max-haplotypes. Variants may be missed. Consider running with a higher value of max-haplotypes!
Number of variants in span (21) would exceed max-haplotypes. Variants may be missed. Consider running with a higher value of max-haplotypes!
Number of variants in span (32) would exceed max-haplotypes. Variants may be missed. Consider running with a higher value of max-haplotypes!
Number of variants in span (39) would exceed max-haplotypes. Variants may be missed. Consider running with a higher value of max-haplotypes!
Number of variants in span (13) would exceed max-haplotypes. Variants may be missed. Consider running with a higher value of max-haplotypes!
Number of variants in span (36) would exceed max-haplotypes. Variants may be missed. Consider running with a higher value of max-haplotypes!
Number of variants in span (11) would exceed max-haplotypes. Variants may be missed. Consider running with a higher value of max-haplotypes!
Number of variants in span (15) would exceed max-haplotypes. Variants may be missed. Consider running with a higher value of max-haplotypes!
Number of variants in span (60) would exceed max-haplotypes. Variants may be missed. Consider running with a higher value of max-haplotypes!
Number of variants in span (73) would exceed max-haplotypes. Variants may be missed. Consider running with a higher value of max-haplotypes!
Number of variants in span (32) would exceed max-haplotypes. Variants may be missed. Consider running with a higher value of max-haplotypes!
Number of variants in span (21) would exceed max-haplotypes. Variants may be missed. Consider running with a higher value of max-haplotypes!
Number of variants in span (24) would exceed max-haplotypes. Variants may be missed. Consider running with a higher value of max-haplotypes!
Warning: 118 variants in span, region not called [70240 70454]
Number of variants in span (20) would exceed max-haplotypes. Variants may be missed. Consider running with a higher value of max-haplotypes!
Number of variants in span (15) would exceed max-haplotypes. Variants may be missed. Consider running with a higher value of max-haplotypes!
Number of variants in span (12) would exceed max-haplotypes. Variants may be missed. Consider running with a higher value of max-haplotypes!
[post-run summary] total reads: 30, unparseable: 0, qc fail: 0, could not calibrate: 0, no alignment: 1, bad fast5: 0

(I have parallel set with -P 2 rather than 8, since my little 2-core computer is already maxed out here, having produced 3 tig directories and .vcfs after a couple hours)

I did indeed do my minimap2 mapping to make my *.bam after running nanopolish -index on the raw reads, I don't see why that's an issue?
I was expecting my genome to reflect a single clonal haploid genome. So I'm not sure about the max-haplotypes warning.

OK, rerunning samtools is easy enough.
About coverage, Canu tells me I had an estimated 180x of raw data, and a final unitig coverage of 41X. The next tig was processed by nanopolish and gave 12 Number of variants in span... notices, but no Warning: ... region not called  problems. ([post-run summary] total reads: 101, unparseable: 0, qc fail: 0, could not calibrate: 0, no alignment: 8, bad fast5: 0).
Possibly just a bum low coverage region?

Also, trying everything on a new machine (a Mac), got me an error that variants had too many arguments. Seemed to repeat for every tig:
variants: too many arguments

Usage: nanopolish variants [OPTIONS] --reads reads.fa --bam alignments.bam --genome genome.fa
Find SNPs using a signal-level HMM

  -v, --verbose                        display verbose output
      --version                        display version
      --help                           display this help and exit
      --snps                           only call SNPs
      --consensus                      run in consensus calling mode
      --fix-homopolymers               run the experimental homopolymer caller
      --faster                         minimize compute time while slightly reducing consensus accuracy
  -w, --window=STR                     find variants in window STR (format: <chromsome_name>:<start>-<end>)
  -r, --reads=FILE                     the ONT reads are in fasta FILE
  -b, --bam=FILE                       the reads aligned to the reference genome are in bam FILE
  -e, --event-bam=FILE                 the events aligned to the reference genome are in bam FILE
  -g, --genome=FILE                    the reference genome is in FILE
  -p, --ploidy=NUM                     the ploidy level of the sequenced genome
  -q  --methylation-aware=STR          turn on methylation aware polishing and test motifs given in STR (example: -q dcm,dam)
      --genotype=FILE                  call genotypes for the variants in the vcf FILE
  -o, --outfile=FILE                   write result to FILE [default: stdout]
  -t, --threads=NUM                    use NUM threads (default: 1)
  -m, --min-candidate-frequency=F      extract candidate variants from the aligned reads when the variant frequency is at least F (default 0.2)
  -d, --min-candidate-depth=D          extract candidate variants from the aligned reads when the depth is at least D (default: 20)
  -x, --max-haplotypes=N               consider at most N haplotype combinations (default: 1000)
      --min-flanking-sequence=N        distance from alignment end to calculate variants (default: 30)
      --max-rounds=N                   perform N rounds of consensus sequence improvement (default: 50)
  -c, --candidates=VCF                 read variant candidates from VCF, rather than discovering them from aligned reads
  -a, --alternative-basecalls-bam=FILE if an alternative basecaller was used that does not output event annotations
                                       then use basecalled sequences from FILE. The signal-level events will still be taken from the -b bam.
      --calculate-all-support          when making a call, also calculate the support of the 3 other possible bases
      --models-fofn=FILE               read alternative k-mer models from FILE

Report bugs to https://github.com/jts/nanopolish/issues

In response to this command:
python "/Users/ad110232/ian/nanopolish/scripts/nanopolish_makerange.py" "/Volumes/Seagate Expansion Drive/Ian/Genome/nanopore_lmu/big/big.contigs.fasta" | parallel --results nanopolish.results -P 4 \
    "/Users/ad110232/ian/nanopolish/nanopolish" variants --consensus -o polished.{1}.vcf -w {1} -r "/Volumes/Seagate Expansion Drive/Ian/Genome/nanopore_lmu/Brachmann_CFL.porechop.fastq" -b "/Volumes/Seagate Expansion Drive/Ian/Genome/nanopore_lmu/nanopolished/bignano.sorted.bam" -g "/Volumes/Seagate Expansion Drive/Ian/Genome/nanopore_lmu/big/big.contigs.fasta" -t 4 --min-candidate-frequency 0.1

Sounded like a typo error, but I can't find the issue. I copy-pasted what was working before and only changed the paths.

It might not like the spaces in the paths

Ah could be. I am just biting the bullet and waiting while transferring all of my data to a computer that I confirmed the commands work on. Hopefully that'll be the end of it. Thanks!

@igwill Just wondering if you resolved the issue of "variants: too many arguments"
I am getting the same error when running nanopolish_makerange.py in parallel...
python "usr/bin/nanopolish/scripts/nanopolish_makerange.py" My.contigs.fasta | parallel --results nanopolish.results -P 8 nanopolish variants --consensus -o polished.{1}.vcf -w {1} -r ../all_run2PCNoMid.fa -b all_run2PCNoMid.sorted.bam -g My.contigs.fasta -t 4 --min-candidate-frequency 0.1
@jts Running the makerange.py with echo "test" {1}  outputs:
test tig00000004:0-50200 test tig00000004:50000-100200 test tig00000004:100000-150200 test tig00000004:150000-200200 test tig00000004:200000-250200 test tig00000004:250000-300200 test tig00000004:300000-350200 test tig00000004:350000-373468
I tried running it without the -o argument and it returns empty .vcf files...
would appreciate any advice....
thanks in advance!
Kathy

Hi Jared
Thanks for rapid response.
I have updated to most recent version 0.10.2  (previously v0.9.2)
On running the tutorial, using v0.10.2, after this command: nanopolish variants --consensus -o polished.vcf -w "tig00000001:200000-202000" -r reads.fasta -b reads.sorted.bam -g draft.fa I am encountering the following error:
[post-run summary] total reads: 101, unparseable: 0, qc fail: 0, could not calibrate: 0, no alignment: 0, bad fast5: 0
I have now deleted my indexed reads ect from previous and am starting again from the data preprocessing stage.
Thanks


Add GetStream overloads for Memory

PR for the proposal #84

 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: faustodavid sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


KanbanBoard  DnD functionality

Description

Added dnd functionality using dnd-kit:  https://dndkit.com/ (sortable tasks && columns)

Ability for user to add columns

Ability for user to add tasks

Ability for user to remove columns
None of the above are handled through the db yet (separate issue)

Styling for kanban board within workspace could still use some work (seperate issue)
Changed eslint rules

'react/jsx-no-bind' to off
'no-underscore-dangle' to allow for _id

Related Issue

[✓] Is this related to a specific issue? Is so please specify: #67

Type of Changes
Use a check for ✓ the following type of changes:

Type

:bug: Bug fix

✓
:sparkles: New feature

:hammer: Refactoring

:100: Add tests

:link: Update dependencies

:scroll: Docs

Updates
After

Testing Steps / QA Criteria

Used ReactDevTools to make sure the correct task/column is being selected. Dnd-kit handles this within its own context.

Checklist:

[ ] This PR is up to date with the development branch, and merge conflicts have been resolved
[x] My code follows the style guidelines of this project
[x] I have executed npm run lint and resolved any outstanding errors.
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] My changes generate no new warnings

LGTM - some stuff to address later on:

Scroll bar styling
Fixed width for the container
other post mvp stylings
But overall and the most important part is that it works


Don't dispose redirects until we're sure to follow

Sometimes we return a 3xx without following it.  For example, if we've
hit the max number of redirects.  If we dispose that 3xx response and
then return it anyway, that makes us not very nice people.

We've got another problem here: when we follow an unsafe method with a GET, and set the body to empty, we don't clear out any payload headers (e.g., Content-Length).

And I'm also getting a queasy security feeling about passing along cookies.

This appears to have turned into quite a slog. Looks good to me, but we also seem to have found our old mima error again.


Logo and stickers

We need to decide on the logo that we want to use. When the logo has been confirmed we can start printing stickers!

@benediktvaldez You had some initial mockups and ideas?

@koddsson yup - will post a few mocksups

I'm excited!

Oh dude! Logo idea!

Powered by apis.is

That be awesome for people to put on their apps that they build with apis.is! Something like this (but more awesome):

Assigning this to myself so I won't keep forgetting!
(coming soon)

For reference, this is roughly the outcome of tonights working session. We'll see how it goes
http://drop.bdvs.info/11i9P

I want "Powered by APIs.is" on my laptop!

@benediktvaldez


App optimization

Make changes to plot rendering requirements and data load approach to speed up app

Note: New package now, vroom, must be installed in R for app to run


[Feature Request] Ability to disable specific cosmetics

It would be really useful for modpacks to be able to configure which cosmetics you want to show up.
Like if you just make it a config file where you can list either file name or bundle/asset name and make it a blacklist or a whitelist.
Similar to how more suits includes the ability to hide specific suits.
The main reason is sometimes a cosmetic mod has cosmetics some communities may not want to see.

Any updates on this? I can't tell it this report is getting ignored or just hasn't been seen somehow
I think it would be great if we could have the ability to disable the builtin cosmetics because like what ProfX66 said "some people may not want to see", and because some people may like to disable the builtin cosmetics to be more vanilla

I'll look into this since I saw someone suggest this on discord recently as well

Woo!!

Added into https://github.com/notnotnotswipez/MoreCompany/pull/95


better error handling for nhanesCodebook

The error message isn't that helpful - perhaps we could check to see if colname is a variable in the table given (and perhaps that the table name given is one we have) so that the errors are easier for the user to understand
cb1 = nhanesCodebook(nh_table = "DEMO_I", colname = "RIAGNDR")
Error in phonto::nhanesTranslate(nh_table, colname, details = TRUE)[[1]] :
subscript out of bounds

cb1 = nhanesCodebook(nh_table = "DEMO_I", colname = "RIAGENDR")

I have revised the function to align with nhanesA:: nhanesCodebook() and added the error messages.


v2 documentation is missing

This page does not have the instructions, sample code for v2. There is no function.json in v2 of azure function app (python)

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

ID: ecc4316a-b047-c738-424a-a1d0c3aad116
Version Independent ID: 5e01d6bd-f71c-491a-8052-e982ca1434f0
Content: Azure Functions HTTP output bindings
Content Source: articles/azure-functions/functions-bindings-http-webhook-output.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

@naseemhaq
Thanks for your feedback! We will investigate and update as appropriate.

@naseemhaq The documentation has provided the hyperlink to the sample:

For example responses, see the trigger examples.

This link redirects you to the Azure Functions HTTP trigger sample where you will find the Azure Functions V2 model using Python.


Generate prefix from subnet divider view

I have a suggestion, maybe it can be implemented.
Do you think it is possible to create prefixes in the Netbox from the subnet divider view?

That is indeed something I have on my idea list.
I'll let this open as a FR.
Thx for posting

Would love to see this feature implemented!


Select Mode easier to extend

Im trying to create my own implementation of select mode (Im basically removing all mid and selection points ONLY while dragging the features, so this way those layers wont get updated 60 times per second, to improve performance in maplibre) , I will reuse most of the methods and properties that are already in the mode i will only override some of them.
So the proposal is to make all properties and methods of select mode protected instead of private, so this way i can make references to those in my own subclass.
I realized that this way is a perfect fit for my implementation of the tool, and discovered it by browsing the documentation where its mentioned that you can create your own select method.
Also while starting this implementation i realized i will be using an slightly changed version of src/modes/select/behaviors/drag-coordinate.behavior.ts and src/modes/select/behaviors/selection-point.behavior.ts again only changing a few or a single method of the classes so it would be nice to have a way to import those classes from the Terra draw API and be able to call its method from a subclass.
Thanks for the suggestion of creating your own implementation of the mode, it really opened my mind.

Would it be easier if we provided a way to disable selection/midpoints whilst dragging in the builtin in Select mode?


Remove old genclassification

With this PR we get rid of the old sources struct and implement the classifier as originally designed using Named Tuples

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@04570b8). Click here to learn what that means.
The diff coverage is 100%.

@@           Coverage Diff            @@
##             master     #70   +/-   ##
========================================
  Coverage          ?   94.3%
========================================
  Files             ?      16
  Lines             ?     193
  Branches          ?       0
========================================
  Hits              ?     182
  Misses            ?      11
  Partials          ?       0

Impacted Files
Coverage Δ

src/models/generation.jl
100% <100%> (ø)

src/base.jl
100% <100%> (ø)

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 04570b8...d376762. Read the comment docs.


updated the tag name of the image

Motivation and Context
When building locally the image is currently census-rm-casesvc, docker-dev looks for eu.gcr.io/census-rm-ci/rm/census-rm-casesvc.  So having this changes makes using locally easier
What has changed
Change to tagging in pom.xml
How to test?
Local copy, mvn clean install.
Then:  docker ps | grep casesvc
The latest image should be eu.gcr.io/census-rm-ci/rm/census-rm-casesvc
Links
https://trello.com/c/8sVc2ua0/624-fix-built-image-name-for-casesvc

do not merge yet

Codecov Report

Merging #4 into master will decrease coverage by 1.66%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master       #4      +/-   ##
==========================================
- Coverage   91.12%   89.45%   -1.67%
==========================================
  Files          33       33
  Lines         901      901
  Branches       79       79
==========================================
- Hits          821      806      -15
- Misses         60       78      +18
+ Partials       20       17       -3

Impacted Files
Coverage Δ

...tp/response/casesvc/service/CaseReportService.java
25% <0%> (-75%)
:arrow_down:

...tp/response/casesvc/scheduled/ReportScheduler.java
43.33% <0%> (-30.01%)
:arrow_down:

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 203f813...13de581. Read the comment docs.

@AdeHarrison Is there a reason not to just remove the re-tagging step from the travis script?


create edge with same subquery between from and to use position params, result is wrong

OrientDB Version: <3.0.17>
Java Version: <1.8.0_172>
OS: <windows 10>
Expected behavior
public class TestOrientClient {
    public static void main(String[] args){
        OrientDB orientDB = new OrientDB("remote:localhost/test", OrientDBConfig.defaultConfig());
        try(ODatabaseSession db = orientDB.open("test","root","root");) {
            // Enter your code here...
            db.begin(OTransaction.TXTYPE.OPTIMISTIC);
            db.command("create edge friends from (select from person where name=?) to (select from person where name=?)","tom","green");
            db.commit();
        }
    }
}

exist data in person class
+----+-----+------+-----+----+----+
|#   |@RID |@CLASS|name |age |id  |
+----+-----+------+-----+----+----+
|0   |#45:0|person|tom  |16  |1   |
|1   |#46:0|person|green|16  |1   |
+----+-----+------+-----+----+----+

result  should be:
+----+-----+-------+-----+-----+
|#   |@RID |@CLASS |out  |in   |
+----+-----+-------+-----+-----+
|0   |#49:0|friends|#45:0|#46:0|
+----+-----+-------+-----+-----+

Actual behavior
+----+-----+-------+-----+-----+
|#   |@RID |@CLASS |out  |in   |
+----+-----+-------+-----+-----+
|0   |#49:0|friends|#45:0|#45:0|
+----+-----+-------+-----+-----+

this is  a wrong result
I think this is due to command caching
Steps to reproduce

orientdb {db=test}> create class person extends V;
orientdb {db=test}> create class friends extends E;
orientdb {db=test}> insert into person (name,age,id) values ('tom',16,1);
orientdb {db=test}> insert into person (name,age,id) values ('green',16,1);

try(ODatabaseSession db = orientDB.open("test","root","root");) {
            // Enter your code here...
            db.begin(OTransaction.TXTYPE.OPTIMISTIC);
            db.command("create edge friends from (select from person where name=?) to (select from person where name=?)","tom","green");
            db.commit();
        }

Hi @robinZhao
Thank you for reporting, it could be a problem with the query execution plan caching, I'm checking it now.
In the meantime, you can work-around it using named parameters
Thanks
Luigi

Hi @robinZhao
I managed to reproduce and fix the problem, the fix will be released with v 3.0.18
Thanks
Luigi


cannot download premium video

some video is available free for one week, but this video cannot be downloaded

Can you give me an example?

https://abema.tv/video/episode/90-493_s0_p1131
for example

yuu download https://abema.tv/video/episode/90-493_s0_p1131
[INFO] Starting yuu v0.2.0...
[INFO] Fetching user token
[INFO] Parsing webpage
[INFO] Parsing m3u8
[ERROR] Cannot download a premium video.

Are you using Japanese proxy or not? It can be 403 error if you're outside Japan.


Add section calling out the need for a client <-> server connection/link

In order for WACI to be implemented the RP will need some way to link the client (not the wallet) to the response given to the wallet.
For use cases where a user is already signed in the challengeToken could contain some identifier for the user but it becomes trickier when it's a new user flow.
On the web you can establish a WebSocket connection, for mobile apps we need to collect some more information on existing patterns.

recommend using sub


Shell input issues

@lzhzero and I have identified several shell input issues, that might relate to each other. So we decided to use this single issue to report them all at once.
Issue 1: shell corruption after bf_switchd exit
After exiting from a bf_switchd process back to a shell, when you type, there will be nothing shown. After typing ENTER, you shall see the characters typed were recorded. They were just not displayed.
A quick fix to this issue is to type reset followed by ENTER. This will reset shell states to bring it back to normal. But it would be nice not having to do this every time.
Issue 2: bfshell exit leads to bf_switchd segfault
In terminal 1, run bf_switchd. In terminal 2, run bfshell to execute some commands. Now quit bfshell in terminal 2, and you shall see bf_switchd getting a segfault error in terminal 1.
We are not sure if this is the expected behavior though.
Issue 3: messed up input in VS Code integrated terminal
In a VS Code integrated terminal, start bf_switchd, and follow by bfshell> tdi_python. Now start typing, and you shall notice random characters popping up.
This might be an issue on the VS Code side though. We are not sure.

@lzhzero helped me debug this, and the issue is partially resolved. We observed several phenomenons:

The mysterious character popping up issue occurs only in VS Code's terminal, not in a normal terminal. So it's likely a VS Code specific issue.
The terminal corruption after bf_switchd exit issue occurs in any terminal.
While bf_switchd is running in terminal A, I can start a bfshell in terminal B. If I exit bfshell in terminal B, nothing abnormal happens. But at the same time, I will get a segfault in terminal A.

I'm leaving this issue open because all these issues seem related, and 2 & 3 are unsolved yet.

Acknowledged, will try to take a look


Unclear instructions in lesson 4

Hi,
first of all: awesome project! I plan to run through this with a bunch of students starting the new school year (September). I figured I'd run through it myself first though.
In lesson 4 it becomes rather unclear what's to be expected when it comes to installing the virtualenv. It says to 'skip this step' when you're running on a rPi locally - which we will be.
Either it's missing instructions on what to do on a rPI or it's missing instructions that you'll be required to install python3-venv on the device.
My suggestion would be to skip this step altogether on the rPI and simply include instructions on how to open a second terminal window.
Please let me know your thought and I'd be happy to submit a PR with the changes.
Cheers,
Peter

Hey @ptoonen - this is great to hear. Let me know if there is any way I can help support you with your students.
When running on the Pi there is no need to set up a virtual environment. The Grove install script installs pip packages globally, so rather than add the complexity of a virtual environment and having to re-install the grove packages into the virtual environment, I felt it was easier to just not use a virtual environment for the Pi. I know a lot of developers are not too worried if their systems have lots of global packages installed as they usually just re-flash the SD card for each new project.
How do you think this can be made clearer? Should I expand the 'skip this step' instructions to include an explanation that virtual environments wont't be used on the Pi?

Hey Jim, my suggestion would be to indeed provide some more clarity that the entire virtual environment section - rather than just 'this step' - can be skipped and add some instructions how to open a second (terminal) window. I think it would save me a lot of questions ;-)

I've added a bit more details, but would love a PR with more improvements that would help clarity!

@ptoonen are we ok to close this issue? Did Jim's changes work for you?

Hey @jlooper I haven't got around to checking the full changes yet, I'll try to to that today/tomorrow and will update this issue accordingly :)

Yup, looking good @jimbobbennett, thanks!


Live tail - Event analytics

Description
Live tail feature is similar to live stream of data logs but addition to this the user is given the option to choose on which interval the data logs to be refreshed automatically. This excludes the manual labor work for refresh and helps the user to view live data for purposes including debugging their system.
Issues Resolved
#23
Check List

[x] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #494 (546d87f) into main (641c0f0) will increase coverage by 13.68%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##               main     #494       +/-   ##
=============================================
+ Coverage     58.89%   72.58%   +13.68%
  Complexity      272      272
=============================================
  Files           205       38      -167
  Lines          7260     2123     -5137
  Branches       1451      236     -1215
=============================================
- Hits           4276     1541     -2735
+ Misses         2784      420     -2364
+ Partials        200      162       -38

Flag
Coverage Δ

dashboards-observability
?

opensearch-observability
72.58% <ø> (ø)

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

Impacted Files
Coverage Δ

...ability/public/components/common/search/search.tsx

...ervability/public/components/explorer/explorer.tsx

...blic/components/explorer/hooks/use_fetch_events.ts

...y/public/components/explorer/slices/query_slice.ts

...vability/common/constants/application_analytics.ts

...ents/helpers/__tests__/sampleZeppelinNotebooks.tsx

...onents/custom_panels/panel_modules/empty_panel.tsx

...public/components/common/field_icon/field_icon.tsx

...mponents/common/helpers/ppl_docs/commands/dedup.ts

...c/components/explorer/visualizations/fieldList.tsx

... and 157 more

Continue to review full report at Codecov.

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

Thanks for the quick changes!

Thanks for valuable reviews too!!


Suggest a value for Scope

It would be very useful for readers if you suggested openid as a value for scope in the article.

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

ID: 8f11f0ed-eec4-dfbc-4498-87690a569091
Version Independent ID: 7c3ba7b6-5874-2cbb-0aef-aef24489d0b0
Content: Configure user authentication in Power Virtual Agents - Power Virtual Agents
Content Source: power-virtual-agents/configuration-end-user-authentication.md
Service: dynamics-365-ai
GitHub Login: @iaanw
Microsoft Alias: iawilt

We've added in more information about scopes to this article.


added Atelierforest-dark

Third pull request of ten for Atelier colorschemes.

Maybe you could also include a screenshot in each PR?

include a screenshot in each PR?

Something went wrong with the commit message on this one earlier (forbidden characters?)
Anyway, should be OK now, ... there are screenshots in each PR, no?

Indeed, there are... Sorry for my silly comment.


chore: bump nixpkgs

it's complaining the mio had a vulnerability https://github.com/fedimint/fedimint/actions/runs/8145238675/job/22261037469?pr=4454#step:5:4216

unsure, cc @dpc

Date:      2024-03-04

Coincidence?


⚠ Add warning icon to warning windows

 Warning! There are currently no warning icons on warning windows. What ever will we do?

 Warning! System failure is emminent!

 Warning! The things that are happen, they ing.

 Warning! The corruptrup̤̳͓t̟.

 The corruptor is coming.

Hello! I would like to work on this.

I think this is done... with https://github.com/1j01/jspaint/commit/9c88f205b29a07aa4056e0487feb2079d8ce157d and probably later commits upgrading other dialogs.


Readme.MD to include info how to run the tests

Could we add some instructions how to run the test suite? It doesn't compile out of the box, you need to do something if you want to run them manually?

Below two sections are update in PR #32. Let us know if test does not compile out of box for new version of driver.

Build the mbed OS Test Cases
Run the POSIX File Test Case

Closing it, feel free to open if required.


more glint toggles

toggles for deactivating the glint on tools & armor and those 4 other items as well would be nice :D i think that should cover all glints in the game

I will add an option for the lodestone compass aswell

Added in 1.3.2


Profile Screen: Contribution Score Bug

Is your feature request related to a problem? Please describe.
Currently the way we are displaying our Contribution Score is not correct.
Likely due to the way we are pulling the data (if we are pulling it at all).

Please take a look to see how we can best fix these.

Mine looks fine except community

Fixed


Exiting magspoof wip clears tracks

So I can add entries and track info. I can attempt to emulate this info. Once I leave magspoof and come back, I'll try to load a saved entry and find all the tack data is missing.

This was a known bug, but should've been fixed a few commits ago. Are you compiling from my GH or using the FAP RM is distributing? He doesn't submodule my code, and as such there's no guarantee it is up to date.

I need a track2 nfc payment simulator that simulates payments, and the voucher comes out, if you have it, get in touch


Error loading PyV8 (ST3 + Ubuntu 14.04)

Despite having manually installed PyV8, the loader tries to load data from remote. Any idea how to fix it?
plugins loaded
PyV8: Creating new thread
TernJS: Creating thread
Syncing project empty
Starting TernJS server for empty with 1 libs and 0 files
TernJS: Loading https://api.github.com/repos/emmetio/pyv8-binaries/contents
TernJS: Loading PyV8 binary from https://raw.github.com/emmetio/pyv8-binaries/master/pyv8-linux64-p3.zip
Package Control: Skipping automatic upgrade, last run at 2014-12-11 14:31:55, next run at 2014-12-11 15:31:55 or after
Exception in thread Thread-35:
Traceback (most recent call last):
  File "./threading.py", line 901, in _bootstrap_inner
  File "./threading.py", line 1142, in run
  File "/home/jan/.config/sublime-text-3/Installed Packages/TernJS.sublime-package/ternjs/pyv8loader.py", line 268, in run
    return self.trigger('error', exit_code=self.thread.exit_code, progress=self)
  File "/home/jan/.config/sublime-text-3/Installed Packages/TernJS.sublime-package/ternjs/pyv8loader.py", line 287, in trigger
    c(*args, **kwargs)
TypeError: on_error() got an unexpected keyword argument 'progress'

Hey guys, I just released my own package that bundles everything nicely and has no issues with PyV8: https://github.com/Slava/tern-meteor-sublime


Add Alexandria University - Egypt

Add Alexandria University - Egypt

@Hassan-Elseoudy I am sorry, the domain alexu.edu.eg is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


[Docs] "Multiple Attributes Validation" > "Custom validator" should be omitted.

There is a newly added section called "Multiple Attributes Validation" in "Validating Input" guide.
I think the first half of it is very useful, providing practical solutions for common scenarios with multiple-attribute validation.
But as for the latter half of it, I mean "Custom validator" sub-section, I think it should be omitted from the guide. It tries to show too complicated a technique for seemingly minor advantages. Some advanced developers may see it interesting, but It's not helpful for the supposed readers of the guide.

Related with #12778

Do you mean https://github.com/yiisoft/yii2/blob/master/docs/guide/input-validation.md#custom-validator-?

Yes, I do.
It's very interesting, but not suitable for the guide.

Could be moved to cookbook article...

I agree.

Done. Thanks for a good suggestion.


Merge is failing on removeRecord

I have this method to emulate transactions using fork & merge on a store. Works fine for create and update records, but does nothing on delete (the record is not deleted, although the events are raised). Meaning: the record is deleted from fork but merge would not recognize this operation and, thus, will merge nothing.
protected async beginTransaction(useHandler: (store: Store) => Promise<boolean>): Promise<void>
{
  const store = await this.getStore();
  const events: IDictionary<RecordOperation> = {};

  const enqueueOperation = (t: Transform) =>
  {
    for(const operation of t.operations as RecordOperation[])
    {
      const key = operation.op + ":" + operation.record.id + ":" + operation.record.type;

      events[key] = operation;
    }
  };

  const fork = store.fork();

  fork.on("update", enqueueOperation);

  try
  {
    const commit = await useHandler(fork);

    if(commit)
    {
      await store.merge(fork, { coalesce: true });

      for(const op of Object.getOwnPropertyNames(events))
      {
        window.dispatchEvent(new StoreDidChangeEvent(events[op] as RecordOperation));
      }
    }
  }
  catch(e)
  {
    // tslint:disable-next-line: no-console
    console.warn("Rollbacking operation");
    throw e;
  }
  finally
  {
    fork.off("update", enqueueOperation);
  }
}

@JCKodel sorry to delay with a response. I wonder if the record is in fact being deleted by the merge, but some extra processing due to relationships is causing shell records to be kept around. This might be the case if the records are appearing with relationships only (and no attributes) after the merge.
Some work is being done in this regard in #637. Also possibly related is #607.
If you are able to confirm that this is the same problem, or create a new failing test, I will investigate.

Closing due to inactivity and lack of a reproduction. I believe the underlying issue has been resolved in #637, but please let me know otherwise.


ShouldClearOnClose() does not clear filters

It only clears the TextView.
This isn't an intended behavior or is it?

My bad, I think this is normal.
And it is possible to clear the filters when closing the searchview by calling "searchView.setFilters(null);" in the onClose event.


MWPW-146370: Collect reviews from 1 to 5 stars

Enable collection of user reviews for 4 and 5 star ratings on all Frictionless pages for all sites
Resolves: MWPW-146370
Test URLs:

Before: https://main--dc--adobecom.hlx.live/acrobat/online/compress-pdf?martech=off
After: https://main--dc--adobecom.hlx.live/acrobat/online/compress-pdf?milolibs=mwpw-146370--milo--joaquinrivero&martech=off
https://mwpw-146370--milo--joaquinrivero.hlx.page/

The two unit tests don't look right. Please fix them.
https://github.com/adobecom/milo/blob/stage/test/blocks/review/components/review/Review.test.js#L21-L37
We need to reach out to other teams who also use this review block for this behavior change.


unable to install opennebula-fireedge on CentOS 8 Stream

/!\ To report a security issue please follow this procedure:
[https://github.com/OpenNebula/one/wiki/Vulnerability-Management-Process]
Description
Using any nodejs dnf module, including nodejs-1:<IP_ADDRESS>, fails with ...
# dnf install  opennebula-fireedge
Last metadata expiration check: 0:18:46 ago on Fri 07 May 2021 02:31:09 PM EDT.
Error:
 Problem: conflicting requests
  - nothing provides nodejs(abi10) >= 10.24 needed by opennebula-fireedge-<IP_ADDRESS>-1.ce.el8.x86_64

To Reproduce
dnf install opennebula-fireedge

Expected behavior
Installation succeeds; nodejs(abi10) >= 10.24 is found (when using appropriate modules).
Details

Affected Component: fireedge
Hypervisor: front end
Version: 6.0.1

Additional context
By default, CentOS 8 Stream (currently) provides nodejs(abi) v10.23.1, and I'd expect that to fail.
However, CentOS 8 Stream also provides a nodejs:14 module, which includes nodejs(abi) v14.16.0.  This should succeed.
dnf -y remove nodejs
dnf -y module reset nodejs
dnf -y module enable nodejs:14
dnf -y install nodejs

# npm version
{
  npm: '6.14.11',
  ares: '1.16.1',
  brotli: '1.0.6',
  cldr: '37.0',
  icu: '67.1',
  llhttp: '2.1.3',
  modules: '83',
  napi: '7',
  nghttp2: '1.41.0',
  node: '14.16.0',
  openssl: '1.1.1g',
  tz: '2020a',
  unicode: '13.0',
  uv: '1.40.0',
  v8: '8.4.371.19-node.18',
  zlib: '1.2.11'
}

My expectation was it would work, but it doesn't.  Maybe an issue with the c8s appstream version info, or maybe a problem with the opennebula-fireedge rpm spec.  In either case, this is the workaround I used ... and fireedge 6.0.1 works fine on c8s with nodejs:14.
dnf -y remove nodejs
dnf -y module reset nodejs
dnf -y module enable nodejs:14
dnf -y install nodejs
dnf -y download opennebula-fireedge
rpm -i opennebula-fireedge-6.0.0.*-1.ce.el8.x86_64.rpm --nodeps

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

FWIW, I took the nodejs-10.24 and npm packages from AlmaLinux 8, installed them, and now also opennebula-fireedge can be installed:
rpm -Uvh http://ftp.linux.cz/pub/linux/almalinux/8.6/AppStream/x86_64/os/Packages/nodejs-10.24.0-1.module_el8.3.0+2047+b07ac28e.x86_64.rpm http://ftp.linux.cz/pub/linux/almalinux/8.6/AppStream/x86_64/os/Packages/npm-6.14.11-<IP_ADDRESS>.1.module_el8.3.0+2047+b07ac28e.x86_64.rpm

I plan to migrate to AlmaLinux anyway, but in the meantime, I hope somebody finds this information helpful.


Replace nicknames with Pokemon names

Describe the issue
Replay logs include the nicknames of Pokemon. Some way of replacing them with the actual name is required. The logs include both but some investigation is required.
Acceptance Criteria

[ ] Replay logs have the nicknames replaced

Closing issue for now until scoping needed for additional parsing beyond rosters.


DOCS-8706: Javadoc for Mule 4.3

I am looking for Javadoc for Mule 4.3, The only one available for a very long while is 4.1.1
Can someone please update the page?

The JavaDoc is for the Mule Kernel, the community version of Mule described here: https://developer.mulesoft.com/download-mule-esb-runtime
Looks like the latest version is 4.2. We are checking in to see if there is API doc available for that version.

https://www.mulesoft.org/jira/browse/DOCS-8706

You can't view this issue
It may have been deleted or you don't have permission to view it.
Getting this error
On Fri, 7 Aug 2020 at 06:45, Duke<EMAIL_ADDRESS>wrote:

https://www.mulesoft.org/jira/browse/DOCS-8706
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mulesoft/docs-mule-runtime/issues/1480#issuecomment-670183241,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADXR5DIEEWFIF3JF7XXKKD3R7MJADANCNFSM4OL6CVIA
.

@michaelhyatt Hi, we are checking into this issue to see if we can get the 4.2 Kernel Javadocs published or packaged somewhere. You will not be able to open that Jira. Apologies for the confusion.

This is done: https://docs.mulesoft.com/mule-runtime/4.3/mule-4-api-javadoc

This is done: https://docs.mulesoft.com/mule-runtime/4.3/mule-4-api-javadoc


When using NVDA, tabbing to the "Browse" faux-link announces "blank"

I am currently testing FilePond for accessibility by using NVDA on Chrome using strictly a keyboard.  When visiting your site (https://pqina.nl/filepond/) and removing the default file that is loaded in the uploader, I use the keyboard to tab from the top of the page to the uploader.  Below is the output from the NVDA speech viewer:

Download
visited  link
Examples
visited  link
Features
visited  link
Roadmap
visited  link
Plugins
visited  link
Docs
visited  link
main landmark
Drag & Drop your files or Browse
button
blank
clickable
main landmark
FilePond on GitHub
5,094
visited  link

The "blank" is announced when the "Browse" text contained in the span with class "filepond--label-action" is the focus (with the highlight around the text).  I know coding for accessibility is not easy, but I was wondering why the "Browse" faux-link cannot just be a button (like the "remove" or "upload" buttons in the file items) so that it can be announced correctly by screen readers?
In addition to this, when tabbing through with a keyboard, "Drag & Drop your files or Browse" text and "blank" still will be announced when it is hidden from view by the file item that is taking its place.  This also needs to not be reachable by screen readers when it is supposed to be hidden.  The issue, in its current state, is that the tabindex is still equal to "0" instead of "-1" when hidden from view.
P. S. I love this uploader!

Hi! Thanks so much for reporting this, I've tabbed through FilePond using VoiceOver and NVDA but did not catch this, I try to make FilePond as accessible as possible but I might have missed a couple things (I obviously did :D) I'll see if I can fix these issues over the next couple of weeks.

Just ran some tests.
It seems to pronounce "Drag & Drop your files or Browse button" correctly, sometimes it says "blank clickable" and sometimes it does not. aria-hidden or other role=button don't seem to work to persuade it to do otherwise. Also setting a different label with aria-label does not work.
It's a span instead of a button because the element is contained inside a label that already serves as a button to focus/activate the related input file element. In this case I only wanted to highlight the browse part when tabbing through the page as you can't drag and drop files with your keyboard.
Users can still drop/browse files when the label is moved to the top/visuall hidden. It's only hidden from an visual point of view. So it's still active on purpose.
Happy to make changes if you can find a workable solution, will close otherwise as I think even with the "clickable blank" part in there the component is still very usable.


Login Method API keys

Find a better home for these... currently leaking in the frontend

Edge API key
Formatic API key
Portis API key
eventually Squarelink
Others?

@andrewdamelio nothing we can really do about that -- they're public.


incompatible with PhpStorm 2023.2

My PhpStorm says:

Incompatible: requires IDE build 231.* or earlier

Gitlab-CI-Local Helper plugin version 0.0.4
PhpStorm 2023.2: "Build #PS-232.8660.205, built on July 28, 2023" (on macOS)

Closed by #19


windowFullscreen error

Hi, after using this package I get some errors in scroll resize when keyboard appears and I discovered this is due to the windowFullscreen variable set to True.

same issue

remove <item name="android:windowFullscreen">true</item> from android\app\src\main\res\values\styles.xml

Should be resolved by 48981828e831768c4ad89a35475e8f9f24e0aca5.

Should be resolved by 48981828e831768c4ad89a35475e8f9f24e0aca5.


Roll Plugins from 7b94e2f343a8 to 9e874339eab4 (14 revisions)

https://github.com/flutter/plugins/compare/7b94e2f343a8...9e874339eab4
2022-04-07<EMAIL_ADDRESS>migrate from ui.hash* to Object.hash* (flutter/plugins#4994)
2022-04-07<EMAIL_ADDRESS>Roll Flutter from d08f7ab9c949 to 2999e8531569 (6 revisions) (flutter/plugins#5190)
2022-04-06<EMAIL_ADDRESS>[webview_flutter_wkwebview] Added support for the rest of the creation params (flutter/plugins#5130)
2022-04-06<EMAIL_ADDRESS>Roll Flutter from e382f1376a9b to d08f7ab9c949 (3 revisions) (flutter/plugins#5187)
2022-04-06<EMAIL_ADDRESS>Roll Flutter from f8e528a4ef80 to e382f1376a9b (2 revisions) (flutter/plugins#5186)
2022-04-06<EMAIL_ADDRESS>Roll Flutter from 676edd4c828f to f8e528a4ef80 (8 revisions) (flutter/plugins#5185)
2022-04-06<EMAIL_ADDRESS>Roll Flutter from 94fefaa49d86 to 676edd4c828f (6 revisions) (flutter/plugins#5183)
2022-04-06<EMAIL_ADDRESS>Roll Flutter from 120b3deb182c to 94fefaa49d86 (1 revision) (flutter/plugins#5182)
2022-04-06<EMAIL_ADDRESS>Roll Flutter from 08c46bcb58cf to 120b3deb182c (1 revision) (flutter/plugins#5181)
2022-04-06<EMAIL_ADDRESS>Roll Flutter from 5788f5ef07c2 to 08c46bcb58cf (1 revision) (flutter/plugins#5180)
2022-04-06<EMAIL_ADDRESS>Roll Flutter from 5c6918933e15 to 5788f5ef07c2 (61 revisions) (flutter/plugins#5179)
2022-04-05<EMAIL_ADDRESS>[espresso] Updates androidx.test.ext dependencies (flutter/plugins#5178)
2022-04-05<EMAIL_ADDRESS>[image_picker] Publish federated version (flutter/plugins#5174)
2022-04-05<EMAIL_ADDRESS>[webview_flutter] Migrate deprecated Scaffold snackbars (flutter/plugins#5150)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-plugins-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Get bsp utiil and proofreading fix

because _root_id is not working right now... other services are confused and think its a bound spatial point related column

Both changes look good


ESP32 resets when large amount of MQTT data is received (IDFGH-10175)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.
v4.4
Operating System used.
Linux
How did you build your project?
VS Code IDE
If you are using Windows, please specify command line type.
None
Development Kit.
seed studio xiao esp32C3
Power Supply used.
USB
What is the expected behavior?
When I send any MQTT file above 1024  bytes, it should split into multiple pieces of 1024 bytes. Instead of this, whenever I send a file higher than 1024 bytes, the entire firmware crashes.
What is the actual behavior?
When I send any MQTT file above 1024  bytes, it should split into multiple pieces of 1024 bytes.
Steps to reproduce.

Step connect to wifi
Step connect to mqtt
Step subscribe to a topic
step send a file more than 1024 kbs to that topic using pythin

Debug Logs.
No response
More Information.
No response

I just created a new project and this problem was solved


linear gradient in symbol (FF)

Did you see this bug on FF?
http://stackoverflow.com/questions/24806913/how-can-i-refer-to-an-internal-gradient-definition-inside-an-svg-sprite-symbol
Have you some options or issue for this?

No, I didn't encountered it.
Is it actual? Firefox is gradually updating including graphical stuff like this.

It's actual.
You can try:
http://codepen.io/Muskos/pen/dpjvvj bad
http://codepen.io/Muskos/pen/WGaGmN good

Would love an option to fix this as well. This is fixable by moving the linear gradient outside the symbol element. See: https://stackoverflow.com/questions/12867704/svg-linear-gradient-does-not-work-in-firefox#answer-12871215
Any ideas?

@Muskos, @Stenvdb: Could this be related?: https://github.com/svg/svgo/pull/733

It looks like this has been resolved in Firefox, so there's no need for us to address this now.


Can't install (just me?)

Getting a "[!] Unable to find a specification for Swifter (~> 1.2.6)" error when trying a pod install. I'm new to CocoaPods, but it seems pretty straight forward. Should I be able to just install by dragging Sources?

Yep. Dragging the content of Source directory will work.

I did a release (1.3.1) to find out if cocoa-pods is ready for swift 3.0 but it's not.

Thanks. It was a silly issue on my machine.


Create build process for pulling in blacklist from other repo

What?
We have a secondary repo https://github.com/clickbait/clickbait-blocker-sites that I want to make into a npm-esque repo that can store all of the sites. We can then use this to have a complete list of sites that are built-in to be blocked, and keep the conversation of WHAT sites to block away from the conversation on HOW we block it.
To do this right, we will need to be able to:

Pull in that repo's files
Build a single "output" file that will be used for the extension (with list built-in)
Determine how best to organize repo for deployment to Chrome Web Store

Questions

What build system needs to be used?
How do we need to organize the repo to handle deployments / tags?

Links
This will be in part related to the issue here: https://github.com/clickbait/clickbait-blocker-sites/issues/1 for how we organize the list in question.

One part of how EasyList works that I'd recommend keeping is to make the default block list for english only sites, create a way to load supplemental lists/filters for other languages, etc


Add load from LoRA on HF notebook

PR Checklist
Notebook requirements

[x] Notebook outputs cleared: Ensure all notebook outputs are cleared.
[x] Test in Google Colab:

[x] Test that it works in Google Colab (Python 3.10.12).
[x] Colab has its own package set and Python version, so ensure compatibility.

Incoming References

[x] README.md updates:

[x] Add a link to the recipe in the Table of Contents (ToC).
[x] Include a Colab button after that link.

GitHub

[x] Commits signed: All commits must be GPG or SSH signed.
[x] DCO Compliance: Developer Certificate of Origin (DCO) applies to the code, documentation, and any example data provided. Ensure commits are signed off.

Related to last comment: Let's adjust the intro language "We now demonstrate how to load a model back for inference." by adding a link to the previous recipe and reminding the reader of the context.

This recipe is testable.  I happened to grant access to my HF_TOKEN during my run-through just now.  Is that necessary?  If so, we can get in available as an org-level secret.
I'm going to merge this as-is, but let's address the points in these 3 comments soon.

Agree on the cross links, my thought was get this one in, then link from the training notebook to this one, and this one back to the training one. The latter I could have done in this PR. (probably on the code one too).
On testable: I don't know if it is? In the test it will download and try to run the notebook. I don't know how that will work in a CPU only environment.


feat(rate-limiting) Optionally don't fall back to ip

We want to rate-limit on a header X-Forwarded-For. When this header
does not exist it usually is an internal service. Falling back to
source ip gives us disastrous results.
This commit adds an option to fail open instead of falling back to ip.

Redone in separate PR


Wrong attribute prompt about the built-in attribute  “name" of class.

According to the python official doc, the attribute  "name" is just the class ,type ,etc.  So the instance can not get the attribute "name", but the pyplance show the "name" attr in instance .

Thanks for the bug report. This will be fixed in the next version of Pyright and Pylance.

@erictraut Thanks for your reply.😁
Besides  attribute "name" has this problem, I find that all attributes marked by the flag below have this problem.

This flag represents "Class" instead of attributes.

Thanks, this will be fixed as well.

This issue has been fixed in version 2020.7.40, which we've just released. You can find the changelog here: https://github.com/microsoft/pylance-release/blob/master/CHANGELOG.md#202074-29-july-2020

@jakebailey  @erictraut
I noticed that you solve this problem by improving grammar checking (linting) , but these attribute hints still exist in smart suggestions. I think you should completely hide these attributes that are not instance attributes from the parameter prompt list

Thanks for the bug report. You are correct that most of these attributes should not appear on instances. The __annotations__ attribute is found on objects (as defined in builtins.pyi), but the others should appear as attributes in classes.
This will be fixed in the next release of Pylance.

https://docs.python.org/3/library/stdtypes.html?highlight=built
I am not sure if other special attributes and methods have similar problems, this is the relevant information I can find according to the official documentation.

We previously hard-coded some of these attribute names. I've removed these hard-coded names and now use the type metaclass and the object base class (both defined in builtins.pyi) to determine which attributes to include for classes and instances. The attributes should now match what you see in builtins.pyi. If that list defined in the type or object class looks wrong to you, you can submit a PR to the typeshed repo to add or remove attributes.

This issue has been fixed in version 2020.8.3, which we've just released. You can find the changelog here: https://github.com/microsoft/pylance-release/blob/master/CHANGELOG.md#202083-28-august-2020


logging simulated doublets lib size

My attempt to fix what i see because of https://github.com/YosefLab/scvi-tools/pull/1078


Adds chained alerts trigger conditions

Issue #, if available:
#834
Description of changes:
This PR adds triggers for chained alerts in workflows.
We fetch all the ACTIVE state alerts created in the current workflow execution and create a map containing information for each monitor has generated alerts or not. Add Trigger condition parser which parses the provided trigger condition expression in the Reverse Polish Notation and then resolves the condition using said map.
This is used to evaluate chained alert trigger condition.
We use Painless scripting language to define chained alerts trigger condition
Trigger condition expressions
* monitor[id=1] && (!monitor[id=2] || monitor[id=3])
    *  Matches monitor id 1 generates an alert and either monitor id 2 doesn’t generate alert  or  monitor id 3 generates alert
* monitor[id=1] && monitor[id=2]
    *  Chained alert created if monitors with id 1 and 2 both generate alerts in current execution
*  monitor[id=1] || monitor[id=2]
    * Chained alert created if either monitor with id 1 or 2 generates an alert in current execution

Operartors supported are:
* (
* )
* &&
* ||
* !

Query Identifiers supported are:
* monitor ---> monitor[id=kjngkra]
* id ---> monitor[id=kjngkra]

It's heavily inspired by https://github.com/opensearch-project/alerting/pull/405 Doc level trigger condition parsing

Notification throttling
Acknowledge chained alerts
Move delegate monitor alerts to ack
setting to limit number of chained alert triggers
add search workflows API support


azcopy files to upload blob to emulated storage

Which version of the AzCopy was used?
AzCopy 10.2.1
Which platform are you using? (ex: Windows, Mac, Linux)
Windows 10, 64b
What command did you run?
 .\azcopy.exe copy "C:\Data.zip" "http://<IP_ADDRESS>:10000/devstoreaccount1/powertill/WebData.zip?st=2019-08-12T17%3A28%3A50Z&se=2199-08-13T17%3A28%3A00Z&sp=racwdl&sv=2018-03-28&sr=c&sig=s2iPFU7om14HGjgBzA%2FXLaXmsrhYPexx2mjNKBdqMSk%3D"

(fake SAS in above command)
What problem was encountered?
File upload fails with this message:
failed to parse user input due to error: the inferred source/destination combination is currently not supported. Please post an issue on Github if support for this scenario is desired
How can we reproduce the problem in the simplest way?

Run storage emulator
Create your own shared access signature
Run above command but replace with your own shared access signature

Have you found a mitigation/solution?
No, except to use Storage Explorer to upload files. But I prefer to use the command line

Hi @igitur, please use the flag --from-to=LocalBlob, as the tool cannot know what kind of endpoint you are targeting.

Thanks and excuse my ignorance.

@igitur no worries, please don't hesitate if you have any other question/concern.
Btw the Azurite project is going to replace the emulator in the future. Perhaps you could give it a try! 😄


nerdfonts-unstable: init at 2.2.0-RC

Description of changes
Add unstable version of nerdfonts
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

nerdfonts 2.2.0 has been released so I don't think this is necessary anymore, I've raised https://github.com/NixOS/nixpkgs/pull/188407 to update the main nerdfonts package.

Would it be worth adding so future unstable releases would be easily accessible?

Would it be worth adding so future unstable releases would be easily accessible?

Are unstable releases common for nerd-fonts? So far it seems they've only done it once with 2.2.0-RC, where I assume a pre-release to validate that the CI release pipeline still works was deemed necessary.

I have no idea, I suppose I can just reopen if its ever necessary


Start Building Templates to Support UI

We need to start taking the existing UI template and adding all the output parameters. This is going to be a bit of heavy lifting as there are many options and a lot of conditional output we will need to pass to downstream templates that do the build.

We are working on finishing the deployment code to then build the portal UI on top

This is a work in progress, we are developing bicep deployment and portal UI at the same time, so they work together.

templates built form bicep code


Issues displaying out-of-page ads on Android in version 2.0.7

Hello,
I'm experiencing difficulties when trying to display out-of-page ads on Android devices using version 2.0.7 of the library. I would like to confirm if the library supports this ad format.
Additional details:

Library Version: 2.0.7
Platform: Android
Ad Format: out-of-page

Any guidance or additional information on how to resolve this issue would be greatly appreciated.
Thank you.

"out-of-page" meaning Interstitial ads?


Results table

Will resolve #47. Will resolve #48.
Should rebase after #42 is merged.

Currently, there are empty functions with TODO markers in the store where the logic for adding/removing footprints and raster overlays to the cesium viewer should be implemented.
To save time, I am passing these functions off to @banesullivan for implementation, as he requested.

To save time, I am passing these functions off to @banesullivan for implementation, as he requested.

I will jump on these ASAP - please merge and let main be our staging branch


[CSL-77] Propagate MPC messages after accepting

@gromak

@gromak
Done except shares. Will be done for shares after CSL-79.


HAProxy advanced timeout / connection Options

Hi,
I would love to see some advanced timeout / connection Options in the GUI :)

timeout http-request (slowloris attacks)

Limiting TCP Connection Rate, abuse Unfair users, etc. (as described in https://www.haproxy.com/blog/use-a-load-balancer-as-a-first-row-of-defense-against-ddos/)

Thanks and regards,
Michael

Thanks for your feature request! :) I'll try to add these features in one of the next feature releases.


Fix #3320 Prevent select2:open when clearing selection

This pull request includes a

[x] Bug fix

The following changes were made

Prevent select2:open when clearing selection

https://github.com/select2/select2/issues/3320

Dear,
As a company, we are having complains from our customers regarding this issue.
We have tested and it is fixing the reported issue.
Can you please accept this pull request, I think it might be helpful not only for us, but for many other users.
KR,
Gerry


[Lagobot] enable haskell

Summary
trying out fixes
How was it tested?
buildkite run should pass

Current dependencies on/for this PR:

main

PR #809   👈

This comment was auto-generated by Graphite.


Nuxt Error: Unknown Middleware with TypeScript

Versions

nuxt:  v^2.15.0
node: v16.11.1

Reproduction
https://github.com/NCLlamas/nuxt-demo-issue
Run: npm install then npm run dev

Nuxt displays error: Unknown middleware homeGuard when running dev mode. Unfortunately no logs are created in the browser or in the console. Error is resolved when switching from TypeScript middleware to JavaScript middleware.

This project was initially built in Nuxt Bridge mode (and worked perfectly fine in dev) but due to errors with building the static app files in bridge mode I had to revert back to no bridge mode. After doing this I get the error about Unknown Middleware HomeGuard (this is a very basic middleware as you can see in the linked repo). This is a basic TS middleware modeled after the official Nuxt TS doc, and I have similar TypeScript middlewares running in other projects fine, but in this project the middleware only functions in JavaScript.
Steps to reproduce

Download linked repo above
Install dependencies
npm run dev
When navigating directly to /queue, /availability, or /settings page you will see the error, home route will not redirect.

What is Expected?
TS Middleware redirects from the home route (/) to /queue the same as it's JS counterpart.
What is actually happening?
There is an error about an unknown middleware with the TS file.
10/19 Update - I have stripped all UI and non-nuxt dependencies out of the issue reproduction and still see the error. The reproduction link should in theory be the most simple version of a nuxt app with a simple middleware

Closing this issue here as I've tracked down the problem to a bridge setup issue not present in plain Nuxt


Mac Install Script

My team has started using this plugin so I wrote a script to simplify installation a bit.  I based it off of what was already in the readme, here's what the script does:

Check to make sure that mutagen is installed, link to installation instructions if not and exit script
Download the latest plugin release
Remove old version of the plugin at ~/.lando/plugins/lando_mutagen
Install latest version of the plugin at the same directory

I also added a Mac Installation Script section to Installation in the readme.

Cool, thanks for the PR!


Minor doc fixes

Nice work. Thanks!

Nice work. Thanks!

Thank you for the PR!

Thank you for the PR!

Indeed. Awesome work! ⚡

Indeed. Awesome work! ⚡

Thank you too!

Thank you too!


Update scala-compiler, scala-library, ... to 2.12.14

Updates

org.scala-lang:scala-compiler
org.scala-lang:scala-library
org.scala-lang:scala-reflect

from 2.12.13 to 2.12.14.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (2.12.13).
You might want to review and update them manually.
build.sbt
build.sc
docs/src/resources/faqs.md

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-lang" } ]

labels: library-update, semver-patch, old-version-remains

Need bump scala version in mill by hand :(
We need read dependencies from a json file for SBT and mill.

Wired, I cannot push to this branch.
Enumerating objects: 15, done.
Counting objects: 100% (15/15), done.
Delta compression using up to 8 threads
Compressing objects: 100% (8/8), done.
Writing objects: 100% (9/9), 1.62 KiB | 1.62 MiB/s, done.
Total 9 (delta 6), reused 0 (delta 0), pack-reused 0
remote: Resolving deltas: 100% (6/6), completed with 5 local objects.
To github.com:chipsalliance/chisel3.git
 ! [remote rejected]   HEAD -> refs/pull/1950/head (deny updating a hidden ref)

I will create another patch.

@jackkoenig This PR needs you push the button.


Setting an array of scalar values

OS: MacOS
Prisma: preview21
I have a model which looks like this:
model UserFilterTemplate {
  id               String    @id @default(uuid())
  user             User
  name             String
  originCity       String?
  originState      String?
  destinationCity  String?
  destinationState String?
  trailerTypes     String[]
  maxWeight        Int?
  loadType         LoadType?
}

I'm having a resolver which has all of the same fields and basically does this:
return await ctx.prisma.userFilterTemplate.create({
  data: {
    name,
    originCity,
    originState,
    destinationCity,
    destinationState,
    trailerTypes: {
      set: trailerTypes // <--- this is basically impossible and throws an error because I'm unable to set the value to null
    },
    maxWeight,
    loadType,
    user: {
      connect: {
        id: userId
      }
    }
  }
})

This is what's get generated by @prisma/client:
export declare type UserFilterTemplateCreatetrailerTypesInput = {
    set?: Enumerable<string> | null;
};

An error I get when trailerTypes is not passed to my resolver and basically undefined:
server: TypeError: Cannot read property 'collectErrors' of null
server:     at errors.push.flatMap_1.flatMap (/Users/impowski/Projects/loaders/node_modules/@prisma/client/runtime/index.js:9794:26)
server:     at Array.map (<anonymous>)
server:     at Object.flatMap (/Users/impowski/Projects/loaders/node_modules/@prisma/client/runtime/index.js:26373:26)
server:     at Arg.collectErrors (/Users/impowski/Projects/loaders/node_modules/@prisma/client/runtime/index.js:9793:38)
server:     at flatMap_1.flatMap.arg (/Users/impowski/Projects/loaders/node_modules/@prisma/client/runtime/index.js:9716:56)
server:     at Array.map (<anonymous>)
server:     at Object.flatMap (/Users/impowski/Projects/loaders/node_modules/@prisma/client/runtime/index.js:26373:26)
server:     at Args.collectErrors (/Users/impowski/Projects/loaders/node_modules/@prisma/client/runtime/index.js:9716:26)
server:     at Arg.collectErrors (/Users/impowski/Projects/loaders/node_modules/@prisma/client/runtime/index.js:9805:18)
server:     at flatMap_1.flatMap.arg (/Users/impowski/Projects/loaders/node_modules/@prisma/client/runtime/index.js:9716:56)

The current workaround:
return await ctx.prisma.userFilterTemplate.create({
  data: {
    name,
    originCity,
    originState,
    destinationCity,
    destinationState,
    trailerTypes: {
      set: trailerTypes || [] // I cannot do set it to null
    },
    maxWeight,
    loadType,
    user: {
      connect: {
        id: userId
      }
    }
  }
})

Related issue: https://github.com/prisma/prisma-client-js/issues/459

I can confirm the behaviour but I am not sure if this is a bug as scalar lists are not allowed to be null.
More minimal reproduction here:
datasource db {
  provider = "postgresql"
  url      = env("DATABASE_URL")
}

generator client {
  provider = "prisma-client-js"
}

model User {
  id String @id @default(cuid())
  name String
  test String[]
}

const { PrismaClient } = require("@prisma/client");

const prisma = new PrismaClient();

async function main() {
  const data = await prisma.user.create({
    data:
      name: "Harshit Pant",
      test: {
        set: null
      }
    }
  });
  console.log(data);
}

main();

This is now handled with a better error:
{
  data: {
?   stringList?: BCreatestringListInput | String[],
+   id: String
  }
}

Argument stringList for data.stringList must not be null. Please use undefined instead.
Argument id for data.id is missing.

Note: Lines with + are required, lines with ? are optional.


Fix the java.lang.NoClassDefFoundError: sun/reflect/Reflection in the strongbox-ldap-authentication-provider when building under JDK 11

Task Description
As part of our effort to evaluate and migrate to JDK11, we need to fix the java.lang.NoClassDefFoundError: sun/reflect/Reflection that occurs in the strongbox-ldap-authentication-provider.
Tasks
The following tasks will need to be carried out:

[x] Fix the exception
[x] Make sure the code still works under JDK8.

Task Relationships
This task:

Is a sub-task of: #1111

Useful Links

spring-projects/spring-security#6804: Usage of sun/reflect/Reflection in transitive dependency of org.springframework.security.ldap.server.ApacheDSContainer

Help

Our chat
Points of contact:

@carlspring
@sbespalov
@steve-todorov

Will be fixed via #1783 .

@sbespalov : I'm assuming we can close this now? You've merged the necessary changes into the issues/1649 branch, right?


modules/terminal: make font configurable

Related: https://github.com/t184256/nix-on-droid-app/pull/2
@PaulGrandperrin  @Gerschtli, do you know why we haven't considered that and instead explored copying the file over?

I think i never actually tried doing a symlink because of this:
https://github.com/t184256/nix-on-droid/issues/120#issuecomment-867219785

bad news are, it has to be a regular file and not a symlink to /nix/store because, as far as the terminal app is concerned, there's no /nix/store. Auto-copying it on activation sounds hacky. Patching the app to do the sort of broken symlink resolution we already do for login script is definitely an option, but I'm basically allergic to Android development. PRs are welcome though.

Is this working?

Seems like it is, yes =)
I guess I was wrong with my original analysis: yes, here I generate what's a broken symlink from nix-on-droid perspective, but it's a valid one from the outer point of view, and that's what matters ¯\(ツ)/¯

Sounds legit, good idea, some minor things I found:

Is the actual font path a gc root?
Should that really be activationBefore? Could be in normal activation I think.
Should we care about keeping a backup/restore backup if there was a font installed manually before? (Maybe remove directory on unlink if empty)
We should use $VERBOSE_ARG consistently in our activation scripts
isNull is deprecated

Also could be worth adding to changelog :)

Thanks for your review.

Is the actual font path a gc root?

It does, as the font path is embedded into the activation script.

Should that really be activationBefore? Could be in normal activation I think.

Fixed.

Should we care about keeping a backup/restore backup if there was a font installed manually before? (Maybe remove directory on unlink if empty)

Made it back up the font, but stopped at that. I like removing the directory even less.

We should use $VERBOSE_ARG consistently in our activation scripts

Sprinkled.

isNull is deprecated

Fixed.

literalExpression

Accepted.

added support for both /nix-paths and non-mangled absolute paths.

Updated.
One more important thing I forgot to ask: does the terminal.font interface look fine? Without a terminal.enabled knob.

(will I ever get this pretty?..)

How about this? (does nothing unless there was a link; then either restores backup or removes empty dir)

LGTM :)

Thanks for your patience, squashing =)

Based on this conversation, it looks like this works.  Hurray!
I'm a bit of a noob though so like... how can I set the font?

On January 13, 2023 11:39:02 PM GMT+01:00, Matt Rixman @.***> wrote:

Based on this conversation, it looks like this works.  Hurray!
I'm a bit of a noob though so like... how can I set the font?

Short story: in file ~/.config/nixpkgs/nix-on-droid.nix set option terminal.font. Example: https://t184256.github.io/nix-on-droid/nix-on-droid-options.html#opt-terminal.font

Thank you!


feat: improve collabration by adding auto comment feature 🚀🔥

Close #710
Changes proposed

Added auto comment feature for Food Recipes Website Repository.
When an issue is opened, the auto-comment should greet the author and provide a brief acknowledgement and request for additional context.
When a pull request is opened, the auto-comment should greet the author, express gratitude, and remind them to follow the project's contributing guidelines.

Screenshots

ON ISSUES

ON PR's

Hey there @Anupkjha2601 🙌🏻
This is a core contribution to this repository, Make sure to review this to Merge.
Let's improve the collaboration using auto comment feature 🚀🔥


Updates to web parts guidance

Q
A

content fix?
yes

New article?
no

Related issues?
n/a

What's in this Pull Request?
Updates to web parts guidance to align it with the latest version of SPFx.

thx Waldek, appreciated.


Missing symlink “nvidia-drm_gbm.so” in Fedora driver package. Necessary for Wayland.

Hi.
I’m using the official Nvidia Cuda Repostory for Fedora to install both, the cuda toolkit and the nvidia driver. It works fine but the driver packages are missing a symlink that should be there since v495 (see release notes, first highlight). The symlink should be /usr/lib64/gbm/nvidia-drm_gbm.so → /usr/lib64/libnvidia-allocator.so.VERSION (or -> ../libnvidia-allocator.so.VERSION).
Without this symlink Wayland session does not start at all.
I created the symlink manually, but it would be nice if it was already in the package, unless there is a reason why it is not the case. The hard part of fixing the issue was finding out that the symlink was missing at all.
I don't know if this issues is also present in the RHEL 8 Repo.
I've also posted the issue at in the Forum but I found this github page later, which I think is the right place to post it.
Thank you in advance

Hi @JoeHannes90
Thank you for reporting this. It will be fixed in the next NVIDIA driver version package release from 515 branch for Fedora, RHEL8, and RHEL7 in the CUDA repository. Specifically in the nvidia-driver-libs RPM, here is a snippet of the .spec
diff --git a/nvidia-driver.spec b/nvidia-driver.spec
@@ -252,6 +252,7 @@
+mkdir -p %{buildroot}%{_datadir}/egl/egl_external_platform.d/

@@ -339,6 +340,13 @@
+# GBM files
+%ifnarch %{ix86}
+install -p -m 0644 15_nvidia_gbm.json %{buildroot}%{_datadir}/egl/egl_external_platform.d/
+mkdir -p %{buildroot}%{_libdir}/gbm/
+ln -sf %{_libdir}/libnvidia-allocator.so.1 %{buildroot}%{_libdir}/gbm/nvidia-drm_gbm.so
+%endif
+
@@ -523,6 +531,10 @@
+%{_libdir}/libnvidia-egl-gbm.so.1
+%{_libdir}/libnvidia-egl-gbm.so.1.1.0
+%{_libdir}/gbm/nvidia-drm_gbm.so
+%{_datadir}/egl/egl_external_platform.d/15_nvidia_gbm.json

diff --git a/nvidia-generate-tarballs-aarch64.sh b/nvidia-generate-tarballs-aarch64.sh
@@ -37,7 +37,6 @@
-        libnvidia-egl-gbm.so.* \

diff --git a/nvidia-generate-tarballs-ppc64le.sh b/nvidia-generate-tarballs-ppc64le.sh
@@ -37,7 +37,6 @@
-        libnvidia-egl-gbm.so.* \

diff --git a/nvidia-generate-tarballs.sh b/nvidia-generate-tarballs.sh
@@ -25,7 +25,6 @@
-    libnvidia-egl-gbm.so.* \

Hi @kmittman,
Thanks you very much for the quick fix. I look forward to the release of the next driver version.

Hi,
I can confirm that the files are present in version 515.48.07 and the issue is solved.
Thank you, again!


Making filesystem_util.cc UNICODE Compatible

make filesystem_util unicode compatible when USE_EXPERIMENTAL_FS=0 and USE_FINAL_FS=0

We have started the process of fixing the loader to use the "W" Win32 functions for proper Unicode support. This pull request may conflict/duplicate that work. I think for now in GitHub, switching to explicitly using the "A" functions will fix building with the UNICODE flag enabled (the loader is doing this already in some places, like RegOpenKeyExA in manifest_file.cpp in the loader and in the private OpenXR repo this has already been fixed to use RegOpenKeyExW but not yet brought over to this GitHub repo). I think fixing the build issue by explicitly using the ANSI "A" functions here in GitHub is the best fix to do here before we bring over the fixed version which uses the wide "W" functions so minimize merge pain.
Does that work for you?

We have started the process of fixing the loader to use the "W" Win32 functions for proper Unicode support. This pull request may conflict/duplicate that work. I think for now in GitHub, switching to explicitly using the "A" functions will fix building with the UNICODE flag enabled (the loader is doing this already in some places, like RegOpenKeyExA in manifest_file.cpp in the loader and in the private OpenXR repo this has already been fixed to use RegOpenKeyExW but not yet brought over to this GitHub repo). I think fixing the build issue by explicitly using the ANSI "A" functions here in GitHub is the best fix to do here before we bring over the fixed version which uses the wide "W" functions so minimize merge pain.
Does that work for you?
I think that works. Thanks!


Hangman Game should show guessed letters in the Overlay

As a player
the Hangman Game
should show the previously guessed letters in the Overlay
So that we can see the letters that have already been guessed

Closed by #250


Med Borgs needs med and chem glasses, to may not break they laws and know who's injured

Description
Borgs need to know sure what cure he use and who need help, especially then someone poisoned and can't be shift+clicked for examine.
Reproduction
Crewmate in crit? Give him inaprovalin - he died. . . . That was lexorin, lol you break laws now go to borg jail in artifact crusher!

Additional context
This is cyborg! He must know everything around and be robot-servant ready make everything for crewmate monkey batrhender,

maybe https://github.com/space-wizards/space-station-14/pull/27843 could add medhud to their PR as well

maybe #27843 could add medhud to their PR as well

That would be nice.

maybe #27843 could add medhud to their PR as well

ok

close issue please

was resolved


Incorrect == and =~ label matching behavior for empty label values

In Prometheus, a non-existent label foo is equivalent to an empty label foo="" (in fact, labels with empty values are removed before a series is stored). That is in Prometheus, {foo=""} returns all series with no foo label, as does {foo=~""}. Cortex currently returns no series for these, because it actually looks for a foo label to be present.
I stumbled over this when looking at https://github.com/weaveworks/cortex/pull/191.

Whoops!  Good to know.  Should be an easy fix.

With the chunk index we use, we don't have a way of querying for things that don't have a label - best I can think of is looking for things that satisfy other matchers, then filtering to remove those with the specified label.


chore: upgrade to candid 0.6.21

This should be stacked on top of #162

Upgraded to 0.7 so dropping this.


Numeric id-based lineage column

Feature requested
Currently, the lineage column in index data contains the full path of each source file, in String type.
This is an initial approach for the lineage column to support "delete" dataset.
There are some disadvantages for this approach:

storage bloat
high memory footprint when reading the index data
high comparison cost - string comparison

though,

easy to maintain
can apply partition filter (not implemented yet)

We can optimize the lineage column by assigning a unique id to each source file.
The following metadata change is required and need to keep the last unique id for newly appended source files.
// cur
case class FileInfo(name: String, size: Long, modifiedTime: Long)

// suggested
case class FileInfo(name: String, size: Long, modifiedTime: Long, id: Long)

With this, the lineage column can be defined with the "id" value/Long type instead of the full path / String type.
Acceptance criteria
Should work well with the current features using the lineage column:

[ ] incremental refresh
[ ] HybridScan

Success criteria
Additional context

@sezruby Thanks for writing this up! Can you also put up the success criteria? Also, how can we still support partitioning? (maybe use ids instead of actual partition keys for push-down?)

Added by #234