mv / fundamentus-api

Python/Pandas code to perform fundamental analysis on companies listed on BOVESPA from the website Fundamentus (www.fundamentus.com.br)
MIT License
90 stars 35 forks source link

Fix for bug that removes decimal separator in decimal values #127

Open obrunodelgado opened 1 year ago

obrunodelgado commented 1 year ago

This Pull Request resolves the bug that removed the "." (decimal separator) character from decimal values, causing issues in the representation of fractional numbers. Due to this bug, values like 0.14 were incorrectly transformed into 014, leading to potential errors in calculations and numerical data visualization.

Evidence of the bug before the execution of fmt_dec:

Screenshot from 2023-05-01 12-08-03

After the execution of fmt_dec: Screenshot from 2023-05-01 12-08-13

Changes made:

  1. Identification and correction of the code snippet responsible for the bug, ensuring that the decimal separator is preserved when handling decimal values.
  2. Updated test_fmt_dec unit tests for this fix.

I believe this fix will bring greater reliability and accuracy to the project, preventing errors related to the handling of decimal values.