Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FEATURE] Simpler configuration file parser and support to have default value #37

Open
triwahyuu opened this issue Jul 26, 2020 · 0 comments
Labels
enhancement New feature or request

Comments

@triwahyuu
Copy link
Member

Is your feature request related to a problem? Please describe.
with current implementation:

  • we need to write all the configuration which is unnecessary since most of them are set in default values
  • use a complex parser definition, hard to read

Describe the solution you'd like

  • have a default values for some config if possible
  • need to create requiredness category for each config field
  • use a dict (or easydict)-based config parser

Describe alternatives you've considered
we could also look into mmdetection and detectron2 implementation

@alphinside alphinside added the enhancement New feature or request label Jul 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants