-
Notifications
You must be signed in to change notification settings - Fork 1
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
New internal implementation of data lists (Mmif.media, Mmif.views, View.annotations) #60
Conversation
Creating this draft PR to resolve conflicts with develop and discuss code changes |
f655440
to
805ab83
Compare
Rebased off of new develop branch |
Codecov Report
@@ Coverage Diff @@
## develop #60 +/- ##
===========================================
- Coverage 96.56% 96.04% -0.52%
===========================================
Files 7 7
Lines 349 430 +81
===========================================
+ Hits 337 413 +76
- Misses 12 17 +5
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Adding some tests written during work on #60
…in the live objects
…e internal data structure away from the Mmif method logic
…el.py to avoid circular import
…eserialize super method
28235c1
to
14cbdbc
Compare
Rebased again |
Fixed merge conflicts |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
test fails and seems that there's a bug in the DataList
class. Can you take a look at it?
…tion in DataList._append_with_key. Ensures unique insertion.
26347bf
to
38ab0ca
Compare
I have implemented
__getitem__
for Mmif and View and implemented the Mmif.media, Mmif.views, and View.annotations lists internally as MmifObject::DataList objects that hide the implementation of these lists.For now, I have implemented DataLists as dicts that serialize into lists. We can continue discussing the pros and cons of using dicts vs lists here or in #40.