Skip to content

Commit

Permalink
update lint
Browse files Browse the repository at this point in the history
  • Loading branch information
mlee03 authored and mlee03 committed Jul 2, 2024
2 parents a1040a1 + cf4c36d commit 24b8675
Show file tree
Hide file tree
Showing 6 changed files with 19 additions and 15 deletions.
8 changes: 4 additions & 4 deletions .github/workflows/fv3_translate_tests.yaml
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
name: "FV3 Translate tests"
name: "FV3 translate tests"
on:
pull_request:
types: [opened, synchronize, reopened, ready_for_review, labeled, unlabeled]

jobs:
all_tests:
fv3_translate_tests:
uses: NOAA-GFDL/pyFV3/.github/workflows/translate.yaml@develop
with:
ndsl_trigger: true
component_trigger: true
component_name: NDSL
1 change: 0 additions & 1 deletion .github/workflows/lint.yaml
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
name: "Lint"
on:
pull_request:
types: [opened, synchronize, reopened, ready_for_review, labeled, unlabeled]

# cancel running jobs if theres a newer push
concurrency:
Expand Down
8 changes: 4 additions & 4 deletions .github/workflows/pace_tests.yaml
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
name: "pace unit tests"
name: "pace main tests"
on:
pull_request:
types: [opened, synchronize, reopened, ready_for_review, labeled, unlabeled]

jobs:
pace_unit_tests:
pace_main_tests:
uses: NOAA-GFDL/pace/.github/workflows/main_unit_tests.yaml@develop
with:
ndsl_trigger: true
component_trigger: true
component_name: NDSL
4 changes: 2 additions & 2 deletions .github/workflows/shield_tests.yaml
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
name: "SHiELD Translate tests"
on:
pull_request:
types: [opened, synchronize, reopened, ready_for_review, labeled, unlabeled]

jobs:
shield_translate_tests:
uses: NOAA-GFDL/pySHiELD/.github/workflows/translate.yaml@develop
with:
ndsl_trigger: true
component_trigger: true
component_name: NDSL
10 changes: 7 additions & 3 deletions .github/workflows/unit_tests.yaml
Original file line number Diff line number Diff line change
@@ -1,29 +1,33 @@
name: "NDSL Unit tests"
name: "NDSL unit tests"
on:
pull_request:
types: [opened, synchronize, reopened, ready_for_review, labeled, unlabeled]

# cancel running jobs if theres a newer push
concurrency:
group: ${{ github.workflow }}-${{ github.ref }}
cancel-in-progress: true

jobs:
all_ndsl_tests:
ndsl_unit_tests:
runs-on: ubuntu-latest
container:
image: ghcr.io/noaa-gfdl/miniforge:mpich
steps:

- name: Checkout repository
uses: actions/checkout@v4
with:
submodules: 'recursive'

- name: Install Python packages
run: pip3 install .[test]

- name: Run serial-cpu tests
run: coverage run --rcfile=setup.cfg -m pytest -x tests

- name: Run parallel-cpu tests
run: mpiexec -np 6 --oversubscribe coverage run --rcfile=setup.cfg -m mpi4py -m pytest -x tests/mpi

- name: Output code coverage
run: |
coverage combine
Expand Down
3 changes: 2 additions & 1 deletion ndsl/stencils/corners.py
Original file line number Diff line number Diff line change
Expand Up @@ -989,6 +989,7 @@ def fill_corners_dgrid_defn(
x_out: FloatField,
y_in: FloatField,
y_out: FloatField,
mysign: float,
):
"""
Args:
Expand All @@ -997,7 +998,7 @@ def fill_corners_dgrid_defn(
y_in (in):
y_out (inout):
"""
from __externals__ import i_end, i_start, j_end, j_start, mysign
from __externals__ import i_end, i_start, j_end, j_start

with computation(PARALLEL), interval(...):
# sw corner
Expand Down

0 comments on commit 24b8675

Please sign in to comment.