-
Notifications
You must be signed in to change notification settings - Fork 393
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
Release 0.13.0 #961
Merged
Merged
Release 0.13.0 #961
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
It was renamed to ValidSplit but we kept the old class for backwards compatibility. The deprecation period is over, so remove the class and assocaited test.
In #835, we made a change to the data loader initilization. To keep backwards compatbility, we had to add extra code to get_iterator, together with a deprecation notice. The deprecation period is over, so that code is removed, as well as the associated test.
Oops, sorry, didn't intend to ping the contributors :) |
@BenjaminBossan , We're honored. We're satisfied users :-). |
Feels good when even small contributions ship to a release, specially as a new contributor. |
thomasjpfan
approved these changes
May 16, 2023
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thx @thomasjpfan I included the latest PRs. |
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.
This is the PR to prepare the skorch v0.13 release.
Apart from updating
VERSION
andCHANGES.md
, two deprecations were removed:CVSplit
: It was renamed toValidSplit
but we kept the old class for backwards compatibility. The deprecation period is over, so remove the class and associated test.get_iterator
: In Initialize iterator only once per fit call #835, we made a change to the data loader initialization. To keep backwards compatibility, we had to add extra code toget_iterator
, together with a deprecation notice. The deprecation period is over, so that code is removed, as well as the associated test.Release text
This is the suggested announcement for the release page:
The new skorch release is here and it has some changes that will be exiting for some users.
compile=True
when initializing the net to enable compilation.accelerate
package has been improved by fixing some bugs and providing a dedicated history class. Our documentation contains more information on what to consider when training on multiple GPUs.SkorchDoctor
class will simplify the diagnosis of underlying issues. Take a look at the accompanying notebookApart from that, a few bugs have been fixed and the included notebooks have been updated to properly install requirements on Google Colab.
We are grateful for external contributors, many thanks to:
Find below the list of all changes since v0.12.1 below:
Added
torch.compile
function, introduced in PyTorch 2.0 release, which can greatly improve performance on new GPU architectures; to use it, initialize your net with thecompile=True
argument, further compilation arguments can be specified using the dunder notation, e.g.compile__dynamic=True
DistributedHistory
which should be used when training in a multi GPU setting (Add DistributedHistory for multi-gpu training #955)SkorchDoctor
: A helper class that assists in understanding and debugging the neural net training, see this notebook (skorch doctor: a tool to understand the net #912)AccelerateMixin
, it is now possible to prevent unwrapping of the modules by settingunwrap_after_train=True
(Fix issues with unwrapping modules with accelerate #963)Fixed
AccelerateMixin
in a multi-GPU setup (Fix accelerate bug, document, add scripts #947)_get_param_names
returns a list instead of a generator so that subsequenterror messages return useful information instead of a generator
repr
string (Unhelpful error message when setting invalid parameter with
set_params
#925)AccelerateMixin
, which could prevent them from being pickleable (Fix issues with unwrapping modules with accelerate #963)