Paul Wais

Results 61 comments of Paul Wais

you're right, the cited code in nerfstudio is not equivalent to the cited multinerf code.. at least not the precise cited lines. The cited conversion appears to do a handedness...

but in opengl camera, -z is forward / look at. yet cast rays have positive lengths. So isn’t that transform a handedness change, then a flip of z? if you...

as i originally instigated this issue, i vote that https://github.com/nerfstudio-project/nerfstudio/pull/2793 effectively closes this one. @jkulhanek any thoughts?

Actually I think @jb-ye https://github.com/nerfstudio-project/nerfstudio/pull/2793 might have a bug? when `assume_colmap_world_coordinate_convention` is off, the camera poses read from colmap will change: https://github.com/jb-ye/nerfstudio/blob/8b4179fad405f1203fcfc973cad69ee051d40132/nerfstudio/data/dataparsers/colmap_dataparser.py#L163 BUT the world cloud will not change as...

@jb-ye but for methods like gaussian splatting, the world frame of the 3d points needs to be the same world frame referenced by the camera transforms, no? fwiw somebody was...

> Those options are reserved for the case when you want to keep the orientation of colmap world when exporting ply file Maybe the flag and functionality should just move...

@jb-ye The current issue with frames etc (and especially the really poor naming e.g. "transformation_matrix" that hides the actual frame name) has lead to a lot of confusion on Discord...

+1 There is also a `ZeroDivisionError` in `download_dataset.py` (line 138) if the download fails like this.

@davnov134 Happy Summer Holiday! The bugs here are: 1) The website generating the 51-line text links file appears to be broken. The urls are all expired. Perhaps it's generating a...

hmm looks like I had some connection issues. the download script still doesn't work for me tho. the selenium script does help a lot!!