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

Update _datasets.py #649

Merged
merged 1 commit into from
Aug 29, 2024
Merged

Update _datasets.py #649

merged 1 commit into from
Aug 29, 2024

Conversation

tessadgreen
Copy link
Collaborator

Update sciplex3 and shifrutmarson to pointers to new scperturb version (1.4), which fixes processing errors in the old versions on scPerturb. The only change is swapping the Zenodo accession numbers for those two files.

Update sciplex3 and shifrutmarson to pointers to new scperturb version (1.4), which fixes processing errors in the old versions
@tessadgreen
Copy link
Collaborator Author

should probably note: latest scperturb version also fixes an issue with Dixit, and that's now separated into three separate datasets on our end. I think the one currently linked should map to https://zenodo.org/records/13350497/files/DixitRegev2016_K562_TFs_High_MOI.h5ad but wasn't 100% sure and didn't want to risk breaking anything.

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 66.40%. Comparing base (f5ffe1d) to head (f145997).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #649   +/-   ##
=======================================
  Coverage   66.40%   66.40%           
=======================================
  Files          46       46           
  Lines        5963     5963           
=======================================
  Hits         3960     3960           
  Misses       2003     2003           
Files Coverage Δ
pertpy/data/_datasets.py 100.00% <ø> (ø)

@Zethson
Copy link
Member

Zethson commented Aug 29, 2024

Thank you!

@Zethson Zethson merged commit ba09f89 into main Aug 29, 2024
6 of 7 checks passed
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