Closed lazappi closed 1 year ago
@lazappi per tests above it looks like not all of the imports were successfully renamed, so tests are failing.
Sorry, I missed the main file. Hopefully it's ok now.
LGTM. Pending a fix on main from a bug introduced by cell2location, will merge after that's fixed.
FYI @lazappi looks like your Tower auth isn't set up correctly. See https://github.com/lazappi/openproblems/actions/runs/4066798071/jobs/7004186056
I'm going to merge this anyway because it's such a small change and I'm confident it won't fail on main.
Base: 94.95% // Head: 95.03% // Increases project coverage by +0.08%
:tada:
Coverage data is based on head (
bc15d95
) compared to base (61be1cf
). Patch coverage: 100.00% of modified lines in pull request are covered.
:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
Thanks @lazappi !
Submission type
Just fixes a couple of typos in author names for the dimensionality reduction datasets
Testing
Submission guidelines
PR review checklist
This PR will be evaluated on the basis of the following checks:
__init__.py
and were tested in the pipeline