VladPolskiy/osmosis
1
0
Fork 0
mirror of https://github.com/openstreetmap/osmosis.git synced 2026-01-14 00:36:36 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #134 from frodrigo/closed_partial_linestring

Browse Source 
On enableKeepPartialLinestring, output a closed linestring if the original way is closed even on missing start/end node #133
This commit is contained in:
Brett Henderson
2023-09-02 10:57:44 +10:00
committed by GitHub
parent 81b0027363 e32e9c2258
commit f2a3487efb
3 changed files with 21 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
doc/detailed-usage.adoc
View File

@ -1684,7 +1684,9 @@ the linestring geometry column. |yes, no |no
built from option enableLinestringBuilder. When an invalid or a missing
node location is encountered the linestring is not built by default.
Enabling this option keeps it as a partial linestring. It will result in
a different geometry than the original one. |yes, no |no
a different geometry than the original one. If the original geometry is
a loop the imported geometry is a loop, even in case of faulty start and
end. |yes, no |no
|nodeLocationStoreType |This option only takes effect if at least one of
the enableBboxBuilder and enableLinestringBuilder options are enabled.
@ -1902,7 +1904,8 @@ is more efficient for smaller datasets. |"InMemory", "TempFile",
built. When an invalid or a missing node location is encountered the
linestring is not built by default. Enabling this option keeps it as a
partial linestring. It will result in a different geometry than the
original one. |yes, no |no
original one. If the original geometry is a loop the imported geometry
is a loop, even in case of faulty start and end. |yes, no |no
|=======================================================================
==== --write-pgsimp-dump (--wsd)
@ -1939,7 +1942,9 @@ the linestring geometry column. |yes, no |no
built from option enableLinestringBuilder. When an invalid or a missing
node location is encountered the linestring is not built by default.
Enabling this option keeps it as a partial linestring. It will result in
a different geometry than the original one. |yes, no |no
a different geometry than the original one. If the original geometry is
a loop the imported geometry is a loop, even in case of faulty start and
end. |yes, no |no
|nodeLocationStoreType |This option only takes effect if at least one of
the enableBboxBuilder and enableLinestringBuilder options are enabled.
@ -1957,7 +1962,8 @@ the default database geometry building implementation, or the
built. When an invalid or a missing node location is encountered the
linestring is not built by default. Enabling this option keeps it as a
partial linestring. It will result in a different geometry than the
original one. |yes, no |no
original one. If the original geometry is a loop the imported geometry
is a loop, even in case of faulty start and end. |yes, no |no
|=======================================================================
==== --truncate-pgsimp (--ts)

5
osmosis-pgsimple/src/main/java/org/openstreetmap/osmosis/pgsimple/v0_6/impl/WayGeometryBuilder.java
View File

@ -216,6 +216,11 @@ public class WayGeometryBuilder implements Closeable {
}
}
if (enableKeepPartialLinestring && way.isClosed() && linePoints.size() > 1
&& !linePoints.get(0).equals(linePoints.get(linePoints.size() - 1))) {
linePoints.add(linePoints.get(0));
}
if (numValidNodes >= 2) {
return createLinestring(linePoints);
} else {

6
osmosis-pgsnapshot/src/main/java/org/openstreetmap/osmosis/pgsnapshot/v0_6/impl/WayGeometryBuilder.java
View File

@ -213,6 +213,12 @@ public class WayGeometryBuilder implements Closeable {
}
}
}
if (enableKeepPartialLinestring && way.isClosed() && linePoints.size() > 1
&& !linePoints.get(0).equals(linePoints.get(linePoints.size() - 1))) {
linePoints.add(linePoints.get(0));
}
return createLinestring(linePoints);
}