TPGEngine / tpg

Evolving Quadruped Robot Controllers with Emergent Tangled Program Graphs
MIT License
0 stars 0 forks source link

Peer Review - Ambiguity in terms used in System Boundaries and Components and elsewhere #134

Open N4T10N01 opened 1 month ago

N4T10N01 commented 1 month ago

Artifact Under Review Critical Assumptions

Team Number for Team Doing the Review Team 2

Description of Issue

Mainly in your "System Boundaries and Components" section (although also in "Critical Assumptions" and "Safety and Security Requirements"), you make reference to "agents" and "environment". It is not entirely clear what "agents" or "environment" are outside of your SRS. For the sake of the part of the rubric which demands the hazard analysis "has all required elements... [including] supporting material", I would recommend either these terms get defined explicitly within the document or there is a hyperlink provided back to a section within the SRS which defines them. It is somewhat unclear what an "agent" directly refers to without additional context, although environment could probably just become "simulation environment" to clarify.

cyrussamante commented 1 month ago

Thanks for the feedback. We have decided to refer to the SRS documentation to act as a single source of truth for terminology.