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

use FastAPI as webserver #233

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

use FastAPI as webserver #233

wants to merge 1 commit into from

Conversation

yankee42
Copy link
Collaborator

Ich habe damit mal was rumgespielt. Da openWB mein erstes Python-Projekt ist, habe ich auch genau null Erfahrung mit Webframeworks für Python.

Die Dokumentation von FastAPI hat mir zugesagt, deswegen habe ich das einfach mal damit probiert und siehe da, es scheint eine recht einfache Kiste zu sein. Es gibt aber noch unzählige Alternativen zu FastAPI, die ich alle nicht getestet habe und von denen ich auch keine Ahnung habe.

So ist es nur eine kleine Demo die zeigt, dass es geht. Vorteile zu jetzt:

  1. Kein Apache mehr - eine "äußere Abhängigkeit" die systemweit konfiguriert werden muss weniger
  2. Man kann die Endpoints der API jetzt in Python coden
  3. Der Webserver läuft im gleichen Prozess wie der Rest von openWB. D.h. auch mit shared memory. D.h. man kann jetzt auch ganz einfach API-Endpoints schreiben die direkt mit dem laufenden Prozess interagieren und natürlich dafür die komplette Codebasis nutzen.

@yankee42
Copy link
Collaborator Author

Flake8 failed weil es immernoch auf Python 3.5 eingestellt ist. Aber das sollte ja für dieses Repo egal sein und 3.8 oder 3.9 oder so der Standard? Was ist das neue Minimum?

@DetMoerk
Copy link
Contributor

Auf dem Raspi in Bullseye ist Python 3.9.2 installiert.

@hhoefling
Copy link
Contributor

Soll nicht auch Buster unterstützt werden

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants