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

Improve camera performance #43

Open
SijmenHuizenga opened this issue May 15, 2020 · 4 comments
Open

Improve camera performance #43

SijmenHuizenga opened this issue May 15, 2020 · 4 comments
Labels
enhancement New feature or request

Comments

@SijmenHuizenga
Copy link
Member

SijmenHuizenga commented May 15, 2020

At this moment the cameras are laggy. With 3 camera framerate can drop to 3 fps and with 1 camera it drops to 20 fps. We want: steady 30 fps for the onboard camera and the livestream angle.

This is the epic adventure to make the simulator performant.

@SijmenHuizenga SijmenHuizenga linked a pull request May 24, 2020 that will close this issue
@SijmenHuizenga SijmenHuizenga changed the title Performance! Improve camera performance Jun 5, 2020
@SijmenHuizenga
Copy link
Member Author

SijmenHuizenga commented Jun 10, 2020

We found that splitting the camera sensor code from the ros_bridge into it's own node greatly improves the performance. After #95 the performance is 30 fps (2x 785x785, local computer, NoDisplay, flying map` without lidar. We are not there yet but getting closer.

@SijmenHuizenga
Copy link
Member Author

Tested camera's in a competition setup: 2 google cloud instances as described in the integration guide. One dedicated to running the simulation, the other one receiving the sensordata and sending setpoints. Using this settings.json. We got around 20 fps on both camera's

@SijmenHuizenga SijmenHuizenga added this to the Competition ready milestone Jun 25, 2020
@SijmenHuizenga SijmenHuizenga added enhancement New feature or request low-prio This task has a low priority. and removed Unreal Engine 4 labels Jul 11, 2020
@SijmenHuizenga
Copy link
Member Author

@SijmenHuizenga
Copy link
Member Author

Sorry, but this turned out to be too complicated to be finished before FSOnline competition.

@SijmenHuizenga SijmenHuizenga removed the low-prio This task has a low priority. label Aug 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants