Revit => BHoM panel convert performance improved plus minor facade panel convert bug fixed

[pawelbaran]

Issues addressed by this PR

Closes #1236
Closes #1237

Test files

On SharePoint
Worth testing the standard test procedure files too @vietle-bh

Changelog

• Revit => BHoM panel convert performance improved
• facade panel convert bug fixed

Additional comments

The proposed performance improvement is quite complex in implementation, but I will try to explain it in a few points 🙈

Before:

• each panel element converted individually
• temporary transaction and document regeneration done for each element separately

After:

• each panel element converted individually
• before any converts happen, the panel locations are being extracted and cached in refObjects to be used when the actual converts get called
• thanks to the above, all panel locations are queried in a single call, therefore only one temporary transaction is opened for all of them

In some cases this may cut down the execution time from hours to minutes.

[pawelbaran]

We just had a call with @enarhi and specifically tested #1236 on another test file to double check that the change did not break the existing feature. Everything looks fine, @enarhi is happy, so please review @vietle-bh and let's merge once your comments get answered 👍

[pawelbaran]

@BHoMBot check required

[bhombot-ci]

@pawelbaran 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 39 requests in the queue ahead of you.

[pawelbaran]

@BHoMBot check required

[bhombot-ci]

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

[pawelbaran]

@BHoMBot check required

[bhombot-ci]

@pawelbaran 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 2 requests in the queue ahead of you.

[pawelbaran]

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

[bhombot-ci]

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

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

[pawelbaran]

@BHoMBot check required

[bhombot-ci]

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

[pawelbaran]

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

[bhombot-ci]

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

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

There are 13 requests in the queue ahead of you.