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.

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

Merge exp into flaml #210

Merged
merged 1,140 commits into from
Nov 16, 2021
Merged

Merge exp into flaml #210

merged 1,140 commits into from
Nov 16, 2021

Conversation

liususan091219
Copy link
Collaborator

No description provided.

sonichi
sonichi requested changes Sep 19, 2021
View reviewed changes
Copy link
Collaborator

@sonichi sonichi left a comment

Choose a reason for hiding this comment

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

This is round 1 of review.
Please also follow the pre-commit instructions as my comments of #208

flaml/nlp/hpo/get_grid_search_space.py Outdated Show resolved Hide resolved
flaml/nlp/hpo/searchalgo_auto.py Outdated Show resolved Hide resolved
flaml/nlp/result_analysis/azure_utils.py Outdated Show resolved Hide resolved
flaml/nlp/result_analysis/azure_utils.py Outdated Show resolved Hide resolved
flaml/nlp/result_analysis/azure_utils.py Outdated Show resolved Hide resolved
flaml/nlp/result_analysis/azure_utils.py Outdated Show resolved Hide resolved
flaml/nlp/result_analysis/azure_utils.py Outdated Show resolved Hide resolved
test/hf/run_analysis.py Outdated Show resolved Hide resolved
test/hf/run_analysis.py Outdated Show resolved Hide resolved
test/hf/run_autohf.py Outdated Show resolved Hide resolved
sonichi
sonichi requested changes Sep 23, 2021
View reviewed changes
Copy link
Collaborator

@sonichi sonichi left a comment

Choose a reason for hiding this comment

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

More changed requested.

.pre-commit-config.yaml Outdated Show resolved Hide resolved
benchmarkeval/amlk8s_hyperpartisan.yml Outdated Show resolved Hide resolved
sonichi
sonichi reviewed Oct 1, 2021
View reviewed changes
Copy link
Collaborator

@sonichi sonichi left a comment

Choose a reason for hiding this comment

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

partial review

flaml/nlp/autotransformers.py Outdated Show resolved Hide resolved
flaml/nlp/autotransformers.py Outdated Show resolved Hide resolved
flaml/nlp/autotransformers.py Outdated Show resolved Hide resolved
sonichi
sonichi reviewed Oct 2, 2021
View reviewed changes
Copy link
Collaborator

@sonichi sonichi left a comment

Choose a reason for hiding this comment

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

review in progress

flaml/nlp/autotransformers.py Outdated Show resolved Hide resolved
flaml/nlp/autotransformers.py Outdated Show resolved Hide resolved
flaml/nlp/autotransformers.py Outdated Show resolved Hide resolved
flaml/nlp/autotransformers.py Outdated Show resolved Hide resolved
flaml/nlp/autotransformers.py Outdated Show resolved Hide resolved
flaml/nlp/autotransformers.py Outdated Show resolved Hide resolved
flaml/nlp/autotransformers.py Outdated Show resolved Hide resolved
flaml/nlp/autotransformers.py Outdated Show resolved Hide resolved
flaml/nlp/autotransformers.py Outdated Show resolved Hide resolved
flaml/nlp/autotransformers.py Outdated Show resolved Hide resolved
sonichi
sonichi reviewed Oct 2, 2021
View reviewed changes
Copy link
Collaborator

@sonichi sonichi left a comment

Choose a reason for hiding this comment

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

two files reviewed.

flaml/nlp/autotransformers.py Outdated Show resolved Hide resolved
flaml/nlp/autotransformers.py Outdated Show resolved Hide resolved
flaml/nlp/autotransformers.py Outdated Show resolved Hide resolved
flaml/nlp/autotransformers.py Outdated Show resolved Hide resolved
flaml/nlp/autotransformers.py Outdated Show resolved Hide resolved
flaml/nlp/dataset/dataprocess_auto.py Outdated Show resolved Hide resolved
flaml/nlp/dataset/dataprocess_auto.py Outdated Show resolved Hide resolved
flaml/nlp/dataset/dataprocess_auto.py Outdated Show resolved Hide resolved
flaml/nlp/dataset/dataprocess_auto.py Outdated Show resolved Hide resolved
flaml/nlp/dataset/dataprocess_auto.py Outdated Show resolved Hide resolved
sonichi
sonichi reviewed Oct 3, 2021
View reviewed changes
Copy link
Collaborator

@sonichi sonichi left a comment

Choose a reason for hiding this comment

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

autotransformers

flaml/nlp/autotransformers.py Outdated Show resolved Hide resolved
@liususan091219
Merge branch 'exp_928d' of https://github.com/liususan091219/FLAML in…
acfe48b 
…to exp_928d
@liususan091219
Merge remote-tracking branch 'origin/exp_928d' into exp_928d
e5af82c
@liususan091219
Merge branch 'exp_928d' of https://github.com/liususan091219/FLAML in…
caedac0 
…to exp_928d
@liususan091219
Merge branch 'exp_928d' of https://github.com/liususan091219/FLAML in…
4b1c782 
…to exp_928d
@liususan091219
Merge branch 'exp_928d' of https://github.com/liususan091219/FLAML in…
ee14e28 
…to exp_928d
@liususan091219
Merge branch 'exp_928d' of https://github.com/liususan091219/FLAML in…
0a495e4 
…to exp_928d
@liususan091219
Merge remote-tracking branch 'origin/exp_928d' into exp_928d
2a45a15
@liususan091219
Merge branch 'exp_928d' of https://github.com/liususan091219/FLAML in…
f50eb1e 
…to exp_928d
@liususan091219
Merge branch 'exp_928d' of https://github.com/liususan091219/FLAML in…
7a76164 
…to exp_928d
@liususan091219
Merge remote-tracking branch 'origin/exp_928d' into exp_928d
813a55d
@liususan091219
Merge remote-tracking branch 'origin/exp_928d' into exp_928d
a553aec
@liususan091219
Merge remote-tracking branch 'origin/exp_928d' into exp_928d
2ea669e
@liususan091219
Merge remote-tracking branch 'origin/exp_928d' into exp_928d
bc12c2b
@liususan091219
Merge remote-tracking branch 'origin/exp_928d' into exp_928d
71e6263
@liususan091219
Merge remote-tracking branch 'origin/exp_928d' into exp_928d
a772e2e
@liususan091219
Merge remote-tracking branch 'origin/exp_928d' into exp_928d
9ba730b
@liususan091219
Merge remote-tracking branch 'origin/exp_928d' into exp_928d
1a9d955
@liususan091219
Merge remote-tracking branch 'origin/exp_928d' into exp_928d
b66d97c
@liususan091219
Merge remote-tracking branch 'origin/exp_928d' into exp_928d
dc76a93
@liususan091219
Merge remote-tracking branch 'origin/exp_928d' into exp_928d
1dc5475
@liususan091219
Merge remote-tracking branch 'origin/exp_928d' into exp_928d
1f96996
@liususan091219
Merge remote-tracking branch 'origin/exp_928d' into exp_928d
fd8aa29
@liususan091219
Merge remote-tracking branch 'origin/exp_928d' into exp_928d
d19c757
@liususan091219
Merge remote-tracking branch 'origin/exp_928d' into exp_928d
d46b1e3
@sonichi sonichi requested a review from a team November 15, 2021 05:41
liususan091219 and others added 6 commits November 15, 2021 00:45
@liususan091219 @sonichi
Update flaml/model.py
409382b 
Co-authored-by: Chi Wang <wang.chi@microsoft.com>
@liususan091219 @sonichi
Update flaml/nlp/huggingface/trainer.py
57c968b 
Co-authored-by: Chi Wang <wang.chi@microsoft.com>
@liususan091219 @sonichi
Update flaml/model.py
edd1dfd 
Co-authored-by: Chi Wang <wang.chi@microsoft.com>
@liususan091219
fix conflict
f34d481
@liususan091219
Merge remote-tracking branch 'origin/exp_928d' into exp_928d
a93156e
@liususan091219
fix conflict
312cf99
sonichi
sonichi reviewed Nov 15, 2021
View reviewed changes
flaml/automl.py Outdated Show resolved Hide resolved
sonichi
sonichi reviewed Nov 15, 2021
View reviewed changes
flaml/automl.py Outdated Show resolved Hide resolved
sonichi
sonichi requested changes Nov 16, 2021
View reviewed changes
flaml/automl.py Outdated Show resolved Hide resolved
flaml/data.py Outdated Show resolved Hide resolved
flaml/data.py Outdated Show resolved Hide resolved
@sonichi sonichi requested a review from a team November 16, 2021 15:31
@sonichi sonichi merged commit 42de307 into microsoft:main Nov 16, 2021
@sonichi sonichi deleted the exp_928d branch November 16, 2021 19:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sonichi sonichi sonichi approved these changes

@qingyun-wu qingyun-wu Awaiting requested review from qingyun-wu

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@liususan091219 @sonichi