PabloUniform class design about bitpit HOT 3 CLOSED

Hi pkestene,
Thanks for your contribute, I'll try to explain what we meant exposing only some of the ParaTree methods in PabloUniform:
ParaTree is the logical manager of the linear octree, meaning that he lives in a logical space where vertex coordinates are integers. PabloUniform should derive from ParaTree in order to introduce an affine transformation from the logical space to the physical one. In this context, we decided to avoid the user to call methods returning logical coordinates of the nodes. For the sake of truth, we did something that is not so good from a design point of view: in order to ease the user experience, we introduced in ParaTree a transformation from the logical space to the unit square. By this way the user, deriving from ParaTree to introduce his own transformation, could do it from the unit square to the physical domain, knowing nothing about logical coordinates.
The method you cited, the hidden getNodes, return a list of logical coordinates of all the nodes, that's why it should not be available in PabloUniform.
If you feel that some of the other ParaTree methods hidden to PabloUniform should be available in PabloUniform, please let us know.
Thanks!

from bitpit.

Ok that makes sense.
Thank you.

from bitpit.

This issue has not seen any recent activity. I'm closing the issue, but feel free to re-open a closed issue if needed.

from bitpit.