Skip to content

Conversation

@awirb
Copy link
Contributor

@awirb awirb commented Nov 19, 2024

if features have no points but are still available in shp file - remove these features log warning that these area ignored
fixes #1044

@awirb awirb added the featureRequest ideas or functions that would be useful label Nov 19, 2024
@awirb awirb requested a review from fso42 November 19, 2024 08:15
@fso42 fso42 changed the title Fix no points Fix no points in shp reading routines [in2] Nov 19, 2024
@fso42 fso42 added the confirmed Something isn't working label Nov 19, 2024
@fso42 fso42 added this to the Version 1.10 milestone Nov 19, 2024
Copy link
Contributor

@fso42 fso42 left a comment

Choose a reason for hiding this comment

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

The test shapefile is missing

@pep8speaks
Copy link

pep8speaks commented Nov 20, 2024

Hello @awirb! Thanks for updating this PR.

Line 420:52: E203 whitespace before ':'
Line 418:48: E203 whitespace before ':'
Line 417:48: E203 whitespace before ':'

Comment last updated at 2024-11-20 20:30:43 UTC

@awirb awirb requested a review from fso42 November 20, 2024 10:11
@qlty-cloud-legacy
Copy link

Code Climate has analyzed commit 3a440d5 and detected 1 issue on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 1

The test coverage on the diff in this pull request is 97.8% (50% is the threshold).

This pull request will bring the total coverage in the repository to 69.6% (0.0% change).

View more on Code Climate.

@fso42
Copy link
Contributor

fso42 commented Nov 20, 2024

Standardtests ok, apart from known kot pfa

@fso42 fso42 merged commit 2698dce into master Nov 20, 2024
1 check passed
@fso42 fso42 deleted the fixNoPoints branch November 20, 2024 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

confirmed Something isn't working featureRequest ideas or functions that would be useful

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Error on empty features in shapefiles

4 participants