osmcha-frontend icon indicating copy to clipboard operation
osmcha-frontend copied to clipboard

"Map failed" when attempting to view specific changeset

Open ZLima12 opened this issue 3 years ago • 2 comments

I'm submitting a (bug report/feature request/ support or question)

Bug Report

Brief Description

When I attempt to view changeset 118466153, the map never loads, and I get an error saying:

Changeset:118466153 Map failed
Please reload the application. If it still doesnt work please refer to usage guide.

Reloading does nothing. I asked in the general channel of the OSMUS Slack, and 5 other people said they encountered the same issue.

What is the current behaviour, (attach relevant screenshots) ?

See description. Additionally, see this screenshot.

What is the expected behaviour ?

For the changeset to load normally.

When does this occur ?

Any time you try to load the changeset.

How do we replicate the issue ?

  1. Open the changeset

Please tell us about your environment:

Linux x86_64, Firefox 98.0.2

Other Information / context:

None.

ZLima12 avatar Mar 26 '22 04:03 ZLima12

https://github.com/osmlab/changeset-map/blob/master/lib/getChangeset.js#L93 - features[1] is undefined in this case.

mmd-osm avatar Apr 18 '22 09:04 mmd-osm

  1. real-changesets-parser returns undefined instead of a feature when a way has no nodes (element.js#L27-L30):
     function createWay(data) {
       if (data.nodes.length === 0) {
         return;
       }
    
    (the GeoJSON spec would allow coordinates to be empty (3.1) or the geometry property to be null (3.2))
  2. The old entry for way 199760894 has no details in the adiff query response:
    <action type="modify">
    <old>
      <way id="199760894" version="6" timestamp="2021-01-15T17:20:21Z" changeset="97574667" ... />
    </old>
    <new>
      <way id="199760894" version="6" timestamp="2021-01-15T17:20:21Z" changeset="97574667" ... >
        <bounds minlat="41.2845186" minlon="-73.7716790" maxlat="41.2846432" maxlon="-73.7715706"/>
        <nd ref="2097537190" lat="41.2845186" lon="-73.7716790"/>
        <nd ref="373645825" lat="41.2846432" lon="-73.7715706"/>
        <tag k="foot" v="yes"/>
        ...
    
  3. Propably because it's considered being outside of the bbox, as the old version of the moved end node 373645825 is right on the bbox:
    • https://overpass-api.de/achavi/?changeset=118466153
      achavi screenshot changeset 118466153
    • Same for #646:
      https://overpass-api.de/achavi/?changeset=132982621
      achavi screenshot changeset 132982621

What would be expected here is to also show the geometry of the old way segment. Which could maybe be reconstructed from the data we get.

But as this is yet another issue with querying changesets from Overpass, I would rather suggest to investigate in searching and building an alternative solution (#652).

nrenner avatar Mar 08 '24 13:03 nrenner

It sounds like a newer version of overpass might fix this. https://community.openstreetmap.org/t/achavi-and-osmcha-behaves-differently-when-reviewing-similar-changesets-made-by-different-editors-is-this-a-concern/121747/3

the OSMCha Overpass server is running version 0.7.57.1, the fix is in version 0.7.62.4.

elevenchars avatar Nov 21 '24 23:11 elevenchars