Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

{cae}[foss/2022a] Nektar++ v5.2.0 WIB #16953

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

sassy-crick
Copy link
Collaborator

@sassy-crick sassy-crick commented Dec 20, 2022

(created using eb --new-pr)

Requires:

@sassy-crick sassy-crick changed the title {cae}[foss/2022a] Nektar++ v5.2.0 {cae}[foss/2022a] Nektar++ v5.2.0 WIB Dec 20, 2022
@sassy-crick
Copy link
Collaborator Author

sassy-crick commented Dec 20, 2022

I have enabled the ctestbut 15 out of the 603 tests are failing with segfaults. This could be my test machine (Tiger CPU) or something else. It might be a good idea to see if that is a problem at other sites too.
Failing tests:

        143 - ADRSolver_Advection3D_m8_DG_hex_nodal (Failed)
        144 - ADRSolver_Advection3D_m12_DG_hex_periodic (Failed)
        145 - ADRSolver_Advection3D_m8_DG_hex (Failed)
        146 - ADRSolver_Advection3D_m8_DG_hex_MF (Failed)
        147 - ADRSolver_Advection3D_m10_DG_hex_VarP (Failed)
        150 - ADRSolver_Advection3D_m12_DG_tet (Failed)
        210 - ADRSolver_Helmholtz3D_Cube_BndFile_Lagrange (Failed)
        223 - ADRSolver_SVV_Prism (Failed)
        313 - IncNavierStokesSolver_Hex_channel_varP (Failed)
        316 - IncNavierStokesSolver_Hex_channel_m6_nodalRestart (Failed)
        345 - IncNavierStokesSolver_Tet_Kovasnay_SVV_DGKer (Failed)
        354 - IncNavierStokesSolver_Tet_channel_varP (Failed)
        371 - IncNavierStokesSolver_Pyr_channel_SVV (Failed)
        512 - StdRegions_StdProject_Diff3D_Hex_Lagrange_P6_Q7 (Failed)
        570 - MultiRegions_Helmholtz2D_CG_P7_Modes_curved_quad (Failed)

added: On Intel Ivy-Bridge I get these failing tests:

        311 - IncNavierStokesSolver_Couette_3DH2D_MVM (Failed)
        321 - IncNavierStokesSolver_SM (Failed)
        322 - IncNavierStokesSolver_SM_base_file (Failed)
        323 - IncNavierStokesSolver_SM_Adj (Failed)
        324 - IncNavierStokesSolver_SM_Adj_base_file (Failed)
        376 - PulseWaveSolver_Junction (Failed)

@smoors
Copy link
Contributor

smoors commented Dec 21, 2022

@boegelbot: please test @ generoso

@boegelbot
Copy link
Collaborator

@smoors: Request for testing this PR well received on login1

PR test command 'EB_PR=16953 EB_ARGS= EB_CONTAINER= /opt/software/slurm/bin/sbatch --job-name test_PR_16953 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 9869

Test results coming soon (I hope)...

- notification for comment with ID 1361315644 processed

Message to humans: this is just bookkeeping information for me,
it is of no use to you (unless you think I have a bug, which I don't).

@boegelbot
Copy link
Collaborator

Test report by @boegelbot
FAILED
Build succeeded for 0 out of 1 (1 easyconfigs in total)
cns2 - Linux Rocky Linux 8.5, x86_64, Intel(R) Xeon(R) CPU E5-2667 v3 @ 3.20GHz (haswell), Python 3.6.8
See https://gist.github.com/cf1e1e05fb0cd3a2730c7ec30fa93ec6 for a full test report.

@smoors smoors added the update label Dec 21, 2022
@smoors
Copy link
Contributor

smoors commented Dec 21, 2022

@sassy-crick why not install the tinyxml dep as a module?

@sassy-crick
Copy link
Collaborator Author

@sassy-crick why not install the tinyxml dep as a module?

Thanks to Jasper that is done now and I have added this to the EasyConfig file.

@smoors
Copy link
Contributor

smoors commented Dec 23, 2022

@boegelbot: please test @ generoso

@boegelbot
Copy link
Collaborator

@smoors: Request for testing this PR well received on login1

PR test command 'EB_PR=16953 EB_ARGS= EB_CONTAINER= /opt/software/slurm/bin/sbatch --job-name test_PR_16953 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 9886

Test results coming soon (I hope)...

- notification for comment with ID 1363882959 processed

Message to humans: this is just bookkeeping information for me,
it is of no use to you (unless you think I have a bug, which I don't).

@boegelbot
Copy link
Collaborator

Test report by @boegelbot
FAILED
Build succeeded for 0 out of 1 (1 easyconfigs in total)
cns1 - Linux Rocky Linux 8.5, x86_64, Intel(R) Xeon(R) CPU E5-2667 v3 @ 3.20GHz (haswell), Python 3.6.8
See https://gist.github.com/0e4983fb57d5998c8d2aac7300397d6d for a full test report.

@smoors
Copy link
Contributor

smoors commented Dec 23, 2022

it looks like it needs the static lib, whereas tinyxml is currently installed as shared lib only:

/base/exceptions.py:124 in init): cmd " ninja -j 4 " exited with exit code 1 and output:
ninja: error: 'ThirdParty/dist/lib/libtinyxml.a', needed by 'library/LibUtilities/libLibUtilities.so.5.2.0', missing and no known rule to make it

@sassy-crick
Copy link
Collaborator Author

it looks like it needs the static lib, whereas tinyxml is currently installed as shared lib only:

/base/exceptions.py:124 in init): cmd " ninja -j 4 " exited with exit code 1 and output:
ninja: error: 'ThirdParty/dist/lib/libtinyxml.a', needed by 'library/LibUtilities/libLibUtilities.so.5.2.0', missing and no known rule to make it

This is odd as I tested it on a machine which does not have TinyXML installed and the log file reads:

-- Found TinyXML: /rds/general/user/jsassman/home/apps/software/TinyXML/2.6.2-GCCcore-11.3.0/lib/libtinyxml.so

Also, just to be sure I checked again:

[cx1-106-9-3 nektar]$ rpm -qa | grep -i tinyxml
[cx1-106-9-3 nektar]$  

As that worked I have updated the file.

Also, for the previous build where I used the OS version, I got:

-- Found TinyXML: /usr/lib/x86_64-linux-gnu/libtinyxml.so

Any idea why it works for me and not for you? Is that a Debian thing? I am a bit puzzled to be honest.

So for me it looks like the dynamic library should work as well.

@terjekv
Copy link
Collaborator

terjekv commented Jan 27, 2023

Test report by @terjekv
FAILED
Build succeeded for 11 out of 12 (1 easyconfigs in total)
ninhursaga.uio.no - Linux RHEL 9.1 (Plow), x86_64, Intel(R) Xeon(R) Silver 4216 CPU @ 2.10GHz, 1 x NVIDIA Tesla T4, 525.60.13, Python 3.9.14
See https://gist.github.com/8630d5731624c1c39865cb61b227f92a for a full test report.

@Parv621
Copy link

Parv621 commented Feb 21, 2023

@sassy-crick made me aware of this merge request. Might be good to do the ctests with at least 6 CPU nodes as some of the tests are supposed to run in parallel from 2 to 6 processors. If there are still remaining failed ctests, I would be keen to see their output using --output-on-failure with the ctest command.

@sassy-crick
Copy link
Collaborator Author

@boegelbot please test @ jsc-zen2
CORE_CNT=16
EB_ARGS=“--parallel 4”

@boegelbot
Copy link
Collaborator

@sassy-crick: Request for testing this PR well received on jsczen2l1.int.jsc-zen2.easybuild-test.cluster

PR test command 'EB_PR=16953 EB_ARGS="“--parallel" /opt/software/slurm/bin/sbatch --mem-per-cpu=4000M --job-name test_PR_16953 --ntasks="16" ~/boegelbot/eb_from_pr_upload_jsc-zen2.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 2301

Test results coming soon (I hope)...

- notification for comment with ID 1460618718 processed

Message to humans: this is just bookkeeping information for me,
it is of no use to you (unless you think I have a bug, which I don't).

@sassy-crick
Copy link
Collaborator Author

@boegelbot please test @ jsc-zen2
CORE_CNT=16
EB_ARGS="--parallel 4"

@boegelbot
Copy link
Collaborator

@sassy-crick: Request for testing this PR well received on jsczen2l1.int.jsc-zen2.easybuild-test.cluster

PR test command 'EB_PR=16953 EB_ARGS="--parallel 4" /opt/software/slurm/bin/sbatch --mem-per-cpu=4000M --job-name test_PR_16953 --ntasks="16" ~/boegelbot/eb_from_pr_upload_jsc-zen2.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 2321

Test results coming soon (I hope)...

- notification for comment with ID 1464092475 processed

Message to humans: this is just bookkeeping information for me,
it is of no use to you (unless you think I have a bug, which I don't).

@boegelbot
Copy link
Collaborator

Test report by @boegelbot
FAILED
Build succeeded for 0 out of 1 (1 easyconfigs in total)
jsczen2g1.int.jsc-zen2.easybuild-test.cluster - Linux Rocky Linux 8.5, x86_64, AMD EPYC 7742 64-Core Processor (zen2), Python 3.6.8
See https://gist.github.com/ce0d5fe3662b926b6b9660332e733be1 for a full test report.

configopts += '-DCMAKE_C_FLAGS="-DH5_USE_110_API" -DCMAKE_CXX_FLAGS="-DH5_USE_110_API" '
# Default options, required for a basic build:
configopts += '-DNEKTAR_USE_HDF5=ON -DNEKTAR_USE_FFTW=ON -DNEKTAR_USE_MPI=ON '
configopts += '-DTHIRDPARTY_BUILD_GSMPI=ON -DNEKTAR_USE_CCM=ON '
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add -DNEKTAR_USE_VTK=ON to this

@Yazan-Essam
Copy link

Ya

@Yazan-Essam
Copy link

(created using eb --new-pr)

Requires:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants