serofoi icon indicating copy to clipboard operation
serofoi copied to clipboard

Improve error messages for `plot_foi_estimates` and `plot_rhats`

Open jpavlich opened this issue 1 year ago • 2 comments

When passing constant seromodel as parameter to plot_foi_estimates and plot_rhats, it yields the following error

Error in plot_foi_estimates(seromodel, serosurvey) : 
  seromodel@name should start with either 'age' or 'time'

It is a little unintuitive, since it assumes the user know that seromodel@model_name begins with 'age' or 'time' for age or time models, respectively.

I would suggest changing it to "seromodel type must be 'age' or 'time'"

jpavlich avatar Oct 21 '24 20:10 jpavlich

@ntorresd If you agree, you can assign me this issue. I will fix it as soon as #215 is merged into dev

jpavlich avatar Oct 21 '24 20:10 jpavlich

#223 has been merged to dev. Please proceed with this @jpavlich.

ntorresd avatar Nov 03 '24 15:11 ntorresd