icub-main icon indicating copy to clipboard operation
icub-main copied to clipboard

merge together camCalib, dualCamCalib, camCalibWithPose

Open traversaro opened this issue 6 years ago • 5 comments

This issue is part of #193 .

traversaro avatar Sep 27 '19 07:09 traversaro

We should also consider to include in this refactoring also camCalibWithPose

Nicogene avatar Sep 27 '19 08:09 Nicogene

just for reference, camCalibWithPose is an ugly hack and should be re-architectured

drdanz avatar Oct 03 '19 12:10 drdanz

Beauty is in the eye of the beholder @drdanz :-)

alecive avatar Oct 03 '19 16:10 alecive

I ~~wrote it~~ took the camCalib and added stuff inside in order to avoid an extra module, therefore I'm allowed to be disgusted!

Actually I wasn't alone though but I'll take the blame!

drdanz avatar Oct 04 '19 06:10 drdanz

Well that changes things a bit :-) I was already picturing a poor phd student crying in a corner because you didn't like his code 🤣

alecive avatar Oct 04 '19 19:10 alecive

This won't happen any time in the future.

pattacini avatar Jul 30 '23 16:07 pattacini

image

alecive avatar Jul 31 '23 11:07 alecive