-
-
Notifications
You must be signed in to change notification settings - Fork 323
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
universal HTTP interface #3
Comments
Tip top. Kann ich quasi direkt so einbauen. Einzig würde ich spindel in spindle (englisch) ändern. Den Wert von name würde ich in Anführungszeichen erwarten.
|
Folgende Vorschläge:
Liefert
alternativ
|
@BernhardSchlegel Gültiges json: compressed json würde ich vorziehen da es Daten spart. Auch könnte man properties unter Umständen einfach verwerfen wenn nicht unterstützt durch das backend. |
Version im Pfad finde ich eine super Idee. Ich würde die API in v1 so klein wie möglich halten. Also Id oder Name. Ist ja quasi das gleiche. Wenn die CPU Id eindeutig ist würde ich diese vorziehen. Je weniger Daten der Nutzer an der Spindel konfigurieren muss desto besser. |
mit |
@Manuel83 Wenn du den Namen verwenden würdest und dynamisch die Spindel erstellst, würde es das coole Feature ergeben dass man sagen kann "IPA_20.11.16_Behaelter1" o.Ä. um Daten für eine bestimmte Sud abzutrennen. @BernhardSchlegel |
Ja, genau so mache ich das bei CraftBeerPi schon mit den DS18B20 Sensoren. Für jeden kann man in der Software einen Namen frei vergeben. Das mapping übernimmt das Backend. |
Das sieht gut aus! Ich kann dir mal als Beispiel das Polynom nennen welches ich in der aktuellen Kalibrierung erhalte: Das ist allerdings abhängig von der Spindel, jede Spindel wird sich etwas anders verhalten. Wieviel das ist (abgesehen vom Offset Winkel) kann ich mangels Erfahrung nicht sagen. Ich habe die HTTP Schnittstelle auch schon implementiert, kann es mangels freiem Raspi momentan nicht testen. Hast du evtl. ein Gateway zum Internet offen an welches ich pushen könnte? |
Top Danke. Man kann später für jede Spindel ein eigenes Polynom einstellen. Leider habe ich keinen Server im Internet laufen. Testen lohnt auch noch nicht wirklich. Ich implementiere gerade den "Happy Day" Ablauf. Das ganze muss dann noch robust gemacht werden. Die Variable muss "tilt" heißen. Aktueller Status:
Fehlende Funktion:
|
Richtig gut! Für die Zukunft, muss noch die Grundlagen erarbeiten, aber die Temperatur muss hier auch in die Formel, wie bei einer normalen Spindel auch. Off topic: |
Ich meine bei orange Pi war die GPIO Ansteuerung anders. Python wird wohl laufen. Aktuelle ist es ein sehr simples eval() mit dem ich die Formel evaluiere. Quasi noch eine Sicherheitslücke. Hier kommt entweder noch ein Regex rein das die Formel prüft und Werte vor der Ausführung prüft oder die Formel wird zerlegt und die Parameter werden einzeln konfiguriert. |
Sehr cool! |
Bezüglich dem PID, sprich keine Daten für den closed loop? |
@Manuel83
Korrekt? |
First implementation of the universal Interface. Anyone likes to try: |
I tried alpha 3.0.0 on a Wemos D1 board with the following logging after a reset. Is it configured right?
The above text is viewed in the logscreen after configuration and a reset. I only use a Wemos D1 board without the accelerator board and the DS18b20. Probably thats the reason why the Wemos board only one time gives the above logging after a reset. Furthermore I still need to puzzle further on the Labview side to read the postdata. This does not work yet. |
@Henielma You are missing the 470 Ohm resistor to wake up from sleep |
Ja, das wäre ein valider Test. Die Server URL kann man aber später komplett frei eingeben oder? |
Würde dann dein "CraftBeerPi" als preset URL festlegen. Oder wird die nicht überall identisch sein? |
Klingt super. Dann muss der Endpoint wie folgt aussehen: |
Port: 80? OK, updated with 3.0.3 |
CraftBeerPi läuft standardmäßig auf Port 5000 |
Thanks for the 470 ohm tip. This resistor added and also the accellerometer board added. The serial screen information does now refresh after waking up. I will test V3.0.0 further because V3.0.3 does not run here. It gives:
|
Thanks thats true, found the issue, will post a fixed version tomorrow |
This is what I get in the log screen.
Hereafter the URL setting I configured in Labview. No data is read in Labview now. Any ideas what I need to change? |
please select the "HTTP" version and specify the url, port and ip. What happens? |
hmm your dropdown is not initialized after page load. Okay will fix that. Clicking once the dropdown should fix also. |
in URL you would set "/api/hydrometer/v1/data", the other seam to be ok. I will add the firmware more debug output tomorrow |
That's cool. Thanks a lot. Tomorrow I will add the features to delete the hydrometer and to use temperature values to control the refridgerator |
Unfortunately, no Raspi available. Anyone able to test? |
Unfortunately no Hydrometer available to test :-) |
Sure contact me [email protected] |
Hey Manuel and Sam, updated my CBP and iSpindel today, how could I test a connection between both ? Ahoi, not |
Awesome- time to celebrate! |
Here you will find the CBPi Branch which provides the Hydrometer feature https://github.com/Manuel83/craftbeerpi/tree/hydrometer I also added the feature to use the hydrometer temperature to control the freezer. By the way. Does the iSpindel support also Fahrenheit or do I have to calculate this inside of CBPi? |
Great job @Manuel83!! Currently there is no Fahrenheit support. Do you see a benefit of providing it through the iSpindel? |
Since we have no issues I'm going to close it. |
Eine universelle Schnittstelle zum Datenaustausch soll festgelegt werden.
Variante 1:
The text was updated successfully, but these errors were encountered: