ctengel / linkmeddle

LINKed MEDia DL
GNU General Public License v3.0
0 stars 0 forks source link

post download json serialization issue #73

Closed ctengel closed 2 years ago

ctengel commented 2 years ago
[2022-06-06 14:07:44,407: WARNING/ForkPoolWorker-1] [download] Finished downloading playlist: xyz - Videos
[2022-06-06 14:07:44,552: ERROR/ForkPoolWorker-1] Task tasks.download[abc] raised unexpected: TypeError('Object of type FFmpegMergerPP is not JSON serializable')
  File "/home/linkmeddle/linkmeddle/apiqueue/tasks.py", line 78, in download
    json.dump(res, filehand)

TypeError: Object of type FFmpegMergerPP is not JSON serializable

Likely result of #63 - returning some object instead of a dict

ctengel commented 2 years ago

issue seems to be that the info_dict has a __postprocessors list which this class of object may belong to. There is a _copy_infodict function which pops it off, but maybe not getting run here?

ctengel commented 2 years ago

Been trying to eliminate the offending object so none of the JSON serializations fail.

New approach: use an idea from stackoverflow to take care of it in the first place, then analyze the resulting JSON to figure out how to eliminate.

Better idea, open an issue with yt-dlp?

ctengel commented 2 years ago

It is deep in there

ctengel commented 2 years ago

result => entries => # => requested_downloads => # => __postprocessors => #