search

convexengineering / gplibrary

Useful subsystem models
MIT License
10 stars 11 forks source link

Breguet range #12

Closed mjburton11 closed 8 years ago

mjburton11 commented 8 years ago

Here's a first try at the Breguet Range Model. Not sure why it is not printing out any results.

whoburg commented 8 years ago

Initial code review complete, see inline comments

mjburton11 commented 8 years ago

Response to code review. Ready for new code review.

whoburg commented 8 years ago

just reviewed, think this is ready to merge. @bqpd, take a quick look?

mjburton11 commented 8 years ago

Added test function and argument units and definitions in the doc string

whoburg commented 8 years ago

@mjburton11, added some changes in #20. If you like those, you can hit merge on #20, which will merge it into this. Once that's done, I'm ready for you to merge this into master by hitting merge on this.