Skip to content

fix(config): correct TIPSv2 embedding filename to crop_384x384 - #16

Open
ZhuoyaoH wants to merge 1 commit into
OpenGeoScope:mainfrom
ZhuoyaoH:fix/tipsv2-crop-name
Open

fix(config): correct TIPSv2 embedding filename to crop_384x384#16
ZhuoyaoH wants to merge 1 commit into
OpenGeoScope:mainfrom
ZhuoyaoH:fix/tipsv2-crop-name

Conversation

@ZhuoyaoH

@ZhuoyaoH ZhuoyaoH commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Summary

The TIPSv2 embedding dataset file was misnamed TIPSv2_b14_crop_448x448.parquet. Source patches in Core-S2L2A-249k are fixed at 384x384; generation took the full 384x384 patch and resized it to the 448x448 model input, so no 448x448 crop ever existed. Per the project naming convention (crop size = source patch crop, e.g. FarSLIP crop 384 / input 224), the correct name is TIPSv2_b14_crop_384x384.parquet.

Changes on the dataset side (already done in Major-TOM/Core-S2RGB-249k-TIPSv2):

  • Uploaded the renamed file TIPSv2_b14_crop_384x384.parquet (byte-verified, 815,473,248 bytes).
  • Corrected the pixel_bbox column from [0,0,448,448] to [0,0,384,384] (embeddings, unique_id and row count verified unchanged).
  • Updated the dataset README (filename, crop-size clarification).
  • The old misnamed file will be removed once this PR merges.

This PR updates the only in-repo reference: configs/config.yaml tipsv2.embedding_path.

Impact

None on retrieval: pixel_bbox is not consumed anywhere in the app code, and embeddings are untouched.

Test plan

  • App loads TIPSv2 embeddings from the renamed file via ms://Major-TOM/Core-S2RGB-249k-TIPSv2/TIPSv2_b14_crop_384x384.parquet.

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.

1 participant