Fix styling bug in 2023 uncertain shorelines #93
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
For full integration test results, refer to the Tests directory README. |
robbibt
added a commit
that referenced
this pull request
Aug 22, 2024
* Test of directly syncing specific tide model subfolders (#86) * Test of directly syncing specific tide model subfolders * Automatically update integration test validation results --------- Co-authored-by: robbibt <robbibt@users.noreply.github.com> * Automatically update integration test validation results * 2023 update (#87) * Update notebooks * 2023 update * Fix pep * Clean notebook * Update docker image base * Update vector.py * Automatically update integration test validation results * Add experiment database access * Update * Update after permissions * Update dockerfile and CI * Update docker * Remove AWS CLI * Add test env * Simplify by installing pytest in setup.py * Add file copying to workflow * Try new paths * Check everything works up to copy step * Try ls * Fix * Try combined approach * Try smaller tests * Fix paths * Automatically update integration test validation results * Try changing locations files are written to * Fix * Update .gitignore * Automatically update integration test validation results * Update stats_tests.csv * Automatically update integration test validation results * Clean repo * Update gitignore * Update test_coastline.py * Update test_coastline.py * Update test_coastline.py * Debug * Debug * Finally fix validation bug * Automatically update integration test validation results * Fix formatting * Fix pep8 * Add image build and artifacts * Automatically update integration test validation results * Update vector.py * Automatically update integration test validation results --------- Co-authored-by: robbibt <robbibt@users.noreply.github.com> * Fix branch reference * Update badge * Update Dockerfile * Automatically update integration test validation results * Update dea-coastlines-image.yaml * Fix styling bug in 2023 uncertain shorelines (#93) * Add fixed style * Automatically update integration test validation results * Update dea-coastlines-image.yaml * Automatically update integration test validation results --------- Co-authored-by: robbibt <robbibt@users.noreply.github.com> * Try old approach for image tagging * Fix workflow name * Update dea-coastlines-image.yaml * Use simpler tagging method that doesn't require git history * Add notebook for automatically generating `.sld` XML styles for GeoServer (#94) * Add notebook for autogenerating coastlines styles * Add notebook for generating XML styles for Geoserver * Revert CLI notebook --------- Co-authored-by: robbibt <robbibt@users.noreply.github.com> Co-authored-by: ClaireP <claire.phillips@ga.gov.au>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Was accidently plotting the uncertain 2023 style based on 2022 data