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

What is the refactored name of federated_output_computation? #4161

Closed
mahimairaja opened this issue Oct 2, 2023 · 5 comments
Closed

What is the refactored name of federated_output_computation? #4161

mahimairaja opened this issue Oct 2, 2023 · 5 comments

Comments

@mahimairaja
Copy link

I could see that federated_output_computation is depreciated. So what could I use instead of this?

@zcharles8
Copy link
Collaborator

@mahimairaja Can you clarify what part of the API you're referring to?

@mahimairaja
Copy link
Author

I am mentioning the function that has depreciated in the previous version - https://github.com/tensorflow/federated/blob/846d00939ff87d675942a52182b65babdecdf4aa/RELEASE.md?plain=1#L1036

@mahimairaja
Copy link
Author

@zcharles8 Can you help me with this?

@zcharles8
Copy link
Collaborator

As the release notes say, you should use report_local_unfinalized_metrics and metric_finalizers now instead if you're directly subclassing tff.learning.models.VariableModel.

However, if you use tff.learning.models.from_keras_model then this is taken care of for you.

@zcharles8
Copy link
Collaborator

@mahimairaja I believes this answers the question in the original github issue, so I'm marking this as resolved.

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

No branches or pull requests

2 participants