You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Comment by Dmitriy Morozov Saturday Nov 22, 2014 at 22:03 GMT
I think here the tricky part is how generic you want to be. Currently, tess assumes that the point coordinates are floats (this is hard-coded all over the place). At the one end of the spectrum, we could simply templatize the input point type, but assume that the coordinates are float. On the other end, we could templatize the internal representation and allow for any coordinate type the user desires to use.
Comment by Tom Peterka Saturday Nov 22, 2014 at 22:29 GMT
The other issue is whether we want to carry metadata around with the particles. This was one of the things that Berk (Kitware) wanted for VTK. I should talk to him again about this; somehow the VTK data model lets you associate data (mass, size, etc.) in some format with the particle. The issue also came up earlier for Carolyn too, where she needed to associate a molecular species (A,B, or C) with each particle. She ended up storing those in a separate file and looking them up herself. The real question is whether we need that metadata in the exchanged points so that tets and their vertices also have their metadata all in the same place. We don't need to write it into a file, VTK will provide the file output. But we may need to carry it around in memory and hand it all off to VTK together.
Comment by Dmitriy Morozov Saturday Nov 22, 2014 at 23:20 GMT
Ah, interesting. I misunderstood the issue, so it's good to have clarification. I think the normal way to do it would be to templatize the point type within tess by some <class T>, which would be stored inside. There is a subtlety about making it efficient if the type is empty (which should be the default) — see empty base optimization — but it's not difficult.
Issue by Tom Peterka
Monday Nov 17, 2014 at 02:53 GMT
The text was updated successfully, but these errors were encountered: