Release 2.3.0 #88
Merged
Release 2.3.0 #88
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- add missing imports, requirements etc.
with version 2.2.0 some braking changes were made, this commit fixes them
known issues: - existing docker containers are broken - logging not fully working - index creation takes very long - data source is not included in repo and fetching script is missing - other smaller bugs possible...
- small script that converts a GPX file to a JSON (in LV95 format)
- remove old prerequisites: those two points are no longer needed as the functionality is now extracted to a separate docker container
- creates a new docker container containing the code for the name finding - exposes a simple API to query swiss names - code refactorization
- include API call to walk-time table generation procedure -> application is now running again.
- add logging - eager caching of index, keep index in memory even after a request
- add street intersections - add leisure areals - improve forest entry names by adding flurname
Fix/fix missed extremas
…i/automatic_walk-time_tables into feature/speedup-name-finding
maede97
previously approved these changes
Aug 4, 2022
Member
maede97
left a comment
maede97
left a comment
There was a problem hiding this comment.
Also:
- Ich habe mal den ersten Batch der Files durchgesehen. Grundsätzlich voll i.O., nur "minor typos".
- Stresstest mit 4 Tabs gleichzeitig war auch super erfolgreich und gab keine Fehler!
- Ich glaube, es ist ready to merge!
webinterface/src/app/pages/export-settings/export-settings.component.html
Outdated
Show resolved
Hide resolved
python_program/automatic_walk_time_tables/Command_Line_Arguments.md
Outdated
Show resolved
Hide resolved
python_program/automatic_walk_time_tables/map_downloader/create_map.py
Outdated
Show resolved
Hide resolved
maede97
reviewed
Aug 4, 2022
Member
maede97
left a comment
maede97
left a comment
There was a problem hiding this comment.
Not tried yet, but just an idea - why not link to the repo at the current commit?
webinterface/src/app/components/footer/app-footer.component.html
Outdated
Show resolved
Hide resolved
Yes, this is indeed working. Co-authored-by: Matthias Hüppi <[email protected]>
Co-authored-by: Matthias Hüppi <[email protected]>
Co-authored-by: Matthias Hüppi <[email protected]>
Co-authored-by: Matthias Hüppi <[email protected]>
…e_map.py Co-authored-by: Matthias Hüppi <[email protected]>
Co-authored-by: Matthias Hüppi <[email protected]>
Co-authored-by: Matthias Hüppi <[email protected]>
Co-authored-by: Matthias Hüppi <[email protected]>
…ted, if you run the frontend with `npm start` for local development
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What's new with this release
Preview
A preview of this version is hosted at https://dev.map.cevi.tools/.
Open Todo's
Known Issues