Home » railML newsgroups » railml.timetable » <trackRef>.dir
Re: <trackRef>.dir [message #785 is a reply to message #783] Tue, 29 May 2012 14:07 Go to previous messageGo to previous message
Joachim Rubröder railML is currently offline  Joachim Rubröder railML
Messages: 0
Registered: November 2019
Hello Dirk,
I agree that the values are misleading and should be renamed, but the Trac
ticket #145 is due to version 3.0 because renaming will be a breaking
change.

I would suggest to use "tDelimitedDirection" (up, down, unknown) instead
of "tLaxDirection" (up, down, unknown, both, none) in 2.2 because "both"
and "none" make no sense. Instead of using "unknown", the optional
attribute "dir" should be skipped.

Kind regards,
Joachim

---

Ich bin völlig einverstanden, aber Umbenennungen wie im Ticket #145
vorgesehen sind als Breaking Changes erst mit Version 3.0 zulässig.

Als kleine Verbesserung bietet sich an den Typ "tDelimitedDirection" (up,
down, unknown) statt "tLaxDirection" (up, down, unknown, both, none) in
Version 2.2 zu verwenden, da die Ausprägungen "both" und "none" hier
keinen Sinn machen. Statt "unknown" zu verwenden sollte man das optionale
Attribut weglassen, was in der Dokumentation vermerkt wird.


Dirk Bräuer wrote:
>
> Dear Joachim,
>
> I have one small issue for RailML 2.2:
>
> We should clearify and ease the attribute
> <trainPart>.<ocpTT>.<sectionTT>.<trackRef>.dir.
>
> So far, it has the enumeration values up, down, unknown, none, and both.
> The only two which make sense are up & down but both should be renamed to
> 'falling' and 'raising' as in infrastructure (see also news message from
> 26.03.2012 and Trac ticket #145).
>
> The others are not applicable and should be deleted. (Instead of using
> 'unknown' one should skip the optional attribute.)
>
> Thank you and best regards,
> Dirk.
>
> ---
> Das Attribut <trainPart>.<ocpTT>.<sectionTT>.<trackRef>.dir ist mit den
> Ausprägungen up, down, unknown, none und both definiert. Die einzig
> sinnvollen sind up und down, wobei diese in raising und falling
> umdefiniert werden sollten. Die übrigen sind nicht zutreffend (statt
> unknown kann das optionale Attribut weggelassen werden) und sollten daher
> entfallen.
>
>



--
----== posted via PHP Headliner ==----
 
Read Message
Read Message
Read Message
Read Message
Read Message
Previous Topic: RFE for stop description
Next Topic: missing bitMask at <trainPart><operatingPeriodRef>
Goto Forum:
  


Current Time: Thu May 16 17:57:42 CEST 2024