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

Introduce callbacks API #1195

Merged
Merged
Changes from 1 commit
Commits
Show all changes
39 commits
Select commit Hold shift + click to select a range
6fbbd77
Get rid of kwargs
MasterSkepticista Dec 5, 2024
fe5011a
Use module-level logger
MasterSkepticista Dec 5, 2024
edca03a
Reduce keras verbosity
MasterSkepticista Dec 5, 2024
31ad8ac
Remove all log_metric and log_memory_usage traces; add callback hooks
MasterSkepticista Dec 5, 2024
8857f03
Add `openfl.callbacks` module
MasterSkepticista Dec 5, 2024
49948cf
Merge branch 'develop' into karansh1/callbacks_api
MasterSkepticista Dec 6, 2024
fc6b2cb
Merge branch 'develop' into karansh1/callbacks_api
MasterSkepticista Dec 6, 2024
975e4ac
Include round_num for task callbacks
MasterSkepticista Dec 6, 2024
1267bf0
Add tensordb to callbacks
MasterSkepticista Dec 6, 2024
07e593d
Merge branch 'develop' into karansh1/callbacks_api
MasterSkepticista Dec 9, 2024
2c84c88
No round_num on task callbacks
MasterSkepticista Dec 9, 2024
0a380c4
Merge branch 'develop' into karansh1/callbacks_api
MasterSkepticista Dec 9, 2024
8d1aea3
Merge branch 'develop' into karansh1/callbacks_api
MasterSkepticista Dec 10, 2024
e74b9bf
Remove task boundary callbacks
MasterSkepticista Dec 10, 2024
d63ced5
Remove tb/model_ckpt. Add memory_profiler
MasterSkepticista Dec 10, 2024
9ce8983
Merge branch 'develop' into karansh1/callbacks_api
MasterSkepticista Dec 10, 2024
e103d63
Restore psutil and tbX
MasterSkepticista Dec 10, 2024
abb15da
Format code
MasterSkepticista Dec 10, 2024
5a2cd4b
Define default callbacks
MasterSkepticista Dec 10, 2024
23b8eb3
Add write_logs for bwd compat
MasterSkepticista Dec 10, 2024
4e32632
Add log_metric_callback for bwd compat
MasterSkepticista Dec 10, 2024
b501527
Migrate to module-level logger for collaborator
MasterSkepticista Dec 11, 2024
f5ebd1d
Merge branch 'develop' into karansh1/callbacks_api
MasterSkepticista Dec 11, 2024
af0c40f
Review comments
MasterSkepticista Dec 11, 2024
2ed63ef
Merge branch 'develop' into karansh1/callbacks_api
MasterSkepticista Dec 16, 2024
e8894d6
Merge branch 'develop' into karansh1/callbacks_api
MasterSkepticista Dec 21, 2024
aab8baf
Add metric_writer
MasterSkepticista Dec 21, 2024
3c5e525
Add collaborator side metric logging
MasterSkepticista Dec 21, 2024
fc76f18
Make log dirs on exp begin
MasterSkepticista Dec 21, 2024
c4eb30b
Do not print use_tls
MasterSkepticista Dec 21, 2024
9b3da0e
Assume reportable metric to be a scalar
MasterSkepticista Dec 21, 2024
1cffb9d
Add aggregator side callbacks
MasterSkepticista Dec 21, 2024
afd1bee
do_task test returns mock dict
MasterSkepticista Dec 21, 2024
25c00f1
Consistency changes
MasterSkepticista Dec 21, 2024
efbfdc7
Add documentation hooks
MasterSkepticista Dec 21, 2024
e7068e1
Update docstring
MasterSkepticista Dec 21, 2024
be06eda
Update docs hook
MasterSkepticista Dec 21, 2024
38f7c30
Remove all traces of log_metric_callback and write_metric
MasterSkepticista Dec 21, 2024
d01ec5e
Do on_round_begin if not time_to_quit
MasterSkepticista Dec 23, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Do not print use_tls
Signed-off-by: Shah, Karan <karan.shah@intel.com>
  • Loading branch information
MasterSkepticista committed Dec 21, 2024
commit c4eb30bf6350b94056acb9da9b96f8b04e989fa4
1 change: 0 additions & 1 deletion openfl/transport/grpc/aggregator_server.py
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,6 @@ def __init__(
TLS connection.
**kwargs: Additional keyword arguments.
"""
print(f"{use_tls=}")
self.aggregator = aggregator
self.uri = f"[::]:{agg_port}"
self.use_tls = use_tls
Expand Down
Loading