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
- 원래는 테스크 큐가 비어 있으면 시뮬레이션을 종료로 침 - 이렇게 되니 아직 시뮬이 돌고있는데 테스크 큐가 비는 상황이 생겨 이 사이에 새 요청을 넣을 수 있고 새 요청은 제대로 처리 못하는 상황이 생김 - 따라서 테스크 큐가 비고 모든 워커가 쉬고 있음을 확인해야 시뮬레이션 종료로 처리
dev2820
commented
Nov 21, 2023
Comment on lines
+22
to
+31
| build: { | ||
| rollupOptions: { | ||
| output: { | ||
| inlineDynamicImports: true, | ||
| }, | ||
| }, | ||
| }, | ||
| worker: { | ||
| format: 'es', | ||
| }, |
There was a problem hiding this comment.
이건 웹 워커를 사용하는 경우 셋업해줘야하는 vite 설정인데 우선은 무시해주세요. 저도 자세히 몰라서
현재 Build를 실패하는 이슈가 있어서 설정이 제대로 동작하는지 확인을 못하고 있습니다.
mahwin
reviewed
Nov 21, 2023
Comment on lines
+56
to
+60
| const isRequestSuccess = evaluator.evaluate(tasks); | ||
|
|
||
| if (!isRequestSuccess) { | ||
| return; | ||
| } |
There was a problem hiding this comment.
isXXX : boolean;
if(!isXXX) return ;
이렇게 하니 가독성 좋네요
mahwin
reviewed
Nov 21, 2023
Comment on lines
+26
to
+31
| function cancelEvaluation() { | ||
| evalWorkers.forEach(({ worker }) => worker.terminate()); | ||
| evalManager.cancelTasks(); | ||
|
|
||
| const newEvalWorkers = range(0, TOTAL_WORKERS).map(createEvaluator); | ||
| evalManager.setNewWorkers(newEvalWorkers); |
There was a problem hiding this comment.
네이밍을 잘 하니, 이 코드만 봐도 어떤 일들을 할 지 예상이 가네요. 진짜 좋습니다.
그리고, 워커는 재활용이 아닌가보네요?!
There was a problem hiding this comment.
네, while(true)로 한 스레드가 먹통이 되어버리면 그 스레드를 통째로 드러내야해서 cancel하면 워커를 종료시키고 새로 등록합니다.
좀 더 정밀하게 하자면 쉬고 있는 워커는 놔두고 일하고 있는 워커만 새로 만드는 방법도 있는데, 그건 그냥 귀찮아서... :)
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.
한 일
스크린 샷
변경된 테스트 입력 창
입력창과 입력 결과가 나오는 화면이 분리된 것을 볼 수 있습니다.
테스트 실행 버튼이 아래로 이동했습니다.
무한 루프를 시행한 경우
테스트 코드를 실행하면 테스트 실행이 "실행 취소" 로 변경됩니다.
실행을 취소하면 결과에 "실행 중단" 이라는 문구가 나타납니다.
이를 통해 테스트 수행 중 테스트를 다시 실행할 수 없게 되고, 테스트를 취소할 수 있게 되었습니다.
참조