search

HephaestusProject / pytorch-transformer

MIT License
0 stars 0 forks source link

Feature/encoder #7

Closed inmoonlight closed 4 years ago

inmoonlight commented 4 years ago

Pull Request

레파지토리에 기여해주셔서 감사드립니다.

해당 PR을 제출하기 전에 아래 사항이 완료되었는지 확인 부탁드립니다:

1. 해당 PR은 어떤 내용인가요?

Transformer encoder

2. PR과 관련된 이슈가 있나요?

1

codecov[bot] commented 4 years ago

Codecov Report

Merging #7 into master will decrease coverage by 3.87%. The diff coverage is 95.10%.

Impacted file tree graph

@@             Coverage Diff             @@
##            master       #7      +/-   ##
===========================================
- Coverage   100.00%   96.12%   -3.88%     
===========================================
  Files            5        7       +2     
  Lines          183      232      +49     
===========================================
+ Hits           183      223      +40     
- Misses           0        9       +9
Flag Coverage Δ
#tests 96.12% <95.10%> (-3.88%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/dataloader.py 86.95% <86.95%> (ø)
src/data/data_utils.py 94.73% <94.73%> (ø)
src/utils.py 98.00% <97.22%> (-2.00%) :arrow_down:
src/data/dataset.py 97.95% <97.95%> (ø)
tests/test_dataset.py 100.00% <100.00%> (ø)
tests/test_utils.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update e4ed23f...0680e26. Read the comment docs.

ssaru commented 4 years ago

@inmoonlight 지형님! 혹시 PR 반영을 완료하신 것이라면, 멘션으로 걸어주시면 "제가 PR을 봐야하는구나!"라고 인지하기 더 좋을 것 같습니다! ㅎㅎ 아니면 슬랙 DM으로 쪼아주셔도 저는 좋습니다! ㅋㅋㅋㅋ (제가 보틀넥이 될까 무섭네요 ㅠㅠ)

inmoonlight commented 4 years ago

@ssaru @aisolab test code 작성 외에는 리뷰 반영 하였습니다! ㅋㅋㅋㅋㅋ test code는.. 조금 우선순위가 내려가서 커버리지가 90 이상이면 우선 넘어가고 마지막에 100으로 맞춰볼게요!

inmoonlight commented 4 years ago

추후 테스트코드 짜는 것을 잊지 않기 위한 이슈: https://github.com/HephaestusProject/pytorch-transformer/issues/10