Skip to content

Issue/41#48

Open
jwokaty wants to merge 4 commits intodevelfrom
issue/41
Open

Issue/41#48
jwokaty wants to merge 4 commits intodevelfrom
issue/41

Conversation

@jwokaty
Copy link
Copy Markdown
Collaborator

@jwokaty jwokaty commented Dec 19, 2025

Incorporates quality control into the files. Currently devel includes unreviewed studies. This PR checks validity of data against https://docs.google.com/document/d/1d8MPEJlgu4EeKDvfZSD6jvo1fz7pVTseHzwr-eyrIhQ/edit?tab=t.0. As a consequence, the 'bleeding edge' would no longer contain all studies. This reduces the current number of rows in full_dump.csv from about 8865 to about 3200.

Alternatively, if it is desirable to continue having a bleeding edge with all, including unreviewed studies, we could make a branch for the release version, like Bioconductor. We'd have to update the action to accommodate this. I would appreciate your feedback.

@jwokaty jwokaty requested review from cmirzayi and lwaldron December 19, 2025 16:02
@jwokaty jwokaty self-assigned this Jan 13, 2026
@lwaldron
Copy link
Copy Markdown
Member

I think "bleeding edge" should include all studies. Filtering incomplete or unreviewed studies can be done downstream if needed, using the "State" and "Reviewer" fields.

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