-
-
Notifications
You must be signed in to change notification settings - Fork 307
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 a nannou_core
crate. Switch from cgmath
to glam
. Remove geom::Graph
. Remove generic Scalar param from Draw
API.
#754
Merged
Commits on Jun 20, 2021
-
Add a new
nannou_core
crate. Switch core fromcgmath
toglam
.See nannou-org#752 for motivation behind adding a `nannou_core` crate. Note that this commit does not yet replace the color, geom, math and rand modules within the `nannou` crate. This will be performed in a following commit.
Configuration menu - View commit details
-
Copy full SHA for 2f9671d - Browse repository at this point
Copy the full SHA 2f9671dView commit details -
Integrate
nannou_core
into mainnannou
crateThis removes the color, math, rand and geom modules in favour of depending on the `nannou_core` crate. The `nannou_core` modules are re-exported within `nannou` in order to provide a near identical API to the existing one. The majority of changes here revolve around the change from `cgmath` to `glam`. The most notable changes include: - `geom::Vector2` -> `glam::Vec2` - `cgmath::Matrix4<f32>` -> `glam::Mat4` - `magnitude` -> `length` This change also removes the generic `Scalar` type parameter from the `Draw` API and related items in the hope of improving downstream compile times. Also removes the seemingly unused `geom::graph` module. Use-cases for this module and its abstractions were mostly made reduntatnt upon allowing for transformations to be applied to the `Draw` instance itself.
Configuration menu - View commit details
-
Copy full SHA for 54732b2 - Browse repository at this point
Copy the full SHA 54732b2View commit details -
Refactor
Rect
andCuboid
methods forPoint*
andVec*
typesThis refactors the `Rect` and `Cuboid` methods to try to use `Point*` and `Vec*` types where possible for `Rect<f32>` and `Rect<f64>` in order to maintain an API closer to the original and avoid large sweeps amounts of breaking changes. Also exposes common `glam` types via the `nannou_core` `prelude` module.
Configuration menu - View commit details
-
Copy full SHA for 22257ac - Browse repository at this point
Copy the full SHA 22257acView commit details -
Configuration menu - View commit details
-
Copy full SHA for 7f996a2 - Browse repository at this point
Copy the full SHA 7f996a2View commit details -
Configuration menu - View commit details
-
Copy full SHA for 75ef26a - Browse repository at this point
Copy the full SHA 75ef26aView commit details -
Configuration menu - View commit details
-
Copy full SHA for f289271 - Browse repository at this point
Copy the full SHA f289271View commit details -
Configuration menu - View commit details
-
Copy full SHA for 0826ecd - Browse repository at this point
Copy the full SHA 0826ecdView commit details -
Configuration menu - View commit details
-
Copy full SHA for 96571b3 - Browse repository at this point
Copy the full SHA 96571b3View commit details -
Configuration menu - View commit details
-
Copy full SHA for 3a390da - Browse repository at this point
Copy the full SHA 3a390daView commit details -
Configuration menu - View commit details
-
Copy full SHA for ad99a32 - Browse repository at this point
Copy the full SHA ad99a32View commit details
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.