-
Notifications
You must be signed in to change notification settings - Fork 8
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
outlierForLayer field provides data for only one layer #31
Comments
Hi @shawnlaffan,
Are you wanting to get the data for the other outlier layers as separate columns or are you simply needing more information about the Also, is it a case that the A user story or use case would be helpful to frame the request, as well. |
Hi @nickdos, My use case is to identify records that are outliers for two or more env layers. Many of the records that are single layer outliers seem to be OK for my purposes (admittedly that's not based on rigorous testing, though). I had a look at the API pages, and the problem might be at the API level where the table is generated since a direct check also gives only one layer. Of course, now I cannot reproduce that since I forget which search I used. Perhaps it is the csv generation component. In any case, direct json access contains the three outlier layers. Snippet from https://biocache.ala.org.au/ws/occurrence/d97cd2e1-c871-4be5-bd50-2b963f210902 :
In terms of packing the info into the existing structures in ALA4R, multiple columns would work, but would get unwieldy pretty quickly, hence packing them into a single entry might be good, e.g. "el882;el883;el887". A space or semicolon would actually be a better separator than a comma, as otherwise csv parsing libs come into play. Hopefully that helps explain things a bit more. Shawn. |
Just an update. This is the record returned via the
|
Thanks @shawnlaffan. Looks like a bug (or feature) where the SOLR index has a |
Thanks @nickdos |
See latest comment on AtlasOfLivingAustralia/biocache-service#195 (comment) for a fix |
Thanks @nickdos |
I'm not very knowledgeable on ALA4R so I'm not sure if the fix suggested requires a code fix in ALA4R or not. @peggynewman any ideas? |
Yes @nickdos @shawnlaffan it's an ALA4R code fix. I'll label this a bug so it can go through in the next release. |
This is related to #27.
As an example, the online search for Acacia cangaiensis produces one record that is flagged as an outlier for three layers, Bio15, Bio17 and Bio26.
https://biocache.ala.org.au/occurrences/d97cd2e1-c871-4be5-bd50-2b963f210902
However, the data downloaded via ALA4R give only one layer, el882, which corresponds to Bio15.
Can more information be packed into this field? Or a new field be provided? A comma separated list should work well enough to state which layers a record is an outlier for.
Code to reproduce is below.
Thanks,
Shawn.
The text was updated successfully, but these errors were encountered: