Re: [Talk-de] GPSbabel - kleine Veränderung für O SM

2008-08-15 Diskussionsfäden Tobias Wendorff
Hallo,

Sven Geggus schrieb:
> Welcher Name Tag denn?





Sowas in der Art wird an jedem Node erzeugt.

> Ich hab das bisher nur dazu verwendet um ein paar kleinere kml Dateien in
> osm zu konvertieren und da stand im name-tag eigentlich durchaus sinnvolles
> Zeug drin. Der Name des konvertierten Polygons zum Beispiel.

Wie bekommst Du GPSBabel dazu, ein Polygon zu konvertieren? Polygone
werden bei meiner Version ignoriert :-(

> Es dürfte aber trivial sein zusätzlich einen no_name flag einzubauen, damit
> gar keine name-tags geschrieben werden.

Schön, dann kannst Du es ja gleich wieder machen :-)

Grüße
Tobias

___
Talk-de mailing list
Talk-de@openstreetmap.org
http://lists.openstreetmap.org/listinfo/talk-de


Re: [Talk-de] GPSbabel - kleine Veränderung für O SM

2008-08-14 Diskussionsfäden Sven Geggus
Tobias Wendorff <[EMAIL PROTECTED]> wrote:

> Aber es bezieht sich _nur_ auf created_by und nicht auf diesen
> "name"-Tag, der immer zusätzlich geschrieben wird ?!?!

Welcher Name Tag denn?

Ich hab das bisher nur dazu verwendet um ein paar kleinere kml Dateien in
osm zu konvertieren und da stand im name-tag eigentlich durchaus sinnvolles
Zeug drin. Der Name des konvertierten Polygons zum Beispiel.

Es dürfte aber trivial sein zusätzlich einen no_name flag einzubauen, damit
gar keine name-tags geschrieben werden.

Gruss

Sven

-- 
"We just typed make"
(Stephen Lambrigh, Director of Server Product Marketing at Informix
  about porting their Database to Linux)
/me is [EMAIL PROTECTED], http://sven.gegg.us/ on the Web

___
Talk-de mailing list
Talk-de@openstreetmap.org
http://lists.openstreetmap.org/listinfo/talk-de


Re: [Talk-de] GPSbabel - kleine Veränderung für O SM

2008-08-14 Diskussionsfäden Tobias Wendorff
Hallo,

Sven Geggus schrieb:
>> Also ich weiß nicht, ob das so im Sinne des Erfinders ist ...
> 
> Warum? Ein zweiter Parameter erschien mir zu viel des Guten zu sein. Die
> Angabe von "-o osm,created_by=" sogt dafür, dass gar kein created_by tag
> mehr in die AUsgabedatei geschrieben wird.

Naja, vielleicht will der Programmierer ja, dass seine Software
die Tags erzeugt.

>> Vergiss bitte nicht, das automatische Tagging an den Nodes auch zu
>> deaktivieren. 
> 
> Hast Du den patch angeschaut?!
> 
> Der greift doch an zwei Stellen im code, bei ways _und_ bei nodes.

Siehe unten.

>> ps: Ich muss endlich C/C++ lernen - ASM, PHP und jetzt auch Perl bringen
>> einen heute nicht mehr weit :-(
> 
> OK, Du hast den patch offensichtlich nicht verstanden.

Nunja, sieht ein wenig wie PHP aus :-)

> gpsbabel -i INTYPE -f INFILE -o osm,created_by=foobar -O out.osm
> 
> für created_by=foobar
> 
> gpsbabel -i INTYPE -f INFILE -o osm,created_by= -O out.osm
> 
> für Dateien ohne created_by tag.

Aber es bezieht sich _nur_ auf created_by und nicht auf diesen
"name"-Tag, der immer zusätzlich geschrieben wird ?!?!

Grüße
Tobias

___
Talk-de mailing list
Talk-de@openstreetmap.org
http://lists.openstreetmap.org/listinfo/talk-de


Re: [Talk-de] GPSbabel - kleine Veränderung für O SM

2008-08-14 Diskussionsfäden Sven Geggus
Tobias Wendorff <[EMAIL PROTECTED]> wrote:

> Also ich weiß nicht, ob das so im Sinne des Erfinders ist ...

Warum? Ein zweiter Parameter erschien mir zu viel des Guten zu sein. Die
Angabe von "-o osm,created_by=" sogt dafür, dass gar kein created_by tag
mehr in die AUsgabedatei geschrieben wird.

> Vergiss bitte nicht, das automatische Tagging an den Nodes auch zu
> deaktivieren. 

Hast Du den patch angeschaut?!

Der greift doch an zwei Stellen im code, bei ways _und_ bei nodes.

> ps: Ich muss endlich C/C++ lernen - ASM, PHP und jetzt auch Perl bringen
> einen heute nicht mehr weit :-(

OK, Du hast den patch offensichtlich nicht verstanden.

gpsbabel -i INTYPE -f INFILE -o osm,created_by=foobar -O out.osm

für created_by=foobar

gpsbabel -i INTYPE -f INFILE -o osm,created_by= -O out.osm

für Dateien ohne created_by tag.

Sven

-- 
C is quirky, flawed, and an enormous success
(Dennis M. Ritchie)

/me is [EMAIL PROTECTED], http://sven.gegg.us/ on the Web

___
Talk-de mailing list
Talk-de@openstreetmap.org
http://lists.openstreetmap.org/listinfo/talk-de


Re: [Talk-de] GPSbabel - kleine Veränderung für O SM

2008-08-14 Diskussionsfäden Tobias Wendorff
Hallo,

Sven Geggus schrieb:
> Wenn man nachfolgenden patch einspielt kann mit Hilfe des Parameters
> created_by beliebige created_by tags setzen und mit Hilfe eines leeren
> created_by tags dafür sorgen, dass gar kein solcher Tag reingeschrieben
> wird.

Also ich weiß nicht, ob das so im Sinne des Erfinders ist ...

Vergiss bitte nicht, das automatische Tagging an den Nodes auch zu
deaktivieren. Momentan schreibt er an jeden Tag einen generierten
Namen und bläst die OSM-Datei massiv auf.

(Habe ich in meinem Patch auch so gemacht, kannst Du ja mal testen).

Grüße
Tobias

ps: Ich muss endlich C/C++ lernen - ASM, PHP und jetzt auch Perl bringen
einen heute nicht mehr weit :-(

___
Talk-de mailing list
Talk-de@openstreetmap.org
http://lists.openstreetmap.org/listinfo/talk-de


Re: [Talk-de] GPSbabel - kleine Veränderung für O SM

2008-08-14 Diskussionsfäden Sven Geggus
Tobias Wendorff <[EMAIL PROTECTED]> wrote:

> echt toll, dass Du sowas weißt. Dann erkläre mir doch bitte auch, wie
> ich jenes in C/C++ realisieren kann und in einen Patch aufbereiten kann.

Gerne doch :)

Wenn man nachfolgenden patch einspielt kann mit Hilfe des Parameters
created_by beliebige created_by tags setzen und mit Hilfe eines leeren
created_by tags dafür sorgen, dass gar kein solcher Tag reingeschrieben
wird.

Ich werde das mal upstream einreichen.

Gruss

Sven

--- osm.c.orig  2008-08-14 22:25:06.549645351 +0200
+++ osm.c   2008-08-14 22:40:05.337795184 +0200
@@ -24,12 +24,13 @@
 #include "avltree.h"
 #include "xmlgeneric.h"
 
-static char *opt_tag, *opt_tagnd;
+static char *opt_tag, *opt_tagnd, *created_by;
 
 static arglist_t osm_args[] = 
 {
{ "tag", &opt_tag,  "Write additional way tag key/value pairs", 
NULL, ARGTYPE_STRING, ARG_NOMINMAX },
{ "tagnd", &opt_tagnd,  "Write additional node tag key/value pairs", 
NULL, ARGTYPE_STRING, ARG_NOMINMAX },
+   { "created_by", &created_by, "Use this value as custom created_by 
value","GPSBabel", ARGTYPE_STRING, ARG_NOMINMAX },
ARG_TERMINATOR
 };
 
@@ -785,11 +786,14 @@
}
gbfprintf(fout, ">\n");
 
-   gbfprintf(fout, "\n");
-
+   if (strlen(created_by) !=0) {
+   gbfprintf(fout, "\n");
+   }
+   
osm_write_tag("name", wpt->shortname);
osm_write_tag("note", (wpt->notes) ? wpt->notes : 
wpt->description);
if (wpt->icon_descr)
@@ -837,10 +841,13 @@
 {
if (skip_rte) return;
 
-   gbfprintf(fout, "\n");
+   if (strlen(created_by) !=0) {
+   gbfprintf(fout, "\n");
+   }
 
osm_write_tag("name", rte->rte_name);
osm_write_tag("note", rte->rte_desc);


-- 
"Every time you use Google, you're using a Linux machine"
 (Chris DiBona, a programs manager for Google)

/me is [EMAIL PROTECTED], http://sven.gegg.us/ on the Web

___
Talk-de mailing list
Talk-de@openstreetmap.org
http://lists.openstreetmap.org/listinfo/talk-de


Re: [Talk-de] GPSbabel - kleine Veränderung für O SM

2008-08-14 Diskussionsfäden Tobias Wendorff
Hallo,

Sven Geggus schrieb:
> Der generische Weg wäre das einfügen einer passenden Option, mit der man das
> überschreiben kann. Das dürfte man dann auch als patch in den offiziellen tree
> reinkriegen.

echt toll, dass Du sowas weißt. Dann erkläre mir doch bitte auch, wie
ich jenes in C/C++ realisieren kann und in einen Patch aufbereiten kann.

Grüße
Tobias

___
Talk-de mailing list
Talk-de@openstreetmap.org
http://lists.openstreetmap.org/listinfo/talk-de


Re: [Talk-de] GPSbabel - kleine Veränderung für O SM

2008-08-14 Diskussionsfäden Sven Geggus
Tobias Wendorff <[EMAIL PROTECTED]> wrote:

> Da ich im Sourcecode keinen Switch zum Abschalten gefunden habe, habe
> ich das kurzer Hand entfernt ...

Der generische Weg wäre das einfügen einer passenden Option, mit der man das
überschreiben kann. Das dürfte man dann auch als patch in den offiziellen tree
reinkriegen.

Sven

-- 
Threading is a performance hack.
(The Art of Unix Programming by Eric S. Raymond)

/me is [EMAIL PROTECTED], http://sven.gegg.us/ on the Web

___
Talk-de mailing list
Talk-de@openstreetmap.org
http://lists.openstreetmap.org/listinfo/talk-de


Re: [Talk-de] GPSbabel - kleine Veränderung für O SM

2008-08-14 Diskussionsfäden Tobias Wendorff
Hallo,

Christoph Eckert schrieb:
>> Da ich im Sourcecode keinen Switch zum Abschalten gefunden habe, habe
>> ich das kurzer Hand entfernt ... die Dateien dürften 30% und mehr
>> kleiner werden und JOSM sollte besser damit klar kommen. Auch die
>> Datenbank wird nun geschont.
> 
> als Patch an Robert Lipe?

Ich denke nicht, dass er mit einem Patch für Windows etwas anfangen
könnte / will. Was ich machen kann ist, eine Mail zu schreiben.

Allerdings weiß ich nicht, ob die Funktion von ihm nicht so gewollt
ist. Schließlich ist es seine persönliche "Fußnote" :-)

Grüße
Tobias

___
Talk-de mailing list
Talk-de@openstreetmap.org
http://lists.openstreetmap.org/listinfo/talk-de


Re: [Talk-de] GPSbabel - kleine Veränderung für O SM

2008-08-14 Diskussionsfäden Christoph Eckert
Moin,

> Da ich im Sourcecode keinen Switch zum Abschalten gefunden habe, habe
> ich das kurzer Hand entfernt ... die Dateien dürften 30% und mehr
> kleiner werden und JOSM sollte besser damit klar kommen. Auch die
> Datenbank wird nun geschont.

als Patch an Robert Lipe?

Gruß,

ce


___
Talk-de mailing list
Talk-de@openstreetmap.org
http://lists.openstreetmap.org/listinfo/talk-de


[Talk-de] GPSbabel - kleine Veränderung für O SM

2008-08-14 Diskussionsfäden Tobias Wendorff
Hallo Leute,

mir ging es ziemlich auf den Sack, dass GPSbabel immer wieder created_by
an jeden Way und Node geschrieben hat. Auch hat mich diese automatische
Node-Bezeichnung genervt.

Da ich im Sourcecode keinen Switch zum Abschalten gefunden habe, habe
ich das kurzer Hand entfernt ... die Dateien dürften 30% und mehr
kleiner werden und JOSM sollte besser damit klar kommen. Auch die
Datenbank wird nun geschont.

http://raumplanung.tobwen.de/OSM/GPSbabel/gpsbabel_osm.zip

Feedback erwünscht - Sourcecode nicht vorhanden, da mir bislang niemand
erklären wollte, wie man GPSbabel unter Windows kompiliert :-(

Grüße
Tobias

___
Talk-de mailing list
Talk-de@openstreetmap.org
http://lists.openstreetmap.org/listinfo/talk-de