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

Update _bp.pyx #28

Merged
merged 7 commits into from
Jul 21, 2020
Merged

Update _bp.pyx #28

merged 7 commits into from
Jul 21, 2020

Conversation

esayyari
Copy link
Contributor

Changing some method definitions from cdef to cpdef, so we have access to those methods from python

wasade and others added 2 commits April 26, 2018 11:29
Changing some method definitions from cdef to cpdef so we have access to those methods from python
@wasade
Copy link
Member

wasade commented Jul 21, 2020

👍

Looks like travis isn't running -- I'll see what's going on there

@wasade
Copy link
Member

wasade commented Jul 21, 2020

Thanks for that merge. Travis is running again, looks like there are some issues (https://travis-ci.com/github/wasade/improved-octo-waddle/jobs/363453203). I think it's just that the .pxd file also needs to be updated

@wasade
Copy link
Member

wasade commented Jul 21, 2020

was hopeful py37 would work but it doesn't due to dependencies (https://travis-ci.com/github/wasade/improved-octo-waddle/jobs/363457462). Can you remove the py37 entry from .travis.yml? that i think should do it

@esayyari
Copy link
Contributor Author

esayyari commented Jul 21, 2020 via email

@wasade
Copy link
Member

wasade commented Jul 21, 2020

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants