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

[CS2103T-T08-1] ImmuniMate #84

Open
wants to merge 497 commits into
base: master
Choose a base branch
from

Conversation

jovantanyk
Copy link

As the number of patients a General Practitioner grows, information management might prove complex, especially so for personal data. ImmuniMate offers a way to record comprehensive information about every patient, while ensuring timely updates and avoiding duplications/contradictions. It also seeks to establish links between patient for contact tracing and finding potential infectious clusters.

Copy link

codecov bot commented Feb 25, 2024

Codecov Report

Attention: Patch coverage is 28.80658% with 173 lines in your changes are missing coverage. Please review.

Project coverage is 63.04%. Comparing base (04ff11d) to head (adacfc2).

❗ Current head adacfc2 differs from pull request most recent head 67fa676. Consider uploading reports for the commit 67fa676 to get more accurate results

Files Patch % Lines
...a/seedu/address/logic/commands/ClusterCommand.java 0.00% 28 Missing ⚠️
...ava/seedu/address/logic/commands/CheckCommand.java 0.00% 27 Missing ⚠️
...eedu/address/logic/commands/DeleteInfoCommand.java 51.85% 12 Missing and 14 partials ⚠️
.../model/person/AddressDiagnosisStatusPredicate.java 0.00% 23 Missing ⚠️
...edu/address/logic/parser/ClusterCommandParser.java 0.00% 19 Missing ⚠️
...ddress/model/visit/VisitContainsNricPredicate.java 0.00% 11 Missing ⚠️
src/main/java/seedu/address/ui/PersonCard.java 0.00% 9 Missing ⚠️
...seedu/address/logic/parser/CheckCommandParser.java 0.00% 7 Missing ⚠️
src/main/java/seedu/address/logic/Messages.java 14.28% 6 Missing ⚠️
.../seedu/address/logic/parser/FindCommandParser.java 0.00% 6 Missing ⚠️
... and 6 more
Additional details and impacted files
@@             Coverage Diff              @@
##             master      #84      +/-   ##
============================================
- Coverage     65.27%   63.04%   -2.23%     
- Complexity      547      579      +32     
============================================
  Files            96      104       +8     
  Lines          2174     2392     +218     
  Branches        242      285      +43     
============================================
+ Hits           1419     1508      +89     
- Misses          678      798     +120     
- Partials         77       86       +9     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

alex-setyawan and others added 29 commits April 3, 2024 13:41
DeleteInfoCommandParser: change to using argmultimap to verify presence of NRIC
Adjust Person for testing
Update user guide with all features
Fix finding profiles by condition, add tests for condition predicate
NatLeong and others added 30 commits April 15, 2024 23:03
Change return value to true for hasPerson().
- precondition involve 'log in to the system' - change to launched the system;
- change to IMS.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants