[SergeAS]

I found two small issues during building and testing this project

1) If we are using as mpi library the Intel MPI , #includes must be reordered so that mpi.h would be included in the first place

2) Older version of Tecplot do not understand "NODES=" and "ELEMENTS=" directives, must use the short form "N=" and "E="

Where do I send the patches ?

Thanks for this interesting project.

[fpalacios]

Originally Posted by SergeAS

I found two small issues during building and testing this project

1) If we are using as mpi library the Intel MPI , #includes must be reordered so that mpi.h would be included in the first place

2) Older version of Tecplot do not understand "NODES=" and "ELEMENTS=" directives, must use the short form "N=" and "E="

Where do I send the patches ?

Thanks for this interesting project.

Thanks a lot for your feedback and kind words, we'll add these changes to the next release 2.1 (summer this year).

We are working in an efficient way to collect the suggestions from the open source community, and recognize your contributions. News will coming soon,

Best,
Francisco

[SergeAS]

1st issue still have yet in SU2v3.2 in 3 places...patch enclosed

[fpalacios]

The MPI issue has been added to the SU2 master branch in GitHub.

Thanks,
Francisco