Smokeview Clipping Validation

[firemist]

When performing clipping, the dialog box is enforcing an unnecessary validation rule.

Sample case: I am looking at a single story home with an attic fire. I want to be able to look down at things happening in the first story and at other times, look up to the underside of the attic. Therefore, I frequently desire clipping planes at zmin=3.0m, to look up from below, or zmax=2.2m, to look down from the top. However, even though neither of these are enabled at the same time, the validation box will not allow the seemingly inconsistent numbers. In the attached image, zmin must be set to a number lower than zmax, even though zmin is not asserted/enabled.

Recommendation: only enforce validation rules for enabled/asserted clipping values.

Thanks

[gforney]

Ill take a look

…

On Sat, Nov 13, 2021, 4:01 PM firemist ***@***.***> wrote: When performing clipping, the dialog box is enforcing an unnecessary validation rule. Sample case: I am looking at a single story home with an attic fire. I want to be able to look down at things happening in the first story and at other times, look up to the underside of the attic. Therefore, I frequently desire clipping planes at zmin=3.0m, to look up from below, or zmax=2.2m, to look down from the top. However, even though neither of these are enabled at the same time, the validation box will not allow the seemingly inconsistent numbers. In the attached image, zmin must be set to a number lower than zmax, even though zmin is not asserted/enabled. Recommendation: only enforce validation rules for enabled/asserted clipping values. Thanks [image: image] <https://user-images.githubusercontent.com/25644607/141658902-917ec92a-9190-49b3-87ac-18b7409812f7.png> — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#10037>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AC6UCRUOTEDGX33MGI2PVBTUL3GZRANCNFSM5H7A5QUQ> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[gforney]

try smokeview posted here. it should allow you set any clipping plane values you want
https://drive.google.com/drive/folders/0B_wB1pJL2bFQc1F4cjJWY2duWTA?resourcekey=0-J_yHat7iJoBp1fk6QS8gMA&usp=sharing

I'm now letting you enter any value for clipping min and max planes. if you do specify a min clipping plane > a max clipping plane the scene will be hidden. it would make the code too complicated to have constraints in some situations but not others.

[firemist]

That works fine. Thank you.