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

ani.json Lese und Schreibe Problem #35

Open
Crypto90 opened this issue Apr 16, 2023 · 2 comments
Open

ani.json Lese und Schreibe Problem #35

Crypto90 opened this issue Apr 16, 2023 · 2 comments

Comments

@Crypto90
Copy link

Könnte man die Funktion der ani.json für lesen/schreiben verbessern?

Ich bin auf folgendes Problem gestoßen:

Läuft das Processing im Loop an und man fügt in der Zeit manuell etwas neues zum Monitoring (in die ani.json) hinzu, fliegt der hinzugefügte Eintrag wieder raus, wenn im Processing Loop etwas hinzugefügt wird.
Das liegt wohl daran, dass zu Beginn des Processings die ani.json eingelesen wird und bis zum Ende vom Processing diese geöffnet bleibt und aktualisiert wird.
Dabei gehen jedoch Einträge die in der Processing Zeit hinzugefügt werden verloren und man kann das ganze widerholen.

Sinnvoller wäre es, wärend des Processings, wenn etwas aktualisiert werden muss innerhalb der ani.json, zum Zeitpunkt der nötigen Aktualisierung die ani.json neu einzulesen, die Änderungen zu schreiben und dann wieder zu schließen.

So könnten auch neue Einträge hinzugefügt werden, während parallel das Processing läuft. Die Wahrscheinlichkeit für einen Conflict wäre so sehr viel geringer.

@Crypto90 Crypto90 changed the title Config lesen und schreiben Problem ani.json Lese und Schreibe Problem Apr 16, 2023
@Pfuenzle
Copy link
Owner

Ja, das würde tatsächlich mehr Sinn machen, ist auch geplant bei dem Rewrite, der seit längerem langsam in Arbeit ist.
Der AniBot wurde eigentlich auch eher als Proof of Concept geschrieben und ist dann mit der Zeit gewachsen, also sind viele Dinge darin eher so semi-logisch.

In der aktuellen Version werde ich nichts dran ändern, aber wenn ich mal die neue Version fertig habe, sollte es das problem nicht mehr geben.

@Crypto90
Copy link
Author

Okay, super gut zu wissen. Danke.

Ich hab dir gerade mal eine Email geschrieben, zum Thema Web Frontend/Webinterface.

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

No branches or pull requests

2 participants