Skip to content

Update gui.py#36

Open
stymbhrdwj wants to merge 1 commit intoNJU-3DV:mainfrom
stymbhrdwj:stymbhrdwj-visibility_tracing_DTU_fix
Open

Update gui.py#36
stymbhrdwj wants to merge 1 commit intoNJU-3DV:mainfrom
stymbhrdwj:stymbhrdwj-visibility_tracing_DTU_fix

Conversation

@stymbhrdwj
Copy link

I was encountering a bug when running the GUI with type "neilf" on the DTU dataset. The issue was pc._visibility_tracing was None, which was being indexed in neilf.py in render_view function. I added a call to the update_visibility method in gui.py to fix this.

Fixed pc._visibility_tracing = None for DTU dataset when running GUI using neilf
@caobiru
Copy link

caobiru commented Feb 22, 2025

@stymbhrdwj Thank you so much! This helped a lot!

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.

2 participants