-
Notifications
You must be signed in to change notification settings - Fork 13
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
documentation request: specimen detail maps #83
Comments
This is interesting. What determines the geography polygon?
I think we need to document the difference between the locality polygon and
the error radius circle - ideally, shouldn't these be the same? It took me
a minute to notice the locality polygon within the geography polygon - some
color changes, maybe?
…On Wed, May 22, 2019 at 11:56 AM dustymc ***@***.***> wrote:
Example: http://arctos.database.museum/guid/UCM:Fish:6064
Noteworthy stuff:
click 'more' after geography, then edit (pending ArctosDB/arctos#2090
<ArctosDB/arctos#2090>) - the geography has a
polygon
[image: Screen Shot 2019-05-22 at 10 49 47 AM]
<https://user-images.githubusercontent.com/5720791/58196540-5d543000-7c7f-11e9-94db-4dd75ac5a18e.png>
it's displayed on the event-map (big blue square).
[image: Screen Shot 2019-05-22 at 10 49 04 AM]
<https://user-images.githubusercontent.com/5720791/58196497-49103300-7c7f-11e9-8862-34c5f36b90e4.png>
locality/edit locality: the locality has a polygon.
zoom in a bit to see it...
[image: Screen Shot 2019-05-22 at 10 53 16 AM]
<https://user-images.githubusercontent.com/5720791/58196746-d784b480-7c7f-11e9-9740-25e9978e1a39.png>
Note the green border: the point is within the geography polygon
note the key/tools link below the map
- best option: what can we do so that doesn't need documentation?
- alternate: can someone somehow document this?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#83?email_source=notifications&email_token=ADQ7JBARUVDPT6X5NGQYQALPWWCM7A5CNFSM4HOWUY7KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GVIY4UA>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ADQ7JBEO5HY4MGVJ7QYF3JTPWWCM7ANCNFSM4HOWUY7A>
.
|
Someone adding WKT to geography. See also ArctosDB/arctos#1795
I think ideal would be to ditch the point-radius model altogether; those things can be represented as polygons, after all. BUT, that would make it difficult to share with anything that can't deal with polygons (so everything that's not Arctos AFAIK), be a lot more data to get to the same shape, etc. I think that's fairly unrealistic without "real" GIS capability (from which it should be trivial to generate point-radius data on demand). In the current model they're completely independent. In this example they line up nicely and I think the WKT can be seen as a more-precise refinement. Nothing requires that, however. For scale, there are around 500 locality polygons (mostly from some fish TCN I think) and many of them do no seem to be valid WKT. Development should probably be low-priority until we get more data, and whoever brings that data in can tell us what they want to do with it. I can mess with the colors fairly easily, but if you zoom around enough they seem to eventually do whatever they want - Google's API is the only thing I have for this, and it's fairly 'lightweight.' |
I agree that keeping this in mind, but low-priority is the way to go. When we eventually have the GIS capability to calculate max distance from a point (or centriod) to the polygon border we could then ditch the point-radius model. |
@dustymc what needs doing here? |
Probably nothing? |
Example: http://arctos.database.museum/guid/UCM:Fish:6064
Noteworthy stuff:
click 'more' after geography, then edit (pending ArctosDB/arctos#2090) - the geography has a polygon
it's displayed on the event-map (big blue square).
locality/edit locality: the locality has a polygon.
zoom in a bit to see it...
Note the green border: the point is within the geography polygon
note the key/tools link below the map
The text was updated successfully, but these errors were encountered: