Add support for MidasCivil 2023 v1.1 and bug fixes

[peterjamesnugent]

Issues addressed by this PR

Closes #341
Closes #351
Closes #353

Test files

Push/Pull Elements

Push/Pull Loads

Push/Pull RigidLink and BarRelease

Changelog

Added functionality for MidasCivil 2023 v1.1
Added changes to RigidLink following changes from Version 9.0.0
Fixed bug where BarReleases were not pushing/pulling correctly (unrelated to versions)

Additional comments

If we want to test things separately please tick the boxes below:

• Tested MidasCivil 2023 v1.1 elements and properties
• Tested MidasCivil 2023 v1.1 loads
• Tested RigidLink fixes
• Tested BarRelease fixes

[peterjamesnugent]

@BHoMBot check required

[bhombot-ci]

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

There are 32 requests in the queue ahead of you.

[StephennipBH]

Only reviewing the rigid link and beam end releases. Here are the following issues I found with the pulling back for beam end release and rigid link. The rigid link comes back false rather than true in XtoX YtoY ZtoZ. The beam end release comes back null

[jtrainor1]

Reviewing the push/pull of loads to MIDAS civil, I've found some issues with the temperature loading. Firstly, the bar uniform temperature loading is only pushing one of the two load specified to the text file.

Only the +2 degree change is being pushed to the text file currently.

Next the area differential temperature is bringing other uniform temperature loads that do not centre the value to 0.

For example, in the above image. A temperature differential of 13 to -1, I would assume it would add +1, not +6 as a uniform load.

[peterjamesnugent]

@jtrainor1 the first issue was because the load had the same name, so it invoked Delete and Update - hence it overwrites and only appears that one load has been pushed. I have updated the script so the loads have unique names.

Regarding the AreaDifferentialTemperatureLoad I found a bug where the unit conversion was not working properly (so if you test it in F it will push/pull properly.

Regarding the comment about the offset being in correct, for a TopTemperature = 13 and BottomTemperature = -1, Midas will centre the differential element about 0, so it will have Top Temperature = + 7 and BottomTemperature = -7 so the required offset will be +6. This has come up quite a bit so I will add a section to the Wiki,

[peterjamesnugent]

Wiki updated here: https://github.com/BHoM/MidasCivil_Toolkit/wiki/MidasCivil-Specific-Adapter-Features#areadifferentialtemperautreload

[elizawren]

Spot checks of node/bars/FEMesh locations as well as section properties, everything checked was ok

[jtrainor1]

Still some issues with pushing the uniform bar temperature loading:

No -2 found.

Also in the grasshopper script itself I see 4 loads, is this correct?:

[StephennipBH]

Bug regarding pulling bar release have been addressed and fixed. Push/pull of bar release and rigid link work as intended

[peterjamesnugent]

@BHoMBot check required

[bhombot-ci]

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

[jtrainor1]

I have checked how loads are pushed and pulled into Midas Civil. I have checked the following load types:

• Bar temperature loads
• Self weight
• Bar uniform and varying loads
• Bar point loads
• Point loads
• Area uniform loads
• Area uniform and differential temperature loading

These are all functioning correctly.

[peterjamesnugent]

@BHoMBot check copyright-compliance
@BHoMBot check versioning
@BHoMBot check installer

[bhombot-ci]

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

• check copyright-compliance
• check versioning
• check installer

There are 3 requests in the queue ahead of you.

[bhombot-ci]

The check installer has already been run previously and recorded as a successful check. This check has not been run again at this time.

[FraserGreenroyd]

@BHoMBot this is a DevOps instruction. I am requesting neutral checks on: versioning, ready-to-merge

[bhombot-ci]

@FraserGreenroyd I have provided neutral checks to the checks requested. These checks will need to be run properly to obtain full results.