Use ModifiedUpdate in dynamic tutorials#21363
Use ModifiedUpdate in dynamic tutorials#21363couet wants to merge 4 commits intoroot-project:masterfrom
Conversation
couet
changed the title
|
Ah ... the python version is different form the C++ one ... not sure why .. it should in principle. The C++ version has only one ModifiedUpdate. I can apply wht you suggest in the python version but I do not garanty the result ... |
|
how do you run |
The difference is because of how the Python interpreter vs the ROOT interpreter work. There is no need to "guarantee", everything stays exactly the same. The only extra bit is one single line of code at the end of the script, as I suggested. This is the whole reason why all of this started, see #21243, so it's necessary to add that line to avoid the canvas randomly dying at the end of the Python script. |
This is not because of Python, nor Cocoa, nor MacOS. This is how ROOT works. webgraphics are the default in master, you have to deactivate them to get the traditional graphics. |
Test Results 22 files 22 suites 3d 5h 34m 23s ⏱️ For more details on these failures, see this check. Results for commit dbb8d20. |
vepadulano
left a comment
vepadulano
left a comment
There was a problem hiding this comment.
Thanks! Changes LGTM but before merging please squash the history such that there are only 2 commits:
- One with the changes related to
ModifiedUpdate. - And another one just with the one extra line of Python code
c1.Draw(block=True)explaining that this is done to let the canvas survive before the Python script is exited.
2 participants
Use ModifiedUpdate in dynamic tutorials to make sure thy work on Mac also.