openfisca / openfisca-core

OpenFisca core engine. See other repositories for countries-specific code & data.
https://openfisca.org
GNU Affero General Public License v3.0
170 stars 75 forks source link

Fix max_depth default value #1308

Open bonjourmauko opened 2 days ago

bonjourmauko commented 2 days ago

Hi there!

I really enjoy OpenFisca, but I recently encountered an issue.

Here is what I did:

openfisca test -v ...

Here is what I expected to happen:

To work for every user.

Here is what actually happened:

@guillett reports that the default value, sys.maxsize, can be None for some users, which leads to buggy behaviour.

Here is data (or links to it) that can help you reproduce this issue:

https://news.ycombinator.com/item?id=22019627

  1. Clone https://github.com/python/cpython
  2. Modify all references to maxsize in *.c and *.h files
  3. Compile Python with maxsize set to *void so sys.maxsize is None

Note

I don't quite get why would someone have sys.maxsize to None, as it is a reference throughly used internaly by CPython to signify an unlimited buffer... It can literally tear the interpreter down in a myriad ways.