ssuunb3035 / team6 Goto Github PK

Currently, every course is being parsed and accounted for in the data sets. We need to ignore certain courses like Co-ops and CTNs. Parsing these increases our 'other' bin quite drastically.

The code structure for updating the gpa in the transcript might lead to the gpa being updated when it shouldn't be, and with the wrong values.

Further analysis is required to ensure that the code is working correctly.

Steps to reproduce:
-Run application.
-Click 'Parse transcripts' button
-Click 'Write to Excel' button

Expected result: The output will be compatible to be opened with the latest Microsoft excel application (xlsx).

Actual Result: The output has the wrong file extension (xsl). The output must be right clicked and 'opened-with' to work with excel, and has many error messages upon opening.

Solutions: Moving the HSSF components to XSSF should mitigate this issue.

Implement global distribution in a new excel sheet. All system functions are in place but they need to be hooked into excel

The Cohort.addTranscript() currently uses hardcoded values to determine the rank of each transcript. We must provide a way for the system to read a user-defined rank configuration for assigning the rank.

Note: The current values must be used as the default if no configuration is provided.

Steps:
-Click parse transcripts button.
-Click however many times more.

Expected result: Every parse will create a new set of data.
Actual Result: Every parse will double the data in the system, giving improper results.

Potential actions:
-We could lock the parse button until a certain condition is met.
-We could find a way to clear all data from the system every time we click the parse button.

After parsing the new set of transcripts, there seems to be a new symbol that the parser cannot deal with. These symbols occur when a student has successfully appealed to have the grade removed from their GPA.

Steps:
-Run the application
-Click the 'parse transcripts' button
Error will be shown in the console.

After speaking with Dr. MacIsaac, the new requirement states that these grades should be completely ignored by the system.

For year distribution, I assume we are placing them into the bin where they have completed the highest year based on their credit hours.

Example: I am in second year, however the system will say I am in first year because that is my highest year that I have fully completed. Not the year that I am currently in.

Is this the proper implementation?
Are the values that are currently in the code correct? find: Cohort.addTranscript()

Is it possible to replace this with a row iterator even though the for-each loop works fine?

HSSF is the library for reading/writing .xsl files (2007). We assume the excel files that will be used are xlsx. To read/write in xlsx we must use the XSSF components from Apache POI instead.