Skip to content
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

add all locality data to specimenresults #755

Closed
dustymc opened this issue Sep 16, 2015 · 8 comments
Closed

add all locality data to specimenresults #755

dustymc opened this issue Sep 16, 2015 · 8 comments

Comments

@dustymc
Copy link
Contributor

dustymc commented Sep 16, 2015

even if it's ugly (eg, JSON??) - ugly is better than invisible.

@ccicero
Copy link

ccicero commented Sep 16, 2015

Dusty, can we discuss? Are you ralking about the table view or detail pag Maybe have a View more details link or something like that, rather than mucking up specimen detail /results and making those harder to read?

@dustymc dustymc added this to the Needs Discussion milestone Sep 16, 2015
@dustymc
Copy link
Contributor Author

dustymc commented Sep 16, 2015

can we discuss

YES!! (Flip "milestone" to "Needs Discussion" to flag that - I just did.)

table view

That's the main (only??) place where data is lacking.

detail page

That's easy to change and everything "fits" - you can see there whatever ya'll tell me you want to see.

mucking up specimen results

It would be something people can turn on/off, like everything else (I think??).

harder to read

I'm just making sure this doesn't fall off my radar for now. I'm thinking perhaps some sort of HTML-popup-link or something in the results/web view and something else (raw JSON??) for download, or SOMETHING of the sort. (That might work for Media too, which is kinda similar and sort of a pain right now.)

Anyway, yes, please, let's talk whenever it works for you, this isn't very urgent and I have no firm plans at the moment. We eventually, somehow, need to get either the data or some indication that there's more data into the results view.

@ccicero
Copy link

ccicero commented Sep 16, 2015

Ok. Let's discuss when I'm back from the field. Thanks.

@dustymc
Copy link
Contributor Author

dustymc commented Sep 16, 2017

From mini pow-wow with @campmlc @jldunnum @AdrienneRaniszewski

  • add all locality data to specimenresults as JSON (that's less-evil than providing no clue why their search and results don't match, or making it impossible to find specimens as now)
  • add a "has multiple events" flag in the GUID column, similar to how Media and Citations are handled.
  • allow search to match all events

Alternatives discussed:

  • treat rows like Occurrences, one row for each "place." GUID becomes non-unique in specimen results and downloads. Seems likely to lead to treating multiple encounters as biological individuals etc.

  • Concatenate all event data in individual columns. A critter that gets around a lot might looks like:

guid country state specloc ...
ABC:XYZ:123 Mexico; United States; United States; United States; Canada Baja California;California;Oregon;Washington;British Columbia bla; blah; blaugh; blaw; bla, eh ...

which seems confusing to me.

@dustymc dustymc modified the milestones: Needs Discussion, Next Task Oct 3, 2017
@dustymc dustymc modified the milestones: Next Task, Active Development Mar 7, 2018
@dustymc
Copy link
Contributor Author

dustymc commented Mar 7, 2018

Added "JSON Locality" option

@dustymc
Copy link
Contributor Author

dustymc commented Mar 7, 2018

"any geog" search performance isn't acceptable when all localities are considered - need to build a cache, precompile relevant geography data

@dustymc
Copy link
Contributor Author

dustymc commented Mar 8, 2018

any geog cache should be compiled in a couple days - added event-count label to results

@dustymc
Copy link
Contributor Author

dustymc commented Mar 14, 2018

This is "limited" (not really, but easier done as) to ~4K characters for Oracle, then has to be passed around and injected into browsers (perhaps hundreds of times - I think there's a "show 5000 rows option on specimenresults) and dealt with as CSV etc. To better accommodate that, I left most free-text fields (remarks and such) out. This is certainly not ALL locality data, but it should be completely sufficient for someone to figure out why a result is included and etc.

Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants