Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork7.9k
Closed as not planned
Labels
Description
Update: See summary in#26187 (comment)
Problem
In#21266 (v3.6) we prohibited negative errors to bring the behavior in line with the documentation.
I've got the request to again support negative errors, so that the error bar can be detached from the marker. There seem to be cases where users exactly want this behavior, e.g.
Proposed solution
I think we should support this somehow. Possibilities:
- Revert to the old behavior and adjust the documentation accordingly. I'm not clear whether not allowing negative errors was really a conscious decision or whether we were only making the code behave according to the documentation. - Technically, negative errors are perfictly fine. The only justification I see for the positive error limitation is to enforce a more narrow semantics that errors are usually positive and we want to guard users against accidental negative errors.
- Alternatively, we could introduce an explicit
allow_negative_errors
flag.
Choosing between those, I'm inclined to say that the flag feels a bit cumbersome and we can assume that users are responsible enough to handle their data without us needing to enforce negative errors. After all, even if they are undesired, it should be relatively easy to spot and find out the reason.