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

Feature/SK-940 | CLI command for train and validate #658

Merged
merged 2 commits into from
Jul 16, 2024
Merged

Conversation

sowmyasris
Copy link
Contributor

@sowmyasris sowmyasris commented Jul 15, 2024

Implemented the CLI command for train entrypoint
Tested the command for mnist-pytorch example
Needs three arguments
-p path to fedn.yaml file
-i input model parameter
-o output directory to write the updated model

Implemented the CLI command for validate entrypoint
Tested the command for mnist-pytorch example
Needs three arguments
-p path to fedn.yaml file
-i input model
-o output directory to write Json with the validation metrics

@github-actions github-actions bot added minor feature New feature or request labels Jul 15, 2024
@sowmyasris sowmyasris changed the title Feature/SK-940 | CLI command for train Feature / SK-940 | CLI command for train Jul 15, 2024
@sowmyasris sowmyasris changed the title Feature / SK-940 | CLI command for train Feature/SK-940 | CLI command for train Jul 15, 2024
@sowmyasris sowmyasris requested a review from Wrede July 15, 2024 13:03
@sowmyasris sowmyasris changed the title Feature/SK-940 | CLI command for train Feature/SK-940 | CLI command for train and validate Jul 16, 2024
Copy link
Member

@Wrede Wrede left a comment

Choose a reason for hiding this comment

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

very good!

@Wrede Wrede merged commit 9011d75 into master Jul 16, 2024
21 checks passed
@Wrede Wrede deleted the feature/SK-940 branch July 16, 2024 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request minor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants