Rust mobility module #95
Merged
Conversation
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
nbuffon
force-pushed
the
rust_mobility_module
branch
from
d4e3963
to
2c09940
Compare
nbuffon
added
documentation
Signed-off-by: Nicolas Buffon <nicolas.buffon@orange.com>
nbuffon
force-pushed
the
rust_mobility_module
branch
from
5573e56
to
5d80933
Compare
|
@Hugues360 @tigroo @tigroo31 ready for review |
Signed-off-by: Nicolas Buffon <nicolas.buffon@orange.com>
Signed-off-by: Nicolas Buffon <nicolas.buffon@orange.com>
Using [Criterion][1] to add benchmarks in `benches` folder Note that `benches` folder need to be copied in the build Docker image otherwise the build fails [1]: https://github.com/bheisler/criterion.rs Signed-off-by: Nicolas Buffon <nicolas.buffon@orange.com> Signed-off-by: Nicolas Buffon <nicolas.buffon@orange.com>
Signed-off-by: Nicolas Buffon <nicolas.buffon@orange.com>
nbuffon
force-pushed
the
rust_mobility_module
branch
from
c1ad495
to
b1c05ae
Compare
Renamed GeoExtension into Quadkey because it is what it is Move the whole thing into mobility mod as quadtree is a positioning solution Signed-off-by: Nicolas Buffon <nicolas.buffon@orange.com>
Signed-off-by: Nicolas Buffon <nicolas.buffon@orange.com>
Signed-off-by: Nicolas Buffon <nicolas.buffon@orange.com>
Using the name Quadtree, despite not being a real implementation of it, to match the mod and because we don't need a specific implementation of it yet or any parsing performance specificaly This could anyway be a good thing to bring a real tree implementation in the future Signed-off-by: Nicolas Buffon <nicolas.buffon@orange.com>
nbuffon
force-pushed
the
rust_mobility_module
branch
from
b1c05ae
to
861d7e4
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
New features
Positionstruct to manipulate geodesic coordinates using SI units instead of ETSIMobiletrait now uses SI units instead of ETSIGeoExtensionrenamed intoQuadkeyas it is now semantically considered a struct on its own and no longer a topic partQuadtreetype to hold a set ofQuadkeysHow to test
Nothing much to do other than checking that build, tests and benches runs correctly
cargo cleancargo buildcargo testcargo bench