-
Notifications
You must be signed in to change notification settings - Fork 1
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
More Makie integration #6
Merged
Merged
Changes from all commits
Commits
Show all changes
29 commits
Select commit
Hold shift + click to select a range
30dbde4
Run all documentation steps under xvfb
asinghvi17 12af00d
Add Makie packages to docs project
asinghvi17 1154a4b
Copy the README over to docs/index.md
asinghvi17 1a39a20
Add a Makie demo page
asinghvi17 7e53145
Ensure all camera attributes are updated in Makie
asinghvi17 587c797
Add recipe header function definitions
asinghvi17 5cf2b9e
Define `duration` for paths
asinghvi17 cdef405
Define a Makie recipe to plot camera paths
asinghvi17 371bdaf
Add a Makie recipe to visualize camera paths
asinghvi17 2ff8622
Add a basic example to the doc page
asinghvi17 d40ce35
Add camera animation
asinghvi17 e69d663
Add converts and a `record` shortcut
asinghvi17 36a20b5
Add a view frustum visualization too
asinghvi17 7d3f852
Remove pipes
asinghvi17 6d11a18
Add developer docs
asinghvi17 7396e6c
julia -> repl blocks in docs md
asinghvi17 50e4486
Move interface definitions to `interfaces.jl` as function stubs
asinghvi17 a2fab89
Remove recipe.jl
asinghvi17 f259b56
Add comments + more methods to method error hint
asinghvi17 28a6e03
Document `PathChange`
asinghvi17 016629f
Update CI.yml
asinghvi17 88f2935
Enable preview docs
asinghvi17 bcdff2b
Fix YAML syntax error
asinghvi17 450131a
More PathChange docs
asinghvi17 84eb0c0
Build docs with `warnonly=true`
asinghvi17 2879d35
Add docstring to `Path`
asinghvi17 6b8e9f1
Update makie.md
asinghvi17 6cae4cf
Try to fix doctest CI
asinghvi17 432b8c0
Add more docstrings + document functions not structs
asinghvi17 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,3 +1,7 @@ | ||
[deps] | ||
CairoMakie = "13f3f980-e62b-5c42-98c6-ff1f3baf88f0" | ||
Documenter = "e30172f5-a6a5-5a46-863b-614d45cd2de4" | ||
FlyThroughPaths = "c11bb9a7-2755-425a-88f3-ebe93bbdb91f" | ||
GLMakie = "e9467ef8-e4e7-5192-8a1a-b1aee30e663a" | ||
Makie = "ee78f7c6-11fb-53f2-987a-cfe4a2b5a57a" | ||
MakieCore = "20f20a25-4f0e-4fdf-b5d1-57303727442b" |
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,16 @@ | ||
# Developer documentation | ||
|
||
## Implementing support for FlyThroughPaths | ||
|
||
FlyThroughPaths operates on the [`ViewState`](@ref) model. In order to implement support for this in a plotting package, you must implement dispatches for the following two functions: | ||
- `capture_view(obj)::ViewState`: extract the current `ViewState`, i.e., camera settings, from `obj`. | ||
- `set_view!(obj, viewstate::ViewState)`: set the camera to the given `ViewState`. | ||
|
||
Integration is already implemented for Makie; you can see that in `ext/FlyThroughPathsMakieExt.jl`. The first ~20 lines are the most instructive, beyond which lie utility functions and visualization specializations. | ||
|
||
## The `PathChange` interface | ||
|
||
```@docs | ||
PathChange | ||
duration | ||
``` |
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.
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.
With the move to Documenter docs, I think we could also slim the README a lot. Otherwise we have two places we have to keep updated.
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.
I can definitely slim the README down (and translate the examples to Documenter syntax)!
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.
Example translation should be done, but I still have to slim down the README.