AtlasOfLivingAustralia / alerts

Alerts services
https://alerts.ala.org.au
Other
1 stars 7 forks source link

Alert failure- invasive species unit list dr27353 #246

Closed turley85 closed 1 month ago

turley85 commented 4 months ago

I think the alert for the NSW Invasive Species unit (BioSecurity alert for NSW_NPWS_InvasiveSpeciesUnit_list) is failing. In testing, it's taking many minutes to load a preview, then showing zero detections: image

But R alerts are detecting occurrences during this same time period. image

As such, I suspect the alert itself is failiing/timing out.

turley85 commented 4 months ago

Note, the shapefile for this alert is quite complex: https://spatial.ala.org.au/?pid=9433211

turley85 commented 4 months ago

I think these two alerts are failing in the same way too:

BioSecurity alert for NSW_NPWS_WesternRivers_Weeds_list and

BioSecurity alert for NSW_NPWS_Western_Weeds_list

kylie-m commented 4 months ago

Thanks for flagging Andrew, the team are investigating

kylie-m commented 4 months ago

Hi @turley85 , thanks for the screenshot, from the R alert, could you please supply a few occurrenceIds / links that you were expecting that did not appear?

turley85 commented 4 months ago

Hi @turley85 , thanks for the screenshot, from the R alert, could you please supply a few occurrenceIds / links that you were expecting that did not appear?

https://biocache.ala.org.au/occurrences/4a42e85a-117c-4734-a663-bb90164f2991 https://biocache.ala.org.au/occurrences/d161667c-c613-434b-aa53-272945e8fb65 https://biocache.ala.org.au/occurrences/d87ee725-d644-4baf-b755-a9d95a5a72b9

adam-collins commented 4 months ago

The use of spatialObject requires more knowledge. General users of the ALA do not need to be aware of this as their objects are simplified by default. spatialObject is not simplified automatically, giving more control to advanced users.

In this case, the tests above fail and one reports that the object is too large.

Analysis of all future shapefiles is required. For this example:

The method of simplification to use for all future shapefiles:

Example use cases:

kylie-m commented 3 months ago

Hi @turley85 - I'm happy to run through this in QGIS with you, either in our catchup tomorrow or another time that suits, if you'd like

turley85 commented 3 months ago

@kylie-m I think we need more than just to correct this file. Could a list of requirements for shapefile be produced that could be given to users to ensure their files match our requirements/standards?

kylie-m commented 3 months ago

Hi Andrew, I've been working through this one for you, it looks like we have a promising solution.

Following Adam's guidance above, the team have downloaded the shapefiles, reduced the complexity of each in QGIS, and uploaded a new copy. I'll include my testing steps below too.

  1. Invasive species unit, for BioSecurity alert for NSW_NPWS_InvasiveSpeciesUnit_list
    • Original shapefile: pid=9433211
    • Updated shapefile, complexity reduced to 0.001 degrees (approx 111m): pid=9439584
    • For your reference, here is the saved session in Spatial Portal where you can view both layers together and compare them: https://spatial.ala.org.au/?ss=1723187675952

Testing steps:

  1. Western rivers, for BioSecurity alert for NSW_NPWS_WesternRivers_Weeds_list and BioSecurity alert for NSW_NPWS_Western_Weeds_list

Test links following steps above:

  1. I also did a quick test of an alert, spatialObject:9439584 seemed to work well for my "Rivers-test2" alert in production

  2. Actions for you -

    • Following on from this, could you try out the new spatial objects in your lists/alerts and see if everything appears to be working at the end of the process?

Longer term, I agree some guidelines for users on what they're supplying would be helpful. I spoke with Adam about this, though we don't have a single file size limit we can give them. As a starting point, we could test new files using the biocache web service link and if that fails, we could ask for complexity to be reduced. We can also run through the QGIS process we have used if needed, though I expect their experts are probably already across it.

kylie-m commented 2 months ago

Hi Andrew, I've taken another look at the NSW NPWS Invasive Species Unit List shapefile, and it's looking like the shapefile optimisation process I've used has worked.

kylie-m commented 1 month ago

Hi @turley85, just flagging the response to this one, have you had a chance to try out the new optimised version of NSW NPWS Invasive Species Unit List shapefile - spatialObject:9454574? It was testing ok for me with alerts, but let me know if any issues come up

turley85 commented 1 month ago

That looks to be working @kylie-m. This list has been transferred over to the new alerts system now.

kylie-m commented 1 month ago

That's great news! I'll close off this issue now