Page 3 of 12

Re: Copy & Paste patch, reworked

Posted: 19 Mar 2009 20:25
by LittleHelper
Timitry wrote:...So, here is an actual version of this patch...
Thanks, Timitry, for your efforts. I will give it a try. 8)
Greetz


Edit: The patched trunk works, but I can't find the copy&paste symbol (copypaste.grf has been copied into the appropriate data folder). Any idea ?

Re: Copy & Paste patch, reworked

Posted: 20 Mar 2009 09:17
by Timitry
It should appear in the landscaping toolbar, if it's not there, something must be wrong with my provided patch / binaries...
If the game can't find the copypaste.grf, it won't start at all, so that shouldn't be the problem!

Re: Copy & Paste patch, reworked

Posted: 20 Mar 2009 21:06
by Iguanna
Bilbo wrote:...Hmm ... I have many useful templates made by older version (the one still from Frostregen). Have anybody made some tool to convert the templates from old format to the new one, or should I write it myself?
Not that I'm aware of. Might be easiest to just paste all your templates onto a game using an old OTTD version then load the game in v0.7 and copy em from there.

The pretty recent version of C&P (maybe current) would crash if you load templates sometimes too - seems to work better to keep the templates you use in a savegame, then if you're playing and want to use one, save your game, load the template game, copy what you want, then load the game you're playing and paste (without using load template). Is yet to crash doing that.

There were a couple of bugs and I need to sit down sometime and reproduce em and post saves etc so they can be fixed, if they are not already. Bridges not being pasted after rotate, and sometimes signals not right when there are 2 on the tile coners and it is rotated were the others IIRC. And also sometimes not enough teraforming is done for tunnels I think I saw a few times, or maybe it was not demolishing stuff which needed to be for teraform, or something.

Re: Copy & Paste patch, reworked

Posted: 23 Mar 2009 01:19
by 2007Alain2007
here is an update and 32 build r15821
CopyPaste_r15821.patch
copy and paste.7z

Re: Copy & Paste patch, reworked

Posted: 23 Mar 2009 22:54
by LittleHelper
Great, 2007Alain! Now copy&paste is working, even some of my old scenarios. Thanks for your efforts, much appreciated. 8)

Only problem now :roll: ... my old c&p templates don't work anymore. I get the message they have been created with a newer version !?
Is there any workaround editing the old templates ? :?:

Greetz

Re: Copy & Paste patch, reworked

Posted: 02 Apr 2009 20:30
by iklucas
will there come a copy/paste patch for 0.7.0?

Re: Copy & Paste patch, reworked

Posted: 02 Apr 2009 20:44
by belugas
iklucas wrote:will there come a copy/paste patch for 0.7.0?
No, definitively not, since 0.7.0 is already out.
And I can add that there is very very very little chance it will ever hit trunk.
No, I won't give reasons.

But there's nothing wrong in using it with some private builds.

Re: Copy & Paste patch, reworked

Posted: 02 Apr 2009 20:48
by SirkoZ
belugas - you should really consider it for inclusion as it can be quite useful in both, single and multiplayer. And no, there have been no desyncs observed - I have used it extensively in network games. ;-)

Re: Copy & Paste patch, reworked

Posted: 02 Apr 2009 20:51
by iklucas
is that sarcasm or realism?

whathever i have no experience with building patches so

Re: Copy & Paste patch, reworked

Posted: 02 Apr 2009 21:02
by Timmaexx
It's realism!
I myself gave up to try patching its too complicated for me....
but there are good manuals, perhaps you couldlearn it! 2007Alain2007 learned it too ;)
And dont bump everything full with your C&P Patching. This thread is the right one :D

Re: Copy & Paste patch, reworked

Posted: 02 Apr 2009 21:03
by Rubidium
SirkoZ wrote:And no, there have been no desyncs observed - I have used it extensively in network games. ;-)
For this statement I am assume using the latest version in this thread, i.e. the patch for r15821. I have not tested it in the real world; this is all based on a quick scan of the diff.

I'd take me probably 10-15 minutes to set up a server that "quickly" desyncs copy-paste users and if I want to I'd need 5 minutes to get all copy-paste users desynced from a server without copy-paste applied and the same amount of time to get all non-copy-paste users desynced from a server with copy-paste applied.

So this again such a marvel of a patch that "doesn't desync" and "isn't buggy" because no-one bothered to report the desyncs/bugs.

Re: Copy & Paste patch, reworked

Posted: 02 Apr 2009 21:20
by iklucas
can't there be made a copy paste mod wich work on 0.7.0, why whould i do can make it for myself if they say it can't be made?

Re: Copy & Paste patch, reworked

Posted: 02 Apr 2009 21:42
by Bilbo
Rubidium wrote:
SirkoZ wrote:And no, there have been no desyncs observed - I have used it extensively in network games. ;-)
For this statement I am assume using the latest version in this thread, i.e. the patch for r15821. I have not tested it in the real world; this is all based on a quick scan of the diff.

I'd take me probably 10-15 minutes to set up a server that "quickly" desyncs copy-paste users and if I want to I'd need 5 minutes to get all copy-paste users desynced from a server without copy-paste applied and the same amount of time to get all non-copy-paste users desynced from a server with copy-paste applied.

So this again such a marvel of a patch that "doesn't desync" and "isn't buggy" because no-one bothered to report the desyncs/bugs.
I think original author does not update it anymore and currently the patch is updated by "first random user that needs a newer version and happens to come by". That probably have somewhat affected its quality in last time :)

As for the bugs: I remember one crash to desktop when using the patch (though I have not managed to repeat the crash or isolate the cause), the pasting sometimes does not paste the tracks well. I think the patch is somewhat buggy, though I have not experienced a desync when using it. Where in the diff have you found that desync bug?

Re: Copy & Paste patch, reworked

Posted: 03 Apr 2009 00:45
by belugas
iklucas wrote:is that sarcasm or realism?
whathever i have no experience with building patches so
Sorry, no, as Rubidium explained, this is reality. As devs, we are not entitled to put everything that the users are sending us in trunk.
We need to keep the game in a working state. We have to make it grow.
This patch does not fall in the safe nor interesting side.
I'm sorry for you, iklucas, but that's the way it is.

Nice one Sirkoz. Very good joke "It's well tested" ! :mrgreen:
And oh yes, it is so useful. I agree. Up to the point that the game becomes boring.

Re: Copy & Paste patch, reworked

Posted: 03 Apr 2009 08:54
by Ammler
There is something much better then this patch: cooperative gameplay. If you like to build something really boring which would need this patch, join us and ask about help. If there are many people building on the same thing, it works without that patch and won't be boring...

Re: Copy & Paste patch, reworked

Posted: 03 Apr 2009 09:19
by SirkoZ
belugas wrote:Nice one Sirkoz. Very good joke "It's well tested" ! :mrgreen:
And oh yes, it is so useful. I agree. Up to the point that the game becomes boring.
I think I posted that on 2nd... anyhow - boring? Not really as one can use it for pasting only the very complex junctions, as I do. :idea:
Of course that doesn't mean at all the designs can't be improved...

Re: Copy & Paste patch, reworked

Posted: 03 Apr 2009 12:08
by Bilbo
Ammler wrote:There is something much better then this patch: cooperative gameplay. If you like to build something really boring which would need this patch, join us and ask about help. If there are many people building on the same thing, it works without that patch and won't be boring...
Well, before PBS was part of trunk, I used copy & paste to insert prepared complex presignal tracks. Now with PBS things are much easier and I use C&P quite rarely, but it can ease some repetitive tasks, like:
* Pasting service station with depots every few tiles of track
* When building something symmetric, you build one part, then copy, mirror/rotate and paste the other half :)

C&P won't do things for you, but helps you if you want same things on more places

Re: Copy & Paste patch, reworked

Posted: 03 Apr 2009 12:15
by 2007Alain2007
but for single player games its preaty good

Re: Copy & Paste patch, reworked

Posted: 03 Apr 2009 12:57
by Frostregen
Hm, desyncs should not be a problem, by the nature of this patch.
It used to only use DoCommandP's, which are the same as if a Player did it. (Random was not used at all too)

The only possibility could be the patch-setting (paste-speed) which was only half synchronized (since not everyone had this setting).
Back then it worked, but maybe a good start to look there ;)
(Check unpatched server + patched client, and the other way round)

Re: Copy & Paste patch, reworked

Posted: 03 Apr 2009 17:44
by Bilbo
Frostregen wrote:Hm, desyncs should not be a problem, by the nature of this patch.
It used to only use DoCommandP's, which are the same as if a Player did it. (Random was not used at all too)

The only possibility could be the patch-setting (paste-speed) which was only half synchronized (since not everyone had this setting).
Back then it worked, but maybe a good start to look there ;)
(Check unpatched server + patched client, and the other way round)
I think in older copy & paste versions it was client setting, so it was never synchronized and kept only locally. Maybe I'll change it back to client settings and post updated patch, so you can change it in-game if your selected speed does not work :)

Well, as I examine the patch, there seems to be some possibly unrelated changes (function GetDepotDirection is moved from npf.cpp to depot.cpp - perhaps to make something easier? OrderGoesToStation is moved as well and there seem to be some unrelated changes in order_cmd.cpp - is the patch really changing the code how orders are copied between vehicles, or someone have accidentally mixed in another patch? That could explain desyncs)

Perhaps the "reworked" patch needs a little cleanup ...