Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

respect LOCATION_FILTER_THRESHOLD config, adjust sf_thresh range from 0.0001 to 0.99 #100

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

bartman081523
Copy link
Contributor

before:
-you set LOCATION_FILTER_THRESHOLD in config, and it was not respected, instead the default value of 0.03 was used.
-only a part of the species list could be adressed with sf_thresh (between 0.01 to 0.99)

after:
-first the config value is used, if the "--sf_thresh" is set, then this value is used, if no config value and no "--sf_thresh" value is set, the default (0.03) is used.
-the full species list can be adressed with sf_thresh (0.0001 to 0.99)
TODO: use a method of argparse, to see whether "--sf_thresh" is set.

before: 
you set LOCATION_FILTER_THRESHOLD in config, and it was not respected, instead the default value of 0.03 was used.

after:
first the config value is used, if the "--sf_thresh" is set, then this value is used, 
if no config value and no "--sf_thresh" value is set, the default (0.03) is used.

TODO: use a method of argparse, to see whether "--sf_thresh" is set.
only between 0.0001 and 0.99 could the full species list be "accessed" through sf_thresh
@bartman081523 bartman081523 changed the title respect location_filter_threshold config, adjust sf_thresh range respect LOCATION_FILTER_THRESHOLD config, adjust sf_thresh range to 0.0001 to 0.99 Apr 19, 2023
@bartman081523 bartman081523 changed the title respect LOCATION_FILTER_THRESHOLD config, adjust sf_thresh range to 0.0001 to 0.99 respect LOCATION_FILTER_THRESHOLD config, adjust sf_thresh range from 0.0001 to 0.99 Apr 19, 2023
@kahst
Copy link
Owner

kahst commented Apr 19, 2023

Ok, thanks. Makes sense. Two quick questions:

  • can you also please adjust the Readme with the new values?
  • why are changes in analyze.py different than in species.py? (if lat and lon are -1, no species filter will be used)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants