team-folivora / annoto

Medical annotation tool

MIT License

0 stars 1 forks source link

Switch to FastAPI #27

Closed bewee closed 2 years ago

bewee commented 2 years ago

This suggests switching from Flask to FastAPI.

FastAPI has a similar syntax to Flask, but is more modern and faster. It supports async request handling and you can generate a openapi yaml of your routes to auto-generate the javascript requests code.

127.0.0.1:8080/docs

codecov-commenter commented 2 years ago

Codecov Report

Merging #27 (a249136) into dev (99c7d7f) will increase coverage by 12.50%. The diff coverage is 100.00%.

@@             Coverage Diff              @@
##              dev       #27       +/-   ##
============================================
+ Coverage   87.50%   100.00%   +12.50%     
============================================
  Files           2         2               
  Lines          24        17        -7     
============================================
- Hits           21        17        -4     
+ Misses          3         0        -3

Impacted Files	Coverage Δ
api/src/app.py	`100.00% <100.00%> (ø)`
api/src/settings.py	`100.00% <100.00%> (ø)`

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 99c7d7f...a249136. Read the comment docs.

bewee commented 2 years ago

@jdsauer @n-koenig What do you think about this?

n-koenig commented 2 years ago

Sieht gut aus, von der Funktionalität hat sich ja nichts geändert und das ganze backend liegt jetzt einfach einen ordner tiefer, richtig? Sagt mir gerne nochmal morgen oder so warum ihr euch für fastAPI entschieden habt und ob sich irgendwas relevantes ändert. Eine Frage hab ich noch: Warum ist die venv jetzt in der globalen gitignore und nicht mehr in der api-gitignore?

bewee commented 2 years ago

Sieht gut aus, von der Funktionalität hat sich ja nichts geändert und das ganze backend liegt jetzt einfach einen ordner tiefer, richtig? Sagt mir gerne nochmal morgen oder so warum ihr euch für fastAPI entschieden habt und ob sich irgendwas relevantes ändert. Eine Frage hab ich noch: Warum ist die venv jetzt in der globalen gitignore und nicht mehr in der api-gitignore?

Ja, so ziemlich. Entscheidende Vorteile sind auch in der PR description drin. Die Syntax ist bei FastAPI sehr ähnlich, stellenweise geradezu identisch zu Flask.

Die venv wird jetzt oben ignored weil @jb3rndt und ich dort unseren venv Ordner haben; Das sollte so aber deine venv im Unteroardner weiterhin auch betreffen