pyro-api icon indicating copy to clipboard operation
pyro-api copied to clipboard

[docs] Add a tutorial/walkthrough to deploy everything on your own

Open frgfm opened this issue 3 years ago • 3 comments

It would be greatly beneficial for contributors to know how to orchestrate everything on their own. I suggest making a tutorial for this

frgfm avatar Oct 20 '22 11:10 frgfm

The README suggests that cloning the package and calling make run should start the API but curl http://localhost:8080/docs returns Empty reply from server. If something else is needed, it would be good to mention it in the README

blenzi avatar Oct 29 '22 12:10 blenzi

Hi @blenzi :wave:

I just tried again myself and the curl yields:

    <!DOCTYPE html>
    <html>
    <head>
    <link type="text/css" rel="stylesheet" href="https://cdn.jsdelivr.net/npm/swagger-ui-dist@4/swagger-ui.css">
    <link rel="shortcut icon" href="https://fastapi.tiangolo.com/img/favicon.png">
    <title>Pyronear API - Swagger UI</title>
    </head>
    <body>
    <div id="swagger-ui">
    </div>
    <script src="https://cdn.jsdelivr.net/npm/swagger-ui-dist@4/swagger-ui-bundle.js"></script>
    <!-- `SwaggerUIBundle` is now available on the page -->
    <script>
    const ui = SwaggerUIBundle({
        url: '/openapi.json',
    "dom_id": "#swagger-ui",
"layout": "BaseLayout",
"deepLinking": true,
"showExtensions": true,
"showCommonExtensions": true,
oauth2RedirectUrl: window.location.origin + '/docs/oauth2-redirect',
    presets: [
        SwaggerUIBundle.presets.apis,
        SwaggerUIBundle.SwaggerUIStandalonePreset
        ],
    })
    </script>
    </body>
    </html>
    % 

You can also see that the GH CI is performing the same action and getting an answer from curl, so I don't really know what caused this :shrug:

Please open a GH issue with the docker logs to debug this :pray:

frgfm avatar Oct 31 '22 09:10 frgfm

we can closed this one, no ?

RonanMorgan avatar Apr 29 '24 09:04 RonanMorgan

Correct, we're way more on a docker compose up instruction now than previously :+1:

frgfm avatar Jun 04 '24 22:06 frgfm