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.

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

Fix max_depth default value #1308

Open
bonjourmauko opened this issue Nov 21, 2024 · 0 comments
Open

Fix max_depth default value #1308

bonjourmauko opened this issue Nov 21, 2024 · 0 comments
Labels
kind:fix Bugs are defects and failure demand.

Comments

@bonjourmauko
Copy link
Member

bonjourmauko commented Nov 21, 2024

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.

@bonjourmauko bonjourmauko added the kind:fix Bugs are defects and failure demand. label Nov 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:fix Bugs are defects and failure demand.
Projects
Development

No branches or pull requests

1 participant