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

V1.0 논의 #12

Open
jinmang2 opened this issue Apr 7, 2022 · 3 comments · May be fixed by #16
Open

V1.0 논의 #12

jinmang2 opened this issue Apr 7, 2022 · 3 comments · May be fixed by #16
Assignees
Labels
enhancement New feature or request v1.x.x

Comments

@jinmang2
Copy link
Owner

jinmang2 commented Apr 7, 2022

  • pororo에서 지원하지 않는 새로운 task 추가
    • 예로 dialogue state tracking, information retrieval(lexical, sparse, dense)
  • pororo에서 지원하지 않는 언어 추가
    • 예로 grammatical error correction (korean은 spacing만 지원)
  • 재추상화
    • summarization의 경우 abs, ext, bullet 등 세부 task가 있어서 forward가 달라져야함
    • 하지만 지금 추상화에서는 이를 상속받기 어려움
    • 그리고 WSD의 경우 seq2seq으로 접근하는 방식 외에 bi-encoder 구조로 접근할 수도 있는데 지금 추상화는 한 개의 category만 접근 가능
  • build fn 변경
    • 지금은 HUB_NAME="jinmang2/dooly-hub"로 강제하여 다운로딩하고 있음
    • 이는 향후 contribution을 원하는 분들의 자유도를 크게 저하시킬 수 있음
@jinmang2 jinmang2 self-assigned this Apr 7, 2022
@jinmang2 jinmang2 added enhancement New feature or request v1.x.x labels Apr 7, 2022
@jinmang2
Copy link
Owner Author

v0.x.x는 Pororo의 모든 것을 cover해야함.
v1.x.x는 새로운 task를 추가하고 user contribution을 할 수 있는 환경으로 둘리를 만들고
v2.x.x는 한국어 nltk, spacy같은 범용 라이브러리로 확장할 것임

@jinmang2
Copy link
Owner Author

Continuous Integration에도 아래와 같은 수정이 필요함

  • flake8과 black은 pre-commit에서 검사 후 commit하는 것이 속편함

@jinmang2
Copy link
Owner Author

v1.x.x에서 hydra로 configure하는 방향도 고려해볼 것

@jinmang2 jinmang2 linked a pull request May 3, 2022 that will close this issue
@jinmang2 jinmang2 linked a pull request Jun 13, 2022 that will close this issue
Closed
@jinmang2 jinmang2 removed a link to a pull request Jun 14, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request v1.x.x
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant