search

victoresque / pytorch-template

PyTorch deep learning projects made easy.
MIT License
4.75k stars 1.09k forks source link

add easydict wrapper to config dict for easy access #30

Closed ionvision closed 6 years ago

ionvision commented 6 years ago

Process the config dict to get alternative Python object attributes with EasyDict.

SunQpark commented 6 years ago

Hi, @ionvision. Thank you for your suggestion. As I understand, the feature EasyDict provide is making dictionary items accessible with the attributes as well as keys(strings). I think that feature looks helpful, especially to the base classes containing a lot of access to the config. However, doing this will add an additional dependency to this project. Considering this is just a template, I think the dependencies should be kept minimal. Tell me if there are things I missed. Thank you

ionvision commented 6 years ago

Hi @SunQpark , keeping the template minimal is quite reasonable, and thanks again for the good work.