BHoM / MidasCivil_Toolkit

GNU Lesser General Public License v3.0
7 stars 3 forks source link

Fix a bug where REBAR-MATL-CODE creates an error when running mct file #373

Closed KalleEdstroem closed 8 months ago

KalleEdstroem commented 8 months ago

Issues addressed by this PR

Closes #180

Fix bug where mct file failes to run with error message.

Error has been encountered during the data input. : *The command of REBAR-MATL-CODE cannot be executed when nodes or sections are included in the model. Run Error : command in line 8-10

Fixed by not including REBAR-MATL-CODE when nodes or sections are included.

Test files

https://burohappold.sharepoint.com/:f:/r/sites/BHoM/02_Current/12_Scripts/02_Pull%20Request/BHoM/Lusas_Toolkit/%23373-FixBugWithMctFailsToParse?csf=1&web=1&e=vm1gbm

peterjamesnugent commented 8 months ago

@BHoMBot check required @BHoMBot check compliance

bhombot-ci[bot] commented 8 months ago
@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` - check `code-compliance` - check `documentation-compliance` - check `project-compliance` - check `branch-compliance` - check `dataset-compliance` - check `copyright-compliance`
bhombot-ci[bot] commented 8 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 8 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.
peterjamesnugent commented 8 months ago

@BHoMBot check ready-to-merge

bhombot-ci[bot] commented 8 months ago
@peterjamesnugent to confirm, the following actions are now queued: - check `ready-to-merge` There are 14 requests in the queue ahead of you.