open-forest-observatory / geograypher

Multiview Semantic Reasoning with Geospatial Data
BSD 3-Clause "New" or "Revised" License
10 stars 4 forks source link

Remove `derived_meshes.py` #28

Closed russelldj closed 5 months ago

russelldj commented 7 months ago

The original idea of having complex mesh texturing operations take place when initializing a mesh was a bad one. This is because it placed specialized and expensive computation in a place where it didn't need to be. Instead, the current approach is to execute three potential steps in order.

It may be helpful to preserve some of the functionality in the derived meshes as utility functions or dev examples. But they should not be so tightly coupled to the functionality of the mesh class.