-
Notifications
You must be signed in to change notification settings - Fork 3
🚨 Targets workflow failed - Review needed #10
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.
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
github-actions
wants to merge
62
commits into
main
Choose a base branch
from
auto-fix-1768447817
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains hidden or 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
Merge branch 'main' into dev-adam # Conflicts: # R/get_climate_chelsa.R # _targets.R # _targets/meta/meta
Merge branch 'main' into dev-adam # Conflicts: # .gitignore
Merge branch 'main' into dev-adam # Conflicts: # .github/decrypt_secret.sh # .gitignore # R/get_climate_chelsa.R # README_files/figure-gfm/unnamed-chunk-3-1.png # _targets.R
- started working through functions auto summary below: Add functions to download and process NDVI and NDWI data from MODIS and VIIRS - Implemented `get_release_ndvi_dates_modis` to download MODIS NDVI dates and convert to UNIX format. - Created `get_release_ndvi_dates_viirs` for downloading VIIRS NDVI data with appropriate quality checks. - Added `get_release_ndwi_modis` to download the most recent MODIS NDWI layer. - Introduced helper functions for date conversion and quality assessment. - Ensured proper directory management and error handling during downloads. - Updated documentation for clarity and usage instructions.
… and VIIRS - Implemented `get_release_ndvi_dates_modis` to download MODIS NDVI dates, including functions for date conversion and error handling. - Created `get_release_ndvi_dates_viirs` for downloading VIIRS NDVI dates with appropriate quality checks. - Developed `get_release_ndwi_modis` to fetch the most recent MODIS NDWI layer, including domain formatting and raster processing. - Added error handling and release management for GitHub uploads across all functions. - Ensured compatibility with existing data structures and improved documentation for clarity.
…emma_envdata into dev-adam-appeears
… national boundary function
…ing climate data download URLs for improved accuracy and reliability.
…and clean up unused steps in targets pipeline
…eps in the GitHub Actions workflow
…add local testing for actions
…and maintain consistency.
…parameters for clarity.
…et in _targets.R for clarity.
…gmap and domain_define, remove unused parameters in domain_rasterize, and enhance target definitions in _targets.R.
…_names for improved consistency
…y function, enhance metadata handling in data processing functions
Expanded .gitignore to include more R, RStudio, targets, data, secrets, and output file patterns. Cleaned up and reorganized entries for clarity and removed merge conflict markers.
Introduces new shapefiles (remnants_shp, capenature_fires_shp), raster and NetCDF outputs (domain.tif, vegmap.tif, domain.nc, vegmap.nc), and updates or adds several functions and objects related to spatial data processing. Also updates metadata for country and domain layers, and removes the process_release_biome_raster function.
…line, update country and domain functions to write GeoParquet, and refactor elevation data retrieval with AppEEARS
…ata handling and NetCDF output
…quests, and issues
…ubdirectory and return correct path; update elevation resampling method to average.
…emnants processing, and improve NetCDF variable definitions and metadata handling.
…ndicator description, add comprehensive CRS attributes, and implement run mode for GitHub Actions.
…vegmap, improve elevation retrieval with AppEEARS, and ensure directory structure for temporary files.
… releases; update .gitignore for new data directories
- Update output path for domain bounding box to "data/target_outputs/domain_bbox.parquet" and ensure directory creation. - Convert bounding box to a simple data frame with WKT geometry before writing as parquet. - Improve GitHub release upload process with credential handling and retry logic for uploads. - Adjust cue mode for domain bounding box task based on run mode.
…ode determination
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.
Workflow Failure
The targets pipeline failed. GitHub Copilot will review this PR.
Error Summary
Full Log
See
.github/workflow-logs/for complete output.Next steps: