sigma67 / spotify_to_ytmusic

Clone a Spotify playlist to YouTube Music
MIT License
954 stars 72 forks source link

Improve importation progress text #70

Closed MLZ94 closed 1 year ago

MLZ94 commented 1 year ago

Much useful when it comes to big playlists, when updating infos every 10 tracks, sometimes it can feel like something is going wrong because it takes too long. This way we can precisely know the progress. It is also more pleasent when it's a single line updating rather than several lines on the terminal

sigma67 commented 1 year ago

I feel like this goes too far, you could simply remove the %10 in the if statement instead.

But I'd like to make this optional, for example add a --verbose option that enables detailed logging. Could even add more output:

YTMusic: Searching for <search term> (<x>/<total>)
MLZ94 commented 1 year ago

Actually the main idea was to make the text dynamicaly updating at the same line (like a progress bar) rather than printing new and new lines, so that doesn't go too far for that purpose. And yeah, it's a good idea to add more output, i'll probably give it a try But only removing the % 10 is not my idea, i wanted to make it feel more reactive and with less output lines (which is cleaner, one single progress line rather than many, could go very far for big playlist, the one i use has almost 90 musics)

sigma67 commented 1 year ago

Sorry, the flush will not be merged. The output shall be sequential