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

Numpy: Python3 Support #60

Closed
ax3l opened this issue Feb 17, 2016 · 5 comments
Closed

Numpy: Python3 Support #60

ax3l opened this issue Feb 17, 2016 · 5 comments

Comments

@ax3l
Copy link
Contributor

ax3l commented Feb 17, 2016

Hi @yyalli,

are you planning to add Python3 support for the numpy wrapper?

Thanks,
Axel

@jychoi-hpc
Copy link
Member

It sounds good to have python3 support. I will start working on this soon but it may take some time.

@ax3l
Copy link
Contributor Author

ax3l commented Feb 23, 2016

have you thought about maybe maintaining the wrapper in an independent repo (say github.com/ornladios/adios-python) and including the ADIOS C "main" library as a git submodule?
This would allow you to run continuous integration tests, e.g., with https://travis-ci.org that could make maintainance/testing way easier by running against various python and cython versions on the fly.

@jychoi-hpc
Copy link
Member

Just updated to support python3 (as well as python2). Only tested with python 3.5 and 2.7. It may need more tests. Anyway, I will appreciate if you can check and report any problem.

@ax3l
Copy link
Contributor Author

ax3l commented Jun 21, 2016

Tested with Debian 8 "Jessie" with Python 2.7 & 3.4 and serial adios.

  • install: works
  • open file: works
  • reading variable via f["/ion__ExB_shear"][()] from utils/bpsplit/test/flow11.bp
    • ok in Python 2.7
    • segfault ok in Python 3

@yyalli since GitHub is your mainline, can I please get some feedback on continuous integration I proposed? I can even write you the files for travis-ci and it's free...

@ax3l
Copy link
Contributor Author

ax3l commented Jun 21, 2016

update: python 3 works, too. did not run setup.py clean before building it after python2 :)

@ax3l ax3l closed this as completed Jun 21, 2016
@ax3l ax3l mentioned this issue Jul 27, 2016
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

No branches or pull requests

2 participants