-
-
Notifications
You must be signed in to change notification settings - Fork 317
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
Add zooming & translation controls to Axis3 #4131
Merged
Merged
Changes from 51 commits
Commits
Show all changes
58 commits
Select commit
Hold shift + click to select a range
385a2b0
zoom prototype
ffreyer 00ed0e3
Merge branch 'master' into ff/Axis3-controls
ffreyer 7040ca7
add zooming as an Interaction
ffreyer 61ab665
add limit reset
ffreyer 6df5852
add translation
ffreyer b3db9d8
make translation more accurate
ffreyer d5c125a
add docstrings
ffreyer 199a9fc
fix x/y/z reversed
ffreyer 9eb6639
update changelog [skip ci]
ffreyer 9625654
Merge branch 'master' into ff/Axis3-controls
ffreyer 87aa380
move code to the right places
ffreyer 2510151
fix corrupted gl state
ffreyer 0ae0409
fix CairoMakie text - clip planes interaction
ffreyer f4553f3
remove comment
ffreyer f6eeafd
reuse and conform to existing infrastructure
ffreyer b1083b9
add docs for Axis3 interactions
ffreyer c95a439
add unit tests & fix zoom coordinate system
ffreyer c182e94
do plot picking ray cast as part of :cursor zoom
ffreyer 12d8d5b
remove old code
ffreyer bb8835e
match zoom direction with Axis, LScene
ffreyer 0abac18
update tests
ffreyer 9adcbe2
Merge branch 'master' into ff/Axis3-controls
SimonDanisch ac7d121
Merge branch 'master' into ff/Axis3-controls
ffreyer 982119e
Merge branch 'ff/Axis3-controls' of https://github.com/MakieOrg/Makie…
ffreyer 2646775
Merge branch 'master' into ff/Axis3-controls
ffreyer 6f6a068
Merge branch 'master' into ff/Axis3-controls
ffreyer a336fa3
add viewmode = :free
ffreyer 6450d12
tweak refimg to include perspectiveness and be less wide
ffreyer 05934bd
fix some test failures
ffreyer 323da39
revert change to decorations
ffreyer 10b9d9c
fix backend dependency of new refimg
ffreyer 14de291
Merge branch 'master' into ff/Axis3-controls
ffreyer 435ebb9
fix var name in new test
ffreyer 9071f6a
Merge branch 'ff/Axis3-controls' of https://github.com/MakieOrg/Makie…
ffreyer a755392
Merge branch 'master' into ff/Axis3-controls
ffreyer d598dcf
avoid rendering issues from too small near
ffreyer 965cee3
skip irrelevant code for viewmode = :free
ffreyer 3626949
fix aspect problems with translations
ffreyer 20c8039
cleanup some stuff
ffreyer 77984cd
reuse in-axis translations for viewmode = :free
ffreyer e283e96
add center-on-cursor interaction
ffreyer e8888b3
fix limit reset not resetting targetlimits
ffreyer 90762e3
fix position_on_plot error for 2D meshes
ffreyer 21815d4
move framelines to 3D scene to avoid line inversion issue
ffreyer 288ede8
Merge branch 'breaking-0.22' into ff/Axis3-controls
ffreyer 7f191d1
use Float64
ffreyer 19f2dd7
bandaid fix for render order
ffreyer 77e48e8
Change LimitReset to more closely match Axis
ffreyer 0ebf94e
update docs
ffreyer 3823a3f
Merge branch 'ff/Axis3-controls' of https://github.com/MakieOrg/Makie…
ffreyer 6c9a4f8
fix offset between ticks and grid/frame
ffreyer fabf49e
fix test error
ffreyer 838d04d
add test for project(scene, point)
ffreyer cfaabec
add option to clip decorations (default on)
ffreyer 2c698cf
use viewport for clipping
ffreyer 79e78f1
don't clip decorations by default because it clips titles and too muc…
ffreyer 49f5e2f
Merge branch 'breaking-0.22' into ff/Axis3-controls
ffreyer 6bfdce2
make test more robust
ffreyer File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Tick alignment with frame lines got worse when I moved the frame to data space. So I spent too long trying to figure out why.
I thought it might be a Float32 vs Float64 thing, so cleaned up the remaining Float32 types in Axis3. Didn't help. I tried forcing lines to convert on the CPU to make sure conversions don't happen till the end. Didn't help.
Then I got frustrated and moved the CairoMakie line point projection to Makie so I could revert moving the frame to data space without having clipping issues. And the alignment issues were still there. Then I tried using that code for ticks as well and it finally went away.
So i tried to figure out why and started comparing. Didn't notice the
resolution - 1
. Noticed that clip space points were very different, due to the CairoMakie code clipping to a -1..1 box. So I changed things up to avoid it and there was still a 0.5-1px difference. And then I finally noticed this line, changed it and I the issue seems to be fixed now.Anyway, fixes #3302.