New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
MM writes an RelayEvent E0 record with an empty total_athlete_age field #16
Comments
@tdsmith : Any chance I could get you to patch this and make a release? I can supply a PR if that's easier for you, but it's a tiny change. Thanks! |
I happen to be laptopless this week so I'd really appreciate a PR and I can
merge it speedily!
…On Sun, Feb 4, 2024, 11:05 Eric V. Smith ***@***.***> wrote:
@tdsmith <https://github.com/tdsmith> : Any chance I could get you to
patch this and make a release? I can supply a PR if that's easier for you,
but it's a tiny change. Thanks!
—
Reply to this email directly, view it on GitHub
<#16 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AABKOQNHGEK3RLGMMTKCBSTYR6PSZAVCNFSM6AAAAABCOTMKZCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRVG44DMNRUGM>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
ericvsmith
added a commit
to ericvsmith/sdif
that referenced
this issue
Feb 5, 2024
…nager sometimes writes an empty value.
Done! See #17. With this change, my sample file is parsed correctly. |
tdsmith
pushed a commit
that referenced
this issue
Feb 5, 2024
…ometimes writes an empty value. (#17)
24.2.1.post0 solves my problem. Thanks again! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This record:
was written by MM 8. The
total_athlete_age
field is all blanks (columns 35-37). But it's marked as required, so my program blows up.The odd thing is that this is line ~1500 in a results file, and this is the first E0 record with this problem. Before this line there are hundreds of E0 records with
0
for this field in the file, I can't figure out why this one is blanks.@tdsmith, could you mark RelayEvent.total_athlete_age as being optional, with
override_m1=True
? Thanks.The text was updated successfully, but these errors were encountered: