Blank credits pass validation but cause valid grade records to disappear
Maintainer thường phản hồi trong vòng 2 ngày
@vivek-3728 đang làm issue này rồi.
Từ ngày 20/10/2026.
Đánh giá
Issue này chưa được đánh giá.
Mô tả
I noticed that the grade tracker accepts blank credits during validation but silently skips the same records when loading them for other commands.
For example, I created a CSV containing:
student_id,subject,score,max_score,date,credits,semester
S1,Mathematics,95,100,2026-10-06,,1
Then I ran:
python -m grade_tracker validate
python -m grade_tracker list
python -m grade_tracker summary S1
The results were inconsistent:
validatereported that the CSV was valid.listshowed no grades.summary S1reported no records for the student.- JSON output commands failed while converting the blank credits value.
I expected all commands to treat blank credits as the documented default of 3.0, so the Mathematics assessment would appear normally in listings, summaries, GPA calculations, and exports.
Looking through the code, the validator explicitly replaces blank credits with "3.0". However, the normal loader uses:
float(row.get("credits", 3.0))
Since the credits field exists but contains an empty string, dict.get() returns "" instead of the default. This causes float("") to fail, and the loader silently skips the record.
This means a CSV can pass validation while valid assessments disappear from calculations and reports without any warning.
I expected validation and all data-loading commands to apply the same default-credit rules consistently, rather than interpreting the same CSV differently.
- Ngôn ngữ chính
- Python
- Star
- 0
- Fork
- 5
- Merge trung bình
- 1 ngày 15 giờ
- Pull request đã merge (30 ngày)
- 27
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Không có hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- 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.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của techcsispit/grade-tracker
-
add accepts Infinity scores and writes records that immediately fail validationCó thể đã có người làm @vivek-3728 đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
techcsispit/grade-tracker#69 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 2 ngày
-
feat: add cross-file integrity verification and repairCó thể đã có người làm @deepmhatre13 đã nhận hôm nay. Đang mởfeature
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
techcsispit/grade-tracker#66 ·
Maintainer thường phản hồi trong vòng 2 ngày
Tất cả issue của techcsispit/grade-tracker
Issue tương tự
-
Support .aac audio uploadsCó thể đã có người làm @julien731 đã nhận hôm nay. Đang mởfeature
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
nimblehq/audio-transcriber#153 ·
-
help wanted
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
antonyrag/ragleap-core#1062 ·
-
needs-triage
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
gke-labs/kube-agents#4321 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
enhancement help wanted
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
Git-Hospital/githealthcheck#61 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
changelog investigate
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 67/100
ramnes/notion-sdk-py#433 ·