Modify

Opened 7 years ago

Last modified 14 months ago

#7468 new defect

Do not delete address nodes but use them as corner node

Reported by: skyper Owned by: Upliner
Priority: normal Milestone:
Component: Plugin buildings_tools Version:
Keywords: address node, building_tools_replacement_mode Cc: jhfi, brogo

Description

I already reported this on #7328.

Polyglot:

Replying to skyper:

By the way: BT should not delete address nodes but use the node as one corner node with its properties moved to the way.

I agree with you there. As I said, there is room for improvement.

I mark this as defect as you loose the history of the address tags right now.

Attachments (0)

Change History (9)

comment:1 Changed 7 years ago by joshdoe

The code for this is implemented in Utilsplugin2, in ReplaceGeometryUtils.upgradeNode(Node subject, OsmPrimitive reference). Perhaps it should depend on utilsplugin2, or if building_tools is to be integrated in core, we can consider moving this class from the plugin to core as well.

comment:2 Changed 7 years ago by skyper

Ticket #6626 has been marked as a duplicate of this ticket.

comment:3 Changed 4 years ago by stephan75

Is this an alternative?

Have utils_plugin2 and building_tools plugin installed,

have a single node from OSM database with any addr:xxx data

then draw a building outline via building_tool around this "old" addr: node, but be sure via buildingtools setting that this node is NOT deleted,

Then select that node and also select the new building outline (that consists of four new nodes,

choose via "moore tools" menu the feature of "Replace Geometry" and transfer all tags from the node to the outline,

after that click each of the four nodes of that outline, and see
that one of them is now the OLD one node with the node ID number from OSM database.

Thus we can see that transferring that old node to the building outline is possible!!!!

So we need a transfer from the feature "Replace Geometry" to building_tool, I think.

Is this possible with little effort?

Stephan75

comment:4 Changed 4 years ago by mdk

First of all there are three possible candidates for the old history:

  1. The building outline. But as far as I know we have no operations to “merge“ the history of two objects or replace the history of the way with the one of a node.
  2. Add existing address node to the building outline (as an entrance). But where to add the node (automatically)?
  3. Replace an existing outline node with old address node, as suggested in this issue.

When I start thinking about the third solution, I found much more arguments against this approach than supporting ones:

  1. We have to choose an arbitrary victim node to be replaced. This node will loose his history instead. What is the criteria for choosing the node to be deleted?
  2. Assuming we have a rectangular building, I would never expect the address history at a corner node. Not finding the history is in my opinion the same as loosing the history.
  3. Assuming some simple terraced buildings. The inner buildings has only nodes which are also part of other buildings. To which building belongs the history, if the replaced node is part of two three or even four buildings?
  4. If the replaced node is member of more than one way (other buildings, fences, parking places, …), the node must be replaced in all ways. This cause (unnecessary) modifications on all the involved ways.
  5. Again think about terraced buildings. How do we want to prevent replacing a node which belongs to more than one building more than once? In this case we will loose an old address history anyway.

Edit: This is more suitable for #6285. For this issue (creating a new building), 1.) is not relevent.

Last edited 4 years ago by mdk (previous) (diff)

comment:5 Changed 3 years ago by openstreetmap.org-user-d1g

Keywords: building_tools_replacement_mode added

comment:6 Changed 14 months ago by Klumbumbus

Ticket #15858 has been marked as a duplicate of this ticket.

comment:7 Changed 14 months ago by Klumbumbus

Cc: jhfi added

comment:8 Changed 14 months ago by Klumbumbus

Ticket #15473 has been marked as a duplicate of this ticket.

comment:9 Changed 14 months ago by Klumbumbus

Cc: brogo added

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The owner will remain Upliner.
as The resolution will be set.
to The owner will be changed from Upliner to the specified user.
The owner will change to skyper
as duplicate The resolution will be set to duplicate.The specified ticket will be cross-referenced with this ticket
The owner will be changed from Upliner to anonymous.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.