Skip to content
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

Edit ADRIA.Domain() and generalize to incorporate loading of GBR-wide domain package #904

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

BG-AIMS
Copy link
Collaborator

@BG-AIMS BG-AIMS commented Jan 8, 2025

Minor changes required to generalize code away from reef cluster/site level Domains and allow GBR-wide ADRIADomains to be accepted.

@BG-AIMS BG-AIMS self-assigned this Jan 8, 2025
@BG-AIMS BG-AIMS added the enhancement New feature or request label Jan 8, 2025
Copy link
Collaborator

@DanTanAtAims DanTanAtAims Jan 8, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The location of the geopackage is correctly documented in the datapackage.json file, how load_domain does not currently use this. This causes an error during load_domain.

ERROR: Provided location data path is not valid or missing: C:/Users/dtan/data/GBR_2024_10_15\spatial\GBR_2024_10_15.gpkg.
Stacktrace:

Note: This Line, ADRIA currently uses the name of the directory as the name of the geopackage, this should be changed to use the path provided in datapackage.json.

This information is stored in the Moore Domain, however the most recent version v070_rc1 incorrectly lists the spatial gpkg as v060_rc1

Comment on lines +220 to +221
# ReefMod DHW timeseries data (2000:2100) do not match the length of waves/cyclone
# data (2025:2099). Can be fixed when AIMS-sourced DHW trajectories become available.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
# ReefMod DHW timeseries data (2000:2100) do not match the length of waves/cyclone
# data (2025:2099). Can be fixed when AIMS-sourced DHW trajectories become available.
# ReefMod DHW timeseries data (2000:2100) do not match the length of waves/cyclone
# data (2025:2099). Can be fixed when AIMS-sourced DHW trajectories become available.

Whitespace

location_data.k .= location_data.k / 100.0 # Make `k` non-dimensional (provided as a percent)
if ("k" ∉ names(location_data)) &
("ReefMod_habitable_proportion" ∈ names(location_data))
# k column not found in gbr-wide canonical-reefs gpkg.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
# k column not found in gbr-wide canonical-reefs gpkg.
# k column not found in gbr-wide canonical-reefs gpkg.

Copy link
Collaborator

@DanTanAtAims DanTanAtAims Jan 8, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some ADRIA functions assumes the column area exists in the geopackage dataframe. The canonical reefs gpkg currently labels this column as ReefMod_area_m2. This prevents the model from running.

ERROR: ArgumentError: column name :area not found in the data frame

@DanTanAtAims DanTanAtAims self-assigned this Jan 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants