ICE-BA icon indicating copy to clipboard operation
ICE-BA copied to clipboard

add the essential comments in project

Open qzhai opened this issue 6 years ago • 3 comments

i found the code understanding is very difficult, could you help add some essential comments?

qzhai avatar Jul 26 '18 06:07 qzhai

I am sorry that current readability is not good enough. We are working on it but it will take some time. What kind of comments do you think is most essential? I guess we will do that first.

liuhaomin avatar Aug 01 '18 08:08 liuhaomin

hi, Haomin, thanks for reply. I think the comments of basic data structure, the class member and the core algorithm which includes local and global BA is very important to understand the code and algorithm, thanks all of you a lot.

qzhai avatar Aug 08 '18 03:08 qzhai

Hi, how is this work coming along? It's really difficult to understand the code without comments especially in local and global BA. For example, the comments of variable naming rules in LocalBundleAdjustor.h will help a lot. Thanks.

J2eremy avatar Nov 29 '18 14:11 J2eremy