Elvas Tower: ORTSDoesVacuumBrakeCutPower is no longer working - Elvas Tower

Jump to content

  • 5 Pages +
  • 1
  • 2
  • 3
  • 4
  • Last »
  • You cannot start a new topic
  • You cannot reply to this topic

ORTSDoesVacuumBrakeCutPower is no longer working Rate Topic: ***-- 1 Votes

#11 User is offline   darwins 

  • Superintendant
  • Group: Status: Elite Member
  • Posts: 1,251
  • Joined: 25-September 17
  • Gender:Male
  • Simulator:Open Rails
  • Country:

Posted 01 December 2021 - 06:17 AM

Sounds like a challenge. Carlo Santucci wrote the TCS script for UK AWS perhaps he can help update this. I only write eng and wag files - not yet learned how to write scripts. Where does that leave things for air brakes? Does that mean that this will be possible with a script?
ORTSDoesBrakeCutPower ( 1 )
ORTSBrakeCutsPowerAtBrakePipePressure ( 46psi )
ORTSBrakeRestoresPowerAtBrakePipePressure ( 60psi )

and what about working without a script?




#12 User is offline   Serana 

  • Conductor
  • Group: Status: Contributing Member
  • Posts: 489
  • Joined: 21-February 13
  • Gender:Male
  • Location:St Cyr l'Ecole (France)
  • Simulator:Open Rails
  • Country:

Posted 01 December 2021 - 06:41 AM

Air brakes are also already supported for the default TCS, but the parameter is not called ORTSDoesBrakeCutPower. It is called DoesBrakeCutPower.

#13 User is online   Csantucci 

  • Member, Board of Directors
  • Group: Status: Elite Member
  • Posts: 7,023
  • Joined: 31-December 11
  • Gender:Male
  • Country:

Posted 01 December 2021 - 12:03 PM

I am not happy with what has occurred here. As I have written also elsewhere, I am not against making changes in the core OR code that could affect existing TCS scripts, but this should be clearly announced, discussed and a balance should be publicly be drawn between changing the core OR code and changing the scrips.
In this case AFAIK nothing of the above has occurred, and the problem has been raised by an attentive OR simmer starting from the consequences of the issue.
The AWS TCS script has been downloaded 985 times from my site, and from this http://www.elvastowe...post__p__258806 I understand that there are also other download sites for the script. All this people should download a new release of the script, once I would have modified it. So I think it should be discussed if the OR code change is worth the disturbance.
Other problem: there are people that prefer using the stable release. So what? Should two versions of the script be available? Unacceptable in my opinion.

#14 User is offline   Serana 

  • Conductor
  • Group: Status: Contributing Member
  • Posts: 489
  • Joined: 21-February 13
  • Gender:Male
  • Location:St Cyr l'Ecole (France)
  • Simulator:Open Rails
  • Country:

Posted 01 December 2021 - 12:37 PM

I just had a better idea than what I did initially: integrate the default code in the abstract class as a virtual function.
That way, the code is included in every script automatically. For custom behaviour, we just have to override this function.

Edit : Well... it's not perfect... because you still have to call it from the Update function... In order to avoid modificating scripts, I will have to call the function from outside of the script.

#15 User is offline   darwins 

  • Superintendant
  • Group: Status: Elite Member
  • Posts: 1,251
  • Joined: 25-September 17
  • Gender:Male
  • Simulator:Open Rails
  • Country:

Posted 22 December 2021 - 08:10 AM

Hi,
This function is still not working when scripts are enabled. Any updates please?


#16 User is offline   Serana 

  • Conductor
  • Group: Status: Contributing Member
  • Posts: 489
  • Joined: 21-February 13
  • Gender:Male
  • Location:St Cyr l'Ecole (France)
  • Simulator:Open Rails
  • Country:

Posted 22 December 2021 - 08:25 AM

Hi,

No changes currently.

But anyway, this functionnality should have never been implemented outside of the TCS interface. The DoesBrakeCutsPower functionnality was implemented in the TCS interface long before pull request 103 implemented it for vacuum brakes seperatedly.

#17 User is offline   darwins 

  • Superintendant
  • Group: Status: Elite Member
  • Posts: 1,251
  • Joined: 25-September 17
  • Gender:Male
  • Simulator:Open Rails
  • Country:

Posted 22 December 2021 - 10:49 AM

As I am not neither a code writer nor a member of the OR Development team I am not in a position to argue about what should or should not have been done at any particular time.

As a content creator I thought that OR would stick to the matter of backwards compatibility. So often I have asked for changes to features to be told that it would break backwards compatibility with MSTS or with previous versions of OR. Right or wrong, this seems to me a case of something that was once working, that is no longer working.

#18 User is offline   darwins 

  • Superintendant
  • Group: Status: Elite Member
  • Posts: 1,251
  • Joined: 25-September 17
  • Gender:Male
  • Simulator:Open Rails
  • Country:

Posted 22 December 2021 - 11:04 AM

Whilst trying to get this fixed I note that this does not work for air brakes either if scripts are enabled

DoesBrakeCutPower ( 1 )
ORTSBrakeCutsPowerAtBrakePipePressure ( 46psi )
ORTSBrakeRestoresPowerAtBrakePipePressure ( 60psi )


(As with vacuum brakes it does work if scripts are disabled.)

#19 User is offline   Serana 

  • Conductor
  • Group: Status: Contributing Member
  • Posts: 489
  • Joined: 21-February 13
  • Gender:Male
  • Location:St Cyr l'Ecole (France)
  • Simulator:Open Rails
  • Country:

Posted 22 December 2021 - 11:07 AM

Which version of OR are you using?
Because on my side, without scripts, it works properly with similar parameters as yours.

#20 User is offline   darwins 

  • Superintendant
  • Group: Status: Elite Member
  • Posts: 1,251
  • Joined: 25-September 17
  • Gender:Male
  • Simulator:Open Rails
  • Country:

Posted 22 December 2021 - 12:34 PM

I am using U2021.12.22-0406. It does work without scripts for me too. The problem is that it does not work with scripts. (For either air or vacuum brakes.)

  • 5 Pages +
  • 1
  • 2
  • 3
  • 4
  • Last »
  • You cannot start a new topic
  • You cannot reply to this topic

1 User(s) are reading this topic
0 members, 1 guests, 0 anonymous users