fix diffusers undefined REFINER_ID

[biswaroop1547]

fixes #71

[casperdcl]

I think this is the only change required, the rest can be reverted? i.e. pass empty string if not defined :)

Suggested change

	RUN python3 download.py --model $MODEL_ID
	RUN python3 download.py --model $MODEL_ID --refiner "$REFINER_ID"

[biswaroop1547]

@casperdcl then for this image build REFINER_ID won't be used correctly by download.py to download refiner only when given. Whereas it'll work for all the other image builds.

[biswaroop1547]

oh wait I totally missed that you've put REFINER_ID inside quotes, my bad! damn this totally works then

[casperdcl]

just pushed a commit (might make it clearer what I mean) - feel free to revert it if it doesn't make sense or doesn't work though. Basically:

• no --build-arg
  • Dockerfile executes download.py --refiner ""
    • argparse parses into args.refiner = "" (Falsey)
• using --build-arg="REFINER_ID=..."
  • Dockerfile executes download.py --refiner "..."
    • argparse parses into args.refiner = "..."

[biswaroop1547]

thanks for the explanation! this makes sense, just tried locally and seems to be working nicely.