Skip to content

Commit

Permalink
Some cleanup
Browse files Browse the repository at this point in the history
Summary:
Some misc cleanup before I extend some model selection features:
* Consolidates TKwargs type
* Simplifies the verify() method's inputs to just take a list of requested models, not the whole candidates request. Reduces confusion whether this function depends on having updated/properly candidate requests (it doesn't)

Reviewed By: ldworkin

Differential Revision: D29278405

fbshipit-source-id: 87d16ee63968a6be09575a60acfe91569e69b900
  • Loading branch information
Steve Mandala authored and facebook-github-bot committed Jun 22, 2021
1 parent f504974 commit 5d55daf
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions ax/utils/common/kwargs.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,8 @@

logger = get_logger(__name__)

TKwargs = Dict[str, Any]


def consolidate_kwargs(
kwargs_iterable: Iterable[Optional[Dict[str, Any]]], keywords: Iterable[str]
Expand Down

0 comments on commit 5d55daf

Please sign in to comment.