CFD Online Discussion Forums

CFD Online Discussion Forums (http://www.cfd-online.com/Forums/)
-   SU2 Installation (http://www.cfd-online.com/Forums/su2-installation/)
-   -   Two small issues (http://www.cfd-online.com/Forums/su2-installation/111938-two-small-issues.html)

SergeAS January 18, 2013 07:12

Two small issues
 
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 January 20, 2013 11:17

Quote:

Originally Posted by SergeAS (Post 402627)
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 June 22, 2014 07:56

1 Attachment(s)
1st issue still have yet in SU2v3.2 in 3 places...patch enclosed

fpalacios June 25, 2014 23:54

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

Thanks,
Francisco


All times are GMT -4. The time now is 00:30.