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

Items from 2021-09-23 meeting #151

Closed
4 of 6 tasks
ahouseholder opened this issue Sep 23, 2021 · 6 comments
Closed
4 of 6 tasks

Items from 2021-09-23 meeting #151

ahouseholder opened this issue Sep 23, 2021 · 6 comments
Assignees
Labels
enhancement New feature or request

Comments

@ahouseholder
Copy link
Contributor

ahouseholder commented Sep 23, 2021

Followup to #149 with a few more items from our conversation this morning.

Vector

PDF

  • PDF output should have "Recommendation" as the first item under Summary

JSON decision_points

  • should allow for "simple" (no dependencies) or "complex" (which depend on answers to other decision points)
  • idea: for "complex" decisions, the "options" structure could mirror the decision table structure e.g., a list of items such as { "situated safety impact": "Major", "mission impact":"MEF Failure", "output": "High"}
  • should be flat, not nested (i.e., decision_points is just a list of simple and complex things. The complex ones will need to refer to the simple things of which they are composed, but the simple ones don't need to say anything about how they show up in complex things. This should allow more composability without overly increasing complexity in the future.
  • Should "complex" types should be composable? I.e., complex(A) depends on complex(B) and simple(C); complex(B) depends on simple(D) and simple(E). This seems like something that's going to come up, although the judges will also accept YAGNI as a counterargument.
@ahouseholder ahouseholder added the enhancement New feature or request label Sep 23, 2021
@ahouseholder
Copy link
Contributor Author

If I've missed anything in the above, please either edit the description directly or leave a comment here with additions.

@sei-vsarvepalli
Copy link
Contributor

The current schema allows for a child identifier in the schema itself

"decision_type": {
"type": "string",
"enum": [
"child",
"complex",
"simple",
"final"
],

In the new setup, I will get rid of it or basically collapse "child" into "simple" as you guys suggested. Comment if you "disagree" or want to change it.

Currently tree structure also has the "children" which is the recursive decision points the way I explained. But I think the preference is to leave this flat.

"children": {
"type": "array",
"items": { "$ref": "#/definitions/decision_points" },
"minItems": 1
}

If I change this, the children will only point to the child either by "shortcut key" decision_point.key or by full decision_point.label using anyOf in JSON schema setup. Just making sure the schema changes are understood and digested.

@sei-vsarvepalli
Copy link
Contributor

Should we preserver also deeper information on choices made by analyst. For e.g., Public Well-being Impact in the current tree is complex and can use multiple reasons for picking a particular "option" . An analyst choosing "Material" option for "Public well-being impact" can be either be due to perceived "Physical harm" to be "Physical distress or injuries for users of the system OR introduces occupational safety haz-ards OR reduction and/or failure of cyber-physical system’s safety margins." or "Financial" harm to be "Financial losses that likely lead to bankruptcy of multiple persons."
Public-well-being-from-word

@sei-vsarvepalli
Copy link
Contributor

A small schema update is pending for this to be complete. The schema update needs to represent the current efficient way to have child decisions (branches) of an SSVC decision tree to end up with a cumulative path on the decision tree. Hopefully can get that tested next week.

@j---
Copy link
Collaborator

j--- commented Jun 27, 2022

We decided that all decisions made by the analyst should be stored.
I think that is implemented in #172 ? Is this issue resolved?

@ahouseholder
Copy link
Contributor Author

Given @j---'s comment above, I'm going to close this. If there is anything remaining to be done, please open a new issue or discussion as appropriate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants