search

tusharagg1 / chest-x-ray-ai

AI for chest X-ray read
MIT License
4 stars 1 forks source link

VnV Plan [Final Doc] #136

Closed tusharagg1 closed 7 months ago

tusharagg1 commented 7 months ago

Feedback from Avenue

Please update the concerned documents based on the feedback from the Professor and TA on Avenue. Important: Also, update the document with the latest information based on our Final Demo/Product.

VnV Plan

  • [x] VnV Plan revised (complete unit testing sections) - @AllisonRCook
    • [x] Summarize unit testing philosophy -@AllisonRCook
    • [x] Point to unit testing code - @AllisonRCook
  • [ ] Change the paper number to be consistent - see this comment
  • [x] Using test-id for all test, should be changed - @AllisonRCook
  • [x] Missing static test - @AllisonRCook
  • [x] An excellent job has been doing addressing the feedback from the Tas, instructor, supervisor (when present), teammates, other teams, and test users. The feedback has been conscientiously, completely and consistently addressed. Unit test summary added to VnV plan. The changes are clearly and completely documented.
  • [x] Update major revisions table.
  • [x] Check the Important note above - @AllisonRCook
  • [x] Refer to the rubric - @AllisonRCook
  • [x] Reviewed by @Nathaniel-Hu

Originally posted by @tusharagg1 in #130

tusharagg1 commented 7 months ago

@Ibrahimango02 @Nathaniel-Hu Add features from below: https://github.com/tusharagg1/chest-x-ray-ai/settings/security_analysis

AllisonRCook commented 7 months ago

Unsure what "consistent page numbers" need to change as all values are present and consistent with the other existing documents