News:

Simutrans Forum Archive
A complete record of the old Simutrans Forum.

Tram lengths

Started by Mariculous, February 07, 2020, 04:41:40 PM

Previous topic - Next topic

0 Members and 1 Guest are viewing this topic.

Mariculous

Most UK trams have a length of slightly (1-3m) more than the graphics scale of 30m.
Due to the tile nature of simutrans these will require two full tiles.
I have shortened the last car of any tram up to 33m to fit into a single tile.
Otherwise there are only exactly two trams in the whole pakset that fit into a single tile. Now any, except for CAF edingburgh(42.8m), siemens-duewag supertram(34.8m) and sheffield Vossloh Tram-Train(37.2m) should fit into a single tile.

However, for some reason they still display as 2 tiles in the depot although their summed-up length is exactly 16=1tile and observating track reservations, they can fit into a single tile. When assigning a single tile station, they will however complain.

The changes can be found there: https://github.com/irgend42/simutrans-pak128.britain/commits/tramLength

Ranran(retired)

QuoteHowever, for some reason they still display as 2 tiles in the depot although their summed-up length is exactly 16=1tile and observating track reservations, they can fit into a single tile.
The tiles display is always correct but the length and tiles may not match.
5+5+5 is not 15 but 18. 16 + 0 is not 16 but 24.
So this fix is meaningless.
See below for details.
https://forum.simutrans.com/index.php/topic,18496.0.html
https://forum.simutrans.com/index.php/topic,3376.0.html
ひめしという日本人が開発者達の助言を無視して自分好みの機能をextendedに"強引に"実装し、
コードをぐちゃぐちゃにしてメンテナンスを困難にし(とりわけ道路と建物関連)、
挙句にバグを大量に埋め込み、それを知らんぷりして放置し(隠居するなどと言って)別のところに逃げ隠れて自分のフォーク(OTRP)は開発を続けている
その事実と彼の無責任さに日本人プレイヤーは目を向けるべき。らんらんはそれでやる気をなくした(´・ω・`)
他人の振り見て我が振り直せ。ひめしのようにならないために、らんらんが生み出したバグや問題は自分で修正しなくちゃね(´・ω・`)

Mariculous

#2
I have noticed and pointed out that it is currently useless.

That far from intuitive length calculation needs to be fixed and once it's fixed it will work.

jamespetts

Quote from: Freahk on February 11, 2020, 12:11:47 PM
I have noticed and pointed out that it is currently useless.

That far from intuitive length calculation needs to be fixed and once it's fixed it will work.

The length calculation system is unchanged from Standard so far as I am aware.
Download Simutrans-Extended.

Want to help with development? See here for things to do for coding, and here for information on how to make graphics/objects.

Follow Simutrans-Extended on Facebook.

Vladki

Bombardier flexity swift can be fixed by setting length=8 (or just commenting the length)
It will look a bit funny, but will fit on one tile.

Mariculous

Maybe also modifying graphic offsets of the first or second car might look better.
Didn't play around with it yet.

Vladki

How about this?  - offset applied only to rear car in red livery


diff --git a/trams/bombardier-flexity-swift.dat b/trams/bombardier-flexity-swift.dat
index 6611a4084..35d39f0a8 100644
--- a/trams/bombardier-flexity-swift.dat
+++ b/trams/bombardier-flexity-swift.dat
@@ -13,7 +13,7 @@ intro_month=3
waytype=tram_track
freight=Passagiere
payload=35
-length=9
+#length=9
# http://en.wikipedia.org/wiki/CR4000
overcrowded_capacity=69
weight=18.2
@@ -83,7 +83,7 @@ intro_month=3
waytype=tram_track
freight=Passagiere
payload=35
-length=7
+#length=9
# http://en.wikipedia.org/wiki/CR4000
overcrowded_capacity=69
weight=18.2
@@ -109,14 +109,14 @@ liverytype[0]=Tramlink-red
liverytype[1]=Tramlink-green
liverytype[2]=Metrolink-yellow

-EmptyImage[E][0]=./images/flexity-swift-croydon-red-rear.1.4
-EmptyImage[SE][0]=./images/flexity-swift-croydon-red-rear.1.5
-EmptyImage[S][0]=./images/flexity-swift-croydon-red-rear.1.6
-EmptyImage[SW][0]=./images/flexity-swift-croydon-red-rear.1.7
-EmptyImage[W][0]=./images/flexity-swift-croydon-red-rear.1.0
-EmptyImage[NW][0]=./images/flexity-swift-croydon-red-rear.1.1
-EmptyImage[N][0]=./images/flexity-swift-croydon-red-rear.1.2
-EmptyImage[NE][0]=./images/flexity-swift-croydon-red-rear.1.3
+EmptyImage[E][0]=./images/flexity-swift-croydon-red-rear.1.4,-4,-2
+EmptyImage[SE][0]=./images/flexity-swift-croydon-red-rear.1.5,0,-2
+EmptyImage[S][0]=./images/flexity-swift-croydon-red-rear.1.6,4,-2
+EmptyImage[SW][0]=./images/flexity-swift-croydon-red-rear.1.7,4,0
+EmptyImage[W][0]=./images/flexity-swift-croydon-red-rear.1.0,4,2
+EmptyImage[NW][0]=./images/flexity-swift-croydon-red-rear.1.1,0,2
+EmptyImage[N][0]=./images/flexity-swift-croydon-red-rear.1.2,-4,2
+EmptyImage[NE][0]=./images/flexity-swift-croydon-red-rear.1.3,-4,0

EmptyImage[E][1]=./images/flexity-swift-croydon-green-rear.1.4
EmptyImage[SE][1]=./images/flexity-swift-croydon-green-rear.1.5

Mariculous

Yes, something like that.
Imho it's fine that way.


jamespetts

Now incorporated - thank you.
Download Simutrans-Extended.

Want to help with development? See here for things to do for coding, and here for information on how to make graphics/objects.

Follow Simutrans-Extended on Facebook.