-
Notifications
You must be signed in to change notification settings - Fork 45
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
For a 1.0.0 release #935
Merged
For a 1.0.0 release #935
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
update `evaluate` docstring
further tweak
Patch to fix `StaticNetworkComposite` issue
Replace built-in measures with measures in StatisticalMeasures.jl
Remove deprecated code supporting `@from_network` and `@pipeline`.
Remove `target` as alias for `transformer` in `TransformedTargetModel`
Improve test coverage
Towards a 1.0.0 release
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #935 +/- ##
==========================================
+ Coverage 87.78% 87.91% +0.12%
==========================================
Files 40 27 -13
Lines 3659 2523 -1136
==========================================
- Hits 3212 2218 -994
+ Misses 447 305 -142
☔ View full report in Codecov by Sentry. |
Add test for display of hyper-parameter ranges and models
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#908
Release notes:
(breaking) Remove the definition of all measures (
mae
,log_loss
, etc), which are instead to be be loaded from StatisticalMeasures.jl. Remove completely measures previously corresponding to loss functions in LossFunctions.jl, which must now be explicitly imported and wrapped. For Julia < 1.9, make StatisticalMeasures a hard dependency and re-export it's names. For Julia 1.9 provide default measures for models using a Pkg extension, including StatisticalMeasures.jl as a weak dependency. See this migration guide for dealing with these breaking changes.(breaking) Remove the LossFunctions.jl dependency to close Investigate loading time of LossFunctions.jl #570.
Adapt the resampling (
evaluate!
) code (and a semi-duplication for modelStack
s) to the new measure API set out in StatisticalMeasuresBase.jl.(breaking) Remove deprecated
@pipeline
and@from_network
macros (Remove deprecated code supporting@from_network
and@pipeline
. #932)(breaking) Remove target as alias for transformer in TransformedTargetModel (Remove
target
as alias fortransformer
inTransformedTargetModel
#933)