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.

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

Consider Removing ListTypes #100

Open
ikiril01 opened this issue Mar 11, 2015 · 1 comment
Open

Consider Removing ListTypes #100

ikiril01 opened this issue Mar 11, 2015 · 1 comment
Assignees
Milestone

Comments

@ikiril01
Copy link
Member

It may be worthwhile to consider removing the existing *ListTypes in MAEC that serve to capture lists of individual elements, and just replacing their usage with the embedded type with a multiplicity of 0..N. This will serve to reduce the hierarchical overhead in MAEC XML instances, but it may have some unintended consequences (e.g., with JSON output), so it should be carefully considered.

@ikiril01
Copy link
Member Author

ikiril01 commented Apr 8, 2015

This should likely be done on a case-by-case basis, as doing it globally would produce some ugly XML. One possible rule is to do it for ListTypes that encompass SimpleTypes, such as the CommentListType, in order to reduce clutter and maintain some organization around the ComplexTypes.

E.g., instead of

<Analysis>
     <Comments>
         <Comment>text</Comment>
         <Comment>other text</Comment>
    </Comments>
</Analysis>

we could have:

<Analysis>
     <Comment>text</Comment>
     <Comment>other text</Comment>
</Analysis>

@ikiril01 ikiril01 self-assigned this Apr 10, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant