pedestrians -> intro_year=

[makie]

Bei den pedestrians (Fußgängern) gibt es derzeit kein intro_year= oder retire_year=.

Bei den Fußgängern gibt es aber auch Radfahrer und Motorradfahrer, das ist historisch vor 1900 nicht korrekt.
Bzw. vor 1885 bei den "Sicherheits" Fahrrädern.

Man kann das bisher nicht auf Jahre eingrenzen. intro_year= oder retire_year= wären hier nützlich.

Auch die Mode der Fußgänger ändert sich. Wäre also nützlich.

[prissi]

Eigentlich hat jedes objet ein solches Datum, d.h. makeobj sollte es auch schon akzeptieren. Einbau laesst sich das fix.

[makie]

nein makeobj weist ab
ich hab das schon getestet bevor ich hier geschrieben habe

[prissi]

Ok, eingebaut in r8826

[makie]

Danke,
werde ich bei Gelegenheit testen

[makie]

Verzeihung das ich erst jetzt zum testen gekommen bin.
makeobj lässt sich nicht bei mir nicht kompilieren.


===> CXX ../descriptor/writer/pedestrian_writer.cc
../descriptor/writer/pedestrian_writer.cc: In member function ‘virtual void pedestrian_writer_t::write_obj(FILE*, obj_node_t&, tabfileobj_t&)’:
../descriptor/writer/pedestrian_writer.cc:76:24: error: ‘intro_date’ was not declared in this scope
  76 |  node.write_uint16(fp, intro_date,          8);
     |                        ^~~~~~~~~~
../descriptor/writer/pedestrian_writer.cc:77:24: error: ‘retire_date’ was not declared in this scope
  77 |  node.write_uint16(fp, retire_date,        10);
     |                        ^~~~~~~~~~~
make: *** [../uncommon.mk:47: ../build/default/descriptor/writer/pedestrian_writer-makeobj.o] Fehler 1

[prissi]

Sollte jetzt gehen. Habe ich wohl vergessen zu submitten, als das SVN crashte.

[makie]

makeobj weist immer noch ab

Makeobj version 60.2.R8828 for Simutrans 120.4.2 Nightly Makie R8828 and higher
Warning: BICYCLE_RIDER_P0:    Entry "intro_year=1885" ignored (check spelling)
Warning: BICYCLE_RIDER_P0:    Entry "intro_month=1" ignored (check spelling)
Warning: BICYCLE_RIDER_P1:    Entry "intro_year=1885" ignored (check spelling)
Warning: BICYCLE_RIDER_P1:    Entry "intro_month=1" ignored (check spelling)

da fehlt wohl noch was in pedestrian_writer.cc

---

wenn man

    uint16 const intro_date =
        obj.get_int("intro_year", DEFAULT_INTRO_DATE) * 12 +
        obj.get_int("intro_month", 1) - 1;

    uint16 const retire_date =
        obj.get_int("retire_year", DEFAULT_RETIRE_DATE) * 12 +
        obj.get_int("retire_month", 1) - 1;

einbaut dann geht es. Das habe ich bei Citycar_writer geklaut.

Hat einen netten Effekt.

Da sind nur Radfahrer zu sehen.
Wir haben das Jahr 1899.
Die Fahrräder haben intro_year=1885. Das Sicherheitsfahrrad wie es damals hieß wurde 1885 erfunden. Davor gab es Hochräder.
Fußgänger haben derzeit kein intro_year= da zieht also der Default von 1900.
Deshalb sind da keine Fußgänger.

[prissi]

Danke, das habe ich echt vermasselt. In 8834 behoben.

[makie]

Änderung pedestrian_writer.cc revision 8841 ist nicht gut.

    uint16 const intro_date = 1;
    uint16 const retire_date = 0xFFFEu;

damit liest er die Parameter nicht mehr aus der .dat

ich würde da eher das empfehlen

    uint16 const intro_date =
        obj.get_int("intro_year", 1) * 12 +        // no DEFAULT_INTRO_DATE here
        obj.get_int("intro_month", 1) - 1;

    uint16 const retire_date =
        obj.get_int("retire_year", DEFAULT_RETIRE_DATE) * 12 +
        obj.get_int("retire_month", 1) - 1;