-
Notifications
You must be signed in to change notification settings - Fork 90
Documentation #479
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
Documentation #479
Conversation
287bd2c to
cf166e7
Compare
|
@FilippoOlivo can you restore at your first commit and then do an amend? Let's avoid to force push in this PR, so that we know we are not overriding files |
aa0881b to
fde6450
Compare
|
I needed to force update due to a rebase with respect to upstream 0.2 |
1f54775 to
795e4a4
Compare
|
@FilippoOlivo @GiovanniCanali how are we with this? |
Finishing the remaining docs. Should be done by today |
7eec394 to
795e4a4
Compare
de5deb4 to
3f455b7
Compare
42a569c to
af942ad
Compare
|
@ndem0 This PR updates the documentation for the
|
pina/trainer.py
Outdated
| test_size=0.0, | ||
| val_size=0.0, | ||
| compile=None, | ||
| repeat=False, |
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.
maybe set it to None and override inside?
|
@dario-coscia @ndem0 before working on workflow I suggest to close this PR (if documentation part is ready). What do you think? |
|
Agree, for me it is ready for merge. |
|
Ready also for me! |
|
Ready to merge! |
This PR contains the documentation updated for 0.2 version: