JGR's Patch Pack

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

SimYouLater
Chief Executive
Chief Executive
Posts: 675
Joined: 03 Apr 2016 20:19

Re: JGR's Patch Pack

Post by SimYouLater »

Can someone help me figure out how to put my NewGRFs and save game from the PC version of this patch pack to the Android version?
Licenses for my work...
You automatically have my permission to re-license graphics or code by me if needed for use in any project that is not GPL v2, on the condition that if you release any derivatives of my graphics they're automatically considered as ALSO GPL v2 (code may remain unreleased, but please do provide it) and carry this provision in GPL v2 uses.
Please ask someone in-the-know to be sure that the graphics are done by me. Especially TTD-Scale, long story.
pelya
Transport Coordinator
Transport Coordinator
Posts: 342
Joined: 18 Nov 2010 19:48
Contact:

Re: JGR's Patch Pack

Post by pelya »

Copy them to your SD card to Android/data/org.openttd.jgrpp/files/.openttd/save
SimYouLater
Chief Executive
Chief Executive
Posts: 675
Joined: 03 Apr 2016 20:19

Re: JGR's Patch Pack

Post by SimYouLater »

pelya wrote:Copy them to your SD card to Android/data/org.openttd.jgrpp/files/.openttd/save
Found it. For some reason the Android/data/org.openttd.jgrpp folder wasn't showing up before. Also, my phone doesn't have external SD cards.
Licenses for my work...
You automatically have my permission to re-license graphics or code by me if needed for use in any project that is not GPL v2, on the condition that if you release any derivatives of my graphics they're automatically considered as ALSO GPL v2 (code may remain unreleased, but please do provide it) and carry this provision in GPL v2 uses.
Please ask someone in-the-know to be sure that the graphics are done by me. Especially TTD-Scale, long story.
SimYouLater
Chief Executive
Chief Executive
Posts: 675
Joined: 03 Apr 2016 20:19

Severe Bug in JGR's Patch Pack

Post by SimYouLater »

I am not seeing town names. At all. Screenshot to demonstrate with new game and no GRFs.
Attachments
Unnamed, 2518-01-01.png
Town names not showing?
(345.2 KiB) Not downloaded yet
Licenses for my work...
You automatically have my permission to re-license graphics or code by me if needed for use in any project that is not GPL v2, on the condition that if you release any derivatives of my graphics they're automatically considered as ALSO GPL v2 (code may remain unreleased, but please do provide it) and carry this provision in GPL v2 uses.
Please ask someone in-the-know to be sure that the graphics are done by me. Especially TTD-Scale, long story.
XaTriX
Engineer
Engineer
Posts: 47
Joined: 31 Jul 2008 18:38
Location: France
Contact:

Re: JGR's Patch Pack

Post by XaTriX »

Hi there :)

First of all, thanks JGR for the patch pack ;)

Is that build known as desync-less for large MP game ? Hundreds of trains, few players, large map, new industries, ... ?

XaT
SimYouLater
Chief Executive
Chief Executive
Posts: 675
Joined: 03 Apr 2016 20:19

Re: JGR's Patch Pack

Post by SimYouLater »

XaTriX wrote:Hi there :)

First of all, thanks JGR for the patch pack ;)

Is that build known as desync-less for large MP game ? Hundreds of trains, few players, large map, new industries, ... ?

XaT
The screenshot isn't from a multiplayer game.

I first had the issue with a 16???x8??? map with many GRFs, however...

I tested it with, and provided that screenshot of, a brand new 64x64 map with no GRFs loaded. This is not a NewGRF issue, it's game-wide.
Licenses for my work...
You automatically have my permission to re-license graphics or code by me if needed for use in any project that is not GPL v2, on the condition that if you release any derivatives of my graphics they're automatically considered as ALSO GPL v2 (code may remain unreleased, but please do provide it) and carry this provision in GPL v2 uses.
Please ask someone in-the-know to be sure that the graphics are done by me. Especially TTD-Scale, long story.
User avatar
JGR
Tycoon
Tycoon
Posts: 2557
Joined: 08 Aug 2005 13:46
Location: Ipswich

Re: Severe Bug in JGR's Patch Pack

Post by JGR »

SimYouLater wrote:I am not seeing town names. At all. Screenshot to demonstrate with new game and no GRFs.
A config, savegame and any other info which you can supply would be helpful.
This is not an issue which has previously been observed.
XaTriX wrote:Hi there :)

First of all, thanks JGR for the patch pack ;)

Is that build known as desync-less for large MP game ? Hundreds of trains, few players, large map, new industries, ... ?

XaT
There are no known desync bugs at present.
Ex TTDPatch Coder
Patch Pack, Github
SimYouLater
Chief Executive
Chief Executive
Posts: 675
Joined: 03 Apr 2016 20:19

Re: Severe Bug in JGR's Patch Pack

Post by SimYouLater »

JGR wrote:
SimYouLater wrote:I am not seeing town names. At all. Screenshot to demonstrate with new game and no GRFs.
A config, savegame and any other info which you can supply would be helpful.
This is not an issue which has previously been observed.
XaTriX wrote:Hi there :)

First of all, thanks JGR for the patch pack ;)

Is that build known as desync-less for large MP game ? Hundreds of trains, few players, large map, new industries, ... ?

XaT
There are no known desync bugs at present.
(Removed due to passwords inside)

Will be uploading a zip of my entire OpenTTD folder if needed.
Last edited by SimYouLater on 29 Dec 2017 02:59, edited 1 time in total.
Licenses for my work...
You automatically have my permission to re-license graphics or code by me if needed for use in any project that is not GPL v2, on the condition that if you release any derivatives of my graphics they're automatically considered as ALSO GPL v2 (code may remain unreleased, but please do provide it) and carry this provision in GPL v2 uses.
Please ask someone in-the-know to be sure that the graphics are done by me. Especially TTD-Scale, long story.
ST2
Engineer
Engineer
Posts: 63
Joined: 07 Apr 2011 14:17

Re: JGR's Patch Pack

Post by ST2 »

just a hint/sidenote @SimYouLater:

maybe you'll want to hide "server_password", "rcon_password" and "admin_password" next time(s) when sharing cfg files ^^
SimYouLater
Chief Executive
Chief Executive
Posts: 675
Joined: 03 Apr 2016 20:19

Re: JGR's Patch Pack

Post by SimYouLater »

ST2 wrote:just a hint/sidenote @SimYouLater:

maybe you'll want to hide "server_password", "rcon_password" and "admin_password" next time(s) when sharing cfg files ^^
Ah, I guess I'll want new passwords. Good thing I only used those passwords with OpenTTD.
Licenses for my work...
You automatically have my permission to re-license graphics or code by me if needed for use in any project that is not GPL v2, on the condition that if you release any derivatives of my graphics they're automatically considered as ALSO GPL v2 (code may remain unreleased, but please do provide it) and carry this provision in GPL v2 uses.
Please ask someone in-the-know to be sure that the graphics are done by me. Especially TTD-Scale, long story.
SimYouLater
Chief Executive
Chief Executive
Posts: 675
Joined: 03 Apr 2016 20:19

Re: Severe Bug in JGR's Patch Pack

Post by SimYouLater »

How would I post a copy of my install? It's too large for the forums.
Licenses for my work...
You automatically have my permission to re-license graphics or code by me if needed for use in any project that is not GPL v2, on the condition that if you release any derivatives of my graphics they're automatically considered as ALSO GPL v2 (code may remain unreleased, but please do provide it) and carry this provision in GPL v2 uses.
Please ask someone in-the-know to be sure that the graphics are done by me. Especially TTD-Scale, long story.
ST2
Engineer
Engineer
Posts: 63
Joined: 07 Apr 2011 14:17

Re: JGR's Patch Pack

Post by ST2 »

SimYouLater wrote: Ah, I guess I'll want new passwords. Good thing I only used those passwords with OpenTTD.
A bit talking on the side, because never tested JGR's Patch Pack outside SP mode, but I think most important is: version and settings used (without PW's this time ^^).

One of the common errors that I've seen, or lets calling it "not used to", is trying to run several and/or modified instances of OpenTTD without taking care of where the "openttd.cfg" file is.
And JGR's Patch Pack, when unzipped and run, will load the "openttd.cfg" file of a Vanilla version, if installed. And vice-versa.

So, my suggestion is: place "openttd.cfg" on the same folder as the executable openttd file (even if empty, openttd will build it), and this way you can run different instances, versions, or with different settings. Whatever comes to your mind ^^
Chico008
Traffic Manager
Traffic Manager
Posts: 143
Joined: 05 Mar 2008 10:56
Location: France

Re: JGR's Patch Pack

Post by Chico008 »

Hi

Don't know if it has been asked yet, but i have a problem compiling

i'm using the r27935 from svn, then apply the diff file
configure and make or OK, but when i launch a dedicated server, i can't connect using the client provided on first page

There is a version mismatch
dedicated server got version : jgrpp-0.22.1-M-
win32/64 clients are : jgrpp-0.22.1

How can i compile to get the same versions on the dedicated serveur ?
Chico008
Traffic Manager
Traffic Manager
Posts: 143
Joined: 05 Mar 2008 10:56
Location: France

Re: JGR's Patch Pack

Post by Chico008 »

Answering to myself

had to modify the findversion.sh like this (but launch it manually once before in case, and make sure you have the .ottdrev-vc file

Code: Select all

elif [ -f "$ROOT_DIR/.ottdrev-vc" ]; then
        VERSION_DATA="`cat "$ROOT_DIR/.ottdrev-vc" | sed -n -e '1 p;'`"
        HASH_DATA="`cat "$ROOT_DIR/.ottdrev-vc" | sed -n -e '2 p;'`"
        REV="`echo "$VERSION_DATA" | cut -f 1 -d'       '`"
        REV_NR="`echo "$VERSION_DATA" | cut -f 2 -d'    '`"
        MODIFIED="`echo "$VERSION_DATA" | cut -f 3 -d'  '`"
        CLEAN_REV="`echo "$VERSION_DATA" | cut -f 4 -d' '`"
#        if [ "$MODIFIED" = "2" ]; then
#                CLEAN_REV="`echo "$CLEAN_REV" | sed -e 's/M$//'`"
#        fi
#        BRANCH="`echo "$REV" | sed -n -e "s|^${CLEAN_REV}M\?-\(.\+\)$|\1|p"`"
#        REV="$CLEAN_REV"
#        if ! $ROOT_DIR/version_utils.sh -o; then
#                MODIFIED="1"
#        else
#                CURRENT_HASH="`$ROOT_DIR/version_utils.sh -s`"
#                if [ "$CURRENT_HASH" != "$HASH_DATA" ]; then
#                        MODIFIED="2"
#                        if [ -n "$BRANCH" ]; then
#                                BRANCH="$BRANCH-"
#                        fi
 #                       BRANCH="${BRANCH}H`echo "$CURRENT_HASH" | cut -c 1-8`"
#                fi
#        fi
 
SimYouLater
Chief Executive
Chief Executive
Posts: 675
Joined: 03 Apr 2016 20:19

Re: JGR's Patch Pack

Post by SimYouLater »

ST2 wrote:
SimYouLater wrote: Ah, I guess I'll want new passwords. Good thing I only used those passwords with OpenTTD.
A bit talking on the side, because never tested JGR's Patch Pack outside SP mode, but I think most important is: version and settings used (without PW's this time ^^).

One of the common errors that I've seen, or lets calling it "not used to", is trying to run several and/or modified instances of OpenTTD without taking care of where the "openttd.cfg" file is.
And JGR's Patch Pack, when unzipped and run, will load the "openttd.cfg" file of a Vanilla version, if installed. And vice-versa.

So, my suggestion is: place "openttd.cfg" on the same folder as the executable openttd file (even if empty, openttd will build it), and this way you can run different instances, versions, or with different settings. Whatever comes to your mind ^^
I don't have a copy of vanilla OpenTTD on my computer. I even removed other copies of JGR when I noticed there were some, no dice. I'll try redoing the install and see if that works.

EDIT
It looks like the cfg file went bad somehow. I had to start over with a fresh one, fortunately my NewGRF settings weren't the problem. Thank you!
Licenses for my work...
You automatically have my permission to re-license graphics or code by me if needed for use in any project that is not GPL v2, on the condition that if you release any derivatives of my graphics they're automatically considered as ALSO GPL v2 (code may remain unreleased, but please do provide it) and carry this provision in GPL v2 uses.
Please ask someone in-the-know to be sure that the graphics are done by me. Especially TTD-Scale, long story.
Wahazar
Tycoon
Tycoon
Posts: 1451
Joined: 18 Jan 2014 18:10

Re: JGR's Patch Pack

Post by Wahazar »

I noticed, that "velocity reduced until depot visited" improved patch breakdown occurs in case of trains, but not in case of road vehicles (not tested ships and airplanes yet).
Is it a bug or feature? In my opinion, it is rather unwanted feature, together with repair payment, road vehicles have now a big advantage over trains - no need to send truck/buses to depot and pay for breakdowns.
Formerly known as: McZapkie
Projects: Reproducible Map Generation patch, NewGRFs: Manpower industries, PolTrams, Polroad, 600mm narrow gauge, wired, ECS industry extension, V4 CEE train set, HotHut.
Another favorite games: freeciv longturn, OHOL/2HOL.
User avatar
JGR
Tycoon
Tycoon
Posts: 2557
Joined: 08 Aug 2005 13:46
Location: Ipswich

Re: JGR's Patch Pack

Post by JGR »

Chico008 wrote:Hi

Don't know if it has been asked yet, but i have a problem compiling

i'm using the r27935 from svn, then apply the diff file
configure and make or OK, but when i launch a dedicated server, i can't connect using the client provided on first page

There is a version mismatch
dedicated server got version : jgrpp-0.22.1-M-
win32/64 clients are : jgrpp-0.22.1

How can i compile to get the same versions on the dedicated serveur ?
This suggests that the patched source is not identical to that of the release, probably because one or more source files have have been manually edited in some way.
In general patching the version detection script is not a good solution as the whole point of the modification check is to avoid the multi-player desync issues which can occur when a modified build is incorrectly labelled as an unmodified release build, and is subsequently used with unmodified release builds.
McZapkie wrote:I noticed, that "velocity reduced until depot visited" improved patch breakdown occurs in case of trains, but not in case of road vehicles (not tested ships and airplanes yet).
Is it a bug or feature? In my opinion, it is rather unwanted feature, together with repair payment, road vehicles have now a big advantage over trains - no need to send truck/buses to depot and pay for breakdowns.
The code suggests that this is intended.
Trains already have so many advantages over road vehicles that this does not seem to be a significant issue.
Ex TTDPatch Coder
Patch Pack, Github
Wahazar
Tycoon
Tycoon
Posts: 1451
Joined: 18 Jan 2014 18:10

Re: JGR's Patch Pack

Post by Wahazar »

JGR wrote:The code suggests that this is intended.
Trains already have so many advantages over road vehicles that this does not seem to be a significant issue.
Maybe this setting was justified in case of standalone breakdown patch, but in case of this patch pack,
with both patches: improved breakdown and repair payment are active - players can cheat by avoiding using depots, because there is no penalty for lack of service.

On the other hand, these permanent speed limits can be annoying, if there is 1 mph limit and you need t wait eons until train will reach depot. It was already pointed out in Improved Breakdown Patch thread.
Minimal velocity should be clamped by let say 10 mph.
Formerly known as: McZapkie
Projects: Reproducible Map Generation patch, NewGRFs: Manpower industries, PolTrams, Polroad, 600mm narrow gauge, wired, ECS industry extension, V4 CEE train set, HotHut.
Another favorite games: freeciv longturn, OHOL/2HOL.
Auge
Director
Director
Posts: 635
Joined: 23 Oct 2006 02:07
Location: Berlin

Re: JGR's Patch Pack

Post by Auge »

Hello
McZapkie wrote:… in case of this patch pack, with both patches: improved breakdown and repair payment are active - players can cheat by avoiding using depots [for road vehicles], because there is no penalty for lack of service.

On the other hand, these permanent speed limits can be annoying, if there is 1 mph limit and you need t wait eons until train will reach depot. It was already pointed out in Improved Breakdown Patch thread.
Minimal velocity should be clamped by let say 10 mph.
Did you ever found a speed limit of 1mph for a broken rail vehicle? I can't remember an experience of a speed limit lower than 40 km/h (kph) for a broken rail vehicle. I play the patchpack at least since two years (first documented translation for version 0.7.1 (2015-11-01) in the changelog of the opening posting).

So IMHO there is no reason to hyperventilate. ;-)

Tschö, Auge
Wahazar
Tycoon
Tycoon
Posts: 1451
Joined: 18 Jan 2014 18:10

Re: JGR's Patch Pack

Post by Wahazar »

Auge wrote:Hello
Did you ever found a speed limit of 1mph for a broken rail vehicle?
Of course, I bought bankrupt company and got engine with 255 failures and 1mph speed limit. Additionally, the cost of the depot was above million GPB, so I was breathing heavily ;)
Formerly known as: McZapkie
Projects: Reproducible Map Generation patch, NewGRFs: Manpower industries, PolTrams, Polroad, 600mm narrow gauge, wired, ECS industry extension, V4 CEE train set, HotHut.
Another favorite games: freeciv longturn, OHOL/2HOL.
Post Reply

Return to “OpenTTD Development”

Who is online

Users browsing this forum: No registered users and 19 guests