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

think about a deserialization profile #479

Closed

Conversation

maxhbr
Copy link
Member

@maxhbr maxhbr commented Aug 17, 2023

this was created when thinking about #403 (comment)

## Metadata

- name: Deserialization
- SubclassOf: /Core/Artifact
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I though about suggesting a subclass of File, but after thinking about it we would want to cover the case where something is serialized as a network stream.


Propreties should be used as follows:

- element, to encode the content that was generated by when parsing the file
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm a bit confused about the element property description - would this reference that actual elements in the graph deserialized from the file?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought that this would reference all element objects, that were created in the de-serialization process

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense - I may suggest some word smithing if we go down this route.

@goneall goneall added the serialization Something about the representation of data in bytes label Aug 17, 2023
@goneall goneall added this to the 3.0-rc2 milestone Aug 17, 2023
@goneall
Copy link
Member

goneall commented Aug 17, 2023

Would I be able to handle the following scenario?:

  • A single "Payload" is created by aggregating several existing previous Payloads (the commonly request merge function for the SPDX Tools) Note that the namespace map used for this merged Payload is irrelevant to this scenario.
  • You then want to "break apart" the single Payload back to the individual previous Payload and retain their original namespace maps - in this scenario, you don't care about the namespace map for the merged Payload

@maxhbr
Copy link
Member Author

maxhbr commented Sep 21, 2023

Discussions moved along and are no happening in a different place. Closing this issue. In discussions it is now called the X-Collection

@maxhbr maxhbr closed this Sep 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
serialization Something about the representation of data in bytes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants