Skip to content
This repository has been archived by the owner on Jan 9, 2023. It is now read-only.

#1201 - Fix issue with setting source date when using .kip directories #1234

Merged
merged 2 commits into from
Mar 26, 2019
Merged

#1201 - Fix issue with setting source date when using .kip directories #1234

merged 2 commits into from
Mar 26, 2019

Conversation

tst-ccamp
Copy link
Collaborator

@tst-ccamp tst-ccamp commented Mar 21, 2019 •
edited
Loading

Verification steps:

  1. Build 2 imagery resources with date set (suggestion: Blue Marble and usgsLanSat from tutorial)
  2. Copy the 2 prebuilt resources into source directory, for Blue Marble just get the product raster.kip directory (you can rename it), for the USGS image get the whole asset (name.kiasset directory)
  3. Create imagery resources using command line utility e.g. genewimageryresource -o Resources/Imagery/usgsLanSat_kip1 /gevol/src/Imagery/usgsLanSat.kiasset/product.kia/ver001/raster.kip
  4. Before building the images, set a unique date for the new Blue Marble resource
  5. Create a new Imagery project and database for the 2 new resources
  6. Build and publish database
  7. Load published database in Google Earth EC
  8. The date for Blue Marble should be the new date set for the resource while the USGS image should be the original date that was set

Notes:

  • If you end up changing a date, you will need to clean the resource version and rebuild the project and database before the new date will be used

@googlebot googlebot added the cla: yes Manual verification that all contributors have signed the CLA. label Mar 21, 2019
@tst-ccamp tst-ccamp changed the base branch from master to release_5.2.5.2 March 21, 2019 23:12
@tst-dsugar
Copy link
Collaborator

This looks like it was really simple to solve, hopping the acquisition date across like that, but completely non obvious ...

tst-mswartz
tst-mswartz approved these changes Mar 25, 2019
View reviewed changes
Copy link
Collaborator

@tst-mswartz tst-mswartz left a comment

Choose a reason for hiding this comment

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

Looks good to me. Very clean solution!

@tst-dsugar tst-dsugar self-requested a review March 25, 2019 20:18
@tst-dsugar
Copy link
Collaborator

I was able to try it.

@tst-ccamp tst-ccamp merged commit 47e6fcd into google:release_5.2.5.2 Mar 26, 2019
@tst-ccamp tst-ccamp deleted the 1201-kip-date-fix branch March 26, 2019 18:45
@tst-lsavoie tst-lsavoie mentioned this pull request Apr 15, 2019
Closed
husf-dsheremata added a commit to husf-dsheremata/earthenterprise that referenced this pull request Jun 4, 2019
@husf-dsheremata
Merge branch 'master' of https://github.com/google/earthenterprise
9730474 
* 'master' of https://github.com/google/earthenterprise:
  1270 Fix xml parsing in common.sh (google#1271)
  1192 update openjpeg to 2.3.1 (google#1269)
  Documented required boost dependency for Ubuntu 14 (google#1264)
  [WIP] Resolve issue google#798 - investigate Qt build performance (google#1245)
  Copy uninstall scripts (google#1232)
  refresh commit
  Issue google#1255 - Limits internal version string to 3 parts to fix various errors with patch releases (google#1256)
  Move build number to release rpm (google#1252)
  fix version
  link up 5.3.1 doc folder (google#1249)
  I1246 create 5.3.1 doc folder from copy of 5.3.0 doc folder (google#1248)
  change version files over to 5.3.1 (google#1247)
  start of 5.3.1
  Fixes google#279 - Admin Console password will be preserved on GEE server upgrade (google#1243)
  google#1201 - Fix issue with setting source date when using .kip directories (google#1234)
  I1228 Add y-scroll-bar to GLC Assembly webpage (google#1230)
  google#1236 Polygons constructor can accept KML as a string (google#1239)
  I1225 Re-add GROUPNAME to install_server.sh (google#1233)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@tst-mswartz tst-mswartz tst-mswartz approved these changes

@tst-dsugar tst-dsugar tst-dsugar approved these changes

Assignees
No one assigned
Labels
cla: yes Manual verification that all contributors have signed the CLA.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@tst-ccamp @tst-dsugar @tst-mswartz @googlebot