Added National Highways dataset

[peterjamesnugent]

NOTE: Depends on

BHoM/BHoM#1453

Issues addressed by this PR

Closes #284

Add the following dataset: https://nationalhighways.co.uk/suppliers/design-standards-and-specifications/carbon-emissions-calculation-tool/

Test files

https://burohappold.sharepoint.com/:u:/r/sites/BHoM/02_Current/12_Scripts/01_Issue/BHoM/LifeCycleAssessment_Toolkit/%23284-Add%20dataset%20for%20National%20Highways/Check%20dataset%20for%20National%20Highways.gh?csf=1&web=1&e=htB355

Changelog

• Added dataset from the National Highways Carbon Tool v2.5
• The EnvironmentalProductDeclaration quantity has been set as the carbon factor x conversion factor and converted to SI units
• The transport factors are to be scaled by the mass they are carrying

Additional comments

I do have a few questions:
[Description("The number of units in reference to quantity type. Example, 1000 kg per unit quantityType of Mass.")]

• The EnvironmentalProductDeclaration input QuantityTypeValue is described above - how does this relate to SI units? Or is the mass the exception because the units for GlobalWarmingPotential (GWP) is measured in tCO2?
  Normalise the carbon factors to SI units.
• The quantity types do not have an eNum for energy, one of the datasets has GWP for gas expressed as kgCO2e/kWh - so the quantity would be energy - what other methods need to be updated to include this eNum?
  Add energy eNum for QuantityValueType to `LifeCycleAssessment_oM
• Similarly, there is data for laden vehicles expressed as tCO2e/(tkm) - so we would need a quantity that represented that.
  Defined in km and then scaled by the mass transported.
• There are some notes on methodology - is this typically stored in the CustomData or is there a specific fragment?
  Added in AdditionalEPDInformation metric.
• There are conversion factors part of the data (that I have included as it simplifies the calculation) - is there anywhere to note this?
  Include

[FraserGreenroyd]

@BHoMBot check dataset-compliance

[bhombot-ci]

@FraserGreenroyd to confirm, the following actions are now queued:

• check dataset-compliance

There are 9 requests in the queue ahead of you.

[bhombot-ci]

Please be advised that the check with reference 9138912711 has more than 50 annotations of notes. API limitations restrict annotations to 50. You may need to rerun this check to obtain the next set when you make changes. At the time of reporting this check, there are 26 additional annotations waiting, made up of 0 errors and 26 warnings.

[peterjamesnugent]

@BHoMBot check dataset-compliance

[FraserGreenroyd]

@BHoMBot check required

[bhombot-ci]

@FraserGreenroyd 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

[peterjamesnugent]

@IsakNaslundBh @StephennipBH @michaelhoehn this is ready to be reviewed and incorporates the changes we discussed.

[michaelhoehn]

@BHoMBot check required

[bhombot-ci]

@michaelhoehn 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

[michaelhoehn]

All the datasets look great on testing and I'm happy to add them as is. I would however request that a simple case and error statement be added to the EvaluateEnvironmentalProductDeclaration method to capture the added Energy enum value.

This message can simply state something along the lines of "This calcuation has not yet been implemented, please perform the calculation manually or request this functionality in the LCA toolkit".

Changes made

[peterjamesnugent]

@BHoMBot check required

[michaelhoehn]

Happy to merge this. Thanks @peterjamesnugent

[michaelhoehn]

@BHoMBot check required

[bhombot-ci]

@michaelhoehn 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

[peterjamesnugent]

@BHoMBot check dataset-compliance
@BHoMBot check copyright-compliance
@BHoMBot check ready-to-merge

[bhombot-ci]

@peterjamesnugent to confirm, the following actions are now queued:

• check dataset-compliance
• check copyright-compliance
• check ready-to-merge