-
Notifications
You must be signed in to change notification settings - Fork 0
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
Improve the vespa management page #63
Comments
Example below: Different linetype for nest and individual (all provinces) |
Note: The blur in active hotbeds is based on the individual observations only (not untreated nests), see here. This is not explicit from the text above the graph:
@SanderDevisscher So, I would either update the code or the text. |
@SanderDevisscher This info is missing for the nests coming from (iNaturalist/Natuurpunt) this data source: |
hmm, It is indeed correct management data is unavailable for the gbif sourced data while nests from iasset (nests.geojson) do have a management data. In the future all nests from natuurpunt will be uploaded automatically into iasset (currently in the pipeline). Changing to iAsset data only when the pipeline goes live will then result in us missing the small number of nests reported via iNaturalist (used to be more in past when we piggybacked on iNaturalist for management data). Therefor I think it is best to, leave it as is for now, but write code to be able to switch to only displaying nests from the nests.geojson only from a certain date1 (when the Natuurpunt <-> iAssset pipeline goes live) onwards. if this is possible ? 1nests reported after this date should be displayed from iasset data only. Nests reported after this date from the gbif sourced data should no longer be displayed. |
Its to cluttered as is! I would either drop the individuals or add like a unit dropdown to choose either. @timadriaens what do you think ? |
Can you add a management succes graph, like this: It should be based on example code:
|
Note: Date currently set at 31/12/2028, see here |
The graph is included. |
@SanderDevisscher to be added to translations file
|
Suggestions for improvement
Originally posted by @SanderDevisscher in #27 (comment)
The text was updated successfully, but these errors were encountered: