-
Notifications
You must be signed in to change notification settings - Fork 0
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
Added distance.cpp for the SphericalDistanceDeterminationAlgorithm #4
base: main
Are you sure you want to change the base?
Conversation
-Changed a few header files (namely attitude-utils) to add midpoint functions for Vec2 and Vec3
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.
Great job, just some changes before we test!
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.
Great job, just a few more styling issues!
commit 3863509 Author: Hieu Nguyen <[email protected]> Date: Mon Oct 14 12:11:23 2024 -0700 Pipeline Implementation (#7) * Added return to main * Squashed commit of the following: commit 86a843f Author: Tri Nguyen <[email protected]> Date: Sat Mar 25 17:24:22 2023 -0700 Update README to viewable version commit 00a9764 Merge: bc566cb 8eaa5cf Author: Tri Nguyen <[email protected]> Date: Sat Mar 25 16:37:50 2023 -0700 Merge pull request #2 from UWCubeSat/interface Testing Suite Setup * Squashed commit of the following: commit 7b3c7e1 Author: Tri Nguyen <[email protected]> Date: Fri Sep 13 10:51:58 2024 -0700 Refractoring Changes commit 86a843f Author: Tri Nguyen <[email protected]> Date: Sat Mar 25 17:24:22 2023 -0700 Update README to viewable version commit 00a9764 Merge: bc566cb 8eaa5cf Author: Tri Nguyen <[email protected]> Date: Sat Mar 25 16:37:50 2023 -0700 Merge pull request #2 from UWCubeSat/interface Testing Suite Setup * Merging from refractor * Pipeline Implementation * Made initial test and edits to pipeline.* * Use constants more * Squashed commit of the following: commit f370c67 Author: Tri Nguyen <[email protected]> Date: Sat Sep 21 16:21:59 2024 -0700 Refractor v2 (#6) commit 0a466d3 Author: Tri Nguyen <[email protected]> Date: Sun Sep 15 19:42:02 2024 -0700 Refractor Code Base for Implementation (#5) Merge from pipeline, refractoring changes PR: #5 commit 86a843f Author: Tri Nguyen <[email protected]> Date: Sat Mar 25 17:24:22 2023 -0700 Update README to viewable version commit 00a9764 Merge: bc566cb 8eaa5cf Author: Tri Nguyen <[email protected]> Date: Sat Mar 25 16:37:50 2023 -0700 Merge pull request #2 from UWCubeSat/interface Testing Suite Setup * Removed catch framework * Fixed minor issues * Documentation and Quality Changes * pending tests * pipeline tests 100% coverage, all functions work * Squashed commit of the following: commit 1d2c89b Author: Tri Nguyen <[email protected]> Date: Sat Oct 5 17:19:22 2024 -0700 Example Test bugs commit adece2b Author: Tri Nguyen <[email protected]> Date: Sat Oct 5 17:11:03 2024 -0700 Stage extension for all stages * Revert to C++11 * Resolving Changes --------- Co-authored-by: Tri Nguyen <[email protected]> commit f370c67 Author: Tri Nguyen <[email protected]> Date: Sat Sep 21 16:21:59 2024 -0700 Refractor v2 (#6) commit 0a466d3 Author: Tri Nguyen <[email protected]> Date: Sun Sep 15 19:42:02 2024 -0700 Refractor Code Base for Implementation (#5) Merge from pipeline, refractoring changes PR: #5 # Conflicts: # src/distance/distance.hpp # src/style/style.hpp
Just merged from main into this branch (took about an hour, you're welcome). Here's what else I've done:
I'm not sure if you saw the new message, but there are quite a few changes that have occurred with the code base. This is part of a modern refractor to leverage CI/CD in our project.
|
Description
-Changed a few header files (namely attitude-utils) to add midpoint functions for Vec2 and Vec3
-Line 49 of Distance.cpp python code was
center = np.linalg.solve(A, y)
, which i used the inverse matrix function to solve for instead.(Probably contains a few errors, if you see them feel free to point out)
Artifacts for PR #4 (DO NOT CHANGE)