Physics unit tests #6
Conversation
ReeceHumphreys
force-pushed
the
physics-unit-tests
branch
from
196ab14 to
5841e27
Compare
| pub struct Quaternion { | ||
| pub data: na::Vector4<f64>, | ||
| } | ||
|
|
||
| #[cfg(test)] |
There was a problem hiding this comment.
why are we editing numerics/quaternion in this pr? Why isn't it just in the numerics unit test pr?
There was a problem hiding this comment.
This PR was just based on the numerics branch so I got the dependency changes. Once that is merged ill just rebase it off of main.
| @@ -37,3 +37,125 @@ pub fn quaternion_derivative<T: SpacecraftProperties>(state: &State<T>) -> Quate | |||
| // Only use body angular velocity for quaternion propagation | |||
| compute_quaternion_derivative(&state.quaternion, &state.angular_velocity) | |||
| } | |||
|
|
|||
| #[cfg(test)] | |||
| mod tests { | |||
There was a problem hiding this comment.
Please comment explaining the different unit tests for quick readability. Also like the other PR explain/document the test value you used and how you decided upon or found them. Rn just a script in the google drive is sufficient
There was a problem hiding this comment.
Will do. Will merge once I've uploaded my scripts and writeup to the google drive. The gist is I used a hybrid of some python scripts and hand calculations to verify values. I also used a few examples from Vallado.
Ignore the changes to numerics. This PR would be merged after the numerics. I just based this branch off of the numerics so the cargo.toml updates were included!
Edit: Need to recommit the orbital mechanics tests