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.

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

Better error messages for misconfigured metric providers. #559

Open
mikelehen opened this issue Jan 24, 2023 · 0 comments
Open

Better error messages for misconfigured metric providers. #559

mikelehen opened this issue Jan 24, 2023 · 0 comments
Labels
metrics packages A task for act-now-packages repo
Milestone

Comments

@mikelehen
Copy link
Contributor

mikelehen commented Jan 24, 2023

I set the regionColumn to "regionId" instead of "region_id" and got the error:

useDataForRegionsAndMetrics(regions=Alabama (regionId=01),Alaska (regionId=02),Arizona (regionId=04),Arkansas (regionId=05),Californi..., metrics=opioid_deaths) failed: Error: INTERNAL ASSERTION FAILED: One or more data rows were missing a region id value in column regionId

It would be nice if:

  1. This wasn't an "INTERNAL ASSERTION"
  2. It specified which provider was misconfigured.
  3. It more specifically said that the regionId column didn't exist at all (instead of that some rows were missing a value).
@mikelehen mikelehen added packages A task for act-now-packages repo metrics labels Jan 24, 2023
@mikelehen mikelehen added this to the launch milestone Jan 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
metrics packages A task for act-now-packages repo
Projects
None yet
Development

No branches or pull requests

1 participant