SCIRun icon indicating copy to clipboard operation
SCIRun copied to clipboard

Show Uncertainty Glyphs

Open tarkpate opened this issue 3 years ago • 4 comments

-Added module to visualize tensor field uncertainty using glyphs -Added uncertainty glyph builder class to tensor glyph builder file -Added test network -Moved some files from Core::Algorithm that should be in Graphics::Datatypes

Note: The glyph surface is not mathematically correct, so I put an experimental label in the module

tarkpate avatar Jan 28 '22 22:01 tarkpate

/Users/runner/work/SCIRun/SCIRun/src/Interface/Modules/Visualization/ShowUncertaintyGlyphsDialog.cc:37:77: error: unused parameter 'name' [-Werror,-Wunused-parameter]
ShowUncertaintyGlyphsDialog::ShowUncertaintyGlyphsDialog(const std::string& name,

dcwhite avatar Mar 18 '22 14:03 dcwhite

I'm going to add comments to explain the uncertainty glyph's variable naming soon. I wanted to get the builds going first.

tarkpate avatar May 09 '22 17:05 tarkpate

It seems that the major "un-named" variables are qn and P. qn seems to be a discretization of q or Q from the paper. Does that match what see @RubioJr9? if so, maybe it would be better as q_i to be more consistent with the paper? p is a point/angle on the glyph, right? I think it's fine to leave it as p in that case.

I didn't see others that weren't part of loops.

I don't think it is a discretization. I think the qn variable is a measure of finite differences in each of the 6 degrees of freedom of the mean tensor.

tarkpate avatar Jun 14 '22 16:06 tarkpate

@jessdtate Just waiting on your approval on this one too

dcwhite avatar Jul 27 '22 16:07 dcwhite