Skip to content

Make ngboost optional #529

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

Merged
merged 20 commits into from
Apr 9, 2025
Merged

Make ngboost optional #529

merged 20 commits into from
Apr 9, 2025

Conversation

AdrianSosic
Copy link
Collaborator

@AdrianSosic AdrianSosic commented Apr 2, 2025

  • Turns ngboost into an optional dependency via our optional import system (see Reduce runtime dependencies #525).
  • Adds an extras dependency group containing all user-facing optional dependencies.
  • Moves the example plotting functionality from the core library to the examples directory (where matplotlib is naturally available) and drops the corresponding plotting module

@AdrianSosic AdrianSosic added the repo Requires changes to the project configuration label Apr 2, 2025
@AdrianSosic AdrianSosic self-assigned this Apr 2, 2025
@AdrianSosic AdrianSosic force-pushed the repo/ngboost branch 2 times, most recently from 508a0f4 to 8a109ab Compare April 5, 2025 20:24
@AdrianSosic AdrianSosic merged commit c7bcfd1 into main Apr 9, 2025
9 of 12 checks passed
@AdrianSosic AdrianSosic deleted the repo/ngboost branch April 9, 2025 14:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
repo Requires changes to the project configuration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants