-
Notifications
You must be signed in to change notification settings - Fork 570
[Proposal] Additional Architecture Adapter tests #1302
Copy link
Copy link
Open
Labels
TransformerBridgeBug specific to the new TransformerBridge systemBug specific to the new TransformerBridge systemcomplexity-simpleSimple issues, which may be good for beginnersSimple issues, which may be good for beginnersenhancementNew feature or requestNew feature or requestgood first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is neededlow-priorityMaintainers are not prioritising this work currently.Maintainers are not prioritising this work currently.
Metadata
Metadata
Labels
TransformerBridgeBug specific to the new TransformerBridge systemBug specific to the new TransformerBridge systemcomplexity-simpleSimple issues, which may be good for beginnersSimple issues, which may be good for beginnersenhancementNew feature or requestNew feature or requestgood first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is neededlow-priorityMaintainers are not prioritising this work currently.Maintainers are not prioritising this work currently.
Proposal
Add config tests for remaining architecture adapters
Motivation
Only about a third of our Architecture Adapters have focused tests in
tests/unit/model_bridge/supported_architectures/. Use the existing available test suites as examples for how the new tests should be written.Pitch
Create a small test suite for each of the remaining architecture adapters that do not have one at present. Feel free to tackle as few or as many at once as you'd like, just leave a comment stating which architecture(s) you're working on to reduce duplication of effort.
Architectures that need tests
Checklist