-
Notifications
You must be signed in to change notification settings - Fork 265
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
그림퀴즈 PR #326
base: master
Are you sure you want to change the base?
그림퀴즈 PR #326
Conversation
그림퀴즈 소스에서 파일 경로 관련 오류발견 |
실수로 PC버전의 CSS가 누락되었던것같습니다. |
시간이 초과되면 오류를 뿜으며 서버가 재부팅됩니다. [2019-01-09 22:29:59] ERROR OCCURRED ON THE MASTER! :8496 [2019-1-9 22:29:59] ERROR: TypeError: Cannot read property 'NaN' of undefined TypeError: Cannot read property 'NaN' of undefined at exports.Room.turnHint (C:\Users\user\Desktop\Private\KKuTu-master\Server\lib\Game\games\drawing.js:113:21) at Timeout._onTimeout (C:\Users\user\Desktop\Private\KKuTu-master\Server\lib\Game\games\drawing.js:100:55) at ontimeout (timers.js:436:11) at tryOnTimeout (timers.js:300:5) at listOnTimeout (timers.js:263:5) at Timer.processTimers (timers.js:223:10) |
이 소스를 사용해도 되나요? |
버그가 있을 수 있습니다만 저희 서버에서는 사용합니다. |
1. 플레이어가 연습 모드로 플레이 도중 중도 퇴장 시 힌트 타이머가 멈추지 않아 발생한 오류 수정 2. "짧음" 특수 규칙 추가 3. 힌트 타이머 간격이 비정상적으로 짧았던 점 수정
그림퀴즈에서 나오는 단어의 길이를 최대 4글자로 조절하는 규칙입니다.
그림퀴즈에서 나오는 단어의 길이를 최대 4글자로 조절하는 규칙입니다.
그림퀴즈에서 나오는 단어의 길이를 최대 4글자로 조절하는 규칙입니다.
@lshqqytiger 해당 오류를 인지하였고, 수정 절차에 들어갔습니다. @dalbodeule님 레프지토리 해당 Branch에 이와 기타 잡것을 수정하는 Pull Request를 넣어 놓았으니, 참고하시면 좋겠네요 :D |
그림퀴즈 소스 내 일부 사항 수정
- add image data diff on DrawQuiz - server and client will send/recive diff data - if diff data not valid, server and client will request/recive full data - and fix log system (if data not json, crash server)
현재 작동 여부가 검증되지 않은 최적화 패치가 Push 되었습니다. 중요: 일부 파일이 Conflict를 일으키고 있습니다. |
이번 그림퀴즈 최적화 패치는 diff를 이용하여 그림 데이터 중 바뀐 부분만 서버와 클라이언트 간 전달하도록 만들었습니다. 그림 데이터 중 바뀐 부분만 전달하도록 하여 데이터 통신량을 줄일 수 있도록 하였습니다. 데이터의 유실에 대비하여 서버나 플레이어가 요청할 경우 원래 데이터를 요청/전달할 수 있게 되어있습니다. |
제가 수정을 잘못해서 일수도 있겠지만 video.mp4 |
문제가 없는 것 같습니다. |
kkutu.pug코드는 저번에 수정할때 같이 추가했습니다. 그런데 저런 오류가 납니다. |
diff-match-patch-compressed.js 파일이 제대로 로드되고 있는지 Network 탭에서 확인해보세요. |
오류가 뜨면서 게임 서버 실행이 안됩니다 internal/modules/cjs/loader.js:818 Error: Cannot find module 'diff-match-patch'
|
package.json에 변경 사항이 있는 PR의 경우 |
|
이런 오류가 뜹니다 [2024-01-12 21:30:45] ERROR OCCURRED! This worker will die in 10 seconds. :8496 [2024-1-12 9:30:45 ├F10: PM┤] ERROR: Error: Cannot find module '../const' Require stack: - C:\Users\모자이크\Desktop\KKuTu-master\KKuTu-master\Server\lib\Game\games\drawing.js - C:\Users\모자이크\Desktop\KKuTu-master\KKuTu-master\Server\lib\Game\kkutu.js - C:\Users\모자이크\Desktop\KKuTu-master\KKuTu-master\Server\lib\Game\master.js - C:\Users\모자이크\Desktop\KKuTu-master\KKuTu-master\Server\lib\Game\slave.js - C:\Users\모자이크\Desktop\KKuTu-master\KKuTu-master\Server\lib\Game\cluster.js Error: Cannot find module '../const' Require stack: - C:\Users\모자이크\Desktop\KKuTu-master\KKuTu-master\Server\lib\Game\games\drawing.js - C:\Users\모자이크\Desktop\KKuTu-master\KKuTu-master\Server\lib\Game\kkutu.js - C:\Users\모자이크\Desktop\KKuTu-master\KKuTu-master\Server\lib\Game\master.js - C:\Users\모자이크\Desktop\KKuTu-master\KKuTu-master\Server\lib\Game\slave.js - C:\Users\모자이크\Desktop\KKuTu-master\KKuTu-master\Server\lib\Game\cluster.js at Function.Module._resolveFilename (internal/modules/cjs/loader.js:815:15) at Function.Module._load (internal/modules/cjs/loader.js:667:27) at Module.require (internal/modules/cjs/loader.js:887:19) at require (internal/modules/cjs/helpers.js:74:18) at Object. (C:\Users\모자이크\Desktop\KKuTu-master\KKuTu-master\Server\lib\Game\games\drawing.js:25:15) at Module._compile (internal/modules/cjs/loader.js:999:30) at Object.Module._extensions..js (internal/modules/cjs/loader.js:1027:10) at Module.load (internal/modules/cjs/loader.js:863:32) at Function.Module._load (internal/modules/cjs/loader.js:708:14) at Module.require (internal/modules/cjs/loader.js:887:19) |
|
/** 라고 써있습니다 |
저는 부근이라고 말씀드렸습니다. 그리고 19번째 줄에 |
/**
const Const = require('../const'); exports.init = function(_DB, _DIC){
|
최근 commit까지 전부 병합하세요. 참고 |
internal/modules/cjs/loader.js:818 throw err; ^ Error: Cannot find module 'diff-match-patch' Require stack: - C:\Users\모자이크\Desktop\KKuTu-master\KKuTu-master\Server\lib\Game\kkutu.js - C:\Users\모자이크\Desktop\KKuTu-master\KKuTu-master\Server\lib\Game\master.js - C:\Users\모자이크\Desktop\KKuTu-master\KKuTu-master\Server\lib\Game\cluster.js at Function.Module._resolveFilename (internal/modules/cjs/loader.js:815:15) at Function.Module._load (internal/modules/cjs/loader.js:667:27) at Module.require (internal/modules/cjs/loader.js:887:19) at require (internal/modules/cjs/helpers.js:74:18) at Object.<anonymous> (C:\Users\모자이크\Desktop\KKuTu-master\KKuTu-master\Server\lib\Game\kkutu.js:24:24) at Module._compile (internal/modules/cjs/loader.js:999:30) at Object.Module._extensions..js (internal/modules/cjs/loader.js:1027:10) at Module.load (internal/modules/cjs/loader.js:863:32) at Function.Module._load (internal/modules/cjs/loader.js:708:14) at Module.require (internal/modules/cjs/loader.js:887:19) { code: 'MODULE_NOT_FOUND', requireStack: [ 'C:\Users\모자이크\Desktop\KKuTu-master\KKuTu-master\Server\lib\Game\kkutu.js', 'C:\Users\모자이크\Desktop\KKuTu-master\KKuTu-master\Server\lib\Game\master.js', 'C:\Users\모자이크\Desktop\KKuTu-master\KKuTu-master\Server\lib\Game\cluster.js' ] }
|
바로 위를 보세요. |
그 죄송하지만 cmd에 npm install를 쓰는건가요? |
그 너무 질문을 많이하는것같아 죄송합니다.. npm 설치 후 npm의 에러는 수정되었지만 [2024-01-13 11:23:16] ERROR OCCURRED! This worker will die in 10 seconds. :8496 [2024-1-13 11:23:16 ├F10: AM┤] ERROR: Error: Unknown authenticationOk message typeMessage { name: 'authenticationOk', length: 23 } Error: Unknown authenticationOk message typeMessage { name: 'authenticationOk', length: 23 } at Connection.parseR (C:\Users\chus6\Desktop\KKuTu-master\KKuTu-master\Server\lib\node_modules\pg\lib\connection.js:454:9) at Connection.parseMessage (C:\Users\chus6\Desktop\KKuTu-master\KKuTu-master\Server\lib\node_modules\pg\lib\connection.js:370:17) at Socket.<anonymous> (C:\Users\chus6\Desktop\KKuTu-master\KKuTu-master\Server\lib\node_modules\pg\lib\connection.js:129:22) at Socket.emit (events.js:314:20) at addChunk (_stream_readable.js:297:12) at readableAddChunk (_stream_readable.js:272:9) at Socket.Readable.push (_stream_readable.js:213:10) at TCP.onStreamRead (internal/stream_base_commons.js:188:23) 라는 에러가 뜨며 그냥 서버랑 워커가 죽습니다.. connection.js 어디에 문제가 있는지 모르겠습니다. |
그.. 아예 이젠 아무것도 안돼서 처음부터 다시하는중입니다... 너무 질문 많이한점 죄송합니다. |
[2024-01-20 10:15:02] ERROR OCCURRED ON THE MASTER! 서버는 작동하지만 에러가 뜨며 채널에 들어가지지 않습니다. |
PR 병합이 힘들다면 PR 병합 대신 병합 원본 레포지토리를 포킹하는것이 권장됩니다. |
그럼 지금 저상태로는 해결하기 조금 복잡한건가요.. |
양쪽 코드가 제대로 병합이 안된것 같습니다. 어느 한쪽의 코드만 사용하는 경우에는 문제 없이 동작합니다. |
전 PR을 내려받아서 스크립트를 합치는 방법을 사용했습니다. |
포킹 후 PR을 바로 진행하면 별도로 합치는 작업이 없습니다. 작업 방식 자체가 잘못되었을 가능성이 높습니다.
|
master.js:351:19 를 수정하면 해결될수도 있는건가요 |
소스코드를 내려받아서 합치는 행위는 적절하지 못합니다. 아울러 Git 사용 방법이나 끄투 코드에 대한 이해가 전무하신 것 같은데, 적절한 개발 실력을 키우고 오시길 부탁드립니다. |
그러면 어떻게 하면 되나요 |
이미 방법을 말씀드렸습니다. |
제가 답변을 너무 어렵게 한 것 같네요. 또한 간단한 질문은 ChatGPT로 부탁드립니다 소스코드 병합 방법새로운 원격(remote) 추가: git remote add 다른포크 https://github.com/다른계정/다른레포.git git fetch 다른포크 git merge 다른포크/브랜치이름 git rebase 다른포크/브랜치이름 |
보니까 별거 아니였네요 너무 질문 많이해서 죄송합니다 |
이전에 모리끄투 등 일부 서버에서 사용중인 그림퀴즈를 공개합니다.
끄투코리아의 그림퀴즈 소스와는 완전 다른 소스입니다.