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

Add test data for Baysor module #1440

Merged
merged 2 commits into from
Jan 2, 2025
Merged

Conversation

sebgoti
Copy link

@sebgoti sebgoti commented Dec 18, 2024

This minimal dataset is for a module I am currently working on for a tool named Baysor. To test it, I need a CSV file that has a specific heading for each column of the file (x-location of transcript, transcript’s name, etc). There are currently no available datasets with this requirements (the Molkart pipeline has a similar dataset but without the column names, also, it seems too tiny to work for this module).

@sateeshperi
Copy link

update README

@sebgoti
Copy link
Author

sebgoti commented Jan 2, 2025

Hi @sateeshperi can you tell me more about what do you exactly mean? The README I have is the same as the one from the master branch

@sateeshperi
Copy link

correct that you have the same as modules branch you, just need to update the README as well with information about the test-data you are uploading

@sebgoti
Copy link
Author

sebgoti commented Jan 2, 2025

Understood! README updated. I ended up creating a new folder for the dataset since it did not fit the current categories

@sateeshperi sateeshperi merged commit d203908 into nf-core:modules Jan 2, 2025
@sebgoti sebgoti deleted the baysor branch January 2, 2025 11:37
@sebgoti sebgoti restored the baysor branch January 2, 2025 11:48
@sebgoti
Copy link
Author

sebgoti commented Jan 2, 2025

Hi @sateeshperi , many apologies but I forgot to upload another file for testing the module. At the end created a new PR #1451

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