svox2 icon indicating copy to clipboard operation
svox2 copied to clipboard

colmap2nsvf.py cx and cy set incorrectly (off by one)

Open amonszpart opened this issue 3 years ago • 0 comments

Hi,

Great stuff, thanks for putting the repo online so early.

I found an off by one error here: https://github.com/sxyu/svox2/blob/59984d6c4fd3d713353bafdcb011646e64647cc7/opt/scripts/colmap2nsvf.py#L320

Params is 4 long for SIMPLE_RADIAL: [fx, fy, cx, cy], so it should say:

K[0, 2] = cameras[0].params[2] / base_scale # <-- not params[1]
K[1, 2] = cameras[0].params[3] / base_scale # <-- not params[2]

amonszpart avatar Feb 10 '22 21:02 amonszpart