lennart-rth / Live-Earth-Wallpapers

A collection of all earth related space Images in one script to set as your Desktop background.
GNU General Public License v3.0
300 stars 13 forks source link

Sentinel ignores supplied resolution #12

Closed SteffenCucos closed 1 year ago

SteffenCucos commented 1 year ago

Describe the bug Sentinel code does not use supplied resolution when loading images from url

To Reproduce Steps to reproduce the behavior:

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

Expected behavior We should pass the args.width/height to the sentinel satellite url

Screenshots

def calcImageDimensions(args):
    pixelWidth = 1920
    if args.width is not None and args.height is not None:
        pixelHeight = pixelWidth / (args.width/args.height)
    else:
        pixelHeight = pixelWidth / (16/9)

    return int(pixelWidth), int(pixelHeight) # ALWAYS RETURNS 1920 WIDTH

Desktop (please complete the following information):

Smartphone (please complete the following information):

Additional context Add any other context about the problem here.

lennart-rth commented 1 year ago

This is done. By specifying -w -he. You can also download custom image sizes for different monitors now