-
Notifications
You must be signed in to change notification settings - Fork 429
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
ENH: Add lazy_loader #3173
Labels
Comments
Hi @alexrockhill, We are aware of this. See #3056 we just need time and to be careful with this. This is not high priority but it should be done this year. |
Sorry only searched issues not PRs, thanks for the info |
2 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
A small issue but my import statements get pretty lengthy by importing every function at the top.
mne
implemented pretty recentlylazy_loader
so that you can doimport dipy
and thendipy.align.affine_registration
without having to doimport dipy.align
as well but you don't have the memory hit of having to load all the submodules at initialization. This seems like a nice convenience and it's a small dependency and a couple of lines in each init file (see https://github.com/mne-tools/mne-python/blob/main/mne/decoding/__init__.py for instance). Maybe worth an hour to add this?The text was updated successfully, but these errors were encountered: