SQL generation bug (or maybe not a bug...) on ATLAS

Cong_Liu · September 26, 2019, 1:19am

I just realized there is a bug in ATLAS. I found when there is no measurement value supplied (leave the “with value as the number” box empty), you can still click the generate button. However, the backend will return the “Nullpointer” error. It turns out when there is no measurement value (the box leave blank), the SQL generation process will fail (500 error on server-side, /webapi/sql). I know, users should always supply the measurement value if they provide this criterion. so I don’t know whether we should say this is a bug.

See example below.
http://www.ohdsi.org/web/atlas/#/cohortdefinition/1772144

Maybe make this box mandatory? if the “with value as the number” attribute added.

Cong

Chris_Knoll · September 26, 2019, 9:25pm

Hi, @Cong_Liu:
We have a simple cohort validation implemetnation, where we could look for missing values in criteria before posting. This check is missing, but it is definitely something that can be added.

In the future, once we get ore certain that the warnings and errors are 100% accurate, we’ll disable generation buttons when we identify an error conditions in the cohort. But we aren’t there yet.

-Chris