search

YingyiLiu / HAMS

An open-source computer program for the analysis of wave diffraction and radiation of three-dimensional floating or submerged structures
Apache License 2.0
100 stars 50 forks source link

Adding regression tests and support for ifort #9

Closed gbarter closed 3 years ago

gbarter commented 3 years ago

Adding regression tests using gfortran-compiled executable. Also support ifort in the makefile.

The tests are currently failing, so it is a good thing we created them. Will have to look into what is going on.

YingyiLiu commented 3 years ago

Garrett, thank you for your effort. It seems I am not able to merge your commits due to some problem (see attached screenshot).

Garrett Barter @.***> 于2021年5月21日周五 下午12:14写道：

Adding regression tests using gfortran-compiled executable. Also support ifort in the makefile.

The tests are currently failing, so it is a good thing we created them. Will have to look into what is going on.

You can view, comment on, or merge this pull request online at:

https://github.com/YingyiLiu/HAMS/pull/9 Commit Summary

  • Create c-cpp.yml
  • Merge branch 'master' of github.com:gbarter/HAMS
  • initial test
  • refine matrix
  • refine matrix
  • refine matrix
  • refine matrix
  • check win
  • run all
  • remove longer tests
  • try win again
  • finalize
  • Merge branch 'master' of github.com:YingyiLiu/HAMS
  • overriding f77 with gfortran
  • adding regression tests for all examples
  • running regression test
  • only use checks valid in gfortran 9 and 10
  • now actually running tests and failing
  • makefile now works with ifort

File Changes

Patch Links:

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/YingyiLiu/HAMS/pull/9, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJ2G3NA6W7S7ZF5QEDLHQVTTOXFZFANCNFSM45IJOJVA .

YingyiLiu commented 3 years ago

Could you please check them again? thanks.

劉 盈溢 @.***> 于2021年5月21日周五 下午1:03写道：

Garrett, thank you for your effort. It seems I am not able to merge your commits due to some problem (see attached screenshot).

Garrett Barter @.***> 于2021年5月21日周五 下午12:14写道：

Adding regression tests using gfortran-compiled executable. Also support ifort in the makefile.

The tests are currently failing, so it is a good thing we created them. Will have to look into what is going on.

You can view, comment on, or merge this pull request online at:

https://github.com/YingyiLiu/HAMS/pull/9 Commit Summary

  • Create c-cpp.yml
  • Merge branch 'master' of github.com:gbarter/HAMS
  • initial test
  • refine matrix
  • refine matrix
  • refine matrix
  • refine matrix
  • check win
  • run all
  • remove longer tests
  • try win again
  • finalize
  • Merge branch 'master' of github.com:YingyiLiu/HAMS
  • overriding f77 with gfortran
  • adding regression tests for all examples
  • running regression test
  • only use checks valid in gfortran 9 and 10
  • now actually running tests and failing
  • makefile now works with ifort

File Changes

Patch Links:

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/YingyiLiu/HAMS/pull/9, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJ2G3NA6W7S7ZF5QEDLHQVTTOXFZFANCNFSM45IJOJVA .

gbarter commented 3 years ago

Will close this one and try again