Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Define the IPython magic lazily #201

Merged
merged 1 commit into from
Apr 17, 2024
Merged

Conversation

superbobry
Copy link
Contributor

IPython is quite a chunky package, and importing it unconditionally makes jaxtyping itself relatively slow to import.

IPython is quite a chunky package, and importing it unconditionally makes
jaxtyping itself relatively slow to import.
@patrick-kidger
Copy link
Owner

This looks good to me! Thank you for the improvement. (Also CC @knyazer who wrote this originally.)

@patrick-kidger patrick-kidger merged commit f83170e into patrick-kidger:main Apr 17, 2024
1 check passed
@knyazer
Copy link
Contributor

knyazer commented Apr 17, 2024

Yep, LGTM.

Thanks for the CC, Patrick :)

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants