Skip to content

Conversation

@umm0
Copy link
Collaborator

@umm0 umm0 commented Sep 5, 2025

Draft pull request to discuss export control classification

@bact bact marked this pull request as ready for review September 5, 2025 15:10
@umm0 umm0 marked this pull request as draft September 8, 2025 13:54
@umm0
Copy link
Collaborator Author

umm0 commented Sep 8, 2025

I marked this pull request again as draft since there is still a discussion happening in the Operations workgroup on how to best represent the data.
It should not be merged, even if checks would pass, before this discussion is concluded.

## Metadata

- name: ExportControlClassificationAssessment
- SubclassOf: /Core/Element
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
- SubclassOf: /Core/Element
- SubclassOf: /Core/Action


## Description

Weight to express relevance in de minimis consideration.
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Operations meeting 25-09-26: A better description is needed to clarify use and context.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Definition of the weight (e.g. specification, type,...) should be in the Assessment.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

needs to be consistent for all classifications in one assessmentResult.

@stevenc-stb stevenc-stb added this to the 3.1 milestone Oct 10, 2025
Copy link

@mkurzman mkurzman left a comment

Choose a reason for hiding this comment

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

No blockers from my side.

@umm0 umm0 marked this pull request as ready for review October 28, 2025 15:39
@umm0 umm0 marked this pull request as draft October 28, 2025 15:40
@umm0
Copy link
Collaborator Author

umm0 commented Oct 28, 2025

rebase to current 'develop' state planned before merge

@umm0 umm0 force-pushed the operations-export-classification branch 2 times, most recently from 46ada0f to c721835 Compare November 2, 2025 10:46
@umm0 umm0 force-pushed the profile-operations branch from ed12e58 to 8094d37 Compare November 2, 2025 10:53
umm0 and others added 4 commits November 2, 2025 12:02
Signed-off-by: Ummo Schwarting <uschwarting@deloitte.de>
Co-authored-by: stevenc-stb <steven@smarttalkbeacon.com>
Signed-off-by: umm0 <uschwarting@deloitte.de>
Co-authored-by: stevenc-stb <steven@smarttalkbeacon.com>
Signed-off-by: umm0 <uschwarting@deloitte.de>
Signed-off-by: Ummo Schwarting <uschwarting@deloitte.de>
@umm0 umm0 force-pushed the operations-export-classification branch from c721835 to dce63a2 Compare November 2, 2025 11:02
umm0 added 5 commits November 2, 2025 20:07
Signed-off-by: Ummo Schwarting <uschwarting@deloitte.de>
Signed-off-by: Ummo Schwarting <uschwarting@deloitte.de>
Signed-off-by: Ummo Schwarting <uschwarting@deloitte.de>
Signed-off-by: Ummo Schwarting <uschwarting@deloitte.de>
…implementations

Signed-off-by: Ummo Schwarting <uschwarting@deloitte.de>
@umm0 umm0 marked this pull request as ready for review November 2, 2025 19:32
@umm0
Copy link
Collaborator Author

umm0 commented Nov 2, 2025

Hi @mkurzman, @stevenc-stb

I rebased the profile operations branch and the source for this pull request onto 'develop'.
I also fixed some smaller things.

I suggest merging the pull request as it is now to have the profile-operations branch updated to the current discussion state. Your opinion?

Edit:I added the suggestions from the discussion here to a separate branch so they are kept: https://github.com/umm0/spdx-3-model/pull/1/files

Best,
Ummo

umm0 added a commit to umm0/spdx-3-model that referenced this pull request Nov 2, 2025
Signed-off-by: Ummo Schwarting <uschwarting@deloitte.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants