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

Use absolute S3 link for label data upload and switch DB log level to debug #5164

Merged
merged 4 commits into from
Sep 12, 2019

Conversation

aaronxsu
Copy link
Member

@aaronxsu aaronxsu commented Sep 11, 2019

Overview

This PR:

  • fixes the upload link of the label data so that it uses absolute S3 link
  • switches database log level back from info to debug

Checklist

  • Description of PR is in an appropriate section of the changelog and grouped with similar changes if possible
  • Swagger specification updated
  • New tables and queries have appropriate indices added
  • Any content changes are properly templated using BUILDCONFIG.APP_NAME
  • Any new SQL strings have tests

Testing Instructions

  • api/assembly
  • batch/assembly
  • spin up servers
  • bring up annotate and point it to this branch (steps)
  • log in annotate as the dev user
  • create a chip classification project
  • label some tasks
  • go to the Exports tab and create an export of the labeled tasks
  • go to Florence Alabama project and create an export of the validated tasks
  • docker-compose build batch
  • ./scripts/console batch bash
  • java -cp /opt/raster-foundry/jars/batch-assembly.jar com.rasterfoundry.batch.Main write_stac_catalog <stac export ID of the chip classification project>
  • java -cp /opt/raster-foundry/jars/batch-assembly.jar com.rasterfoundry.batch.Main write_stac_catalog <stac export ID of the Florence Alabama project>
  • after the exports are done: aws s3 sync <s3 prefix to the export> . --profile raster-foundry
  • make sure both exports have data.geojson and view them in a geojson viewer

Closes #5163
Closes #5162

@aaronxsu aaronxsu requested review from jisantuc and removed request for notthatbreezy September 11, 2019 19:29
@aaronxsu aaronxsu assigned jisantuc and unassigned notthatbreezy Sep 11, 2019
Copy link
Contributor

@jisantuc jisantuc left a comment

Choose a reason for hiding this comment

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

Works -- your call on whether you agree about changelog placement, but I don't need to review again either way 🚀

CHANGELOG.md Outdated Show resolved Hide resolved
@aaronxsu aaronxsu merged commit 3d92654 into develop Sep 12, 2019
@aaronxsu aaronxsu deleted the feature/asu/fix-data-upload-s3-link branch September 12, 2019 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants