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

Correct Checkpoint Name #624

Merged
merged 1 commit into from Dec 14, 2018
Merged

Correct Checkpoint Name #624

merged 1 commit into from Dec 14, 2018

Conversation

@jamesmcclain
Copy link
Member

@jamesmcclain jamesmcclain commented Dec 13, 2018

Overview

Fixes dependency checking by noting the correct fine-tuning checkpoint name as a product of the TRAIN stage.

Checklist

  • Ran scripts/format_code and commited any changes
  • Documentation updated if needed
  • PR has a name that won't get you publicly shamed for vagueness

To Test: Run the Vegas experiment twice in a row both with and without these changes.

rastervision run local -e spacenet.vegas -a test True -a use_remote_data True -a root_uri s3://my-bucket/vegas
rastervision run local -e spacenet.vegas -a test True -a use_remote_data True -a root_uri s3://my-bucket/vegas

Closes #608

@jamesmcclain jamesmcclain merged commit f13b34e into azavea:develop Dec 14, 2018
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@jamesmcclain jamesmcclain deleted the jamesmcclain:redundant branch Dec 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants