-
Notifications
You must be signed in to change notification settings - Fork 91
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
Make it easier to select expression from Pareto front for evaluation #289
Conversation
Benchmark Results
Benchmark PlotsA plot of the benchmark results have been uploaded as an artifact to the workflow run for this PR. |
e6d35cc
to
22b5b7b
Compare
[Diff since v0.23.1](v0.23.1...v0.23.2) **Merged pull requests:** - Formatting overhaul (#278) (@MilesCranmer) - Avoid julia-formatter on pre-commit.ci (#279) (@MilesCranmer) - Make it easier to select expression from Pareto front for evaluation (#289) (@MilesCranmer) **Closed issues:** - Garbage collection too passive on worker processes (#237) - How can I set the maximum number of nests? (#285)
[Diff since v0.23.1](v0.23.1...v0.23.2) **Merged pull requests:** - Formatting overhaul (#278) (@MilesCranmer) - Avoid julia-formatter on pre-commit.ci (#279) (@MilesCranmer) - Make it easier to select expression from Pareto front for evaluation (#289) (@MilesCranmer) **Closed issues:** - Garbage collection too passive on worker processes (#237) - How can I set the maximum number of nests? (#285)
Sorry for not catching this earlier. This looks very breaking to me. In MLJ you cannot attach metadata to data at predict time. |
I wasn't sure the proper way to incorporate this. |
Something similar was encountered in hierarchical clustering. In that case we wanted the user to choose, after fitting, the height of the dendrogram. There was quite a bit of discussion and the best we came up with was to return a method in the report to let the user predict with a user-specified height. (A hyperparameter controls the default height for the ordinary I do have the idea to allow more flexible |
This enables the following syntax:
which lets the user specify the equation they wish to use for prediction from the Pareto front.
For multiple outputs:
TODO:
selection_method
(Seems to not work, see Issues with making predictions using the expression selected using the function passed to selection_method PySR#543 by @MrChewi)@ablaom I would be interested to know if there is any way to make this sort of behavior compatible with MLJ? As it stands I think this might break some MLJ interfaces for users who wish to use this. (They would still be able to use
selection_method
parameter, but only beforefit!
)