The main from ay1920s1-cs2113-t14-1

Comments from User Guide

I'm cleaning up the User Guide to make it more user-friendly. I left a few comments/notes about implementation lying around while writing it. I'm removing them to store them here.

Since there is a lot of fairly complex data to manage, the input system is stateful - different commands mean different things based on the current context.
This would require some tracking and interfacing between the UI and the Logic (e.g. the internal logic needs to know which patient is being displayed as patient #1). This can be done by creating an array of integers mapping the on-screen indices to the internal ArrayList indices.
If user input is ambiguous, we can show a window of possible options to disambiguate between, or give the user the option to edit the text in a buffer. The disambiguation window and code can be reused - create an abstract DukeObject class for our data items.
For switches, disambiguation can be used as confirmation if part of the input matches the first part of the switch, but subsequent parts are wrong.
next and prev can be removed if we have a scrolling display on the GUI
All search and disambiguation functions should be able to run on essentially the same interface: window of matching results, each with an index, and the ability to navigate to each specific result
Should we have autocomplete for commands too?

As a house officer who has to manage a lot of information, I can easily view and navigate through data associated with particular conditions that particular patients have, so that I can have a clearer view of what that particular condition is.

Role: House officer
Function: Easily view and navigate through data associated with particular conditions that particular patients have
Benefit: Can have a clearer view of what that particular condition is

Peer Evaluation Round 1

Peer Evaluation Round 1 will be open on TEAMMATES. You should have received the submission link. If you do not submit feedback responses, you will forfeit participation marks that you earned based on feedback ratings received from others.

The expectations are two fold. First, we hope you will give a honest and constructive feedback to your teammates to help them improve. Second, we hope that you will use the feedback you received to be an effective team player.

Create a template for the reports

As a house officer who needs to quickly input a lot of data and is prone to mistyping, I can make mistakes when typing commands but still have them accepted.

#43 Format checks for fields like dates, and values like medicine names
#42 Undo previous commands
#39 Autocorrect for commands
#41 Confirmation for autocorrect, and option to edit input if autocorrect has not parsed user's intention correctly

As a house officer keeping track of information for my consultant, I can keep track of whether or not I’ve checked for the results of certain investigations so that I can make sure the consultant is kept up-to-date.

Role: House officer
Function: Keep track of whether or not I’ve checked for the results of certain investigations
Benefit: Can make sure the consultant is kept up-to-date

Create disambiguation window GUI

Create the GUI for #85

A DateTimeParser class to handle parsing of dates and times

Currently, all parsing of dates and times are executed through TimedTask class. A DateTimeParser class would ensure that the single responsibility principle is not violated.

Rapid input parsing

We need to design, essentially, a finite-state machine that allows the user to rapidly navigate between different fields and input data, controlling with just a few characters. I can work on the design for now, but the actual implementation may take some time.

Developer Guide

Should have the user profile, value proposition, and an appendix containing user stories, use cases and NFR.

Support contextual parsing of commands

The same command in two different contexts can do two different (conceptually similar) things, with a different argument signature. We need a way to differentiate between them.

Create a chatbot interface

UI elements

Chat window
Input box
Dialog

User Guide

Should have the concrete command syntax and explanation for features that you would implement (in v1.4) and the features that are planned for v2.0.

Create disambiguation window logic

A general purpose disambiguation screen is required for the functioning of a lot of commands. It should be able to display the string representation of several DukeObjects and allow the user to select them by index in order to execute the Command on them.

Fully integrate JavaFX into Dr. Duke

Give Dr. Duke a basic GUI with JavaFx.

As a house officer in a hurry, I can quickly access the relevant data of the patient I am seeing.

#37 View previous medical history: view investigation results, courses of medicine completed, general notes on medical history, completed plan items
#35 View and navigate data associated with each condition: go from viewing the impression to a detailed view of some evidence or treatment, and vice versa
#30 Check allergies: display allergies and other vital data on patient's main page, and maybe in a little box at the side even when viewing/entering other kinds of data
#33 Search for relevant data: search through treatments/evidence for a string?

Feedback on week 7 project progress for CS2113-T14-1

Team progress for v1.1

Individual progress of @aquohn for v1.1

Team repo forked to your GitHub account(:heavy_check_mark: well done!)
Photo uploaded(:heavy_check_mark: well done!)
PR created: #69 #68 #16 #15 #6 #10 #55 #58 #57 #23 #19 (:heavy_check_mark: well done!)
You have reviewed at least 1 PRhttps://github.com//pull/65 #64 #67 #12 #51 #17 #11 #8 #5 #62 #63 #61 #60 #56 (:heavy_check_mark: well done!)

Individual progress of @JeremyKwok for v1.1

Team repo forked to your GitHub account(:heavy_check_mark: well done!)
Photo uploaded(:heavy_check_mark: well done!)
PR created: #21 #18 (:heavy_check_mark: well done!)
You have reviewed at least 1 PR(:exclamation: try to do by next milestone)

Individual progress of @gowgos5 for v1.1

Team repo forked to your GitHub account(:heavy_check_mark: well done!)
Photo uploaded(:heavy_check_mark: well done!)
PR created: #65 #64 #67 #51 #17 #11 #8 #5 #62 #61 #60 #56 #25 #3 #13 #9 #4 #1 (:heavy_check_mark: well done!)
You have reviewed at least 1 PRhttps://github.com//pull/68 #12 #6 #55 #58 #57 #23 (:heavy_check_mark: well done!)

Individual progress of @JacobToresson for v1.1

Team repo forked to your GitHub account(:heavy_check_mark: well done!)
Photo uploaded(:heavy_check_mark: well done!)
PR created: #12 (:heavy_check_mark: well done!)
You have reviewed at least 1 PRhttps://github.com//pull/21 (:heavy_check_mark: well done!)

Tutor: @sanjukta99

_{Note: the above observation was done by the CS2113-feedback-bot and covers changes up to 2019-10-03 00:00:00 only. If you think the above observation is incorrect, please let us know by replying in this thread. Please include links to relevant PRs/comments in your response.}

Event updated to be between two times

Latest merge changed a fundamental part of the data model: events are now between two specific times rather than at a specific time. I'll sort out the fallout, especially in the test code - in the meantime, don't touch any events, especially not those in the test code.

Provide autocomplete functionality for commands

On top of having a HashMap to look up commands, should we have an array of valid commands, and we see what commands could possibly match with the string that was input? E.g.

int n = cmdStr.length();
for (String validCmd : validCmdArr) {
  if (n <= validCmd.length()) {
    if (cmdStr.equals(validCmd.substring(0, n))) {
      if (validCmd.length() == n) {
        //exact match for valid command, execute the command here
        break;
      } else {
        //add validCmd to array of possible valid command strings to display to user here
      } 
    }
  }
} 
//if by the end of the loop there is only one valid command in the array, prompt the user if this is what he wants to do

Feedback on week 6 project progress for CS2113-T14-1

Leftovers from the previous week

You are done with previous weeks work(:heavy_check_mark: well done!)

Progress of this week

Individual progress of @aquohn

Team repo forked to your GitHub account(:heavy_check_mark: well done!)
You have reviewed at least 1 PR(:heavy_check_mark: well done!)

PR created for:

[CS2113-T14-1]-aquohn-A-Jar
[CS2113-T14-1]-aquohn-A-Jar
[CS2113-T14-1]-aquohn-B-RecurringTasks

Individual progress of @JeremyKwok

Team repo forked to your GitHub account(:heavy_check_mark: well done!)
You have reviewed at least 1 PR(:exclamation: try to do by next milestone)

PR created for:

[CS2113-T14-1]-JeremyKwok-B-DoWithinPeriodTasks
[CS2113-T14-1]-JeremyKwok-B-FixedDurationTasks
[CS2113-T14-1]-JeremyKwok-B-DoWithinPeriodTasks
[CS2113-T14-1]-JeremyKwok-B-FixedDurationTasks

Individual progress of @gowgos5

Team repo forked to your GitHub account(:heavy_check_mark: well done!)
You have reviewed at least 1 PR(:heavy_check_mark: well done!)

PR created for:

[CS2113-T14-1]-gowgos5-B-ViewSchedules
[CS2113-T14-1]-gowgos5-B-Reminders
[CS2113-T14-1]-gowgos5-A-Travis

Individual progress of @JacobToresson

Team repo forked to your GitHub account(:heavy_check_mark: well done!)
You have reviewed at least 1 PR(:exclamation: try to do by next milestone)

PR created for:

[CS2113-T14-1]-JacobToresson-B-Snooze

Tutor: @sanjukta99

_{Note: the above observation was done by the CS2113-feedback-bot and covers changes up to 2019-09-19 00:00:00 only. If you think the above observation is incorrect, please let us know by replying in this thread. Please include links to relevant PRs/comments in your response.}

As a house officer who needs to upload records into the hospital’s health system, I can generate unified reports that are fully compatible with the system so that I don't have to manually input those records.

Role: House officer
Function: Generate unified reports that are fully compatible with the hospital’s health system
Benefit: Don't have to manually input those records

As a house officer who has to manage many patients, I can easily view all critical issues all my patients are facing by level of importance so that I can address them as soon as possible.

Role: House officer
Function: Easily view all critical issues all my patients are facing by level of importance
Benefit: Can address them as soon as possible

Create parser core

Required for all parsing-related stories.

Create a parser class that is able, given an input string, a command name, and a list of switches with the required parameters, to:

identify if the string is a valid command
if not, throw an error identifying the first encountered error condition
if so, extract the values supplied for each switch

As a house officer who needs to quickly input a lot of data and is prone to mistyping, I can make typing errors but still have my input recognised so that I don’t have to waste time to retype my command.

Role: House officer
Function: Make typing errors but still have my input recognised
Benefit: I don’t have to waste time to retype my command

Update UI mock-up

UI for v1.2

Create CRUD commands for main context

Every context allows the performance of some basic CRUD actions. Create the CRUD commands for the main context.

As a house officer who has to manage a lot of information, I can easily keep track of my patients' critical conditions or follow-up items so I can address them appropriately.

#34 Flagging Plan items and Observations as critical and displaying them on the patient's home page.
#38 View all critical Plans and Observations amongst patients being managed.

Create a window to display patients

UI elements

Scrollable list window
Panel to store patient's information

Create CRUD commands for Patient context

#75, but for the Patient context.

As a house officer who needs to quickly input a lot of data and is prone to mistyping, I can undo the previous commands so I can quickly rectify mistakes made when inputting data.

Role: House officer
Function: Undo the previous commands
Benefit: Can quickly rectify mistakes made when inputting data

Create abstract Evidence Object extends abstract DukeData

Represents evidence constituting a medical diagnosis.
An Evidence object corresponds to the notes of the doctor on a piece of medical evidence as well as an integer between 1-4 representing the priority or significance of the evidence.

Attributes:

- name: information on the evidence
- impression: the impression object the evidence is tagged to
- priority: the priority level of the evidence

Functions:

+ toString():

Returns information of the evidence which can be Displayed, Reported or Stored.
@return String the information on the evidence

+ toDisplayString():

Returns the information on the evidence formatted for display
@return String the pretty print for the evidence information

+ toReportString():

Returns the information on the evidence formatted for Report Generation
@return String the formatted evidence information for writing to a report

+ updatePriority(priorityVal: int):

@param int the integer value of the priority between 1 to 4
@return null

+ getters & setters: to access data stored in Evidence

As a house officer who needs to quickly input a lot of data and is prone to mistyping, I can have my input automatically checked to ensure it is of the right format so that I can always be assured that I am inputting the right commands.

Role: House officer
Function: Have my input automatically checked to ensure it is of the right format
Benefit: Always be assured that I am inputting the right commands

As a house officer who has to manage a lot of information...

#33 - Search through all the records of a patient
#34 - Flag and view the critical issues to follow up for each patient
#35 - Easily view and navigate through data associated with particular conditions that particular patients have
#47 - Easily link new information and follow-up items to particular conditions

Junit tests fail for RecurringTasks

As @aquohn has already noted in NewRecurringTaskCommandTest class, dates generated were 1 day off for some reason.

P.S. nice meme at NewRecurringTaskCommand class 😉

As a house officer, I can check my patients’ allergies so I can issue them with appropriate medicine.

Role: House officer
Function: Check my patients’ allergies
Benefit: Can issue them with the appropriate medicine

Create CRUD commands for Impression context

#75, but for the Impression context.

Command "done 0" will result in a program crash

Upper bound is restricted properly. However, do check lower bound as well.

As a house officer who has to manage many patients...

#37 - Easily view the previous medical history of my patients
#38 - Easily view all critical issues all my patients are facing by level of importance

As a house officer who has to manage a lot of information, I can easily link new information and follow-up items to particular conditions so that there will be a clearer picture of each condition and its management plan.

Role: House officer
Function: Easily link new information and follow-up items to particular conditions
Benefit: There will be a clearer picture of each condition and its management plan

As a house officer who needs to quickly input a lot of data and is prone to mistyping...

#39 - Can make typing errors but still have my input recognised
#41 - Confirm my input type and modify it quickly if it is incorrect
#42 - Undo my previous commands
#43 - Have my input automatically checked to ensure it is of the right format

As a house officer who needs to quickly input a lot of data and is prone to mistyping, I can confirm my input type and modify it quickly if it is incorrect so that I do not have to retype or tediously transfer entries that were input in the wrong place.

Role: House officer
Function: Confirm my input type and modify it quickly if it is incorrect
Benefit: Don't have to retype or tediously transfer entries that were input in the wrong place

Create undo and redo functionality

These commands will be available in every context, and require the maintenance of two Command stacks, one for undo and one for redo.

Program crashes when data folder is missing.

Create folder in Storage class if data folder is missing.
Add ./data/* to .gitignore

As a house officer who has to manage many patients, I can easily view the previous medical history of my patients so that I can understand what has been done to treat their conditions.

Role: House officer
Function: Easily view the previous medical history of my patients
Benefit: Can understand what has been done to treat their conditions

Feedback on week 5 project progress for CS2113-T14-1

Progress of this week

A version tagged A-Packages is in the repo(:heavy_check_mark: well done!)
A version tagged v1.0 is in the repo(:heavy_check_mark: well done!)

Tutor: @sanjukta99

_{Note: the above observation was done by the CS2113-feedback-bot and covers changes up to 2019-09-12 00:00:00 only. If you think the above observation is incorrect, please let us know by replying in this thread. Please include links to relevant PRs/comments in your response.}

Create CRUD commands for Treatment/Evidence contexts

#75, but for each Treatment/Evidence context. May require several abstract commands.

Create a report interface

To be able to toggle and view reports that have been uploaded

As a house officer who has to manage a lot of information, I can search through all of the records of a patient so that I can find all the details relevant to a particular aspect of his/her care plan.

Role: House officer
Function: Search through all of the records of a particular patient
Benefit: Find all the details relevant to a particular aspect of his/her care plan

As a house officer who has to manage a lot of information, I can flag and view the critical issues to follow up for each patient so that I can complete the follow-up as soon as possible.

Role: House officer
Function: Flag and view the critical issues to follow up for each patient
Benefit: Can complete the follow-up as soon as possible

As a house officer with a consultant that talks too fast, I can differentiate the types of input with just a single control character so that I don't have to waste time switching between windows.

Role: House officer
Function: Differentiate the types of input with just a single control character
Benefit: Don't have to waste time switching between windows

ay1920s1-cs2113-t14-1 / main Goto Github PK

main's People

Contributors

Watchers

Forkers

main's Issues

Team progress for v1.1

Individual progress of @aquohn for v1.1

Individual progress of @JeremyKwok for v1.1

Individual progress of @gowgos5 for v1.1

Individual progress of @JacobToresson for v1.1

Leftovers from the previous week

Progress of this week

Progress of this week

Recommend Projects

Recommend Topics

Recommend Org