Skip to content

Commit

Permalink
Merge branch 'develop' into bugfix/tbeltzun/ndebug
Browse files Browse the repository at this point in the history
  • Loading branch information
tbeltzun committed Oct 12, 2023
2 parents e192b62 + 7d3f608 commit a9a03c4
Show file tree
Hide file tree
Showing 88 changed files with 3,261 additions and 640 deletions.
34 changes: 28 additions & 6 deletions .github/workflows/ci_tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ concurrency:
cancel-in-progress: true

env:
GEOSX_TPL_TAG: 238-63
GEOSX_TPL_TAG: 228-73

jobs:

Expand All @@ -23,9 +23,12 @@ jobs:
steps:
- name: Check that the PR is not a draft (cancel rest of jobs otherwise)
run: |
echo "Is PR a draft?"
echo ${{ toJSON(github.event.pull_request.draft) }}
if [[ ${{ toJSON(github.event.pull_request.draft) }} == true ]]; then "false" ; else "true"; fi
draft_status=$(curl -H "Accept: application/vnd.github+json" \
https://api.github.com/repos/${{ github.repository }}/pulls/${{ github.event.number }} \
| jq '.draft')
echo "Is PR a draft? (Check PR's draft status )"
echo $draft_status
if [[ $draft_status == false ]]; then "true" ; else "false"; fi
# PR must be assigned to be merged.
# This job will fail if this is not the case.
Expand All @@ -43,6 +46,23 @@ jobs:
echo $id
if [[ $id != null ]]; then "true" ; else "false"; fi
# PR must be labeled "ready to be merged" to run CUDA jobs.
# This job will fail if this is not the case.
# Note: Label should only be added if PR is in merge queue.
check_ready_to_be_merged:
needs: [check_pull_request_is_not_a_draft]
runs-on: ubuntu-22.04
steps:
- name: Check that the PR is ready to be merged
run: |
labels=$(curl -H "Accept: application/vnd.github+json" \
https://api.github.com/repos/${{ github.repository }}/pulls/${{ github.event.number }} \
| jq '.labels')
ready_label="ready to be merged"
echo "Is PR Ready to be Merged? (Check for $ready_label label)"
echo "Labels found are: $labels"
if [[ $labels == *"$ready_label"* ]]; then "true" ; else "false"; fi
check_submodules:
needs: [check_pull_request_is_not_a_draft]
runs-on: ubuntu-22.04
Expand Down Expand Up @@ -164,8 +184,9 @@ jobs:
# CUDA jobs should only be run if PR is ready to merge.
cuda_builds_merge_ready:
name: ${{ matrix.name }}
needs: [check_pull_request_is_not_a_draft]
if: contains( github.event.pull_request.labels.*.name, format('flag{0} ready to be merged', ':'))
needs:
- check_pull_request_is_not_a_draft
- check_ready_to_be_merged
strategy:

# In-progress jobs will not be cancelled if there is a failure
Expand Down Expand Up @@ -226,6 +247,7 @@ jobs:
needs:
- check_pull_request_is_not_a_draft
- check_pull_request_is_assigned
- check_ready_to_be_merged
- check_submodules
- code_style
- documentation
Expand Down
3 changes: 2 additions & 1 deletion .readthedocs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,9 +8,10 @@ build:
apt_packages:
- npm
- plantuml
- texlive
tools:
nodejs: "16"
python: "3.7"
python: "3.10"
jobs:
post_install:
- npm install -g [email protected]
Expand Down
3 changes: 3 additions & 0 deletions host-configs/TOTAL/pangea3-gcc8.4.1-openmpi-4.1.2.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -108,4 +108,7 @@ set(ENABLE_PETSC OFF CACHE BOOL "")
set(ENABLE_HYPRE ON CACHE BOOL "")
set(ENABLE_HYPRE_DEVICE "CUDA" CACHE BOOL "")

# activate workaround for fmt formatter
set(ENABLE_FMT_CONST_FORMATTER_WORKAROUND ON CACHE BOOL "")

include( ${CMAKE_CURRENT_LIST_DIR}/../tpls.cmake )
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@
<EmbeddedSurfaceGenerator
name="SurfaceGenerator"
logLevel="1"
discretization="fluidTPFA"
discretization="FE1"
targetRegions="{ Domain, Fracture }"
targetObjects="{ FracturePlane }"
fractureRegion="Fracture"
Expand Down Expand Up @@ -97,6 +97,11 @@
<TwoPointFluxApproximation
name="fluidTPFA"/>
</FiniteVolume>
<FiniteElements>
<FiniteElementSpace
name="FE1"
order="1"/>
</FiniteElements>
</NumericalMethods>

<ElementRegions>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@
<EmbeddedSurfaceGenerator
name="SurfaceGenerator"
logLevel="1"
discretization="fluidTPFA"
discretization="FE1"
targetRegions="{ Domain, Fracture }"
targetObjects="{ FracturePlane }"
fractureRegion="Fracture"
Expand Down Expand Up @@ -98,6 +98,11 @@
name="fluidTPFA"
usePEDFM="1"/>
</FiniteVolume>
<FiniteElements>
<FiniteElementSpace
name="FE1"
order="1"/>
</FiniteElements>
</NumericalMethods>

<ElementRegions>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
useMass="0"
targetRegions="{ Domain, Fracture }">
<NonlinearSolverParameters
newtonTol="1.0e-10"
newtonTol="1.0e-8"
newtonMaxIter="15"
maxTimeStepCuts="2"
lineSearchMaxCuts="2"/>
Expand All @@ -23,7 +23,7 @@
<EmbeddedSurfaceGenerator
name="SurfaceGenerator"
logLevel="1"
discretization="fluidTPFA"
discretization="FE1"
targetRegions="{ Domain, Fracture }"
targetObjects="{ FracturePlane }"
fractureRegion="Fracture"/>
Expand Down Expand Up @@ -99,6 +99,11 @@
<TwoPointFluxApproximation
name="fluidTPFA"/>
</FiniteVolume>
<FiniteElements>
<FiniteElementSpace
name="FE1"
order="1"/>
</FiniteElements>
</NumericalMethods>

<ElementRegions>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@
<EmbeddedSurfaceGenerator
name="SurfaceGenerator"
logLevel="1"
discretization="fluidTPFA"
discretization="FE1"
targetRegions="{ Domain, Fracture }"
targetObjects="{ FracturePlane }"
fractureRegion="Fracture"
Expand Down Expand Up @@ -98,6 +98,11 @@
name="fluidTPFA"
usePEDFM="1"/>
</FiniteVolume>
<FiniteElements>
<FiniteElementSpace
name="FE1"
order="1"/>
</FiniteElements>
</NumericalMethods>

<ElementRegions>
Expand Down
Loading

0 comments on commit a9a03c4

Please sign in to comment.