CFAST Visualization possible temperature colorbar error
Closed this issue · 10 comments
This is Smokeview build 2154:
On the colorbar in F, I saw an unexpectedly low number, so as test, I set the global minimum to 20 C:
With the selected units as C, it looks fine:
But switching the units to F, it gives an unexpectedly low (and non-physical) value:
Switching to K, these values match the F values:
So, in the conversion, it appears that Smokeview is erroneously subtracting 100 C somewhere in the Fahrenheit conversion. It somehow appears to be deducting 118 C in the conversion to Kelvin. Here is a table depicting what I found:
Displayed Values:
C K F
Max 1200 1355 1980
Min 20 175.1 -144.4
What should be displayed:
Max 1200 1473 2192
Min 20 293 68
Differences:
Max 0 -118 -212
Min 0 -117.9 -212.4
I apologize that my table does not look so nice.
In between these displays, I made no changes to anything. I did not move the displayed timestep, did not reread data, or anything other than using the option to switch the displayed temperature units.
Thank you for your assistance.
Thanks for the report. Ill take a look
can you uppload the input file you were using
OK, so here's the file. And here's some additional testing information:
I reran this on my local drive and had both the slice files and zone files written out to the directory.
This piece of information, hopefully, will help you look for the problem. The temperature error only appears to occur on the zone file temperature colorbar. While looking at the slice files, the colorbar works correctly changing between units.
Here's the file with the .in change to .txt for upload compatability.
Thanks for looking at this.
try smokeview posted at URL below. this was an array indexing problem. the colorbar labels were computed correctly they were just displayed in the wrong location (one row to high)
https://github.com/firemodels/test_bundles/releases/tag/SMOKEVIEW_TEST2
That took care of the temperature issue. Thank you very much.
One question I had about something that apparently changed between the CFAST distribution version of Smokeview and this version. In the distribution version, wall vents have a sort of depth. In other words, instead of being drawn as rectangles, they were drawn as parallelepipeds. I quite liked that presentation and thought it made them easier to see.
My first question is that I'm curious why that functionality was removed?
Secondly, is that a feature request that could come back?
If you want me to close this comment and make the request separate, just let me know and I'll submit it that way.
If it's not something that can be reasonably implemented, you can just close this thread with your response.
Thanks again!
Just to clarify, I'm not talking about the old default vents that used to be put in by CEdit added into the actual model. I was just talking about how the explicit vents are visualized in Smokeview. So, it's not a matter of putting vents in or out of the model, it's just how they're drawn in smoke view. I was asking about changing the way explicit vents, defined on the Wall Vents tab, are visualized in Smokeview back to the way they were shown in the CFAST distribution version of Smokeview that gave the vents a little bit of thickness making them more visible. That's why your comment about taking vents out has confused me.
Thank you.
The way that the vents were drawn in 3D caused some users to be confused, especially when the compartments had unusual dimensions. We thought it best to simple draw a vent as a plane, which is how one specifies it.
Thanks for the explanation. Thanks for all of your combined work.