nadimkobeissi / mkbsd

Download all the wallpapers in MKBHD's "Panels" app
Do What The F*ck You Want To Public License
3.64k stars 459 forks source link

Request to Rename files the downloaded files as orginal file names instead of number #55

Open rsaurav12 opened 2 weeks ago

rsaurav12 commented 2 weeks ago

Rename file to original file name instead of numbers

"Of~Lux.jpg" https://storage.googleapis.com/panels-api/data/20240916/media-1a-i-p~s#:~:text=content/a~justinmaller_81d8962f/-,Of~Lux.jpg,-%3Fcrop%3Dfaces%252Ccenter

"Airport.jpg" https://storage.googleapis.com/panels-api/data/20240916/media-1a-i-p~s#:~:text=content/a~justinmaller_8267e364/-,Airport.jpg,-%3Fexpires%3D1728508444399%26fm

retardgerman commented 2 weeks ago

I opened a PR #57!:D Should fix that.

rsaurav12 commented 2 weeks ago

Hi, i want to know it fixed?

retardgerman commented 2 weeks ago

Hi, i want to know it fixed?

You may use my fork as long you are waiting for approval of my PR :)

rsaurav12 commented 2 weeks ago

U did a awesome job! But i'm facing some issues:

  1. I think some files are missing total number of images extracted from ur repo are 377 and in the following repo total images are 380: https://github.com/anthonywww/mksld

  2. Some files are not renamed properly For eg. Saved image to /content/mkbsd/downloads/vasjenkatro/08.png 🔍 Found image URL! 🖼️ Saved image to /content/mkbsd/downloads/vasjenkatro/03.png 🔍 Found image URL! 🖼️ Saved image to /content/mkbsd/downloads/vasjenkatro/10.png 🔍 Found image URL! 🖼️ Saved image to /content/mkbsd/downloads/vasjenkatro/06.png 🔍 Found image URL! 🖼️ Saved image to /content/mkbsd/downloads/vasjenkatro/11.png 🔍 Found image URL! 🖼️ Saved image to /content/mkbsd/downloads/vasjenkatro/07.png 🔍 Found image URL!

retardgerman commented 2 weeks ago

Thank you!

I'll check the scripts again to see what the problem is. Are you running the Python or the JS?

With the naming I'm actually not sure, if those are the actual names and that's why they are named eg. "1.png". But let me check that. Thanks for your feedback.

inuzen commented 2 weeks ago

@rsaurav12 you can check my issue here https://github.com/nadimkobeissi/mkbsd/issues/46 might be what you are looking for

rsaurav12 commented 2 weeks ago

Thank you!

I'll check the scripts again to see what the problem is. Are you running the Python or the JS?

With the naming I'm actually not sure, if those are the actual names and that's why they are named eg. "1.png". But let me check that. Thanks for your feedback.

Your welcome and i'm running python on google collab

rsaurav12 commented 2 weeks ago

@rsaurav12 you can check my issue here #46 might be what you are looking for

Can you create a python script pls

retardgerman commented 2 weeks ago

Thank you! I'll check the scripts again to see what the problem is. Are you running the Python or the JS? With the naming I'm actually not sure, if those are the actual names and that's why they are named eg. "1.png". But let me check that. Thanks for your feedback.

Your welcome and i'm running python on google collab

So I've just checked again. The images with the digits in their names are actually called like that in the source.

As for the number of images, it depends on what data you pull from the json. In my script, only the images marked dhd do get downloaded. If you want to load others, you would have to change the script, but I've not tested that yet.

rsaurav12 commented 2 weeks ago

Thank you! I'll check the scripts again to see what the problem is. Are you running the Python or the JS? With the naming I'm actually not sure, if those are the actual names and that's why they are named eg. "1.png". But let me check that. Thanks for your feedback.

Your welcome and i'm running python on google collab

So I've just checked again. The images with the digits in their names are actually called like that in the source.

As for the number of images, it depends on what data you pull from the json. In my script, only the images marked dhd do get downloaded. If you want to load others, you would have to change the script, but I've not tested that yet.

Yeah i understood , but acc. to this repo : https://github.com/anthonywww/mksld This repo extract only dhd images , u can check its python script also .. idk why it's pulling more images