-
Notifications
You must be signed in to change notification settings - Fork 502
[4.45.0] 하드웨어 PR #2920
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
[4.45.0] 하드웨어 PR #2920
Conversation
코코드론/entry/hardwarelite
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit
eslint
'time' is never reassigned. Use 'const' instead.
| let time = script.getNumberValue('TIME', script); |
'Buffer' is not defined.
| const commandBuffer = Buffer.from(command, 'utf8'); |
'time' is never reassigned. Use 'const' instead.
| let time = script.getNumberValue('TIME', script); |
'Buffer' is not defined.
| const commandBuffer = Buffer.from(command, 'utf8'); |
'time' is never reassigned. Use 'const' instead.
| let time = script.getNumberValue('TIME', script); |
'Buffer' is not defined.
| const commandBuffer = Buffer.from(command, 'utf8'); |
'time' is never reassigned. Use 'const' instead.
| let time = script.getNumberValue('TIME', script); |
'Buffer' is not defined.
| const commandBuffer = Buffer.from(command, 'utf8'); |
'Buffer' is not defined.
| const commandBuffer = Buffer.from( |
Unexpected string concatenation.
| encodeURIComponent(command) + '\r', |
'Buffer' is not defined.
| const commandBuffer = Buffer.from(command, 'utf8'); |
'Buffer' is not defined.
| const commandBuffer = Buffer.from(command, 'utf8'); |
Missing radix parameter.
| const value = parseInt(script.getNumberValue('VALUE', script)); |
Missing radix parameter.
| const time = parseInt(script.getNumberValue('TIME', script)); // ms |
Missing radix parameter.
| const turnValue = parseInt(script.getNumberValue('TURN_VALUE', script)); |
Missing radix parameter.
| const turnTime = parseInt(script.getNumberValue('TURN_TIME', script)); // ms |
'Buffer' is not defined.
| Buffer.from(firstCommand, 'utf8'), |
'Buffer' is not defined.
| Buffer.from(secondCommand, 'utf8'), |
This line has a length of 114. Maximum allowed is 100.
| '화면에 [게임/캐릭터]중 특정 아이콘을 표시할 위치와 크기를 설정\nX좌표: -160 ~ 160\nY좌표: -120 ~ 120\n크기: 0 ~ 200', |
This line has a length of 176. Maximum allowed is 100.
| 'Set the position and size of a specific icon in [game/character] on the screen\nX coordinate: -160 ~ 160\nY coordinate: -120 ~ 120\nSize: 0 ~ 200', |
🚫 [eslint] <prettier/prettier> reported by reviewdog 🐶
Delete ··
| robotis_icon_game_character_44: 'Running Person 2', |
'iconNum' is never reassigned. Use 'const' instead.
| let iconNum = script.getField('ICON', script); |
Identifier 'data_instruction' is not in camel case.
| let data_instruction = INST_WRITE; |
'data_instruction' is never reassigned. Use 'const' instead.
| let data_instruction = INST_WRITE; |
Identifier 'data_address' is not in camel case.
| let data_address = 166; |
'data_address' is never reassigned. Use 'const' instead.
| let data_address = 166; |
Identifier 'data_length' is not in camel case.
| let data_length = 2; |
'data_length' is never reassigned. Use 'const' instead.
| let data_length = 2; |
Identifier 'data_value' is not in camel case.
| let data_value = 10496; |
Identifier 'data_value' is not in camel case.
| data_value = iconNum; |
Identifier 'data_sendqueue' is not in camel case.
| let data_sendqueue = [ |
'data_sendqueue' is never reassigned. Use 'const' instead.
| let data_sendqueue = [ |
Identifier 'data_instruction' is not in camel case.
| [data_instruction, data_address, data_length, data_value], |
Identifier 'data_address' is not in camel case.
| [data_instruction, data_address, data_length, data_value], |
Identifier 'data_length' is not in camel case.
| [data_instruction, data_address, data_length, data_value], |
Identifier 'data_value' is not in camel case.
| [data_instruction, data_address, data_length, data_value], |
'packets' is never reassigned. Use 'const' instead.
| let packets = []; |
🚫 [eslint] <prettier/prettier> reported by reviewdog 🐶
Replace ABC with abc
| this.robotisBuffer.push([INST_WRITE, 1010, 2, 0xABC2]); // 아래 20번 모드 진입후 코알라 얼굴로 변경준비 |
This line has a length of 114. Maximum allowed is 100.
| '화면에 [게임/캐릭터]중 특정 아이콘을 표시할 위치와 크기를 설정\nX좌표: -160 ~ 160\nY좌표: -120 ~ 120\n크기: 0 ~ 200', |
This line has a length of 176. Maximum allowed is 100.
| 'Set the position and size of a specific icon in [game/character] on the screen\nX coordinate: -160 ~ 160\nY coordinate: -120 ~ 120\nSize: 0 ~ 200', |
🚫 [eslint] <prettier/prettier> reported by reviewdog 🐶
Delete ························
| robotis_icon_game_character_44: 'Running Person 2', |
'iconNum' is never reassigned. Use 'const' instead.
| let iconNum = script.getField('ICON', script); |
Identifier 'data_instruction' is not in camel case.
| let data_instruction = INST_WRITE; |
'data_instruction' is never reassigned. Use 'const' instead.
| let data_instruction = INST_WRITE; |
Identifier 'data_address' is not in camel case.
| let data_address = 166; |
'data_address' is never reassigned. Use 'const' instead.
| let data_address = 166; |
Identifier 'data_length' is not in camel case.
| let data_length = 2; |
'data_length' is never reassigned. Use 'const' instead.
| let data_length = 2; |
Identifier 'data_value' is not in camel case.
| let data_value = 10496; |
Identifier 'data_value' is not in camel case.
| data_value = iconNum; |
Identifier 'data_sendqueue' is not in camel case.
| let data_sendqueue = [ |
'data_sendqueue' is never reassigned. Use 'const' instead.
| let data_sendqueue = [ |
Identifier 'data_instruction' is not in camel case.
| [data_instruction, data_address, data_length, data_value], |
Identifier 'data_address' is not in camel case.
| [data_instruction, data_address, data_length, data_value], |
Identifier 'data_length' is not in camel case.
| [data_instruction, data_address, data_length, data_value], |
Identifier 'data_value' is not in camel case.
| [data_instruction, data_address, data_length, data_value], |
'packets' is never reassigned. Use 'const' instead.
| let packets = []; |
🚫 [eslint] <prettier/prettier> reported by reviewdog 🐶
Delete ················
🚫 [eslint] <prettier/prettier> reported by reviewdog 🐶
Replace ABC with abc
| this.robotisBuffer.push([INST_WRITE, 1010, 2, 0xABC1]); // 아래 20번 모드 진입후 꼭두 얼굴로 변경준비 |
| i--; | ||
| } | ||
|
|
||
| if (chunk === SOME_EXPECTED_RESPONSE) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| const queryString = []; | ||
| const readablePorts = this.readablePorts; | ||
|
|
||
| const commandStr = Entry.hw.sendQueue['COMMAND']; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
["COMMAND"] is better written in dot notation.
| const commandStr = Entry.hw.sendQueue['COMMAND']; | |
| const commandStr = Entry.hw.sendQueue.COMMAND; |
| for (let i = 0; i < commandStr.length; i++) { | ||
| queryString.push(commandStr.charCodeAt(i)); | ||
| } | ||
| delete Entry.hw.sendQueue['COMMAND']; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
["COMMAND"] is better written in dot notation.
| delete Entry.hw.sendQueue['COMMAND']; | |
| delete Entry.hw.sendQueue.COMMAND; |
| Cocodron_Dron_Controller_land: 'Landing', | ||
| Cocodron_Dron_Controller_emergency: 'Emergency Stop', | ||
| Cocodron_Dron_Controller_headless: 'Headless Mode', | ||
| //Cocodron_Dron_Controller_move_drone: 'Move joystick %1 in %2 direction by %3 for %4 ms', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| isNotFor: ['cocodroncontrollerLite'], | ||
| func(sprite, script) { | ||
| const command = 'mapping_start'; | ||
| const commandBuffer = Buffer.from( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| const command = `down ${stick} ${time}\r`; | ||
| console.log('Constructed command:', command); | ||
|
|
||
| const commandBuffer = Buffer.from(command, 'utf8'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| isNotFor: ['cocodroncontrollerLite'], | ||
| func(sprite, script) { | ||
| let stick = script.getNumberValue('STICK', script) * 10; | ||
| let time = script.getNumberValue('TIME', script); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'time' is never reassigned. Use 'const' instead.
| let time = script.getNumberValue('TIME', script); | |
| const time = script.getNumberValue('TIME', script); |
| } | ||
|
|
||
| const command = `cw ${stick} ${time}\r`; | ||
| const commandBuffer = Buffer.from(command, 'utf8'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| isNotFor: ['cocodroncontrollerLite'], | ||
| func(sprite, script) { | ||
| let stick = script.getNumberValue('STICK', script) * 10; | ||
| let time = script.getNumberValue('TIME', script); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'time' is never reassigned. Use 'const' instead.
| let time = script.getNumberValue('TIME', script); | |
| const time = script.getNumberValue('TIME', script); |
| } | ||
|
|
||
| const command = `ccw ${stick} ${time}\r`; | ||
| const commandBuffer = Buffer.from(command, 'utf8'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
entrylabs/entry-hw#807
신규기기 1종 추가
기존 기기 3종 수정
전체 4종 중 3종은 웹연결만 변경