Skip to content

add viser viewer for training, with background image and point cloud viz#135

Merged
wilsonCernWq merged 6 commits into
nv-tlabs:mainfrom
tangkangqi:darcyt/viser_train_viewer
Jan 27, 2026
Merged

add viser viewer for training, with background image and point cloud viz#135
wilsonCernWq merged 6 commits into
nv-tlabs:mainfrom
tangkangqi:darcyt/viser_train_viewer

Conversation

@tangkangqi
Copy link
Copy Markdown
Contributor

@tangkangqi tangkangqi commented Aug 7, 2025

Add config param "with_viser_gui"
example command:
python train.py --config-name apps/colmap_3dgut.yaml path=data/mipnerf360/bicycle out_dir=runs experiment_name=bicycle_3dgut dataset.downsample_factor=2 export_usdz.enabled=true with_viser_gui=true

Point cloud update while training:
point_cloud

Background image update while training:
image

@tangkangqi tangkangqi mentioned this pull request Aug 7, 2025
@tangkangqi tangkangqi force-pushed the darcyt/viser_train_viewer branch 2 times, most recently from 5b3b933 to 9888c86 Compare August 8, 2025 00:27
@tangkangqi tangkangqi force-pushed the darcyt/viser_train_viewer branch from 9888c86 to 62910bc Compare August 8, 2025 00:29
Copy link
Copy Markdown
Collaborator

@wilsonCernWq wilsonCernWq left a comment

Choose a reason for hiding this comment

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

Hi, thanks for the nice contribution, I tested the code and I think I can see point cloud renderings correctly, but I am unable to see, for example, rendered colors. I am wondering if you could take a look and see if something merged recently caused the conflict?

Thank. you!

@tangkangqi
Copy link
Copy Markdown
Contributor Author

Just noticed this, I will check and update tomorrow.

@tangkangqi tangkangqi force-pushed the darcyt/viser_train_viewer branch from 733b413 to 62910bc Compare October 23, 2025 15:54
@tangkangqi
Copy link
Copy Markdown
Contributor Author

tangkangqi commented Oct 23, 2025

Hi @wilsonCernWq , sorry my PC reinstalled thus late...
Sorry my last screenshot forget to select "Do Training" checkbox.
I checked, we need select "Show Render ", "Do Training", "Live Update" at the same time.

image

@tangkangqi
Copy link
Copy Markdown
Contributor Author

tangkangqi commented Oct 23, 2025

And I add:

  1. resolution sidebar config,
  2. render of density distance hits normals
image image

Updated camera-to-world conversion logic by using the view matrix directly, questioning the previous inversion method.
Copy link
Copy Markdown
Collaborator

@wilsonCernWq wilsonCernWq left a comment

Choose a reason for hiding this comment

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

I added one small bug fix, I think the camera transformation is wrong? That is the reason why I saw black images. It seems the issue is obvious on synthestic data, but less obvious on colmap scenes.

@wilsonCernWq wilsonCernWq merged commit 01de6f8 into nv-tlabs:main Jan 27, 2026
janickm pushed a commit to janickm/3dgrut that referenced this pull request Mar 30, 2026
…viz (nv-tlabs#135)

* add viser viewer for training conifg

* add viser gui utils (for training viz), with point cloud and background image option

* add resolution adjust with browser size or sidebar config

* clear background when render checkbox unchecked

* add render of density distance hits normals

* Fix camera-to-world conversion logic

Updated camera-to-world conversion logic by using the view matrix directly, questioning the previous inversion method.

---------

Co-authored-by: tangkangqi <tangkangqi@yeah.net>
Co-authored-by: Qi Wu <wilson.over.cloud@gmail.com>
janickm pushed a commit to janickm/3dgrut that referenced this pull request Mar 30, 2026
add viser viewer for training, with background image and point cloud viz (nv-tlabs#135) (nv-tlabs#187)

See merge request nrs/3dgrut-internal!275
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