CI: Add PDAF builds#82
Conversation
includes: commenting out unused variables in PDAF code
in order to get Oasis-files compiled correctly
Error ``` parflow_pdaf/pfsimulator/parflow_lib/solver_richards.c:1817:16: error: unused variable ‘pfl_start’ [-Werror=unused-variable] ``` Background: In `PseudoAdvanceRichards`, there are many unused variables.
branch matching current generic ParFlow from TSMP2
|
The issue seems to be the way ParFlow linker flags are manually specified: TSMP2/cmake/BuildPDAFMODEL.cmake Lines 98 to 104 in f45128a The correct linker flags for ParFlow 3rd party dependencies (e.g. CUDA, Hypre) are generated by the ParFlow CMake scripts. Unfortunately TSMP2 cannot directly access these flags since they are dynamically generated. We need to find a way to extract these linker info from ParFlow. |
Error from CI ``` ./build_tsmp2.sh CLM35 ParFlow PDAF --OASIS_SRC /home/runner/work/TSMP2/TSMP2/oasis3-mct --no_update Unknown parameter passed: CLM35 ```
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
@kvrigor With an update in loading SLURM/HYPRE, the eCLM/CLM3.5-ParFlow-PDAF builds now complete! Side-Note: Temporary CI failureI get a strange ICON standalone failure, but this should not be part of this CI. Note: ICON standalone was failing due to temprorary connection failure to https://gitlab.dkrz.de/icon/icon-model re-running these checks, they complete (as they should with no changes wrt ICON). |
This turns the PR into
I am looking forward to the additional PDAF-related CI-tests! |
Started from issue #81
Opening PR for testing CI.
Related PDAF-PR: