Skip to content
This repository has been archived by the owner on Nov 16, 2024. It is now read-only.

Remove hardcoded intervals #25

Closed
dajmcdon opened this issue Sep 26, 2024 · 1 comment · Fixed by #29
Closed

Remove hardcoded intervals #25

dajmcdon opened this issue Sep 26, 2024 · 1 comment · Fixed by #29
Assignees

Comments

@dajmcdon
Copy link
Collaborator

  • level is a mandatory arg to the method. This should be used to select the confidence level.
  • It may also be nice to allow multiple levels rather than one (but this complicates plotting).
@dajmcdon dajmcdon self-assigned this Sep 26, 2024
@dajmcdon dajmcdon linked a pull request Sep 27, 2024 that will close this issue
@kaitejohnson
Copy link
Collaborator

Was thinking about this-- I think we want to make sure that all of the sumrmt_summary objects are by default generating the same CI band level (I had assumed they were all 95% CI but now am not sure this was ever verified). I think it makes sense for this to be the default but allow the user to specify.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants