search

terraref / brapi

Breeder's API implementation for TERRA-REF
https://terraref.org/brapi/v1/calls
BSD 3-Clause "New" or "Revised" License
0 stars 2 forks source link

brapi hackathon (move to openapi from swagger 2.0) #21

Closed robkooper closed 5 years ago

robkooper commented 5 years ago

This contains the work from the hackathon. The main part is that BRAPI switched to openapi 3.0 and the code generator no longer works with this. This now uses the connexion libraries to use the brapi.yaml file to create the then endpoints automagically.

To implement new endpoints you will need to create a file in /api/ with the name of the endpoint, for example /calls/ is now find in /api/calls.py. For GET function without a name the function in the file has to be called search() for a GET with an ID it is called get(). See files in /api/ for more details.

robkooper commented 5 years ago

We should first merge #20 into this pull request to simplify the code base (or it can be merged into master after wards).

tcnichol commented 5 years ago

Could I get instructions on how to run this locally?

tcnichol commented 5 years ago

I seem to be unable to connect to betydb and not sure what to change.

dlebauer commented 5 years ago

Did you see the readme on the openapi branch?

David LeBauer Director of Data Sciences Arizona Experiment Station THE UNIVERSITY OF ARIZONA

Bioscience Research Labs, 207 1230 N Cherry Ave | Tucson, AZ 85721 Office: 520-621-4381 dlebauer@email.arizona.edu

(sent from my phone - please pardon brevity and typos)

From: Todd Nicholson notifications@github.com Sent: Friday, May 3, 2019 6:18 PM To: terraref/brapi Cc: LeBauer, David Shaner - (dlebauer); Review requested Subject: Re: [terraref/brapi] brapi hackathon (move to openapi from swagger 2.0) (#21)

I seem to be unable to connect to betydb and not sure what to change.

— You are receiving this because your review was requested. Reply to this email directly, view it on GitHubhttps://github.com/terraref/brapi/pull/21#issuecomment-489154422, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AADRPZ3EHCQEZFULWDE2D4DPTRQVLANCNFSM4HJ6MNSA.

tcnichol commented 5 years ago

@dlebauer thanks i was not using the right command to start up bety

looks good, marking both approved.