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

Schema version invalid #177

Closed
m-mohr opened this issue Jul 27, 2023 · 11 comments
Closed

Schema version invalid #177

m-mohr opened this issue Jul 27, 2023 · 11 comments
Labels
bug Something isn't working

Comments

@m-mohr
Copy link
Collaborator

m-mohr commented Jul 27, 2023

The version number in the schema is invalid. It is 1.0.0-dev instead of 1.0.0-rc.1

See https://github.com/opengeospatial/geoparquet/releases/download/v1.0.0-rc.1/schema.json

See also: #147 (comment)

@m-mohr m-mohr added the bug Something isn't working label Jul 27, 2023
@m-mohr
Copy link
Collaborator Author

m-mohr commented Jul 27, 2023

The geoparquet.md also says "1.0.0-dev": https://github.com/opengeospatial/geoparquet/releases/download/v1.0.0-rc.1/geoparquet.md

I assume there could be more instances where the replacement doesn't work. Or you forgot to just update the references in #176, but as there was no response to my comment I assumed there would be some kind of replacement magic happen in the release process.

The archives, e.g. https://github.com/opengeospatial/geoparquet/archive/refs/tags/v1.0.0-rc.1.zip, are similarly broken.

@tschaub
Copy link
Collaborator

tschaub commented Jul 27, 2023

Yeah, I was just starting to address this.

I wrote up these steps for creating a release: https://github.com/opengeospatial/geoparquet/wiki/Release-Process

These steps include updating the version numbers scattered throughout the repo before creating the release tag.

At this point, I think it would be fine to delete the rc.1 tag and start over with the steps on the above wiki page.

I would do this, except I am wrestling with the python dependencies required by the scripts that update the example data and example metadata. If anybody can update the instructions to get set up to run the python in the scripts directory, please do so.

@kylebarron
Copy link
Collaborator

I would do this, except I am wrestling with the python dependencies required by the scripts that update the example data and example metadata. If anybody can update the instructions to get set up to run the python in the scripts directory, please do so.

What's the issue you're hitting here?

@tschaub
Copy link
Collaborator

tschaub commented Jul 27, 2023

Failure to install pyarrow. It looks like I had a brew installed version of poetry. Trying again using the poetry installer.

@m-mohr
Copy link
Collaborator Author

m-mohr commented Jul 27, 2023

Great, thanks.

I wrote up these steps for creating a release: https://github.com/opengeospatial/geoparquet/wiki/Release-Process

Maybe put them in a release.md in the main repo? That's much more common, I never even expect someone using/filling the Wiki...

@kylebarron
Copy link
Collaborator

Oh I just tried to install it on my M1 Mac and it failed because it's still using pygeos and that is trying to compile from source for some reason.

Is anyone still using write_nz_building_outline.py or can/should we delete it? @cholmes was the only one I knew who was using it I think.

@tschaub
Copy link
Collaborator

tschaub commented Jul 27, 2023

As you may guess, the release process is not entirely ironed out. I find the wiki easier to edit than creating pull requests for each tweak. But contributions adding a release.md that points to the wiki are welcome. And when everything is solid, we can get rid of the wiki page in favor of a release.md file if people want.

Update: #181 adds a RELEASE.md doc.

@tschaub
Copy link
Collaborator

tschaub commented Jul 27, 2023

I created #178 with some additional detail on the failure I'm hitting with a "fresh" poetry install. Not entirely sure how to start over (perhaps I have stale/conflicting/outdated/bad things installed), but I'll keep trying.

@tschaub
Copy link
Collaborator

tschaub commented Jul 27, 2023

https://github.com/opengeospatial/geoparquet/blob/v1.0.0-rc.1/format-specs/schema.json now has the correct version number (updated in 6a9c43e).

@tschaub tschaub closed this as completed Jul 27, 2023
@cholmes
Copy link
Member

cholmes commented Jul 31, 2023

Hey all, thanks for the help on this - I appreciate it, and sorry for messing it up. Thanks for fixing up my mess.

@tschaub
Copy link
Collaborator

tschaub commented Jul 31, 2023

No problem, @cholmes. @m-mohr was right that the release doc on the wiki page was not easy to find - I even forgot having written it. It was good to tidy up a few things as a result of this. I think #180 is one thing that warrants further discussion before 1.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants