Skip to content

Add package: 2023_Chylenski#85

Closed
Tlkhi wants to merge 2 commits intoposeidon-framework:mainfrom
Tlkhi:add-2023_Chyleński_NatComm

Hidden character warning

The head ref may contain hidden characters: "add-2023_Chyle\u0144ski_NatComm"
Closed

Add package: 2023_Chylenski#85
Tlkhi wants to merge 2 commits intoposeidon-framework:mainfrom
Tlkhi:add-2023_Chyleński_NatComm

Conversation

@Tlkhi
Copy link
Copy Markdown
Contributor

@Tlkhi Tlkhi commented May 25, 2025

Linked to #70

PR Checklist

  • Add the appropriate label to your PR (new package or package update).
  • 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.

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).

@Tlkhi Tlkhi changed the title Add package: 2023_Chyleński_NatComm Add package: 2023_Chylenski_NatComm May 25, 2025
@Tlkhi Tlkhi changed the title Add package: 2023_Chylenski_NatComm Add package: 2023_Chylenski May 25, 2025
@Tlkhi
Copy link
Copy Markdown
Contributor Author

Tlkhi commented May 25, 2025

Closing this PR because it included unrelated commits (from a different package).
A clean version has been submitted here instead: #86

@Tlkhi Tlkhi closed this May 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant