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

Denon Plugin: suggested resend feature #937

Closed
wants to merge 11 commits into from

Conversation

onkelandy
Copy link
Member

.. as a base for discussion how SDP could resend commands

@Morg42
Copy link
Member

Morg42 commented Jun 28, 2024

Wenn das bei dir läuft, kannst du das natürlich so machen.

Wenn ich die Entscheidung "nochmal senden" treffe, muss ich eine Grundlage dafür haben, in aller Regel eine Fehlermeldung des Gerätes bzw. der Gegenstelle.

Insofern taugt "jeder" resend-Algorithmus nicht als generelle Lösung, weil...

  • es Geräte gibt, die den Empfang gar nicht bestätigen
  • es Geräte gibt, den korrekten Empfang nicht bestätigen
  • es Geräte gibt, die keine Fehler zurückmelden
  • eine eindeutige Zuordnung von Antwort zu Anfrage nur bei synchroner Bearbeitung möglich ist,
  • die meisten Protokolle eine solche Zuordnung nicht besitzen (Ausnahme zB JSON-RPC).

Konsequenz ist aus meiner Sicht, dass man resend-Funktionalitäten

  1. nicht grundsätzlich implementieren kann
  2. auf die jeweilige Situation bezogen implementieren muss und
  3. eine Lösung, die vielen Fällen gerecht wird, als optionales Modul implementiert (bei sdp: als protocol)

Wie gesagt - wenn das für denon funktioniert, dann bitte :) die Erweiterungen für sdp können wir woanders klären.

@onkelandy onkelandy closed this Aug 11, 2024
@onkelandy onkelandy deleted the denon branch August 11, 2024 18:40
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.

2 participants