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

Final checklist for my review! #213

Closed
5 tasks
c-joly opened this issue Jul 12, 2024 · 1 comment
Closed
5 tasks

Final checklist for my review! #213

c-joly opened this issue Jul 12, 2024 · 1 comment
Assignees

Comments

@c-joly
Copy link

c-joly commented Jul 12, 2024

Hi @costashatz , these are some general comments about my review on JOSS openjournals/joss-reviews#6771

I have tested the main functionalities of Robot_DART, this is quite impressive; FYI :

  • I successfully ran a simulation with one and several robots
  • I succeeded to play with the pendulum and changed some physical parameters (added friction, changed the type of actuator…)
  • I validated the physical behavior of the pendulum in the simulation, since its behavior is easy to understand :-)
  • I made some collisions between two pendulums, the simulator gave an appropriate result
  • I could simulated in window and windowless mode, and ran several simulation in parallel
  • I succeeded to save images and videos, including embedded videos at the end of the pendulum

Here are the few changes in the repository that I expect to validate the last bullet points of the checklist of my review :

@costashatz
Copy link
Member

@c-joly Let us know if you are happy with our updates and feel free to close the issue when ready (and update your review) :)

Thank you again for the in depth comments that greatly improved our library and documentation. Highly appreciated!

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

No branches or pull requests

2 participants