search

nus-cs2103-AY2324S1 / pe-dev-response

0 stars 0 forks source link

Summary does not support summarizing old information #1760

Open nus-se-script opened 1 year ago

nus-se-script commented 1 year ago

An example use case might be looking at my current year's spending in comparison with my previous year's, since I might want to hit yearly budgeting goals. If I used the app long-term, I could see myself appreciating this functionality.

[original: nus-cs2103-AY2324S1/pe-interim#2528] [original labels: type.FeatureFlaw severity.Medium]

jamz903 commented 1 year ago

Team's Response

Thank you for your suggestion! We have opted to classify this as a Medium FeatureFlaw that is NotInScope for the following reason:

There are several components to address for our response:

Why NotInScope? Refer to the screenshot below regarding the qualifications for a bug being NotInscope:

image.png

Firstly, extending the current functionality of the summary statistics feature to support years of information is less important than the work that has already been done to:

  1. Calculate the summary statistics for both per category and per month (filtered for the past 1 year)
  2. Create a summary window that updates in real-time

The functionality to display data across multiple years should be postponed to future iterations of the application. It is important to note that limiting the plot to one year's worth of data was an intentional decision. We aimed to prevent the plot from becoming excessively small in case a stress-testing user adds centuries of data. Further testing is required to determine the appropriate timeframe for plotting the data, and this aspect should be addressed in later iterations. Additionally, it qualifies the second bullet point in the screenshot as it is impossible for users to view data prior to 1 year ago since viewing data prior to one year ago has not been implemented. Since the user cannot attempt to use the feature of viewing data prior to 1 year ago, we classify it is NotInScope.

Why Medium severity? As you've mentioned, this problem occurs for long-term users of the application. Thus, we agree with your severity of "Medium".

Why FeatureFlaw? Refer to the screenshot below regarding the qualifications for a bug being a Feature Flaw:

image.png

The summary statistics feature works as how it is documented in the UG and DG. Thus, we think it is a Feature Flaw, like you've indicated.

Regardless, thank you for your feedback!

Duplicate status (if any):

--