New error behaviour when calling `flush_tx_buffer` for several interface types

Aperta Adatta ai principianti
#1,922 0 commenti 1 reazione 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
1/5
Tempo stimato
Meno di un'ora
Idoneità per principianti
68/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Ferma
Stack tecnologico
python
Ambito
api

Direzione di ricerca

Inizia con la classe BusABC e il comportamento di flush_tx_buffer() introdotto da PR #1724; riproduci l'eccezione usando un'interfaccia come socketcan. Verifica che le interfacce non supportate tornino al comportamento no-op precedente senza modificare le interfacce che implementano lo svuotamento, quindi verifica che la riproduzione non generi più un'eccezione.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

bug
Describe the bug

Due to the addition of raise NotImplementedError in the BusABC class introduced with https://github.com/hardbyte/python-can/pull/1724 by @zariiii9003, trying to call flush_tx_buffer() throws an exception when called for any interface type that doesn't explicitly implement this.

There are several interface types in python-can which don't, including e.g. socketcan.

This breaks other tools which relied on the previous behaviour of "doing nothing" when flush_tx_buffer() was called on one of these interfaces.

To Reproduce

Call flush_tx_buffer() on an interface type that doesn't explicitly implement this method.

I noticed this because the tool pydronecan no longer works for socketcan if python-can is installed on the system.

Expected behavior

I expect the old behaviour where flushing does nothing. This can easily be done by removing the raise line, or more complicated, by implementing flush_tx_buffer() with a pass for all interface types in this repo where flushing is irrelevant.

If behavour was like this from the beginning, I would be OK with raising error. However, I think it's problematic when previous versions didn't behave like this, causing problems for tools like pydronecan. The behaviour of interfaces like this should be kept stable.

Additional context

This problem exists for every version after the PR linked above was merged. All releases after 4.4.0 are affected.

Lingua principale
Python
Stelle
1.6k
Fork
697
Nessuna PR unita negli ultimi 30g

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di hardbyte/python-can

Tutte le issue di hardbyte/python-can

Issue simili

Altre issue su Python

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.