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

No longer importing everything in __init__ files. This will require u… #53

Merged
merged 2 commits into from
Mar 31, 2023

Conversation

gvanhoy
Copy link
Collaborator

@gvanhoy gvanhoy commented Mar 31, 2023

…sers to import things differently, but this is not the way to manage import namespacing for many reasons.

…sers to import things differently, but this is not the way to manage import namespacing for many reasons.
@gvanhoy gvanhoy linked an issue Mar 31, 2023 that may be closed by this pull request
@gvanhoy gvanhoy self-assigned this Mar 31, 2023
@gvanhoy gvanhoy added the enhancement New feature or request label Mar 31, 2023
…nal dependency in CuPy for now. Will replace with Torch in the future.
@gvanhoy gvanhoy merged commit 4925ef5 into main Mar 31, 2023
@gvanhoy gvanhoy deleted the 48-remove-recursive-imports-in-init branch March 31, 2023 20:53
ereoh pushed a commit that referenced this pull request Sep 25, 2024
#53)

* No longer importing everything in __init__ files. This will require users to import things differently, but this is not the way to manage import namespacing for many reasons.

* More import statement adjustments in named datasets. Removed conditional dependency in CuPy for now. Will replace with Torch in the future.

---------
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Remove recursive imports in __init__
1 participant