mdolab / CMPLXFOIL

GNU General Public License v2.0
29 stars 20 forks source link

Standardizing and modernizing CMPLXFOIL #18

Closed bernardopacini closed 1 year ago

bernardopacini commented 1 year ago

Description of feature

@A-CGray, @eytanadler, and I met with with @whophil and Co to discuss how to recombine two divergent forks of CMPLXFOIL (this one and their internal one). The main points from the meeting are:

The main issue in merging the two branches is the different applications, however this can be handled as Python-only differences using the same underlying Fortran code. With this in mind, we propose the following solution and areas to work on (@whophil and others please edit these as needed).

Potential solution

@eytanadler, @A-CGray, @whophil, let me know if this sounds reasonable, we can keep the discussion going on this issue page.

bernardopacini commented 1 year ago

Thank you @whophil for adding CMPLXFOIL to Conda (PR linked here). Please let us know if you run into any issues with the new workflow, otherwise we can check off our third to-do item here.

A-CGray commented 1 year ago

Can we mark this as completed now @bernardopacini ?

bernardopacini commented 1 year ago

Can we mark this as completed now @bernardopacini ?

Yes, that sounds good to me. We don't have extensive tests but given that no one on our side is working with CMPLXFOIL at the moment I think we can tag that as an issue for when someone does pick it up later on.