search

RhoInc / aeexplorer

Adverse Event Explorer
https://rhoinc.github.io/aeexplorer/test-page/
MIT License
17 stars 9 forks source link

Prevent charts with too many groups when the group variable changes #133

Closed jwildfire closed 6 years ago

jwildfire commented 6 years ago

Not sure if we should prevent variables with too many categories from being added to the control, or if we should just show a message that there are too many groups when changing to that control. Or we could possibly just pick the first N levels of the variable and show those groups ...

jwildfire commented 6 years ago

Test Notes

Confirm that the chart doesn't draw when the user selects a group variable with more levels than config.defaults.maxGroups by changing the group by variable to AGE using the settings below in CAT. Also confirm that you can then change back to another variable and the chart renders as expected. 

{
    variables:{group:"ARM"},
    variableOptions:{
      group:["RACE","SEX","AGE"]
  },
};
rchronowic commented 6 years ago

Testing for this issue passed in CAT for both Firefox and Chrome. See below screen shot for Error message.

image