-
Notifications
You must be signed in to change notification settings - Fork 410
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
refactor: remove unused AccelerationWeighting #1454
Conversation
This is not a refactoring because the fix changes the behavior of ORSWeightingFactory. Hence the rename from fix to refactor should be reverted. |
@sfendrich What are your thoughts on waiting for refactorings like this after the modularization branch is merged? |
I'm fine with waiting. Just had a bit of time left and thought this would be a good opportunity for minor cleanup. |
My impression, when we discussed it last week, was that there will be a 7.x release very soon so that the modularization PR can be merged right after that. Any new change will potentially create a merge conflict because EVERY JAVA FILE (and most other files) are in a different place now and the pom(s) have also changed in subtle and not so subtle ways. |
I don't think this needs to wait, the merge of the modularization branch will be non-trivial anyway |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes!
The class AccelerationWeighting is unfinished since 2017 and, therefore, removed as unused code.
6d1c3b0
to
b8ae196
Compare
IMO. Not an argument for making it worse and worse. |
The class AccelerationWeighting is unfinished since 2017 and, therefore, removed as unused code.
Pull Request Checklist
have been resolved.
[Unreleased] heading.
along with a short description of what it is for, and documented this in the Pull Request (below).
(at least Germany), and the graphs build without problems (i.e. no out-of-memory errors).
importer etc.), I have generated longer distance routes for the affected profiles with different options
(avoid features, max weight etc.) and compared these with the routes of the same parameters and start/end
points generated from the current live ORS.
If there are differences then the reasoning for these MUST be documented in the pull request.
and why the change was needed.