BHoM / BHoM_Adapter

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

Adding method to create adapter from Type #301

Closed IsakNaslundBh closed 2 years ago

IsakNaslundBh commented 2 years ago

Issues addressed by this PR

Closes #300

Adding method for creating an adapter based on a type via reflection.

Useful for creation of adapters dynamically from the code side and also have some use from excel vba workflows as well as testscripts in grasshopper.

Will be required for automatic interop testing by the BHoMBot.

Test files

https://burohappold.sharepoint.com/:f:/s/BHoM/Emk-_8egMhhDpHySuegqFAEBLUoH3DJu4lq-RcG6-CRYJg?e=psemXO

Changelog

Additional comments

IsakNaslundBh commented 2 years ago

Trying to run this on my machine, with the soft wares I have installed, I am able to create all adapters but the following using this method when not providing any file path and/or config:

Failing adapters:

DIALuxAdapter EnergyPlusAdapter ETABS18Adapter FileAdapter LusasV17Adapter LusasV18Adapter LusasV19Adapter MidasCivilAdapter MongoAdapter RAMAdapter RFEMAdapter SharepointAdapter SocketAdapter TasT3DAdapter

Etabs show up as failing as it can only load one version at the time from the side of the code.

Lusas, Midas, RAM, EnergyPlus, RFEM and TAS I do not have installed on my machine so might be one of the reasons this is failing.

IsakNaslundBh commented 2 years ago

@BHoMBot check compliance @BHoMBot check null-handling

bhombot-ci[bot] commented 2 years ago
@IsakNaslundBh to confirm, the following checks are now queued: - `code-compliance` - `documentation-compliance` - `project-compliance` - `branch-compliance` - `dataset-compliance` - `copyright-compliance` - `null-handling`
FraserGreenroyd commented 2 years ago

@BHoMBot check required

bhombot-ci[bot] commented 2 years ago
@FraserGreenroyd to confirm, the following checks are now queued: - `code-compliance` - `documentation-compliance` - `project-compliance` - `core` - `null-handling` - `serialisation` - `installer` - `versioning`
bhombot-ci[bot] commented 2 years 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 2 years 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.
bhombot-ci[bot] commented 2 years ago
The check `project-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 2 years ago
The check `null-handling` 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 2 years ago
@IsakNaslundBh just to let you know, I have provided a `check-installer` result to this Pull Request as it was detected to be linked to other Pull Requests in a series. The comment which triggered this check came from @FraserGreenroyd on SAP2000_Toolkit
bhombot-ci[bot] commented 2 years ago
@IsakNaslundBh just to let you know, I have provided a `check-versioning` result to this Pull Request as it was detected to be linked to other Pull Requests in a series. The comment which triggered this check came from @FraserGreenroyd on SAP2000_Toolkit
IsakNaslundBh commented 2 years ago

@BHoMBot check ready-to-merge

bhombot-ci[bot] commented 2 years ago
@IsakNaslundBh to confirm, the following checks are now queued: - `ready-to-merge` There are 16 requests in the queue ahead of you.