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

Bring python versions and and callbacks into 1.7 release #1228

Merged
merged 2 commits into from
Dec 23, 2024
Merged

Conversation

psfoley
Copy link
Contributor

@psfoley psfoley commented Dec 23, 2024

No description provided.

tanwarsh and others added 2 commits December 23, 2024 13:09
* update python version to 3.12

Signed-off-by: yes <[email protected]>

* dummy commit

Signed-off-by: yes <[email protected]>

* dummy commit

Signed-off-by: yes <[email protected]>

* update python version to 3.12

Signed-off-by: yes <[email protected]>

* dummy commit

Signed-off-by: yes <[email protected]>

* dummy commit

Signed-off-by: yes <[email protected]>

* removed files

Signed-off-by: yes <[email protected]>

* reverted doc change

Signed-off-by: yes <[email protected]>

* added missing requirements for Workflow Interface Tests

Signed-off-by: yes <[email protected]>

* added tensorboard

Signed-off-by: yes <[email protected]>

---------

Signed-off-by: yes <[email protected]>
* Get rid of kwargs

Signed-off-by: Shah, Karan <[email protected]>

* Use module-level logger

Signed-off-by: Shah, Karan <[email protected]>

* Reduce keras verbosity

Signed-off-by: Shah, Karan <[email protected]>

* Remove all log_metric and log_memory_usage traces; add callback hooks

Signed-off-by: Shah, Karan <[email protected]>

* Add `openfl.callbacks` module

Signed-off-by: Shah, Karan <[email protected]>

* Include round_num for task callbacks

Signed-off-by: Shah, Karan <[email protected]>

* Add tensordb to callbacks

Signed-off-by: Shah, Karan <[email protected]>

* No round_num on task callbacks

Signed-off-by: Shah, Karan <[email protected]>

* Remove task boundary callbacks

Signed-off-by: Shah, Karan <[email protected]>

* Remove tb/model_ckpt. Add memory_profiler

Signed-off-by: Shah, Karan <[email protected]>

* Restore psutil and tbX

Signed-off-by: Shah, Karan <[email protected]>

* Format code

Signed-off-by: Shah, Karan <[email protected]>

* Define default callbacks

Signed-off-by: Shah, Karan <[email protected]>

* Add write_logs for bwd compat

Signed-off-by: Shah, Karan <[email protected]>

* Add log_metric_callback for bwd compat

Signed-off-by: Shah, Karan <[email protected]>

* Migrate to module-level logger for collaborator

Signed-off-by: Shah, Karan <[email protected]>

* Review comments

Signed-off-by: Shah, Karan <[email protected]>

* Add metric_writer

Signed-off-by: Shah, Karan <[email protected]>

* Add collaborator side metric logging

Signed-off-by: Shah, Karan <[email protected]>

* Make log dirs on exp begin

Signed-off-by: Shah, Karan <[email protected]>

* Do not print use_tls

Signed-off-by: Shah, Karan <[email protected]>

* Assume reportable metric to be a scalar

Signed-off-by: Shah, Karan <[email protected]>

* Add aggregator side callbacks

Signed-off-by: Shah, Karan <[email protected]>

* do_task test returns mock dict

Signed-off-by: Shah, Karan <[email protected]>

* Consistency changes

Signed-off-by: Shah, Karan <[email protected]>

* Add documentation hooks

Signed-off-by: Shah, Karan <[email protected]>

* Update docstring

Signed-off-by: Shah, Karan <[email protected]>

* Update docs hook

Signed-off-by: Shah, Karan <[email protected]>

* Remove all traces of log_metric_callback and write_metric

Signed-off-by: Shah, Karan <[email protected]>

* Do on_round_begin if not time_to_quit

Signed-off-by: Shah, Karan <[email protected]>

---------

Signed-off-by: Shah, Karan <[email protected]>
@psfoley psfoley merged commit 6b69126 into v1.7.x Dec 23, 2024
2 of 3 checks passed
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.

3 participants