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.

Sign up for GitHub

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

Jump to bottom

S3 raster storage functionality #66

Merged
merged 21 commits into from
Jun 6, 2022
Merged

S3 raster storage functionality #66

merged 21 commits into from
Jun 6, 2022

Conversation

micah-prime
Copy link
Contributor

Functionality for #65

  • Raster uploader creates COGs and either stores locally or in S3 bucket
  • Raster tests use locally stored COGS
  • configuration script allows for VSI parameters so that DB can read from private S3 buckets

@micah-prime micah-prime added the enhancement New feature or request label Jun 2, 2022
@micah-prime micah-prime self-assigned this Jun 2, 2022
@micah-prime
try slightly different pwd for volume mount since this works locally …
a930cd0 
…on mac, but not in github actions
@micah-prime
Try a more direct path mount since PWD does not work on the github ru…
5587439 
…nner
@micah-prime
We should install dev requirements for testing
90f814a
@micah-prime
Python 3.6 is EOL
4437eb0
@micah-prime
#65 - fixing tests
d568c9f
@micah-prime
#65 - more test fixes
fb2f2a0
@micah-prime
#65 - a lot of expected counts have changed with the new raster stora…
2c0693d 
…ge scheme
@micah-prime
#65 - tilling all rasters caused an issue in TestFunctions::test_pixe…
30a0d18 
…l_as_point. Limit the return to 1 in this test so that we only check the scalar value of the first return
@micah-prime
Tests that check the expected date are off by one on timezone. I chec…
39a9c28 
…ked this on the main branch and the same tests fail. I Changed timezone to MST instead of US/Mountain to not ignore changes in daylight savings and avoid this error. Long term the DB could only handle and return UTC datetimes and leave it up to the user to conver to their desired timezones
@micah-prime
Missed the timezone in the workflow
a24e702
@micah-prime
#65 - install gdal in workflow
a4ef1f8
@micah-prime
wrong line
6a51755
@micah-prime
#65 - fix failing docs hopefully. Had possible non-assignment in modi…
9745864 
…fy_conf script. Update docker database image
@micah-prime
walk the nbconvert requirement back a little
8215ee4
@micah-prime
Copy link
Contributor Author

Closes #65

@micah-prime micah-prime merged commit 8818a83 into master Jun 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@micahjohnson150 micahjohnson150 micahjohnson150 approved these changes

Assignees

@micah-prime micah-prime

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@micah-prime @micahjohnson150