Experiment.srts_from_df (which is a bad name) should allow the srt_at argument to take model-specific values. For example, if a model outputs SII values, then the SRT shouldn't be at 50% but at some value between 0 and 1. The current API is
We should allow for model specific criteria, like: {('Model', 'Output'}: criterion'}. This has the downside that if the "default" srt_at shouldn't be 50, then all models must be part of the dictionary. Therefore, adding another keyword argument might be a better idea.
Experiment.srts_from_df
(which is a bad name) should allow thesrt_at
argument to take model-specific values. For example, if a model outputs SII values, then the SRT shouldn't be at 50% but at some value between 0 and 1. The current API isWe should allow for model specific criteria, like:
{('Model', 'Output'}: criterion'}
. This has the downside that if the "default"srt_at
shouldn't be 50, then all models must be part of the dictionary. Therefore, adding another keyword argument might be a better idea.