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

ohsome filters #253

Merged
merged 16 commits into from
Nov 9, 2020
Merged

ohsome filters #253

merged 16 commits into from
Nov 9, 2020

Conversation

tyrasd
Copy link
Member

@tyrasd tyrasd commented Jun 29, 2020

Allows to use an ohsome filter directly in an oshdb query: mapReducer.filter("amenity=bench and type:node or leisure=park and geometry:polygon");. This includes the optimizations implemented in the ohsome-api.

todo:

  • add to documentation
  • write unit tests
  • use stable release instead of snapshot of ohsome-filter
  • test serialization (for ignite)

Changes proposed in this pull request:

Type of change

  • New feature (non-breaking change which adds functionality)
  • This change requires a documentation update

New or changed dependencies:

  • ohsome-filter (version 1.4.0)

Checklist

🚧 wip 🚧

@tyrasd tyrasd added enhancement New feature or request user experience Enhances the usability of OSHDB labels Jun 29, 2020
@tyrasd tyrasd changed the title Feature filter library 🚧 ohsome filters Jun 29, 2020
@tyrasd tyrasd changed the title 🚧 ohsome filters ohsome filters Aug 21, 2020
@tyrasd tyrasd added the waiting for review This pull request needs a code review label Aug 21, 2020
@tyrasd tyrasd added this to the release 0.6.0 milestone Aug 28, 2020
@tyrasd tyrasd changed the title ohsome filters 🚧 ohsome filters Oct 20, 2020
@joker234 joker234 added waiting for review This pull request needs a code review and removed waiting for review This pull request needs a code review labels Oct 27, 2020
@tyrasd tyrasd mentioned this pull request Nov 3, 2020
@tyrasd tyrasd changed the title 🚧 ohsome filters ohsome filters Nov 5, 2020
@tyrasd
Copy link
Member Author

tyrasd commented Nov 5, 2020

this should be ready to be tested now. only open question from my side would be if we should update the documentation and examples now, or during the release of v0.6?

@tyrasd tyrasd requested review from rtroilo and joker234 and removed request for rtroilo November 5, 2020 16:41
Copy link
Member

@joker234 joker234 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good apart from some smaller issues. I suppose the tests cover all important units and cases, that are not yet already tested in the ohsome filter library.

What's with the example/benchmark checkpoint? Is this still an open TODO?

@joker234
Copy link
Member

joker234 commented Nov 9, 2020

this should be ready to be tested now. only open question from my side would be if we should update the documentation and examples now, or during the release of v0.6?

Would you redo the documentation and examples totally for the next release? If yes, I think we can do that until then. If you only want to add something about ohsome filters, why not add this within this PR?

tyrasd and others added 2 commits November 9, 2020 18:59
Co-authored-by: Johannes Visintini <johannes.visintini@heigit.org>
@tyrasd
Copy link
Member Author

tyrasd commented Nov 9, 2020

What's with the example/benchmark checkpoint? Is this still an open TODO?

that was mean more like a question to you guys. I guess we can leave it as is and rework the examples & benchmark at a later time: they do still work with the "old" syntax (.osmTag, etc). but at some point one could switch them as well.

@tyrasd
Copy link
Member Author

tyrasd commented Nov 9, 2020

Would you redo the documentation and examples totally for the next release?

That's also a somewhat open question. IMHO we can leave it as is (the new filter is documented as a variant/option in this PR), and maybe redo the whole documentation for the release after the next (and maybe also deprecate the "old" filters in that process as well)

@joker234 joker234 self-requested a review November 9, 2020 19:50
@joker234
Copy link
Member

joker234 commented Nov 9, 2020

I would say we redo the documentation separately at least for the 1.0 version. @tyrasd do you want to create an issue for that?

@joker234 joker234 merged commit 5857f3b into master Nov 9, 2020
@joker234 joker234 deleted the feature-filter-library branch November 9, 2020 20:06
@joker234 joker234 removed the waiting for review This pull request needs a code review label Nov 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request user experience Enhances the usability of OSHDB
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants