[Issue-32] Revise the path of resource files #33
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Lot of thx! As you may well know, komoran was developed to be used without external library dependencies :) As you mentioned, would you change concat logic using guava to stringbuilder? I can do it myself, but I want to record your precious pull request. |
|
Yeah, gotcha! I will do @shin285. |
|
Yes I did. Always thanks for your contributions! |
|
One question here. |
|
Komoran using JDK8 features :) |
|
Thank you 👍 |
4 tasks
shin285
pushed a commit
that referenced
this pull request
Apr 3, 2020
shin285
added a commit
that referenced
this pull request
Apr 6, 2020
* initial commit * migrated from 9bow/DictAdmin * GitHub 이슈 템플릿 추가 * README 개선 * URL 수정 * README 개선 * KOMORAN 버전 수정 * 누락된 css/js 파일 추가 * 에러 메시지 출력 방법 개선 및 입력값 검증 추가 related #5 & #4 * Exception 메시지 변경 * 파일 업로드 시 문제 해결 방법 변경 related #2 * 에러 메시지 노출 * 테이블 이름 대소문자 구분 * 사용자 사전 관리 기능 추가 closes #6 * 저장소 경로를 변경합니다. * ID 생성 방법 통일 * 띄어쓰기 수정 * DicWord 테스트 정리 * DicUser 테스트 추가 * 예외 케이스 추가 * 오타 및 품사 표시 방법 수정 * .gitignore 업데이트 * grammar.in 관리기능 추가 * GrammarIn 테스트 코드 추가 * 메뉴 정리 * 메뉴 정리, favicon 변경 * 품사 빈도 관리(GrammarIn) 오류 수정 * 페이지 상단 사용법 설명 변경 * Analyze 기능 추가 closes #12 * KOMORAN 관리도구 설명 초안 추가 related #11 * 사용하지 않는 lib 정리 * 기분석 사전(fwd.user) 관리 기능 추가 related #7 * FULL 모델 제거 closes #16 * 보정 메뉴 내의 순서 변경 closes #17 * 모든 컬럼이 수정 가능하도록 변경 closes #20 * KOMORAN의 master 브랜치를 가져오도록 수정 * 사용자 모델 추가 related #14 * grammar.in 변경사항이 적용된 별도 KOMORAN.jar 관리 related #14 & #56 * 품사 빈도 파싱 규칙 수정 related #9 * 품사 변경 내용 적용 및 중복 품사 삭제한 기본 품사 빈도 파일 업데이트 related #9, #14 * 사용자 모델 생성 및 생성된 모델 목록 조회 기능 추가 related #14 * 사용자 모델 생성에 필요한 기본 사전 데이터 추가 로직 변경 related #14 * 형태소 분석 비교 메뉴 추가 closes #13 * 사용하지 않는 CSS 정리 * 오타 수정 * 사용하지 않는 코드 / 파일 제거 * 모델 관리 메뉴 추가 (모델 삭제 및 배포 기능) closes #15 #24 * 코드 정리 등 * 코드 정리 및 설명 수정 등 * 사용하지 않는 코드 정리 * 좌측 메뉴 숨김 시 테이블 다시 그리기 closes #23 * 오타 수정 * README 갱신 relate #27 * Apache 2.0 라이선스 적용 related #29 * 사용자 모델 목록 제공 시 유효성 검사 추가 related #28 * Log Level 변경 * Log Level 변경 * 사용자 모델 관련 테스트 추가 related #14 #15 #24 * FAQ 및 라이선스 추가 related #31 * 로그 레벨 변경 및 로컬 환경에서만 접근 가능하도록 수정 related #32 * 접근 포트를 8888에서 3579로 변경 related #33 * 명시적으로 인코딩 지정 related #18 #35 * FAQ 내용 추가 및 별도 문서로 분리 related #31 * Log Level 조정 related #37 (TODO: Profile 분리) * KOMORAN 실행 완료 안내 문구 추가 related #36 * 1.0 릴리즈를 위한 버전 변경 * 최신 버전의 KOMORAN을 사용하도록 변경 related #38 #18 #14 * 페이지 로딩 시 동작 일부 변경 * FAQ 내용 일부 수정 related #31 #34 * 분석 메뉴에 모델을 선택하여 분석할 수 있도록 기능 추가 related #42 * 사용자 모델에 대한 유효성 검사를 담당하는 컴포넌트를 새로 만들고, 해당 컴포넌트를 사용하여 유효성을 검사 related #39 * 잘못 삭제한 parseModelName() 함수 복원 * 오타 수정 및 모델 생성 시 기본 디렉토리 생성 코드 복구 * 사용자 모델의 유효성 검사시 corpus_build/ 디렉토리 및 하위 파일들에 대한 유효성 검사 추가 #39 - corpus_build/ 디렉토리 유효성 검사 추가 - corpus_build/ 디렉토리 하위의 dic.word, dic.irregular, grammar.in 파일 유효성 검사 추가 * 아래 메뉴들에 Enter 키를 이용한 Form Submit 기능 적용 #41 - 기본 형태소 분석 메뉴 (analyze.html) - 형태소 분석 비교 메뉴 (compare.html) * 띄어쓰기, 공백 정리 등 * 보정 메뉴들에 Enter 키를 이용한 Form Submit 기능 추가 related #41 * 사용자 모델 선택 변경 시에도 형태소 분석 진행 related #41 * KOMORAN 버전을 3.3.5로 지정 * 분석 메뉴를 문장 단위와 파일 단위로 분리 related #44, #45 * KOMORAN 3.3.5 이상 지원을 명시 * 여러 문장의 형태소 분석 지원 related #45 * init commit * implementation compare file page based on form * 파일 비교 페이지 개발 * Komoran project를 module로 변경 Co-authored-by: Junghwan Park <reserve.dev@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#32
@shin285
When importing a resource file in Java, that method assumes "/" seperated path.
On Windows, the code
File.separatoris '\' and will generate an invalid path.I modified some code and added Google's Guava library to concatenate String efficiently.
If you don't want to increase Jar file size by adding Guava, alternative method such as StringBuilder will be possible.
Feel free to give me a feedback. Thx