Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fix and add basic test for ins kinematic eq #700

Merged
merged 5 commits into from
Sep 11, 2023
Merged

Fix and add basic test for ins kinematic eq #700

merged 5 commits into from
Sep 11, 2023

Conversation

Affie
Copy link
Member

@Affie Affie commented Sep 5, 2023

No description provided.

dehann
dehann approved these changes Sep 5, 2023
View reviewed changes
Copy link
Member

@dehann dehann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops, apologies for wrong variable names. Looks good to me

@dehann dehann added this to the v0.23.0 milestone Sep 5, 2023
dehann
dehann reviewed Sep 5, 2023
View reviewed changes
Affie and others added 2 commits September 5, 2023 20:16
@Affie
fix i_R_b w_R_b
5645318
@Affie @dehann
Update test/inertial/testODE_INS.jl
dcb822d 
Co-authored-by: Dehann Fourie <6412556+dehann@users.noreply.github.com>
@codecov
Copy link

codecov bot commented Sep 5, 2023 •
edited
Loading

Codecov Report

Merging #700 (5645318) into master (e066c4f) will not change coverage.
Report is 3 commits behind head on master.
The diff coverage is n/a.

❗ Current head 5645318 differs from pull request most recent head c94b812. Consider uploading reports for the commit c94b812 to get more accurate results

@@           Coverage Diff           @@
##           master     #700   +/-   ##
=======================================
  Coverage   39.85%   39.85%           
=======================================
  Files          53       53           
  Lines        2065     2065           
=======================================
  Hits          823      823           
  Misses       1242     1242

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@Affie Affie marked this pull request as ready for review September 11, 2023 09:25
@Affie Affie merged commit 93ecb12 into master Sep 11, 2023
@Affie Affie deleted the 23Q3/fac/ins_ode branch September 11, 2023 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dehann dehann dehann approved these changes

Assignees
No one assigned
Labels
differential equations imu
Projects
None yet
Milestone
v0.23.0
Development

Successfully merging this pull request may close these issues.
2 participants
@Affie @dehann