Skip to content
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.

Sign up for GitHub

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

Jump to bottom

reader: handle unknown messages and fields better #6

Open
tormoder opened this issue Sep 29, 2015 · 0 comments
Open

reader: handle unknown messages and fields better #6

tormoder opened this issue Sep 29, 2015 · 0 comments
Labels
enhancement
Milestone
Unplanned

Comments

@tormoder
Copy link
Owner

Include definition messages in addition to byte payload of unknown fields. Maybe make this a decode option.
@tormoder tormoder added this to the Unplanned milestone Sep 29, 2015
tormoder added a commit that referenced this issue Oct 11, 2015
@tormoder
testdata: Add file from dtcooper/python-fitparse/#6
3330c2e
tormoder added a commit that referenced this issue Apr 2, 2017
@tormoder
reader: add DecodeOptions
a9e486f 
Initial options are for optional logging and decoding of
fields/messages. A follow-up commit will enable them for the decoder.

Updates #6.
tormoder added a commit that referenced this issue Apr 2, 2017
@tormoder
unknown: refactor handling of unknown fields and messages
edf5276 
Must now be enabled using DecodeOptions.

Updates #6.
@tormoder tormoder modified the milestones: Unplanned, fit release 0.3.0 Apr 3, 2017
@tormoder tormoder changed the title Decode: Handle unknown messages and fields better reader: handle unknown messages and fields better Apr 3, 2017
@tormoder tormoder modified the milestones: fit release 0.3.0, Unplanned May 10, 2019
@tormoder tormoder mentioned this issue Nov 10, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
Unplanned
Development

No branches or pull requests
1 participant
@tormoder