Skip to content
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

CI check that metrics defined in a custom Jetstream config are required to have associated statistical treatments #353

Open
danielkberry opened this issue Feb 6, 2024 · 1 comment
Open

CI check that metrics defined in a custom Jetstream config are required to have associated statistical treatments #353

danielkberry opened this issue Feb 6, 2024 · 1 comment

Comments

@danielkberry
Copy link
Contributor

Currently, a user can define a metric and add it to the list of experiment metrics, but if that metric is missing a statistical treatment it will be silently dropped and no analysis will be done on it. Users intend to do analysis with these metrics.
@mikewilli
Copy link
Collaborator

Hopefully this will be solved by the solution described in #354

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@danielkberry @mikewilli