Skip to content

Conversation

@Grufoony
Copy link
Collaborator

@Grufoony Grufoony commented Dec 11, 2024

  • Split the Dynamics header into: Dynamics + RoadDynamics + FirstOrderDynamics
  • Make seed an attribute of the constructor

Copy link

@github-advanced-security github-advanced-security bot left a comment

Choose a reason for hiding this comment

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

Cppcheck (reported by Codacy) found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

@Grufoony Grufoony marked this pull request as draft December 11, 2024 13:11
@Grufoony Grufoony marked this pull request as ready for review December 12, 2024 08:55
@Grufoony Grufoony requested a review from sbaldu December 12, 2024 08:55
@sbaldu sbaldu merged commit 009d8af into main Dec 12, 2024
25 checks passed
@sbaldu sbaldu deleted the split_Dynamics branch December 12, 2024 09:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants