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

[#62] Use updated hsmodels #63

Merged
merged 8 commits into from
May 16, 2024
Merged

[#62] Use updated hsmodels #63

merged 8 commits into from
May 16, 2024

Conversation

pkdash
Copy link
Member

@pkdash pkdash commented Apr 16, 2024

No description provided.

@pkdash
Copy link
Member Author

pkdash commented Apr 18, 2024

Tests failed as hydroshare beta was down.

@pkdash pkdash marked this pull request as ready for review April 25, 2024 22:29
@pkdash pkdash requested a review from sblack-usu April 26, 2024 02:32
@pkdash
Copy link
Member Author

pkdash commented May 2, 2024

Tests are failing as beta is down.

@pkdash
Copy link
Member Author

pkdash commented May 6, 2024

Tests ran successfully today!

"source": [
"# retrieve the updated georaster aggregation to verify the data was updated\n",
"\n",
"# need to first download this updated/new aggregation\n",
"download_to = r\"D:\\Temp\\GeoRaster_Testing\"\n",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is this intentional? This looks like a Windows file path.

Copy link
Member Author

@pkdash pkdash May 15, 2024

Choose a reason for hiding this comment

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

Seems like left over from my testing code. Fixed these file paths.

@pkdash pkdash merged commit 0117347 into master May 16, 2024
1 check passed
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