search

p-chambers / occ_airconics

Aircraft Configuration through Integrated Cross-disciplinary Scripting, Python package built on PythonOCC
http://occ-airconics.readthedocs.io/en/latest/index.html
BSD 3-Clause "New" or "Revised" License
14 stars 9 forks source link

Mixed capitals, underscores and lowercase in functions names/API. #16

Open p-chambers opened 7 years ago

p-chambers commented 7 years ago

The API has a lot of mixed use of lowercase/underscores, capitals and mixed case. This is partially due to the original airconics library using a different style to other dependencies, and also conflicting with PEP8.

A consistent style should be decided upon to avoid confusion. The easiest to implement would probably be capital letters and no underscores in function names.

p-chambers commented 7 years ago

Best to write more tests before doing this, as code coverage is currently quite low for the Topology class.