search

boramalper / himawaripy

Set near-realtime picture of Earth as your desktop background
http://labs.boramalper.org/himawaripy
MIT License
1.62k stars 239 forks source link

Option to disable offset #48

Closed p4block closed 7 years ago

p4block commented 8 years ago

I just want to get the latest available image, could it be possible to pass an option to the script so it ignores the offset?

This instead of having to edit the config file

boramalper commented 8 years ago

I want to keep it simple, so I'm not planning to add runtime parameters for now.

May I learn the reason of needing runtime parameters instead of config file?

p4block commented 8 years ago

I respect the decision on simplicity, but runtime parameters are a must imho

Yesterday I found that I was getting an offset picture of the Earth instead of the realtime one, so I tried running the software with -h and --help, but nothing happened. Some info on usage is somewhat expected.

A quick locate found the configuration file buried very deep inside the filesystem, and I edited it. Everything works fine and realtime now.

However, without runtime parameters support it is impossible to script the program in different ways. Two users can't have different configurations, a single user can't get different quality images on demand... who knows what use cases someone may find.

I think it would be useful at most and simple convention following at least.

However, the software works and it could be argued any fancy-anything is outside its scope. No problem with that, really.

This piece of software is great anyhow :+1:

boramalper commented 8 years ago

A quick locate found the configuration file buried very deep inside the filesystem

Two users can't have different configurations

I think the points you raised are fair enough, I'll try to add runtime parameters support.

Thanks for the suggestion! :)

boramalper commented 7 years ago

Done in v2. :)

Many many thanks!