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

Item 7170 (classes 1;3;20 at least) frozen since 22/06/2024 ? #2363

Open
NicolasPyIGN opened this issue Oct 16, 2024 · 9 comments
Open

Item 7170 (classes 1;3;20 at least) frozen since 22/06/2024 ? #2363

NicolasPyIGN opened this issue Oct 16, 2024 · 9 comments

Comments

@NicolasPyIGN
Copy link

Hi,
We consume the false positive of item 7170 classes 1;3;20 and we discovered the update timestamp (last run of analyzer we presume) is frozen at 22 june 2024 22:29.
Do you confirm that ? if yes, is it nominal or something is broken backend side ?

@NicolasPyIGN NicolasPyIGN changed the title Item 7170 (classes 1;3;20 at least) frozen since 22/06/2024 Item 7170 (classes 1;3;20 at least) frozen since 22/06/2024 ? Oct 16, 2024
@frodrigo
Copy link
Member

Indeed, this classes are not updated for a while. The update come from an external tool by @cquest .

You also maybe interested y the new class 13

https://osmose.openstreetmap.fr/fr/issues/open?item=7170

@NicolasPyIGN
Copy link
Author

In
https://wiki.openstreetmap.org/wiki/FR:Osmose/issues, Classe 13 "voirie cadastre sans route à 20 m"
also
../analysers/analyser_merge_road_FR.py:32 link to IGNF:BDTopo, not french cadaster institute

If I understand clearly, item 7170 class 13 do the comparison between OSM and official road network. If so, a false positive on that class indicate a potential error in the official road network. Is it correct ?

@frodrigo
Copy link
Member

@NicolasPyIGN
Copy link
Author

We looked at https://osmose.openstreetmap.fr/fr/issues/open?item=7170 class 13.
Until now, we used country=france and source=14708 ; we understood that countries were independant, and non hierarchical.

Looking at https://osmose.openstreetmap.fr/fr/issues/open?item=7170&source=&class=13&username=&bbox=, we discovered there's now many country and source.

What are we missing ? How works that new item7170+class13 ?

@frodrigo
Copy link
Member

item7170+class13 is a standard case. Using only one source for the whole country a particular case.

What are we missing ?

What are you expected ?

@NicolasPyIGN
Copy link
Author

Until now we looked at:
_false-positive
and
__item7170 source=14708 class 1 country=france*
__item7170 source=14708 class 3 country=france*
__item7170 source=14708 class 4 country=france*
__item7170 source=14708 class 20 country=france*

We expected having to add
false-positive
item7170
class13
country=france*
attached to only one unique source

Looking at
https://osmose.openstreetmap.fr/api/0.3/issues.json?limit=10000&country=france*&full=true&status=false&start_date=2024-08-17&end_date=2024-10-17&useDevItem=false&class=13&item=7170
show at least 2 sources: 428617 and 428654.

Why many sources? are they perennial ?

@frodrigo
Copy link
Member

Better no use the source parameter.

1 source = (1 analyser, 1 area)

@NicolasPyIGN
Copy link
Author

so only
_false positive
__item7170 class 1 country=france*
__item7170 class 3 country=france*
__item7170 class 4 country=france*
__item7170 class 20 country=france*
__item7170 class 13 country=france*
does the job ?

@frodrigo
Copy link
Member

does the job ?

Yes.

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

No branches or pull requests

2 participants