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.

Sign up for GitHub

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

Jump to bottom

fix negative decimals format #1548

Closed
casperdcl opened this issue Feb 5, 2024 · 0 comments · Fixed by #1549
Closed

fix negative decimals format #1548

casperdcl opened this issue Feb 5, 2024 · 0 comments · Fixed by #1549
Assignees
@casperdcl
Labels
c1-quick 🕐 Complexity low p2-bug-warning ⚠ Visual output bad to-fix ⌛ In progress

Comments

@casperdcl
Copy link
Member

casperdcl commented Feb 5, 2024 •
edited
Loading

#621 introduced a bug (-0.123 -> -.123):

>>> from tqdm import tqdm
>>> tqdm.format_num(0.0000012345)
'1.23e-6' # ok
>>> tqdm.format_num(-0.1234)
'-.123'   # bug

spotted by @MargaretDuff in TomographicImaging/CIL#1659 :)
@casperdcl casperdcl added to-fix ⌛ In progress p2-bug-warning ⚠ Visual output bad c1-quick 🕐 Complexity low labels Feb 5, 2024
@casperdcl casperdcl self-assigned this Feb 5, 2024
@casperdcl casperdcl mentioned this issue Feb 5, 2024
Merged
8 tasks
casperdcl added a commit that referenced this issue Feb 5, 2024
@casperdcl
fix format_num
d2a8585 
- fixes #1548
@casperdcl casperdcl mentioned this issue Feb 5, 2024
Merged
casperdcl added a commit that referenced this issue Feb 5, 2024
@casperdcl
fix format_num
5eb7e3f 
- fixes #1548
casperdcl added a commit that referenced this issue Feb 5, 2024
@casperdcl
fix format_num
b38a922 
- fixes #1548
@github-project-automation github-project-automation bot moved this to Done in Casper Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@casperdcl casperdcl

Labels
c1-quick 🕐 Complexity low p2-bug-warning ⚠ Visual output bad to-fix ⌛ In progress
Projects
Casper
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

next release v4.66.2 tqdm/tqdm
1 participant
@casperdcl