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

Ndgeojson done #38

Merged
merged 7 commits into from Jan 18, 2019
Merged

Ndgeojson done #38

merged 7 commits into from Jan 18, 2019

Conversation

sckott
Copy link
Contributor

@sckott sckott commented Jan 18, 2019

No description provided.

working on a new read line by line processor for reading ndgeojson files
commented out geo_bbox in the print.geojson fxn for now as its really slow #37
#36 for lage files running into protection stack overflow errors with ndgeo_read
just grab first 5 features with jqs limit param, then get the whatever from those features
also moved grabbing number of features and first 5 in as.geojson out of print method to avoid
long running waits on printing outputs
@sckott sckott merged commit 68ba564 into master Jan 18, 2019
@sckott sckott added this to the v0.3 milestone Jan 18, 2019
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.

None yet

1 participant