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

Improve development experience #82

Merged
merged 9 commits into from
Mar 1, 2023
Merged

Improve development experience #82

merged 9 commits into from
Mar 1, 2023

Conversation

GustavoSilvera
Copy link
Collaborator

@GustavoSilvera GustavoSilvera commented Feb 19, 2023 •
edited
Loading

  • Add developer-focused usage documentation, USAGE.md is geared more towards end-users
  • Replace DReyeVRData.inl (inline header file) with DReyeVRData.cpp (source file) with new linker fix
  • Fix old link in xerces download
  • Add github workflows for DReyeVR installation (only builds LibCarla/PythonAPI and runs unit tests)
  • Fix critical bugs from recorder/replayer with interpolation disabled, as well as mirrors causing crash
@GustavoSilvera GustavoSilvera added documentation Improvements or additions to documentation enhancement New feature or request labels Feb 19, 2023
@GustavoSilvera GustavoSilvera self-assigned this Feb 19, 2023
@GustavoSilvera GustavoSilvera added the bug Something isn't working label Feb 25, 2023
ajdroid
ajdroid reviewed Feb 27, 2023
View reviewed changes
Copy link
Contributor

@ajdroid ajdroid left a comment

Choose a reason for hiding this comment

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

Wonder if the diagram of UE4/Carla /DReyeVR relationships belongs in the dev docs now.
Also dev docs under getting started, switch back to the carla dir before doing git status and tracking DReyeVR changes
ajdroid
ajdroid approved these changes Mar 1, 2023
View reviewed changes
Copy link
Contributor

@ajdroid ajdroid left a comment

Choose a reason for hiding this comment

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

Okay looks good
@ajdroid ajdroid merged commit 7ca1792 into main Mar 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request
2 participants
@GustavoSilvera @ajdroid