Skip to content
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

Creating task areas with no features should not be possible #37

Closed
spwoodcock opened this issue Jul 2, 2024 · 5 comments · Fixed by #41
Closed

Creating task areas with no features should not be possible #37

spwoodcock opened this issue Jul 2, 2024 · 5 comments · Fixed by #41
Assignees
Labels
bug Something isn't working

Comments

@spwoodcock
Copy link
Member

spwoodcock commented Jul 2, 2024

Describe the bug

  • This is specific to the split by square algorithm.
  • It's possible that sometimes a task area is generated with no features inside it.
  • This happens when an AOI is not square.

To Reproduce

Project https://fmtm.hotosm.org/project/100

Expected behavior

  • The task areas with no features are deleted at the end of the split_by_square function to avoid this problem.

Screenshots

You can see two small task areas that have no buildings:
image

Additional considerations

I imagine this problem does not exist on the complex splitting algorithm, as we handle the case already.

@Sujanadh
Copy link
Contributor

Sujanadh commented Jul 9, 2024

We do not pass data extracts to splitbysquare in order to check if the resulting polygons contain features or not.
Options:

  • Update the logic to check if the resulted tasks contains features or not in splitbysquare by passing data extracts.
  • get the splitted tasks in fmtm and update there if the task contains features or not; if some tasks donot have features, delete them or merge them.

@manjitapandey
Copy link

Is the problem here only specific to features not available because the task area is small or does it implemented on tasks with more area too where there is no features. Cases could be of forest areas?

@Sujanadh
Copy link
Contributor

Sujanadh commented Jul 9, 2024

I have made some improvements to this issue:

  • updated the splitbysquare to include data extracts
  • considering square polygons where atleast a data extract feature is present.

Output:
Before :
image

After:
Screenshot from 2024-07-09 13-47-48

AOI geojson:
durbar_poly.txt
data extracts:
durbar_exports.txt

@Sujanadh
Copy link
Contributor

Sujanadh commented Jul 9, 2024

I couldn't see the project you mentioned, I think its already deleted can you provide me aoi and the dimension you used so that i can try it, if this new improvements solves it.
@spwoodcock

@spwoodcock
Copy link
Member Author

Possibly this one?

https://fmtm.hotosm.org/project/104

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Development

Successfully merging a pull request may close this issue.

3 participants