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

Enabled personalized policy for fedex #481

Merged
merged 4 commits into from
Jan 13, 2023

Conversation

joneswong
Copy link
Collaborator

TODO: calculate the client encoding based on client-wise data

@joneswong joneswong added the Feature New feature label Dec 29, 2022
@joneswong
Copy link
Collaborator Author

@TheSunWillRise please help us review this pr. Thanks!

Copy link
Collaborator

@rayrayraykk rayrayraykk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, please remove [WIP] from the caption of this PR.

@@ -58,6 +58,7 @@ def callback_funcs_for_model_para(self, message: Message):
return_raw=True))

results['arms'] = arms
results['client_id'] = self.ID - 1
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why?

@joneswong joneswong changed the title [WIP] enabled personalized policy for fedex Enabled personalized policy for fedex Jan 13, 2023
@joneswong joneswong added the FedHPO FedHPO related label Jan 13, 2023
@rayrayraykk rayrayraykk merged commit f09348b into alibaba:master Jan 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature New feature FedHPO FedHPO related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants