Project

Profile

Help

HostedRedmine.com has moved to the Planio platform. All logins and passwords remained the same. All users will be able to login and use Redmine just as before. Read more...

Feature #764943

Separate tech trading chance from tech stealing chance

Added by Jacob Nevins about 3 years ago. Updated almost 3 years ago.

Status:
Closed
Priority:
Normal
Category:
General
Sprint/Milestone:
Start date:
Due date:
% Done:

100%

Estimated time:

Description

http://forum.freeciv.org/f/viewtopic.php?f=11&t=90258

https://github.com/longturn/freeciv-S2_5/commit/ddafed38f77adc0a7db147ea3d562be168648963

Original problem statement from wieder: "We are setting up LT44 tat will have two teams and tech trading inside teams. Is there a way to make tech trading possible but also to make it very hard to steal from the other team? Diplo chance seems to be 40% at the lowers setting for 2.5."

Solution committed to longturn server: new server setting "diplchance_steal", which is used instead of "diplchance" in diplomat_get_tech().

(If we go for a similar solution, since that wouldn't be a format change, we could even introduce it in 2.6 series post-2.6.0?)


Related issues

Blocks Freeciv - Task #656466: S3_0 datafile format freeze (d3f)Closed2019-10-05

<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>

History

#1 Updated by Marko Lindqvist about 3 years ago

Jacob Nevins wrote:

(If we go for a similar solution, since that wouldn't be a format change, we could even introduce it in 2.6 series post-2.6.0?)

We've usually treated new server settings as format change since they can be set in game.ruleset (and also in the .serv file)

#2 Updated by Marko Lindqvist about 3 years ago

Subject is probably not what it's meant to be. Tech trading chance and tech stealing chance already have nothing to do with each other, I think. The problem they have solved sounds more like separating tech stealing chance from chances of other spy actions.

#3 Updated by Marko Lindqvist about 3 years ago

  • Sprint/Milestone set to 3.0.0

#4 Updated by Marko Lindqvist about 3 years ago

  • Blocks Task #656466: S3_0 datafile format freeze (d3f) added

#5 Updated by Sveinung Kvilhaugsvik almost 3 years ago

Marko Lindqvist wrote:

Subject is probably not what it's meant to be. Tech trading chance and tech stealing chance already have nothing to do with each other, I think. The problem they have solved sounds more like separating tech stealing chance from chances of other spy actions.

I think this could be implemented by making the Action_Odds_Pct effect work for the tech theft actions.

#6 Updated by Sveinung Kvilhaugsvik almost 3 years ago

Untested patch

#7 Updated by Sveinung Kvilhaugsvik almost 3 years ago

  • Status changed from Resolved to Closed
  • % Done changed from 90 to 100

Also available in: Atom PDF