Project

General

Profile

Feature #764943

Separate tech trading chance from tech stealing chance

Added by Jacob Nevins 9 months ago. Updated 8 months ago.

Status:
Closed
Priority:
Normal
Category:
General
Target version:
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)In Progress2019-01-02

History

#1 Updated by Marko Lindqvist 9 months 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 9 months 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 9 months ago

  • Target version set to 3.0.0

#4 Updated by Marko Lindqvist 9 months ago

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

#5 Updated by Sveinung Kvilhaugsvik 8 months 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 8 months ago

Untested patch

#7 Updated by Sveinung Kvilhaugsvik 8 months ago

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

Also available in: Atom PDF