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

params/metrics: support top-level dicts #9453

Closed

dberenbaum wants to merge 1 commit into main from top-level-params-dict

Collaborator

dberenbaum commented May 15, 2023

Fixes #9452


          params/metrics: support top-level dicts

586344d

dberenbaum requested a review from skshetry

May 15, 2023 14:13

Collaborator Author

dberenbaum commented May 15, 2023

Looks like this is more complex than I thought. Surprised it worked at the directory level but broke tests at the file level.

dberenbaum closed this

skshetry deleted the top-level-params-dict branch

June 26, 2023 10:10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet