Skip to content

[BHP1-1510] Update VMS with new Direction Variables Attribute#195

Open
Kcheung42 wants to merge 3 commits intokc-BHP1-1355-feat-consolidate-direction-variables-settingsfrom
kc-BHP1-1510-vms-updates
Open

[BHP1-1510] Update VMS with new Direction Variables Attribute#195
Kcheung42 wants to merge 3 commits intokc-BHP1-1355-feat-consolidate-direction-variables-settingsfrom
kc-BHP1-1510-vms-updates

Conversation

@Kcheung42
Copy link
Collaborator

@Kcheung42 Kcheung42 commented Mar 18, 2026

Purpose

Related Issues

Closes BHP1-1510

Submission Checklist

  • Included Jira issue in the PR title (e.g. BHP1-### <title>)
  • Code passes linter rules (clj-kondo --lint components/**/src bases/**/src projects/**/src)
  • Feature(s) work when compiled (clojure -M:compile-cljs)

Testing

  1. Open local vms and run migration script 2026-03-03-add-direction-variables-attr-to-group-variables

  2. Ensure for these "Directional Parent" (the ones with no direction set) variables exist and it's corresponding Directional Children exists. All Directional Parents have it's children listed under "Direction Variables" header. All children are properly configured with the cpp code and conditionals that trigger them to be enabled.

Surface > Fire Behavior (output) > Surface Fire : Rate of Spread
Surface > Fire Behavior (output) > Surface Fire : Flame Length
Surface > Fire Behavior (output) > Surface Fire : Fireline Intensity
Surface > Fire Behavior (output) > Surface Fire : Spread Distance

Mortality > Mortality (output) > Tree Mortality : Probability of Mortality
Mortality > Mortality (output) > Tree Mortality : Scorch Height
Mortality > Mortality (output) > Tree Mortality : Tree Crown Length Scorch
Mortality > Mortality (output) > Tree Mortality : Tree Crown volume Scorch
Mortality > Mortality (output) > Tree Mortality : Bole Char Height

Screenshots

Notice some quality of life changes to the views to differentiate between directional children and its parent
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant