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

Serving test #43

Merged
merged 4 commits into from
Dec 22, 2021
Merged

Serving test #43

merged 4 commits into from
Dec 22, 2021

Conversation

presto105
Copy link
Contributor

최근 update된 code에 맞게 bigbart visualization 코드를 업데이트 하였습니다.

  • 변경 내용
  1. serving utils에서 '##'처리하는 index 생성 코드 수정
  2. Highlight를 위한 text processing 코드 수정
  3. 최근 수정되어 필요없어진 load config제거

split_words_indices = split_words_indices[1:]
return split_words_indices

def token_to_words(
text_tokens: List[str],
model_type: str
) -> List[str] :
if model_type != 'kobigbirdbart' :
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

아래 부분도 동일하게 적용되는 내용으로 model part 코드와 일관성을 위해 bigbart로 설정해주시면 좋을 것 같습니다.

@presto105
Copy link
Contributor Author

수정완료했습니다!

@gistarrr gistarrr merged commit a8a906b into dev Dec 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants