Skip to content

Commit

Permalink
use generation_node_name on the GeneratorRun (facebook#2003)
Browse files Browse the repository at this point in the history
Summary:

Replace the use of generation_step_index with generation_node_name because GenerationStrategy should no longer access index

Differential Revision: D51432075
  • Loading branch information
Mia Garrard authored and facebook-github-bot committed Nov 28, 2023
1 parent 18788f5 commit 65b220b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion ax/modelbridge/generation_strategy.py
Original file line number Diff line number Diff line change
Expand Up @@ -510,7 +510,7 @@ def _get_model_state_from_last_generator_run(self) -> Dict[str, Any]:
model_on_curr = self._curr.model_enum
if (
lgr is not None
and lgr._generation_step_index == self._curr.index
and lgr._generation_node_name == self._curr.node_name
and lgr._model_state_after_gen
):
if self.model or isinstance(model_on_curr, ModelRegistryBase):
Expand Down

0 comments on commit 65b220b

Please sign in to comment.