-
Notifications
You must be signed in to change notification settings - Fork 175
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
Add API for Pythonic Job Creation #2563
Conversation
5494ac2
to
d2d1d4d
Compare
d2d1d4d
to
9e7d82d
Compare
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.
Great examples, Some comments
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.
Overall LGTM, there are few topics we will think about how to address later.
/build |
/build |
1 similar comment
/build |
/build |
1 similar comment
/build |
/build |
/build |
/build |
/build |
/build |
/build |
* Job API and examples * fix import * don't export by default. fix typo * allow providing objects directly instead of ids * restructure files * use optional import * move ScriptExecutor * use optional import in ScriptExecutor * add license header * make kmeans preprocessing optional * introduce as_id() command and revert some changes * enable recursive referencing * fix component count * increase count starting at 1 * rename files * improve docstring, list clients only once * Moved the get_component_init_parameters to class_utils. * check constructor args only * add tf example; only add components once * save tf net args * add tensorboard persistor * add requirements; support tf in script executor * update tasks for converter * reformatting; move ScriptExecutor * address comments * fix imports for inprocess executor; update TF network --------- Co-authored-by: Yuhong Wen <yuhongw@nvidia.com>
Fixes # .
Description
FedJob
allows users to generate job configurations in a Pythonic way. Theto()
routine allows users to send different components to either the server or clients.Depends on #2578
Types of changes
./runtest.sh
.