RFC: cleanup of sdcard.py code

Đang mở
#763 2 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Đánh giá

Độ khó
5/5
Thời gian dự kiến
Hơn một tuần
Mức phù hợp với người mới
20/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Cần làm rõ
Mức độ hoạt động
Đình trệ
Công nghệ
python
Lĩnh vực
embedded-iot

Hướng nghiên cứu

Bắt đầu với module sdcard.py, cuộc thảo luận được liên kết và prototype sdcard_20231102_0752.zip đính kèm. Xem xét các thay đổi được đề xuất đối với adaptive timeout và CRC trong bối cảnh đó, sau đó tìm kiếm phản hồi từ các maintainer trước khi xác định việc hoàn thành nên có nghĩa là gì; issue không chỉ rõ phạm vi được chấp nhận hoặc các bài kiểm thử.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

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

Ngôn ngữ chính
Python
Star
2.9k
Fork
1.1k
Merge trung bình
7 ngày 6 giờ
Pull request đã merge (30 ngày)
3

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của micropython/micropython-lib

Tất cả issue của micropython/micropython-lib

Issue tương tự

Thêm issue về Python

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.