Skip to content

Nextflow#90

Open
dimalvovs wants to merge 12 commits intomainfrom
nextflow
Open

Nextflow#90
dimalvovs wants to merge 12 commits intomainfrom
nextflow

Conversation

@dimalvovs
Copy link
Copy Markdown
Collaborator

This pull request updates the output file handling for the SPACEMARKERS process in nextflow/main.nf, specifically changing how the IMScores data is saved and referenced throughout the workflow. The main changes are focused on improving consistency and correctness in file formats and outputs.

Output file handling improvements:

  • The output tuple for spaceMarkersScores now emits the ${prefix}/IMScores.rds file instead of the previous ${prefix}/spaceMarkers.rds, ensuring the correct file is referenced downstream.
  • The IMScores data is now saved as an RDS file using saveRDS() instead of being written as a CSV with write.csv(), aligning the output format with the .rds extension and typical R conventions.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request fixes incorrect file format handling for the IMScores data in the SPACEMARKERS process. The changes ensure that the file is correctly saved as an RDS file using saveRDS() and that the output declaration references the correct filename.

Key changes:

  • Fixed the output file path from spaceMarkers.rds to IMScores.rds to match the actual file being generated
  • Corrected the file writing method from write.csv() to saveRDS(), aligning the function with the .rds file extension

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

2 participants