sshaoshuai / PointRCNN

PointRCNN: 3D Object Proposal Generation and Detection from Point Cloud, CVPR 2019.
MIT License
1.7k stars 425 forks source link

Problem in third step when doing build and install file #221

Closed gehadza closed 1 year ago

gehadza commented 1 year ago

when I run python setup.py install in \roipool3d folder give me this error image Please help me

gehadza commented 1 year ago

I smell big trouble. Programmers should not use long anywhere in code that is intended to be portable across platforms. On 64-bit Linux platforms, long resolves to long long (a 64-bit type), while on 64-bit Windows platforms, long resolves to int (a 32-bit type). CUDA maintains type size compatibility between host and device code, so this difference also applies to device code on those two platforms.

QEBAPEAJXZ is presumably just C++ name decoration resulting from data_ptr(void)const. As C++ name mangling is toolchain specific, I don’t think you would want to chase that down.

I suspect the real reason this doesn’t resolve during linking is that the long here is matched up with either int or long long elsewhere, which makes the code link fine on one platform but not another. Can you make the code base “long clean”, i.e. replace all occurrences of long with more appropriate types? Guessing at the intentions, when the programmers used long they likely meant “signed integer type wider than int”, so as an initial attempt one could try replacing all instances of long with long long. You would still need to review whether those changes are in fact appropriate.

If the object being referred to is an actual pointer, a portable way of expressing that as an integer is to use uintptr_t, a type that is guaranteed by C and C++ language standards to be able to hold a pointer of any kind, e.g. for the purpose of performing bit manipulations on pointers.