Skip to content

add BaseFloodElevation to structure - #154

Merged
HenryGeorgist merged 1 commit into
USACE:mainfrom
sebastianrowan:feature/add-bfe-to-structureprovider
Sep 24, 2026
Merged

HenryGeorgist merged 1 commit into
USACE:mainfrom
sebastianrowan:feature/add-bfe-to-structureprovider

Conversation

@sebastianrowan

Copy link
Copy Markdown
Contributor

No description provided.

@HenryGeorgist HenryGeorgist left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks god but it doesnt look like you modified the compute yet.

Comment thread structures/structure.go
OccType OccupancyTypeStochastic
FoundType, FirmZone, ConstructionType string
StructVal, ContVal, FoundHt consequences.ParameterValue
BaseFloodElevation float64 // Would BFE ever be stochastic and require type to be ParameterValue?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no i dont think so, at least not until fema fully embraces uncertainty in bfe

@HenryGeorgist
HenryGeorgist merged commit ef988bf into USACE:main Sep 24, 2026
1 check passed
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.

2 participants