ktmeaton / ncov-recombinant

Reproducible workflow for SARS-CoV-2 recombinant sequence detection.
MIT License
18 stars 2 forks source link

v0.7.0 Release #232

Closed ktmeaton closed 1 year ago

ktmeaton commented 1 year ago

v0.7.0 - Recursive Recombinants

Reviews

Edits

Notes

This is a minor release aimed towards a nextclade dataset upgrade from 2022-10-27 to 2023-01-09 which adds nomenclature for newly designated recombinants XBH - XBP. This release also adds initial support for the detection of "recursive recombination" including XBL and XBN which are recombinants of XBB.

A comprehensive test summary report can be downloaded directly with: ncov-recombinant_v0.6.1_v0.7.0.zip or viewed at the following link once the release is complete.

Documentation

Dataset

Resources

sc2rf

Plot

Report

Validate

Designated Lineages
Proposed Lineages

Continuous Integration

Commits

ktmeaton commented 1 year ago

Something has gone wrong and a merge was created. I'm going to try and revert it.

ktmeaton commented 1 year ago

This was a pretty astonishing series of mistakes that led to this. Overall, I wasn't diligent about double-checking what branch I was on, or which git commands failed.

# First mistake, when using depth 1, I was missing the dev branch
git clone --depth 1 https://github.com/ktmeaton/ncov-recombinant
# Second mistake, this pull applied the changes from v0.7.0 to my master branch
# Also, I should have used the dev branch not the v0.7.0 tag
git pull origin v0.7.0
# Third mistake, I should have checked out the dev branch not the v0.7.0 tag
# Checking out the tag failed (from --depth 1) and I was still on master and didn't notice
git checkout v0.7.0
# Adding the edits to the TSP from my local machine
git add docs/testing_summary_package/ncov-recombinant_v0.6.1_v0.7.0.html
# Fourth mistake: pre-commit linting hooks were not installed, so HTML wasn't linted and raised CI error
git commit -m "docs: update tsp with lineage change and cluster_id note"
# Fifth mistake: I should have checked which branch I was on before pushing, this pushed to main.
git push

This is how I fixed the mistake:

  1. On master, checkout the previous version (v0.6.1): git checkout v0.6.1
  2. Force push to master: git push -f origin master
  3. Checkout dev properly: git checkout dev
  4. Add the TSP updates to dev.
  5. Create a new redo PR #238.