-
Notifications
You must be signed in to change notification settings - Fork 64
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
simple pytraj test for cpptraj #5
Comments
On Fri, Feb 6, 2015 at 10:20 AM, Hai Nguyen notifications@github.com
I've made some changes to cpptraj that will break your version, but I'm Thanks for writing some tests - it will make our lives much easier! -Dan
Daniel R. Roe, PhD |
Thanks for your help. Hai |
By default, most Linux operating systems do not come pre-installed with This package is usually called something like The sander API suffers from the same limitation -- if the Python development headers are absent, it simply cannot be built. |
thx Jason. Hai |
To clarify -- this is probably unnecessary for the |
On Fri, Feb 6, 2015 at 12:13 PM, Jason Swails notifications@github.com
-Dan
Daniel R. Roe, PhD |
Only when it means less work for me :). In this case, though, I think anybody that comes to Github to get the pytraj source code so they can compile it knows full well that python-dev is required, or they will quickly find out without an error message from |
On Fri, Feb 6, 2015 at 11:41 AM, Daniel Roe daniel.r.roe@gmail.com wrote:
OK, so this will be a bigger change than I thought. Recently I had to However, since this was a substantial interface change, it has now broken So bottom line, I will try working on this unless I hear from you. If you -Dan
Daniel R. Roe, PhD |
Ideally, pytraj user would do some actions with Frame objects, then will combine them into a class objects that CpptrajState can accept (for example using "AddFrame" method of DataSet_Coords_TRJ). CpptrajState would take care of "massive" action, analysis, output stuff .. If this is not easy, I can translate CpptrajState.cpp to Cython code. Let me know if I miss anything. thx. Hai |
@Mojyt Hai |
OK, pushed. Hopefully all goes well... On Fri, Feb 6, 2015 at 3:37 PM, Hai Nguyen notifications@github.com wrote:
Daniel R. Roe, PhD |
@Mojyt: yes. went really well https://travis-ci.org/hainm/pytraj/builds/49823926 |
On Fri, Feb 6, 2015 at 10:03 PM, Hai Nguyen notifications@github.com
It looked like there were some failures (from the Travis-CI log): 55.81s$ python RunAllAndFindFailure.py Is this expected? Jason M. Swails |
Hi, yes, this is unrelated to what Dan has changed. I made some changes to the code and forgot to commit. Thanks |
Hi @Mojyt
I've just finished updating pytraj to catch up with cpptraj-dev. Please following this to run the tests.
Note: pytraj's compiling time (linking to libcpptraj.so) is currently few times slower than compiling libcpptraj. :D
Let me know if you have any question. thx.
Hai
The text was updated successfully, but these errors were encountered: