Hi everyone,
this is a suggestion for a minor change.
median_hilow is reexported and wrapped from Hmisc::smedian.hilow. This function defaults to conf.int = .95.
My expected behavior would be that the function returns the range of the data for ymin and ymax. This is currently not the case, but can be achieved by setting conf.int = 1. This would be the behavior I expect and this could possibly be a source of confusion.
I would have some suggestions how to deal with this:
- Changing the default to
conf.int = 1, but this would break backwards compatibility
- Adding another function
median_minmax or median_range. Probably better, but results in similarly named functon with different behavior.
- Adding a notice to the docs.
- Add an example how to set the conf.int.
- Or an example with
fun.min = min, fun.max = max.
I am happy to help with the implementation of these or other solutions. If you consider this a non-problem please close this issue.
Greetings
Alex
Hi everyone,
this is a suggestion for a minor change.
median_hilowis reexported and wrapped fromHmisc::smedian.hilow. This function defaults toconf.int = .95.My expected behavior would be that the function returns the range of the data for ymin and ymax. This is currently not the case, but can be achieved by setting
conf.int = 1. This would be the behavior I expect and this could possibly be a source of confusion.I would have some suggestions how to deal with this:
conf.int = 1, but this would break backwards compatibilitymedian_minmaxormedian_range. Probably better, but results in similarly named functon with different behavior.fun.min = min, fun.max = max.I am happy to help with the implementation of these or other solutions. If you consider this a non-problem please close this issue.
Greetings
Alex