fix(dcmjs): Add a set of accessors to the sequence list so the API is more consistent

[wayfarer3130]

This is another attempt to make the naturalize data objects more consistent. This change minimally affects the API, but makes it possible to consistently be accessed, rather than having the naturalize/denaturalize produce inconsistent results.
There are no circular reference here. The change makes sequence objects remain lists, but adds an accessor so that child properties can consistently be accessed on the top level object. That is, suppose the per frame groups is a sequence with 1 element, then:

const seq = elements.PerFrameFunctionalGroupsSequence
expect(seq.length).to.equal(1)
Object.keys(seq[0]).forEach( key => expect(seq[key]).to.equal(seq[0][key]);

Also, if you modify any existing key of seq[0], it will be reflected in seq, and vice-versa. however, adding a key to seq[0] will not automatically add it to seq or vice-versa.

[pieper]

thanks for putting this together. I think you and Rudolfo are both at Radical, right? Did you confirm with him that this approach will address the issues he had before?

[ladeirarodolfo]

Thanks for the implementation much better :D. I have added only some minor inputs

[wayfarer3130]

@pieper - Rodolfo and I are agreed on this one, as he state, so is this one ready to go in now? Fixes a number of related bugs in OHIF, so I'm hoping to get it in soon.

[pieper]

LGTM - thanks for working through all this. 👍

[ohif-bot]

🎉 This PR is included in version 0.18.11 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀