micropython / micropython/micropython-lib
`logging.FileHandler` not adding new lines to log file
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 2.9k
- Forks
- 1.1k
- Ø Merge
- 7 T. 6 Std.
- Gemergte PRs (30 T.)
- 3
Beschreibung
Just tried the updated logging module to replace the StreamHandler and FileHandler adapted from CPython that I was previously using. Thank you @iabdalkader for this update.
StreamHandler works as expected, however I noticed that I wasn't getting any new lines in my log file.
I looked at the previous code and saw that there's a flush after the write in StreamHandler.emit.
Using this subclass for FileHandler, it works like before:
class MyStreamHandler(StreamHandler):
def flush(self):
if self.stream and hasattr(self.stream, "flush"):
self.stream.flush()
def emit(self, record):
super().emit(record)
self.flush()
Is there a reason for not doing a flush in emit, maybe something specific to MicroPython that I'm not aware of?
- MicroPython and board:
MicroPython v1.19.1-915-g2bcd88d55 on 2023-03-02; Raspberry Pi Pico W with RP2040 loggingversion:0.5installed withmip
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne damit, das Problem mit dem aktualisierten Logging-Modul auf dem gemeldeten MicroPython Raspberry Pi Pico W-Setup zu reproduzieren. Vergleiche anschließend FileHandler mit StreamHandler.emit und der vorherigen angepassten Implementierung. Erledigt ist die Aufgabe, wenn die FileHandler-Logausgabe wie erwartet verfügbar ist, ohne die benutzerdefinierte flush-Unterklasse; prüfe vorhandene Logging-Tests, falls es welche gibt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- observability
- Issue-Typ
- Bug
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 45/100