1.13 fails to build
nbecker opened this issue · 13 comments
nbecker commented
fedora linux 25 (gcc-6.3.1)
./randomstate/pcg64.c:3033:14: error: request for member ‘high’ in something not a structure or union
__pyx_v_out.high = __pyx_t_2;
^
...
./randomstate/pcg64.c:3046:14: error: request for member ‘low’ in something not a structure or union
__pyx_v_out.low = __pyx_t_2;
bashtage commented
Hmmm, not really sure what is going on. Is this possibly a GCC bug, or something set by a flag?
Which cython are you using?
nbecker commented
python3-Cython-0.25.2-3.fc25.x86_64
BTW, do you know if I can use pip to re-install an older version (1.12)? pip seems to have left me with a broken randomstate (even though the failed install should have done a rollback) :(
bashtage commented
pip install randomstate=1.12 should work
…On Tue, May 23, 2017 at 12:09 PM ndbecker ***@***.***> wrote:
python3-Cython-0.25.2-3.fc25.x86_64
BTW, do you know if I can use pip to re-install an older version (1.12)?
pip seems to have left me with a broken randomstate (even though the failed
install should have done a rollback) :(
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#92 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AFU5RbQP-WkBd_cJIojavfl7YBLOTbfOks5r8r6EgaJpZM4Njgv8>
.
nbecker commented
1.12 was reinstalled, but I get (on my app):
AttributeError: 'randomstate.xorshift1024.RandomState' object has no
attribute 'random_uintegers'
Wasn't random_uintegers available in 1.12?
On Tue, May 23, 2017 at 7:10 AM Kevin Sheppard <notifications@github.com>
wrote:
… pip install randomstate=1.12 should work
On Tue, May 23, 2017 at 12:09 PM ndbecker ***@***.***>
wrote:
> python3-Cython-0.25.2-3.fc25.x86_64
>
> BTW, do you know if I can use pip to re-install an older version (1.12)?
> pip seems to have left me with a broken randomstate (even though the
failed
> install should have done a rollback) :(
>
> —
> You are receiving this because you commented.
>
>
> Reply to this email directly, view it on GitHub
> <
#92 (comment)
>,
> or mute the thread
> <
https://github.com/notifications/unsubscribe-auth/AFU5RbQP-WkBd_cJIojavfl7YBLOTbfOks5r8r6EgaJpZM4Njgv8
>
> .
>
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#92 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAHK0CM62w17bzIi8SaLvGEDygid6mPNks5r8r6-gaJpZM4Njgv8>
.
bashtage commented
I put it back in right after 1.12.
What happens if you install from github instead of pip?
git clone https://github.com/bashtage/ng-numpy-randomstate
cd ng-numpy-randomstate <https://github.com/bashtage/ng-numpy-randomstate>
python setup.py install
Kevin
…On Tue, May 23, 2017 at 12:13 PM ndbecker ***@***.***> wrote:
1.12 was reinstalled, but I get (on my app):
AttributeError: 'randomstate.xorshift1024.RandomState' object has no
attribute 'random_uintegers'
Wasn't random_uintegers available in 1.12?
On Tue, May 23, 2017 at 7:10 AM Kevin Sheppard ***@***.***>
wrote:
> pip install randomstate=1.12 should work
>
>
>
> On Tue, May 23, 2017 at 12:09 PM ndbecker ***@***.***>
> wrote:
>
> > python3-Cython-0.25.2-3.fc25.x86_64
> >
> > BTW, do you know if I can use pip to re-install an older version
(1.12)?
> > pip seems to have left me with a broken randomstate (even though the
> failed
> > install should have done a rollback) :(
> >
> > —
> > You are receiving this because you commented.
> >
> >
> > Reply to this email directly, view it on GitHub
> > <
>
#92 (comment)
> >,
> > or mute the thread
> > <
>
https://github.com/notifications/unsubscribe-auth/AFU5RbQP-WkBd_cJIojavfl7YBLOTbfOks5r8r6EgaJpZM4Njgv8
> >
> > .
> >
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <
#92 (comment)
>,
> or mute the thread
> <
https://github.com/notifications/unsubscribe-auth/AAHK0CM62w17bzIi8SaLvGEDygid6mPNks5r8r6-gaJpZM4Njgv8
>
> .
>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#92 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AFU5RVLnEsr0CxFCInNYwQcEPfSGQ4yzks5r8r9vgaJpZM4Njgv8>
.
nbecker commented
compiled without error (but there are a few warnings you might want to look
at sometime)
On Tue, May 23, 2017 at 7:16 AM Kevin Sheppard <notifications@github.com>
wrote:
… I put it back in right after 1.12.
What happens if you install from github instead of pip?
git clone https://github.com/bashtage/ng-numpy-randomstate
cd ng-numpy-randomstate <https://github.com/bashtage/ng-numpy-randomstate>
python setup.py install
Kevin
On Tue, May 23, 2017 at 12:13 PM ndbecker ***@***.***>
wrote:
> 1.12 was reinstalled, but I get (on my app):
> AttributeError: 'randomstate.xorshift1024.RandomState' object has no
> attribute 'random_uintegers'
> Wasn't random_uintegers available in 1.12?
>
> On Tue, May 23, 2017 at 7:10 AM Kevin Sheppard ***@***.***
>
> wrote:
>
> > pip install randomstate=1.12 should work
> >
> >
> >
> > On Tue, May 23, 2017 at 12:09 PM ndbecker ***@***.***>
> > wrote:
> >
> > > python3-Cython-0.25.2-3.fc25.x86_64
> > >
> > > BTW, do you know if I can use pip to re-install an older version
> (1.12)?
> > > pip seems to have left me with a broken randomstate (even though the
> > failed
> > > install should have done a rollback) :(
> > >
> > > —
> > > You are receiving this because you commented.
> > >
> > >
> > > Reply to this email directly, view it on GitHub
> > > <
> >
>
#92 (comment)
> > >,
> > > or mute the thread
> > > <
> >
>
https://github.com/notifications/unsubscribe-auth/AFU5RbQP-WkBd_cJIojavfl7YBLOTbfOks5r8r6EgaJpZM4Njgv8
> > >
> > > .
> > >
> >
> > —
> > You are receiving this because you authored the thread.
> > Reply to this email directly, view it on GitHub
> > <
>
#92 (comment)
> >,
> > or mute the thread
> > <
>
https://github.com/notifications/unsubscribe-auth/AAHK0CM62w17bzIi8SaLvGEDygid6mPNks5r8r6-gaJpZM4Njgv8
> >
> > .
> >
>
> —
> You are receiving this because you commented.
>
>
> Reply to this email directly, view it on GitHub
> <
#92 (comment)
>,
> or mute the thread
> <
https://github.com/notifications/unsubscribe-auth/AFU5RVLnEsr0CxFCInNYwQcEPfSGQ4yzks5r8r9vgaJpZM4Njgv8
>
> .
>
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#92 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAHK0KKSFEcmT3lIzejbM6uB24otnEAnks5r8sAJgaJpZM4Njgv8>
.
nbecker commented
But now I'm back to (with randomstate==1.13+2.gea9accf)
AttributeError: 'randomstate.xorshift1024.RandomState' object has no
attribute 'random_uintegers'
…On Tue, May 23, 2017 at 7:20 AM Neal Becker ***@***.***> wrote:
compiled without error (but there are a few warnings you might want to
look at sometime)
On Tue, May 23, 2017 at 7:16 AM Kevin Sheppard ***@***.***>
wrote:
> I put it back in right after 1.12.
>
> What happens if you install from github instead of pip?
>
> git clone https://github.com/bashtage/ng-numpy-randomstate
> cd ng-numpy-randomstate <https://github.com/bashtage/ng-numpy-randomstate
> >
> python setup.py install
>
> Kevin
>
>
> On Tue, May 23, 2017 at 12:13 PM ndbecker ***@***.***>
> wrote:
>
> > 1.12 was reinstalled, but I get (on my app):
> > AttributeError: 'randomstate.xorshift1024.RandomState' object has no
> > attribute 'random_uintegers'
> > Wasn't random_uintegers available in 1.12?
> >
> > On Tue, May 23, 2017 at 7:10 AM Kevin Sheppard <
> ***@***.***>
> > wrote:
> >
> > > pip install randomstate=1.12 should work
> > >
> > >
> > >
> > > On Tue, May 23, 2017 at 12:09 PM ndbecker ***@***.***>
> > > wrote:
> > >
> > > > python3-Cython-0.25.2-3.fc25.x86_64
> > > >
> > > > BTW, do you know if I can use pip to re-install an older version
> > (1.12)?
> > > > pip seems to have left me with a broken randomstate (even though the
> > > failed
> > > > install should have done a rollback) :(
> > > >
> > > > —
> > > > You are receiving this because you commented.
> > > >
> > > >
> > > > Reply to this email directly, view it on GitHub
> > > > <
> > >
> >
> #92 (comment)
> > > >,
> > > > or mute the thread
> > > > <
> > >
> >
> https://github.com/notifications/unsubscribe-auth/AFU5RbQP-WkBd_cJIojavfl7YBLOTbfOks5r8r6EgaJpZM4Njgv8
> > > >
> > > > .
> > > >
> > >
> > > —
> > > You are receiving this because you authored the thread.
> > > Reply to this email directly, view it on GitHub
> > > <
> >
> #92 (comment)
> > >,
> > > or mute the thread
> > > <
> >
> https://github.com/notifications/unsubscribe-auth/AAHK0CM62w17bzIi8SaLvGEDygid6mPNks5r8r6-gaJpZM4Njgv8
> > >
> > > .
> > >
> >
> > —
> > You are receiving this because you commented.
> >
> >
> > Reply to this email directly, view it on GitHub
> > <
> #92 (comment)
> >,
> > or mute the thread
> > <
> https://github.com/notifications/unsubscribe-auth/AFU5RVLnEsr0CxFCInNYwQcEPfSGQ4yzks5r8r9vgaJpZM4Njgv8
> >
> > .
> >
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <#92 (comment)>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AAHK0KKSFEcmT3lIzejbM6uB24otnEAnks5r8sAJgaJpZM4Njgv8>
> .
>
bashtage commented
Hmmm...
nbecker commented
OK, I cleaned out all the old versions of randomstate in site_packages, and reinstalled the current git, and everything is OK.
As to why it didn't build from pip, that remains a problem.
bashtage commented
Run
sudo pip uninstall randomstate -y
sudo pip uninstall randomstate -y
(yes twice) then reinstall. (Obviously only sudo if needed)
bashtage commented
Something is wrong with the pip sdist -- it fails on Ubuntu/Mint as well.
bashtage commented
Thanks for reporting -- I think I need to not publish the c files in the sdist since I don't think these are compatible across Python 2.7 and 3.x. Not 100% but this is my suspicion. I published from Windows/2.7 so this might be why you had an issue. I'll probably clean things up and publish a 1.13.2 soon that requires Cython to do a sdist install.
bashtage commented
This is not fixed -- have to use local Cython directly since there is some platform conditional Cython being used.