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

Connectivity modelling update #105

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Conversation

erialC-P
Copy link
Collaborator

During our CEM release phase, we made some changes to the connectivity model that we didn't transfer into the intertidal repo. This PR commits those changes and makes a minor update to the mangrove shapefile source.
This PR:

  • replaces the single year Global Mangrove Watch shapefile from the connectivity baseline class to now use the unioned GMW extent that we use with our DEA Mangroves product. To support this task, I created a flatgeobuf file of the unioned GMW shapefile and hosted it from our S3 dev bucket.
  • adds code to import and add highest astronomical tide to the connectivity baseline class. This code is transferred directly from gdata1/projects/coastal/intertidal/Connectivity_testing_Robbi.ipynb. To support this task, I also added the CCBY 4.0 licensed HAT modelling from CSIRO onto our S3 dev bucket.

Copy link

For full integration test results, refer to the Tests directory README.

Copy link
Member

@robbibt robbibt left a comment

Choose a reason for hiding this comment

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

Looks great @erialC-P! Ready to squash and merge

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants