Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move nf-core/demultiple test data from pipeline to test-datasets #920

Merged
merged 2 commits into from
Jun 5, 2023

Conversation

drpatelh
Copy link
Member

@drpatelh drpatelh commented Jun 5, 2023

@adamrtalbot
Copy link

What happens when v1.4.0 releases?

@maxulysse
Copy link
Member

Is the samplesheet format unchanged since 1.0.0 ?

@drpatelh
Copy link
Member Author

drpatelh commented Jun 5, 2023

We add a new folder for 1.4.0 if required. If not, we carry on using the 1.3.0 test data until we have to change it. This at least shows you the last version in which the test data was changed.

@adamrtalbot
Copy link

I think that's misleading - the data version number will be different from the pipeline version number.

@adamrtalbot
Copy link

Is the samplesheet format unchanged since 1.0.0 ?

Yes, they look unchanged according to commit history.

@drpatelh
Copy link
Member Author

drpatelh commented Jun 5, 2023

I think that's misleading - the data version number will be different from the pipeline version number.

Don't know of a more intelligent way to track test-data updates without making in-place changes that will break CI for older releases. Agree it's not perfect but it does the job, especially since test data isn't updated very frequently. Open to suggestions.

@drpatelh
Copy link
Member Author

drpatelh commented Jun 5, 2023

Also, at the moment, the samplesheets for this pipeline were all over the place:

  • Within the pipeline
  • Within this repo in the top-level folder as well as within samplesheets/

This unifies where to find all of the samplesheets in a single location.

@maxulysse
Copy link
Member

Is the samplesheet format unchanged since 1.0.0 ?

Yes, they look unchanged according to commit history.

Then let's put in a 1.0.0 folder, that way, we know it's csv that works since 1.0.0

@adamrtalbot
Copy link

Yeah it's definitely an improvement,

I think that's misleading - the data version number will be different from the pipeline version number.

Don't know of a more intelligent way to track test-data updates without making in-place changes that will break CI for older releases. Agree it's not perfect but it does the job, especially since test data isn't updated very frequently. Open to suggestions.

Yeah it's definitely an improvement. I would probably just go v1, v2, v3 etc. completely independently from the pipeline version but that's not very intuitive either.

@adamrtalbot
Copy link

Then let's put in a 1.0.0 folder, that way, we know it's csv that works since 1.0.0

I guess that makes the most sense.

@drpatelh
Copy link
Member Author

drpatelh commented Jun 5, 2023

Yeah it's definitely an improvement. I would probably just go v1, v2, v3 etc. completely independently from the pipeline version but that's not very intuitive either.

Exactly. This way we know in which release it was last updated.

Then let's put in a 1.0.0 folder, that way, we know it's csv that works since 1.0.0

Nope, there are other samplesheets in this new folder that were changed/added more recently in later releases. Let's start with a blank slate with 1.3.0 to indicate when everything was collated together.

@drpatelh drpatelh merged commit 325fdde into demultiplex Jun 5, 2023
@drpatelh drpatelh deleted the demultiplex_test branch June 5, 2023 11:00
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.

3 participants