mozilla / jetstream

This machine kills superstition. 💨
https://experimenter.info/jetstream/jetstream
Mozilla Public License 2.0
22 stars 18 forks source link

bump mozanalysis version, add additional fallback case #2086

Closed danielkberry closed 1 month ago

danielkberry commented 1 month ago

Incorporates additional error checks from most recent mozanalysis version. Also catches case where covariate adjustment was configured correctly, but metric was also configured for the other preenrollment analysis basis. In that case, we do not want to attempt to adjust inferences for PREENROLLMENT_DAYS_28 using PREENROLLMENT_WEEK. Additionally, adds check for missing covariate column, logs a warning and falls back to unadjusted inferences in that case.

Added 2 unit tests for the new fallback cases.