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

Add migration for change in summaries properties #330

Closed
lossyrob opened this issue May 4, 2021 · 0 comments · Fixed by #447
Closed

Add migration for change in summaries properties #330

lossyrob opened this issue May 4, 2021 · 0 comments · Fixed by #447
Assignees
@duckontheweb
Milestone
1.0.0-rc.1

Comments

@lossyrob
Copy link
Member

lossyrob commented May 4, 2021

radiantearth/stac-spec#967

Summaries changed from min/max to minimum/maximum. Previous to #309 we were not parsing out the stats object stats object explicitly, and that PR adds a RangeSummary object with a minimum and maximum correctly named, so we only need to add functionality to migrate summary objects.
@cholmes cholmes added this to the 1.0.0 milestone May 24, 2021
@duckontheweb duckontheweb modified the milestones: 1.0.0, 1.0.0-rc.1 May 24, 2021
@duckontheweb duckontheweb self-assigned this Jun 15, 2021
@duckontheweb duckontheweb mentioned this issue Jun 15, 2021
Merged
4 tasks
duckontheweb pushed a commit that referenced this issue Jun 16, 2021
Merge pull request #447 from duckontheweb/add/gh-330-summary-migration
12866c4 
Add migration for pre-1.0.0-rc.1 Stats Object
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@duckontheweb duckontheweb

Labels
None yet
Projects
None yet
Milestone
1.0.0-rc.1
Development

Successfully merging a pull request may close this issue.

Add migration for pre-1.0.0-rc.1 Stats Object duckontheweb/pystac
3 participants
@cholmes @lossyrob @duckontheweb