BHoM / LifeCycleAssessment_Toolkit

GNU Lesser General Public License v3.0
22 stars 5 forks source link

ModifyInsulationEPD Method #352

Closed vgreen-BH closed 1 month ago

vgreen-BH commented 2 months ago

NOTE: Depends on

https://github.com/BHoM/Localisation_Toolkit/pull/113

Issues addressed by this PR

Closes #317

All US Insulation EPDs are area based EPDs assuming RSI = 1.0. The LCA calculation methods do not support calculations for these in cases where RSI does not = 1.0. This method creates a new area based EPD with the quantity value updated to reflect 1 m2 area with the project RSI value.

Test files

https://burohappold.sharepoint.com/:u:/r/sites/BHoM/02_Current/12_Scripts/03_Alpha/BHoM/BHoM_Engine/Facade_Engine/BHoM%20PR%20Test%20Files/240430_Insulation%20EPD%20Method%20Test%202.gh?csf=1&web=1&e=TagkGe

enarhi commented 2 months ago

@BHoMBot check required

bhombot-ci[bot] commented 2 months ago
@enarhi to confirm, the following actions are now queued: - check `code-compliance` - check `documentation-compliance` - check `project-compliance` - check `core` - check `null-handling` - check `serialisation` - check `versioning` - check `installer`
enarhi commented 2 months ago

@BHoMBot check compliance

bhombot-ci[bot] commented 2 months ago
@enarhi to confirm, the following actions are now queued: - check `code-compliance` - check `documentation-compliance` - check `project-compliance` - check `branch-compliance` - check `dataset-compliance` - check `copyright-compliance`
bhombot-ci[bot] commented 2 months ago
The check `code-compliance` has already been run previously and recorded as a successful check. This check has not been run again at this time.
bhombot-ci[bot] commented 2 months ago
The check `documentation-compliance` has already been run previously and recorded as a successful check. This check has not been run again at this time.
enarhi commented 2 months ago

@BHoMBot check ready-to-merge

bhombot-ci[bot] commented 2 months ago
@enarhi to confirm, the following actions are now queued: - check `ready-to-merge`