Fix 'try again' logics

baku89 · Oct 11, 2023 · 5e9f2b4 · 5e9f2b4
1 parent 0a669aa
commit 5e9f2b4
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/src/PTPDevice.ts b/src/PTPDevice.ts
@@ -17,7 +17,7 @@ enum PTPType {
 const PTPCommandMaxByteLength = 12 + 4 * 3
 const PTPDefaultTimeoutMs = 10000
 const PTPTryCount = 30
-const PTPTryAgainIntervalMs = 500
+const PTPTryAgainIntervalMs = 100
 
 interface PTPSendCommandOption {
 	label?: string
@@ -233,7 +233,7 @@ export class PTPDevice extends EventEmitter<EventTypes> {
 				res.code === ResCode.DeviceBusy
 
 			if (tryAgain) {
-				sleep(PTPTryAgainIntervalMs)
+				await sleep(PTPTryAgainIntervalMs)
 				continue
 			}
 
@@ -283,7 +283,7 @@ export class PTPDevice extends EventEmitter<EventTypes> {
 				res.code === ResCode.DeviceBusy
 
 			if (tryAgain) {
-				sleep(PTPTryAgainIntervalMs)
+				await sleep(PTPTryAgainIntervalMs)
 				continue
 			}
 
@@ -347,7 +347,7 @@ export class PTPDevice extends EventEmitter<EventTypes> {
 				res2.code === ResCode.DeviceBusy
 
 			if (tryAgain) {
-				sleep(PTPTryAgainIntervalMs)
+				await sleep(PTPTryAgainIntervalMs)
 				continue
 			}
 

diff --git a/src/Tethr.ts b/src/Tethr.ts
@@ -151,10 +151,11 @@ export abstract class Tethr
 
 		for (const [name, value] of sortedConfigs) {
 			// NOTE: this might be converted to parallel execution in the future
+			console.log('setting', name, value)
 			await this.set(name, value)
 
 			// The delay is necessary to avoid "busy" error
-			await sleep(50)
+			await sleep(25)
 		}
 	}