ProteomicsLFQ exit status 6

Hi again,
I run the pipeline like this:
nextflow run main.nf
		 -profile singularity
		 --input ../data/raw/*.raw
		 --database ../data/reference/proteome/Mouse_prot_database.fasta
		 --expdesign ../data/exp_design/experimental_design.tsv
		 --outdir res_MsdClvg3
		 --allowed_missed_cleavages 3
		 -c crg.config
		 --add_decoys

These processes succeed:
N E X T F L O W  ~  version 20.07.1
Launching `main.nf` [curious_galileo] - revision: 18f2d39694
executor >  sge (11)
[5d/b8a76b] process > raw_file_conversion (1)         [100%] 1 of 1 ✔
[-        ] process > mzml_indexing                   -
[9a/582f45] process > generate_decoy_database (1)     [100%] 1 of 1 ✔
[-        ] process > openms_peakpicker               -
[-        ] process > search_engine_msgf              -
[72/3160ac] process > search_engine_comet (1)         [100%] 1 of 1 ✔
[00/86f81b] process > index_peptides (1)              [100%] 1 of 1 ✔
[06/51a0d1] process > extract_percolator_features (1) [100%] 1 of 1 ✔
[80/2f443c] process > percolator (1)                  [100%] 1 of 1 ✔
[-        ] process > fdr_idpep                       -
[-        ] process > idpep                           -
[bc/c907dd] process > idscoreswitcher_to_qval (1)     [100%] 1 of 1 ✔
[-        ] process > consensusid                     -
[-        ] process > fdr_consensusid                 -
[42/abfd67] process > idfilter (1)                    [100%] 1 of 1 ✔
[-        ] process > idscoreswitcher_for_luciphor    -
[-        ] process > luciphor                        -
[89/e158d3] process > proteomicslfq (1)               [100%] 1 of 1, failed: 1 ✘
[-        ] process > msstats                         -
[-        ] process > ptxqc                           -
[f4/6023bc] process > get_software_versions           [100%] 1 of 1 ✔
[31/9e5f72] process > output_documentation            [100%] 1 of 1 ✔

But I get this  error:
Error executing process > 'proteomicslfq (1)'

Caused by:
  Process `proteomicslfq (1)` terminated with an error exit status (6)

Command executed:

  ProteomicsLFQ -in file1.mzML \
                -ids file1_comet_idx_feat_perc_switched_filter.idXML \
                -design experimental_design.tsv \
                -fasta Mouse_prot_database_decoy.fasta \
                -protein_inference aggregation \
                -quantification_method feature_intensity \
                -targeted_only true \
                -mass_recalibration false \
                -transfer_ids false \
                -protein_quantification unique_peptides \
                -out out.mzTab \
                -threads 12 \
                -out_msstats out.csv \
                -out_cxml out.consensusXML \
                -proteinFDR 0.05 \
                -debug 0 \
                > proteomicslfq.log

Command exit status:
  6

Command output:
  (empty)

How can I check the meaning of an exit status 6?

I tried looking at the ProteomicsLFQ OpenMS source code but found it hard to find which exit status this is supposed to be.
@timosachsenberg can you help here, maybe?

Exit codes are here:
https://github.com/OpenMS/OpenMS/blob/develop/src/openms/include/OpenMS/APPLICATIONS/TOPPBase.h#L157
This would mean "illegal parameter".
But it is often hard to say where the exit status actually came from. It could also be from the containerization software, or from the scheduler, or from the OS.
It would be better if you could upload the log file in the workspace of the failed process. Nextflow should have told you which folder it is, right after the messages you posted.

If I look inside work/89/e158d3a4b127294b1798ce2f640885 and open the file proteomicslfq.log it says:
Invalid parameter: Spectra files provided as input need to match the ones in the experimental design file.

I double checked and my experimental design file indeed contains the full path the .raw files.

Ah maybe the problem is that I provide file1.raw in the experimental design table while here proteomicslfq has -in file1.mzML ?

I am almost certain that we only care about the basename since we often expect converted files but it is worth a try.
What could be the culprit, is that we currently expect ALL of the files in the experimental design to be present, so unfortunately I think you cannot yet specify a full experiment in the design and then analyze only a subset (here: one file). You would need to filter the design as well.
@timosachsenberg We discussed that multiple times before I think.

I never wanted to specify only a subset. I wanted to quantify the proteins in all my .raw files, that's why the experimental design contains all the files.
What would you recommend trying?
One experimental design per file and launch the pipeline on each individual file? I have 6 samples so I could do that easily.

Ah! It feels like it did not pick up every raw file then.
--input ../data/raw/*.raw
Should yield at least as many "raw_file_conversion" steps as there were raw files recognized by the pipeline. In your case, it only started one process: [5d/b8a76b] process > raw_file_conversion (1)         [100%] 1 of 1 ✔
which means that it probably did only 'match' one "*.raw" file in that folder.

Uhm... that is very weird as the path really shows all 6 raw files.
ll ../data/raw/*.raw

../data/raw/file1.raw
../data/raw/file2.raw
../data/raw/file3.raw
../data/raw/file4.raw
../data/raw/file5.raw
../data/raw/file6.raw

I think I should give it another try then. I'm cleaning up everything and launch the pipeline again.

In the .nextflow.log I check that the command identifies all the input files and it in fact managed correctly:
head .nextflow.log
Nov-08 17:25:03.820 [main] DEBUG nextflow.cli.Launcher - $>
nextflow run main.nf -profile singularity --input ../data/raw/file1.raw ../data/raw/file2.raw ../data/raw/file3.raw ../data/raw/file4.raw ../data/raw/file5.raw ../data/raw/file6.raw --database ../data/reference/proteome/Mouse_prot_database.fasta --expdesign ../data/exp_design/experimental_design.tsv --outdir res_MsdClcg3 --allowed_missed_cleavages 3 -c crg.config --add_decoys

Nov-08 17:25:04.011 [main] INFO  nextflow.cli.CmdRun - N E X T F L O W  ~  version 20.07.1
Nov-08 17:25:04.040 [main] INFO  nextflow.cli.CmdRun - Launching `main.nf` [zen_albattani] - revision: 18f2d39694
Nov-08 17:25:04.072 [main] DEBUG nextflow.config.ConfigBuilder - Found config local: proteomicslfq/nextflow.config
Nov-08 17:25:04.075 [main] DEBUG nextflow.config.ConfigBuilder - User config file: proteomicslfq/crg.config
...

But for some weird reasone nextflow runs only on one sample!
(The first one)
[a6/6abb89] process > raw_file_conversion (1)         [100%] 1 of 1 ✔
[-        ] process > mzml_indexing                   -
[ab/e841f0] process > generate_decoy_database (1)     [100%] 1 of 1 ✔
[-        ] process > openms_peakpicker               -
[-        ] process > search_engine_msgf              -
[75/8af8a7] process > search_engine_comet (1)         [100%] 1 of 1 ✔
[63/5ea7de] process > index_peptides (1)              [100%] 1 of 1 ✔
[bb/38ec7d] process > extract_percolator_features (1) [100%] 1 of 1 ✔
[3e/94b118] process > percolator (1)                  [100%] 1 of 1 ✔
[-        ] process > fdr_idpep                       -
[-        ] process > idpep                           -
[2d/88a2d0] process > idscoreswitcher_to_qval (1)     [100%] 1 of 1 ✔
[-        ] process > consensusid                     -
[-        ] process > fdr_consensusid                 -
[f4/f24c30] process > idfilter (1)                    [100%] 1 of 1 ✔
[-        ] process > idscoreswitcher_for_luciphor    -
[-        ] process > luciphor                        -
[0a/6f6976] process > proteomicslfq (1)               [100%] 1 of 1, failed: 1 ✘
[-        ] process > msstats                         -
[-        ] process > ptxqc                           -
[33/d7b772] process > get_software_versions           [100%] 1 of 1 ✔
[52/66d28b] process > output_documentation            [100%] 1 of 1 ✔
Execution cancelled -- Finishing pending tasks before exit

And it gives me the same error of before.
-[nf-core/proteomicslfq] Pipeline completed with errors-
Error executing process > 'proteomicslfq (1)'

Caused by:
  Process `proteomicslfq (1)` terminated with an error exit status (6)

Command executed:

  ProteomicsLFQ -in file1.mzML \
                -ids file1_comet_idx_feat_perc_switched_filter.idXML \
                -design experimental_design.tsv \
                -fasta Mouse_prot_database_decoy.fasta \
                -protein_inference aggregation \
                -quantification_method feature_intensity \
                -targeted_only true \
                -mass_recalibration false \
                -transfer_ids false \
                -protein_quantification unique_peptides \
                -out out.mzTab \
                -threads 12 \
                -out_msstats out.csv \
                -out_cxml out.consensusXML \
                -proteinFDR 0.05 \
                -debug 0 \
                > proteomicslfq.log

Command exit status:
  6

Command output:
  (empty)

Work dir:
work/0a/6f6976cdc2ae3001d79638134a8aa8

Tip: you can try to figure out what's wrong by changing to the process work dir and showing the script file named `.command.sh`

How come?

Hmm.. Can you try to put quotes around the wildcard expression? It might be that your shell already expands the arguments and nextflow then only picks the first.

Okay --input "../data/raw/*.raw" makes it run on all 6 samples.
However, I still get the same exit status 6 error from ProteomicsLFQ when running on all samples 😔
Error executing process > 'proteomicslfq (1)'

Caused by:
  Process `proteomicslfq (1)` terminated with an error exit status (6)

Command executed:

  ProteomicsLFQ -in file1.mzML file2.mzML  file3.mzML  file4.mzML  file5.mzML  file6.mzML \
                -ids file1_comet_idx_feat_perc_switched_filter.idXML file2_comet_idx_feat_perc_switched_filter.idXML file3_comet_idx_feat_perc_switched_filter.idXML file4_comet_idx_feat_perc_switched_filter.idXML file5_comet_idx_feat_perc_switched_filter.idXML file6_comet_idx_feat_perc_switched_filter.idXML \
                -design experimental_design.tsv \
                -fasta Mouse_prot_database_decoy.fasta \
                -protein_inference aggregation \
                -quantification_method feature_intensity \
                -targeted_only true \
                -mass_recalibration false \
                -transfer_ids false \
                -protein_quantification unique_peptides \
                -out out.mzTab \
                -threads 12 \
                -out_msstats out.csv \
                -out_cxml out.consensusXML \
                -proteinFDR 0.05 \
                -debug 0 \
                > proteomicslfq.log

Command exit status:
  6

Command output:
  (empty)

Work dir:
work/9f/53bcb4f3b030136dbfa938c360936d

And if I check the log
cat work/9f/53bcb4f3b030136dbfa938c360936d/proteomicslfq.log
Invalid parameter: Spectra files provided as input need to match the ones in the experimental design file.

can you post the experimental design file?

It's like this:

Fraction_Group
Fraction
Spectra_Filepath
Label
MSstats_Condition
MSstats_BioReplicate

1
1
/full/path/to/data/raw/file1.raw
1
WT
1

2
1
/full/path/to/data/raw/file2.raw
1
WT
2

3
1
/full/path/to/data/raw/file3.raw
1
WT
3

4
1
/full/path/to/data/raw/file4.raw
1
KO
1

5
1
/full/path/to/data/raw/file5.raw
1
KO
2

6
1
/full/path/to/data/raw/file6.raw
1
KO
3

file1.mzML file2.mzML  file3.mzML  file4.mzML  file5.mzML  file6.mzML
but in the experimental design there are the path to the raw files

file1.mzML file2.mzML file3.mzML file4.mzML file5.mzML file6.mzML
but in the experimental design there are the path to the raw files

Yes because I input raw files and the pipeline does the conversation. I was told that the everything is done using the basename. I'm now trying to change the experimental design table with .mzML extension.

Yes I am pretty sure that it will not look at the extension but it may look at the path

Ok, we checked and the tool expects the extension after the conversion (mzML).
The automated conversion from raw to mzML was only added recently, and we do not yet update the experimental design internally to match it. We will open an issue to fix that problem in the future.

Sorry, for the confusion, it was a misunderstanding of the phrase basename from my side!
The meaning of basename for the tool (and actually in most other use cases on Linux) is indeed the filename with extension.
https://github.com/OpenMS/OpenMS/blob/develop/src/openms/source/SYSTEM/File.cpp#L312
We will provide an automatic conversion of the filenames in the experimental design in the conversion step in the future to address #107

If I create a design table with the full path to "fake" .mzML converted files like this one:

Fraction_Group
Fraction
Spectra_Filepath
Label
MSstats_Condition
MSstats_BioReplicate

1
1
/full/path/to/data/raw/file1.mzML
1
WT
1

2
1
/full/path/to/data/raw/file2.mzML
1
WT
2

3
1
/full/path/to/data/raw/file3.mzML
1
WT
3

4
1
/full/path/to/data/raw/file4.mzML
1
KO
1

5
1
/full/path/to/data/raw/file5.mzML
1
KO
2

6
1
/full/path/to/data/raw/file6.mzML
1
KO
3

The pipeline returns the following error:
Error executing process > 'proteomicslfq (1)'

Caused by:
  Process `proteomicslfq (1)` terminated with an error exit status (6)

Command executed:

  ProteomicsLFQ -in file1.mzML file2.mzML  file3.mzML  file4.mzML  file5.mzML  file6.mzML \
                -ids file1_comet_idx_feat_perc_switched_filter.idXML file2_comet_idx_feat_perc_switched_filter.idXML file3_comet_idx_feat_perc_switched_filter.idXML file4_comet_idx_feat_perc_switched_filter.idXML file5_comet_idx_feat_perc_switched_filter.idXML file6_comet_idx_feat_perc_switched_filter.idXML \
                -design experimental_design.tsv \
                -fasta Mouse_prot_database_decoy.fasta \
                -protein_inference aggregation \
                -quantification_method feature_intensity \
                -targeted_only true \
                -mass_recalibration false \
                -transfer_ids false \
                -protein_quantification unique_peptides \
                -out out.mzTab \
                -threads 12 \
                -out_msstats out.csv \
                -out_cxml out.consensusXML \
                -proteinFDR 0.05 \
                -debug 0 \
                > proteomicslfq.log

Command exit status:
  1

Command output:
  (empty)

Command error:
  /opt/conda/conda-bld/openms-meta_1601515042055/work/src/utils/ProteomicsLFQ.cpp(1555): MzML file in experimental design file '/full/path/to/data/raw/sample1.mzML' not passed as 'in' parameter.
  Note: relative paths in the experimental design file are resolved relative to the design file path.
  Use absolute paths or make sure the design file is in the same path as the mzML files.

Work dir:
work/a9/a1da9142c66acd672fd4c7ef8d4d39

And if fact the error is:
cat work/a9/a1da9142c66acd672fd4c7ef8d4d39/proteomicslfq.log
Error: File not found (the file '/full/path/to/data/raw/file1.mzML' could not be found)

Instead if I use an experimental design table without the full path and the samples with a "fake" .mzML extension like this one:

Fraction_Group
Fraction
Spectra_Filepath
Label
MSstats_Condition
MSstats_BioReplicate

1
1
file1.mzML
1
WT
1

2
1
file2.mzML
1
WT
2

3
1
file3.mzML
1
WT
3

4
1
file4.mzML
1
KO
1

5
1
file5.mzML
1
KO
2

6
1
file6.mzML
1
KO
3

The pipeline succeed and there were no errors!
Hurray!

Great to hear!  Monday, 09 November 2020, 06:24PM +01:00 from Ni-Ar<EMAIL_ADDRESS>:

If I create a design table with the full path to "fake" .mzML converted files like this one:
Fraction_Group Fraction Spectra_Filepath Label MSstats_Condition MSstats_BioReplicate
1 1 /full/path/to/data/raw/file1.mzML 1 WT 1
2 1 /full/path/to/data/raw/file2.mzML 1 WT 2
3 1 /full/path/to/data/raw/file3.mzML 1 WT 3
4 1 /full/path/to/data/raw/file4.mzML 1 KO 1
5 1 /full/path/to/data/raw/file5.mzML 1 KO 2
6 1 /full/path/to/data/raw/file6.mzML 1 KO 3
The pipeline returns the following error:
Error executing process > 'proteomicslfq (1)'
Caused by:
Process proteomicslfq (1) terminated with an error exit status (6)
Command executed:
ProteomicsLFQ -in file1.mzML file2.mzML  file3.mzML  file4.mzML  file5.mzML  file6.mzML
-ids file1_comet_idx_feat_perc_switched_filter.idXML file2_comet_idx_feat_perc_switched_filter.idXML file3_comet_idx_feat_perc_switched_filter.idXML file4_comet_idx_feat_perc_switched_filter.idXML file5_comet_idx_feat_perc_switched_filter.idXML file6_comet_idx_feat_perc_switched_filter.idXML
-design experimental_design.tsv
-fasta Mouse_prot_database_decoy.fasta
-protein_inference aggregation
-quantification_method feature_intensity
-targeted_only true
-mass_recalibration false
-transfer_ids false
-protein_quantification unique_peptides
-out out.mzTab
-threads 12
-out_msstats out.csv
-out_cxml out.consensusXML
-proteinFDR 0.05
-debug 0
> proteomicslfq.log
Command exit status:
1
Command output:
(empty)
Command error:
/opt/conda/conda-bld/openms-meta_1601515042055/work/src/utils/ProteomicsLFQ.cpp(1555): MzML file in experimental design file '/full/path/to/data/raw/sample1.mzML' not passed as 'in' parameter.
Note: relative paths in the experimental design file are resolved relative to the design file path.
Use absolute paths or make sure the design file is in the same path as the mzML files.
Work dir:
work/a9/a1da9142c66acd672fd4c7ef8d4d39
And if fact the error is:
cat work/a9/a1da9142c66acd672fd4c7ef8d4d39/proteomicslfq.log
Error: File not found (the file '/full/path/to/data/raw/file1.mzML' could not be found)
Instead if I use an experimental design table  without the full path and the samples with a "fake" .mzML extension like this one:
Fraction_Group Fraction Spectra_Filepath Label MSstats_Condition MSstats_BioReplicate
1 1 file1.mzML 1 WT 1
2 1 file2.mzML 1 WT 2
3 1 file3.mzML 1 WT 3
4 1 file4.mzML 1 KO 1
5 1 file5.mzML 1 KO 2
6 1 file6.mzML 1 KO 3
The pipeline succeed and there were no errors!
Hurray!
—
You are receiving this because you commented.
Reply to this email directly,  view it on GitHub , or  unsubscribe .

Thanks! We will fix that.


Shadow Database Url

I was attempting to add the Shadow Database URL programatically but it results to the following schema code
datasource database {
  provider          = "postgresql"
  url               = env("DATABASE_URL")
  shadowDatabaseUrl = {"env":""}
}

leading to an error, attempted many different ways, by string and others but it doesn't work as planned
Following is my index.ts for the schemix
createSchema({
    basePath: __dirname,
    datasource: {
        provider: "postgresql",
        url: { env: "DATABASE_URL" },
        shadowDatabaseUrl: { env: "" }
    },
    generator: {
        provider: "prisma-client-js",
    },
}).export(__dirname, "schema");

For reproduction is just having the shadowDatabaseUrl: { env: "" } part in your index and it creates this scenario
Thanks in any advice I will try to fix this myself, if I figure out how to do without doing any kind of pull request will be great but If required I will create an pull request to this repo

Apologies for this bug, for the time being you can do shadowDatabaseUrl: 'env("<NAME_HERE>")', but I will definitely get this fixed ASAP.

Added support in #57 !


"Realm accessed from incorrect thread" after transcoding

When I use the CLI version, all files that need transcoding fail with a similiar error message.
Example:
(393/?) Exporting YOASOBI - Yuusha (TV Size).mp3(transcode required from .ogg)
Unable to set metadata for YOASOBI - Yuusha (TV Size).mp3 :: Realm accessed from incorrect thread.
Exporting will continue.

This does not happen when I use the GUI version.
Steps to reproduce

Start CLI
Under advanced export settings, select "Type 2"
Export audio files

Version: 2.2.2
OS: Fedora Linux 40

Thanks for the report. BeatmapExporter 2.2.3 should fix this behavior.
The transcoded files should have still been exporting. Just the metadata (song artist, title) would have failed to apply to those files, so the result of this bug should've been pretty minor overall. This actually affected both CLI and GUI, but only the CLI displayed the error while the GUI just kept going without waiting for the encode to finish (a behavior I was aware of but thought it was fine at the time). Addressing the "thread" issue also fixes this, so the progress bar makes a lot more sense now.


Revamp autotuning

'runtime' mode for 'core' so that we don't waste time
more tests
enhanced op.apply interface (now can pass basic/aggressive/... directly as op.apply(autotune=aggressive) instead of setting them globally via configuration)

see tests for more info

Codecov Report

Merging #694 into master will increase coverage by 0.1%.
The diff coverage is 86%.

@@            Coverage Diff            @@
##           master     #694     +/-   ##
=========================================
+ Coverage   86.97%   87.07%   +0.1%
=========================================
  Files          97       97
  Lines        9618     9671     +53
  Branches     1944     1956     +12
=========================================
+ Hits         8365     8421     +56
+ Misses       1011     1009      -2
+ Partials      242      241      -1

Impacted Files
Coverage Δ

devito/yask/wrappers.py
86.72% <0%> (+0.47%)
:arrow_up:

devito/core/operator.py
88.46% <100%> (+5.12%)
:arrow_up:

devito/dle/backends/common.py
98.3% <100%> (+0.05%)
:arrow_up:

devito/operator.py
92.66% <100%> (+0.45%)
:arrow_up:

devito/__init__.py
90.9% <100%> (+2.99%)
:arrow_up:

devito/core/autotuning.py
85.35% <84.41%> (+1.87%)
:arrow_up:

Continue to review full report at Codecov.

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

So  if I understand that correctly, autontuning is now done on the fly over the first nblock time stpes?

only when setting the runtime mode. The number of time steps spent autotuning depend on several factors (numbers of combinations attempted, ie basic vs aggressive mode, options[at-squeezer], and other stuff as well)

Ok cool, GTG for me then.

Note: this now depends on #707 , and clearly I want #707 to go in first


Investigate creating selectList lazily

Doing some perf profiling of the settings editor, and I see that creating the Lists for the custom SelectBoxes and calling setOptions shows up as taking much more time than other control types.
It's not the slowest part of the settings editor by far, but it would help if we could wait to create the List until the select box is clicked. I looked into it briefly but see that this.selectList is referenced in a lot of places, so @cleidigh can you investigate what it would take to instantiate the List only when it needs to be shown?
If it turns out to require a lot of refactoring, it's probably not worth the effort.

@roblourens
from the beginning I've noticed a lot of duplicate usually in terms of layout, but I think others as well.
one of the things we have now  are all the callbacks for the detailed descriptions, compounded by
having to do measurements.  I was already thinking about cashing this since it's a lot of duplicate processing.
how does creating the list upon open really change the timing?  we will need the options to be set at rendering time since the select can be used without being opened (keyboard selection)
is the main idea that if you have a page view with five select boxes, now they would instantiate five lists
as opposed to instantiation upon opening?
I will investigate and try to identify any areas for optimization.  it's time anyway with all the new additions
for some rejiggering (I have an aversion to the term re- factoring stems from my firmware days ;-)

couple of starter questions:

are there profiling tools for VSC I can use to measure current and progress
I assume for lazy we construct a list upon open but, do not dispose until settings code request so
can you give some specifics on the other controls for me to reference
the current details provider is completely inefficient in having to do a call for each item , should be done once probably within set options as an option and a single call

we will need the options to be set at rendering time since the select can be used without being opened (keyboard selection)

That is true, hopefully that's possible without creating the list?

is the main idea that if you have a page view with five select boxes, now they would instantiate five lists as opposed to instantiation upon opening?

If you mean "as opposed to instantiation upon opening the page", that's right, they should only create their List when needed.

are there profiling tools for VSC I can use to measure current and progress

I open the developer tools and use the Performance tool to see where vscode is spending time, it's very helpful.

I assume for lazy we construct a list upon open but, do not dispose until settings code request so

Correct

can you give some specifics on the other controls for me to reference

Not sure what you are looking for, I just see that they are created faster than the select boxes, because they don't have to do the heavy work of creating a List.

the current details provider is completely inefficient in having to do a call for each item , should be done once probably within setOptions as a new option and/or a single callback that can return all descriptions

I think it made sense when the select box was asking for one at a time - now you have to get all of them to measure for layout, right? So it probably makes sense to just pass an array of them like setOptions

I did a quickie measurement, I'm sure with lots of inaccuracies but, I saw 15ms for one selectbox instantiation - way too long.
I  have already played around with deferred list creation, shouldn't be that big deal.  That and improving the provider calls should make a big difference.

it looks like render enum gets called more than necessary, setOptions even more so
is there something forcing this in the settings code or is there something I can change within selectBox
to reduce this?  bear in mind I'm not able to get the best info from dev tools based on lack of use
and the difficulty in its use with just the keyboard, I frequently cannot focus on what I need to.
regardless , shifting list creation and tweaking set options will improve both static and open cases

I'm not sure exactly what your scenario is, but it gets called by the constructor with [], then we call it when we set the real options, then we call it from setDetailsProvider (not sure why?).
And the list may be rendered multiple times during startup as config loads, or during search as local and remote searches complete.

@roblourens
I'm happy with this first pass.  PR #58722
Constructor is more than 3.5x faster.  Removed duplicate set options calls
Next I'm going to change the provider function such that you either pass the descriptions in the constructor or pass them with set options.  I don't see any reason to use a callback and the descriptions
should always be available at the same time the options are unless you can think of some problem with that.

Passing them with setOptions sounds ok.

export interface ISelectBoxOptionEntry {
	optionEntry: string;
	optionEntryIsMarkdown?: boolean;
	optionEntrySecondary?: string;
	optionEntrySecondaryIsMarkdown?: boolean;
	optionEntryDetails?: string;
	optionEntryDetailsIsMarkdown?: boolean;
	isSelected?: boolean;
	isDefault?: boolean;
	isDisabled?: boolean;
	optionEntryAriaLabel?: string;
	optionEntryAriaLabeledby?: string;
}

I don't think optionEntry should support markdown - that's the value of the enum right? Same with the secondary. It makes sense for those to be just simple strings.
What is optionEntryAriaLabel/labeledBy? Aren't those the same for every entry, like just the value and details?

I guess I was thinking allowing MarkDown for the optionEntry would allow for more customization of option entrys.  If that seems like it is not useful, I will go with the simple strings.
I also have to add describedby.  The idea is that you might want to refer to an element by ID for a series of IDs.  These may not be within the select box itself.  I'm still playing with exactly how I want to structure this.


Remove '=for openssl ifdef'

No longer needed after rewrite of cmd-nits
Depends on #15298

Nice.
Consequently, please also remove
        if ( /=for openssl ifdef (.*)/ ) {
            foreach my $f ( split / /, $1 ) {
                #$localskips{$f} = 1;
            }
            next;
        }

from find-doc-nits.

Consequently, please also remove

Done.  I would have thought it would be in the other PR, but sure.

This pull request is ready to merge

Merged after fixing conflicts.  Thanks.


Styles on Body don't work for Microsoft Edge

In Microsoft Edge, putting the style on the body doesn't impact the rendering of certain elements (e.g. the toolbar and notices on CNN.com).

If the style is changed to target the /html/ node instead, the effect is applied to the entire page.
Unfortunately, the document DOM node doesn't seem to have a .classList object on it, so I'm not sure if this is a trivial fix?

It's what I expected, it's the fact that they use fixed positioning and we changed a few releases ago to match webkit in producing a new layer for fixed positioned elements for mobile compat (we would end up with mobile menus sliding out under content). My suspicion is that there is probably a optimization heuristic on the compositer that this is falling into due to that layering; because if you remove the position: fixed it works as expected.
www-style thread linking seems to be broken however


Kiln Support

Closes #16
Changes:

Integrate Kiln network
Update templates structure to reuse docker-compose's services templates for testnets. To add a new network or testnet with the configuration settings similar to Kiln, Sepolia and Ropsten, you need add templates for the .env files and update a NetworksToServices map in configs pkg
Handle jwtsecret
Handle fee recipient
Upgrade keys command to generate keystore_password.txt in generated validator keystore folder and to use nethermindeth/staking-deposit-cli docker image instead of building it from staking-deposit-cli repo
Update README
Change some select prompts to confirm prompts ([Y/n])

Types of changes
What types of changes does your code introduce?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Documentation Update
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Other (please describe):

Testing
Requires testing

[x] Yes
[ ] No

In case you checked yes, did you write tests?

[x] Yes
[ ] No

Codecov Report

Merging #46 (94a5f27) into develop (7b1f396) will decrease coverage by 4.70%.
The diff coverage is 76.27%.

@@             Coverage Diff             @@
##           develop      #46      +/-   ##
===========================================
- Coverage    82.33%   77.63%   -4.71%
===========================================
  Files           27       31       +4
  Lines         1302     1529     +227
===========================================
+ Hits          1072     1187     +115
- Misses         186      297     +111
- Partials        44       45       +1

Flag
Coverage Δ

unittests
77.63% <76.27%> (-4.71%)
:arrow_down:

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

Impacted Files
Coverage Δ

internal/pkg/commands/cmd_runner_helper_unix.go
75.00% <ø> (ø)

internal/pkg/commands/commands.go
100.00% <ø> (ø)

internal/ui/tables.go
80.64% <0.00%> (-19.36%)
:arrow_down:

cli/keys.go
52.57% <54.38%> (+1.41%)
:arrow_up:

cli/cli.go
50.81% <63.41%> (-0.89%)
:arrow_down:

cli/cli_utils.go
71.31% <100.00%> (-16.93%)
:arrow_down:

cli/listClients.go
89.18% <100.00%> (+0.95%)
:arrow_up:

cli/listNetworks.go
100.00% <100.00%> (ø)

configs/get_clients.go
100.00% <100.00%> (ø)

configs/init.go
100.00% <100.00%> (ø)

... and 16 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 7b1f396...94a5f27. Read the comment docs.


Possibility to set Time Zones

My time zone is CET. In the Web-Frontend, the displayed time in the upper right and in some charts is CET-2h. It would be great to have the possibility to set the time zone somewhere in the config file such that it is consistent all over airflow.
On a job execution level, all timestamps seem to be in the right time zone.

Related to issue #227.


I can't access the shop.

When I open shop, it will just load infinitely.
Both folders in may /LocalSave/ are updated.

FileLoadError.md states all known fixes for loading issues. If you resolve your issue in a way not listed feel free make a PR or describe it and I'll add it to the list.


Allow task to be duplicated to another project upon task creation

Small modification to enable the TaskDuplicateAnotherProject action to make use of the task creation event.
How might this be used?   I have a column for "Today" tasks in my main project. I have another project specifically for working with my today tasks. The today project has columns for the hours of the day, and I move the tasks into the hours column that I plan to work on them.  The Today project is treated as a workspace, and the main project is the official source.  It's nice to duplicate tasks to the Today project when they are created in the main project's Today column. At the end of each day, the Today project is wiped clean.

Oh heck, I didn't mean to include 3 commits.  If you see what I'm requesting, I'll close this pull request.  Thank you.

https://ariejan.net/2011/07/05/git-squash-your-latests-commits-into-one/


Set RequestContext.isTimedOut(true) on DNS, session, write timeout

Related issue #4935
Motivation:
We need to make sure CancellationScheduler.finishNow(TimeoutException) is invoked on more timeout types,
such as session/connection/DNS timeout so that true is set to ctx.isTimedOut().

On #4935, we found that RequestContext.isTimedout is not set true on DNS, session creation, write timeout so we need to fix it

Modifications:

Set RequestContext.isTimedOut(true) on DNS, session, write timeout by ctx.cancel(TimeoutException.class)

Result:

Now RequestContext.isTimedout is set true well on DNS, session, write timeout too
Close #4935

I think 1 or 2-2 is probably a realistic direction for this PR. Let me discuss with the other tomorrow morning

If there's some discussion progress about direction, feel free to share to me~! 🙇

We have decided to go with the second option. @jrhee17 is laying the groundwork for that.
https://github.com/line/armeria/pull/5212
He will let you know when it's done. 😉

We have decided to go with the second option

aha~ I checked #5212. thanks for sharing!
feel free to let me know when it's ready to go with the 2. option~! 🙇

Since #5212 has been completed, perhaps we can resume this work now?
cc @jrhee17

oh~ I'll update this PR on this weekend. thanks! :)

Since https://github.com/line/armeria/pull/5212 has been completed, perhaps we can resume this work now?

@jrhee17, I updated this PR to latest based on #5212~! PTAL 🙇


Build fails : commit hash 6a80dbc2c74e3af8eff532e8be399122eeee5d14

[ 25%] Building CXX object src/library/hw_identifier/CMakeFiles/hw_identifier.dir/hw_identifier_facade.cpp.o
In file included from /home/nyue/projects/Open-License-Manager/OLM/Open-License-Manager_git/src/library/hw_identifier/hw_identifier_facade.hpp:14:0,
from /home/nyue/projects/Open-License-Manager/OLM/Open-License-Manager_git/src/library/hw_identifier/hw_identifier_facade.cpp:8:
/home/nyue/projects/Open-License-Manager/OLM/Open-License-Manager_git/include/licensecc/datatypes.h:109:20: error: ‘LCC_API_FEATURE_NAME_SIZE’ was not declared in this scope
char feature_name[LCC_API_FEATURE_NAME_SIZE +
^~~~~~~~~~~~~~~~~~~~~~~~~
/home/nyue/projects/Open-License-Manager/OLM/Open-License-Manager_git/include/licensecc/datatypes.h:109:20: note: suggested alternative: ‘LCC_API_PROJECT_NAME_SIZE’
char feature_name[LCC_API_FEATURE_NAME_SIZE +
^~~~~~~~~~~~~~~~~~~~~~~~~
LCC_API_PROJECT_NAME_SIZE
src/library/hw_identifier/CMakeFiles/hw_identifier.dir/build.make:62: recipe for target 'src/library/hw_identifier/CMakeFiles/hw_identifier.dir/hw_identifier_facade.cpp.o' failed
make[2]: *** [src/library/hw_identifier/CMakeFiles/hw_identifier.dir/hw_identifier_facade.cpp.o] Error 1
CMakeFiles/Makefile2:1535: recipe for target 'src/library/hw_identifier/CMakeFiles/hw_identifier.dir/all' failed
make[1]: *** [src/library/hw_identifier/CMakeFiles/hw_identifier.dir/all] Error 2
Makefile:162: recipe for target 'all' failed
make: *** [all] Error 2

There was an incompatible change in the api parameter names. Sorry for that.
You should find the file licensecc_properties.h in your project and delete it if you haven't customized it or rename LCC_API_PROJECT_NAME_SIZE into LCC_API_FEATURE_NAME_SIZE if the file was customized.
The file is not committed to git because it's subject to your customizations and should be automatically generated if not found.

I've added some instructions on the user forums https://groups.google.com/forum/#!topic/licensecc/UJR3vbihOQc
later i'll modify the docs too. Let's close this for now, if you still have the issue please reopen it.


Does dotnet core support server-side web service application?

I'm really sorry to ask this but there is some old project running on iis and we want move to .net core

Yes, it's one of its mainline scenarios. Depending on how old it is and what technology it uses, those technologies may or may not be supported on .NET Core. But .NET Core is absolutely for server-side web services. See here for more info and links to documentation, getting started, etc...

https://www.microsoft.com/net/core/platform

https://docs.microsoft.com/en-us/aspnet/core/

I'm going to close this issue now as I think you got directed to the right place.


Portable configuration of firefox doesn't open links correctly

Bug Report
Package Name: firefox
Current Behaviour

When I open links from other applications, for example someone sends me a link in a messenger, Firefox opens it in a new window and not in my profile.
Expected Behaviour

Firefox opens links in current window in a new tab and in my profile.
Additional context/output

Portable firefox always had this problem.
    "bin": [
        [
            "firefox.exe",
            "firefox",
            "-profile \"$dir\\profile\""
        ]
    ],
    "shortcuts": [
        [
            "firefox.exe",
            "Firefox",
            "-profile \"$dir\\profile\""
        ]
    ],

This configuration assumes that I will be running Firefox from shim or shortcut. But Firefox also registers itself as a default web browser to handle links. And it registers it's currently installed instance, something like this C:\Users\<user>\scoop\apps\firefox\96.0\firefox.exe. As you can see it doesn't include profile argument. It doesn't now it's running in portable mode, it doesn't has information about this portable profile in %AppData%\Roaming\Mozilla\Firefox\profiles.ini so I can't even make this portable profile a default.
Recent PR #7573 made firefox portable by default.
Yes non-portable version had this problem #4377, but it can be solved another way. You can create named profile, make it a default and add argument -p <profile name> to your Firefox shortcut.
Possible Solution

Rollback changes to separate portable and non-portable versions. Firefox currently doesn't has mechanisms to make it fully portable.
System details
Windows version: 10
OS architecture: 64bit
PowerShell version:
Major  Minor  Patch  PreReleaseLabel BuildLabel
-----  -----  -----  --------------- ----------
7      2      1

Additional software: none
Scoop Configuration

{
  "lastupdate": "2022-01-11T10:13:47.6535320+04:00",
  "SCOOP_REPO": "https://github.com/lukesampson/scoop",
  "SCOOP_BRANCH": "master",
  "alias": {
    "st": "scoop-st",
    "i": "scoop-i",
    "ls": "scoop-ls",
    "rm": "scoop-rm",
    "up": "scoop-up",
    "clean": "scoop-clean",
    "show": "scoop-show"
  },
  "shim": "kiennq"
}

This is currently being handled by @amreus in https://github.com/ScoopInstaller/Scoop/discussions/4633

> cd (scoop prefix firefox)
> .\firefox.exe -P
// select your profile and check [use the selected profile without asking at startup] option.
// set your profile as default at 'about:profiles'

> cat $env:APPDATA\Mozilla\Firefox\profiles.ini
...
Name=scoop
IsRelative=0
Path=C:\Users\HUMOR\scoop\persist\firefox\profile
...

i am always stay on my profile, and dont know why extras/firefox(-*) sets launch option -p '$profile' for bin as default.
see also:
https://github.com/HUMORCE/nuke/blob/master/bucket/firefox.json

and dont know why extras/firefox(-*) sets launch option -p '$profile' for bin as default.

I was a little surprised at this change.  I did not think portability was a core goal of scoop.  I thought the persist feature was mostly for apps that stored their config in the apps folder so the configs would not get deleted on upgrade.  I do not see an issue with apps persisting data in %APPDATA% which is an acceptable and expected location for app config files.
With the change to using -profile in the shim, running firefox -P no longer works to bring up the Profiles dialog.  This is a problem for some users as it breaks a firefox feature. I do occasionally use a fresh profile for testing.

Should we remove persist from the manifests then? If many people are facing problem, I'll remove it.


flushAll() stops flushing on first error

The && operator causes conditional evaluation (aka "short-circuiting"). This means if the left-hand-side is false the right hand side is not evaluated. If any call to interp.flushWriter returns false then all streams following that stream in the range, as well as interp.output, will not be flushed.
Removes the type cast to the flusher interface because that is done in interp.flushWriter. This check would be performed twice for any interp.output value whose type implements the interface.
Fixes #208

Let me know if you want me to add regression tests.

Good catch. This looks excellent. I think it's tough to test (even with a white-box test), so let's just leave as is.


failed to cherry pick PR 3686 from cyclinder, to branch release-v0.8

commits 93f2665f28c6ba8e2b746597381ffe7aff165c38 of cyclinder conflict when merging to branch release-v0.8, please manually cherry pick it by yourself.
PR https://github.com/spidernet-io/spiderpool/pull/3686 ,  action https://github.com/spidernet-io/spiderpool/actions/runs/9966977481
Auto-merging docs/usage/install/overlay/get-started-calico-zh_cn.md
Auto-merging docs/usage/install/overlay/get-started-calico.md
Auto-merging docs/usage/install/overlay/get-started-cilium-zh_cn.md
Auto-merging docs/usage/install/overlay/get-started-cilium.md
Auto-merging test/doc/coordinator.md
CONFLICT (content): Merge conflict in test/doc/coordinator.md
Auto-merging test/e2e/coordinator/macvlan-overlay-one/macvlan_overlay_one_test.go
CONFLICT (content): Merge conflict in test/e2e/coordinator/macvlan-overlay-one/macvlan_overlay_one_test.go
error: could not apply 93f2665f... Merge pull request #3686 from cyclinder/coordinator/overlay_policy_routing
hint: After resolving the conflicts, mark them with
hint: "git add/rm <pathspec>", then run
hint: "git cherry-pick --continue".
hint: You can instead skip this commit with "git cherry-pick --skip".
hint: To abort and get back to the state before "git cherry-pick",
hint: run "git cherry-pick --abort".
hint: Disable this message with "git config advice.mergeConflict false"

https://github.com/spidernet-io/spiderpool/pull/3815


YAxis dynamic max value not working

I have a horizontal(!) bar chart with labels and would like for the chart to leave enough space for the labels to render. After some research I found out that the best approach would be to increase the Max value for the Y-axis. When I leave Max empty, the result is this:

Or, with sparkline disabled for clarity:

Now setting Max to 42, like so:
Yaxis = [
	new YAxis
	{
		Max = 42,
	}
],

works as expected and results in the following:

However, when I try to give it a function, it will not work. I tried the following:
Yaxis = [
		new YAxis
		{
			Max = "(max) => max * 2",
		}
],

or
Yaxis = [
		new YAxis
		{
			Max = "(max) => 42",
		}
],

or even
Yaxis = [
		new YAxis
		{
			Max = "42",
		}
],

but these all results in:

What am I doing wrong? Or is this a bug?
Notes:

Max is of type object, so a function or delegate does not work.
I found https://github.com/apexcharts/Blazor-ApexCharts/issues/495 which answered why 1. does not work. However the given answer does not work for me. I would expect it to work similar to the Formatter option for labels.

Hi @joadan, would you be able to confirm this is a bug? Are you able to reproduce this issue?

Hi, we don't support the dynamic function to set min/max, but you have the data so you should be able to set it as an absolute value.  Just calculate the value from your data and set that value.
Sorry if I'm missing something here.

Thanks for your quick reply! This indeed solves my issue. Although it did require an extra parameter on my custom chart component


Not using GPU

The app runs fine but is slow.
I checked in my windows Task manager, it's putting all the load on CPU and nothing on my nvidia 960M.
I'm on win 10.
I tried click right on the app name and running using graphics processor but that does not start the process.
If i double click on app, it start the process but it's too slow
Was anyone able to use it with omegle?

Not sure if that is supposed to work or if it is too old, Avatarify Desktop requirements state that a GTX1070 or better is recommended. How many FPS are you getting?

i didn't check the FPS, but it's quite bad. It's not even running on GPU, atleast it should have tried

If you are getting 3-9 fps that's completely normal for a low end GPU. I don't even think it's possible for Avatarify to run on the CPU since it uses CUDA, but I'm not 100% sure about that

I'm having the same issue.
It looks like the GPU is not being used at all, while the CPU is at 100%. I am using a GTX 1070.
FPS is below 1 (Estimate)

Same problem. Does installing CUDA helps?

GTX 1080 here, 80% CPU usage and 15% GPU usage.

I am having the performance issue, it's too slow

I'm running an RTX 3090 and Avatarify doesn't seem to use the GPU at all. It's also not listed in the task manager as using the GPU.

Avatarify is not installing on my device

this my gforce nvidia experience but avatarify is very slow


Add to CDN.js

Submit a PR to cdnjs

For reference, you can grab this from https://unpkg.com/ouioui or https://unpkg.com/ouioui/dist/oui.min.js


[WFCORE-4026] Add version 5.0.0 of the Elytron subsystem model and schema.

This is for WildFly Core 7
Presently it is not possible to set up transformer tests to transform to WildFly 14 so initially the new tests are set to transform to EAP_7_1_0 but this can be easily switched later rather than delaying adding the tests at all.
https://issues.jboss.org/browse/WFCORE-4026

Retest this please

I have resubmitted as a new PR as GitHub seems to have lost the ability to display an accurate diff!
https://github.com/wildfly/wildfly-core/pull/3526


fix iteration

Fixes #391
with this change, data duplication is gone.
it is iterating over an array, but we are removing elements from that very array, so it causes to skip the next element after removing.

i have applied your suggestion. Thank you.
And I have added a test case.
When i comment out i-- line, the test fails, but with it, it doesn't fail.


feat: requires Node.js >= v18

BREAKING CHANGE: Drop support for Node.js < v18

Related issue https://github.com/expressjs/response-time/issues/25


What does this line of code mean : gen_im_0_1 = (gen_im + 1) / 2

I'm very confused why we need to add 1 and divide  2 here

mapping [-1,1] to [0,1]

@lincong666  Hi I noticed: there is no BN or adaIn or tanH activation in the generator, so how does the network ensure that the activation value is between -1 and 1?


Add an allocation module to the stdlib

This adds a convenient and powerful API for memory management.

I don't really know how to test a memory allocator effectively. Advice would be convenient. Additionally, I haven't finished the testing framework yet, so testing is inconvenient.

I have a few suggestions, take them for what they are: why don't you just add the code directly and try to build on it as much as you can, that is, try to use these lower level APIs in your higher level ones as much as possible so that every API is actually used somewhere, the more calls to it from different contexts the better. Your usage code might give you better insights into how to shape these APIs than humans can. Any good advice you will get from others will come from the same place anyway, that is, people remembering from the experience of using an API why that API was good or bad, eg. I can tell you that memmove is more useful to me than memcpy simply because I needed overlapping copying in enough contexts to justify having only that variant in the stdlib, or I could tell you that I never needed cond with more than 3 args but I did need a case-like (or sql-coalesce-like if you will) macro once or twice (and that btw I prefer iif instead of cond -- in usage you can see why, lacking that, I'll have to resort to explaining and justifying which is far less effective. But this list of little details is so long that it will take forever to shape the APIs with conversations. I think it's more effective for us to write our apps and try to push down common functionality and data structures and whatnot into libraries and if some of them end up being good enough for other people to use, then maybe they'll have a place in a stdlib and then we'll all drink from the holly grail of code reuse.
Anyway, if you do choose to consider this, some implications follow like keeping it light on the docs (a terse but organized API list might be enough at first eg) -- the more docs you write from the start the less willing you'll be to change the APIs later and throw all that work away. Oh and try to communicate in a less justificative manner (eg. cut down on adjectives like "convenient and powerful API"). Sorry if this sounds like editorializing :)


Change Update and Delete methods to use href

All of the resources in our API have an href attribute that points back to the object. Here is an example of an AddOn:
<add_on href="https://your-subdomain.recurly.com/v2/plans/gold/add_ons/ipaddresses">
 <!-- ...... -->
</add_on>

This is helpful because you can store it on the object and the object doesn't need to know how to build back up it's route to update or delete itself. Doing so requires inflexible and unnecessary logic and also it requires that the resource has to track all of it's parent resources and their identifiers. Here is an example in the AddOn model:
https://github.com/recurly/recurly-client-net/blob/master/Library/AddOn.cs#L67-L73
public void Update()
{
  Client.Instance.PerformRequest(Client.HttpRequestMethod.Put,
    UrlPrefix + Uri.EscapeUriString(PlanCode) + UrlPostfix + Uri.EscapeUriString(AddOnCode),
    WriteXml,
    ReadXml);
}

This object must now store and track it's url parts, the PlanCode, the AddOnCode, and the logic to build this all back up. This could be replaced with something like this:
public void Update()
{
  Client.Instance.PerformRequest(Client.HttpRequestMethod.Put, GetHref(), WriteXml, ReadXml);
}

Where Href would be read from the XML and stored as an instance variable.
This would prevent issues like this https://github.com/recurly/recurly-client-net/pull/141 as well as simplify the code.
\cc @csmb @MuffinMan10110 for comments

I'm a fan. It seems like a great way to reduce complexity around locating objects. :+1:

What you propose sounds like a more logical way to go about it.

Okay, I'm going to put this on my calendar for next sprint.

We're not going to implement the HREFs at this time for this library. Closing this and the related PR.


PowerToys automatically exit when I click the author name in What’s new!

Microsoft PowerToys version
v0.66.0
Installation method
PowerToys auto-update
Running as admin
None
Area(s) with issue?
General
Steps to reproduce
PowerToys automatically exit when I click the author's name in What’s new!
✔ Expected Behavior
Open browser
❌ Actual Behavior
Close app
Other Software
No response

I can replicate this. Thanks for opening the issue.

This has been addressed as part of the 0.67 release cycle. Thank you!


Bump versions to 0.45.1

@FuelLabs/sdk-rust I added the build_tx PR to this release as we need it for debugging predicates.


Debugging a CI differential

This PR will be deleted once the bug is fixed.

The Heisenbug was fixed in PR #3380.


Add cautious-robot logo

Adds AI-generated cautious-robot logo to the README. Set it at 500px so it won't be too large on the screen.

Any reason you wouldn't put the logo image into the repo? Also, I would right-float the image. (And you can still put the image at full rez and then resize for display.)

Any reason you wouldn't put the logo image into the repo? Also, I would right-float the image. (And you can still put the image at full rez and then resize for display.)

When I fixed the height, the aspect ratio doesn't resize under 500px (width will be narrower but height stays fixed). I didn't put it in directly so it'd be visible, but I can adjust both.
Curious, why right float? And do you think 500px is a good fixed size?

Curious, why right float? And do you think 500px is a good fixed size?

Without the float, you have this relatively large vertical space that one needs to scroll over without learning anything new. With float, you can have both the logo and the text side-by-side, so the logo artfully embellishes but does not take over the space.
As for the size, 500px is a bit large, and would be especially so on a mobile device. Personally I would instead use relative width (then height will be automatic).

Curious, why right float? And do you think 500px is a good fixed size?

Without the float, you have this relatively large vertical space that one needs to scroll over without learning anything new. With float, you can have both the logo and the text side-by-side, so the logo artfully embellishes but does not take over the space.

Ahh I misunderstood! Yes, definitely a good idea!

As for the size, 500px is a bit large, and would be especially so on a mobile device. Personally I would instead use relative width (then height will be automatic).

Sounds good.

Thanks, @hlapp! That was the style fix I needed :)


Download not supported for MyStream Server. (not an extension issue)

Steps to reproduce
open AnimeWorld India 13.2 (Multi) [it's the only extension which uses MyStream]
select preferred server : MyStream in extension settings. (it's default so you can skip)
try downloading episode which contains MyStream Server (Ben 10 alien force for easiness)
it's likely it'll fail (instant download) unless it picks StreamSB somehow.
Expected behavior
Download episode completely using MyStream.
Actual behavior
Can't download (instant download) episodes From MyStream. (occurs 98% of time 2% could be auto selecting StreamSB due to unavailability of MyStream)
Crash logs
No response
Aniyomi version
preview: r6151; beta: r6453, r6473
Android version
Android 12
Device
Iqoo 3 5G
Other details
it seems ffmpeg's issue since it can stream/play online but can't download. which means url extraction is working properly.
closed issue of MyStream in aniyomi-extensions: https://github.com/aniyomiorg/aniyomi-extensions/issues/1904
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have tried the troubleshooting guide.
[X] I have updated the app to version <IP_ADDRESS>.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

Me too but nothing

Me too but nothing

sadly only @/jmir1 can fix it


Oomph init script for cobigen-eclipse + cobigen-typescript plugin does not work

Expected behavior
After importing automatically all projects in the cobigen-development branch, the projects
cobigen-eclipse, cobigen-eclipse-test, cobigen-typescriptmerger should be intitialized. Meaning there are no build plugins and I can directly run tests within eclipse of these projects.
Actual behavior
During run of the maven-init script, it seems, that the eclipse projects are not build properly. So the two projects keep their build errors after import. In addition, the typescriptplugin build is not running properly.
I am not sure why this happens, so part of this issue is to investigate it. Just a hint: one issue might be the fact, that the maven profiles for building eclipse projects have been changed. See deployment section for currently valid profiles. E.g. p2Bundle has been replaced by p2-bundle or experimental has been replaced by p2-build-mars,p2-build-experimental -Dp2.upload=experimental
Another potential issue might be the fact, that the process is not running with the correct PATH environment including the login credentials for the nexus. This would then lead to authorized or authenticated failed messages. Please check.
Steps to reproduce (bug) / Use Case of feature request (enhancement)

create new installation with oomph and clone CobiGen -> development stream
start cobigen-development eclipse after installation and wait for automatic setup
wait until everything has been setup and the projects are cloned. Keep the oomph window open
see the log of the oomph init setup. Three projects as mentioned above should have been initialized but the build always fails.

Related/Dependent Issues

Closing as it is already fixed


Get data from Images in asset manager

Hi,
Is it possible to get the probe-image-size results from image urls like this:
https://www.debijenkorf.nl/content/dam/images_debijenkorf/webshop/campagnes/creatieve-campagnes/2022/zomer/hero/flight-2/home/1.1_home_topbanner__hero_celebratelove_flight2_5120x1700.jpg.gravity/780x312-49.3880489560835x42.3160173160173
Because of the parameters after the .jpg a different size and focus is dynamically generated from the base image (https://www.debijenkorf.nl/content/dam/images_debijenkorf/webshop/campagnes/creatieve-campagnes/2022/zomer/hero/flight-2/home/1.1_home_topbanner__hero_celebratelove_flight2_5120x1700.jpg).
I get an error when probing the url with parameters after the filetype. Wondering if should create an exception in my application, or if this use case might be something to add in probe-image-size.

CLI:
./support/dump.js https://www.debijenkorf.nl/content/dam/images_debijenkorf/webshop/campagnes/creatieve-campagnes/2022/zomer/hero/flight-2/home/1.1_home_topbanner__hero_celebratelove_flight2_5120x1700.jpg.gravity/780x312-49.3880489560835x42.3160173160173

As far as i see, server returns 403 status (fobidden). That means it does not like request. Usually, that means extra restrictions, based on

user agent
ip address
referrer

If you wish to investigate, i'd suggest to inspect firefox headers, and create the same list for prober request. Then repeat requests and drop headers one-by-one until find what is critical for that server.
With high probability problem is with user agent.

Thanks for your answer Puzrin!
Good to know 'probe-image-size' would normally recognize file extensions with additional values(.jpg**.gravity/7**...).
Took the headers of an image which got a good result in the probe (https://www.debijenkorf.nl/content/dam/images_debijenkorf/webshop/system/Thuiswinkel_Waarborg.png) and added those  to the probe script, dropped headers and changed many of them. But no luck so far. Think Cloudflare is detecting Playwright.  Strange thing is that only images with parameters give an error, and the images without extra values after the filetype end up giving a good result.
One question about the cli : could you describe how to run it?
I am on Windows.

Made a folder in my project /support/
Copied dump.js from your repo into the folder
Ran the command in terminal
No luck. Hope you can share some pointers.

Clone the repo, then run npm i to install dependencies, then run dumper script with desired params. That's a standard way for all nodejs projects.

Oke thanks!


Shouldn't the "name" property of the NamedEntity Bean be non nullable?

	@Column(name = "name", nullable = false)
	private String name;

I'm not super excites about changing this. It's more likely to lead to other bugs showing up unexpectedly, and doesn't really change the behaviour of the app as far as I can see.

Ditto.


extended context to enable implementation of swiftlike scoping rules …

…(pushLocals function)
added a testcase for this new behavior
modified ForTag to make use of this new mechanism

Hi @kaygro, I'm having a hard time trying to understand what you are trying to solve that is different from the push method on Context which already takes a closure. Could you perhaps elaborate and perhaps provided an example use-case where the regular push method does not suffice?

If you want to to implement an extension, which has set tag (as StencilSwiftKit for example),
you want to be able to do mutations inside a for loop, which are visible outside of it.
A resonable implementation of a set tag would do something like
context[variableName] = value,
but if that happens inside a for loop, it gets popped off afterwards when using regular push.

Exampe Stencil code for this scenario would look like:
{% set foundElem %}false{% endset %}
{% for elem in array %}
{% if satisfiesInterestingProperty elem %}
{% set foundElem %}true{% endset %}
{% endif %}
{% endfor %}
{# if normal push/pop is used, foundElem will always be "false" at this point #}
{% if foundElem == "true" %}
...
{% endif %}

Not sure about this change, feels too invasive, also I think it can be implemented in a simpler way, but I didn't explore it enough to be sure. Also I think current behaviour is the same as in Jinja.
Speaking of example shared, I believe it can be achieved by using filter on array and then checking its count.
{{ if array|yourCustomFilterThatChecksForPredicate|count > 0 }}

Of course you'll need to implement this filter and register it in environment, so there is no way to do it just with templates.
Another way would be to use filter that I've implemented in #189

Well, I think it can be implemnted entriely from within the set node, but that would mean, the implementation of the set node would need to make copies at the various heights of the context stack, where that variable is visible, (for every variable change that is). It would be less cubersome, if we could observe/override the subscript operators of the Context class, but that is not possible due to access beeing public, not open and Context not being a protocol.

@kaygro I get what this is trying to achieve (seeing the issues with set), but I agree with the others that this might not be the best way to achieve this.
It might be better to have a mechanism in Context to directly modify the stack, so that set (for example) could modify a variable outside it's scope. For example

I like that idea. I didn't realize, you could have multiple parameters in subscript operators.

Closing this due to inactivity.


MTP40-F I2C test

From the MTP40-F datasheet:
1.Send a start signal;
2.Send an address to write(slave address + R/W=0x64) and check responses;
3.Send a read command (0x03) and check the responses;
4.Send a stop signal;
5.Send a start signal;
6.Send an address to read (slave address + R/W(1) =0x65) and check responses;
7.Read 3 bytes from the module and send responses;
8.Send a stop signal.

Structure of response:
CO2 concentration = high byte of CO2 concentration*256 + low concentration byte
Data valid byte, 0x00 means data is valid, 0xFF means data is invalid.

Thanks for the PR, will look into it later today.

@M66B
Merged your comment into master branch, will not make a new release for it.
Downloaded the MTP40-F datasheet as I only had the C and D version.
This (MTP40F) datasheet gives slightly better info about the I2C interface. (request 3 bytes instead of 7.
Could you give this a try?
It would be slightly faster as it uses less data.
uint16_t readMTP40_F(uint8_t address)  //  address 0x32
{
  Wire.beginTransmission(address);
  //    Wire.write(0x52);
  if (Wire.endTransmission() != 0) return 0;

  if (Wire.requestFrom(address, 3) == 3)
  {
    uint16_t ppm = Wire.read() * 256;
    ppm += Wire.read();
    uint8_t status = Wire.read();
    if (status == 0xFF) Serial.println("Invalid PPM");
    return ppm;
  }
  return 0;
}

I look into ordering an MTP40_F for additional testing.
I assume that the 0x52 in the original demo code identifies a register, so it would be interesting to see if other "register addresses" provide additional data.
PS, you're from Netherlands? I live in Gemert.

@M66B
Q: did you test the TTL rs232 interface?
if so which library?
I am considering a new library for the F model, but I need to order hardware.

@RobTillaart I don't think it will work without command code, but I'll try tonight.
Via RS232 it works. I can share some Python for this.
If you contact me via here, we can discuss via email:
https://contact.faircode.eu/?product=other

Just changing some command strings in a copy of this library should work (sort of).
Might write a first version of the lib tonight although I planned to add two multiplexer classes first.
I might need them in a project so they have priority.
Just ordered my MTP40F (and some more) at tinytronics.nl, hope to find time later this week "to play".

This is very basic, but it works:
https://gist.github.com/M66B/863b2ab8f36c51b2443e9db860c15a72
I switched to a Wemos D1 mini V4 in the meantime.

A disadvantage of the MTP40-F is that the maximum value is 2000 ppm. This can easily be reached in a room with many persons.

@RobTillaart without command code, the gas concentration / valid indication seems to be received via I2C. It is getting out of sync, though. With another command code than 0x03 (=read gas concentration) all zeroes are being received.
Wire.beginTransmission(address);
// No command code
if (Wire.endTransmission() != 0) return 0;
if (Wire.requestFrom(address, 3) == 3)

Thanks for this info, helps to prep examples for the mtp40f library.
Yesterday I compared datasheets and the Serial commands and answers are definitely different.
Also did a small test to replace the (fast) CRC algorithm with a much smaller function to save RAM.
To be continued

I've just rewired to use RX/TX instead of I2C because it looks like I2C can be used to read value only, and not to calibrate, etc. I was looking into converting the Python code into cpp just now ...

Created a placeholder for the MTP40F library - https://github.com/RobTillaart/MTP40F
No hardware in yet, but I try to prepare code as much as possible, including build-CI and examples.
When I get the hardware I can patch the code where needed.
The library is based on MTP40C library, some parts of the interface are different or not available.
First task now is to get all examples compiling on the MTP40F interface.

I had no success in communication via RX/TX (which I had wired to MISO/MOSI). So, I rewired to I2C too.

https://github.com/RobTillaart/MTP40F/pull/1  build is successful

develop branch is "stable" as all examples compile.
unit test is limited, however as far as testable no errors.
LINT fails as library is not registered at Arduino libs yet.
test if the lib works as intended can be done when hardware arrives.

I tried again with pin D0/D5, taking care of what is RX and TX, and there was still no response.

FYI, I tested the CRC to use less RAM - https://github.com/RobTillaart/MTP40C/issues/9 - worked!

I tried again with pin D0/D5, taking care of what is RX and TX, and there was still no response.

First tests with the library shows nothing but an error 2.

I2C tested, first I got nothing. but with this code it works - Note the Wire.write(0x03) !!
uint16_t readMTP40F(int address)
{
  Wire.beginTransmission(address);
  Wire.write(0x03);
  if (Wire.endTransmission() != 0) return 0;

  if (Wire.requestFrom(address, 3) == 3)
  {
    uint16_t ppm = Wire.read() * 256;
    ppm += Wire.read();
    uint8_t status = Wire.read();
    if (status == 0xFF) Serial.print(" - error - ");
    return ppm;
  }
  return 0;
}

==>
PPM: 684
PPM: 684
PPM: 686
PPM: 693
PPM: 2000
PPM: 2000
PPM: 2000
PPM: 2000
PPM: 2000
PPM: 2000
PPM: 1533
PPM: 1549
PPM: 1567
PPM: 1645
PPM: 1646
PPM: 1641
PPM: 1632
PPM: 1575
PPM: 1557
PPM: 1429
PPM: 1177
PPM: 1164
PPM: 1152
PPM: 1104
PPM: 1093
PPM: 1082
PPM: 888
PPM: 864
PPM: 873
PPM: 873
PPM: 871
PPM: 869
PPM: 867
PPM: 865
PPM: 863
PPM: 860
PPM: 857

I have this working for some days already, but what isn't possible and disturbing, is that it isn't possible to calibrate, etc. via I2C.

plotter.

I have this working for some days already, but what isn't possible and disturbing, is that it isn't possible to calibrate, etc. via I2C.

yep, but at least got some information out of it, so I know my sensor works
(at least partially).

last hunch got serial working (sort of)
...Arduino\libraries\MTP40F\examples\MTP40F_getCO2_SW_SERIAL\MTP40F_getCO2_SW_SERIAL.ino
MTP40F_LIB_VERSION:	0.1.0

TIME	CO2 LEVEL
5000	XX: 915
10000	XX: 913
15000	XX: 909
20000	XX: 904
25000	XX: 865
30000	XX: 863
35000	XX: 860
40000	XX: 861
45000	XX: 934
50000	XX: 1657

I have to carefully analyze what I changed ...

Can you please share the source code? And also how you connected the sensor?

I need to clean code up today, what I did was remove the CRC algorithm and add hardcoded CRC values.
So at least in that corner there is a BUG in that area.
Think I have fixed the CRC.
D:\Rob\WORK\Arduino\libraries\MTP40F\examples\MTP40F_getPressureReference\MTP40F_getPressureReference.ino
MTP40F_LIB_VERSION:	0.1.0

TIME	PRESSURE (mbar)
5000	1013
10000	1013
15000	1013

will push code to develop soon, but further testing is needed

Pushed code to develop branch.
not all functionality is tested  - only getCO2 en get/set air pressure reference.
Other functions might need some patching in "data reconstruction"
I have other tasks now, to be continued later.

Tested most of the examples, looks like the library now works correctly.
Will merge asap.

That's nice to hear. Can you please tell how you connected the sensor?

Merged develop into master.
Build a first release asap

That's nice to hear. Can you please tell how you connected the sensor?

              UNO                     MTP40F
        +---------------+        +---------------+
        |               |        |               |
        |          +5V  |--------| 1 VIN         |
        |          GND  |--------| 2 GND         |
        |               |        | 3             |
        |               |        | 4             |
        |               |        |               |
        |               |        | 5             |
        |         RX 6  |--------| 6 TX          |
        |         TX 7  |--------| 7 RX          |
        |               |        | 8             |
        |               |        | 9             |
        |               |        |               |
        +---------------+        +---------------+

  use SoftwareSerial sws(7,6);

@M66B
Created - https://github.com/RobTillaart/MTP40F/issues/2
to discuss the new library as it does not belong here.


Integrated terminal is not executing pdftk command

As tittle says integrated terminal of visual code not recognizes pdftk command, it's already on the path. I can check pdftk --version on bash or cmd directly on windows but not in the terminal of vs code or even when i'm debugging. Pdftk is installed on windows as server and is used to merge pdfs and is on windows path
Actually I have read similar issues, but none of that solutions have worked.

"version": "0.2.0",     "configurations": [         {             "type": "node",             "request": "launch",             "name": "Launch Node",             "program": "${workspaceFolder}\\bin\\www"         }     ]

If the shell is saying it cannot find the command then it doesn't exist on the PATH. Try investigating what $PATH is within the terminal and try restart VS Code completely.


Website should have community channel information

Filing here because there doesn't seem to be a repo for the website? (where is the website content?)
Currently, the website has no information on how to join the various mailing lists, attend community meetings, or anything else.  This would be the same information that's present here  It only has a "contact us" form.  This is a bad approach; new community members will prefer to opt-in, rather than signing up for spam.

@jameslabocki this would be something to link on konveyor.io


bugfix: move inline_table_scan after subquery-unnest.

Which issue does this PR close?
Closes #5364.
Rationale for this change
What changes are included in this PR?
move inline_table_scan after subquery-unnest.
Are these changes tested?

Are there any user-facing changes?

@Dandandan how do you think about this change?


workspaceFolder path issue under Windows

Under Windows, it seems that the path using ${workspaceFolder}/ generates an issue.
See below.

The problem is solved when using relative path.

Yep, I've just been hit by this, too.  To clarify the workaround by @eymeric-giraux, I solved the problem by using the following settings.
{
    "sphinx-needs.needsJson": "build/needs/needs.json",
    "sphinx-needs.srcDir": "source"
}

BUT doing this makes the "Sphinx-Needs Explorer" view go "blank", whereas if I put
{
    "sphinx-needs.needsJson": "${workspaceFolder}/build/needs/needs.json",
    "sphinx-needs.srcDir": "${workspaceFolder}/source"
}

then the explorer view works, but the above error appears and auto-complete, etc., don't work.


use process.nextTick for handleCallback

Prevent Maximum call stack exceeded by calling all callbacks on nextTick

backed this out as it break node 0.10.x and earlier support. can you provide a code example that causes the issue as no-one else seems to have run into this.


Implement OverflowBox Widget in the Widget Catalog

OverflowBox Widget needs to be added to the Widget Catalog. Read How to contribute to the Widget Catalog in the README for detailed steps. Make sure to include all the arguments(name, implementation, description, codeString, link and category). Refer to any widget in widgetList.dart and its own *widget*.dart file for better understanding. Showcase all the properties of the widget and make sure the implementation clearly shows the usage of this widget. (Refer to the official docs in case of any doubts)
Please attach screenshots of the same once done in the PR.

Sure, good luck!


Non-python asset pipeline

This is some work done towards moving away from the python asset pipeline.
Still WIP pretty much.

is this ready for merge? I'm happy to clear out the conflict, just need to know if it works :)

@pudo if you have some time tomorrow, please take a look at this PR. I'll be finishing the docs in the meantime.


Fix README.  Update Payload custom_detail object to enforce JSONObjec…

Hi Daniel,
I have addressed the bullet points mentioned in the email.

updated


add new CI config

tests compatibility with xarray 0.12.0, fixes #124

You will also need to edit .travis.yml.
Now that xarray 0.12 is released, it is the default in the other envs as well. If we want to maintain backwards compatibility, we will have to explicitly specify old xarray versions (e.g. 0.11.3, 0.10.9).
We might also consider adding an env that tests against the latest xarray master (via pip + github). This would have allowed us to catch #123 before the xarray release.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #125   +/-   ##
=======================================
  Coverage   93.61%   93.61%
=======================================
  Files           4        4
  Lines         986      986
  Branches      223      223
=======================================
  Hits          923      923
  Misses         35       35
  Partials       28       28

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 4f010ef...92c603c. Read the comment docs.

Fantastic! Thanks for putting all of this together!
A future conversation we should have is when to drop python 2 support.


Installation script not working properly...

Hi,
tried to install Octoprint on an Odroid C2 with Armbian.
Something was running, but there where some errors in between.
I found out, that in the installer script in line 65 ("-e python3-setuptools") a backslash is missing.
After adding the missing backslash, the script seems to run properly now.
Regards, Thorsten

Thanks! Fixed.


Color Picker not working as intended

I ran the example code, however, the Color picker/dropper is not working. I am on the current latest Flutter version 3.10.0

Yes indeed, is it fixed in 0.7.0 ?


Sync autoloads deprecated

[ProvideAutoLoad(UIContextGuids80.SolutionExists)] attribute causes following error in VS2019 16.1:
"Synchronous auto loads will be deprecated in future versions, consider using BackgroundLoad flag and AsyncPackage base class."
Please update documentation for version 16.1

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

ID: f7aca15b-bfa6-33ac-8f45-78d5d6c4ffc0
Version Independent ID: c3dc31a4-cf7a-4b25-8e39-91cb19565203
Content: Creating an Extension with a VSPackage - Visual Studio
Content Source: docs/extensibility/creating-an-extension-with-a-vspackage.md
Product: visual-studio-windows
Technology: vs-ide-sdk
GitHub Login: @gregvanl
Microsoft Alias: gregvanl

Thanks for submitting your feedback and we apologize for the delayed response. Due to current resourcing and substantial extensibility changes we are currently working on that will result in significant doc changes, we are closing this lower priority issue. We will return to this issue on a later date and address it based on it's relevance after those extensive extensibility updates are made. Feel free to comment here as necessary.


Add cargo-deadlinks

cargo-deadlinks helps prevent stale and broken links in the documentation.

Unfortunately, cargo-deadlinks is currently missing an important feature from my perspective. https://github.com/deadlinks/cargo-deadlinks/issues/52


Deploy v2.82.2+rev2 to production

This release contains a NFSD kernel configuration that is required by a customer's application.

Test plan available in https://resinio.testlodge.com/projects/16238/runs/608734?tab=0

Will be released with BalenaOS version v2.82.10 from https://github.com/balena-os/balena-iot-gate-imx8/pull/23

v2.82.10+rev1 released to production for iot-gate-imx8


Engine selection

Fix #29

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 4 (50.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 81.649%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/qa4sm_reader/img.py
2
4
50.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
1160

Relevant Lines:
1363

💛  - Coveralls


Added grunt server and new serve command

@joshdmiller would be great to have this module by default, I think many people are having issues to run the app.


use current mercurial plugin version

Found while investigating ros-infrastructure/ros_buildfarm#488.

This repo should be audited for plugin versions as this doesnt seem to be the only outdated one

While I don't want to shut down the idea of keeping contents here in better sync with reality. Two factors make it less of a priority. First: puppet cannot currently be re-run on the master host without clobbering additional configuration, which means that changes to the master config files do not get propagated or reset beyond the initial configuration.
Second, in order to allow the master to be safely reconfigurable I'd like to replace templated configuration files in puppet with use of Jenkins console scripts.


"Edit this page on GitHub" links broken

At the bottom of website pages e.g. https://docs.sigstore.dev/cosign/usage/ there is an "Edit this page on GitHub" which currently 404s.
An example link is https://github.com/sigstore/sigstore-website/tree/docs/edit/docs/content/en/cosign/usage.md
I think this should be https://github.com/sigstore/sigstore-website/edit/docs/content/en/cosign/usage.md or similar.

CC @dlorenc

Resolved with #193

#193 reintroduces #81, the issue that #88 fixed: the GitHub icon in the right hand corner now links to https://github.com/sigstore/sigstore-website (the main branch, not the docs branch).

See my above comment for a solution that handles both (but is substantially more work). A much simpler thing to do would be to make a new sigstore/docs repo and migrate the docs branch there. There's already some support for doing so.

Thanks, @znewman01 for surfacing this. My thought was that the GitHub logo going to the main branch is not ideal but less of a concern compared to the completely broken "edit this page" link (going to a 404), and I agree there should be a longer-term fix. Though I don't have data about the click-through rate for these which could potentially inform a better solution.
Thank you for surfacing that previous conversation on repos — I also have been thinking that a separatedocs repo will lower the barrier to entry for docs contributions. I will write some more details about that for feedback.


FrontEnd Mutations

REMOVE_WORKOUT ( _id, username, email, savedWorkouts(workoutId, title, description) )

completed


Default timeout

Hi, could it be possible to pass a timeout argument instead of hard-coded it to 10?
https://github.com/richardpenman/whois/blob/d2984dfeab630f7f02b0830487e8786f34f7ce6d/whois/whois.py#L178

Definitely, you're right that shouldn't be hardcoded. I would gladly accept a PR for this.

Is this released? (https://pypi.org/project/python-whois/#history)


Add icons to navbar content

Feature Description
Add icons to navigation menu items (e.g. Home, Contact Us) to enhance user experience and provide a modern look on the movie ticket booking website.
Adding icons next to each navigation item will make the menu visually attractive, breaking the monotony of text-only navigation.
Icons can convey meaning quickly, allowing users to identify sections of the website at a glance.
Icons can enhance the intuitive nature of the navigation menu. Users often associate certain icons with specific actions or pages (e.g., a house icon for Home, an envelope for Contact Us).
This improvement can lead to faster navigation, as users can easily find what they are looking for without reading through text.

@codervivek5
Please Review - Add icons to navbar content

I  would like to contribute to this ,please assign me the task.

can you plz label this issue i want to work on this issue , plz assign this to me

can you please assign this task to me....I would like to solve this issue


Migrate the source code to the Next.js v13 API

Next.js v13 comes with a lot of goodies and useful features. The entire list of provided features is impossible to share here, so instead its recommended to checkout the release notes.
This thread will be used to keep track of the progress made to migrate the source code to the latest version of Next.js over time. Do note, the source code already relies on Next.js v13 but thanks to the backwards compatibility, I have not faced any major concerns yet. That said, some advantages of migrating to the latest version are:

Out of the box sitemap.xml generation and much more like OG and Twitter graph handling.
Improved and easier routing which no longer relies on functions like getStaticProps and such.

....there are many other advantages of migrating which is impossible to list here in its entirety so I will share a release note some time later then.
NOTE: Follow the official migration guide for a detailed walk-through of the process.

Closing since its already complete!


Can't uninstall or reinstall ubuntu.

Version
Microsoft Windows [版本 10.0.19044.1348]
WSL Version

[ ] WSL 2
[ ] WSL 1

Kernel Version
No response
Distro Version
No response
Other Software
No response
Repro Steps
I reset Ubuntu 20 on the setting page, then I found I can't open ubuntu.
So I tried to install Ubuntu on Microsoft Store, it infor me something happened on ...(I can't read '...')
If I try to uninstall it, nothing happened. (I can't find Ubuntu on the applicaiton list).
Now I have uninstall WSL, but it still show on my start menu.
Expected Behavior
Please help me purge it and reinstall Ubuntu
Actual Behavior
Nothing.
Diagnostic Logs
No response

Have you tried to install the Ubuntu instance directly over PowerShell?
wsl --install -d Ubuntu-20.04
When the instance is rolled out properly, you should see it in the list:
wsl --list
You can run the session with (When its your default session.):
wsl
I think the package itself should be saved under this path: %LOCALAPPDATA%\Packages
Example from my computer: C:\Users\myUserName\AppData\Local\Packages\CanonicalGroupLimited.Ubuntu20.04onWindows_79rhkp1fndgsc
You can also try to share the error codes you got. :)
You can also check out the doc to WSL


Doesnt work with Q promises

I have a method which uses Q promises, like:
function testFn()
{
. . . . .
    promise = getPromise({}) // returns Q promise

   promise.then(function(data){
         console.log(data)
         otherFn();
   })

 . . . . .
}

function otherFn()
{
    console.log("Called")
}

I'm using karma for my test cases, so I used this version of sinon-as-promise (https://github.com/bendrucker/sinon-as-promised/blob/v3.0.1/release/sinon-as-promised.js).
My test case looks like this:
spyCalledFn = sandbox.spy(window,"otherFn");
var sinon = window.sinonAsPromised(Q.Promise)
var stub = sinon.stub(window,"getPromise");
stub.resolves('foo')

testFn()

assert.equal(spyCalledFn.calledOnce,true)

when the testFn is called, the following get printed:
foo

Called // <---- otherFn is called

However the Called statement gets printed, the assertion on the line:
assert.equal(spyCalledFn.calledOnce,true)

fails. I tried callCount method too, it says 0. But  the console is printing its get called, not really sure whats going wrong here. Why Sinon couldn't able to spy this function?
Where I'm making mistake?

Promises resolve asynchronously and you're not waiting for it to resolve before you make your assertion. testFn needs to return a promise. And you need to do testFn().then(makeAssertions).


node-sass is out of date

The version of node-sass that is used is one major release behind. The version used (at the time of writing) contains a bug where running in production environment causes sass files to be compressed without media-query bodies. This breaks bootstrap's grid layout as well as user-land styles.
Ideally this should use the latest (2.1.1). This also requires bumping node-sass-middleware to 0.5.0

Just ran into some minor problems as a result of this as well and took some time to realize I need to upgrade the node-sass and node-sass-middleware modules.
The version of node-sass being used has a bug where CSS color names i.e. "white" "red", etc. or not computed when defining SASS variables.
i.e.
In version used by keystone-generator...
$default-color: white; //will not be computed
$default-color: #ffffff; //works

In latest version of node-sass and node-sass-middleware
$default-color: white; //works
$default-color: #ffffff; //works

I should note that I was able to update to latest versions without any complications.

I've released a new version of the generator with all dependencies updated, sass seems to work correctly again now.


Unable to login

twikit.errors.BadRequest: Status: 400, message: "{"errors":[{"code":366,"message":"flow name LoginFlow is currently not accessible"}]}" 2fa is on, but I gave the secret to the func

@mightatnight #169

@mightatnight #169

Thank you, after setting the user agent it logs in. Should i install login fix in this case?

You only need to install version 2.1.2.

@mightatnight #169

Thank you, after setting the user agent it logs in. Should i install login fix in this case?

请问您是怎么解决的


Telegraf configuration options

This change updates the module's configuration template to accept (nested) hashes for plugins that should be enabled, as well as the associated options.  This translates nicely into Hiera, for example:
telegraf::plugins:
  cpu:
    percpu: 'true'
    totalcpu: 'true'
  mem:
  io:
  net:
  disk:
  swap:
  system:

It also fixes a small typo on line 38.

Hi @yankcrime!
I have refactored the puppet-telegraf recipe today. For the basic configuration of telegraf. Personally I like to still have comments in configuration files ... so I won't pull your PR.
I use in my / our environment a second puppt recipe
 s_telegraf

which adds now snippets in /etc/opt/telegraf/telegraf.d/
e.g. opentsdb as output sink (01-opentsdb.conf)
and mysql metrics only for hosts with enabled mysql (50-mysql.conf)


Increase visibility of what's inside a backup

I might have missed something, but from what I found out from the docs is that when I want to look inside a backup, I run ark backup describe <name>
I think that the information this gives me is pretty limited.
I'd expect to see to see the Kubernetes resources that have been backed up. Something like this -
$ ark backup describe wp
Name:         wp
Namespace:    heptio-ark
Labels:       <none>
Annotations:  <none>

Namespaces:
  Included:  *
  Excluded:  <none>

Resources:
  Included:        *
  Excluded:        <none>
  Cluster-scoped:  auto

Label selector:  <none>

Snapshot PVs:  auto

TTL:  720h0m0s

Hooks:  <none>

Phase:  New

Backup Format Version:  0

Expiration:  0001-01-01 00:00:00 +0000 UTC

Validation errors:  <none>

Persistent Volumes: <none included>

Resources:
  Deployments:
  - database
  - wordpress
  Services:
  - database (3306/TCP)
  - wordpress (80:31229/TCP)
  Secrets:
  - database-root-password
  - database-user-password
  ConfigMaps:
  - database

This would really help me keep track of backups and what to restore at what time.

We'd need to figure out how to do this without blowing up etcd. Etcd values are capped at 1.5MB, so if you're backing up an entire cluster, I don't think we can store all of that information on the backup object itself. We could include a manifest in object storage, and have describe download the manifest and display it.

@ncdc that makes sense. How do I start hacking on this? I've been trying to read through the code, but I'm not super familiar with how it is structured. Where should I start looking? Also, would love if there is an architecture doc that'd guide me through this.

If you just want to hack, you could either add a method to the BackupService interface:
https://github.com/heptio/ark/blob/master/pkg/cloudprovider/backup_service.go#L38-L65
or add yet another arg to the UploadBackup method.
This is where we call UploadBackup:
https://github.com/heptio/ark/blob/master/pkg/controller/backup_controller.go#L371
We do plan on creating some architecture docs. Just not done yet...

It would be great if there was more information about the backup in metadata stored in the bucket. (not in etcd) e.g., the contents of the k8s manifests, and even file data from volumes (file names, dates, and hashes(!))

@robbyt we are planning on adding metadata in the bucket about what's in it (resource types + namespaces + names). I doubt we'll look at file metadata in the short term, though. That's significantly more work.

I think we should be careful about dumping too much information into the output of the describe command.  I might propose spinning out a new CLI command, inspect, that allows the user to see the contents of a particular backup.

I like the idea of inspect, or we could add a flag to describe to show it.

@skriss for 1.1 are we just interested in showing the resource kinds, namespaces and names? Perhaps we can create a separate issue for an inspect command?

kinds, namespaces and names are a good place to start. Since this issue was originally created, we've added a --details flag to the velero backup describe command - so one option is to show this new info only when that flag is specified, rather than creating an entirely separate command.

Sounds good, it sounded like the discussion here was that an inspect command could show the full k8s manifests that were backed up, but certainly using --details for kinds, namespaces and names makes sense.

Yeah, starting with showing kinds/namespaces/names for velero backup describe --details makes sense.  Since we already have velero backup download, I'm not sure there's much additional utility in an additional CLI command that displays the contents of the files in the tarball.

Splitting this work up into 2/3 PRs:

[ ] store backup resource list in object storage (#1709)
[ ] DownloadRequest changes to fetch backup resource list from object storage
[ ] CLI changes to display backup resource list in velero backup describe --details

Might combine the last 2 depending on length


Check if web contents is attached before allowing it to be considered as focused

Fixes https://github.com/brave/browser-laptop/issues/14197
Auditors: @petemill, @bridiver

Should go into the next Tor build- which needs to be rebased on top of C67 branch (when ready) 😄


"Peer has not allowed receiving yet" - but the channel is deeply confirmed

I have a Turbo channel between my BLW and my node running your fork of Eclair with support for Turbo. I used the channel to spend 4 times already and the channel was confirmed long ago. Yet, I see "Can't ask for Lightning paymennts because your peer has not allowed receiving yet..." message.
Not sure if this is related, but I had another channel opened with the same node previously, but closed it cooperatively because it ended up in some weird state. The old channel is still visible in the list of channels.
I tried restarting the node and BLW as well. Didn't solve it.

Either Eclair does not send a ChannelUpdate or BLW fails to apply it for some reason.
Can you try the following:
Issue https://acinq.github.io/eclair/#updaterelayfee for a channel with BLW and change one of fee parameters, that should trigger Eclair to re-send an update. channelId can be obtained by issuing https://acinq.github.io/eclair/#channels-2 with nodeId=BLW node id.

it ended up in some weird state

can you elaborate?

Great, this part of protocol is very hairy right now so such edge cases may happen sometimes, will be fixed for good once https://github.com/lightningnetwork/lightning-rfc/pull/681 is merged.


'get_allocated' function for Part

Inspect the db to get the number of parts allocated (for this part type);

Allocated to builds
Allocated to customer orders
Allocated for any other purpose??

Implemented in 11b9fb10d877719f6e4f0f185a830d20931e28b4


Added custom DevEUI if wanted and Debug information if needed

Added custom DevEUI if wanted instead using the hardware one from RN2xx3 module
Centralized some _Serial to sendRawCommand (easier for debugging serial transmission between RN2xx3 module and arduino)
Added initOTAA(uint8_t * AppEUI, uint8_t * AppKey, uint8_t * DevEUI) from uint8_t buffer

What is the reason/use case that you need to set a custom DevEUI and not use the hardware EUI of the RN2483?
I like a lot of the things you added. But we need to clean up a little.

Some of the comments are wrong.
In some places F() is used for string constants, other places not. We need to make it consistent.
We need to decide if the uint8_t buffer init function is necessary. TTN dropped this from their library.

Guys,
I've just updated this PR with your suggestions


Added gsap sliding animation to navbar

Description
A clear and concise description of what the PR does.

This PR does the following:

Adds  a sliding down gsap animation to navbar when the page loads
Fixes #334
Updates : updated navbar.jsx by adding a gsap animation inside useEffect hook

Related Issues

Placeholder: "Fixes #334 "

Changes
List the detailed changes made in this PR.

Added a new feature to ...
Refactored the ...
Fixed a bug in ...

Testing Instructions
Detailed instructions on how to test the changes. Include any setup needed and specific test cases.

Pull this branch.
Run npm install to install dependencies.
Run npm test to execute the test suite.
Verify that when the screen is loaded the navbar should have a sliding down animation which give the page more interactive look

Checklist
Make sure to check off all the items before submitting. Mark with [x] if done.

[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] I am working on this issue under GSSOC

contributor's Request: please add a level 3 and gssoc-extd tags to this PR since implementing GSAP code to this repo was hard and takes a lot of time and efforts to be fully functioning

Description
A clear and concise description of what the PR does.

This PR does the following:

Adds  a sliding down gsap animation to navbar when the page loads
Fixes [Feature Request]: Adding GSAP animation to the navbar and body for better UI engagement #334
Updates : updated navbar.jsx by adding a gsap animation inside useEffect hook

Related Issues

Placeholder: "Fixes [Feature Request]: Adding GSAP animation to the navbar and body for better UI engagement #334 "

Changes
List the detailed changes made in this PR.

Added a new feature to ...
Refactored the ...
Fixed a bug in ...

Testing Instructions
Detailed instructions on how to test the changes. Include any setup needed and specific test cases.

Pull this branch.
Run npm install to install dependencies.
Run npm test to execute the test suite.
Verify that when the screen is loaded the navbar should have a sliding down animation which give the page more interactive look

Checklist
Make sure to check off all the items before submitting. Mark with [x] if done.

[x]  I have performed a self-review of my code
[x]  I have commented my code, particularly in hard-to-understand areas
[x]  My changes generate no new warnings
[x]  I am working on this issue under GSSOC

contributor's Request: please add a level 3 and gssoc-extd tags to this PR since implementing GSAP code to this repo was hard and takes a lot of time and efforts to be fully functioning

Kindly resolve these conflicts @Hamza1821