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.

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

Refactor dynamics #332

Merged
merged 63 commits into from
Feb 25, 2023
Merged

Conversation

200km
Copy link
Member

@200km 200km commented Feb 23, 2023

Overview

Refactor dynamics

Issue

#225

Details

Refactor dynamics

Validation results

See CI

Scope of influence

NA

Supplement

NA

Note

Merge after #331

@200km 200km added this to the Major update for v6.0.0 milestone Feb 23, 2023
@200km 200km self-assigned this Feb 23, 2023
@200km 200km changed the base branch from feature/major-update-v6.0.0 to feature/refactor-environment February 23, 2023 21:10
Copy link
Member

@suzuki-toshihir0 suzuki-toshihir0 left a comment

Choose a reason for hiding this comment

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

いくつかコメントしています!

src/dynamics/attitude/controlled_attitude.cpp Outdated Show resolved Hide resolved
src/dynamics/orbit/initialize_orbit.cpp Outdated Show resolved Hide resolved
Base automatically changed from feature/refactor-environment to feature/major-update-v6.0.0 February 25, 2023 09:26
Copy link
Member

@suzuki-toshihir0 suzuki-toshihir0 left a comment

Choose a reason for hiding this comment

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

修正確認しました.approveします.

@200km 200km merged commit 0254637 into feature/major-update-v6.0.0 Feb 25, 2023
@200km 200km deleted the feature/refactor-dynamics branch February 25, 2023 13:46
@200km 200km mentioned this pull request Feb 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dynamics dynamics calculation major update incompatible API changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants