Skip to content

Commit 2d66be6

Browse files
authored
Merge pull request #1318 from apradhana/TSC_2022-02-22
Add TSC meeting notes from 2022-02-08 and 2022-02-22.
2 parents d4e59bc + a85b9b6 commit 2d66be6

File tree

2 files changed

+161
-0
lines changed

2 files changed

+161
-0
lines changed

tsc/meetings/2022-02-08.md

Lines changed: 76 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,76 @@
1+
Minutes from 120-th OpenVDB TSC meeting, February 8th, 2022, (EDT)
2+
3+
Attendees: *Ken* M., *Jeff* L., *Andre* P, *Nick* A., *Rich* J.
4+
5+
Additional Attendees: Sebastian Gaida, Greg Hurst (United Therapeutics), Karl
6+
Marett (UCLA), Bruce Cherniak (Intel), Kyle Wardlow
7+
8+
Regrets: *Dan* B.
9+
10+
Agenda:
11+
12+
1) Confirm quorum
13+
2) Secretary
14+
3) Forum
15+
4) Undefined Behavior Sanitizer in Coord Hash
16+
5) ASWF Open Source Forum March 10
17+
6) VDB Blend
18+
7) Next Meeting
19+
20+
--------------------
21+
22+
1) Confirm quorum
23+
24+
Quorum is present.
25+
26+
2) Secretary
27+
28+
Secretary is Andre Pradhana.
29+
30+
3) Forum
31+
32+
a) Issue 1301
33+
34+
Question about making ZLIB to be optional when using BLOSC. Nick thinks that
35+
the solution brought up in the issue is correct, i.e. we need to get rid of
36+
the if statement in the CMakeLists that says that if you use BLOSC then you
37+
also need to use ZLIB.
38+
39+
b) PR-1282
40+
41+
Ken has reviewed PR-1282 and made a few suggestions. Jeff will approve the PR.
42+
Ken will wait for the submitter to make the required changes. If, after awhile,
43+
there is no reply from the submitter, Ken will merge the PR and will submit
44+
another PR with his fix.
45+
46+
Jeff signed off this PR. Greg Hurst will test Ken's proposed fix.
47+
48+
4) Undefined Behavior Sanitizer in Coord Hash
49+
50+
Nick says that we have an undefined behavior sanitizer in VDB. There was a post
51+
specifically static analyzer warning of undefined behavior. Nick has been trying
52+
to fix these, primarily signed overflow. He is now asking about this issue in
53+
Coord's hash. There is a method in the Coord class that returns hash, but it may
54+
overflow. Jeff says that we should reinterpret_cast.
55+
56+
Ken proposes to fix the sanitizer and Nick will review it.
57+
58+
5) ASWF Open Source Forum March 10
59+
60+
This means Ken has to present something related to VDB
61+
62+
6) VDB Blend
63+
64+
Andre talks about VDB Blend, which is an implementation of SIGGRAPH talk by
65+
Sony-Imageworks: Levelsets in Production: Spider-Man 3. The disadvantage of this
66+
method is the need to have a pretty wide narrow-band level sets for it to work well.
67+
Jeff and Ken likes the idea of doing dilation on the intersection region between
68+
the two level sets. One may use extrapolate technique to define a meaningful
69+
signed-distance value in this dilation region. Once this is done, we can apply
70+
the blending method in this dilated region. Ken also mentions that he has published
71+
a paper on this in 2002 called Level Set Surface Editing Operators. Jeff thinks that
72+
we should have a separate SOP for VDB Blend.
73+
74+
7) Next Meeting
75+
76+
Next meeting is in one week, February 18th, 2022. 1pm-2pm EST (GMT-5).

tsc/meetings/2022-02-22.md

Lines changed: 85 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,85 @@
1+
Minutes from 122-nd OpenVDB TSC meeting, February 22nd, 2022, (EDT)
2+
3+
Attendees: *Ken* M., *Jeff* L., *Nick* A, *Dan* B., *Rich* J., *Andre* P.
4+
5+
Additional Attendees: Sebastian Gaida, Xavier Martin Ramirez (ILM),
6+
Greg Hurst (United Therapeutics), Kyle Wardlow (United Therapeutics),
7+
Bruce Cherniak (Intel)
8+
.
9+
Regrets: None
10+
11+
Agenda:
12+
13+
1) Confirm quorum
14+
2) Secretary
15+
3) Forum
16+
4) Rasterizing VDB from Particles
17+
5) Next Meeting
18+
19+
--------------------
20+
21+
1) Confirm quorum
22+
23+
Quorum is present.
24+
25+
2) Secretary
26+
27+
Secretary is Andre Pradhana.
28+
29+
3) Forum
30+
31+
a) Google Forum v.s. Github Discussion
32+
33+
Ken will post on the Google Forum to encourage people to embrace Github
34+
Discussion. Dan will post on the Github Discussion to say that you should also
35+
see the Google Forum.
36+
37+
b) Blosc
38+
39+
There are two issues related to Blosc. Nick will take a look at it.
40+
41+
c) VDB Print and VDB memusage
42+
43+
Someone uses VDB Print and sees a discrepancy between the expected VDB memory
44+
value with the one printed. This shows that delayed loading is working. Ken thinks
45+
that we should just provide "(out-of-core)" printing. All you need to know is the
46+
number of leaf nodes and the type of data to know what is the in-core memory usage.
47+
Providing these numbers without loading the VDB is useful.
48+
49+
d) NanoVDB (Github Discussion 1307)
50+
51+
This question is from Sebastian Gaida.
52+
53+
4) Rasterizing VDB from Particles
54+
55+
Dan and Xavier Martin Ramirez did a presentation on rasterizing VDB from
56+
particles. Several points:
57+
- This is different from Nick's work on particle rasterizer. This seems to
58+
have more applications in rendering, e.g. in the context of volumetric
59+
motion blur. Nick's work is in the context of particle-skinning and creating
60+
a level set from points.
61+
- This tool is to rasterize attributes to VDBs, e.g. in the context of
62+
rasterizing camera motion blur particles. It can rasterize density, mask,
63+
integer, float, and vector. It does clipping based on frustum.
64+
- Unique to this tool is rasterization to frustum grid.
65+
- Xavier shows that their approach to associate a voxel with a pixel on
66+
the screen will keep rendering the VDB to more efficient than rendering
67+
points. Memory usage is lower because we have fewer voxels than particle
68+
numbers. The output from volume and particle renderings look identical.
69+
The timing to finish a volume render using this approach is also faster
70+
compared to particle based rendering.
71+
- Nick is interested in a graph that demonstrates performance over time
72+
in relation to underlying resolution.
73+
- Dan talks about the different modes it supports: accumulators, max, and
74+
average.
75+
- It supports the use of mask.
76+
- This is based on scatter method. Nick's implementation is based on gather
77+
because he targets level sets applications, SIMD optimization, and
78+
prioritizing thread synchronization.
79+
- It targets different use cases as Nick's point rasterization, so both
80+
methods should coexist in OpenVDB.
81+
- Nick will talk about his point rasterization PR the next time.
82+
83+
5) Next Meeting
84+
85+
Next meeting is in one week, March 1st, 2022. 1pm-2pm EST (GMT-5).

0 commit comments

Comments
 (0)