-
Notifications
You must be signed in to change notification settings - Fork 27
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
Add Hypothesis tests #179
Comments
I'm interested in this issue. Is there a contribution guide? Should I work from the |
No guideline, sorry. Fork, and make your PR against |
Commit often, commit early, make a draft PR while you are still working on it, so I can give you some pointers ;-) |
After you cloned your fork, cd into the directory and
|
Sounds good. I'm more of a |
I am more of a minimalist 😄 |
Draft PR here: Just adding the Hypothesis dependency and making a small start checking for encode/decode invariance with I'm not super familiar with different spatial reference systems. Do you have in mind that these strategies would generate different values depending on the SRS provided? Happy to take the conversation there. |
Another approach may be using hypothesis-jsonschema and the geo-json schema of The GeoJSON Format |
Are you still working on this, or did you loose interest? |
Didn't lose interest, just lost time. I will budget time for this tomorrow. |
Test
pygeoif
against these strategies,repr
andgeometry.wkt
are Encode/Decode invariantThe strategies should be similar to hypothesis-geojson, but one should be able to create coordinates for any given CRS (SRS) with EPSG 4326 as a predefined shortcut
The text was updated successfully, but these errors were encountered: