Deal with edge case of zero length paths. #147
Conversation
There was a problem hiding this comment.
Hello, @RadostW,
I am really sorry for not responding so much time. I had a rough time doing pet projects.
Thank you for your contribution! It means a lot! If you are still interesting in the project, could you please take a look at the review.
| @@ -50,12 +50,12 @@ def parse_float(string: str) -> Optional[float]: | |||
|
|
|||
| def parse_levels(string: str) -> list[float]: | |||
| """Parse string representation of level sequence value.""" | |||
| # TODO: add `-` parsing | |||
| # TODO: add `-` parsing | |||
There was a problem hiding this comment.
Running black -l 80 map_machine will fix this.
| if address: | ||
| texts.append(self.label(", ".join(address))) | ||
|
|
||
| # if name: |
| @@ -174,6 +174,9 @@ def __init__(self, tags: dict[str, str], point: np.ndarray) -> None: | |||
|
|
|||
| def draw(self, svg: Drawing, scheme) -> None: | |||
| """Draw gradient sector.""" | |||
|
|
|||
There was a problem hiding this comment.
If sectors are drawn incorrectly, we should disable them with the option.
| @@ -531,7 +531,12 @@ def draw(self, svg: Drawing, is_border: bool) -> None: | |||
| filter_: Filter = self.get_filter(svg, is_border) | |||
|
|
|||
| style: dict[str, Union[int, float, str]] = self.get_style(is_border) | |||
| path_commands: str = self.line.get_path(self.placement_offset) | |||
|
|
|||
There was a problem hiding this comment.
We can do
if (path_commands := self.line.get_path(self.placement_offset) == None:
returnand the same for other tmp_d variables.
| @@ -104,8 +104,13 @@ def __init__( | |||
|
|
|||
| def draw(self, svg: Drawing, flinger: Flinger) -> None: | |||
| """Draw simple building shape.""" | |||
|
|
|||
There was a problem hiding this comment.
There is a comment below about tmp_ variables. We can nicely avoid them.
|
Checks had failed mostly because of style issues. The project uses Black style formatter. To apply Black style, one have to simply run black -l 80 map_machinewith the Black installed ( |
When cropping larger OSM datasets sometimes paths with zero nodes are formed.
Such paths break path -> svg code.
Checking for len < 2 resolves this problem.