Skip to content
This repository has been archived by the owner on Sep 18, 2024. It is now read-only.

Dedup for evolution search #5092

Merged
merged 7 commits into from
Oct 12, 2022
Merged

Conversation

ultmaster
Copy link
Contributor

Description

Add deduplication for evolutionary search.

Test Options

  • fast test
  • full test - HPO
  • full test - NAS
  • full test - compression

Checklist

  • test case
  • doc

How to test

@ultmaster ultmaster marked this pull request as ready for review August 31, 2022 07:02
@QuanluZhang QuanluZhang added v2.9.1 and removed v2.10 labels Sep 7, 2022
@cruiseliu
Copy link
Contributor

What will happen if the search space is exhausted?
HPO has a dedup util: https://github.com/microsoft/nni/blob/master/nni/common/hpo_utils/dedup.py

@ultmaster
Copy link
Contributor Author

What will happen if the search space is exhausted? HPO has a dedup util: https://github.com/microsoft/nni/blob/master/nni/common/hpo_utils/dedup.py

It will give up the dedup, and fallback to the original "un-dedup" process.

I read the HPO dedup util. It's hard to reuse it because the format is not unified. Also, the generated config uses mutator as keys, and mutators are not JSON-serializable. (I'm working on both problems in v3.0 iteration.)

@@ -43,6 +46,10 @@ class RegularizedEvolution(BaseStrategy):
The number of individuals that should participate in each tournament. Default: 25.
mutation_prob : float
Probability that mutation happens in each dim. Default: 0.05
dedup : bool
Do not try the same configuration twice. When variational is true, deduplication is not supported. Default: true.
Copy link
Contributor

Choose a reason for hiding this comment

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

what is "variational"?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Mis-copied. Removed.

@ultmaster ultmaster merged commit aa98462 into microsoft:master Oct 12, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants