LDAR-Sim / LDAR_Sim

MIT License
11 stars 22 forks source link

fix: Fix bug in metric prefix formatter #233

Closed ThomasGalesloot closed 4 months ago

ThomasGalesloot commented 4 months ago

Pull Request Key Information

Reason for change

There was a bug in the metric prefix formatter that was causing the tick labels to be incorrect. This was due to the fact that the formatter was not correctly handling the case where the value was between -1 and 1, resulting in any ticks in that range being lost.

What was changed

Added a check to the metric prefix formatter to handle the case where the value is between -1 and 1. This ensures that the tick labels are correctly displayed.

Intended Purpose

The tick labels are now correctly displayed on the visualization.

Level of version change required

Patch bump

Testing Completed

Manually tested to ensure all values now returning as expected.

All unit tests passing: unit_test_results.txt

Target Issue

N/A

Additional Information

N/A