Skip to content

Adding new fortran wrapper for CoSimIO#339

Merged
philbucher merged 2 commits intomasterfrom
fortran-cosimio-wrapper
Jun 19, 2022
Merged

Adding new fortran wrapper for CoSimIO#339
philbucher merged 2 commits intomasterfrom
fortran-cosimio-wrapper

Conversation

@Ankit-Nanda-altair
Copy link
Copy Markdown
Contributor

Fortran Interface for:

  • co_sim_io_c.h
  • co_sim_io_c_info.h
  • co_sim_io_c_model_part.h
    New fuctions:
  • CoSimIO_Info_GetString_Wrapper: wrapper for CoSimIO_Info_GetString()
  • CoSimIO_ModelPart_Name_Wrapper: wrapper for CoSimIO_ModelPart_Name()

@philbucher
Copy link
Copy Markdown
Member

wow awesome thanks!

I will review in the next days

Could you please add a basic test, e.g. for connection, like this one

@Ankit-Nanda-altair
Copy link
Copy Markdown
Contributor Author

Hi,

I have added a simple connect disconnect test for the fortran wrapper, however I could not test it on my machine as I need to associate the cosimio dll to the project and have a hard time associating it in visual studio.

Please check the test and let me know if it worked for you or any errors it is showing

The connect disconnect test is in test\co_sim_io\fortran\connect_disconnect.f90

@Ankit-Nanda-altair
Copy link
Copy Markdown
Contributor Author

Hi @philbucher ,

Do you have any idea why intel mpi checks are failing because I have not changed that part.

@philbucher
Copy link
Copy Markdown
Member

sometimes the tests can fail, especially if MPI is involved
I restarted the job, hopefully it works now

@Ankit-Nanda-altair
Copy link
Copy Markdown
Contributor Author

Thanks @philbucher , Seams it worked this time!

@philbucher
Copy link
Copy Markdown
Member

Awesome, then lets go ahead :)
Thanks a lot for your contribution!

@philbucher philbucher merged commit 14f51ab into master Jun 19, 2022
@philbucher philbucher deleted the fortran-cosimio-wrapper branch June 19, 2022 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request interface Fortran

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants