-
Notifications
You must be signed in to change notification settings - Fork 0
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/data loader #6
Conversation
reserve <bos>, <eos>, <pad>
Remove and update script
54a5892
to
dd6fe99
Compare
f8d63f4
to
609d798
Compare
609d798
to
700f119
Compare
Codecov Report
@@ Coverage Diff @@
## master #6 +/- ##
==========================================
Coverage 100.00% 100.00%
==========================================
Files 5 5
Lines 61 183 +122
==========================================
+ Hits 61 183 +122
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
load dataset need wmt14 corpus to test
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.
@inmoonlight
코멘트 완료했습니다.
수고하셨습니다. 🙇♂️
@aisolab 보섭님도 한번 봐주시지요~ |
- additionally, fix deen to en-de to show source and target language
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.
@inmoonlight Approve했습니다.
고생하셨습니다.
Pull Request
레파지토리에 기여해주셔서 감사드립니다.
해당 PR을 제출하기 전에 아래 사항이 완료되었는지 확인 부탁드립니다:
1. 해당 PR은 어떤 내용인가요?
<eos> <bos> <pad>
를 reserve하지 않았어서 다시 학습하여 올립니다2. PR과 관련된 이슈가 있나요?
#1