Conversation
And correct some docstrings. Closes #67
(multi)polygon of the osm relation
Change boundary polygon to fitting dtype for unpickling Update test data
Added reference notebook for the method used with LTNs. Approach can calculate the enclosed population of any city/(multi)polygon on the world. The data source is downloaded automatically and dynamically.
+ Url conversion
For area and population calculation Connected to the partitioner structure
Explicitly not in repository
Fix missing full raster reference
Create it if it does not exist in download function. Tested.
Used for population density
Fix working directory to package install location and connect further directories relatively
When single edges are split up in the middle, population and area are split in half and a new cell id is assigned for one half. This is still compatible with digraphs
Corrected the projection order in the graph import. Edge length was calculated in wrong projection.
Parallelized loop, old for loop was very inefficient
Moved module out of metric subpackage, it does not directly belong to the `Metric` object
Basic graph stats as well as population density
Removed valid test case
Half-precision induces overflow in some cases
Black changed the pngs in the CI, this is unwanted.
This was
linked to
issues
Jun 20, 2023
Closed
Contributor
Coverage reportThe coverage rate went from None of the new lines are part of the tested code. Therefore, there is no coverage data about them. |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Closes #57, #64, and #67