CFD Online Logo CFD Online URL
www.cfd-online.com
[Sponsors]
Home > Forums > SU2 Installation

Two small issues

Register Blogs Members List Search Today's Posts Mark Forums Read

Reply
 
LinkBack Thread Tools Display Modes
Old   January 18, 2013, 07:12
Default Two small issues
  #1
Member
 
SergeAS's Avatar
 
Serge A. Suchkov
Join Date: Oct 2011
Location: Moscow, Russia
Posts: 73
Blog Entries: 5
Rep Power: 5
SergeAS is on a distinguished road
Send a message via Skype™ to 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.
SergeAS is offline   Reply With Quote

Old   January 20, 2013, 11:17
Default
  #2
Super Moderator
 
Francisco Palacios
Join Date: Jan 2013
Location: Stanford, CA
Posts: 301
Rep Power: 5
fpalacios is on a distinguished road
Quote:
Originally Posted by SergeAS View Post
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
fpalacios is offline   Reply With Quote

Old   June 22, 2014, 07:56
Default
  #3
Member
 
SergeAS's Avatar
 
Serge A. Suchkov
Join Date: Oct 2011
Location: Moscow, Russia
Posts: 73
Blog Entries: 5
Rep Power: 5
SergeAS is on a distinguished road
Send a message via Skype™ to SergeAS
1st issue still have yet in SU2v3.2 in 3 places...patch enclosed
Attached Files
File Type: txt intel_mpi_issue.diff.txt (1.4 KB, 14 views)
SergeAS is offline   Reply With Quote

Old   June 25, 2014, 23:54
Default
  #4
Super Moderator
 
Francisco Palacios
Join Date: Jan 2013
Location: Stanford, CA
Posts: 301
Rep Power: 5
fpalacios is on a distinguished road
The MPI issue has been added to the SU2 master branch in GitHub.

Thanks,
Francisco
fpalacios is offline   Reply With Quote

Reply

Thread Tools
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off
Trackbacks are On
Pingbacks are On
Refbacks are On


Similar Threads
Thread Thread Starter Forum Replies Last Post
.STL: non-closed manifold surface giulio.topazio OpenFOAM Native Meshers: snappyHexMesh and Others 24 March 3, 2015 05:41
why did Time step is too small in stiffbs? satum FLUENT 1 December 10, 2012 10:15
Simulating small features on large models siw CFX 1 February 16, 2012 18:10
Small influence of convection in PISO if Courant number is small sebi OpenFOAM 0 February 22, 2011 15:55
Time step too small? derz CFX 2 September 8, 2010 00:03


All times are GMT -4. The time now is 17:18.