Skip to content
This repository has been archived by the owner on Sep 22, 2022. It is now read-only.

New dev API is ready #66

Closed
karthik opened this issue Dec 18, 2014 · 11 comments
Closed

New dev API is ready #66

karthik opened this issue Dec 18, 2014 · 11 comments

Comments

@karthik
Copy link
Contributor

karthik commented Dec 18, 2014

New API is ready.

related issue: https://github.com/berkeley-gif/bee/issues/266#issuecomment-67558081

Due: Dec 31.

@postfalk
Copy link

postfalk commented Jan 7, 2015

Does it mean rOpenSci works properly? Just double-checking!

@karthik
Copy link
Contributor Author

karthik commented Jan 7, 2015

Not yet. Almost there but there are still a few issues to resolve.

@postfalk
Copy link

postfalk commented Jan 7, 2015

Ok. You might post an issue if you think the error is on my side OR if you
think the API would be the more appropriate place. I am happy to work on
this.

On Wed, Jan 7, 2015 at 12:43 PM, Karthik Ram notifications@github.com
wrote:

Not yet. Almost there but there are still a few issues to resolve.


Reply to this email directly or view it on GitHub
#66 (comment).

Falk Schuetzenmeister, PhD
Web Application Developer
Geospatial Innovation Facility http://gif.berkeley.edu/

University of California Berkeley
111 Mulford Hall
94720 Berkeley, CA

schuetzenmeister@berkeley.edu
1-510-642-4897

@karthik
Copy link
Contributor Author

karthik commented Jan 7, 2015

@postfalk A couple of tests are failing and it seems like incorrect JSON being returned from your side when using genus_exact:

> ee_observations(genus__exact = "Pinus")
Error in parseJSON(txt) : lexical error: invalid char in json text.
                                       <html>  <head><title>502 Bad Ga
                     (right here) ------^
Calls: ee_observations ... <Anonymous> -> fromJSON_string -> parseJSON -> .Call
In addition: Warning message:
In ee_observations(genus__exact = "Pinus") :
  server error: (502) Bad Gateway
Calls: ee_observations -> warn_for_status

@postfalk
Copy link

postfalk commented Jan 7, 2015

It looks more like you get a 502 error which is probably a time out due to
the small instance the dev server is running on. I will explore.
Thx

On Wed, Jan 7, 2015 at 3:31 PM, Karthik Ram notifications@github.com
wrote:

@postfalk https://github.com/postfalk A couple of tests are failing and
it seems like incorrect JSON being returned from your side:

ee_observations(genus__exact = "Pinus")Error in parseJSON(txt) : lexical error: invalid char in json text.
<title>502 Bad Ga
(right here) ------^Calls: ee_observations ... -> fromJSON_string -> parseJSON -> .CallIn addition: Warning message:In ee_observations(genus__exact = "Pinus") :
server error: (502) Bad GatewayCalls: ee_observations -> warn_for_status


Reply to this email directly or view it on GitHub
#66 (comment).

Falk Schuetzenmeister, PhD
Web Application Developer
Geospatial Innovation Facility http://gif.berkeley.edu/

University of California Berkeley
111 Mulford Hall
94720 Berkeley, CA

schuetzenmeister@berkeley.edu
1-510-642-4897

@postfalk
Copy link

postfalk commented Jan 7, 2015

https://dev-ecoengine.berkeley.edu/api/observations/?genus_exact=pinus

worked for me can you rerun?

Falk

On Wed, Jan 7, 2015 at 3:36 PM, Falk Schuetzenmeister <
schuetzenmeister@berkeley.edu> wrote:

It looks more like you get a 502 error which is probably a time out due to
the small instance the dev server is running on. I will explore.
Thx

On Wed, Jan 7, 2015 at 3:31 PM, Karthik Ram notifications@github.com
wrote:

@postfalk https://github.com/postfalk A couple of tests are failing
and it seems like incorrect JSON being returned from your side:

ee_observations(genus__exact = "Pinus")Error in parseJSON(txt) : lexical error: invalid char in json text.
<title>502 Bad Ga
(right here) ------^Calls: ee_observations ... -> fromJSON_string -> parseJSON -> .CallIn addition: Warning message:In ee_observations(genus__exact = "Pinus") :
server error: (502) Bad GatewayCalls: ee_observations -> warn_for_status


Reply to this email directly or view it on GitHub
#66 (comment).

Falk Schuetzenmeister, PhD
Web Application Developer
Geospatial Innovation Facility http://gif.berkeley.edu/

University of California Berkeley
111 Mulford Hall
94720 Berkeley, CA

schuetzenmeister@berkeley.edu
1-510-642-4897

Falk Schuetzenmeister, PhD
Web Application Developer
Geospatial Innovation Facility http://gif.berkeley.edu/

University of California Berkeley
111 Mulford Hall
94720 Berkeley, CA

schuetzenmeister@berkeley.edu
1-510-642-4897

@karthik
Copy link
Contributor Author

karthik commented Jan 7, 2015

That worked, but removing that test led other tests to fail. You might be right that the small instance isn't able to return results fast enough.

@postfalk
Copy link

postfalk commented Jan 7, 2015

Correction:

https://dev-ecoengine.berkeley.edu/api/observations/?genus__exact=Pinus

On Wed, Jan 7, 2015 at 3:39 PM, Falk Schuetzenmeister <
schuetzenmeister@berkeley.edu> wrote:

https://dev-ecoengine.berkeley.edu/api/observations/?genus_exact=pinus

worked for me can you rerun?

Falk

On Wed, Jan 7, 2015 at 3:36 PM, Falk Schuetzenmeister <
schuetzenmeister@berkeley.edu> wrote:

It looks more like you get a 502 error which is probably a time out due
to the small instance the dev server is running on. I will explore.
Thx

On Wed, Jan 7, 2015 at 3:31 PM, Karthik Ram notifications@github.com
wrote:

@postfalk https://github.com/postfalk A couple of tests are failing
and it seems like incorrect JSON being returned from your side:

ee_observations(genus__exact = "Pinus")Error in parseJSON(txt) : lexical error: invalid char in json text.
<title>502 Bad Ga
(right here) ------^Calls: ee_observations ... -> fromJSON_string -> parseJSON -> .CallIn addition: Warning message:In ee_observations(genus__exact = "Pinus") :
server error: (502) Bad GatewayCalls: ee_observations -> warn_for_status


Reply to this email directly or view it on GitHub
#66 (comment).

Falk Schuetzenmeister, PhD
Web Application Developer
Geospatial Innovation Facility http://gif.berkeley.edu/

University of California Berkeley
111 Mulford Hall
94720 Berkeley, CA

schuetzenmeister@berkeley.edu
1-510-642-4897

Falk Schuetzenmeister, PhD
Web Application Developer
Geospatial Innovation Facility http://gif.berkeley.edu/

University of California Berkeley
111 Mulford Hall
94720 Berkeley, CA

schuetzenmeister@berkeley.edu
1-510-642-4897

Falk Schuetzenmeister, PhD
Web Application Developer
Geospatial Innovation Facility http://gif.berkeley.edu/

University of California Berkeley
111 Mulford Hall
94720 Berkeley, CA

schuetzenmeister@berkeley.edu
1-510-642-4897

@karthik
Copy link
Contributor Author

karthik commented Jan 18, 2015

Hi @postfalk
The error shows up every single time. So if this is an issue with the dev being a small instance, there will be no way to resolve in the current state. I think you should just switch the API over and I'll quickly fix anything that stalls and will push a fix to CRAN. I don't think the plumbing itself has changed so we should be fine. Let me know once you do that, and I'll rerun the tests and fix any bugs. I don't think we need to wait longer unless there is more to do on the dev side.

@postfalk
Copy link

Ok, moving on with some improvements and will do that in a couple of weeks.
Could you please copy paste the request that fails.
Thx

On Sun, Jan 18, 2015 at 2:57 PM, Karthik Ram notifications@github.com
wrote:

Hi @postfalk https://github.com/postfalk
The error shows up every single time. So if this is an issue with the dev
being a small instance, there will be no way to resolve in the current
state. I think you should just switch the API over and I'll quickly fix
anything that stalls and will push a fix to CRAN. I don't think the
plumbing itself has changed so we should be fine. Let me know once you do
that, and I'll rerun the tests and fix any bugs. I don't think we need to
wait longer unless there is more to do on the dev side.


Reply to this email directly or view it on GitHub
#66 (comment).

Falk Schuetzenmeister, PhD
Web Application Developer
Geospatial Innovation Facility http://gif.berkeley.edu/

University of California Berkeley
111 Mulford Hall
94720 Berkeley, CA

schuetzenmeister@berkeley.edu
1-510-642-4897

@karthik
Copy link
Contributor Author

karthik commented Jan 20, 2015

Same as before. See a few comments above.

@maelle maelle closed this as completed Sep 22, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants