Skip to content

Swig init #37

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.

Sign up for GitHub

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

Jump to bottom
Open
Jvinniec wants to merge 29 commits into
base: master
Choose a base branch
from
Open

Swig init #37

Jvinniec wants to merge 29 commits into from

Conversation

@Jvinniec
Copy link
Owner

@Jvinniec Jvinniec commented Nov 16, 2020

This pull request adds python wrappers to the code so that its classes and methods can be accessed from Python. To enable, add -Dpython-bindings=1 when building with CMake. Note that there are currently only superficial tests for the Python bindings that only test that the code can be imported in Python.

Requirements:

  • SWIG v4 (or greater)
  • Python 3.5 (or greater.)

Jvinniec and others added 29 commits October 4, 2020 12:56
@Jvinniec
Update enums to 'enum class' inheriting from 'unsigned int'
fb5fd91
@Jvinniec
Add first set of SWIG files
528c533 
- CEAngle, CEDate, CENamespace, and CESkyCoord
@Jvinniec
Update version number for development
85a27c8
@Jvinniec
Exclude .so files from being installed in the lib directory
72c2d27
@Jvinniec
Add doxygen class description for CEExecOptions
83b0389
@Jvinniec
Add license information to all CMakeLists.txt files
88032ae
@Jvinniec
Generate more informative doc strings
9228611
@Jvinniec
Update swig files and update build
bc84aa1
@Jvinniec
Add CEBase class that all classes inherit from
cee6c4f
@Jvinniec
Add 'ClassName' and 'describe' methods for each class
604d8e4 
Add tests for these methods
@Jvinniec
Update planetephem executable to use CEPlanet::describe
efaf78c
@Jvinniec
Add 'CEDate::ReturnType' method.
c558b67
@Jvinniec
Add ability to control whether corrections are used
627de17
@Jvinniec
Rename 'AngularSeparation' methods
9aaacfe
@Jvinniec
Add 'CETime::HmsStr' method
9cccc77
@Jvinniec
Replace angular separation method in 'angsep'
ab5f24a
@Jvinniec
Add correction control methods to CENamespace.i
66953d1
@Jvinniec
Update ChangeLog
886029a
@Jvinniec
Update swig build.
e9e1093 
Fix precision error in CETime::TimeSec2Time method
@Jvinniec
Update documentation in CEAngle & cppephem.i SWIG files
6771ac8
@Jvinniec
Update linux dist to latest
88a2e0b
@Jvinniec
Update OSX images to be tested to latest versions
d34d238
@Jvinniec
Fix typo stating 45 deg was equal to pi/2
dd3b454
@Jvinniec
Fix deprecated 'matrix' parameter with 'jobs' parameter
d47266b
@Jvinniec
Fix build so python wrappers are installed at the appropriate stage
bc9ff63
@Jvinniec
Fix build to require SWIG >4
3bc4c21 
Fix setup to install all components correctly
@Jvinniec
Add preliminary Python Tests
8b37cc2
@Jvinniec
Force use of Python 3.5 or later
05cab6e
@Jvinniec
Update README.md to reflect added Python-binding functionality
5a4ef79
@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 16, 2020

SonarCloud Quality Gate failed.

Bug A 0 Bugs
Vulnerability C 1 Vulnerability (and Security Hotspot 3 Security Hotspots to review)
Code Smell B 89 Code Smells

81.7% 81.7% Coverage
0.0% 0.0% Duplication

@Jvinniec Jvinniec self-assigned this Nov 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@Jvinniec Jvinniec

Labels

enhancement

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Jvinniec