firemodels / fds

Fire Dynamics Simulator
https://pages.nist.gov/fds-smv/
Other
650 stars 618 forks source link

Smokeview: 3D Slice with Velocity Vectors only renders in 1 plane #12839

Closed FireResearch-BK closed 5 months ago

FireResearch-BK commented 5 months ago

When rendering a 3D slice with velocity vectors in Smokeview, vector arrows are only drawn in the YZ plane. Vectors in XZ and XY are only drawn at the mesh boundary.

grafik

Reproduction can be done with the following test case: Vel_Vectors_Test.txt

The behavior exists in FDS 6.9.1 and 6.8.0. No earlier releases were tested.

This is on Windows 10 with a NVidia GPU (latest drivers) and on Windows 11 with an Intel iGPU (latest drivers), but I do not think this is in any way related to the GPU.

gforney commented 5 months ago

I'll take a look

when you say FDS 6.9.1 and 6.8.0 did you mean smokeview 6.9.1 and 6.8.0 .

On Mon, Apr 22, 2024 at 5:49 AM FireResearch-BK @.***> wrote:

When rendering a 3D slice with velocity vectors in Smokeview, vector arrows are only drawn in the YZ plane. Vectors in XZ and XY are only drawn at the mesh boundary.

grafik.png (view on web) https://github.com/firemodels/fds/assets/114649229/0e2b9bea-3aee-4d9b-b38c-e98faf5f3612

Reproduction can be done with the following test case: Vel_Vectors_Test.txt https://github.com/firemodels/fds/files/15060673/Vel_Vectors_Test.txt

The behavior exists in FDS 6.9.1 and 6.8.0. No earlier releases were tested.

This is on Windows 10 with a NVidia GPU (latest drivers) and on Windows 11 with an Intel iGPU (latest drivers), but I do not think this is in any way related to the GPU.

— Reply to this email directly, view it on GitHub https://github.com/firemodels/fds/issues/12839, or unsubscribe https://github.com/notifications/unsubscribe-auth/AC6UCRTEM3WDKHOAMKDUX43Y6TMJ3AVCNFSM6AAAAABGSN3LOWVHI2DSMVQWIX3LMV43ASLTON2WKOZSGI2TMMBYG43DAOI . You are receiving this because you are subscribed to this thread.Message ID: @.***>

-- Glenn Forney

FireResearch-BK commented 5 months ago

when you say FDS 6.9.1 and 6.8.0 did you mean smokeview 6.9.1 and 6.8.0 .

Yes, the Smokeview version packaged with the respective FDS release.

gforney commented 5 months ago

I found the bad commit. I"m working on a fix.

gforney commented 5 months ago

try smokeview posted here and report back whether it is working for you . https://github.com/firemodels/test_bundles/releases/tag/SMOKEVIEW_TEST2

FireResearch-BK commented 5 months ago

Yep, that fixes the issue. Thank you, Glenn!