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

rerun parenx simplification #130

Open
martinfleis opened this issue Aug 1, 2024 · 6 comments
Open

rerun parenx simplification #130

martinfleis opened this issue Aug 1, 2024 · 6 comments
Assignees

Comments

@martinfleis
Copy link
Contributor

We need to rerun parenx simplification following #127

@jGaboardi
Copy link
Collaborator

#131 (comment)

@anastassiavybornova
Copy link
Collaborator

WIP on the https://github.com/uscuni/simplification/tree/parenx-rerun branch (haven't finished it yet :-/ there seems to be a shapely bug to uncover. will try to look at it this week)

@anastassiavybornova
Copy link
Collaborator

...and confirm that correct input data is being used, to prevent issues like this one

image

@anastassiavybornova
Copy link
Collaborator

@martinfleis just to be sure, should the input for parenx be from data/{fua}/original or from data/{fua}/no_degree_2?

@martinfleis
Copy link
Contributor Author

no_degree_2 is using stuff implemented in sgeop so I'd say original. How did the issue above happened?

@anastassiavybornova
Copy link
Collaborator

How did the issue above happened?

not sure, i will rerun evaluation and keep in mind to check whether it persists

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants