gavinbenda / platinum-md

Minidisc NetMD Conversion and Upload
MIT License
230 stars 38 forks source link

Tracks with Japanese characters are converted to ATRAC, but not transferred. #139

Open eastraindesign opened 2 years ago

eastraindesign commented 2 years ago

If Japanese characters (presumably other non-ASCII as well?) exist in either the title or artist, the track converts, but doesn't start the transfer stage. Appears to possibly be a file naming issue during the ATRAC conversion?

Contents of Temppmd-temp:

01 Tweet! Tweet! スパゲッティなう!! (2012).at3
01 Tweet! Tweet! スパゲッティなう!! (2012).raw.wav
Tweet! Tweet! スパゲッティなう!! - Beat Charger.wav

Debug Log:

Processing track: 0
renderer.js:56 Directory createdC:\Users\defor\AppData\Local\Temppmd-temp
renderer.js:56 D:\Music - Old\FLAC\Beat Charger - Tweet! Tweet! スパゲッティなう!! (2012) [MP3]\01 Tweet! Tweet! スパゲッティなう!! (2012).mp3
renderer.js:56 Starting conversion in <LP2> mode
renderer.js:56 Starting WAV conversion process using ffmpeg: D:\Music - Old\FLAC\Beat Charger - Tweet! Tweet! スパゲッティなう!! (2012) [MP3]\01 Tweet! Tweet! スパゲッティなう!! (2012).mp3 --> C:\Users\defor\AppData\Local\Temppmd-temp\01 Tweet! Tweet! スパゲッティなう!! (2012).raw.wav
renderer.js:56 Spawned Ffmpeg with command:  ffmpeg -i D:\Music - Old\FLAC\Beat Charger - Tweet! Tweet! スパゲッティなう!! (2012) [MP3]\01 Tweet! Tweet! スパゲッティなう!! (2012).mp3 -y -ar 44100 -acodec pcm_s16le C:\Users\defor\AppData\Local\Temppmd-temp\01 Tweet! Tweet! スパゲッティなう!! (2012).raw.wav
renderer.js:56 Processing: 00:03:47.36 done 39168 kilobytes
renderer.js:56 progress Converting Track
renderer.js:56 ffmpeg completed successfully
renderer.js:56 progress Idle
renderer.js:56 progress Converting to ATRAC
renderer.js:56 Converting to atrac
renderer.js:56 131072 132300
WARNING: ATRAC3 is uncompleted, result will be not good )))
bitrate 132300

renderer.js:56 progress Converting to ATRAC
renderer.js:56 \  0% done
progress Converting to ATRAC
renderer.js:56 /  99% done
renderer.js:56 progress Converting to ATRAC
renderer.js:56 
Done

renderer.js:56 progress Converting to ATRAC
renderer.js:56 atracdenc returned Success code 0
renderer.js:56 progress Adding Wav Wrapper
renderer.js:56 Processing: 00:03:47.35 done 3672 kilobytes
renderer.js:56 ffmpeg completed successfully
renderer.js:56 Title: Tweet! Tweet! スパゲッティなう!! - Beat Charger
renderer.js:56 Conversion Complete: C:\Users\defor\AppData\Local\Temppmd-temp\Tweet! Tweet! スパゲッティなう!! - Beat Charger.wav
renderer.js:56 progress Sending to Player
renderer.js:56 Attempting to send to NetMD device
renderer.js:56 audio file too small (corrupt or not supported)

renderer.js:56 netmdcli error, returned 1
renderer.js:56 progress Idle
renderer.js:56 undefined
renderer.js:56 Attempt to send file failed, retrying...
renderer.js:56 audio file too small (corrupt or not supported)

renderer.js:56 netmdcli error, returned 1
renderer.js:56 progress Idle
renderer.js:56 undefined
renderer.js:56 Attempt to send file failed, retrying...
renderer.js:56 audio file too small (corrupt or not supported)

renderer.js:56 netmdcli error, returned 1
renderer.js:56 progress Idle
renderer.js:56 undefined
renderer.js:56 Attempt to send file failed, retrying...
renderer.js:56 audio file too small (corrupt or not supported)

renderer.js:56 netmdcli error, returned 1
renderer.js:56 progress Idle
renderer.js:56 undefined
renderer.js:56 Attempt to send file failed, retrying...
renderer.js:56 audio file too small (corrupt or not supported)

renderer.js:56 netmdcli error, returned 1
renderer.js:56 progress Idle
renderer.js:56 undefined
renderer.js:56 Attempt to send file failed, retrying...
renderer.js:56 transfer-completed
renderer.js:56 {eventType: "transfer-completed", isBusy: false}eventType: "transfer-completed"isBusy: false__proto__: constructor: ƒ Object()hasOwnProperty: ƒ hasOwnProperty()isPrototypeOf: ƒ isPrototypeOf()propertyIsEnumerable: ƒ propertyIsEnumerable()toLocaleString: ƒ toLocaleString()toString: ƒ toString()valueOf: ƒ valueOf()__defineGetter__: ƒ __defineGetter__()__defineSetter__: ƒ __defineSetter__()__lookupGetter__: ƒ __lookupGetter__()__lookupSetter__: ƒ __lookupSetter__()get __proto__: ƒ __proto__()set __proto__: ƒ __proto__()
renderer.js:56 Deleting:
 C:\Users\defor\AppData\Local\Temppmd-temp
gavinbenda commented 2 years ago

I think this may be an issue with linux-minidisc not having support, I know some work has been done in the past on this, and it wasn't quite as straight forward as it first looked. https://github.com/linux-minidisc/linux-minidisc/pull/56

gavinbenda commented 2 years ago

Also just noticed the path - C:\Users\defor\AppData\Local\Temppmd-temp - there may be a similar issue to the one that just got fixed for Linux, if so, will push a new build.

mullaliui commented 1 year ago

I am having the same issue on newest build on windows 10