search

JVT038 / MetaTube

Python web ui to download videos from YouTube and add metadata on top of it
GNU General Public License v3.0
309 stars 20 forks source link

Qol bugs #73

Open lasersPew opened 1 year ago

lasersPew commented 1 year ago
  1. Even when I edited the metadata manually, it still asks me to pick one from the list that it searched from musicbrainz, etc even though there's literally no results.
  2. It's slow. is there no concurrency in this project?
  3. When I recreate the Docker container or just restart it, I get this error: 
    ERROR [flask_migrate] Error: Can't locate revision identified by '8046105cb767'
INFO  [alembic.runtime.migration] Context impl SQLiteImpl.
INFO  [alembic.runtime.migration] Will assume non-transactional DDL.

I can resolve it by moving the app.db out of the folder and letting it recreate the database file then moving it back but now, I can't do it.

lasersPew commented 1 year ago

About #2, You can practically make a api call to sponsorblock while it scrapes the data from youtube and after it get the youtube data, call musicbrainz, spotify, and deezer api.

On #1, you can also just add a place where users can add link to either musicbrainz, spotify, or deezer and scrape and use the data.

Tbh, I am irritated that I have to troubleshoot and fix this everyday for days

JVT038 commented 1 year ago

Yeah, you're right, this project should be partly rewritten as it is kind of a mess. I don't know if I can find the time and/or motivation to do this though.

JVT038 commented 1 year ago

Error 3 has been fixed. Check issue #79 for the fix.

JVT038 commented 1 year ago

I'm sorry for the slow / inactive development. I'm currently considering rewriting this entire project in Django instead of Flask. If I do this, the program should hopefully run better and more efficient. I have decided that I'll only fix the very urgent / breaking bugs (such as a bug where the program is unable to start properly). QoL bugs are something that I'm honestly unsure about. If you want to make a PR to fix it, feel free to do so.

lasersPew commented 1 year ago

To add to this, maybe not use bootstrap as it doesn't look good, even at mobile. maybe the responsive material design would do the trick? Also, An API for media players and other clientsto use as this may be used alongside selfhosted media players.

JVT038 commented 6 months ago

So after almost a year, I've started with implementing concurrency :D. Pull the latest image and hopefully it works faster now.