search

openfisca / openfisca-web-api

[DEPRECATED] Web API for OpenFisca
https://www.openfisca.fr/
GNU Affero General Public License v3.0
13 stars 11 forks source link

Update formula api #46

Closed fpagnoux closed 8 years ago

fpagnoux commented 8 years ago

This was on mes-aides/embauche prod, but no PR has ever been issued to openfisca. @MattiSG, this is your code. Maybe you remember ?

MattiSG commented 8 years ago

@fpagnoux Would you have the reference to the original commit?

fpagnoux commented 8 years ago

https://github.com/sgmap/openfisca-web-api/commit/d551972d8e4443aaaf9cec9d87f188936df98f52

MattiSG commented 8 years ago

I think this could date back from April and have been cherry-picked. No idea why this wasn't merged, nor if it has major impact. I find the changeset to slightly improve readability by making functions more cohesive, but nothing critical.

laem commented 8 years ago

This PR in fact has a major impact on performance !

laem commented 8 years ago

See #52 (same, but coming from sgmap/openfisca-web-api)