Fix issue pushing gravity loads with meshes with more than one face

[IsakNaslundBh]

NOTE: Depends on

Issues addressed by this PR

Closes #258

Special case for getting ids from FEMesh

Get Ids from FEFaces instead of host FEMesh when producing IDList.

Adding in OrderBy to ensure IDString is as simple as possible

Test files

https://burohappold.sharepoint.com/:f:/s/BHoM/Esm46kmSSANDghV6Hu9HtH0BxJZNmiT3_IK1NWnd849prw?e=UiKicn

Changelog

Additional comments

[IsakNaslundBh]

@BHoMBot check compliance

[bhombot-ci]

@IsakNaslundBh to confirm, the following checks are now queued:

• code-compliance
• documentation-compliance
• project-compliance
• branch-compliance
• dataset-compliance
• copyright-compliance

[alelom]

Approved based on:

• code: LGTM after comments
• test file: objects and loads appear correctly pushed

[alelom]

@BHoMBot check required

[bhombot-ci]

@alelom to confirm, the following checks are now queued:

• code-compliance
• documentation-compliance
• project-compliance
• core
• null-handling
• serialisation
• versioning
• installer

[bhombot-ci]

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]

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.

[JosefTaylor]

Builds OK, but cannot test as I don't have GSA installed.
I suggested a change as a public method name is misspelled, but I don't mind if you don't.

[alelom]

@BHoMBot ready-to-merge

[bhombot-ci]

@alelom sorry, I didn't understand.
Was that comment an instruction for me? If so, could you state again what check you would like me to do?
For a list of available instructions, please see this wiki page.

[alelom]

@BHoMBot check ready-to-merge

[bhombot-ci]

@alelom to confirm, the following checks are now queued:

• ready-to-merge

There are 46 requests in the queue ahead of you.

[IsakNaslundBh]

/azp run GSA_Toolkit.CheckCore