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 verification step for raw GEO data vs. prepared CSV #105

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

ttriche
Copy link

@ttriche ttriche commented Jul 21, 2024

Show the CSV file provided contains the same information as the raw data files deposited on GEO for each GSM.

Trust, but verify -- provenance is important and we cannot cut corners if we want to teach reproducible analysis skills.

Show that the CSV file provided does in fact contain the same information as the raw data files deposited on GEO for each GSM.
Copy link

🆗 Pre-flight checks passed 😃

This pull request has been checked and contains no modified workflow files, spoofing, or invalid commits.

It should be safe to Approve and Run the workflows that need maintainer approval.

ttriche added 3 commits July 21, 2024 16:20
One needs to use "[[" rather than `[[` in webR (I know not why). 

Also, breaking up the challenge solution with comments is more considerate of the learner's attention.
like it says on the tin.  should ensure build check passes...
Check that compiled counts are the same as from the raw data.
@ttriche
Copy link
Author

ttriche commented Jul 22, 2024

Got rid of the GEOquery dependency and now this just uses base R.

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