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.

Sign up for GitHub

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

Jump to bottom

[Feature]: MLJ integration #225

Closed
ericphanson opened this issue Jul 2, 2023 · 3 comments · Fixed by #226
Closed

[Feature]: MLJ integration #225

ericphanson opened this issue Jul 2, 2023 · 3 comments · Fixed by #226
Labels
feature (importance: high) High-importance feature

Comments

@ericphanson
Copy link

Feature Request

I think it could be really cool to have an MLJ integration for SymbolicRegression. As an example of how it could be used, in https://discourse.julialang.org/t/ann-sirus-jl-v1-2-interpretable-machine-learning-via-rule-extraction/100932/2 I compared SymbolicRegression to SIRUS for performing interpretable regression on a basic ML example dataset (Boston housing).
@MilesCranmer
Copy link
Owner

Sure, sounds like a great idea! What methods do we need to write to interface it? And I guess it would go in ext/?

@MilesCranmer MilesCranmer added the feature (importance: high) High-importance feature label Jul 3, 2023
@MilesCranmer MilesCranmer mentioned this issue Jul 3, 2023
Merged
@MilesCranmer
Copy link
Owner

Potentially fixed by #226

@ablaom
Copy link

ablaom commented Jul 3, 2023

Generally, only MLJModelInterface.jl is needed and it is very light. I think making it a hard dependency should be fine.

MilesCranmer added a commit that referenced this issue Jul 6, 2023
@MilesCranmer
Merge tag 'v0.20.0' into units4
8b4efa0 
[Diff since v0.19.1](v0.19.1...v0.20.0)

**Closed issues:**
- [Feature]: MLJ integration (#225)
- [Feature]: ternary operators  (#227)

**Merged pull requests:**
- MLJ Integration (#226) (@MilesCranmer)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature (importance: high) High-importance feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

MLJ Integration MilesCranmer/SymbolicRegression.jl
3 participants
@ericphanson @MilesCranmer @ablaom