-
Notifications
You must be signed in to change notification settings - Fork 1.4k
[hist] clip color to minimum if below with COLZ TH2 #19441
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
Fixes root-project#19438 and improve docu
Test Results 21 files 21 suites 3d 13h 18m 34s ⏱️ For more details on these failures, see this check. Results for commit c460faf. ♻️ This comment has been updated with latest results. |
Thks couet
Note: I am looking at it also. There in an issue with option TEXT also. |
@@ -1110,6 +1110,11 @@ painted with the color corresponding to the new maximum. | |||
When the minimum of the histogram is set to a greater value than the real minimum, | |||
the bins having a value between the real minimum and the new minimum are not drawn | |||
unless the option `0` is set. | |||
In other words, option `COLZ0` forces the painting of bins with content < set minimum with |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why adding this note ? it repeats what is already said
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It does not specify that it depends on whether minimum is negative or positive, right?
hcol22->SetMaximum(100); | ||
hcol22->SetMinimum(40); | ||
} | ||
End_Macro | ||
Note that the behavior of `COLZ` is not symmetric: it does not draw values below the specified minimum, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same comment. The original doc says it already.
Can you add the fix to option TEXT ?
and at below (near line 10154) replace:
by
|
Sure, thanks, I can add that |
Fix by couet
This Pull request:
Changes or fixes:
Fixes #19438
and improve docu
fyi @azontikov
Checklist: