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
Support setting arguments of pretraining by a config file #22
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sample 수준의 config 파일이라도 같이 포함되어야 할 것 같습니다. 아까 만드셨던 거라도 더해주시면 좋을 것 같아요~!!
@iron-ij 아 그거 추가되어있을텐데요?? |
없는 거 확인하고 추가했습니다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
model_type의 가능 범주를 알려주는 조건절 명령어 추가
Co-authored-by: Minho Ryu <ryumin93@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM :)
arguments를 전달함.
CONFIG_MAPPING
을 활용하여 class 생성자 호출Refs: #16