epwalsh / epwalsh/python-registrable
Adding FromParams and configuration file pipeline
Open
- Dominant language
- Python
- Stars
- 13
- Forks
- 1
- PR merge metrics
- No merged PRs in 30d
Description
The best part about having a registry is that one can write intelligent instantiation logic (once) and not worry about changing that logic as more children classes get added to the code. In order to achieve that, I believe, having the FromParams class in this package will be very useful.
Are you planning to add that? I could take a stab at it as well - do you foresee any issue with adding the `FromParams` mixin from AllenNLP as is?
Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.