Skip to content

Commit

Permalink
Make Github actions run on renamed develop branch (#87)
Browse files Browse the repository at this point in the history
* Make Github actions run on renamed develop branch

* Update dea-intertidal-image.yml

* Automatically update integration test validation results

---------

Co-authored-by: robbibt <[email protected]>
  • Loading branch information
robbibt and robbibt authored Aug 2, 2024
1 parent 00fdde3 commit cddc8db
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 6 deletions.
10 changes: 5 additions & 5 deletions .github/workflows/dea-intertidal-image.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ env:
on:
push:
branches:
- main
- develop
- tests
paths:
- 'intertidal/**'
Expand All @@ -20,7 +20,7 @@ on:

pull_request:
branches:
- main
- develop
paths:
- 'intertidal/**'
- 'data/**'
Expand Down Expand Up @@ -135,8 +135,8 @@ jobs:
needs: [test]
runs-on: ubuntu-latest

# Only run on a push to the main branch OR a release
if: (github.event_name == 'push' && github.ref == 'refs/heads/main') || (github.event_name == 'release')
# Only run on a push to the develop branch OR a release
if: (github.event_name == 'push' && github.ref == 'refs/heads/develop') || (github.event_name == 'release')

steps:
- name: Checkout code
Expand All @@ -161,7 +161,7 @@ jobs:
image_name: ${{ env.IMAGE_NAME }}
image_tag: ${{ env.RELEASE }}

- name: Get git commit hash for push to main
- name: Get git commit hash for unstable image name
if: github.event_name != 'release'
run: |
echo "TAG=dev$(git rev-parse --short HEAD)" \
Expand Down
2 changes: 1 addition & 1 deletion tests/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ Integration tests
This directory contains tests that are run to verify that DEA Intertidal code runs correctly. The ``test_intertidal.py`` file runs a small-scale full workflow analysis over an intertidal flat in the Gulf of Carpentaria using the DEA Intertidal [Command Line Interface (CLI) tools](../notebooks/Intertidal_CLI.ipynb), and compares these results against a LiDAR validation DEM to produce some simple accuracy metrics.

The latest integration test completed at **2024-05-07 10:13**. Compared to the previous run, it had an:
The latest integration test completed at **2024-08-02 14:52**. Compared to the previous run, it had an:
- RMSE accuracy of **0.14 m ( :heavy_minus_sign: no change)**
- MAE accuracy of **0.12 m ( :heavy_minus_sign: no change)**
- Bias of **0.12 m ( :heavy_minus_sign: no change)**
Expand Down
1 change: 1 addition & 0 deletions tests/validation.csv
Original file line number Diff line number Diff line change
Expand Up @@ -58,3 +58,4 @@ time,Correlation,RMSE,MAE,R-squared,Bias,Regression slope
2024-04-23 05:49:24.645404+00:00,0.975,0.141,0.121,0.95,0.116,1.11
2024-05-06 22:50:22.431050+00:00,0.975,0.141,0.121,0.95,0.116,1.11
2024-05-07 00:13:21.589490+00:00,0.975,0.141,0.121,0.95,0.116,1.11
2024-08-02 04:52:31.336765+00:00,0.975,0.141,0.121,0.95,0.116,1.11
Binary file modified tests/validation.jpg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.

0 comments on commit cddc8db

Please sign in to comment.