csu-hmc / opty

A library for using direct collocation in the optimization of dynamic systems.
http://opty.readthedocs.io
Other
86 stars 20 forks source link

Added option to show Cython compilation output and display more informative error. #149

Closed moorepants closed 1 month ago

moorepants commented 1 month ago

@tjstienstra Any thoughts on this? Not sure if you are interested in reviewing things here.

tjstienstra commented 1 month ago

Haven't tested it, but it looks like a really nice feature. In terms of review, I can look at it if you like. As you currently always capture the output you may consider providing that in the error message as well.

moorepants commented 1 month ago

As you currently always capture the output you may consider providing that in the error message as well.

Good idea.

moorepants commented 1 month ago

I'll go with this.