Skip to content
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

updates to torch_cnn_histology openfl-workspace for Task Runner API #976

Conversation

kta-intel
Copy link
Collaborator

This PR:

  • Updates torch_cnn_histology TaskRunnerAPI example to follow the new convention set in Improve Task Runner Examples and Documentation #943
  • Adds taskrunner.py and dataloader.py in place of old .py files
  • Uses new train_ and validate_ functions
  • Updates plan.yaml to reflect these changes

@kta-intel kta-intel marked this pull request as draft May 20, 2024 22:22
@kta-intel kta-intel marked this pull request as ready for review May 21, 2024 18:40
@kta-intel kta-intel changed the title [WIP] updates to torch_cnn_histology openfl-workspace for Task Runner API updates to torch_cnn_histology openfl-workspace for Task Runner API May 21, 2024
@kta-intel kta-intel requested a review from psfoley May 21, 2024 18:51
Copy link
Contributor

@psfoley psfoley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes look great, @kta-intel. Approved!

@psfoley psfoley merged commit 6d8a219 into securefederatedai:develop May 21, 2024
23 of 26 checks passed
manuelhsantana pushed a commit that referenced this pull request Jul 10, 2024
…976)

* update torch_cnn_histology openfl-workspace for taskrunner api

Signed-off-by: kta-intel <[email protected]>

* fix lint error

Signed-off-by: kta-intel <[email protected]>

* upgrade torch and torchvision to latest. fix lint

Signed-off-by: kta-intel <[email protected]>

---------

Signed-off-by: kta-intel <[email protected]>
Signed-off-by: manuelhsantana <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants