Patch: Improved Breakdowns v2.4 (r14563)

Forum for technical discussions regarding development. If you have a general suggestion, problem or comment, please use one of the other forums.

Moderator: OpenTTD Developers

2007Alain2007
Chief Executive
Chief Executive
Posts: 658
Joined: 11 Nov 2007 12:06

Re: Patch: Improved Breakdowns v2.4 (r14563)

Post by 2007Alain2007 » 26 Jun 2009 00:43

Thank you very much eddi

were did you look to find out this infor
For Community Integrated Version http://code.google.com/p/civopenttd/

Eddi
Tycoon
Tycoon
Posts: 7414
Joined: 17 Jan 2007 00:14

Re: Patch: Improved Breakdowns v2.4 (r14563)

Post by Eddi » 26 Jun 2009 00:56

reading and interpreting error messages correctly is one of the most elementary skills of a software developer.
also, checking the revision log for changes related to the suspected error cause, and grepping for the definition of the missing function are obvious (to me) things to do. takes a total of two to ten minutes...
You might not exactly be interested in Ferion, but if you are, have fun :)

2007Alain2007
Chief Executive
Chief Executive
Posts: 658
Joined: 11 Nov 2007 12:06

Re: Patch: Improved Breakdowns v2.4 (r14563)

Post by 2007Alain2007 » 26 Jun 2009 00:58

i fort it might have been some thing like that but never thought what one it be and thank you once again and i should learn to look longer for myself befor asking for help or i never learn
Last edited by 2007Alain2007 on 28 Jun 2009 00:39, edited 1 time in total.
For Community Integrated Version http://code.google.com/p/civopenttd/

audigex
Tycoon
Tycoon
Posts: 1923
Joined: 09 Dec 2007 21:28
Contact:

Re: Patch: Improved Breakdowns v2.4 (r14563)

Post by audigex » 27 Jun 2009 19:59

At risk of being obnoxious, the word is "thought" not "fort" which is a defensive construction used in warfare... pet peeve of mine, sorry :-)
Jon

2007Alain2007
Chief Executive
Chief Executive
Posts: 658
Joined: 11 Nov 2007 12:06

Re: Patch: Improved Breakdowns v2.4 (r16388)

Post by 2007Alain2007 » 29 Jun 2009 23:36

I getting tired of trying to update this patchs i bringed it from r14563 to now at 16388 dose any one fanice takeing it the rest
improved-breakdowns-v2.4 16388.patch
there proble bugs but it works with no proble that i see on R16388
(58.09 KiB) Downloaded 119 times
hope your like to test it out

The patch is using Road and ship togiver so when there changed all the names i think i cant slove it

Unless it is safe to copy the hole of HandleBrokenRoadVeh in to the ship one but i have not got a clue how i am gonig to do that just yet

If any one wants a win 32 build i am more then happy to make one for you
For Community Integrated Version http://code.google.com/p/civopenttd/

2007Alain2007
Chief Executive
Chief Executive
Posts: 658
Joined: 11 Nov 2007 12:06

Re: Patch: Improved Breakdowns v2.4 (r16389)

Post by 2007Alain2007 » 09 Jul 2009 23:13

ERROR

In file included from C:/msys/1.0/home/Alainspc/ottdsrc/trunk/src/pbs.cpp:8:
C:/msys/1.0/home/Alainspc/ottdsrc/trunk/src/yapf/follow_track.hpp: In member function `bool CFollowTrackT<Ttr_type_, T90deg_turns_allowed_, Tmask_reserved_tracks>::Follow(TileIndex, Trackdir)':
C:/msys/1.0/home/Alainspc/ottdsrc/trunk/src/yapf/follow_track.hpp:108: error: 'const union Vehicle::<anonymous>' has no member named 'road'
make[1]: *** [pbs.o] Error 1
make[1]: Leaving directory `/home/Alainspc/ottdsrc/trunk/objs/release'
make: *** [all] Error 2

UPDATED to R16389 but got stuck with chages in r16390 evern that the patch is no toching them files

I think its src/roadveh.h but i am not 100% i allso changed the code so ships and roac are apart so should be easyer to update
IB V2.4 R16389.patch
(58.4 KiB) Downloaded 127 times
For Community Integrated Version http://code.google.com/p/civopenttd/

2007Alain2007
Chief Executive
Chief Executive
Posts: 658
Joined: 11 Nov 2007 12:06

Re: Patch: Improved Breakdowns v2.4 (r14563)

Post by 2007Alain2007 » 09 Sep 2009 10:19

22 downloads but no coments on it or ideas on fixes
For Community Integrated Version http://code.google.com/p/civopenttd/

User avatar
SirkoZ
Tycoon
Tycoon
Posts: 1505
Joined: 06 Mar 2004 23:51
Location: The sunny side of Alps

Re: Patch: Improved Breakdowns v2.3 (r14458)

Post by SirkoZ » 12 Sep 2009 12:28

Hirundo wrote:Personally I consider this patch finished now. I'm not planning any new features or changes, although I will keep it updated to the trunk and fix any bugs that appear. Therefore I would like to ask to (one of) the devs; what is your opinion on this patch? Is it eligible for trunk inclusion? Does it need changes to functionality, code style or anything else? Answers to some or all of these questions would be very much appreciated.
I would like that one of the developers to answer Hirundo's request for an opinion on inclusion/code-style, etc. It has been nearly one year since his request for it. :o

By the way where are you Hirundo?

audigex
Tycoon
Tycoon
Posts: 1923
Joined: 09 Dec 2007 21:28
Contact:

Re: Patch: Improved Breakdowns v2.4 (r14563)

Post by audigex » 12 Sep 2009 20:20

Inclusion isn't based on an "is it finished" - finished isn't the same as "suitable for trunk". The feature needs to add materially to gameplay, not ruin savegame compatability and various other bits and bobs. It also has to do enough to be worth it's code.

Eg a small patch with minor features is much more likely to get in than a large patch with the same feature, the devs don't want to tax people's machines with un-necessary things.

I can't comment on the liklihood for this particular case - simply because I'm not a dev, I know very little about the source code or this patch etc - but there are many reasons why something wouldn't be included; although I'd like to see it myself, that doesn't mean it's right for trunk,
Jon

User avatar
SirkoZ
Tycoon
Tycoon
Posts: 1505
Joined: 06 Mar 2004 23:51
Location: The sunny side of Alps

Re: Patch: Improved Breakdowns v2.4 (r14563)

Post by SirkoZ » 12 Sep 2009 20:57

As I said - I would like a developers opinion on the state of the patch, since Hirundo is gone and petern did show some interest in the patch beginning.

Hirundo
Transport Coordinator
Transport Coordinator
Posts: 298
Joined: 27 Jan 2008 13:02

Re: Patch: Improved Breakdowns v2.4 (r14563)

Post by Hirundo » 13 Sep 2009 09:58

*cough* Although I'm not a dev either, I would not trunkify this patch in its current state if I were. It needs quite a lot of work to update and clean it and to split it into parts. Currently I lack the time to do so, leaving this patch in a dormant state.
Create your own NewGRF? Check out this tutorial!

2007Alain2007
Chief Executive
Chief Executive
Posts: 658
Joined: 11 Nov 2007 12:06

Re: Patch: Improved Breakdowns v2.4 (r14563)

Post by 2007Alain2007 » 13 Sep 2009 11:29

My update to your patch is that ok to keep there or did you want me to removie it becuse i proble made it dirty in codeing words
For Community Integrated Version http://code.google.com/p/civopenttd/

User avatar
SirkoZ
Tycoon
Tycoon
Posts: 1505
Joined: 06 Mar 2004 23:51
Location: The sunny side of Alps

Re: Patch: Improved Breakdowns v2.4 (r14563)

Post by SirkoZ » 13 Sep 2009 21:29

Hirundo wrote:*cough* Although I'm not a dev either, I would not trunkify this patch in its current state if I were. It needs quite a lot of work to update and clean it and to split it into parts. Currently I lack the time to do so, leaving this patch in a dormant state.
:lol: Here he be!
I see - well - nice of you to clear that out. It's a nice patch idea-wise, so I hope you find time to clean it up. ;-)

User avatar
bigos
Traffic Manager
Traffic Manager
Posts: 181
Joined: 13 Sep 2009 21:15
Location: Poland
Contact:

Re: Patch: Improved Breakdowns v2.4 (r14563)

Post by bigos » 23 Oct 2009 23:15

Merged with trunk r17811.
Attachments
IB2.4_r17811.patch
(57.25 KiB) Downloaded 95 times

Tekky
Route Supervisor
Route Supervisor
Posts: 420
Joined: 19 Dec 2006 04:24

Re: Patch: Improved Breakdowns v2.4 (r14563)

Post by Tekky » 23 Oct 2009 23:47

Hirundo wrote:It needs quite a lot of work to update and clean it and to split it into parts.
You may also want to think about whether you would like to include my "no reliability decrease for stopped trains" patch into your patch, since my patch is a very simple patch. On the other hand, it may also be appropriate for me to include your patch into mine. :-)

2007Alain2007
Chief Executive
Chief Executive
Posts: 658
Joined: 11 Nov 2007 12:06

Re: Patch: Improved Breakdowns v2.4 (r14563)

Post by 2007Alain2007 » 23 Oct 2009 23:59

very simple patch are easyer to use myself add to much and the patch grows bigger and biger and out of control
For Community Integrated Version http://code.google.com/p/civopenttd/

User avatar
bigos
Traffic Manager
Traffic Manager
Posts: 181
Joined: 13 Sep 2009 21:15
Location: Poland
Contact:

Re: Patch: Improved Breakdowns v2.4 (r14563)

Post by bigos » 24 Oct 2009 16:41

Because this patch, in my opinion, make game a bit easier I have modified it a bit. I have added effect of mechanical failure - maximum speed of train is permanently reduced to 50% (more locos in train - less speed reduction). Reduction can be removed by visiting depot.

What do you think guys about my upgrade ?

EDIT: Fixed
Attachments
IB2.4_speed_reduction_r17863.patch
IB 2.4 with additional speed reduction after mechanical breakdown
(58.61 KiB) Downloaded 97 times

User avatar
bigos
Traffic Manager
Traffic Manager
Posts: 181
Joined: 13 Sep 2009 21:15
Location: Poland
Contact:

Re: Patch: Improved Breakdowns v2.4 (r14563)

Post by bigos » 31 Oct 2009 10:26

There was small bug in last version. Fixed and merged with trunk.
Attachments
IB_24_speed_reduction.patch
IB 2.4 with additional speed reduction for r17911
(58.63 KiB) Downloaded 95 times

Tekky
Route Supervisor
Route Supervisor
Posts: 420
Joined: 19 Dec 2006 04:24

Re: Patch: Improved Breakdowns v2.4 (r14563)

Post by Tekky » 05 Nov 2009 21:03

bigos wrote:Because this patch, in my opinion, make game a bit easier I have modified it a bit. I have added effect of mechanical failure - maximum speed of train is permanently reduced to 50% (more locos in train - less speed reduction). Reduction can be removed by visiting depot.

What do you think guys about my upgrade ?
Although I haven't actually tested it yet, I like this idea. In fact, I suggested this feature myself some time ago in this post.

When you implement this, I consider it very important that a train that has once suffered a breakdown with permanent damage should consider itself as scheduled to service inside a depot, even if the scheduled service period has not expired yet. For example, a "service in depot" order should act like a "got to depot" order, in this case. Otherwise, the permanent damage will not get repaired until the scheduled service period expires. Has this already been taken into account by your patch?

User avatar
bigos
Traffic Manager
Traffic Manager
Posts: 181
Joined: 13 Sep 2009 21:15
Location: Poland
Contact:

Re: Patch: Improved Breakdowns v2.4 (r14563)

Post by bigos » 05 Nov 2009 21:59

I did not want to to make order service in depot because it can realy mess up in your routs. Such order will for example pass the station and will go to depot behind it - if its 1 way station then it will be lost or go to loading station... it can really screw game. For now its like you said. Broken until next service scheduled or forced by you (constructing tracks when train always go to depot). Other way to repair this is to use waypoints. You can create advanced orders list with conditional orders and when for example max speed is lower then half of maximum then go to depot and repair. I think its better way then forcing all trains to repair after mechanical failure. And you will have more fun with using waypoits ;)
What do you think now? :)

Post Reply

Return to “OpenTTD Development”

Who is online

Users browsing this forum: Google [Bot] and 5 guests