Skip to content

Commit

Permalink
Post-review text improvements
Browse files Browse the repository at this point in the history
  • Loading branch information
mgcam committed Aug 29, 2024
1 parent 5d421a4 commit 37330d1
Showing 1 changed file with 10 additions and 3 deletions.
13 changes: 10 additions & 3 deletions docs/qc_outcomes_change_howto.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,13 +6,13 @@
a clause to fail it so that it can be tried out (see some examples below).

All listed below procedures were originally written for products containing
one component. Procedures for changing the already existing QC outcomes should
also work for products containing multiple components as long as these
one component. Existing procedures for changing the already existing QC outcomes
should also work for products containing multiple components as long as these
components are from the same run and are for the same tag index. Use any lane
number (position) that went into the merged product. For example, for a product
described by JSON below

```
```json
{
"__CLASS__": "npg_tracking::glossary::composition-101.3.0",
"components": [
Expand All @@ -39,6 +39,13 @@ my $rs=npg_qc::Schema->connect()->resultset("MqcLibraryOutcomeEnt")
->search_autoqc({id_run=>49494,position=>7,tag_index=>1});
```

The custom `search_autoqc` method returns a resultset containing QC outcomes for
products which contain a component with position 7 and tag_index 1, which will
be the merged product described by the above JSON. If the QC database records
are correct, the resultset should contain one row. The same resultset is
returned if 8 is used as the value of the position. There should not be any need
to repeat the process for every position that went into the merge and in a case
of a toggle this would be wrong.

## Toggle the outcome of a single library

Expand Down

0 comments on commit 37330d1

Please sign in to comment.