osm-fr / osmose-frontend

Part of osmose that shows results on the website http://osmose.openstreetmap.fr
GNU General Public License v3.0
41 stars 38 forks source link

History graph current value doesn't relate to analysis count #448

Closed flacombe closed 1 year ago

flacombe commented 1 year ago

Hello

One additional question I have (and don't find existing issue on that topic) : why sometimes history graph doesn't relate to analysis current count? It's not necessary a mistake but it needs explanations.

For instance in this area, I got today 43 warnings: https://osmose.openstreetmap.fr/fr/issues/open?source=410981&item=7040&class=4

But history graph ends at > 120 https://osmose.openstreetmap.fr/fr/issues/graph.png?source=410981&item=7040&class=4 image

Is there kind of aggregation in warnings that prevent to show up as high as the graph?

Famlam commented 1 year ago

I think someone was very active with fixing this issue in the past few days, because right now the graph looks ok for me? 25 issues in the list and a similar number in the graph. I think the graph is only updated once every day: image

PtiBzh commented 1 year ago

Hi! I recently fixed somes issues from another analysis[1] and the same thing happen where the graph is lagging behind the analysis warning count. It seem to me that the delay is like 36 hours or more. It's really confusing when you want to check the evolution.

The analysis graph currently show ~1100 warnings, the warning count show 236 and I think I finished to correct all issues this evening. It can be interisting to check how many days is needed for the graph to fall (almost) 0.

[1] https://osmose.openstreetmap.fr/en/issues/open?item=1250&source=&class=1&username=&bbox=&country=france%2a Screenshot_20230126-235213_1

flacombe commented 1 year ago

Thanks for feedbacks

That's probably cache or processing lag, nothing very worrying indeed.

Let maintainers tell us if they consider it as an issue or not.

frodrigo commented 1 year ago

All right.

jocelynj commented 1 year ago

Just to clarify: