scikit-learn/scikit-learn — comment on commit b6542787e7d9359b139e616dd3858c4952aec2de
File: sklearn/linear_model/sgd_fast.pyx line 305
By pprett:

I quickly skimmed over PEP 353... it seems that `Py_ssize_t` is signed; `offset` is guaranteed to be larger or equal to zero. On a 32bit machine Py_ssize_t is an int thus we recover the same problem that Andreas ran into (well... on a 32bit machine allocating such a huge X might not work anyways). Otherwise I'm +1 for Py_ssize_t since it adds additional semantics to offset.