|
| 1 | +Minutes from 128-th OpenVDB TSC meeting, April 5th, 2022, (EDT) |
| 2 | + |
| 3 | +Attendees: *Ken* M., *Jeff* L., *Nick* A., *Rich* J., *Andre* P. |
| 4 | + |
| 5 | +Additional Attendees: Kyle Wardlow, Karl Marrett (UCLA), Bruce Cherniak (Intel) |
| 6 | +Sebastian Gaida, Greg Hurst (United Therapeutics), J. T. Nelson (Blender) |
| 7 | + |
| 8 | +Regrets: *Dan* B. |
| 9 | + |
| 10 | +Agenda: |
| 11 | + |
| 12 | +1) Confirm quorum |
| 13 | +2) Secretary |
| 14 | +3) Forum |
| 15 | +4) Half Implementation |
| 16 | +5) PR-1347: Meeting Notes |
| 17 | +6) PR-1343: Speed up Windows CI |
| 18 | +7) PR-1342: MinGW fix |
| 19 | +8) PR-1339: Meeting notes and consolidate pending-changes |
| 20 | +9) PR-1336: Exposed TypeLists of the default set of OpenVDB types |
| 21 | +10) PR-1333: Small CMake change |
| 22 | +11) PR-1008: Rasterization |
| 23 | +12) PR-1317: openvdb_tool |
| 24 | +13) OpenVDBLink: Link between Mathematica and OpenVDB |
| 25 | +14) USD |
| 26 | +15) Next Meeting |
| 27 | + |
| 28 | +-------------------- |
| 29 | + |
| 30 | +1) Confirm quorum |
| 31 | + |
| 32 | +Quorum is present. |
| 33 | + |
| 34 | +2) Secretary |
| 35 | + |
| 36 | +Secretary is Andre Pradhana. |
| 37 | + |
| 38 | +3) Forum |
| 39 | + |
| 40 | +a) Windows static build |
| 41 | + |
| 42 | +Several Windows users have asked for a better CMake support, see e.g. Issue |
| 43 | +[1160](https://github.com/AcademySoftwareFoundation/openvdb/issues/1160). |
| 44 | + |
| 45 | +We need to improve the CMake support especially for a better use of OpenVDB |
| 46 | +downstream. The CMake workflow should allow a user to easily bring VDB |
| 47 | +library, its dependencies, and compiled-flags; which all depend on the |
| 48 | +platform. This is primarily a Windows problem because Windows requires more |
| 49 | +specific branching in CMake. We need a custom find-modules in our CMake and |
| 50 | +rewrite the findPackage logic, e.g. findOpenVDB.cmake should not exist. This |
| 51 | +can be replaced by an OpenVDBconfig.cmake file (config module) instead. |
| 52 | + |
| 53 | +Ken asks if distributing binaries actually solves the problem. Nick thinks that |
| 54 | +it can alleviate the problem, but not solve it. |
| 55 | + |
| 56 | +Ken will bring this issue to the TAC meeting to ask for from other ASWF |
| 57 | +projects on April 20th 2022. |
| 58 | + |
| 59 | +b) Post in the forum about building VDB 9.0 with embedded half. |
| 60 | + |
| 61 | +Jeff said that the user included both openvdb.lib and libopenvdb.lib, which |
| 62 | +causes the problem. |
| 63 | + |
| 64 | +4) Half Implementation |
| 65 | + |
| 66 | +Ken is going to ask Autodesk about their requirements for a half-grid |
| 67 | +implementation. Greg Hurst plans to spear-head the effort once he is done |
| 68 | +with Mathematica-OpenVDBLink PR. |
| 69 | + |
| 70 | +5) PR-1347: Meeting Notes |
| 71 | + |
| 72 | +Somebody will take a look. |
| 73 | + |
| 74 | +6) PR-1343: Speed up Windows CI |
| 75 | + |
| 76 | +Adding Caching support for vcpckg installs, which should speed up the Windows |
| 77 | +CI. Nick and Jeff are unsure about store password logic that the PR includes. |
| 78 | +It would be nice if Dan takes a look. |
| 79 | + |
| 80 | +7) PR-1342: MinGW fix |
| 81 | + |
| 82 | +Nick is happy with this PR. Jeff approves. |
| 83 | + |
| 84 | +8) PR-1339: Meeting notes and consolidate pending-changes |
| 85 | + |
| 86 | +Merged. |
| 87 | + |
| 88 | +9) PR-1336: Exposed TypeLists of the default set of OpenVDB types |
| 89 | + |
| 90 | +This allows a user to do static-dispatch using a pre-approved list of grids |
| 91 | +to be supported by an algorithm at compile time. At runtime, however, it still |
| 92 | +does a string comparison. We could add an enum to replace the string |
| 93 | +comparison logic. |
| 94 | + |
| 95 | +We are happy to remove the Native keyword from the type-lists. |
| 96 | + |
| 97 | +10) PR-1333: Small CMake change |
| 98 | + |
| 99 | +Modernize CMake: change find_package(PythonLibs) to find_package(Python). |
| 100 | + |
| 101 | +11) PR-1008: Rasterization |
| 102 | + |
| 103 | +Rasterization PR from Nick. We have talked about approving both rasterization |
| 104 | +PRs from Nick and Dan. Andre approves the PR. |
| 105 | + |
| 106 | +12) PR-1317: openvdb_tool |
| 107 | + |
| 108 | +Ken is pushing changes, including CMake. The PR is ready for some feedback. |
| 109 | + |
| 110 | +13) OpenVDBLink: Link between Mathematica and OpenVDB |
| 111 | + |
| 112 | +Greg will create a new branch containing OpenVDBLink, which provides a link |
| 113 | +between OpenVDB and Wolfram Mathematica, and will create a PR based on it. |
| 114 | + |
| 115 | +Greg ask about an advection example. Andre suggests to use Enright field to |
| 116 | +advect a density field. Nick suggests to advect points with a curl noise. |
| 117 | +Ken will help Greg. |
| 118 | + |
| 119 | +14) USD |
| 120 | + |
| 121 | +Bruce Cherniak raises awareness on an USD PR to [support OpenVDB without |
| 122 | +depending on OpenEXR](https://github.com/PixarAnimationStudios/USD/pull/1728). |
| 123 | + |
| 124 | +The PR is from NVIDIA Gameworks. Nick and Ken will take a look. |
| 125 | + |
| 126 | +15) Next Meeting |
| 127 | + |
| 128 | +Next meeting is in one week, April 12th, 2022. 1pm-2pm EST (GMT-5). |
0 commit comments