harrisonchiu / xray

MIT License
1 stars 0 forks source link

Peer Review - Non-abstract FR #20

Open RezaJodeiri opened 1 month ago

RezaJodeiri commented 1 month ago

Artifact Under Review

Functional Requirement FR2 from the project requirements specification.

Team Number for Team Doing the Review

Team 27

Description of Issue

It seems like the current wording of FR2 might be leaning a bit into the how rather than the what by specifying the use of a "diffusion model" for processing CXR images.

image

A functional requirement typically focuses on the outcome — what the system needs to do — without getting into specific implementation details. Maybe rewording FR2 to focus on the goal of detecting disease-specific patterns could make it more abstract and aligned with the role of functional requirements.

RezaJodeiri commented 1 month ago

@Tan630