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.

Sign up for GitHub

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

Jump to bottom

Fixes capture for filter_fn. #136

Merged

dvadym merged 1 commit into OpenMined:main from XinyuYe:fix-filter

Jan 14, 2022

Contributor

XinyuYe commented Jan 14, 2022

Description

Minor fix for the filter function, now we are directly using the budget in the outer scope.

Checklist

I have followed the Contribution Guidelines and Code of Conduct
I have commented my code following the OpenMined Styleguide
I have labeled this PR with the relevant Type labels
My changes are covered by tests


          Fixes capture for filter_fn.

001c58e

dvadym approved these changes

View reviewed changes

Collaborator

dvadym left a comment

Thank you!

dvadym merged commit 658d7a0 into OpenMined:main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet