mjaakko / NeoStumbler

New stumbler application for contributing data to geolocation services
MIT License
127 stars 11 forks source link

More settings #209

Open solonovamax opened 2 months ago

solonovamax commented 2 months ago

Description

It would be nice if additional settings were offered, past just "send reports automatically" and "ignore wifi scan throttling"

here are some suggestions for settings (some of these were influenced by TowerCollector):

may think of additional ones later. will update if I do.

Suggested implementation

No response

Additional information

No response

New request

mjaakko commented 2 months ago

Could you split this into multiple issues? That way it would be easier to keep track of progress and to have discussions, thanks!

In general I'd prefer to avoid having too many settings and instead have sensible defaults. It becomes kind of pain to test all permutations when there's a lot of settings. But if there's clear benefit of making something configurable, it's not a problem

solonovamax commented 2 months ago

Could you split this into multiple issues? That way it would be easier to keep track of progress and to have discussions, thanks!

I made a master issue as several of these already have their own issues, and also because I didn't want to create a ton of issues on the repo as it's kinda spammy.

In general I'd prefer to avoid having too many settings and instead have sensible defaults. It becomes kind of pain to test all permutations when there's a lot of settings. But if there's clear benefit of making something configurable, it's not a problem

I do agree that sensible defaults are good, however many of these options wouldn't require testing:

also, for many of the cases where "sensible defaults" might apply, I think there are very good justifications for wanting it to be configurable.