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

peer review: <Mitchell Weingust Team 22> All Critical Assumptions Not Covered #127

Open
Tracked by #113
mitcholll opened this issue Oct 27, 2024 · 0 comments
Open
Tracked by #113
Labels
review To be used with different parties for review purposes

Comments

@mitcholll
Copy link

artifact under review.

Hazard Analysis Document

team number (for the team doing the review).

Team 22

description of issue.

Section 5. Critical Assumptions section states "No critical assumptions are being made for tinymorph that would limit the scope of mitigating or eliminating potential hazards." The following section 6.1 mentions Hazards Out of Scope, which some may fall into Critical assumptions.

You mention in the sentence below the list of 'hazards out of scope' that the first three are managed by users, which the team doesn't have control over. It would be a good idea to state that a critical assumption is "the team has no control over the user's hardware". That would justify why your chosen hazards are out of scope. This is a critical assumption as it relates to how the project's success does not depend on the user's physical devices.

@CSchank

@mitcholll mitcholll added the review To be used with different parties for review purposes label Oct 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
review To be used with different parties for review purposes
Projects
None yet
Development

No branches or pull requests

1 participant