search

CatalogueOfLife / checklistbank

UI for checklistbank.org
https://www.checklistbank.org/
7 stars 2 forks source link

Checklistbank: I am not able open GLI & WOL in the project #1444

Closed yroskov closed 3 weeks ago

yroskov commented 1 month ago

Description

While inside the CoL3 project, I can no longer open some source GSDs anymore for unknown reasons. So far, I met problem with GLI & WOL (whereas other GSDs, such as ITIS, Entiminae, Scarabs, WCO are accessable as expected). https://www.checklistbank.org/catalogue/3/metadata

Steps: COL - Source - Select, typed "GLI" or "WOL" in a search box. Drop list is empty, and these GSDs are unavailable:

image

image

The problem was met first time on 2024-07-17 and repeated on 2024-07-18

What browsers are you seeing the problem on?

Firefox

ChecklistBank URL

https://www.checklistbank.org/catalogue/3/metadata

yroskov commented 1 month ago

It looks like the problem caused by the length of ID (over 4 digits): GLI id 55434 WOL id 125101 Bryonames id 170394 CunaxidBase id 296427 Geometridae id 185410 etc. = all have the same problem

mdoering commented 1 month ago

@thomasstjerne this seems to be a limit of the max numbers of sources being loaded into the UI - I don't see any search/suggest to the API going on. With Plazi there are now >50k source datasets, so loading these all into the client memory is not feasable.

We could restrict the sources to non merge source only. But better would be a more dynamic interaction with the API to search for matches rather than keeping them all in memory. I do fear though the memory list of sources is used in other places as well?

mdoering commented 1 month ago

https://api.checklistbank.org/dataset?contributesTo=3&limit=1000 is being called. @thomasstjerne Shall I develop some suggest/search method that is quick to call?

I will expose this method on prod tomorrow: http://api.checklistbank.org/dataset/299029/source/suggest?merge=false&q=specie

You can also include merge sources

thomasstjerne commented 3 weeks ago

@mdoering the q param doesn´t seem to have any effect https://api.checklistbank.org/dataset/299029/source/suggest?merge=false&q=fish

mdoering commented 3 weeks ago

I will expose this method on prod tomorrow:

yes, deploying the fix that uses the query right now

mdoering commented 3 weeks ago

works now: https://api.checklistbank.org/dataset/3/source/suggest?q=fish