News:

The Forum Rules and Guidelines
Our forum has Rules and Guidelines. Please, be kind and read them ;).

[Bug] livery scheme selector list ignores intro month

Started by Ranran(retired), October 26, 2019, 05:27:12 PM

Previous topic - Next topic

0 Members and 1 Guest are viewing this topic.

Ranran(retired)

     (´・ω・`)Hello.

┏━━━━━━━━━━━━━━━━━━━━┓
┃Ranran appears! らんらんがあらわれた! .┃
┃         ▼          ┃
┗━━━━━━━━━━━━━━━━━━━━┛

I found that livery scheme selector ignores the value of intro_month and creates a list by checking only intro_year.
Although there are options, you have to wait until the actual month of appearance, but the player can not understand anything because we can't check the actual month of livery scheme appearance in the game.
ひめしという日本人が開発者達の助言を無視して自分好みの機能をextendedに"強引に"実装し、
コードをぐちゃぐちゃにしてメンテナンスを困難にし(とりわけ道路と建物関連)、
挙句にバグを大量に埋め込み、それを知らんぷりして放置し(隠居するなどと言って)別のところに逃げ隠れて自分のフォーク(OTRP)は開発を続けている
その事実と彼の無責任さに日本人プレイヤーは目を向けるべき。らんらんはそれでやる気をなくした(´・ω・`)
他人の振り見て我が振り直せ。ひめしのようにならないために、らんらんが生み出したバグや問題は自分で修正しなくちゃね(´・ω・`)

freddyhayward

The other issue is that there is no indication as to which livery schemes are applicable to a convoy, leading to endless scrolling through the entire list. The system is virtually inaccessible to new players, and frustrating for experienced players. If I am trying to paint a bus, why are hovertravel and ffestinog railway options?

Spenk009

May I suggest that while this receives attention, that use_timeline also acts on the selection disables the timeline of liveries and enables all? It would make pakset work easier and improve control for players wishing to build outside of timelines.

Vladki

1. the livery selector combo is obverlapping with list of stops and is thus not usable for direct choose, but only for scrolling
2. it should be limited to liveries that are used by vehicles using the line
3. when picking vehicles in depot, info about available liveries would be very nice.

When speaking about timeline, it would be nice to be able to turn off the timeline only for liveries. However in that case it would be necessary to show not only the livery scheme (like LMS/GWR/BR/Blackpool/Sheffield/...) but a specific livery (BR-Green, BR-Blue, ...). This would be extremely handy in multiplayer games, where players could use liveries to distinguish from other players.

Ranran(retired)

I have not yet found the cause of this bug. I suspect that livery scheme list has been updated in the processing of new_year or that the value of intro_month has not been gotten correctly.

Quote from: freddyhayward on October 26, 2019, 10:16:58 PMThe other issue is that there is no indication as to which livery schemes are applicable to a convoy, leading to endless scrolling through the entire list. The system is virtually inaccessible to new players, and frustrating for experienced players. If I am trying to paint a bus, why are hovertravel and ffestinog railway options?
This patch will reduce some annoyance.
I am currently planning to make a patch that like: eliminates the name of the airplane company from inside the bus depot.
ひめしという日本人が開発者達の助言を無視して自分好みの機能をextendedに"強引に"実装し、
コードをぐちゃぐちゃにしてメンテナンスを困難にし(とりわけ道路と建物関連)、
挙句にバグを大量に埋め込み、それを知らんぷりして放置し(隠居するなどと言って)別のところに逃げ隠れて自分のフォーク(OTRP)は開発を続けている
その事実と彼の無責任さに日本人プレイヤーは目を向けるべき。らんらんはそれでやる気をなくした(´・ω・`)
他人の振り見て我が振り直せ。ひめしのようにならないために、らんらんが生み出したバグや問題は自分で修正しなくちゃね(´・ω・`)

Ranran(retired)

Subsequent research has shown that the intro month of the "livery" is completely ignored. As a result, the intro month of the "livery scheme" is also ignored. In other words, all intro_month is 1 which is the default value.

For example, according to simuconf.tab and dat settings, if you apply British-Rail for LNER-A1 in 1950, the BR-Early livery will be applied like this:

and it should be green in November 1951, but in January 1951 is already green.

Don't jump the gun!  ::(
ひめしという日本人が開発者達の助言を無視して自分好みの機能をextendedに"強引に"実装し、
コードをぐちゃぐちゃにしてメンテナンスを困難にし(とりわけ道路と建物関連)、
挙句にバグを大量に埋め込み、それを知らんぷりして放置し(隠居するなどと言って)別のところに逃げ隠れて自分のフォーク(OTRP)は開発を続けている
その事実と彼の無責任さに日本人プレイヤーは目を向けるべき。らんらんはそれでやる気をなくした(´・ω・`)
他人の振り見て我が振り直せ。ひめしのようにならないために、らんらんが生み出したバグや問題は自分で修正しなくちゃね(´・ω・`)

Ranran(retired)

#6
I think I have identified and fixed the issue where the intro_month for liveries was not registered correctly. This was a simple mistake.
https://github.com/Ranran-the-JuicyPork/simutrans-extended/commit/31b1a57167044f887dee08e3542978228dc3cf97

Please confirm it.
ひめしという日本人が開発者達の助言を無視して自分好みの機能をextendedに"強引に"実装し、
コードをぐちゃぐちゃにしてメンテナンスを困難にし(とりわけ道路と建物関連)、
挙句にバグを大量に埋め込み、それを知らんぷりして放置し(隠居するなどと言って)別のところに逃げ隠れて自分のフォーク(OTRP)は開発を続けている
その事実と彼の無責任さに日本人プレイヤーは目を向けるべき。らんらんはそれでやる気をなくした(´・ω・`)
他人の振り見て我が振り直せ。ひめしのようにならないために、らんらんが生み出したバグや問題は自分で修正しなくちゃね(´・ω・`)

jamespetts

My apologies: I have realised that I omitted to respond to this thread. I believe that this is now working. Thank you for your fix.
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.