Skip to content

Python wrapper fixes

Boman Romain requested to merge python_wrapper into master

Created by: msanchezmartinez

This PR consists on two fixes to the python part, as well as some enhancements and adding licensing info:

  1. Airfoils in the Selig format are now parsed differently
    1. Not assumed to have one point on the leading edge and two on the trailing edge (suction side and pressure side)
    2. Not assumed to have same # of points on the SS and the PS
    3. Added regression test for this: everything gives the same results on my Mac, Windows 10 laptop, gaston and an Ubuntu VM.
    4. This test has very tight boundaries. The objective of this is to have it fail more times whenever there is an issue with it. Sometimes the results would be a bit off but still look reasonably close... The results are the same on all platforms I could check, so it should not be much of an issue.
  2. The CMake procedure failed on some systems due to the duplication of the module name in newer versions of swig
  3. run.py now does not need the input file to be in tests
  4. Added licensing information

Merge request reports

Loading