-
Notifications
You must be signed in to change notification settings - Fork 89
Open
Labels
Effort > Brief 🐇Small tasks expected to take a few hours up to a couple of days.Small tasks expected to take a few hours up to a couple of days.Great First Contribution! 🌱Beginner friendly tickets with narrow scope and huge impact. Perfect to join our community!Beginner friendly tickets with narrow scope and huge impact. Perfect to join our community!Impact > Minor 🔷Small, backward compatible change. Treat like a patch release (e.g., 0.5.8 → 0.5.9).Small, backward compatible change. Treat like a patch release (e.g., 0.5.8 → 0.5.9).
Description
Are you on the latest chainladder version?
- Yes, this bug occurs on the latest version.
Describe the bug in words
display.py has IPython.display imported to support type-hinting. Unfortunately, the core package install doesn't declare this dependency so users are reporting failure unless they separately install ipython. We need to determine whether adding the ipython dependency to chainladder is worth the extra dependency simply for type-hinting.
How can the bug be reproduced?
importing chainladder causes:
File "/usr/local/lib/python3.12/site-packages/chainladder/core/display.py", line 6, in
import IPython.display
ModuleNotFoundError: No module named 'IPython'
What is the expected behavior?
it should not do this
Metadata
Metadata
Assignees
Labels
Effort > Brief 🐇Small tasks expected to take a few hours up to a couple of days.Small tasks expected to take a few hours up to a couple of days.Great First Contribution! 🌱Beginner friendly tickets with narrow scope and huge impact. Perfect to join our community!Beginner friendly tickets with narrow scope and huge impact. Perfect to join our community!Impact > Minor 🔷Small, backward compatible change. Treat like a patch release (e.g., 0.5.8 → 0.5.9).Small, backward compatible change. Treat like a patch release (e.g., 0.5.8 → 0.5.9).