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

google map says oops on specimen records #2011

Closed
amgunderson opened this issue Mar 28, 2019 · 19 comments
Closed

google map says oops on specimen records #2011

amgunderson opened this issue Mar 28, 2019 · 19 comments
Labels
Bug Arctos is not performing as it should. Priority-Critical (Arctos is broken) Critical because it is breaking functionality.

Comments

@amgunderson
Copy link
Contributor

See any specimen record. Happens for me in chrome and safari.
Screen Shot 2019-03-28 at 9 45 49 AM

@amgunderson amgunderson added the Bug Arctos is not performing as it should. label Mar 28, 2019
@KyndallH
Copy link

Same here.

@mkoo
Copy link
Member

mkoo commented Mar 28, 2019 via email

@amgunderson
Copy link
Contributor Author

I have a credit card, how much is it?

@dustymc dustymc added the Priority-Critical (Arctos is broken) Critical because it is breaking functionality. label Apr 1, 2019
@dustymc
Copy link
Contributor

dustymc commented Apr 2, 2019

@mkoo any news? Do I need to start looking for another solution?!?

@DerekSikes
Copy link

same here

@sharpphyl
Copy link

I have a credit card too. What gives??

@dustymc
Copy link
Contributor

dustymc commented Apr 17, 2019

@mkoo @ccicero anybody?

@dustymc
Copy link
Contributor

dustymc commented Apr 22, 2019

@mkoo @ccicero? Do I need to start looking for another solution?

@Jegelewicz
Copy link
Member

Do we need to email them - not sure they monitor GitHub like we wished they did....

@ccicero
Copy link

ccicero commented Apr 26, 2019

MK email 4/26:

Honestly the best solution is to opt for a new geo service like through leaflet (open source, FREE). We are moving BerkeleyMapper from google api to leaflet and hopefully will have something for people to test soon.

Even with the Google Geo-grant, they are not making it easy to manage, almost as if they would like everyone to pay! It's honestly the worst customer support ever.

Dusty, I've sent you the Leaflet API link before and while I think it may not have ALL the lookup services you are using it at least as the map viz (quick start guide). We could request a Mapbox tile key and have some cool basemaps!

What do you think?

Meanwhile I will write Google grants AGAIN

@ccicero
Copy link

ccicero commented Apr 26, 2019

Dusty email 4/26:

I'm not morally opposed to moving away from Google, but it's going to take quite a bit of coding/time and I am likely to be completely comsumed by Oracle over the next few weeks. We've been pushing TACC on this upgrade, things started happening yesterday, the map stuff REALLY needs to happen in a dev environment, and overall doing the stuff I've been doing in prod is just dangerous and dumb. I feel like prioritizing the upgrade over all else is corect, but I'm not really thrilled about pushing Arctos with broken maps either.

Getting a functional API key, even if it's somewhat limited and even if it's somewhat short-term, seems critical to me at this point.

https://developers.google.com/maps/documentation/javascript/get-api-key

I suspect we'll lose some functionality with a not-Google solution too, but I'm not sure - leaflet does seem to have a fair number of plugins to do random stuff.

@ccicero
Copy link

ccicero commented Apr 26, 2019

So can we do a short-term solution and then work toward the longer-term solution/switch?

It's hard to figure out how much the Google Maps API licensing would cost.

How easy would it be to embed a BerkeleyMapper map onto the detail pages instead of a Google Maps, so we have something. It does't do the geocoding that we have now, but at least the detail page would show a map rather than an 'oops' !

I agree that with SPNHC and other meeting presentations coming up, we can't show Arctos the way it is now.

@campmlc
Copy link

campmlc commented Apr 26, 2019

How much would it cost us to pay it ourselves for 6 months or a year, without the grant?

@ccicero
Copy link

ccicero commented Apr 26, 2019

That's what's hard to figure out. It looks like it's based on map credits, usage limits, and quotas, and can vary depending on your plan.

We have a previous license:
https://developers.google.com/maps/premium/previous-licenses/

Pricing plans vary depending on usage:
https://cloud.google.com/maps-platform/pricing/
https://developers.google.com/maps/premium/overview

Their website is very confusing.

@dustymc
Copy link
Contributor

dustymc commented Apr 26, 2019

Thanks to @mkoo we again have basic mapping capability. Please leave a comment here if you find any undocumented broken map-related functionality.

After Oracle is upgraded and the dust settles we need to revisit this. It would be very useful if ya'll could check out https://leafletjs.com/ and we could come out of the upgrade with a longer-term plan. I'd rather not fix the remaining stuff and then immediately abandon it for a different solution!

  • I've turned off maps for Media - enabling that needs a client_id (and how to get one is anything but clear) or a different authentication method (requires significant coding).
  • internal stuff (eg, edit locality) can just stay broken for the time being
  • all service calls (elevation suggestions, distance-from-suggested reports, locality suggestions and automation, search terms, etc.) are broken for now

@mkoo
Copy link
Member

mkoo commented Apr 26, 2019 via email

@campmlc
Copy link

campmlc commented Apr 26, 2019 via email

@mkoo
Copy link
Member

mkoo commented Apr 26, 2019 via email

@mkoo
Copy link
Member

mkoo commented Apr 27, 2019

since the basic Bug is fixed, I'm closing this issue (and related ones). I will create a new one for the background services we need, which is really just a matter of more learning how the enterprise Google cloud console works so will assign to myself!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Arctos is not performing as it should. Priority-Critical (Arctos is broken) Critical because it is breaking functionality.
Projects
None yet
Development

No branches or pull requests

9 participants