AtlasOfLivingAustralia / ala-bie-hub

BIE (Biodiversity Information Explorer) hub - merge of old ala-bie and bie-plugin
Mozilla Public License 2.0
1 stars 6 forks source link

Lists content not appearing on BIE #16

Closed sat01a closed 1 year ago

sat01a commented 1 year ago

In BIE content is usually served from the lists tool

The distribution map (blue) placement looks terrible.

adam-collins commented 1 year ago

Both issues are specieslist-webapp.

  1. These items are not included because the path variable LSID was not handling LSIDs containing //
  2. These items are not imported because ws/speciesListItems now has a limit on the max number joined records due to the api gateway. The default is max=400. Two issues are the paging is missing from the bie-index call and the paging is based on the join resulting in overlapping pages that require merging.

Pipelines and biocache-service also need paging implemented for the changed webservice behaviour.

adam-collins commented 1 year ago
  1. \\ handling https://github.com/AtlasOfLivingAustralia/specieslist-webapp/commit/0882c5b7056208661dd433c0b97a90501ca968fc
  2. fix webservice paging https://github.com/AtlasOfLivingAustralia/specieslist-webapp/commit/c9634d8389fb181c9ea832723df5d26f5b332f23
  3. add paging to speciesListItems call https://github.com/AtlasOfLivingAustralia/bie-index/pull/359
  4. add paging to speciesListItems call https://github.com/AtlasOfLivingAustralia/spatial-service/commit/a990a72698d360a1bad34ffb9d16aaeda03f507e
  5. add paging to speciesListItems call https://github.com/AtlasOfLivingAustralia/biocache-service/tree/species-list-items-paging

Pipelines uses a different webservice to get this information from lists and does not need a change.

adam-collins commented 1 year ago

lists-test and bie-test are running with the fixed code and BIE SOLR index. https://bie-test.ala.org.au/species/https://biodiversity.org.au/afd/taxa/31869a0e-2ce5-40be-b6f6-c0231a48b414 (ignore images-test javascript errors)

adam-collins commented 1 year ago
  1. ala-install change https://github.com/AtlasOfLivingAustralia/ala-install/commit/03124a9b80320735b528f402f98688f7207a6f95
sat01a commented 1 year ago

Thanks @adam-collins for following this through, sushant will review and incorporate this changes and release for internal testing. ccing: @sughics @kylie-m @peggynewman @elywallis

peggynewman commented 1 year ago

The Conservation Status issue seems to have been resolved, hooray!

But I'm consistently getting an error that I have to click multiple times to get rid of when I go to a BIE page:

Image

But there are still some problems: Anthochaera (Xanthomyza) phrygia - Regent Honeyeater List boxes are showing up but the EOL text feed isn't

In the ones I look at, the Datasets box sentence is missing a count, ie, it says datasets have provided data to the Atlas of Living Australia for this species. but there should be a number in front.

Gymnobelideus leadbeateri - Leadbeater's Possum is working, except for this problem.

sughics commented 1 year ago

@peggynewman the errors you are seeing - I'm pretty sure thats related to the content type returned by biocache-service - https://github.com/AtlasOfLivingAustralia/ala-bie-hub/issues/17 - I currently implementing a fix

sat01a commented 1 year ago

Fixed and Deployed