[Django] #29130: Floatformat is rounding off incorrectly.

7 views
Skip to first unread message

Django

unread,
Feb 12, 2018, 5:00:07 PM2/12/18
to django-...@googlegroups.com
#29130: Floatformat is rounding off incorrectly.
-------------------------------------+-------------------------------------
Reporter: Akash | Owner: nobody
Deshpande |
Type: Bug | Status: new
Component: Template | Version: 1.11
system | Keywords: template float
Severity: Normal | filter
Triage Stage: | Has patch: 0
Unreviewed |
Needs documentation: 0 | Needs tests: 0
Patch needs improvement: 0 | Easy pickings: 0
UI/UX: 0 |
-------------------------------------+-------------------------------------
I think the below changes broke/caused issues in floatformat

https://github.com/django/django/commit/12f7928f5a455e330c0a7f19bc86b37baca12811

Easiest way to replicate the issue is
1. Create a simple view with a template

views.py
\\
def test(request):
return render('test.html')


test.html
\\
{{ 331.44999999999|floatformat:-1 }}

The above should return 331.5, but returns 331.4. This changed in django
1.11. (It returned 331.5 in django 1.8.X )

--
Ticket URL: <https://code.djangoproject.com/ticket/29130>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

Django

unread,
Feb 12, 2018, 5:00:36 PM2/12/18
to django-...@googlegroups.com
#29130: Floatformat is rounding off incorrectly.
-------------------------------------+-------------------------------------
Reporter: Akash Deshpande | Owner: nobody
Type: Bug | Status: new
Component: Template system | Version: 1.11
Severity: Normal | Resolution:
Keywords: template float | Triage Stage:
filter | Unreviewed
Has patch: 0 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Akash Deshpande):

* Attachment "Screen Shot 2018-02-12 at 1.49.17 PM.png" added.

Django

unread,
Feb 12, 2018, 9:25:07 PM2/12/18
to django-...@googlegroups.com
#29130: floatformat is rounding off incorrectly

-------------------------------------+-------------------------------------
Reporter: Akash Deshpande | Owner: nobody
Type: Bug | Status: new

Component: Template system | Version: 1.11
Severity: Normal | Resolution:
Keywords: template float | Triage Stage:
filter | Unreviewed
Has patch: 0 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------

Old description:

> I think the below changes broke/caused issues in floatformat
>
> https://github.com/django/django/commit/12f7928f5a455e330c0a7f19bc86b37baca12811
>
> Easiest way to replicate the issue is
> 1. Create a simple view with a template
>
> views.py
> \\
> def test(request):
> return render('test.html')
>

> test.html
> \\
> {{ 331.44999999999|floatformat:-1 }}
>
> The above should return 331.5, but returns 331.4. This changed in django
> 1.11. (It returned 331.5 in django 1.8.X )

New description:

I think 12f7928f5a455e330c0a7f19bc86b37baca12811 caused a regression in
`floatformat`.

`{{ 331.44999999999|floatformat:-1 }}`

The above should return 331.5, but returns 331.4. This changed in django
1.11. (It returned 331.5 in django 1.8.X )

--

Comment (by Tim Graham):

The current behavior looks correct to me. If rounding to one decimal
place, look at the second decimal place (4). Since that's less than five,
round down. Have I misunderstood the rules for rounding?

--
Ticket URL: <https://code.djangoproject.com/ticket/29130#comment:1>

Django

unread,
Feb 13, 2018, 9:45:10 AM2/13/18
to django-...@googlegroups.com
#29130: floatformat is rounding off incorrectly
-------------------------------------+-------------------------------------
Reporter: Akash Deshpande | Owner: nobody
Type: Bug | Status: closed

Component: Template system | Version: 1.11
Severity: Normal | Resolution: invalid

Keywords: template float | Triage Stage:
filter | Unreviewed
Has patch: 0 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Tim Graham):

* status: new => closed
* resolution: => invalid


--
Ticket URL: <https://code.djangoproject.com/ticket/29130#comment:2>

Reply all
Reply to author
Forward
0 new messages