|
| 1 | +Minutes from 95th OpenVDB TSC meeting, June 15th, 2021, (EDT) |
| 2 | + |
| 3 | +Attendees: *Ken* M., *Jeff* L., *Dan* B., *Andre* P. |
| 4 | + |
| 5 | +Regrets: *Nick* A. |
| 6 | + |
| 7 | +Additional Attendees: JT Nelson (Blender), Richard Jones (DNeg), |
| 8 | +Johannes Meng (Intel), Sergio Rojas, Bruce Cherniak (Intel), |
| 9 | +Chris Flesher |
| 10 | + |
| 11 | +Regrets: None |
| 12 | + |
| 13 | +Agenda: |
| 14 | + |
| 15 | +1) Confirm Quorum |
| 16 | +2) Secretary |
| 17 | +3) Forum |
| 18 | +4) Open Source Day |
| 19 | +5) 8.1.0 Release |
| 20 | +6) NanoVDB |
| 21 | +7) Multiple Tickets |
| 22 | +8) TSC Membership |
| 23 | +9) Python Binding |
| 24 | +10) Fast Sweeping |
| 25 | +11) Next meeting |
| 26 | + |
| 27 | + |
| 28 | +1) Confirm Quorum |
| 29 | + |
| 30 | +Quorum is present. |
| 31 | + |
| 32 | +2) Secretary |
| 33 | + |
| 34 | +Secretary is Andre Pradhana. |
| 35 | + |
| 36 | +3) Forum |
| 37 | + |
| 38 | +Dan mentions that at a future meeting, we should discuss what we should do |
| 39 | +with the forum and the github discussion page. The latter is readily available |
| 40 | +in the OpenVDB Website github repository. |
| 41 | + |
| 42 | +There is a question about doing csgDifference where the second argument is |
| 43 | +an implicit function. Dan provides an answer to the question where he suggests |
| 44 | +to use tools::visitNodeDepthFirst() or to use the dynamic node manager. Dan |
| 45 | +also points out one example of an implicit function, which is an EnrightField, |
| 46 | +primarily used for debugging. |
| 47 | + |
| 48 | +This question is related to the support for implicit function, especially |
| 49 | +in the case of doing CSG-type operation with an implicit function. |
| 50 | +The challenge is how to introduce a scripting language that allows you |
| 51 | +to represent an analytic function/ closed form expression that can be |
| 52 | +combined with an explicit grid. Ken asks whether AX provides the proper |
| 53 | +solution for the purpose of for implicit function support in OpenVDB or |
| 54 | +if there is a light-weight solution that can be added to the core library. |
| 55 | + |
| 56 | +Jeff thinks that OpenVDB should stick with being an explicit grid library. |
| 57 | +Dan thinks that it can be valuable to have something that samples from |
| 58 | +a closed form expression. Jeff thinks that the proper way to address this |
| 59 | +is through an AX kernel. Ken argues that OpenVDB has support for meshes |
| 60 | +because you can generate an explicit SDF grid from that. Jeff says that |
| 61 | +AX is the ultimate generator for such things. Ken agrees that AX is |
| 62 | +probably the natural place to put this. He also mentions that |
| 63 | +Rhythm and Hues used to have a scripting language that supports this idea. |
| 64 | +Dan mentions that ILM has Zeno technology field that is also very useful. |
| 65 | +Ken thinks that if there is a light-weight way to add such thing to the |
| 66 | +core library, it will be very useful. |
| 67 | + |
| 68 | +Dan mentions that we cannot put a grid type in AX that is not listed in |
| 69 | +the core-library. |
| 70 | + |
| 71 | +In conclusion, we should table the discussion and revisit it at a later time. |
| 72 | + |
| 73 | +4) Open Source Day |
| 74 | + |
| 75 | +Open Source day hosted by ASWF will take place between Wednesday, August 4th |
| 76 | +to Thursday, August 5th 2021. Ken confirmed that OpenVDB will participate. |
| 77 | +The program is 1 hour, including Q+A. But by June 23rd, Ken needs to submit a |
| 78 | +form outlining the intent/program and the speakers. |
| 79 | + |
| 80 | +OpenVDB presentation at SIGGRAPH will be available at the beginning of August |
| 81 | +or the end of July. The course video will only be available for SIGGRAPH |
| 82 | +attendees and OpenVDB has signed over the rights for it over to ACM SIGGRAPH, |
| 83 | +so we may not be able to put the video in the OpenVDB website. However, there |
| 84 | +is a chance that it will be available to be accessed through the ACM Digital |
| 85 | +Library. |
| 86 | + |
| 87 | +Ken thinks that we can discuss it next Tuesday. He proposes if we take the |
| 88 | +discussion on the email thread. We should talk about the things that is not |
| 89 | +included in the course, which may include the road map and also fo ask |
| 90 | +OpenVDB users/clients for what they would like to see from the project. |
| 91 | +We should also mention about our intent to accept contribution for |
| 92 | +OpenVDB Python binding that is based on PyBind11. See Python Binding |
| 93 | +(Section 9) discussion below. |
| 94 | + |
| 95 | +JT mentions that he finds that OpenVDB is very useful in his line of work |
| 96 | +and expresses intention to keep helping with video production related |
| 97 | +to OpenVDB. |
| 98 | + |
| 99 | +5) 8.1.0 Release |
| 100 | + |
| 101 | +Ken says that he followed the guideline for releasing a new version of OpenVDB. |
| 102 | +However, he encounters a conflict during the release. We need to add a sentence |
| 103 | +on how to resolve conflicts if they arise during the release process. |
| 104 | + |
| 105 | +Dan says that the state of the release is good. We need announcement of the |
| 106 | +release. Merging the release branch into master is required to keep everything |
| 107 | +in-sync. |
| 108 | + |
| 109 | +6) NanoVDB |
| 110 | + |
| 111 | +Ken says that there is a user who branches off an old version of NanoVDB. Ken |
| 112 | +will fix the issue and will merge in some new changes. Jeff says that it has |
| 113 | +been branched off an old version, so it hasn't been kept up-to-date with head |
| 114 | + |
| 115 | +Ken says that there is fixed-bit-rate compression and variable-bit-rate |
| 116 | +compression. He is able to make variable-bit-rate compression to be branchless |
| 117 | +to make it faster. |
| 118 | + |
| 119 | +7) Multiple Tickets |
| 120 | + |
| 121 | +Rich asks for a way to keep track of tickets, ideas, and development process |
| 122 | +as Nick A. moves from DNEG. Rich and Nick will continue to work on OpenVDB |
| 123 | +AX and they would like to be able to have a process to collaborate. |
| 124 | + |
| 125 | +Ken mentions the need to triage the tickets/tasks/issues as they come in. |
| 126 | +Dan mentions that the OpenVDB JIRA will no longer be used. He proposes to use |
| 127 | +the Github Issue section. Currently, the Github Issue section includes |
| 128 | +discussion, questions, bug reports, etc. Dan thinks that we should switch on |
| 129 | +Github Discussion tab and Github Issue will contain tasks that we want to |
| 130 | +do. Dan thinks that the Forum should no longer be used and should be moved |
| 131 | +to Github Discussion. By triage, Ken means to properly tag them properly |
| 132 | +and to create corresponding Github issues for the ones we will work on. |
| 133 | + |
| 134 | +Ken asks Rich about the nature of the tickets that he wants to migrate |
| 135 | +over. Rich says they vary from old issues, simple API improvements, and |
| 136 | +bugs. Jeff thinks that Github Issue is the right place to put them. |
| 137 | + |
| 138 | +Ken asks if there is a way for us to tag a discussion with labels such as |
| 139 | +core, houdini, maya, AX, etc. Dan says it is doable. |
| 140 | + |
| 141 | +Dan also thinks that Github Issue is the right place to put them in. |
| 142 | +Dan will send an email summarizing what we talk about during the meeting to |
| 143 | +loop Nick. We will discuss this topic again the next time we meet. |
| 144 | + |
| 145 | +8) TSC Membership |
| 146 | + |
| 147 | +Nick leaves DNEG on Tuesday. We should discuss Rich's promotion to be part |
| 148 | +of TSC. Jeff mentions that the TSC position goes with the person and not |
| 149 | +with the company. Ken says that there is an unwritten consensus that |
| 150 | +one company gets one vote. |
| 151 | + |
| 152 | +9) Python Binding |
| 153 | + |
| 154 | +Somebody made a response to our discussion about Python binding for OpenVDB. |
| 155 | +Ken talked to Matthew Cong. |
| 156 | + |
| 157 | +Imath is the glue that binds everything together, so the Python binding for |
| 158 | +OpenVDB at ILM is relying heavily on IMath. IMath is currently the glue that |
| 159 | +keeps everything together, e.g. between UT_Vector and OpenVDB::math vector. |
| 160 | +Jeff mentions that if only C++ has a standard Vec3 class. |
| 161 | + |
| 162 | +Dan mentions that we made the decision that PyBind11 will be accepted to the |
| 163 | +library if somebody were to do the work. We should mention this on the |
| 164 | +Open Source day. |
| 165 | + |
| 166 | +10) Fast Sweeping |
| 167 | + |
| 168 | +Andre has a PR-1106, which adds support for extending a field in one direction, |
| 169 | +i.e. when the corresponding SDF/Fog value is above or below a given iso-value, |
| 170 | +as well as dilation. |
| 171 | + |
| 172 | +Andre says that he will look at the FastSweeping unit test in debug mode that |
| 173 | +is currently running very slow. Andre will also add support for extending |
| 174 | +SDF into a mask if the mask grid does not have the same transform as the |
| 175 | +SDF grid. Ken thinks that doing trilinear interpolation inside the FastSweeping |
| 176 | +algorithm can slow down the code, so perhaps the right way to go about it |
| 177 | +is to re-sample the mask grid at the SOP level. Jeff thinks that there are cases, |
| 178 | +e.g. when we upres a really light-weight mask field against an SDF grid that is |
| 179 | +really dense. Andre will investigate this. |
| 180 | + |
| 181 | +10) Baking Minor Version to The ABI |
| 182 | + |
| 183 | +Jeff brought up the fact that we bake the ABI minor version into the namespace. |
| 184 | +He asks if the minor version should be part of the namespace because the intent |
| 185 | +is that you should be able to mix-and-match anything that matches the major |
| 186 | +version. Patch number is not part of the namespace. |
| 187 | + |
| 188 | +Dan wonders if Peter Cucka has a reason for why he does it this way. We should |
| 189 | +think of repercussions of stripping it. |
| 190 | + |
| 191 | +11) Next meeting |
| 192 | + |
| 193 | +Next meeting is June 22nd, 2021. 12pm-1pm EST (GMT-5). |
0 commit comments