Modify

Opened 15 years ago

Closed 15 years ago

Last modified 15 years ago

#4324 closed defect (duplicate)

New ways not displayed

Reported by: stoecker Owned by: jttt
Priority: blocker Milestone:
Component: Core Version: latest
Keywords: way mappaint Cc: malenki

Description

Under rare conditions new ways are not displayed in JOSM. Zooming a bit in and out usually shows them again. I was not yet able to make it reproducable. Probably an issue with wrong handling in QuadBuckets.

Attachments (6)

disappearing_ways.osm (17.2 KB ) - added by Ldp 15 years ago.
Example of this happening for me
gkrellShoot_2010-01-24_134508_new_way_complete.png (191.8 KB ) - added by malenki 15 years ago.
new_way_complete
gkrellShoot_2010-01-24_135926_new_way_complete.png (191.0 KB ) - added by malenki 15 years ago.
new_way_complete
gkrellShoot_2010-01-24_135936_way_splitted_disappeared.png (187.9 KB ) - added by malenki 15 years ago.
way_splitted_disappeared
gkrellShoot_2010-01-24_135950_new_way_still_exists.png (206.3 KB ) - added by malenki 15 years ago.
way_still_exists
nil_broken.osm (593.4 KB ) - added by malenki 15 years ago.

Download all attachments as: .zip

Change History (32)

comment:1 by Gubaer, 15 years ago

This happens very often while mapping. I'll collect sample bounding boxes with sample objects to reproduce and I'll post them here.

comment:2 by vsandre, 15 years ago

Keywords: way mappaint added
Priority: normalcritical
Version: latest

I have downloaded a small area and than all members of a containing route relation. And now the problem appears, no ways were shown. Afterwards I saved and deleted the data layer, with the aim to reproduce the bug, but all works fine. Also the saved file looks fine.

This bug is very annoying and could leads into wrong edits.

comment:3 by vsandre, 15 years ago

BTW selecting the route relation shows all ways.

by Ldp, 15 years ago

Attachment: disappearing_ways.osm added

Example of this happening for me

comment:4 by Ldp, 15 years ago

Load my example file, and drag one of the nodes. The way disappears, while the nodes are still painted. Zoom out pretty far to get the way painted again.

comment:5 by Ldp, 15 years ago

I went back to r2000 and worked my way back in steps, and I found my use case (see disappearing_ways.osm) was introduced in r2450 ('Added parameter Bounds to MapView, draw only currently visible primitives in MapPaintVisititor').

I had personally classified this (disappearing ways) as a Heisenbug, because I could never reproduce it before.

comment:6 by malenki, 15 years ago

Cc: malenki added

This bug could be related to #4111. Trying to reproduce it with r2864 did not result in a disappearing way but in a way who couldn't be selected anymore. Nodes can be selected, dragged around and used to continue the way. The way itself remains unselectable.

comment:7 by jttt, 15 years ago

Owner: changed from team to jttt

I can confirm that with disappearing_ways.osm. I'm working on a fix.

comment:8 by jttt, 15 years ago

Resolution: fixed
Status: newclosed

(In [2868]) Fixed #4324 New ways not displayed

comment:9 by malenki, 15 years ago

Priority: criticalmajor

In r2887 I experience a not straightforward reproducible error in wireframe view.

It appeared as I worked on Nile with Lakewalker. Sometimes newly created ways weren't displayed at all (only the nodes), sometimes parts of the way vanished after splitting. See attachments

by malenki, 15 years ago

new_way_complete

by malenki, 15 years ago

new_way_complete

by malenki, 15 years ago

way_splitted_disappeared

by malenki, 15 years ago

way_still_exists

comment:10 by malenki, 15 years ago

Resolution: fixed
Status: closedreopened

PS: I forgot to overwrite "gkrellShoot_2010-01-24_134508_new_way_complete.png" - can be deleted (or just disregard it)

comment:11 by malenki, 15 years ago

Priority: majorblocker

Changed to blocker since though the ways are diplayed in mappaint they don't exist. Validator sais there are thousands of empty nodes. Ways which weren't changed but happened to be displayed also disappeared leaving only the nodes behind. See attachment. Some hours of work wasted :€

by malenki, 15 years ago

Attachment: nil_broken.osm added

comment:12 by malenki, 15 years ago

For the attachment nil_broken.osm: If it can be repaired easily it would nice if it would be uploaded (without the deletions at HammerHeads ways and nodes). Would hate if I really had wasted so much time...

comment:13 by jttt, 15 years ago

What Dataset consistency test says? (right click on osm layer)

comment:14 by malenki, 15 years ago

Hrm. This i forgot to perform as I reported the bug. The Joms instance where this happened is long since closed. To the attached file the consistency test sais:

{Way id=5229400 version=6 V nodes=[]} not found using Dataset.searchWays()
Way {Way id=5229400 version=6 V nodes=[]} has zero nodes

comment:15 by jttt, 15 years ago

Hm, next time this kind of bug happens make also heap dump (see #4057 for details).

comment:16 by jttt, 15 years ago

Do you remember when the problems started? From the beginning, after data updated or some other event?

Also do I understand it right that the ways were displayed in mappaint mode, but not in wireframe mode?

comment:17 by jttt, 15 years ago

It looks like #4057 strikes again. And again lakewalker was used.

in reply to:  17 ; comment:18 by anonymous, 15 years ago

Replying to jttt:

It looks like #4057 strikes again. And again lakewalker was used.

If you think this is true, it's not true in every case. I have not used lakewalker, and don't even have it loaded, and yet (as can be seen in disappearing_ways.osm) I also have intermittent issues. Split in 2 tickets?

in reply to:  18 ; comment:19 by Ldp, 15 years ago

Replying to anonymous:

Replying to jttt:

It looks like #4057 strikes again. And again lakewalker was used.

If you think this is true, it's not true in every case. I have not used lakewalker, and don't even have it loaded, and yet (as can be seen in disappearing_ways.osm) I also have intermittent issues. Split in 2 tickets?

Anonymous replies sometimes considered harmful. This was me.

comment:20 by jttt, 15 years ago

The original issue is fixed in r2868 (the one with disappearing_ways.osm), the problem reported by malenki is different (and yes, it should have been in separated ticket).

in reply to:  16 comment:21 by malenki, 15 years ago

Replying to jttt:

Do you remember when the problems started? From the beginning,

no, just some time after I started editing

Also do I understand it right that the ways were displayed in mappaint mode, but not in wireframe mode?

yes

in reply to:  19 comment:22 by malenki, 15 years ago

Replying to Ldp:

Replying to anonymous:

Replying to jttt:

It looks like #4057 strikes again. And again lakewalker was used.

yes (though I didn't mention it explicitly)

If you think this is true, it's not true in every case. I have not used lakewalker, and don't even have it loaded, and yet (as can be seen in disappearing_ways.osm)

The attachment contains broken data so it is not relevant imo which plugins you at your josm are active.

in reply to:  20 comment:23 by malenki, 15 years ago

Replying to jttt:

The original issue is fixed in r2868 (the one with disappearing_ways.osm) [was created, you mean?]

the problem reported by malenki is different

How should I have known?

(and yes, it should have been in separated ticket).

Next time one has to tell me before reopening/adding a a ticket. One time I was "scolded" not having reopened a ticket, now I am having done so. Sums up to zero :)

comment:24 by stoecker, 15 years ago

Only use existing tickets when you are sure the issue is the same. Use a new one otherwise. You get blamed correctly when you open a new ticket and the same issue has been reported shortly before in another one. Ignore any blaming when the previous ticket is older or could not be found with an easy search.

It is not the task of a user to maintain the tickets or guess whether they really represent equal topics, but he should also not increase developers load by obvious duplicates.

comment:25 by jttt, 15 years ago

Resolution: duplicate
Status: reopenedclosed

I might have fixed the bug, see #4057 for details.

comment:26 by anonymous, 15 years ago

Closed as duplicate of #4057.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain jttt.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.