-
Notifications
You must be signed in to change notification settings - Fork 24
Add python bindings #35
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
base: master
Are you sure you want to change the base?
Changes from all commits
082afe4
a8a924e
4b33819
111b734
caeb577
eddb760
8e7f475
dc91554
1cb3597
c940bd3
86cbd91
037eb69
60103bc
5dacd2d
2ec61b4
bae25d3
4d54664
a65d69a
9143d1a
8a0f1a4
6701c19
daadb33
7bb120f
1692246
b10e6a3
7a2c67e
cf92b45
ded948f
7892582
4c3414e
6e5beb6
470ae1a
740f8d2
54d8f30
aa3fb33
b6648fa
cc2e1b5
c0158a6
1ab4052
674aea1
2394da2
7aab1a3
efcb14a
83ea2be
4761465
95c81d1
e3bf246
9464880
10e2148
433810e
22d8b68
5ac8042
a24420c
6f2aea1
c740b14
65c8345
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,9 +1,12 @@ | ||
| [submodule "Catch2"] | ||
| path = submodules/Catch2 | ||
| url = https://github.com/catchorg/Catch2 | ||
| [submodule "msgpack-c"] | ||
| [submodule "submodules/pybind11"] | ||
| path = submodules/pybind11 | ||
| url = https://github.com/pybind/pybind11.git | ||
| [submodule "submodules/msgpack-c"] | ||
| path = submodules/msgpack-c | ||
| url = https://github.com/msgpack/msgpack-c | ||
| [submodule "mmtf_spec"] | ||
| url = https://github.com/msgpack/msgpack-c.git | ||
| [submodule "submodules/Catch2"] | ||
| path = submodules/Catch2 | ||
| url = https://github.com/catchorg/Catch2.git | ||
| [submodule "submodules/mmtf_spec"] | ||
| path = submodules/mmtf_spec | ||
| url = https://github.com/rcsb/mmtf | ||
| url = https://github.com/rcsb/mmtf.git | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,3 +1,4 @@ | ||
| --- | ||
|
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I 'linted' this file using yamllint, sorry for the noise |
||
| language: cpp | ||
| sudo: false | ||
| dist: trusty | ||
|
|
@@ -27,8 +28,25 @@ linux64_cpp17addons: | |
| sources: | ||
| - ubuntu-toolchain-r-test | ||
|
|
||
| linux64_cpp17addons: | ||
| addons: &linux64cpp17 | ||
| apt: | ||
| sources: | ||
| - ubuntu-toolchain-r-test | ||
|
|
||
|
|
||
| linux64_cpp17addons_py: | ||
| addons: &linux64cpp17py | ||
| apt: | ||
| sources: | ||
| - ubuntu-toolchain-r-test | ||
| - gcc-7 | ||
| - g++-7 | ||
|
|
||
| python_test_command_sub: &python_test_command TEST_COMMAND=$TRAVIS_BUILD_DIR/ci/build_and_run_python_tests.sh CC=gcc | ||
| # Set empty values for allow_failures to work | ||
| env: TEST_COMMAND=$TRAVIS_BUILD_DIR/ci/build_and_run_tests.sh | ||
| python: | ||
|
|
||
| matrix: | ||
| fast_finish: true | ||
|
|
@@ -54,7 +72,27 @@ matrix: | |
| compiler: gcc | ||
| addons: *linux64cpp17 | ||
| dist: bionic | ||
|
|
||
| - os: linux | ||
| compiler: gcc | ||
| addons: *linux64cpp17py | ||
| dist: bionic | ||
| env: *python_test_command | ||
| python: 3.8 | ||
| language: python | ||
| - os: linux | ||
| compiler: gcc | ||
| addons: *linux64cpp17py | ||
| dist: bionic | ||
| env: *python_test_command | ||
| python: 3.11 | ||
| language: python | ||
| - os: linux | ||
| compiler: gcc | ||
| addons: *linux64cpp17py | ||
| dist: bionic | ||
| env: *python_test_command | ||
| python: 3.12 | ||
| language: python | ||
|
|
||
| before_install: | ||
| # Setting environement | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,10 @@ | ||
|
|
||
| python3 --version | ||
| pip3 --version | ||
| pip3 install -r requirements.txt | ||
| pip3 install -r requirements-dev.txt | ||
| cd $TRAVIS_BUILD_DIR | ||
| pip3 install . | ||
| pytest python_src/tests/tests.py -s -vv | ||
|
|
||
|
|
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -163,7 +163,7 @@ struct StructureData { | |
| std::string title; | ||
| std::string depositionDate; | ||
| std::string releaseDate; | ||
| std::vector<std::vector<float> > ncsOperatorList; | ||
| std::vector<std::vector<float>> ncsOperatorList; | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is C++11 syntax. As far as I know this project is still C++03 (but you have my vote for dropping that).
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm guessing you meant to comment this somewhere else? |
||
| std::vector<BioAssembly> bioAssemblyList; | ||
| std::vector<Entity> entityList; | ||
| std::vector<std::string> experimentalMethods; | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Catch2listed twice?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will fix this once the other two PRs are merged