RFC: cleanup of sdcard.py code

Offen
#763 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Anfängerfreundlichkeit
20/100
Issue-Typ
Refactoring
Klarheit
Muss geklärt werden
Aktivitätsstatus
Veraltet
Tech-Stack
python
Bereich
embedded-iot

Rechercherichtung

Beginne mit dem Modul sdcard.py, der verlinkten Diskussion und dem beigefügten Prototypen sdcard_20231102_0752.zip. Prüfe die vorgeschlagenen Änderungen an adaptivem Timeout und CRC in diesem Kontext und hole dann Feedback der Maintainer ein, bevor du festlegst, was als abgeschlossen gelten soll; das Issue legt keinen akzeptierten Umfang und keine Tests fest.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

The sdcard.py module works, but seems a bit incomplete and slow. After quite a bit of discussion,
https://github.com/orgs/micropython/discussions/12687
I have prepared a prototype of a newer scheme for it. It is much faster, since it uses an 'adaptive' timeout scheme (zero delays unless a bunch of fast delay loops have passed without data). It correctly implements crc7 on all commands, and allows the use of crc16 on data.

I have forked the repo, and am preparing a pull request. Before I go there, I would appreciate any commentary from the PTB about the changes. I am attaching a zip with the current status.

Thanks for any comments.

sdcard_20231102_0752.zip

Vorherrschende Sprache
Python
Sterne
2.9k
Forks
1.1k
Ø Merge
7 T. 6 Std.
Gemergte PRs (30 T.)
3

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus micropython/micropython-lib

Alle Issues in micropython/micropython-lib

Ähnliche Issues

Weitere Issues zu Python

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.