-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
refine-pixsfm work with colmap #2715
Conversation
@Guangyun-Xu Do you have side-by-side visualizations w/o pixsfm? I am curious if it produces better GS models. |
@jb-ye
|
Not sure if I'm missing something but don't you need to pass in I tried running this without that change (as-is) and it looks like pixsfm was not run |
@thomakah Thanks for your advice. I forgot to add the file |
@Guangyun-Xu Could you give a full step-by-step instruction that reproduces your results on |
@Guangyun-Xu I made that change locally before I wrote my comment and then received an error in PixSfM (installed from a recent version via +1 to @jb-ye 's request. It could be that we are working off of different versions |
OK. |
In order to use pixsfm, we need to install these packages. I refer to the Dockerfile for these installation instructions:
|
To process data by colmap and then refine by pixsfm, we should run commend like this: # download Poster dataset
ns-download-data nerfstudio --capture-name=poster
# processing data
ns-process-data images --data data/nerfstudio/poster/images/ --output-dir data/colmap_pixsfm/poster/ --sfm-tool colmap --refine-pixsfm The log like this:
|
Tain and eval:
Eval result:
|
That's really nice! @Guangyun-Xu Thanks |
Whats up with this PR? |
Is Colmap 3.9.1 compatible with this PR? |
So far, I can not build Edit: |
I got unknown error.. This is playroom dataset which is a sample from Inria GS github repo.
The error:
using :
|
@ichsan2895 Maybe CUDA out of memory?Or you can try CUDA_VISIBLE_DEVICES=0, multiple GPUs may cause some problems. Due to the Spring Festival, I am very sorry for not being able to deal with your problems in time. |
@Guangyun-Xu hello, I know the problem, we must build from source If you have newer Ubuntu (for example >= 23.04) and use So, we must built from source |
Refer to this example, use pixsfm to optimize colmap results