-
Notifications
You must be signed in to change notification settings - Fork 148
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
Moving source directory to src/anndata #1151
Merged
Merged
Changes from all commits
Commits
Show all changes
38 commits
Select commit
Hold shift + click to select a range
0f916d3
Start moving
ivirshup 3159ae5
bug fixes
ivirshup 96994af
editable install for doctests
flying-sheep 28df752
oops
flying-sheep 50be530
fix gpu tests
flying-sheep f1e3b8e
Merge branch 'main' into src2
flying-sheep 5fdbc60
--pyargs
flying-sheep 878d2fd
Merge branch 'main' into src2
flying-sheep 8ae217e
IT WORKS
flying-sheep abad630
Discard changes to .azure-pipelines.yml
flying-sheep 4471d95
Discard changes to .github/workflows/test-gpu.yml
flying-sheep 9650876
importlib
flying-sheep 55e1f0e
Merge branch 'main' into src2
flying-sheep 09343e9
Merge branch 'main' into src2
flying-sheep bdf7447
exclude conftest
flying-sheep d2ea31b
Merge branch 'main' into src2
flying-sheep e627fdd
Merge branch 'main' into src2
flying-sheep 61d8ecc
bump pytest version
flying-sheep 55afbaa
make less dependent on import order
flying-sheep 4f619f2
fix weird import
flying-sheep 77a744d
Merge branch 'main' into src2
flying-sheep 7019f39
dev version
flying-sheep fbd89ab
Merge branch 'main' into src2
flying-sheep d4ab295
pytest version
flying-sheep edd2ca2
maybe fix coverage paths
flying-sheep 062717f
more correct
flying-sheep d4170cb
Merge branch 'main' into src2
flying-sheep 6f07f81
Merge branch 'main' into src2
flying-sheep fbd5a3f
deduplicate collection
flying-sheep 6e55886
no importlib for now
flying-sheep 99dcc98
revert
flying-sheep 2992a35
revert pyargs
flying-sheep 6c28074
pyargs was load bearing
flying-sheep b0efbc2
cache bust
flying-sheep 30d7891
Merge branch 'main' into src2
flying-sheep 8a845af
update TODOs
flying-sheep 045c19c
fix
flying-sheep a834c42
bump nuber to be extra safe
flying-sheep File filter
Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure I totally get this argument. What does this do other than let us use
anndata
instead of./src/anndata
intestpaths
?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
because doctests are weird and would otherwise result in importing some modules as files and others via
sys.path
.pytest imports the files that hosts doctests. If they are in a
src
directory, they’ll be imported twice: once that way and once when importing the actual library.Using
--pyargs
means that pytest will identify the doctest parent modules as identical to the library module with the same path.