Translated Lists-and-Keys page

[x] कई कौम्पोनॅन्टस को रेंडर करना
[x] मूल लिस्टस कौम्पोनॅन्ट
[x] Keys
[x] key के साथ कंपोनेंट्स निकालना
[x] keys सिर्फ़ भाई-बहनों के बीच अद्वितीय होनी चाहिए
[x] JSX में मैप की एम्बेडिंग

@anchal7299 Any updates on this, let us know if you need any help from our end with the translation

Thank you for the support.I have made the changes as per feedback.

@anchal7299 Great work, there is some more comments I have added in the same feedback.
Can you have a look at the feedbacks which are not resolved.
Also mark every feedback with :+1: emoji, once you have fixed it, so it will be easy for us during review.
Once you are done with these changes, I will start review of rest of the translation

👍
On Thu, Mar 28, 2019 at 8:23 AM Arshad Kazmi<EMAIL_ADDRESS>wrote:

@anchal7299 https://github.com/anchal7299 Great work, there is some
more comments I have added in the same feedback.
Can you have a look at the feedbacks which are not resolved.
Also mark every feedback with 👍 emoji, once you have fixed it, so it
will be easy for us during review.
Once you are done with these changes, I will start review of rest of the
translation
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/reactjs/hi.reactjs.org/pull/25#issuecomment-477425841,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ar9xg1XISU5YmzJgUmxGZD72HhVM2bSpks5vbC6ogaJpZM4cA91J
.

@anchal7299 Great going, there are few more changes which broke during other changes, once you are done with those changes I can start reviewing reset of the translation

@anchal7299 There is only 1 change pending for this review, then I will start with next part of the translation review

Sir,
My git push command is not working.
Initially it was working well but now I am getting a TaskCanceledException
when i use     git push origin lists-and-keys
Please help me out in resolving this issue.
On Thu, Apr 4, 2019 at 10:50 PM Arshad Kazmi<EMAIL_ADDRESS>wrote:

@anchal7299 https://github.com/anchal7299 There is only 1 change
pending for this review, then I will start with next part of the
translation review
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/reactjs/hi.reactjs.org/pull/25#issuecomment-479987740,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ar9xgwGxnt1ouHTA2rqgCCJcNTmW-Dabks5vdjRlgaJpZM4cA91J
.

Are you using Windows?
Can you try this
https://stackoverflow.com/a/36421510/3136282

Thanks, it worked.
On Tue, Apr 9, 2019 at 8:07 AM Arshad Kazmi<EMAIL_ADDRESS>wrote:

Are you using Windows?
Can you try this
https://stackoverflow.com/a/36421510/3136282
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/reactjs/hi.reactjs.org/pull/25#issuecomment-481078797,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ar9xg26mfi5zO1zIbqMk14sUe1qbLKIEks5ve_zygaJpZM4cA91J
.

@anchal7299 Great work. Its good to go from my end.
For next steps, @saranshkataria will be doing a second review and after completion of second review we will get this merged

Ohk sir
Thank you
On 05-May-2019 6:30 pm, "Arshad Kazmi"<EMAIL_ADDRESS>wrote:

@anchal7299 https://github.com/anchal7299 Great work. Its good to go
from my end.
For next steps, @saranshkataria https://github.com/saranshkataria will
be doing a second review and after completion of second review we will get
this merged
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/reactjs/hi.reactjs.org/pull/25#issuecomment-489424359,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AK7XDA7SUBWETGPXYWDXMH3PT3K7ZANCNFSM4HAD3VEQ
.

Is my second review done ?
On 05-May-2019 6:30 pm, "Arshad Kazmi"<EMAIL_ADDRESS>wrote:

@anchal7299 https://github.com/anchal7299 Great work. Its good to go
from my end.
For next steps, @saranshkataria https://github.com/saranshkataria will
be doing a second review and after completion of second review we will get
this merged
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/reactjs/hi.reactjs.org/pull/25#issuecomment-489424359,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AK7XDA7SUBWETGPXYWDXMH3PT3K7ZANCNFSM4HAD3VEQ
.

@anchal7299 @saranshkataria Is out of town for a couple of weeks.
The second review will be done once he is back.

Ohk
On 11-May-2019 11:36 am, "Arshad Kazmi"<EMAIL_ADDRESS>wrote:

@anchal7299 https://github.com/anchal7299 @saranshkataria
https://github.com/saranshkataria Is out of town for a couple of weeks.
The second review will be done once he is back.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/reactjs/hi.reactjs.org/pull/25#issuecomment-491483082,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AK7XDA44MWBBGCAOBSNJSFDPUZO5ZANCNFSM4HAD3VEQ
.

done till line 100

done till {#extracting-components-with-keys}

done till {#embedding-map-in-jsx}

Awesome work @anchal7299 ! Thank you! And merging this in.

Thanks
On 04-Jun-2019 3:32 am, "Saransh Kataria"<EMAIL_ADDRESS>wrote:

Merged #25 https://github.com/reactjs/hi.reactjs.org/pull/25 into
master.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/reactjs/hi.reactjs.org/pull/25?email_source=notifications&email_token=AK7XDA4AEFQAPNP5BMEJPALPYWIF5A5CNFSM4HAD3VE2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGORY6UKJQ#event-2386380070,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AK7XDA4CSYBCMUNN4632RODPYWIF5ANCNFSM4HAD3VEQ
.


Clearly indicate that site is in Alpha

Is your feature request related to a problem? Please describe.
It's unclear if the tool is intended to be fully-functional yet.
Describe the solution you'd like
Add "Alpha" to the header, and add a /release page that describes what is meant by being in Alpha stage, what will be meant by Beta stage, and what being released from Beta will mean

Describe alternatives you've considered
Additional context
References to a couple sites conveying their beta-ness:

https://www.sorcel.co/

https://albus.org/

Alpha vs Beta explanation

Alpha: it's expected that the tool is useful in some narrow situations, with few (if any) examples supporting this; core features exist but are incomplete or need improvement
Beta: it's expected that the tool is useful in more general situations, with many examples supporting this; core features are complete but could be enhanced
Fully-released: few noticeable bugs, users have tested, use cases well-established

Technical ideas

Done in #225


default read timestamp incorrect for writer+reader use case

Describe the bug
When a process performs writes followed by reads (using a new open() handle), the reads do not return the written data.
This occurs because the SOMATileDBContext.read_timestamp is set at process initialization time, and so will always be before the write timestamp, which is set to a greater value.
To Reproduce
TODO: @atolopko-czi
Roughly:
with soma.Collection.create('youcantreadme') as c:
    c.metadata["key"] = "anything"
with soma.Collection.open('youcantreadme') as c:
    assert c.metadata["key"] == "anything" # fails

with soma.Collection.open('youcantreadme', SOMATileDBContext(read_timestamp=int(time.time() * 1000)) as c:
    assert c.metadata["key"] == "anything" # succeeds, thanks to the explicit "now" read timestamp

Versions (please complete the following information):

TileDB-SOMA version: 0.0.5a9
Language and language version (e.g. Python 3.8, R 4.2.2): Py 3.10.6, but shouldn't matter
OS (e.g. MacOS, Ubuntu Linux): MacOS, of course

Additional context
Need to test on 0.0.5a10, just in case this is already fixed!

CC @mlin

Hmm, still looking, but I wasn't immediately able to reproduce by adding this to test_collection.py
def test_real_timestamped_ops(tmp_path):
    with soma.Collection.create(tmp_path.as_uri()) as sc:
        sc.metadata["key"] = "anything"
        sc.add_new_sparse_ndarray("A", type=pa.uint8(), shape=(2, 2)).write(
            pa.SparseCOOTensor.from_scipy(
                sparse.coo_matrix(([1], ([0], [0])), shape=(2,2))
            )
        )
    with soma.Collection.open(tmp_path.as_uri()) as sc:
        assert sc.metadata["key"] == "anything"
        assert "A" in sc
        assert sc["A"].nnz == 1

The default initialization of the read timestamp occurs here:
https://github.com/single-cell-data/TileDB-SOMA/blob/badb5d66318d33469adec04b78534a3ca932256d/apis/python/src/tiledbsoma/options/soma_tiledb_context.py#L35
The timing of this is controlled by attrs, and I assumed that the idea of the "factory lambda" is to generate it at object initialization time, not process wide...but admittedly I don't see that explicated in the attrs docs. (cc @thetorpedodog)
@atolopko-czi can you think of any other distinguishing characteristics of how you encountered this? can you try putting in some debug print of c.context.read_timestamp to confirm if the default read timestamp was generated prematurely, or else if something else more subtle is going on?

@mlin is right - it gets called at init time, so something else must be going on...
In [2]: import time
   ...:
   ...: import attrs
   ...:
   ...:
   ...: def factory():
   ...:     print("Factory called")
   ...:     return int(time.time() * 1000)
   ...:
   ...:
   ...: @attrs.define
   ...: class Foo:
   ...:     x: int = 0
   ...:     read_timestamp: int = attrs.field(factory=lambda: factory())
   ...:

In [3]: Foo()
Factory called
Out[3]: Foo(x=0, read_timestamp=1676000166828)

In [4]: Foo()
Factory called
Out[4]: Foo(x=0, read_timestamp=1676000169850)

The bug is in the client code, which maintains a global SOMATileDBContext, locking in the read timestamp for the duration of the process that is performing writes followed by reads. Closing this issue!


Added test case for #1805

See #1805

This change is

Current coverage is 82% (diff: 100%)

Merging #1819 into master will increase coverage by <1%

@@             master      #1819   diff @@
==========================================
  Files           275        275
  Lines         18367      18868   +501
  Methods        2846       2926    +80
  Messages          0          0
  Branches       2106       2232   +126
==========================================
+ Hits          15001      15465   +464
- Misses         2887       2909    +22
- Partials        479        494    +15

Powered by Codecov. Last update 29a8f30...cf0d7cf


[Transition] restore transition between item and detail.

Transition where broken due to inset padding linked
to shadow on item.

lgtm :tada:


Connectivity-proxy module template resources are sometimes not cleaned up after the module is disabled

Description
During testing of release candidate of new connectivity-proxy module (version 1.0.3) we noticed an issue where after disabling the module in Busola sometimes the operator and custom resource definition were not removed from the Kyma cluster. As a result all module resources (installed by operator) were gone but the resources of the module manifest (installed by module manager) seem to stay.
This happens randomly and really not very often (less than 5% of cases) but we could reproduce it several times during last few days. It would be good to understand what is the root cause and how we could possibly avoid this problem.
Steps to reproduce
The module manifest is currently available only in experimental channel

Enable module,
wait until all resources are installed
disable the module
wait until all resources are cleaned up.

All actions (1-4) should be repeated many times (like 20). At some point after cleanup you can see that operator pod will not be terminated.
Environment Type
Managed
Environment Info
Kubernetes Version: 1.27.x
Kyma CLI Version: 2.19.0
Kyma Runtime version: 2.20.0
Landscape: Canary
Provider: AWS
Region: Europe (Frankfurt) - Canary
Global Account: 1815c4ab-caa7-4b52-87ff-65ed47424dd6
Subaccount: 59fff536-6988-4d6a-8e09-eff6cb4f3696
Attachments

/label area/connectivity-proxy

After alignment between @koala7659  and @kyma-project/jellyfish , we agreed to treat this issue not as a blocker for the CP module rollout.

Can't reproduce this behaviour, tested on STAGE.
The logic of provision and deprovision is idempotent, can't imagine such situation that succeeded several times, but failed in the end, if it ends like this, my assumption is during the whole testing period, it is affected by some external process, however, it's hard to reproduce it purely by this description.
Close it now.


POC for DDP training

Proposal for distributed training using PyTorch DDP
CCFRAUD CPU run: https://ml.azure.com/runs/helpful_yogurt_lqd2q006qb?wsid=/subscriptions/48bbc269-ce89-4f6f-9a12-c6f91fcb772d/resourcegroups/fldev-damajrecak-rg/workspaces/aml-fldevdmdemo0&tid=72f988bf-86f1-41af-91ab-2d7cd011db47#
CCFRAUD GPU run: https://ml.azure.com/runs/frank_pocket_bjn92dmyfd?wsid=/subscriptions/48bbc269-ce89-4f6f-9a12-c6f91fcb772d/resourcegroups/fldev-damajrecak-rg/workspaces/aml-fldevdmdemo0&tid=72f988bf-86f1-41af-91ab-2d7cd011db47#

@jfomhover @garg-amit @thomasp-ms @ShizeSu06 would be really great to hear your thoughts and whether we should apply it in the same way to all examples

Let's switch to AzureML distribution settings!

Responded in a thread


jnp.arange is broken inside shmap unless using jit

Description
This may be expected, but I found the error message quite confusing:
from functools import partial
import numpy as np
import chex
import jax
import jax.numpy as jnp
from jax.sharding import Mesh, PartitionSpec as P
from jax.experimental import mesh_utils
from jax.experimental.shard_map import shard_map

chex.set_n_cpu_devices(8)
devices = mesh_utils.create_device_mesh((8,))
mesh = Mesh(devices, axis_names=('i',))

# uncomment this to make it work!
# @jax.jit
@partial(shard_map, mesh=mesh, in_specs=(P('i'),), out_specs=P('i'))
def f(a_block):
  i = jnp.arange(a_block.shape[0])
  return i + a_block

f(np.zeros(32))

Raises:
XlaRuntimeError                           Traceback (most recent call last)
[<ipython-input-2-9c0c4e291861>](https://localhost:8080/#) in <cell line: 20>()
     18   return i + a_block
     19
---> 20 f(np.zeros(32))

    [... skipping hidden 5 frame]

2 frames
[<ipython-input-2-9c0c4e291861>](https://localhost:8080/#) in f(a_block)
     15 @partial(shard_map, mesh=mesh, in_specs=(P('i'),), out_specs=P('i'))
     16 def f(a_block):
---> 17   i = jnp.arange(a_block.shape[0])
     18   return i + a_block
     19

[/usr/local/lib/python3.9/dist-packages/jax/_src/numpy/lax_numpy.py](https://localhost:8080/#) in arange(start, stop, step, dtype)
   2243       ceil_ = ufuncs.ceil if isinstance(start, core.Tracer) else np.ceil
   2244       start = ceil_(start).astype(int)  # type: ignore
-> 2245     return lax.iota(dtype, start)
   2246   else:
   2247     if step is None and start == 0 and stop is not None:

    [... skipping hidden 16 frame]

[/usr/local/lib/python3.9/dist-packages/jax/_src/dispatch.py](https://localhost:8080/#) in backend_compile(backend, built_c, options, host_callbacks)
    469   # TODO(sharadmv): remove this fallback when all backends allow `compile`
    470   # to take in `host_callbacks`
--> 471   return backend.compile(built_c, compile_options=options)
    472
    473 _ir_dump_counter = itertools.count()

XlaRuntimeError: INTERNAL: RET_CHECK failure (external/xla/xla/hlo/ir/hlo_instruction.cc:2828) ShapeUtil::CompatibleIgnoringFpPrecision(shape(), new_producer->shape()) s32[8,4]{1,0} is not compatible with s32[1,4]{1,0}

What jax/jaxlib version are you using?
No response
Which accelerator(s) are you using?
No response
Additional system info
No response
NVIDIA GPU info
No response

Also fails if the body
def f(a_block):
  i = jnp.arange(a_block.shape[0])
  return i + a_block

is replaced with
def f(a_block):
  jnp.arange(4)

Yet replacing jnp.arange(4) with jnp.zeros(4) or similar works fine... I wonder if this is an XLA bug.
Does it look like there's anything wrong with this HLO?
HloModule jit__unnamed_wrapped_function_, entry_computation_layout={()->f32[8,4]{1,0}}

ENTRY main.5 {
  iota.1 = f32[4]{0} iota(), iota_dimension=0
  reshape.2 = f32[1,4]{1,0} reshape(iota.1)
  custom-call.3 = f32[1,4]{1,0} custom-call(reshape.2), custom_call_target="Sharding", sharding={manual}
  ROOT custom-call.4 = f32[8,4]{1,0} custom-call(custom-call.3), custom_call_target="SPMDShardToFullShape", sharding={
devices=[8,1]0,1,2,3,4,5,6,7}
}

Interestingly, jax.jit doesn't always fix it: it's only a jax.jit and at least one argument passed into the shmap which fixes it, e.g. this fails:
@jax.jit
@partial(shard_map, mesh=mesh, in_specs=(), out_specs=P('i'))
def f():
  return jax.lax.iota(jnp.dtype('int32'), 4)
f()

I think there's a bug in XLA "manual spmd mode" with zero-input functions.

I filed internal bug b/277140784.

Oops, this is a JAX bug after all; somehow in the problematic code we're telling XLA to compile with num_partitions=1. (Thanks @tongfei-guo for patiently correcting me!)


Big dataset quantified before and now with ms2rescore not quant

Description of the bug
I have run this dataset with the previous version of quantms 1.2, without ms2rescore and sage. Right now is not working:
The exit status of the task that caused the workflow execution to fail was: 8

Error executing process > 'NFCORE_QUANTMS:QUANTMS:TMT:PROTEINQUANT:PROTEINQUANTIFIER (MSV000085836.sdrf_openms_design)'

Caused by:
  Process `NFCORE_QUANTMS:QUANTMS:TMT:PROTEINQUANT:PROTEINQUANTIFIER (MSV000085836.sdrf_openms_design)` terminated with an error exit status (8)

Command executed:

  ProteinQuantifier \
      -method 'top' \
      -in ID_mapper_merge_epi_filter_resconf.consensusXML \
      -design MSV000085836.sdrf_openms_design.tsv \
      -out MSV000085836.sdrf_openms_design_protein_openms.csv \
      -mztab MSV000085836.sdrf_openms_design_openms.mzTab \
      -peptide_out MSV000085836.sdrf_openms_design_peptide_openms.csv \
      -top:N 3 \
      -top:aggregate median \
      -top:include_all \
       \
      -ratios \
      -threads 8 \
       \
      -debug 0 \
      2>&1 | tee pro_quant.log

  cat <<-END_VERSIONS > versions.yml
  "NFCORE_QUANTMS:QUANTMS:TMT:PROTEINQUANT:PROTEINQUANTIFIER":
      ProteinQuantifier: $(ProteinQuantifier 2>&1 | grep -E '^Version(.*)' | sed 's/Version: //g' | cut -c 1-50)
  END_VERSIONS

Command exit status:
  8

Command output:
  Quantifying peptides...
  Warning: No peptides quantified.
  /tmp/OpenMS/src/openms/source/ANALYSIS/QUANTITATION/PeptideAndProteinQuant.cpp(446): No protein matches found, cannot quantify proteins.
  Error: Unexpected internal error (No protein matches found, cannot quantify proteins.)

Command wrapper:
  Quantifying peptides...
  Warning: No peptides quantified.
  /tmp/OpenMS/src/openms/source/ANALYSIS/QUANTITATION/PeptideAndProteinQuant.cpp(446): No protein matches found, cannot quantify proteins.
  Error: Unexpected internal error (No protein matches found, cannot quantify proteins.)

Work dir:
  /hps/nobackup/juan/pride/reanalysis/absolute-expression/cell-lines/MSV000085836/work/27/076c4aca914f0b880f6820aea85f7f

Container:
  /hps/nobackup/juan/pride/reanalysis/singularity/ghcr.io-openms-openms-tools-thirdparty-sif-latest.img

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`

ID Filter only found 1 Peptide: tail -n 400 -f ID_mapper_merge_epi_filter_resconf.log
ConsensusXMLFile::store():  found 1 invalid unique ids
IDConflictResolver took 30:22 m (wall), 30:12 m (CPU), 01:35 m (system), 28:37 m (user); Peak Memory Usage: 35086 MB.

Looks like IDFilter is the issue:
tail -n 400 ID_mapper_merge_epi_idfilter.log
Filtering by PSM score (better than 0.05)...
No 'score:peptide' threshold set. Not filtering by peptide score.
No 'score:protein' threshold set. Not filtering by protein score.
Filtering by protein group score...
Removing unreferenced protein hits...
Removing peptide hits without protein references...
Before filtering:
1 identification runs with 79890 proteins,
8175488 spectra identified with 8175488 spectrum matches.
After filtering:
1 identification runs with 0 proteins,
0 spectra identified with 0 spectrum matches.

Here the command of IDFilter:
#!/usr/bin/env bash

set -e # Exit if a tool returns a non-zero status/exit code
set -u # Treat unset variables and parameters as an error
set -o pipefail # Returns the status of the last command to exit with a non-zero status or zero if all successfully execute
set -C # No clobber - prevent output redirection from overwriting files.

IDFilter \
    -in ID_mapper_merge_epi.consensusXML \
    -out ID_mapper_merge_epi_filter.consensusXML \
    -threads 22 \
    -score:proteingroup "0.01" -score:psm "0.05" -delete_unreferenced_peptide_hits \
    2>&1 | tee ID_mapper_merge_epi_idfilter.log

cat <<-END_VERSIONS > versions.yml
"NFCORE_QUANTMS:QUANTMS:TMT:PROTEININFERENCE:IDFILTER":
    IDFilter: $(IDFilter 2>&1 | grep -E '^Version(.*)' | sed 's/Version: //g' | cut -d ' ' -f 1)
END_VERSIONS

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

Here the quantms PROTEININFERENCE:
#!/usr/bin/env bash

set -e # Exit if a tool returns a non-zero status/exit code
set -u # Treat unset variables and parameters as an error
set -o pipefail # Returns the status of the last command to exit with a non-zero status or zero if all successfully execute
set -C # No clobber - prevent output redirection from overwriting files.

ProteinInference \
    -in ID_mapper_merge.consensusXML \
    -threads 16 \
    -picked_fdr true \
    -picked_decoy_string DECOY_ \
    -protein_fdr true \
    -Algorithm:use_shared_peptides true \
    -Algorithm:annotate_indistinguishable_groups true \
     \
    -Algorithm:score_aggregation_method best \
    -Algorithm:min_peptides_per_protein 1 \
    -out ID_mapper_merge_epi.consensusXML \
    -debug 0 \
    2>&1 | tee ID_mapper_merge_inference.log

cat <<-END_VERSIONS > versions.yml
"NFCORE_QUANTMS:QUANTMS:TMT:PROTEININFERENCE:PROTEININFERENCER":
    ProteinInference: $(ProteinInference 2>&1 | grep -E '^Version(.*) ' | sed 's/Version: //g' | cut -d ' ' -f 1)
END_VERSIONS

Did you also use epifany before? And this is TMT right? You'll probably have a lot of errors from ms2rescore. I'm convinced that it's not working correctly for TMT, how we do it right now.

Did you use 1.2.0? Could you share the ID_mapper_merge_epi.consensusXML and other related files?

@daichengxin @jpfeuffer dataset https://ftp.pride.ebi.ac.uk/pub/databases/pride/resources/proteomes/quantms-benchmark/60b5b664a9744072dc80fad42a60ab/

NoMS2Rescore:
Filtering by PSM score (better than 0.01)...
No 'score:protein' threshold set. Not filtering by protein score.
Removing decoy hits...
Filtering by protein group score...
Removing unreferenced protein hits...
Removing peptide hits without protein references...
Before filtering:
1 identification runs with 40419 proteins,
9822955 spectra identified with 9822955 spectrum matches.
After filtering:
1 identification runs with 13161 proteins,
7780873 spectra identified with 7780873 spectrum matches.
ConsensusXMLFile::store():  found 1 invalid unique ids
IDFilter took 34:12 m (wall), 29:05 m (CPU), 01:38 m (system), 27:27 m (user); Peak Memory Usage: 42328 MB.

MS2Rescore:
Filtering by PSM score (better than 0.01)...
No 'score:protein' threshold set. Not filtering by protein score.
Removing decoy hits...
Filtering by protein group score...
Removing unreferenced protein hits...
Removing peptide hits without protein references...
Before filtering:
1 identification runs with 40566 proteins,
10184014 spectra identified with 10184014 spectrum matches.
After filtering:
1 identification runs with 0 proteins,
0 spectra identified with 0 spectrum matches.

This is with the latest version of OpenMS? Can you make both files and the commands to run IDFilter available?

is the psm score still the q-value or something else (e.g., p-value) after ms2rescore.

Off topic:

30 minutes for filtering? It's really time for a new file format.

Yes, dev branch. PSM Still q-value. Protein is also q-value.

.
And I plotted protein q-value for without ms2rescore (orange line) and with ms2rescore (blue line).
All target PSM are removed at protein q-value 0.01 for with ms2rescore.
But almost target PSM are remained at protein q-value 0.01 for without ms2rescore.

Can you plot the PSMs q-value distribution?

@daichengxin, @timosachsenberg has pointed out that the scores don't look quite right in https://github.com/bigbio/quantms/issues/459#issuecomment-2527968481, Can you plot the protein scores without the smooth?

can you also plot target / decoy curves and how they change before / after ms2rescore? For me it looks like some peptide decoys got promoted to a good score which results in bad protein q-values

Ok, looks like the problem is not ms2rescore. Here my recent benchmark using same dataset but not ms2rescore:
nextflow run /hps/nobackup/juan/pride/reanalysis/quantms/main.nf
		 -profile pride_slurm,dev
		 --input MSV000085836.sdrf.tsv
		 --search_engines comet,sage,msgf
		 --root_folder /hps/nobackup/juan/pride/reanalysis/absolute-expression/cell-lines/MSV000085836/
		 --local_input_type raw
		 --outdir /hps/nobackup/juan/pride/reanalysis/absolute-expression/ae-entrapment/cell-lines/MSV000085836
		 --database /hps/nobackup/juan/pride/reanalysis/multiomics-configs/databases/Homo-sapiens-uniprot-reviewed-contam-entrap-decoy-20241105.fasta
		 --protein_level_fdr_cutoff 0.01
		 --posterior_probabilities percolator
		 --psm_level_fdr_cutoff 0.05
		 --protocol TMT
		 --quantify_decoys true
		 --sage_processes 100
		 --skip_post_msstats true
		 --enable_pmultiqc false
		 -resume
		 -with-tower

Here the results from proteinquantifer:
The exit status of the task that caused the workflow execution to fail was: 8

Error executing process > 'NFCORE_QUANTMS:QUANTMS:TMT:PROTEINQUANT:PROTEINQUANTIFIER (MSV000085836.sdrf_openms_design)'

Caused by:
  Process `NFCORE_QUANTMS:QUANTMS:TMT:PROTEINQUANT:PROTEINQUANTIFIER (MSV000085836.sdrf_openms_design)` terminated with an error exit status (8)

Command executed:

  ProteinQuantifier \
      -method 'top' \
      -in ID_mapper_merge_epi_filter_resconf.consensusXML \
      -design MSV000085836.sdrf_openms_design.tsv \
      -out MSV000085836.sdrf_openms_design_protein_openms.csv \
      -mztab MSV000085836.sdrf_openms_design_openms.mzTab \
      -peptide_out MSV000085836.sdrf_openms_design_peptide_openms.csv \
      -top:N 3 \
      -top:aggregate median \
      -top:include_all \
       \
      -ratios \
      -threads 8 \
       \
      -debug 0 \
      2>&1 | tee pro_quant.log

  cat <<-END_VERSIONS > versions.yml
  "NFCORE_QUANTMS:QUANTMS:TMT:PROTEINQUANT:PROTEINQUANTIFIER":
      ProteinQuantifier: $(ProteinQuantifier 2>&1 | grep -E '^Version(.*)' | sed 's/Version: //g' | cut -c 1-50)
  END_VERSIONS

Command exit status:
  8

Command output:
  Quantifying peptides...
  Warning: No peptides quantified.
  /tmp/OpenMS/src/openms/source/ANALYSIS/QUANTITATION/PeptideAndProteinQuant.cpp(446): No protein matches found, cannot quantify proteins.
  Error: Unexpected internal error (No protein matches found, cannot quantify proteins.)

Command wrapper:
  Quantifying peptides...
  Warning: No peptides quantified.
  /tmp/OpenMS/src/openms/source/ANALYSIS/QUANTITATION/PeptideAndProteinQuant.cpp(446): No protein matches found, cannot quantify proteins.
  Error: Unexpected internal error (No protein matches found, cannot quantify proteins.)

Work dir:
  /hps/nobackup/juan/pride/reanalysis/absolute-expression/cell-lines/MSV000085836/work/10/f70f443e848d1982b23d4d7895fe80

Container:
  /hps/nobackup/juan/pride/reanalysis/singularity/ghcr.io-openms-openms-tools-thirdparty-sif-latest.img

Tip: when you have fixed the problem you can continue the execution adding the option `-resume` to the run command line

Here is the ProteinInference data @timosachsenberg @jpfeuffer:
https://ftp.pride.ebi.ac.uk/pub/databases/pride/resources/proteomes/quantms-benchmark/8711e85cea3f3da8ec117ec7477be9/
Here is the folder for IDFilter @timosachsenberg @jpfeuffer:
https://ftp.pride.ebi.ac.uk/pub/databases/pride/resources/proteomes/quantms-benchmark/298bfca6ce4a4c7fd531b62d62f168/


Show Favourite Contacts

showfav command, does exactly what it sounds like.
Other minor changes:

Person::getFavourite renamed to Person::getIsFavourite to make clearer it's a boolean

Note: This PR is a subset of #51 so we can just review and merge that one and ignore this one.


Add link to moderation channel to message report interaction response

This looks good to me, and seems to work according to my tests. This is a really neat feature and really improves the life of the moderators :) Thanks for this!
Will be merging after the review by @DrVilepis

Merging now


[WIP] Update the code with latest KEP

Made the following changes:

Introduce the new APIs
Fix the code generation to use the generate-internal-groups.sh script
Renamed the resource name to follow package name conventions
Structured the generated client code
Updated the go version and introduced vendor directory
Updated README

nicely done @mukhoakash!

/ok-to-test

please fix the build errors. Otherwise, it LGTM

/lgtm

/retest

please fix the build errors. Otherwise, it LGTM

I believe this build error is due to golang version which is set here in the test-infra file. We will have to fix those versions to handle the failures.

/retest

/lgtm
/approve


FR: Show full grid (above and below diagonal)

Maybe I'm missing something totally obvious, but cannot understand how to plot the full grid. That is, the whole square of N*N plots, instead of the default triangle.
Any suggestions?

That is not currently supported but could be added very easily.

Added the fullgrid keyword argument in latest version.


Test page: Don't trigger ad-blockers with class

I was wondering why the new "Share" button didn't show up. A selector .share-wrapper seems to be in one of the common block lists, so I just renamed the wrapper from "share-wrapper" to "link-wrapper".

The fix is very simple. I'll just merge this.


Quota filled

A user reported that his quota is filled but the submissions submitted are not 15 GB in total
more details in the email sent to<EMAIL_ADDRESS>

@ihsaan-ullah Can you please add the details here? I can't find it.
If I recall correctly, the problem was simply that users can't remove submissions so they can get stuck with full storage quota (this problem is reported in #713).

@Didayolo It's showing that i am using 14.7GB of my quota while the total of my submissions size is 8.2168GB

Indeed I can see the problem in your account.

Hi @Didayolo ,
I'm also having the same issue. Not only is the quota filled, but the file sizes are also showing NaN...

And when I try uploading new files, the "attachment" buttom doesn't respond:

Any insights from the Codabench devs/admins will be very much appreciated.

@karimrochd do you remember if the quota was wrong from the start or it appeared this way after you deleted some of your submissions?

@ihsaan-ullah it was wrong before deleting any submission.

Related:

#1388

@OhMaley
What is remaining before closing this issue?

I think everything has be done ! We can close this issue


Automatically restart crashed kernels (after executing user code)

https://github.com/microsoft/vscode-jupyter/issues/1706
We should probably restart the kernels that crash, just like Jupyter.
(probably needs to be done only if we've successfully executed a user cell, else we could end up going in a loop re-starting kernels that fail to start)
The logic may need some fine-tuning, but the overload feature request makes sense as this is what Jupyter offers today.

We should not restart if a cell has not been run, else we go into a crazy loop (when we fail to start kernels)

Not going to do this, we want users to know kernel died.


Packing a Class Library targeting .NetCoreApp1.1 generates warning

Create a Class Libary (.Net Core )
Build
Right click properties change TFM from .NetCoreApp1.0 to .NetCoreApp1.1
Build
Pack
Expected
Packs with out any warning
Actual
1>------ Build started: Project: ClassLibrary3, Configuration: Debug Any CPU ------
1>ClassLibrary3 -> c:\users\corbvt\documents\visual studio 2017\Projects\MegaSln\ClassLibrary3\bin\Debug\netcoreapp1.1\ClassLibrary3.dll
1>Successfully created package 'c:\users\corbvt\documents\visual studio 2017\Projects\MegaSln\ClassLibrary3\bin\Debug\ClassLibrary<IP_ADDRESS>.nupkg'.
1>C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\MSBuild\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(104,5): warning : Issue found with package 'ClassLibrary3'.
1>C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\MSBuild\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(104,5): warning : Issue: Assembly outside lib folder.
1>C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\MSBuild\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(104,5): warning : Description: The assembly 'content\Microsoft.DiaSymReader.Native.x86.dll' is not inside the 'lib' folder and hence it won't be added as reference when the package is installed into a project.
1>C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\MSBuild\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(104,5): warning : Solution: Move it into the 'lib' folder if it should be referenced.
1>C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\MSBuild\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(104,5): warning : Issue: Assembly outside lib folder.
1>C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\MSBuild\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(104,5): warning : Description: The assembly 'contentFiles\any\netcoreapp1.1\Microsoft.DiaSymReader.Native.x86.dll' is not inside the 'lib' folder and hence it won't be added as reference when the package is installed into a project.
1>C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\MSBuild\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(104,5): warning : Solution: Move it into the 'lib' folder if it should be referenced.
1>C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\MSBuild\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(104,5): warning : Issue: Assembly outside lib folder.
1>C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\MSBuild\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(104,5): warning : Description: The assembly 'content\Microsoft.DiaSymReader.Native.amd64.dll' is not inside the 'lib' folder and hence it won't be added as reference when the package is installed into a project.
1>C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\MSBuild\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(104,5): warning : Solution: Move it into the 'lib' folder if it should be referenced.
1>C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\MSBuild\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(104,5): warning : Issue: Assembly outside lib folder.
1>C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\MSBuild\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(104,5): warning : Description: The assembly 'contentFiles\any\netcoreapp1.1\Microsoft.DiaSymReader.Native.amd64.dll' is not inside the 'lib' folder and hence it won't be added as reference when the package is installed into a project.
1>C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\MSBuild\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(104,5): warning : Solution: Move it into the 'lib' folder if it should be referenced.
1>Done building project "ClassLibrary3.csproj".
========== Build: 1 succeeded or up-to-date, 0 failed, 0 skipped ==========
Build
26205.00.d15rel

These binaries are being added as content items - and hence they are getting packed. @tmat?

I have same issue.
Microsoft.DiaSymReader.Native.props requires <Pack>false</Pack>
https://github.com/dotnet/roslyn/issues/15137#issuecomment-278183707

This is being tracked by https://github.com/dotnet/corefx/issues/14627 and the package is being fixed in NetCoreapp 1.1.1


Initialization of S3 client is not thread safe

Confirm by changing [ ] to [x] below to ensure that it's a bug:

[x] I've gone though Developer Guide and API reference
[x] I've searched for previous similar issues and didn't find any solution

Describe the bug
If you attempt to create 2 or more S3 clients from within parallel threads in a binary instrumented with Thread Sanitizer, you will discover that initialization of S3 client accesses a global variable without synchronization while parsing JSON somewhere. Here's the relevant part of the call stack:
WARNING: ThreadSanitizer: data race (pid=12)
  Write of size 8 at 0x7fe93658a1b0 by thread T16 (mutexes: write M144250874682678808, write M150443169551677688):
    #0 cJSON_ParseWithOpts external/aws/aws-cpp-sdk-core/source/external/cjson/cJSON.cpp:1006 (libexternal_Saws_Slibaws.so+0x299bef)
    #1 Aws::Utils::Json::JsonValue::JsonValue(std::__cxx11::basic_string<char, std::char_traits<char>, Aws::Allocator<char> > const&) external/aws/aws-cpp-sdk-core/source/utils/json/JsonSerializer.cpp:39 (libexternal_Saws_Slibaws.so+0x32d09c)
    #2 Aws::Config::EC2InstanceProfileConfigLoader::LoadInternal() external/aws/aws-cpp-sdk-core/source/config/AWSProfileConfigLoader.cpp:341 (libexternal_Saws_Slibaws.so+0x286d88)
    #3 Aws::Config::AWSProfileConfigLoader::Load() external/aws/aws-cpp-sdk-core/source/config/AWSProfileConfigLoader.cpp:47 (libexternal_Saws_Slibaws.so+0x282b4f)
    #4 Aws::Auth::InstanceProfileCredentialsProvider::Reload() external/aws/aws-cpp-sdk-core/source/auth/AWSCredentialsProvider.cpp:265 (libexternal_Saws_Slibaws.so+0x235987)
    #5 Aws::Auth::InstanceProfileCredentialsProvider::RefreshIfExpired() external/aws/aws-cpp-sdk-core/source/auth/AWSCredentialsProvider.cpp:283 (libexternal_Saws_Slibaws.so+0x23613b)
    #6 Aws::Auth::InstanceProfileCredentialsProvider::GetAWSCredentials() external/aws/aws-cpp-sdk-core/source/auth/AWSCredentialsProvider.cpp:250 (libexternal_Saws_Slibaws.so+0x23be9a)
    #7 Aws::Auth::AWSCredentialsProviderChain::GetAWSCredentials() external/aws/aws-cpp-sdk-core/source/auth/AWSCredentialsProviderChain.cpp:35 (libexternal_Saws_Slibaws.so+0x244550)
    #8 Aws::Client::AWSAuthV4Signer::AWSAuthV4Signer(std::shared_ptr<Aws::Auth::AWSCredentialsProvider> const&, char const*, std::__cxx11::basic_string<char, std::char_traits<char>, Aws::Allocator<char> > const&, Aws::Client::AWSAuthV4Signer::PayloadSigningPolicy, bool) external/aws/aws-cpp-sdk-core/source/auth/AWSAuthSigner.cpp:170 (libexternal_Saws_Slibaws.so+0x2262ed)
    #9 std::shared_ptr<Aws::Client::AWSAuthV4Signer> Aws::MakeShared<Aws::Client::AWSAuthV4Signer, std::shared_ptr<Aws::Auth::DefaultAWSCredentialsProviderChain>, char const*&, std::__cxx11::basic_string<char, std::char_traits<char>, Aws::Allocator<char> > const&, Aws::Client::AWSAuthV4Signer::PayloadSigningPolicy&, bool>(char const*, std::shared_ptr<Aws::Auth::DefaultAWSCredentialsProviderChain>&&, char const*&, std::__cxx11::basic_string<char, std::char_traits<char>, Aws::Allocator<char> > const&, Aws::Client::AWSAuthV4Signer::PayloadSigningPolicy&, bool&&) /usr/include/c++/9/ext/new_allocator.h:147 (libexternal_Saws_Slibaws.so+0x39a78b)
    #10 Aws::S3::S3Client::S3Client(Aws::Client::ClientConfiguration const&, Aws::Client::AWSAuthV4Signer::PayloadSigningPolicy, bool, Aws::S3::US_EAST_1_REGIONAL_ENDPOINT_OPTION) external/aws/aws-cpp-sdk-s3/source/S3Client.cpp:134 (libexternal_Saws_Slibaws.so+0x39a78b)
    #11 std::shared_ptr<Aws::S3::S3Client>::shared_ptr<Aws::Allocator<Aws::S3::S3Client>, Aws::Client::ClientConfiguration&, Aws::Client::AWSAuthV4Signer::PayloadSigningPolicy, bool const&>(std::_Sp_alloc_shared_tag<Aws::Allocator<Aws::S3::S3Client> >, Aws::Client::ClientConfiguration&, Aws::Client::AWSAuthV4Signer::PayloadSigningPolicy&&, bool const&) /usr/include/c++/9/ext/new_allocator.h:147 (libexternal_Scom_Ugithub_Utiledb_Utiledb_Slibtiledb.so+0x2879bd)
    #12 std::shared_ptr<Aws::S3::S3Client> std::allocate_shared<Aws::S3::S3Client, Aws::Allocator<Aws::S3::S3Client>, Aws::Client::ClientConfiguration&, Aws::Client::AWSAuthV4Signer::PayloadSigningPolicy, bool const&>(Aws::Allocator<Aws::S3::S3Client> const&, Aws::Client::ClientConfiguration&, Aws::Client::AWSAuthV4Signer::PayloadSigningPolicy&&, bool const&) /usr/include/c++/9/bits/shared_ptr.h:702 (libexternal_Scom_Ugithub_Utiledb_Utiledb_Slibtiledb.so+0x2879bd)
    #13 std::shared_ptr<Aws::S3::S3Client> Aws::MakeShared<Aws::S3::S3Client, Aws::Client::ClientConfiguration&, Aws::Client::AWSAuthV4Signer::PayloadSigningPolicy, bool const&>(char const*, Aws::Client::ClientConfiguration&, Aws::Client::AWSAuthV4Signer::PayloadSigningPolicy&&, bool const&) external/aws/aws-cpp-sdk-core/include/aws/core/utils/memory/stl/AWSAllocator.h:106 (libexternal_Scom_Ugithub_Utiledb_Utiledb_Slibtiledb.so+0x2879bd)

Other TSAN output suggests that "Location is global 'global_error' of size 16 at 0x7fe93658a1b0 (libexternal_Saws_Slibaws.so+0x0000007dc1b0)" is to blame.
#266 suggests that S3 client is thread safe. The call stack shown above confirms that's not the case.
SDK version number
1.7.336
Platform/OS/Hardware/Device
Ubuntu 20.04
To Reproduce (observed behavior)
Spin up a few std::threads and try initializing S3 clients in them in parallel in a test instrumented with TSAN. Get the stack trace above.
Expected behavior
If it's thread safe, it should not fail TSAN. Either eliminate that global variable, or access it while holding a global mutex lock. TSAN is used by many to improve thread safety of their code.
Logs/output
If applicable, add logs or error output.
To enable logging, set the following system properties:
REMEMBER TO SANITIZE YOUR PERSONAL INFO
options.loggingOptions.logLevel = Aws::Utils::Logging::LogLevel::Trace;
Aws::InitAPI(options)

Additional context
Add any other context about the problem here.

@depthwise ,
Yea it looks like you are right and we need to take a closer look at this... in the mean time and just for the sake of completion, can you show me a code snippet on how you are initializing the S3Clients ?

@depthwise
I am sorry I think I may have misunderstood the point of the bug report initially, but when we say the s3 client is thread safe, we mean that the same s3client can be used to do multiple requests in parallel.
I would really need to see sample code for me to replicate and see if this is really a bug.

Folks, I have zero interest in helping you fix your software beyond the information I already provided. I've reported the issue months ago in exhaustive detail. The repro is trivial: spin up several threads and simultaneously initialize separate client instances in them. We've patched this in our own copy of your code. This took at most 10 minutes to diagnose and fix. If, instead, you prefer to treat the problem as though it doesn't exist, that's fine. It demonstrably does exist though, as the stack trace posted in the original bug report should show to anyone who knows anything at all about concurrency.

If someone else is reading this old issue and finds it concerning.
Although it technically is a thread problem, the AWS SDK fulfills the requirement to be thread safe as described in https://github.com/DaveGamble/cJSON#thread-safety. Specifically cJSON_GetErrorPtr is not called so the not protected global_error  variable is only written to and never read.


TypeError: Cannot read property 'exportFileName' of undefined

$ node bin/enml2md Recipes.enex ./Recipes
/home/omega9/Warehouse/Sources/Evernote/enml2md/lib/Note.js:63
        attachmentFileName = attachment.exportFileName();
                                       ^
TypeError: Cannot read property 'exportFileName' of undefined
    at null.<anonymous> (/home/omega9/Warehouse/Sources/Evernote/enml2md/lib/Note.js:63:40)
    at exports.each (/home/omega9/Warehouse/Sources/Evernote/enml2md/node_modules/cheerio/lib/api/traversing.js:122:24)
    at Note.loadENMLContent (/home/omega9/Warehouse/Sources/Evernote/enml2md/lib/Note.js:58:21)
    at Function.Note.composeNoteWithCheerio (/home/omega9/Warehouse/Sources/Evernote/enml2md/lib/Note.js:103:10)
    at null.<anonymous> (/home/omega9/Warehouse/Sources/Evernote/enml2md/lib/Parser.js:28:23)
    at exports.each (/home/omega9/Warehouse/Sources/Evernote/enml2md/node_modules/cheerio/lib/api/traversing.js:122:24)
    at /home/omega9/Warehouse/Sources/Evernote/enml2md/lib/Parser.js:25:19
    at fs.js:334:14
    at /home/omega9/Warehouse/Sources/Evernote/enml2md/node_modules/graceful-fs/graceful-fs.js:103:5
    at FSReqWrap.oncomplete (fs.js:95:15)

@Omega9:
I encountered the same issue. It seems Evernote doesn't always synch attachment content for all notes.
When I click on older notes, it downloads the attachments on demand. When export notes in this state, the <resource> element the .enex is missing it's <data> element.
This contradicts the info in the export dialog:

This will export the selected notes into a single XML file that preserves exact note content with attachments.

I think this is an Evernote bug (and another reason to stop using Evernote as soon as possible).


Add check for maximun databases size

Makes it possible to check that the source databases will fit to target.

Ah, true. I will add one.


No 'Open' action displays for a graph/document

Repro Steps:

Choose a Cosmos DB account and expand it.
Create a graph/document under one database node.
Right click the new graph/document and check the context menu.

Expect:
There is 'Open' action in the context menu.
Actual:
There is no 'Open' action in the context menu. --Please see the below attachment:

This is VS Code behavior. Just selecting the item will open it, there's no menu for it.


need to upgrade required sqlite3 driver version?

Describe the bug
npm i -g<EMAIL_ADDRESS>will raise a 403 error, so I execute npm i -g sqlite3 and installed<EMAIL_ADDRESS>However, extension still requires<EMAIL_ADDRESS>To Reproduce
Steps to reproduce the behavior:

create a connection profile, connecting to my firefox cookies.sqlite file.
Connect.

Expected behavior
commonly run...
Screenshots

Desktop (please complete the following information):

[x] SQLTools Version: 0.21.5
[x] VSCode Version: 1.41.0
[x] Driver Used/Version: SQLite3/?
[x] OS: Windows_NT x64 10.0.18363

I'm having the same issue. I'm guessing it's because in this file they have this line requiring a specific version of SQLite3:
const SQLite3Version = '4.0.8';

I'll upgrade the version.
@zombie110year install it globally doesn't work. The extension is a package and all its dependencies are installed in the extension folder. You can try installing the driver in the extension folder if you want

fixed


ListJobRequest fails with io.grpc.StatusRuntimeException INVALID_ARGUMENT: Invalid filter

Environment details

Google Cloud Talent
MacOS 11.1
Java version: 11
talent version: 4

Steps to reproduce
Created tenant, company and couple of Jobs. Trying to list the jobs by the company
Code example
used this as a refference:
https://github.com/googleapis/java-talent/blob/master/samples/generated/src/main/java/com/google/cloud/examples/talent/v4beta1/JobSearchListJobs.java
com.google.api.gax.rpc.InvalidArgumentException: io.grpc.StatusRuntimeException: INVALID_ARGUMENT: Invalid filter: companyName=projects/myproj-244423/tenants/e7cf52d2-4242-468b-87f8-cdbd45ec46aa/companies/f5b67213-c9d1-4267-bf2d-9a2974b51f4f
Noticed that the test I'm using is failing for a long time: https://github.com/googleapis/java-talent/issues/342
Is this API supported?

Adding quotes to the filter value fixed this

Adding quotes to the filter value fixed this


Formula fields broken in ODS spreadsheets

When I create an ODS file in LibreOffice and create a fomula cell with the contents =SUM(A3:A10) it will not be detected as a formula cell in Openspout. It will be treated as a NumericCell which is incorrect.
When I save the same spreadsheet as an XLSX, it all works. I get the formula cells and can see the formula.
So reading Formula fields in ODS spreadsheet seems to be broken in Openspout.
Also writing Formula fields to ODS is broken. When I add a field with the Formula value  =SUM(A3:A10), LibreOffice is complaining that the format is broken and can't read the file.
When I look in the generated XML of Openspout for de ODS, I can't see the formula at all. It is not in the contents.xml file.
I guess this is a bug?

I guess this is a bug?

Most likely: would you like to fix it please?
Your help would be very welcome :muscle:

I guess this is a bug?

Most likely: would you like to fix it please? Your help would be very welcome 💪

Can i fix?


MINOR: Fix broken JMX URL in docs

The "Monitoring and Management Using JMX Technology" link on the operations page is broken because of a missing double quote in the html a tag.

Open http://kafka.apache.org/documentation/#operations and search for "JMX Technology"
Click the "Monitoring and Management Using JMX Technology" link, you will get "Page not found
" error
notice the URL in the browser address bar, there is a extra double quote at end of the URL https://docs.oracle.com/javase/8/docs/technotes/guides/management/agent.html"

duplicate with #8587


Upgrade json-jwt

Fixes this exploit.  But requires ruby 2.3+.

Coverage remained the same at 100.0% when pulling 3e1c771e1d422205511000310e890f8b31d0d914 on fix/security_patches into f2947111cc42b0a87efb162429f6fc30e0af34d3 on master.


hard clipping stats

Hard clipping stats present in BAM file are not summerized

Should be present in the summary table?
zgrep ^ME qc.tsv.gz | datamash transpose | grep "HardClip"

scaffold380     2048    chr01   1       60      394161H487M1I38M182I55M1I40M1I180M1D206M52D839M7I52M1D232M58D939M33D566M1D13M1I8M1D51M1D238M60I1433M56D10M1D16M1I216M1D182M1D278M73I1021M1D21M2I2M3D75M1D2M1D9M2I16M1I57M1I40M23D140M1D2M3I7M2D23M2I3M2D73M1I127M1I493M56D854M75D32M2D1824M51D91M2I5M1D3M1D88M1I5M3D644M1D454M1D577M1I4M1D85M1D550M1I347M42D741M5D18M1I267M71D173M6D69M3D34M1I119M1D322M5D247M362I287M709966H
the bam file was generated with Minimap2, but the summery says:
#HardClippedBases
0
HardClipRate
0

Ah I see, thanks for posting this SAM record. Alfred counts this read as a secondary alignment but then only takes the primary alignment forward to compute error rates, mapping statistics, and so on. That's probably the desired method for illumina data but I agree for long reads that's not ideal. I have added now a flag to alfred qc (-s) to evaluate secondary alignments.
alfred qc -s -r ref.fa ...

Do you need a new static binary or are you working from the github source?

I downloaded the static binary, didn't use the source yet

Okay, no problem. Here is a new static binary.

thanks a lot but:
the -s flag gives:
[2018-Aug-24 09:56:51] BAM file parsing
0%   10   20   30   40   50   60   70   80   90   100%
|----|----|----|----|----|----|----|----|----|----|
***Segmentation fault
but without -s the script works fine

Thanks. It looks like some aligners set the sequence and quality character strings for secondary alignments to '' and this indeed caused a segfault in alfred. I fixed this now because '' for the sequence and quality is allowed by the SAM spec. What is a bit unfortunate is that some aligners then don't use the new BAM_CEQUAL ('=') and BAM_CDIFF ('X') CIGAR characters which would help to estimate the mismatch and match rate but well, it is what it is. I hope this is working now with most long read aligners but please let me know if you still have problems. I also added -u to evaluate supplementary alignments. For long reads you might want to evaluate both, secondary and supplementary alignments:
./src/alfred qc -su -r <genome.fa> <input.bam>
because that would make it easier to estimate the match and mismatch rate but I tried to find

                             else if (op == '=') op = BAM_CEQUAL;
                             else if (op == 'X') op = BAM_CDIFF;

// Evaluate secondary alignments
// Sequence and quality strings might be '*' for secondary alignments
,

Here is the update release alfred v0.1.10

Thanks, it does what it should do !

I noticed that alfred summaries of hardclipped/softclipped bases are a bit different then expected (much lower values over a complete bamfile. I took 2 lines and analyzed the results:
scaffold380     2048    chr01   1       60           394161H487M1I38M182I55M1I40M1I180M1D206M52D839M7I52M1D232M58D939M33D566M1D13M1I8M1D51M1D238M60I1433M56D10M1D16M1I216M1D182M1D278M73I1021M1D21M2I2M3D75M1D2M1D9M2I16M1I57M1I40M23D140M1D2M3I7M2D23M2I3M2D73M1I127M1I493M56D854M75D32M2D1824M51D91M2I5M1D3M1D88M1I5M3D644M1D454M1D577M1I4M1D85M1D550M1I347M42D741M5D18M1I267M71D173M6D69M3D34M1I119M1D322M5D247M362I287M709966H487M1I38M182I55M1I40M1I180M1D206M52D839M7I52M1D232M58D939M33D566M1D13M1I8M1D51M1D238M60I1433M56D10M1D16M1I216M1D182M1D278M73I1021M1D21M2I2M3D75M1D2M1D9M2I16M1I57M1I40M23D140M1D2M3I7M2D23M2I3M2D73M1I127M1I493M56D854M75D32M2D1824M51D91M2I5M1D3M1D88M1I5M3D644M1D454M1D577M1I4M1D85M1D550M1I347M42D741M5D18M1I267M71D173M6D69M3D34M1I119M1D322M5D247M362I287M709966H
scaffold4441    16      chr01   12746   1       217S107M1D5M1I17M1I95M1D104M1D123M60S
alfred will give HardClipped = 2 SoftClipped=0, but sum 394161H + 709966H = 1.104.127 hardclipped bases and 217S + 60S = 277 softclipped bases
Any ideas about this ? Maybe Alferd is not useful for Minimap2 based bamfiles due to the CIGAR characters, or is alfred using some filtering on the bam file ?

too quick with my comment, the sequence length of scaffold380 is just 16740bp so Minimap2 CIGAR is a bit different then expected ;-)

sorry, forgot that the hardclipped bases are not in the bamfile

Alfred counts the events per read not the length of these events. For instance, for RNA-Seq it counts the number of splicing events but not the length. The same is true for deletions and insertions, it counts the number of D and I events and then separately the InDel size distribution. For sequencing error rate estimations this was more reliable and comparable across technologies and sequencing assays.

Ah, that clarifies it, thanks a lot


"Error: React Native is not installed. Please run npm install in your project directory. Couldn't start project. Please fix the errors and restart the project".

I have gone through these following points

[x] Check latest documentation: https://docs.nativebase.io/
[x] Check for existing open/closed issues for a possible duplicate before creating a new issue: https://github.com/GeekyAnts/NativeBase/issues
[x] Use the latest NativeBase release: https://github.com/GeekyAnts/NativeBase/releases
[x] Check examples from NativeBase KitchenSink https://github.com/GeekyAnts/NativeBase-KitchenSink
[x] For discussion purpose make use of NativeBase Slack: http://slack.nativebase.io/
[x] For queries related to theme, check Theme Variables from Docs and live NativeBase Theme Editor http://nativebase.io/customizer/

Issue Description
React-native is already installed on my system. Sometimes when I start my project using any of these command : "npm start", "yarn start", "expo start" this error is occurring. When I run "npm install" in my terminal the issue is resolved.

node, npm, react-native, react and native-base version, expo version if used, xcode version
Node : v12.16.2; npm : 6.13.0; react-native-cli: 2.0.1; expo: 37.0.0
Expected behaviour
Actual behaviour
Steps to reproduce

Is the bug present in both iOS and Android or in any one of them?
Any other additional info which would help us debug the issue quicker.
ERROR IN THE TERMINAL
yarn start
yarn run v1.22.4
$ expo start
Starting project at E:\Study Material\Practice\React-Native\blog
Expo DevTools is running at http://localhost:19002
Press d to open DevTools now, or shift-d to always open it automatically.
Error: React Native is not installed. Please run npm install in your project directory.
Couldn't start project. Please fix the errors and restart the project.
Error: Couldn't start project. Please fix the errors and restart the project.
at startExpoServerAsync (C:@expo\xdl@57.8.32\src\Project.ts:1948:11)
at Object.startAsync (C:@expo\xdl@57.8.32\src\Project.ts:2350:5)
at action (C:\Users\pro\AppData\Roaming\npm\node_modules\expo-cli\src\commands\start.ts:163:3)
at C:\Users\pro\AppData\Roaming\npm\node_modules\expo-cli\src\commands\start.ts:334:16
at Command. (C:\Users\pro\AppData\Roaming\npm\node_modules\expo-cli\src\exp.ts:84:7)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
PACKAGE.JSON
{
"main": "node_modules/expo/AppEntry.js",
"scripts": {
"start": "expo start",
"android": "expo start --android",
"ios": "expo start --ios",
"web": "expo start --web",
"eject": "expo eject"
},
"dependencies": {
"@react-native-community/masked-view": "0.1.6",
"axios": "^0.19.2",
"expo": "^37.0.0",
"react": "16.9.0",
"react-dom": "16.9.0",
"react-native": "https://github.com/expo/react-native/archive/sdk-37.0.1.tar.gz",
"react-native-gesture-handler": "~1.6.0",
"react-native-reanimated": "~1.7.0",
"react-native-safe-area-context": "0.7.3",
"react-native-screens": "~2.2.0",
"react-native-web": "^0.11.7",
"react-navigation": "^4.3.9",
"react-navigation-stack": "^2.5.1"
},
"devDependencies": {
"@babel/core": "^7.8.6",
"babel-preset-expo": "^8.1.0"
},
"private": true
}
###Important
If you want your issue to be looked at quicker, attach a snack reproducible with your issue. Makes it easier for us!

Hi @ScoobyDu16 , please follow react-native environment setup. Closing for now. Let us know if the issue still persists.


Misc documentation fixes

Notes

how-to-inference is just about the client wrapper, so I clarified it a bit since it's not just for 3rd party integration
Fixed broken image
Consistent formating in https://huggingface.co/docs/hub/adding-a-model which has broken sections at the moment

In general, I also think we should avoid empty headings with no content. Adding a brief description provides more context for users, and makes it easier to browse. For example:

At the moment the first heading is a h2 and the second is a h3, but there's an internal bug (cc @gary149 https://github.com/huggingface/moon-landing/issues/799) that displays both similarly.


[Fluent] Regenerated fluent interfaces, fixed impls, tests

Hi @martinsawicki, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (marcins). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

src\ResourceManagement\Azure.Fluent\Fluent.Tests\VirtualMachineExtensionImageTests.cs file is missing license header.

file: src\ResourceManagement\Azure.Fluent\Fluent.Tests\VirtualMachineExtensionImageTests.cs
            // Lazy listing
            foreach (var extensionImage in extensionImages)
            {
                Assert.NotNull(extensionImage);
                count++;
                if (count >= maxListing)
                {
                    break;
                }
            }
            Assert.True(count == maxListing);

What is this code supposed to test? It would be nice to convert this logic to LINQ.

src\ResourceManagement\Azure.Fluent\Fluent.Tests\VirtualMachineScaleSetTests.cs file does not have license header.

file Fluent/src/ResourceManagement/ResourceManager/Microsoft.Azure.Management.Fluent.ResourceManager/ResourceManager2.cs and class name should be change to ResourceManager


Generating many models makes trainning slower

I've written a genetic algorithm to evolve Keras models, but every time a new model is created it spends more time to train than before, as if keras/tensorflow keeps something in memory for processing, summing up and slowing down all training. I generate many models (one at a time) and train every one of them for 15 epochs on Cohn-Kanade dataset.
The average training time for the first batch of models is about 6 seconds.
The average training time for the fourth batch of models is about 54 seconds.
Observe that the same model used less time in the first epoch than the 4th.
Tried this but didn't worked:
  model = None
  gc.collect()

Gist with some results:
https://gist.github.com/oxydron/1ba27a6c543143a749197a28a6a03592
Note that t: <time> represents the time spent to do a 15 epoch trainning.
Last observation: as evolution goes on, the model complexity not necessarily increases, as someone can use it as an argument for the increasing time spent on training.
Keras version: 2.1.5
Tensorflow version: 1.7
Edit 1:
Related to #3579

Have you tried K.clear_session() between each model?

I did and solved the problem (even with a segmentation fault on epoch 81, but this is another problem...).


Fix Post Commit Blackhole Workflow

Ticket
#12611
Problem description
The Fast Dispatch workflow tests are run inside of the Docker and using the built wheel.
What's changed
The Blackhole workflow also needed the step to build the wheels before calling the Fast Dispatch tests.
Checklist

[x] Post commit CI passes
[ ] Blackhole Post commit (if applicable)
[ ] Model regression CI testing passes (if applicable)
[ ] Device performance regression CI testing passes (if applicable)
[ ] New/Existing tests provide coverage for changes

@TT-billteng These changes were moved to branch mchiou/11919-enable-ttnntteager-on-bh-ci by @abhullar-tt's request.
Looks like the CI tests for Blackhole are getting close!
https://github.com/tenstorrent/tt-metal/actions/runs/10855561390/job/30129009533
cc: @ttmchiou

We can probably close this PR for now.


Section 508 502.3.1: (devenv/tab item 'Microsoft.ML.ModelBuilder.ImageFolderModel') The Name property must not include its class name.

The following accessibility issue needs investigation.
App: devenv
Element path: tab item 'Microsoft.ML.ModelBuilder.ImageFolderModel'
Issue Details: The Name property must not include its class name. Section 508 502.3.1
How To Fix: Provide a UI Automation Name property for the element that:
· Concisely identifies the element, AND
· Does not include the element's class name (such as Microsoft.. or Windows..).
This accessibility issue was found using Accessibility Insights for Windows, a tool that helps debug and find accessibility issues earlier. Get more information and download this tool at https://aka.ms/AccessibilityInsights.

This should be closed by other accessibility bugs.

This should be closed by other accessibility bugs.


Support Dynamic data type

https://clickhouse.com/docs/en/sql-reference/data-types/dynamic

Give this a shot
https://bit.ly/4eaAnWN
Passcode: changeme
you must have a gcc compiler

According to docs this is encoded as
<binary_encoded_data_type><value_in_binary_format_according_to_the_data_type>

So, it should lead to deserialize_any (to ask for the real type from the deserializer).
Then, it can be parsed into some our enum Dynamic { UInt64(u64), ... }.
Btw, the same type can be used after moving to Native or RowBinaryWithNamesAndTypes for any columns to solve https://github.com/ClickHouse/clickhouse-rs/issues/53 and https://github.com/ClickHouse/clickhouse-rs/issues/74


PDI-15648 - Group by step - Minimum aggregation doesn't work when null value in first row

In case "minNullIsValued" is false, firstly, we need to set the first non-null value for further comparing

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f engine/build.xml clean-all resolve jacoco && ant -Dtestreports.dir=bin/reports/integration-test -f engine/build.xml jacoco-integration checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  engine/src/org/pentaho/di/trans/steps/groupby/GroupBy.java
  engine/test-src/org/pentaho/di/trans/steps/groupby/GroupByAggregationNullsTest.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.job.entries.sql.JobEntrySQL

Instruction Change:  + 3.28%
Line Change:  + 3.87%

org.pentaho.di.trans.step.StepMeta

Instruction Change: -.19%:small_red_triangle_down:
Line Change: -.27%:small_red_triangle_down:

org.pentaho.di.trans.steps.datagrid.DataGridMeta

Instruction Change:  + .09%

org.pentaho.di.trans.steps.dimensionlookup.DimensionLookupMeta

Branch Change:  + .63%
Complexity Change:  + .40%
Instruction Change:  + .49%
Line Change:  + .23%

org.pentaho.di.trans.steps.excelwriter.ExcelWriterStepMeta

Branch Change: -1.28%:small_red_triangle_down:
Complexity Change: -.78%:small_red_triangle_down:
Instruction Change: -.08%:small_red_triangle_down:

org.pentaho.di.trans.steps.getfilesrowscount.GetFilesRowsCountMeta

Branch Change: -1.85%:small_red_triangle_down:
Complexity Change: -1.30%:small_red_triangle_down:
Instruction Change: -.40%:small_red_triangle_down:
Line Change: -.40%:small_red_triangle_down:

org.pentaho.di.trans.steps.groupby.GroupBy

Branch Change:  + .90%
Complexity Change:  + .47%
Instruction Change:  + .33%
Line Change:  + .35%

org.pentaho.di.trans.steps.groupby.GroupByMeta

Branch Change:  + 2.08%
Complexity Change:  + 2.11%
Instruction Change:  + .14%
Line Change:  + .37%

org.pentaho.di.trans.steps.ldapoutput.LDAPOutputMeta

Instruction Change:  + .06%

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -1.18%:small_red_triangle_down:
Complexity Change: -1.35%:small_red_triangle_down:

org.pentaho.di.trans.steps.pgbulkloader.PGBulkLoaderMeta

Branch Change:  + .88%
Complexity Change:  + .93%

org.pentaho.di.trans.steps.replacestring.ReplaceStringMeta

Instruction Change:  + .07%

org.pentaho.di.trans.steps.setvalueconstant.SetValueConstantMeta

Instruction Change: -.14%:small_red_triangle_down:

org.pentaho.di.trans.steps.sort.SortRowsMeta

Instruction Change: -.08%:small_red_triangle_down:

org.pentaho.di.trans.steps.sortedmerge.SortedMergeMeta

Instruction Change:  + .18%

org.pentaho.di.trans.steps.tableoutput.TableOutputMeta

Branch Change:  + .91%
Complexity Change:  + .82%

org.pentaho.di.trans.steps.textfileinput.TextFileInputMeta

Branch Change:  + .53%

org.pentaho.di.trans.steps.uniquerows.UniqueRowsMeta

Instruction Change:  + .18%

org.pentaho.di.trans.steps.webservices.WebServiceMeta

Branch Change: -1.79%:small_red_triangle_down:
Complexity Change: -1.19%:small_red_triangle_down:
Instruction Change: -.07%:small_red_triangle_down:

Integration test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.trans.steps.groupby.GroupBy

Branch Change: -.01%:small_red_triangle_down:
Complexity Change: -.03%:small_red_triangle_down:
Instruction Change: -.01%:small_red_triangle_down:
Line Change: -.02%:small_red_triangle_down:

@mbatchelor, @bmorrise , @e-cuellar , Could you, please, take a look?

@mbatchelor, @bmorrise , @e-cuellar , Review, please


Error if object is undefined

With native @tracked you can defined your tracked properties with no default values.
@tracked foo;
@tracked bar;

Even though as objects ember won't track these, the user is used to write it this way. When you start to use this addon, without adding = {} you get the following error

vendor.js:172922 Uncaught (in promise) TypeError: initializer is not a function
desc.get = function get() {
      if (!value) {
        value = deepTracked(initializer());
      }

is where the error is.
I think if you're using this addon, and you're trying to track the properties of an object, perhaps it could default to empty object? Or fail with a warning instead.

I think defaulting to {} sounds good 🎉
I'd gladly accept a PR that adds that, too <3


[Employees Benefit page] Missing end point to a paragraph

The first paragraph after the main title is missing the endpoint after the word salary.

Thanks 👍🏼

Fixed. Pushed to set-up-deal-pages @lcoenen @RohrerHope

Pushed to staging

Approved 👍


Questions about the code

fluid use kubebuilder v2, but the controllers folder is under the pkg
what is the purpose of this design ?

all in one context
type ReconcileRequestContext struct {
	context.Context
	types.NamespacedName
	Log logr.Logger
	*datav1alpha1.Dataset
	Recorder record.EventRecorder
	client.Client
	RuntimeType   string
	FinalizerName string
	common.Category
	Runtime runtime.Object
}

While reading the code, all in one design tends to confuse me
I don't know which values are required and which are not


SetPageFileSize.ps1 fails on new windows-2019 image

Until some time yesterday or the day before, everything was working great, but now the script does not seem to be working anymore. At the moment I am getting the following error:
Error: Exception calling "SetPageFileSize" with "3" argument(s): "The operation completed successfully"
At D:\a\_actions\al-cheb\configure-pagefile-action\v1.2\scripts\SetPageFileSize.ps1:193 char:1
+ [Util.PageFile]::SetPageFileSize($minimumSize, $maximumSize, $diskRoo ...
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
    + FullyQualifiedErrorId : Win32Exception

I've also confirmed with wmic pagefile list /format:list that the size of the page file does not change.

Actually, it's working well. It only looks like the examples in the README.md file are incorrect. The GB suffix is missing.
The last time that I used this action with parameters was last year, and I'm pretty sure I simply copied the examples from the README.md file then too, which was working fine, but maybe I'm imagining things. :)

I experienced this today with the following configuration:
- name: Configure pagefile
  uses<EMAIL_ADDRESS>  with:
    minimum-size: 10
    maximum-size: 12

Error from GitHub Actions:
Run<EMAIL_ADDRESS>Pagefile configuration:
- Minimum size: 10
- Maximum size: 12
- Disk root: D:

Error: Exception calling "SetPageFileSize" with "3" argument(s): "The operation completed successfully"
At D:\a\_actions\al-cheb\configure-pagefile-action\v1.2\scripts\SetPageFileSize.ps1:193 char:1
+ [Util.PageFile]::SetPageFileSize($minimumSize, $maximumSize, $diskRoo ...
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
    + FullyQualifiedErrorId : Win32Exception

Adding a GB suffix solves the problem.
Pagefile configuration:
- Minimum size: 10GB
- Maximum size: 12GB
- Disk root: D:
PageFile:<PHONE_NUMBER>0 /<PHONE_NUMBER>8 bytes for \Device\HarddiskVolume3\pagefile.sys


TypeError: 'Exception' object is not iterable

When I try to run master tip against Antenna running on my local machine, I get this:
(M=bc78b) ~/mozilla/socorro-zero/antenna-loadtests> make test -e URL_SERVER=http://localhost:8000/submit
env LDFLAGS="-L/lib" \
    CFLAGS="-I/include" \
    /home/willkg/mozilla/socorro-zero/antenna-loadtests/venv/bin/pip3 install cryptography
Requirement already satisfied: cryptography in ./venv/lib/python3.5/site-packages
Requirement already satisfied: idna>=2.0 in ./venv/lib/python3.5/site-packages (from cryptography)
Requirement already satisfied: pyasn1>=0.1.8 in ./venv/lib/python3.5/site-packages (from cryptography)
Requirement already satisfied: setuptools>=11.3 in ./venv/lib/python3.5/site-packages (from cryptography)
Requirement already satisfied: cffi>=1.4.1 in ./venv/lib/python3.5/site-packages (from cryptography)
Requirement already satisfied: six>=1.4.1 in ./venv/lib/python3.5/site-packages (from cryptography)
Requirement already satisfied: pycparser in ./venv/lib/python3.5/site-packages (from cffi>=1.4.1->cryptography)
/home/willkg/mozilla/socorro-zero/antenna-loadtests/venv/bin/pip3 install -r requirements.txt
Collecting https://github.com/tarekziade/ailoads/archive/master.zip (from -r requirements.txt (line 4))
  Downloading https://github.com/tarekziade/ailoads/archive/master.zip
     / 40kB 1.1MB/s
Requirement already satisfied: six in ./venv/lib/python3.5/site-packages (from -r requirements.txt (line 1))
Requirement already satisfied: requests in ./venv/lib/python3.5/site-packages (from -r requirements.txt (line 2))
Requirement already satisfied: flake8 in ./venv/lib/python3.5/site-packages (from -r requirements.txt (line 3))
Collecting aiohttp (from molotov==0.4->-r requirements.txt (line 4))
  Downloading aiohttp-1.2.0-cp35-cp35m-manylinux1_x86_64.whl (155kB)
    100% |████████████████████████████████| 163kB 1.6MB/s
Requirement already satisfied: statsd in ./venv/lib/python3.5/site-packages (from molotov==0.4->-r requirements.txt (line 4))
Collecting urwid (from molotov==0.4->-r requirements.txt (line 4))
  Downloading urwid-1.3.1.tar.gz (588kB)
    100% |████████████████████████████████| 593kB 1.9MB/s
Collecting humanize (from molotov==0.4->-r requirements.txt (line 4))
  Downloading humanize-0.5.1.tar.gz
Collecting redis (from molotov==0.4->-r requirements.txt (line 4))
  Downloading redis-2.10.5-py2.py3-none-any.whl (60kB)
    100% |████████████████████████████████| 61kB 2.9MB/s
Requirement already satisfied: pycodestyle<2.3.0,>=2.0.0 in ./venv/lib/python3.5/site-packages (from flake8->-r requirements.txt (line 3))
Requirement already satisfied: pyflakes!=1.2.0,!=1.2.1,!=1.2.2,<1.4.0,>=0.8.1 in ./venv/lib/python3.5/site-packages (from flake8->-r requirements.txt (line 3))
Requirement already satisfied: mccabe<0.6.0,>=0.5.0 in ./venv/lib/python3.5/site-packages (from flake8->-r requirements.txt (line 3))
Collecting chardet (from aiohttp->molotov==0.4->-r requirements.txt (line 4))
  Downloading chardet-2.3.0.tar.gz (164kB)
    100% |████████████████████████████████| 174kB 6.7MB/s
Collecting multidict>=2.1.4 (from aiohttp->molotov==0.4->-r requirements.txt (line 4))
  Downloading multidict-2.1.4-cp35-cp35m-manylinux1_x86_64.whl (357kB)
    100% |████████████████████████████████| 358kB 2.6MB/s
Collecting async-timeout>=1.1.0 (from aiohttp->molotov==0.4->-r requirements.txt (line 4))
  Downloading async_timeout-1.1.0-py3-none-any.whl
Collecting yarl>=0.8.1 (from aiohttp->molotov==0.4->-r requirements.txt (line 4))
  Downloading yarl-0.8.1-cp35-cp35m-manylinux1_x86_64.whl (137kB)
    100% |████████████████████████████████| 143kB 2.2MB/s
Building wheels for collected packages: urwid, humanize, chardet
  Running setup.py bdist_wheel for urwid ... done
  Stored in directory: /home/willkg/.cache/pip/wheels/e7/bb/9d/d7f7fcc9b3aeae20cf4685667635002415629e6395ff2dda7d
  Running setup.py bdist_wheel for humanize ... done
  Stored in directory: /home/willkg/.cache/pip/wheels/d4/80/38/cfbfd95752f71f3812505b948b43383ddc99eedf835fc13b09
  Running setup.py bdist_wheel for chardet ... done
  Stored in directory: /home/willkg/.cache/pip/wheels/28/8c/bf/a69199bd4901d84e13362f95a9ea7bc9a691fed2d655a90bc4
Successfully built urwid humanize chardet
Installing collected packages: chardet, multidict, async-timeout, yarl, aiohttp, urwid, humanize, redis, molotov
  Running setup.py install for molotov ... done
Successfully installed aiohttp-1.2.0 async-timeout-1.1.0 chardet-2.3.0 humanize-0.5.1 molotov-0.4 multidict-2.1.4 redis-2.10.5 urwid-1.3.1 yarl-0.8.1
bash -c "URL_SERVER=http://localhost:8000/submit /home/willkg/mozilla/socorro-zero/antenna-loadtests/venv/bin/ailoads -v -d 30"
[th]Traceback (most recent call last):
  File "/home/willkg/mozilla/socorro-zero/antenna-loadtests/venv/bin/ailoads", line 11, in <module>
    load_entry_point('ailoads==0.1', 'console_scripts', 'ailoads')()
  File "/home/willkg/mozilla/socorro-zero/antenna-loadtests/venv/lib/python3.5/site-packages/ailoads/run.py", line 71, in main
    return run(args)
  File "/home/willkg/mozilla/socorro-zero/antenna-loadtests/venv/lib/python3.5/site-packages/ailoads/run.py", line 85, in run
    for ok, failed in res:
TypeError: 'Exception' object is not iterable
Makefile:56: recipe for target 'test' failed
make: *** [test] Error 1
2 (M=bc78b) ~/mozilla/socorro-zero/antenna-loadtests>

Am I doing something wrong? If not, any idea what's going on.

@rpappalax this appears to be what we saw when running the docker-run command yesterday, any ideas?

I did some poking around and my TypeError is from the fact that all the tests are set to run never. Changing any of them from 0 to a non-zero number causes tests to run.

👍 oy.

remedied in #11

thanks, @willkg I'll add a conditional to molotov to ensure that scenario weights are > 0

@rpappalax  @willkg thanks for reporting, it's now fixed in molotov.


Clone + build.cmd fails two tests on my machine

These tests fail for me in desktop runs of the master branch:

Microsoft.Build.UnitTests.ToolLocationHelper_Tests.GetPathToDotNetFrameworkSdk
Microsoft.Build.UnitTests.ToolLocationHelper_Tests.VerifyToolsetAndToolLocationHelperAgree

      Expected: D:\Src\msbuild\bin\x86\Windows_NT\Debug
      Actual:   C:\Program Files (x86)\Microsoft SDKs\Winúúú
                 (pos 0)
      Stack Trace:
        ToolLocationHelper_Tests.cs(843,0): at Microsoft.Build.UnitTests.ToolLocationHelper_Tests.GetPathToDotNetFrameworkSdk()

      Assert.True() Failure
      Expected: True
      Actual:   False
      Stack Trace:
        ToolLocationHelper_Tests.cs(1010,0): at Microsoft.Build.UnitTests.ToolLocationHelper_Tests.VerifyToolsetAndToolLocationHelperAgree()

=== TEST EXECUTION SUMMARY ===
   Microsoft.Build.Utilities.UnitTests  Total: 414, Errors: 0, Failed: 2, Skipped: 64, Time: 22.346s

@rainersigwald

Closing as a duplicate of #995 which has a couple other failures listed.


Simplify the ability to write typemaps

The new syntax should be similar to code below.
This should help to resolve several issues:

#11 at now it is impossible to handle conversation of Option<T> and Option<&T> at once,
because of Rust trait doesn't allow this, see rust rust-lang/rust#50238 and rust-lang/rust#48869
#167 should simplify transfer of #[repr(C)] between C++<->Rust
#133 It would be possible in user's Rust code write how to handle Result<Option<

foreign_typemap!(
    define_c_type!(
	#[repr(C)]
	struct concat_ident!(CFnOnce, swig_c_type!(T)) {
	    cb: extern "C" fn(swig_c_type!(T), *mut c_void),
	    ctx: *mut c_void,
	})

    (rust, <T, F: FnOnce(T)> $r_param: F) <= concat_ident!(CFnOnce, swig_c_type!(T))
    {
	  |x| $c_param.cb(convert_to_c!(x), $c_param.ctx)
    }

     define_helper_f_helper!(
	 void concat_ident!(result_ready, swig_c_type!(T))(swig_c_type!(T) ret, void *ctx)
	 {
	     swig_f_type!(T) cpp_ret = convert_to_f!(ret);
	     auto ptr = std::make_shared<swig_f_type!(T)>(std::move(cpp_ret));
	     auto fi =
		 static_cast<QFutureInterface<std::shared_ptr<swig_f_type!(T)>> *>(ctx);
	     fi->reportResult(std::move(ptr));
	     fi->reportFinished();
             delete fi;
	 }
     )
    (f, input_to_output) => r#"QFuture<std::shared_ptr<swig_f_type!(T)>>"# r#"
        auto fi = new QFutureInterface<std::shared_ptr<swig_f_type!(T)>>;
       concat_ident!(CFnOnce, swig_c_type!(T)) cb;
       cb.cb = concat_ident!(result_ready, swig_c_type!(T));
       cb.ctx = fi;
       $f_result = fi->future();
       $c_result = cb;
 }"#
);

foreign_typemap!(
    (rust, &str) => RustStrView {
        RustStrView::from_str($r_param)
    }
    (f) => QString r#"
$f_result = QString::fromUtf8($c_result.data, $c_result.len);
"#);

Missed features:

Temporary variables in rules
Combining generic rules with $out
Optimization

Combining generic rules with $out

This is only missed. it will be addressed if someone need it.


[3pt] - (#Front-End Member #2) - Create Resort Details page

In the details page, the user can click the "Reserve" button (replace "Configure" button -  with the "Reserve" button).

The details page for room is created.


[WIP] Update Safety Checker to match new compiler implementation

Description
Previously, we enforced that all CALLs came of the form:

PUSH1 value
PUSH20 address
PUSH or DUP gas
CALL

Now, with the new compiler, CALLs come in the form:

PUSH1 value
CALLER address
GAS gas
CALL

The alternative approach to dealing with this is to whitelist CALLER, blacklist CALL, and every time we see CALLER, we enforce that it is preceded by a PUSH1 0x00 and is followed by GAS CALL
Questions
None
Metadata
Fixes

Fixes #YAS-573

Contributing Agreement

[X] I have read and understood the Optimism Contributing Guide and Code of Conduct and am following those guidelines in this pull request.

I think both internal functions, toAddress and resolveExecutionManager can be removed.

Closing in favor of https://github.com/ethereum-optimism/optimism-monorepo/pull/218


Rds

Does this also bypass remote desktop services license?
Any keygen or patcher for that?

Please, do not ask on Github for Keygens or cracks.
Use reddit, google, etc..
http://lmgtfy.com/?q=rds+license+pastebin
Please, close this issue. Thx.

i was not asking for links. i was asking if rdpwrap ALSO patches RDS license server?

no, and again, there is no need to patch RDS

Not an RDPWrapper issue


Default branch name

@mholt Hi!
I have tested possibility publish to this repo and branch with name tmp has been setted as default. Can u change default branch to main (I have not these permission)? All changes alreasy published to both branches.
Please forgive me for the misconfiguration.

No problem; all done. Let me know if there's anything else!


feat: commitlint init ✨

Description
I have init the commitlint in Your Project if you want to add customer rule you can checkout Commit Lint Official Docs
Fixes #765
Screenshots

Checklist

[x] Tests have been added or updated to cover the changes
[x] Documentation has been updated to reflect the changes
[x] Code follows the established coding style guidelines
[x] All tests are passing

Hey @SurajPratap10 Please Review the Pr and Merge it


Complete Desktop Version Of Portfolio Website

Complete desktop version of portfolio website using media queries to fit desktop screens and maintain responsiveness.

Hello,
Thank you for the review but I am not really clear with the review because
the change for the content section that the previous reviewer gave was
implemented and also the part of introducing new bugs is not actually clear
to me.
Regards.
On Fri, Dec 9, 2022 at 2:56 AM Debas Gebreslasie @.***>
wrote:

@.**** requested changes on this pull request.
Hi @shaaibu7 https://github.com/shaaibu7 👋,
Status: Require Change ♻
Great work on making the changes requested by a previous reviewer 👏
You've done well implementing some of the requested changes, but there are
still some which aren't addressed yet.
https://camo.githubusercontent.com/41693fedeeb870a9be3ebad02d61140bfa75cd829aed4958781a08f350ebd822/68747470733a2f2f6d65646961342e67697068792e636f6d2f6d656469612f5754646735474252343558364e6278714a4b2f67697068792e676966
Highlights

PR title and summary ✔
Documented your files professionally✔

Required Changes ♻
Check the comments under the review.
Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this
PR. However, I strongly recommend you to take them into account as they can
make your code better. Some of them were simply missed by the previous
reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something
is not 100% clear.
Please, remember to tag me in your question so I can receive the
notification.
Please, do not open a new Pull Request for re-reviews. You should use the
same Pull Request submitted for the first review, either valid or invalid
unless it is requested otherwise.
As described in the Code reviews limits policy
https://microverse.zendesk.com/hc/en-us/articles/1500004088561 you have a
limited number of reviews per project (check the exact number in your
Dashboard). If you think that the code review was not fair, you can request
a second opinion using this form https://airtable.com/shrQAqnBwek5a0O0s.
In style.css
https://github.com/shaaibu7/Portfolio-project/pull/4#discussion_r1044327349
:

@.*** screen and (min-width: 768px) {

body {

width: 100%;

margin: 0;

padding: 0;

box-sizing: 0;

[OPTIONAL ] - Good job adding a media query to your project of this
milestone +1 , but according to the project requirements, you should use a
number between (768...1440)px instead of 768 px as the breakpoint between
the desktop layout and another layout.

In style.css
https://github.com/shaaibu7/Portfolio-project/pull/4#discussion_r1044325354
:

.content {

display: flex;

padding-left: -50px;

justify-content: left;

height: 496px;

margin-left: 142px;

margin-right: 142px;

column-gap: 46px;

}

Kindly work on the previous code reviewer's suggestion, not the
issue addressed yet.

In style.css
https://github.com/shaaibu7/Portfolio-project/pull/4#discussion_r1044325125
:

#form-wrap {

width: 100%;

}

.form-text {

width: 569px;

text-align: center;

}

#form-section {

display: grid;

justify-content: center;

}

form {

width: 100%;

While doing a great implementation which the previous code reviewer
suggested, you introduce new bugs in the entire design of the project.
Kindly fix 🔧 the issue in the screenshot below and compare it with
the design template you choose.

Your design in desktop version
[image: Screenshot from 2022-12-09 13-08-53]
https://user-images.githubusercontent.com/65129228/206685238-98146d93-d680-4d6f-a57e-f574028bce89.png
[image: Screenshot from 2022-12-09 13-10-26]
https://user-images.githubusercontent.com/65129228/206685347-461429ca-76ac-44c7-a5af-c5f88a4041fe.png
[image: Screenshot from 2022-12-09 13-10-53]
https://user-images.githubusercontent.com/65129228/206685426-d2d9ecc5-db58-48fb-a64b-dafc45945e2c.png
—
Reply to this email directly, view it on GitHub
https://github.com/shaaibu7/Portfolio-project/pull/4#pullrequestreview-1211476818,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATZZZH47Y7LL4LJIIHASPCLWMMF4NANCNFSM6AAAAAASY24JKA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Add a skill for extracting evidence from a document for a grounded claim

Describe the contribution to the taxonomy
Given a claim grounded in a document, extract evidence from the document that supports the claim.

The evidence must be a verbatim quote from the document
The evidence may contain ellipsis to indicate skipped irrelevant text
The evidence must be self-contained (include the necessary context from the document)
The evidence must be sufficient to verify that the claim is supported by the document
Beyond that, the evidence must contain as little additional text as possible

Evidence extraction skill is a prerequisite for the ability to self-check grounded responses for being properly grounded (i.e. for not containing hallucinations).
Input given at the prompt
The document is provided as the context. The prompted question asks:
From the given document, copy the entire relevant passage that supports the following claim:

" <the claim goes here> "

Copy the passage verbatim, do not paraphrase.
Replace all irrelevant parts in the passage with ellipsis.
If no relevant passage is found, write None.

Response from the original model
When the claim combines information from several parts of the document, the evidence extracted by Merlinite 7b often omits one or more parts. As a result, the extracted incomplete evidence does not support the claim. For example, for document maven-features.html and claim:
The ability to easily write plugins in Java or scripting languages is a key feature of Maven.

the extracted evidence was just:
Extensible, with the ability to easily write plugins in Java or scripting languages...

while the complete evidence should be:
The following are the key features of Maven in a nutshell:

...

* Extensible, with the ability to easily write plugins in Java or scripting languages

Response from the fine-tuned model
(did not have an opportunity to fine tune the model)
Contribution checklist

[x] The contribution was tested with lab generate
[x] No errors or warnings were produced by lab generate
[x] All commits are signed off (DCO)
[x] The qna.yaml file contains at least 5 seed_examples
[x] The qna.yaml file was linted and prettified (yaml-validator can do both)

Thank you for your contribution to InstructLab! Unfortunately, once a Github repo is made public, all open PRs are automatically closed since they link against a private repo. We have detected that your PR might have been one of the ones affected by this change. If you are still interested in contributing your improvement, please fill out the following short form by no later than May 3rd, and we will get back to you with the additional steps necessary once we have had time to assess the PRs of those still interested:
https://forms.gle/V7SrPPMZDo6iGDYu8


Missing or redundant

What is displayed will disappear or be repeated.
[Thanks for your job, it is valuable.]

The extension shouldn't have this issue with Github's new UI. Can you test with the new UI and latest extension?


Removed slow and unnecessary branching from compute shaders

I removed the branch from the preprocessing and postprocessing compute shaders which were only for convenient indexing and replaced them by a one-liner to calculate the same index for both cases without branching.
This should improve performance by some percentages because code will perform better in parallel without branches.
I also changed the local_size from 32x32x3 to 32x32x1 because GPUs really tend to favor a power of 2 as local_size and the size should fit better to match different channel counts.

For what it's worth I tried this PR and on my test image ran 3 times each, on average this runs 0.13% slower, but that's within the margin of error anyway. At least the output files were identical (hash).

It generally depends on the GPU and its drivers. For me it was a little faster in average but theoretically can branches slow down the GPUs scheduler because it can cause the workers to get out of sync.


No requirements.txt

There is no requirements.txt in the repo.

@Qi-Zhangyang Apologies for the delay. Thanks for your interest and reminder! : )
We have added the requirements.txt file.


Increase execution count when execution marked for restart

This PR carries the execution count from stored execution to the new one when execution is marked for restart. This has to be done because the execution starts with a fresh instance.

Removing the execution count field from the struct is the next change, but like you said it is bigger change, so that's why I'm proposing this change as quick patch for the issue for now.
Testing it now.

Did one test and it did increase the count:
    {
        "ID": 5,
        "status": "Restarted",
        "stepId": 5,
        "error": "At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/arm-deployment-operations for usage details.",
        "errorDetails": "Code: Conflict - Message: {\r\n  \"status\": \"Failed\",\r\n  \"error\": {\r\n    \"code\": \"ResourceDeploymentFailure\",\r\n    \"message\": \"The 'AzureAsyncOperationWaiting' resource operation completed with terminal provisioning state 'Failed'.\",\r\n    \"details\": [\r\n      {\r\n        \"code\": \"DeploymentFailed\",\r\n        \"message\": \"At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/arm-deployment-operations for usage details.\",\r\n        \"details\": [\r\n          {\r\n            \"code\": \"Conflict\",\r\n            \"message\": \"{\\r\\n  \\\"status\\\": \\\"Failed\\\",\\r\\n  \\\"error\\\": {\\r\\n    \\\"code\\\": \\\"ResourceDeploymentFailure\\\",\\r\\n    \\\"message\\\": \\\"The 'AzureAsyncOperationWaiting' resource operation completed with terminal provisioning state 'Failed'.\\\",\\r\\n    \\\"details\\\": [\\r\\n      {\\r\\n        \\\"code\\\": \\\"ReconcileControlPlaneCertificatesError\\\",\\r\\n        \\\"message\\\": \\\"key pool is empty\\\"\\r\\n      }\\r\\n    ]\\r\\n  }\\r\\n}\"\r\n          }\r\n        ]\r\n      }\r\n    ]\r\n  }\r\n}\n",
        "code": "DeploymentFailed",
        "provisioningState": "Failed",
        "details": "",
        "timestamp": "2023-03-29T21:33:10.2560646Z",
        "duration": "48 minutes 29.30 seconds",
        "correlationId": "0c8159f8-592a-4a33-91f5-6ed08c53d87a",
        "executionCount": 1
    },
    {
        "ID": 6,
        "status": "Started",
        "stepId": 5,
        "error": "",
        "errorDetails": "",
        "code": "",
        "provisioningState": "",
        "details": "",
        "timestamp": "0001-01-01T00:00:00Z",
        "duration": "",
        "correlationId": "",
        "executionCount": 2
    }
]

Cool, go ahead with it.

Will rebuild the container and re-test with the latest change.


Settling questions of Persistence

[x] Persistence Selection, including interfaces. See #2 .
[x]  To create & update resources in the persistence layers. To be ticketed / expanded.
[x] Ability to delete a resource in the persistence layers is foundational so should be considered, but the more involved aspects of delete may be pushed out of scope.

Currently supported by using AWS SDK library in Go to connect to DynamoDB, S3.
Question of requiring degradation paths - details to what to degree? Or just knowing what we would move to and how?

See https://github.com/sul-dlss-labs/taco/wiki/Prototype-Persistence-Layer-Selection


Register .dwp files as debuginfo artifacts so they get uplifted

What does this PR try to resolve?
With this change linux split-debuginfo=packed builds will place the final .dwp in the top level target dir alongside the binary and report it as a file in the json messages. This puts it on equal footing with .pdb and .dysm files.
How should we test and review this PR?
Step 0: be on linux with the latest rustc stable release.
Step 1: Add this profile to any project that emits a binary:
[profile.dist]
inherits = "release"
split-debuginfo = "packed"
debug = true

Step 2: Run cargo build --profile=dist --message-format=json
Before:

.dwp file doesn't end up in the top level of target/dist/
json message doesn't include .dwp file in files for the binary's build

After:

Both of those things now happen, establishing parity.

Additional information
I am uncertain on:

exactly under what conditions this file type should get pushed (currently "not windows-msvc AND not apple")
what the right default is for the hyphen-renaming flag
whether moving (copying?) the file will break any existing tooling anyone has written (unclear to me if any exists)

See https://github.com/rust-lang/rust/pull/98051#issuecomment-1153776677 for some additional context on "wait what the hell is a dwp file"

I guess cc @davidtwco since this feature seems to be your project?

Ah interesting, seems there's some extra work to do to avoid conflicting names

Ok so it seems like i've kicked a really cursed bee's nest here. I don't have a proper mingw setup right now, so I'm kinda piecing things together from logs and other platforms.

warning: output filename collision.
The targets should have unique names.
Consider changing their names to be unique or compiling them separately.
Colliding filename is: D:/a/cargo/cargo/target/tmp/cit/t340/foo/target/debug/foo.dwp
This may become a hard error in the future; see <https://github.com/rust-lang/cargo/issues/6313>.
The bin target foo in package foo v1.0.0 (D:/a/cargo/cargo/target/tmp/cit/t340/foo) has the same output filename as the lib target foo in package foo v1.0.0 (D:/a/cargo/cargo/target/tmp/cit/t340/foo).

I'm running afoul of this test which is a regression test for #9325 / #9418. It builds a package with a binary and cdylib target of the same name. Originally this was trying to catch problems with .d files, but now it's finding issues with debug files, which is alluded to in the original issues.
The naming scheme we are using for .dwp files on "x64 windows gnu" (mingw?) is such that the binary and cdylib get the same name for their .dwp file and things explode. I believe an analogous thing already happens on msvc windows, as discussed in #9325. Hence why the test is disabled for that platform.

Testing on platforms I do have available:

binary
cdylib
binary debuginfo
cdylib debuginfo

macos
mypackage
libmypackage.dylib
mypackage.dSYM
libmypackage.dylib.dSYM

linux
mypackage
libmypackage.so
mypackage.dwp
libmypackage.dwp

msvc
mypackage.exe
mypackage.dll
mypackage.pdb 💥
mypackage.pdb 💥

mingw
?
?
mypackage.dwp 💥
mypackage.dwp 💥

So it seems like this is a continuation of the woes on msvc that result from its naming scheme. However the test only fails now because mingw I guess previously didn't have a debuginfo file to collide?
A few options:

accept it: disable the test on mingw too
ignore it: don't uplift dwp files on mingw
fix it: make mingw's output naming format more robust

I imagine there are Horrible Implications to 3 because things love assuming paths/names.

I don't have a strong opinion but I'd probably lean towards doing whatever it is we do for pdb files on MSVC (as far as I can tell, that's the "accept it" option).

Friendly ping @Gankra. Just checking in to see if you are still interested in adding some tests and comments for this. I guess @khuey is willing to help if you don't have time :)

I left more detailed comments in the other PR but the values in the table above for linux/cdylib debuginfo and mingw/[binary|cdylib] debuginfo are wrong (in the sense that the table accurately describes what rustc outputs but rustc is outputting the wrong thing). In order for debuggers to find the dwp automatically the filename needs to be exactly the binary filename + ".dwp". For the same reason, the hyphen renaming flag needs to match the binary, not be false unconditionally.

Close in favor of #11572. Thanks for the initial work and analysis!