Comments (11)
gforney
commented on September 4, 2024
I was missing a '\n' character in a print statement. thanks for the
catch. I'll post again when I'm able to post an updated smokeview. I'm
working on a couple of other bugs at the moment
…On Fri, Sep 9, 2022 at 12:47 PM obscureed ***@***.***> wrote:
This is a slightly tricksy bug to replicate, but I think the following
steps demonstrate it:
(1) Run any project in FDS6.7.9. (I've added a small example,
tunnel0003.fds, renamed as tunnel0003.txt, in case it matters.)
(2) Open the results in the corresponding SMV6.7.21. Save the settings.
Copy the resulting .ini file to step2.ini. Close smokeview.
(3) Open the results in a new session of smokeview. Save the settings.
Copy the resulting .ini file to step3.ini.
(4) Display some results -- for example, 3D visualisation of HRRPUV. Pause
the results display by pressing "t". Save the settings. Copy the resulting
.ini file to step4.ini.
Now compare the .ini files. There are differences between step2.ini and
step3.ini, but they seem to be benign. The (presumed) bug is that step2.ini
and step3.ini contain this text:
SMOKEFIREPROP
0 1SMOKEPROP
8700.000000
which I suspect is a garbled version of the equivalent in step4.ini:
SMOKEPROP
8700.000000
If SMOKEFIREPROP is a parameter name, then the bug is also that step4.ini
does not contain it.
I'm seeing this behaviour on the Jun 28 2022 build, Revision
SMV6.7.21-0-gccbc561da-release, LINUX64 platform.
tunnel0003.txt
<https://github.com/firemodels/smv/files/9537091/tunnel0003.txt>
—
Reply to this email directly, view it on GitHub
<#1430>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AC6UCRXTNGCI2E23NP4LKOLV5NSYZANCNFSM6AAAAAAQI23AZ4>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
--
Glenn Forney
from smv.
obscureed
commented on September 4, 2024
I would have preferred more descriptive names than SMOKEPROP and SMOKEFIREPROP -- SMOKE3DEXTINCTION, for example.
from smv.
gforney
commented on September 4, 2024
wanted to leave open the option of adding other smoke or fire properties if needed to these keywords.
from smv.
obscureed
commented on September 4, 2024
From my perspective as a user, descriptive names are helpful. If a new smoke property is added, it is helpful if it is given a new descriptive name, rather than lumped with unrelated numbers into one catchall name. I sometimes have to hunt through the .ini file for a relevant-sounding parameter name, and then experiment with it to see if it does what I want. The documentation does not describe all the relevant parameters -- it does not currently mention SMOKEPROP or SMOKEFIREPROP, for example.
from smv.
gforney
commented on September 4, 2024
should be fixed in smokeview posted here
https://drive.google.com/drive/folders/0B_wB1pJL2bFQc1F4cjJWY2duWTA?resourcekey=0-J_yHat7iJoBp1fk6QS8gMA&usp=sharing
from smv.
gforney
commented on September 4, 2024
did the smokeview I posted fix your issue?
from smv.
obscureed
commented on September 4, 2024
Sorry, it will be a few days before I have time to test this.
from smv.
obscureed
commented on September 4, 2024
I've tried that test version of smokeview on the test case I posted. The three .ini files are all different from each other in various ways, which I find slightly odd. In particular, step4.ini is missing the settings of SMOKEFIREPROP that are present in the earlier step2.ini and step3.ini. Can it be correct that displaying some results destroys a setting? Even so, the .ini files are not garbled, so I'll mark this as closed. Thanks for your efforts!
from smv.
obscureed
commented on September 4, 2024
Reopening because I think there are a couple of similar missing '\n' issues in saved .ini files:
• end of numeric line after "CO2COLOR" (or start of DIRECTIONCOLOR)
• end of numeric line after "SHOWHRRPLOT" (or start of SHOWMISSINGOBJECTS)
I haven't seen any problems because of these, so I'm just highlighting for info. No need from me for a patched version, thanks.
from smv.
gforney
commented on September 4, 2024
thanks. should be fixed in smokeview's posted here
https://drive.google.com/drive/folders/0B_wB1pJL2bFQc1F4cjJWY2duWTA?resourcekey=0-J_yHat7iJoBp1fk6QS8gMA&usp=sharing
…On Tue, Oct 18, 2022 at 9:33 AM obscureed ***@***.***> wrote:
Reopening because I think there are a couple of similar missing '\n'
issues in saved .ini files:
• end of numeric line after "CO2COLOR" (or start of DIRECTIONCOLOR)
• end of numeric line after "SHOWHRRPLOT" (or start of SHOWMISSINGOBJECTS)
I haven't seen any problems because of these, so I'm just highlighting for
info. No need from me for a patched version, thanks.
—
Reply to this email directly, view it on GitHub
<#1430 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AC6UCRTIKS6BHIUGHJXF7A3WD2RLDANCNFSM6AAAAAAQI23AZ4>
.
You are receiving this because you modified the open/close state.Message
ID: ***@***.***>
--
Glenn Forney
from smv.
obscureed
commented on September 4, 2024
Great, thanks. I've verified the patched version and am closing the issue.
from smv.
Related Issues (20)
- smokeview problem with the loading of Isosurface HOT 5
- Start Smokeview window on active monitor HOT 8
- Grid orientation effect in volume rendering of HRRPUV HOT 4
- Temperature slice colobar retains level set color bar if loaded after level set slice HOT 5
- GEOM BNDF not showing correct bounds HOT 8
- Hiccup in slice file with restart case HOT 2
- CFAST Visualization possible temperature colorbar error HOT 10
- Smokeview not displaying boundary data at mesh interface HOT 6
- OBST outline with "blockage color" does not use lighting HOT 3
- How to display slice within geometry with latest smv HOT 5
- Devices look huge in OSX HOT 2
- Long directory names in smokeview SSF scripts crash HOT 2
- smokeview has lost "load all" commands in right-click GUI HOT 10
- GEOMETRY COLOR overridden by SURF color in SMV HOT 2
- Loading multiple SMV results to compare HOT 2
- Hot key 'u' does not reload HRRPUV 3D smoke data HOT 3
- rendering HVAC HOT 5
- Smokeview does not show BNDF data at boundaries of meshes HOT 22
- Load z for soot vis not avaiable HOT 5
- SMV Crashes when loading many slices, possible memory cap HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from smv.