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

Horizontal Line Threshold #380

Open
wants to merge 29 commits into
base: develop
Choose a base branch
from
Open

Horizontal Line Threshold #380

wants to merge 29 commits into from

Conversation

kunkoala
Copy link
Collaborator

@kunkoala kunkoala commented Sep 18, 2024

Description

This PR adds a dynamic horizontal line threshold based on an arbitrary number (user input) on the line chart. Horizontal line will also differ between selected districts and compartments. It will have a custom settings button located somewhere on the chart (now the bottom part). Inputs are very welcome!

Related Issues

#264

Design Decisions

Performance & Quality

Checklist

I, the author of this PR checked the following requirements for good software quality:

  • The code is properly formatted (I ran the formatter)
  • The code is written with our software quality standards (I ran the linter)
  • The code is written using our code style
  • Extensive in source documentation has been added
  • Unit and/or integration tests have been added
  • All texts have been internationalized with at least the following languages:
    • English
    • German
  • I tried addressing all new accessibility problems displayed in the console and documented if they can't be fixed
  • I attached performance measurements to prevent performance degradation
  • I added the changes to the next release section of the changelog

I, the reviewer checked the following things:

  • I ran the software once and tried all new and related functionality to this PR
  • I looked at all new and changed lines of code and commented on possible problems
  • I read the added documentation and checked if it is understandable and clear
  • I checked the added tests for completeness
  • I checked the internationalized strings for spelling errors
  • I checked the performance metrics for problems or unexplained degradation
  • I checked that the changes are noted in the changelog

…eshold is saved in UserPreferenceSlice.ts and could be adjusted through LineChartSettings
…Series to create all ranges (wonder if this approach worked)
@kunkoala kunkoala self-assigned this Sep 18, 2024
…ed the seriesRange settings to accept optional parameter so that the horizontal line is undefined when there's no horizontalYAxis
Copy link

github-actions bot commented Sep 27, 2024

Test Results

126 tests  +22   126 ✅ +22   31s ⏱️ +7s
 41 suites + 3     0 💤 ± 0 
  1 files   ± 0     0 ❌ ± 0 

Results for commit 5866381. ± Comparison against base commit 7336aef.

♻️ This comment has been updated with latest results.

@kunkoala
Copy link
Collaborator Author

Here I'm only using the useSeriesRange approach, this has a little bit weird thing where you can see the lines of the percentile overlaps and the strokes also got doubled.

I will make a new approach after this as discussed, by creating a new series similar to the percentile approach.

… to the bottom left that is customizable and able to add new menus such as filters, added a horizontal line settings menu that is able to do add, update, delete on horizontal line threshold that is shown on the list.
…d Item, now highlights selected area based on the current selected district and compartment
@kunkoala kunkoala marked this pull request as ready for review October 22, 2024 12:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant