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

sharing collecting events #1017

Closed
dustymc opened this issue Jan 3, 2017 · 9 comments
Closed

sharing collecting events #1017

dustymc opened this issue Jan 3, 2017 · 9 comments
Assignees
Labels
Priority-Critical (Arctos is broken) Critical because it is breaking functionality.

Comments

@dustymc
Copy link
Contributor

dustymc commented Jan 3, 2017

Can we do with collecting_events what we've done with localities WRT sharing?

@dustymc dustymc added this to the Needs Discussion milestone Jan 3, 2017
@campmlc
Copy link

campmlc commented Jan 3, 2017 via email

@dustymc dustymc added the Priority-Critical (Arctos is broken) Critical because it is breaking functionality. label Jan 4, 2017
@ekrimmel
Copy link

ekrimmel commented May 9, 2017

Ok, here's an example of why people probably didn't want to share collecting events in the past. Higher geography of "Africa, Kenya" got georeferenced despite the fact that none of the specimens have a specific locality. I could be misunderstanding what happened, but to me our specimens with this locality now look "wrong" because a point shows up on a map and makes it appear as though we know exactly where these came from (even though the error on the georeference is huge).

I think this is probably a social/procedural problem for us rather than a database issue. Although I'd also argue that specimens without a specific locality should not be getting point georeferenced.

@dustymc
Copy link
Contributor Author

dustymc commented May 9, 2017

share collecting events

The georeferencing - anything you'd see on a map - happens to the locality, not event (which contains "verbatim coordinates").

point shows up on a map

Points are just wrong, but sometimes I don't have the resources to do anything else (circles are computationally expensive!). I THINK I have all the public interfaces dealing with shapes (circles) now, but definitely not the internal stuff. And edit locality is doubly-weird because the webservice suggestions are points, so even if we do have a shape then displaying it makes things hard to compare.

without a specific locality

Drawing precision inferences from the structure of the data is usually a mistake. Geography includes oceans and such, but also very precise things - the intersection of state/county/park/etc. might be a few acres. I collected a lot of "no specific locality" specimens because I was strapped to a very expensive WAAS-enabled GPS at the time - why would I want to introduce garbage by recording where I THINK I am when I KNOW where I am (just not in strings!)? I'm sure there are many more examples - structure and accuracy/precision are not related.

point georeferenced

Error=0 should ALWAYS be interpreted to mean "we have no idea" rather than "infinitely precise." I'm up for clever ideas on how we better convey that.

In any case, I don't see the problem here. The text data are "Kenya, and there's some weird field that demands we try to be more precise in the most imprecise way possible but we don't have anything else to say!!" and the spatial data do a pretty decent job of reflecting that.

screen shot 2017-05-09 at 8 29 21 am

That level of precision is useful for all sorts of things - it'll show up in my spatial query for "circle including all africa" and if I find a bog lemming from there I'll be concerned and for the specimens with an elevation of 14000-15000 feet I can eliminate everything except two floppy donuts (just need better spatial tools!) and .... - those are REALLY useful data, even if they're not tens-of-meters precision!

@ekrimmel
Copy link

ekrimmel commented May 9, 2017

Thank you for explaining, this makes much more sense to me now and I'll take back my above complaints :)

Would it be computationally expensive to default the google map interface to a more zoomed out view (one that captures the extent of the error, e.g. what's above)? I get why there's actually no real problem here, but I do still think it's misleading when the first map that shows up is way zoomed in:
capture

Also, point taken on "no specific locality" equating to "not appropriate for point georeferencing." My head is always in legacy data land...

@dustymc
Copy link
Contributor Author

dustymc commented May 9, 2017

If I already have the extents, it doesn't "cost" much to do stuff with them. #1129

@dustymc
Copy link
Contributor Author

dustymc commented Dec 5, 2017

AWG consensus: go (but review #1023 first)

@dustymc
Copy link
Contributor Author

dustymc commented Mar 15, 2018

#1023 is implemented, so I don't think anything is holding us back on this.

  • Has the shared locality model caused any problems?
  • Are the emails useful?

Etc. - anything different we should do here, or shall I just do what we've done for localities to events?

I'll try to start on this next week if there's no further feedback.

@dustymc
Copy link
Contributor Author

dustymc commented Apr 3, 2018

Lacking further feedback, I'll implement this the same way localities have been: changes will be logged and daily summary emails will be sent.

@dustymc dustymc self-assigned this Apr 3, 2018
@dustymc dustymc modified the milestones: Next Task, Active Development Apr 3, 2018
@dustymc
Copy link
Contributor Author

dustymc commented Apr 17, 2018

This is now implemented

@dustymc dustymc closed this as completed Apr 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority-Critical (Arctos is broken) Critical because it is breaking functionality.
Projects
None yet
Development

No branches or pull requests

3 participants