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

fix & speedup show_gui #112

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix & speedup show_gui #112

wants to merge 1 commit into from

Conversation

Linyou
Copy link

@Linyou Linyou commented Dec 26, 2022

This pull request makes the following changes to the project:

  1. Replaces dearpygui with taichi GUI, which is based on imgui and allows for direct transfer of tensors to the GPU frame buffer without requiring a trip back to the CPU.
  2. Fixes a rotation bug where the rotation would not stop when the mouse movement stopped.
  3. Adds visualization of dataset poses.

There are also the following interface changes:

  1. w and s can be used to move forward and backward instead of using the mouse scroll.
  2. q and e can be used to move up and down, and a and d can be used to move left and right.
  3. Use right-click instead of left-click to control rotation.

@kwea123
Copy link
Owner

kwea123 commented Dec 26, 2022

my 4090 cannot build taichi at the moment... will try once it gets supported

@kwea123
Copy link
Owner

kwea123 commented Dec 26, 2022

how much speed up do you get by switching dearpygui to taichi?

@Linyou
Copy link
Author

Linyou commented Dec 26, 2022

how much speed up do you get by switching dearpygui to taichi?

I get a 2-3 fps increase in the Lego scenes after switching to taichi, but the movement appears to be smoother. Also, taichi GUI calculates FPS in C++, while dearpygui does not provide a reliable way to evaluate FPS, so the FPS measured in Python for dearpygui may not be accurate.

@Michelvl92
Copy link

Any update on this?

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