|
| 1 | +Minutes from 118th OpenVDB TSC meeting, January 25th, 2022, (EDT) |
| 2 | + |
| 3 | +Attendees: *Ken* M., *Jeff* L., *Andre* P, *Nick* A., *Dan* B., |
| 4 | +*Rich* J. |
| 5 | + |
| 6 | +Additional Attendees: Sebastian Gaida, Greg Hurst (United Therapeutics), Bruce Cherniak (Intel), |
| 7 | +JT Nelson (Blender), Sergio Rojas (Different Dimension). |
| 8 | + |
| 9 | +Regrets: |
| 10 | + |
| 11 | +Agenda: |
| 12 | + |
| 13 | +1) Confirm quorum |
| 14 | +2) Secretary |
| 15 | +3) Forum |
| 16 | +4) Level Set Propagation |
| 17 | +5) Half VDBs |
| 18 | +6) Roadmap |
| 19 | +7) Next meeting |
| 20 | + |
| 21 | + |
| 22 | +-------------------- |
| 23 | + |
| 24 | +1) Confirm quorum |
| 25 | + |
| 26 | +Quorum is present. |
| 27 | + |
| 28 | +2) Secretary |
| 29 | + |
| 30 | +Secretary is Dan Bailey. |
| 31 | + |
| 32 | +3) Forum |
| 33 | + |
| 34 | +Discussion on Google groups about deep merging point trees. Author of the post |
| 35 | +shared some interesting context about their application and provided some |
| 36 | +feedback and details about the type of datasets that were tried. Dan to |
| 37 | +investigate performance regarding his merging implementation. Nick thinks we |
| 38 | +should consider accepting his deep merge implementation. |
| 39 | + |
| 40 | +Issue 1287 - we should investigate if the proposed solution is worth |
| 41 | +implementing. |
| 42 | + |
| 43 | +PR 1282 - Ken to try this suggested change out and reply with his feedback. Greg |
| 44 | +mentioned that he ran into the same issue recently and will help review. |
| 45 | + |
| 46 | +Discussion 1286 - Question about distributing a compiled Python module. Someone |
| 47 | +should reply to this, but we don't have time or knowledge to look into doing |
| 48 | +this right now. |
| 49 | + |
| 50 | +4) Level Set Propagation |
| 51 | + |
| 52 | +Ken has a new tool to do level set propagation, will share shortly. |
| 53 | + |
| 54 | +5) Half VDBs |
| 55 | + |
| 56 | +Greg has been looking at adding half grids as a native runtime type. Various |
| 57 | +changes required, some more hacky as proof of concept, but it is working. Some |
| 58 | +discussion around what precision to do computation at, whether to promote to |
| 59 | +double for example and then lower to half for in-memory storage. Computing |
| 60 | +natively at half is slower. API and tools are not very consistent about this, |
| 61 | +but this is more forgiving with float and double than with half, so this issue |
| 62 | +will need to be resolved. |
| 63 | + |
| 64 | +Possibility of using a compression type. Jeff highlighted that something similar |
| 65 | +is done with Houdini volumes, where constant tiles can be stored with a |
| 66 | +compressed type different from the volume type. |
| 67 | + |
| 68 | +6) Roadmap |
| 69 | + |
| 70 | +Live discussion and updates to the roadmap, see this document: |
| 71 | + |
| 72 | +https://docs.google.com/document/d/1F1hmGnS1kIMobnfgpTJ6SYSrEPhC2B0WQAlt-idAo5w/edit?usp=sharing |
| 73 | + |
| 74 | + |
| 75 | +Of particular note during this discussion: |
| 76 | + |
| 77 | +6a) MultiResGrid - no major progress on this front recently, but worth |
| 78 | +discussing with Autodesk and making a final call on whether this is something |
| 79 | +we can pursue or not. 6b) PointRasterize - Nick's implementation for point |
| 80 | +rasterizing has languished on a PR for a while. Contention is around how to |
| 81 | +include multiple similar but different tools. Rasterization is a complex and |
| 82 | +multi-faceted problem. No reason not to add more than one similar tool, but |
| 83 | +general agreement that the current practice of having tools self-documented in |
| 84 | +header files would make this harder to navigate and more confusing. Nick |
| 85 | +highlighted that we should really improve our documentation and rely more on |
| 86 | +well-written Doxygen or Sphinx to help our users. Ken suggested that we could |
| 87 | +introduce new translation units which would make the header files simpler and |
| 88 | +easier to navigate, though this has implications for explicit template |
| 89 | +instantiation. Other suggestions include having a single entry header |
| 90 | +(PointRasterize.h) which pulled in all other implementations. Nick pointed at |
| 91 | +other libraries that use a detail namespace for better abstracting |
| 92 | +implementation detail away from the user. Dan to review Nick's PR with a view |
| 93 | +to merge this in shortly. |
| 94 | + |
| 95 | +7) Next meeting |
| 96 | + |
| 97 | +Next meeting is in one week, February 1st, 2022. 1pm-2pm EST (GMT-5). |
0 commit comments