This is a suggestion on some simplifications to the code.
I don't see much reason to keep PPTree as its own thing and utils as a separate .so as it was tailored for ObstacleDetector only. Also, even thought the idea behind the tests/ folder is good, it has a very simplistic implementation, its never used and was unable to validate PPTree when we were having trouble with it, so I think it is ok to let it go.
All this cleanup and simplification is intended to better organize our source and simplify the docs. I found it confusing to justify the current structure when I was writing the documentation.
mbelluzzo
commented
7 years ago
This is a suggestion on some simplifications to the code.
I don't see much reason to keep PPTree as its own thing and utils as a separate .so as it was tailored for ObstacleDetector only. Also, even thought the idea behind the tests/ folder is good, it has a very simplistic implementation, its never used and was unable to validate PPTree when we were having trouble with it, so I think it is ok to let it go.
All this cleanup and simplification is intended to better organize our source and simplify the docs. I found it confusing to justify the current structure when I was writing the documentation.