Skip to content

Add: 2025_Schuetz_Sarmatians#200

Merged
TCLamnidis merged 12 commits intomainfrom
2025_Schuetz_Sarmatians
Jan 2, 2026
Merged

Add: 2025_Schuetz_Sarmatians#200
TCLamnidis merged 12 commits intomainfrom
2025_Schuetz_Sarmatians

Conversation

@SioziosK
Copy link
Copy Markdown
Contributor

@SioziosK SioziosK commented Dec 1, 2025

Linked to #196

PR Checklist

  • The PR title is in the format Add/Update package: {package_name}.
  • The PR description includes a link to the issue requesting the package its
    update. (Add to Linked to #XXX above.)

If adding or updating a package:

SSF file Todo list

  • This PR contains a sequencingSourceFile (.ssf) for the requested
    package.
  • The name of the .ssf file(s) matches the package name (i.e.
    packages/2023_my_package/2023_my_package.ssf).
  • The .ssf file MUST contain a new line at the end of the file.
    A check for this exists in the CI. This check should pass before
    you continue with this list.
  • I confirm that the poseidon_IDs, udg, and library_built are filled
    and correct.
  • I made sure to leave notes where necessary to explain any special
    cases/judgement calls made for data entries.

Include a Janno file

It is recommended that you include a janno file for the requested package to the PR.
While optional, recipes that include a janno file will be prioritised for processing.

  • (optional) This PR contains a janno file (.janno) for the requested
    package.
    • The name of the .janno file(s) matches the package name (i.e.
      packages/2023_my_package/2023_my_package.janno).
    • The .janno file contains one record per poseidon_id specified in the SSF file.

Recipe creation and validation

  • Comment @delphis-bot create recipe to this pull request to awaken
    Poseidon's trusty helper. (This should be repeated whenever changes are
    made to the SSF file contents).

After a few second, Delphis-bot will add a number of files to the PR.
Using the 'Files changed' tab, check that all of the following files were added:

  • The file packages/{package_name}/{package_name}.tsv was added to the PR.
  • The file packages/{package_name}/{package_name}.tsv_patch.sh was added
    to the PR from template.
  • The file packages/{package_name}/script_versions.txt was added to the
    PR.
  • The file packages/{package_name}/{package_name}.config was added to the
    PR from template.

Additional configuration

Additional configuration may be required when processing the data through nf-core/eager.
If you think this may be the case here, please either leave a comment about it in the PR or
add the relevant parameters within the params section at the end of the package config file.
For example, if the published data from the paper have internal barcodes, please mention that
in a comment, or provide the relevant nf-core/eager parameters in the params section.

  • If any nf-core/eager parameters need to be altered from their defaults, I
    have commented so in this PR (or added the relevant parameters within the
    params section at the end of the package config file).

@github-actions github-actions bot added the has janno The PR includes a janno file label Dec 1, 2025
@SioziosK SioziosK marked this pull request as draft December 1, 2025 09:20
@SioziosK SioziosK self-assigned this Dec 1, 2025
@SioziosK
Copy link
Copy Markdown
Contributor Author

@delphis-bot create recipe

@SioziosK SioziosK requested a review from TCLamnidis December 23, 2025 11:11
@TCLamnidis TCLamnidis marked this pull request as ready for review January 2, 2026 09:06
@TCLamnidis
Copy link
Copy Markdown
Member

TCLamnidis commented Jan 2, 2026

@SioziosK Thank you for adding the relationship info to the janno file.

Please note:

  • You should use a . as a decimal separator, instead of ,.
  • Date_BC_AD_Median is an integer column, so when using the average of Start/Stop, please round up/down to the nearest integer.
  • When specifying multiple entries for a column with ;, please do not use a space after the ;. (i.e. first;thirdToFifth, not first; thirdToFifth)
  • Make sure not to commit any files other than the janno file. Sometimes git will pick up hidden lock files used by your spreadsheet editor.

@TCLamnidis TCLamnidis changed the title 2025 schuetz sarmatians Add: 2025_Schuetz_Sarmatians Jan 2, 2026
@TCLamnidis
Copy link
Copy Markdown
Member

@delphis-bot create recipe

@TCLamnidis TCLamnidis merged commit fcc9582 into main Jan 2, 2026
5 checks passed
@TCLamnidis TCLamnidis deleted the 2025_Schuetz_Sarmatians branch January 2, 2026 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

has janno The PR includes a janno file new package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants