Skip to content
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

Only import pycolmap in run_hloc #3146

Merged
merged 5 commits into from
May 15, 2024
Merged

Conversation

KevinXu02
Copy link
Contributor

@KevinXu02 KevinXu02 commented May 15, 2024

ns process data will immediately crash when trying to import pycolmap in windows. this pr stops trying to import pycolmap when initializing. as pycolmap is only used in hloc, this shouldn't be a problem.

*** Aborted at 1715751746 (unix time) try "date -d @1715751746" if you are using GNU date ***
@ 0x7ffe98de4132 log2f
@ 0x7ff7ce112698 OPENSSL_Applink
@ 0x7ffe4a74e390 __C_specific_handler
@ 0x7ffe9b6f440f __chkstk
@ 0x7ffe9b66e466 RtlFindCharInUnicodeString
@ 0x7ffe9b6f33fe KiUserExceptionDispatcher
@ 0x7ffe3c3225ee void __cdecl __ExceptionPtrRethrow(void const * __ptr64)
@ 0x7ffcea0ee80e public: void __cdecl c10::ivalue::Future::markCompleted(void) __ptr64
@ 0x7ffcea101ae7 public: class pybind11::object const & __ptr64 __cdecl pybind11::error_already_set::trace(void)const __ptr64
@ 0x7ffce97cdec5 (unknown)
@ 0x7ffce9d26b29 PyInit_pycolmap
@ 0x7ffe4a811080 (unknown)
@ 0x7ffe4a8126a5 __NLG_Return2
@ 0x7ffe9b6f3c56 RtlCaptureContext2
@ 0x7ffce97d0936 (unknown)
@ 0x7ffe258d6f60 _PyMethodDef_RawFastCallKeywords
@ 0x7ffe258d5f26 _PyObject_MakeTpCall
@ 0x7ffe258e266b PyWrapper_New
@ 0x7ffe2591e34a _PyObject_GenericGetAttrWithDict
@ 0x7ffe2591d939 PyObject_GetAttrString
@ 0x7ffce9805dfe PyInit_pycolmap
@ 0x7ffce985d98d PyInit_pycolmap
@ 0x7ffce983628d PyInit_pycolmap
@ 0x7ffce97f3ed0 (unknown)
@ 0x7ffce98007a8 PyInit_pycolmap
@ 0x7ffce98002ae PyInit_pycolmap
@ 0x7ffe259e21cd PyImport_AppendInittab
@ 0x7ffe259e1896 PyImport_Import
@ 0x7ffe259194b5 _PyCFunction_DebugMallocStats
@ 0x7ffe258d606e PyVectorcall_Call
@ 0x7ffe259b3a20 PyEval_GetFuncDesc
@ 0x7ffe259b0571 _PyEval_EvalFrameDefault

@KevinXu02 KevinXu02 mentioned this pull request May 15, 2024
@KevinXu02 KevinXu02 marked this pull request as draft May 15, 2024 08:39
@KevinXu02 KevinXu02 changed the title Remove pycolmap import on windows Only import pycolmap in run_hloc May 15, 2024
@KevinXu02 KevinXu02 marked this pull request as ready for review May 15, 2024 08:47
Copy link
Collaborator

@jb-ye jb-ye left a comment

Choose a reason for hiding this comment

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

Thx

@jb-ye jb-ye merged commit e6d780d into nerfstudio-project:main May 15, 2024
2 checks passed
@KevinXu02 KevinXu02 deleted the colmap branch June 10, 2024 15:29
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