Skip to content

CI/CD: Update ty, reduce Git LFS bandwidth pressure #124

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.

Sign up for GitHub

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

Open
kopperp wants to merge 2 commits into
base: main
Choose a base branch
from
+19 −15
Open

CI/CD: Update ty, reduce Git LFS bandwidth pressure #124

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
71b3b68
CI/CD: Update rules for ty v0.0.27
kopperp Apr 2, 2026
7114eac
CI/CD: Reduce Git LFS bandwidth pressure
kopperp Apr 2, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
28 changes: 14 additions & 14 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -78,14 +78,14 @@ jobs:
image: python:3.14.3-slim-trixie
steps:
# Git LFS requires git
- name: Install Git LFS
shell: bash
run: |
apt-get update
apt-get install -y --no-install-recommends git git-lfs
# - name: Install Git LFS
# shell: bash
# run: |
# apt-get update
# apt-get install -y --no-install-recommends git git-lfs
- uses: actions/checkout@v6
with:
lfs: true
# with: Reduce Git LFS bandwidth
# lfs: true
- uses: ./.github/actions/setup
# with:
# minimal: 'false'
Expand Down Expand Up @@ -235,17 +235,17 @@ jobs:
password: ${{ secrets.GITHUB_TOKEN }}
steps:
# Git LFS requires git
- name: Install Git LFS
shell: bash
run: |
dnf update -y
dnf install -y git-lfs
# - name: Install Git LFS
# shell: bash
# run: |
# dnf update -y
# dnf install -y git-lfs
# - uses: actions/setup-node@v6
# with:
# node-version: 'latest'
- uses: actions/checkout@v6
with:
lfs: true
# with: Reduce Git LFS bandwidth
# lfs: true
- uses: ./.github/actions/setup
with:
coverage: 'true'
Expand Down
6 changes: 5 additions & 1 deletion pyhope/mesh/transform/mesh_transform.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@
# ----------------------------------------------------------------------------------------------------------------------------------
from __future__ import annotations
from typing import Final
from typing import cast
# ----------------------------------------------------------------------------------------------------------------------------------
# Third-party libraries
# ----------------------------------------------------------------------------------------------------------------------------------
Expand Down Expand Up @@ -91,6 +92,9 @@ def CalcStretching(nZones: int, zone: int, nElems: npt.NDArray, lEdges: npt.NDAr
handler = stretchingHandlers.get(stretchingType)
progFac, l0, dx = handler() if handler else (np.array(()), np.array(()), np.array(()))

if progFac is None or l0 is None or dx is None:
hopout.error(f'Missing parameter {[s for s in (progFac, l0, dx) if s is None]}, exiting...')

if stretchingType == 'combination':
for iDim in range(3):
if np.isclose(progFac[iDim], 0., atol=mesh_vars.tolInternal):
Expand Down Expand Up @@ -146,7 +150,7 @@ def CalcStretching(nZones: int, zone: int, nElems: npt.NDArray, lEdges: npt.NDAr
hopout.error('Stretching factor = 0 is invalid, exiting...')

# Return stretching factor
return progFac
return cast(np.ndarray, progFac)


def TransformMesh() -> None:
Expand Down
Loading