BHoM / BHoM_Engine

Internal manipulation of the BHoM
GNU Lesser General Public License v3.0
26 stars 13 forks source link

Type matching based on unqualified names enabled in Create.EngineType #3387

Closed pawelbaran closed 1 month ago

pawelbaran commented 1 month ago

Issues addressed by this PR

Closes #3385 Closes #3386

Test files

Passing serialisation and versioning checks should be enough of verification.

Changelog

Additional comments

Fix for #3385 is almost self-explanatory, while #3386 requires a bit of explanation I believe. The origin of the bug is renaming of Revit assemblies that led me to finding out that Create.EngineType matches types based on fully qualified name - after renaming the assemblies the old and new names are different, resulting in a failure. Two steps were required to address that:

  1. Enabling matching by unqualified name
  2. Enabling picking first type in case of finding multiple matching types, similar to what we did with @IsakNaslundBh in https://github.com/BHoM/BHoM_Engine/pull/3344
pawelbaran commented 1 month ago

@BHoMBot check compliance @BHoMBot check serialisation @BHoMBot check versioning

bhombot-ci[bot] commented 1 month ago
@pawelbaran to confirm, the following actions are now queued: - check `code-compliance` - check `documentation-compliance` - check `project-compliance` - check `branch-compliance` - check `dataset-compliance` - check `copyright-compliance` - check `serialisation` - check `versioning` There are 23 requests in the queue ahead of you.
pawelbaran commented 1 month ago

@BHoMBot check compliance @BHoMBot check serialisation @BHoMBot check versioning

bhombot-ci[bot] commented 1 month ago
@pawelbaran to confirm, the following actions are now queued: - check `code-compliance` - check `documentation-compliance` - check `project-compliance` - check `branch-compliance` - check `dataset-compliance` - check `copyright-compliance` - check `serialisation` - check `versioning` There are 38 requests in the queue ahead of you.
bhombot-ci[bot] commented 1 month ago
The check `branch-compliance` has already been run previously and recorded as a successful check. This check has not been run again at this time.
pawelbaran commented 1 month ago

@BHoMBot check versioning @BHoMBot check serialisation

bhombot-ci[bot] commented 1 month ago
@pawelbaran to confirm, the following actions are now queued: - check `versioning` - check `serialisation`
pawelbaran commented 1 month ago

@BHoMBot check installer @BHoMBot check core @BHoMBot check compliance @BHoMBot check unit-tests

bhombot-ci[bot] commented 1 month ago
@pawelbaran to confirm, the following actions are now queued: - check `installer` - check `core` - check `code-compliance` - check `documentation-compliance` - check `project-compliance` - check `branch-compliance` - check `dataset-compliance` - check `copyright-compliance` - check `unit-tests`
pawelbaran commented 1 month ago

@BHoMBot check null-handling

bhombot-ci[bot] commented 1 month ago
@pawelbaran to confirm, the following actions are now queued: - check `null-handling` There are 16 requests in the queue ahead of you.
bhombot-ci[bot] commented 1 month ago
FAO: @FraserGreenroyd @pawelbaran is seeking dispensation on this Pull Request to skip a required check. Please can you provide authorisation for the check to be skipped, or provide assistance as appropriate. The check they wish to have dispensation on is unit-tests. If you are providing dispensation on this occasion, please reply with: > @BHoMBot this is a DevOps instruction. I am authorising dispensation to be granted on check ref. `28350420492`
pawelbaran commented 1 month ago

I am dispensating the failing UT check on the basis of the fact that all reported errors were present in the past (e.g. here). @peterjamesnugent @IsakNaslundBh maybe that's something we could try to clear/fix in the coming sprints?

pawelbaran commented 1 month ago

@BHoMBot this is a DevOps instruction. I am authorising dispensation to be granted on check ref. 28350420492

bhombot-ci[bot] commented 1 month ago
@pawelbaran I have now provided a passing check on reference `28350420492` as requested.
pawelbaran commented 1 month ago

@BHoMBot check ready-to-merge

bhombot-ci[bot] commented 1 month ago
@pawelbaran to confirm, the following actions are now queued: - check `ready-to-merge`