search

AhmedAl-Hayali / GenreGuru

Music Featurization, Recommendation, and Generation
MIT License
3 stars 1 forks source link

Peer Review - Vague verification plans #182

Closed emilyperica closed 1 week ago

emilyperica commented 2 weeks ago

Artifact Under Review docs/VnVPlan/

Team Number for Team Doing the Review Team 7

Description of Issue The checklists associated with each verification can be easily misinterpreted due to their lack of specific benchmarks. For example, in section 3.3.3 one of the checklist items is ‘Processing time is optimized for expected load’ - neither ‘optimized’ nor ‘expected load’ are given specific values. If these metrics are explained in other sections or previous project documents then they should be referenced accordingly.

AhmedAl-Hayali commented 1 week ago

Feedback captured in #159 #158 #157 #156 #155, and #154.