-
Notifications
You must be signed in to change notification settings - Fork 84
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
feat(datasets): Added PyTorchDataset #735
Conversation
Hey @bpmeek, could you resolve the DCO failures following the instructions here - https://github.com/kedro-org/kedro-plugins/pull/735/checks?check_run_id=26337551853 |
bbbd8df
to
2b7a162
Compare
Signed-off-by: bpmeek <bpmeek.developer@gmail.com>
Signed-off-by: bpmeek <bpmeek.developer@gmail.com>
2b7a162
to
c729731
Compare
I'm not sure what to do about the unit-test failures, none of them are related to my changes. running |
@bpmeek Yeah the failures don't seem to be related, I'll take a look at the CI failures! Thanks for fixing the DCO check! |
Signed-off-by: Merel Theisen <49397448+merelcht@users.noreply.github.com>
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.
LGTM, thanks!
Signed-off-by: Merel Theisen <merel.theisen@quantumblack.com>
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.
Thanks for the experimental contribution @bpmeek ! I've added the dataset to pyproject.toml
as well with torch
as dependency.
Signed-off-by: Merel Theisen <merel.theisen@quantumblack.com>
Signed-off-by: Merel Theisen <merel.theisen@quantumblack.com>
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.
Many thanks, @bpmeek !
* Added PyTorchDataset Signed-off-by: bpmeek <bpmeek.developer@gmail.com> * updated RELEASE.md Signed-off-by: bpmeek <bpmeek.developer@gmail.com> * Add dependencies for PyTorchDataset Signed-off-by: Merel Theisen <merel.theisen@quantumblack.com> * Add PyTorchDataset to API docs Signed-off-by: Merel Theisen <merel.theisen@quantumblack.com> * Fix docs build Signed-off-by: Merel Theisen <merel.theisen@quantumblack.com> --------- Signed-off-by: bpmeek <bpmeek.developer@gmail.com> Signed-off-by: Merel Theisen <49397448+merelcht@users.noreply.github.com> Signed-off-by: Merel Theisen <merel.theisen@quantumblack.com> Co-authored-by: Merel Theisen <49397448+merelcht@users.noreply.github.com> Co-authored-by: Merel Theisen <merel.theisen@quantumblack.com> Signed-off-by: Merel Theisen <merel.theisen@quantumblack.com>
Description
Closes 714
Development notes
Added PyTorchDataset to use pytorch's recommended save and load methods. Added tests to
kedro-datasets/tests/kedro_datasets_experimental/pytorch/test_pytorch_dataset.py
Checklist
RELEASE.md
file