Hydra: Fix Nevergrad sweeper to pass common sweeper tests

Created on 7 May 2020  路  5Comments  路  Source: facebookresearch/hydra

576 introduces common sweeper tests that should pass for all Sweeper.

Currently those tests are failing for the Nevergrad sweeper, please re-enable the tests (commented out in #576) and fix the Nevergrad sweeper such that they pass.

Most helpful comment

I suggest you wait until @shagunsodhani implements this for the Ax sweeper, no point in both of you trying to make it work at the same time.
Let's see what he ends up with for Ax first.

All 5 comments

Is it used anywhere and working? The tests don't even seem to pass when using the basic hydra sweeper

It does , but not with overriding the function:

Basuc launcher test (uses the basic sweeper by default)

Example sweeper test

I am not seeing an example when I overridden the task_function, so there could still be some issues with that.

I suggest you wait until @shagunsodhani implements this for the Ax sweeper, no point in both of you trying to make it work at the same time.
Let's see what he ends up with for Ax first.

Yeah I plan to work on it tonight.

Closing for now. we can revisit what improving testing for optimizing sweepers once they have a common library.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

emilemathieu picture emilemathieu  路  4Comments

vict0rsch picture vict0rsch  路  4Comments

natolambert picture natolambert  路  4Comments

tamuhey picture tamuhey  路  4Comments

nathansomavarapu picture nathansomavarapu  路  3Comments