DGNum/gestioCOF

Qwann/k fet/decimal issue2 #462

Merged

narmanli merged 1 commit from qwann/k-fet/decimal_issue2 into k-fet

2017-02-11 22:04:16 +01:00

narmanli commented

2017-02-01 21:01:32 +01:00

(Migrated from git.eleves.ens.fr)

grant is a Decimal and thus can not be casted implicitly to float.

`grant` is a Decimal and thus can not be casted implicitly to float.

narmanli commented

2017-02-01 21:04:22 +01:00

(Migrated from git.eleves.ens.fr)

Target branch changed from master to k-fet

Target branch changed from `master` to `k-fet`

lstephan commented

2017-02-01 21:07:15 +01:00

(Migrated from git.eleves.ens.fr)

Ça affecte quoi dans K-Psul ? Sinon ça me paraît bon =)

Ça affecte quoi dans K-Psul ? Sinon ça me paraît bon =)

narmanli commented

2017-02-01 21:10:27 +01:00

(Migrated from git.eleves.ens.fr)

Dans K-Psul proprement dit rien mais si quelqu'un-e est au COF ça rend une erreur dans la vue account_read

Dans K-Psul proprement dit rien mais si quelqu'un-e est au COF ça rend une erreur dans la vue account_read

narmanli commented

2017-02-01 21:12:23 +01:00

(Migrated from git.eleves.ens.fr)

mentioned in merge request !159

mentioned in merge request !159

lstephan commented

2017-02-01 21:15:10 +01:00

(Migrated from git.eleves.ens.fr)

Je n'ai rien comme erreur chez moi mais c'est possible que ça soit le cas ailleurs =)

À ce compte là, à quoi bon continuer à maintenir du Decimal dans le code si on a des problèmes de cast partout ?

Je n'ai rien comme erreur chez moi mais c'est possible que ça soit le cas ailleurs =) À ce compte là, à quoi bon continuer à maintenir du `Decimal` dans le code si on a des problèmes de cast partout ?

narmanli commented

2017-02-01 21:16:28 +01:00

(Migrated from git.eleves.ens.fr)

Si je me souvien bien on a rajouté le Decimal récement, non ? C'est juste des cast qu'on a oublié de faire en passant à Decimal, non ?

Si je me souvien bien on a rajouté le Decimal récement, non ? C'est juste des cast qu'on a oublié de faire en passant à Decimal, non ?

lstephan commented

2017-02-01 21:20:07 +01:00

(Migrated from git.eleves.ens.fr)

J'ai jamais vu le code sans Decimal il me semble, après il faut demander à @delobell pour ça.

J'ai jamais vu le code sans `Decimal` il me semble, après il faut demander à @delobell pour ça.

delobell commented

2017-02-02 11:46:19 +01:00

(Migrated from git.eleves.ens.fr)

Les Decimal sont là depuis le début et partout en théorie.
Comme c'est indiqué dans le commentaire, le float est là pour qu'il n'y ait pas de problème avec round/floor de future.builtins.
Vu qu'on ne supporte plus Py2, on peut passer à :

def ukf(balance, is_cof):
    grant = is_cof and (1 + Settings.SUBVENTION_COF() / 100) or 1
    return floor(balance * 10 * grant)

Y'a-t-il des problèmes en faisant ça ?

Les Decimal sont là depuis le début et partout en théorie. Comme c'est indiqué dans le commentaire, le ``float`` est là pour qu'il n'y ait pas de problème avec ``round/floor`` de ``future.builtins``. Vu qu'on ne supporte plus Py2, on peut passer à : ```python def ukf(balance, is_cof): grant = is_cof and (1 + Settings.SUBVENTION_COF() / 100) or 1 return floor(balance * 10 * grant) ``` Y'a-t-il des problèmes en faisant ça ?

narmanli commented

2017-02-05 22:38:52 +01:00

(Migrated from git.eleves.ens.fr)

Added 1 commit:

44202811 - not supporting future.builtins anymore in templatetags

Added 1 commit: * 44202811 - not supporting future.builtins anymore in templatetags

narmanli commented

2017-02-05 22:39:33 +01:00

(Migrated from git.eleves.ens.fr)

Works for me. I pushed it the merge request. Thanks :)

Works for me. I pushed it the merge request. Thanks :)

delobell commented

2017-02-11 22:04:12 +01:00

(Migrated from git.eleves.ens.fr)

mentioned in commit 49e6101156

mentioned in commit 49e6101156bd2ada3797441dd84b62e7757bb068

delobell commented

2017-02-11 22:04:16 +01:00

(Migrated from git.eleves.ens.fr)

Status changed to merged

Status changed to merged

Sign in to join this conversation.

No reviewers

No labels

devtype -- backend

devtype -- docs

devtype -- frontend

devtype -- user interface

difficulty -- easy

difficulty -- hard

difficulty -- normal

Good first issue

priority -- high

priority -- low

priority -- medium

priority -- staff-wanted

status -- development

status -- discussion

status -- need review

status -- production

status -- ready to merge

type -- hygiene

type -- improvement

type -- new feature

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

DGNum/gestioCOF!462

No description provided.