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

[core] Remove 'System' hierarchy. Remove single-robot 'Engine' class. #728

Merged
merged 1 commit into from
Feb 21, 2024

Conversation

duburcqa
Copy link
Owner

@duburcqa duburcqa commented Feb 20, 2024

  • [core] Remove unnecessarily confusing 'System' hierarchy.
  • [core] Remove single-robot 'Engine' class.

Fixes #556

@duburcqa duburcqa force-pushed the no_system_single_robot_engine branch 4 times, most recently from d721738 to e66c254 Compare February 20, 2024 19:49
@duburcqa duburcqa enabled auto-merge (squash) February 20, 2024 20:05
@duburcqa duburcqa force-pushed the no_system_single_robot_engine branch from e66c254 to 97cf7ce Compare February 20, 2024 20:10
@duburcqa duburcqa force-pushed the no_system_single_robot_engine branch from 97cf7ce to b72ee1e Compare February 20, 2024 23:18
@duburcqa duburcqa merged commit 6205fbe into dev Feb 21, 2024
31 of 32 checks passed
@duburcqa duburcqa deleted the no_system_single_robot_engine branch February 21, 2024 06:37
@duburcqa duburcqa mentioned this pull request Feb 21, 2024
duburcqa added a commit that referenced this pull request Feb 21, 2024
* [core] More conservative step adjustment for adaptive steppers. (#724)
* [core] Add relaxation to PGS solver to mitigate convergence instabilities. (#726)
* [core] Remove unnecessarily confusing 'System' hierarchy. (#728)
* [core] Remove single-robot 'Engine' class. (#728)
* [python/plot] Synchronise x-axes on all tabs by default. (#724)
* [gym/common] Ease debugging of Gym environments. (#724)
* [gym/common] Do not enforce 'dtMax'. (#727)
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.

[core] Get rid of confusing system hierarchy
1 participant