patchFreeciv - Patches: patch #1340, Technology requirements

 
 
Show feedback again

patch #1340: Technology requirements

Submitted by:  pepeto <pepeto>
Submitted on:  Sat Oct 17 14:16:56 2009  
 
Category: rulesetsPriority: 1 - Later
Status: NonePrivacy: Public
Assigned to: NoneOpen/Closed: Open
Planned Release: Contains string changes: None

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Sun Mar 9 16:45:02 2014, comment #1:

Perhaps this should be split into smaller tasks. Would the plan below be acceptable?

Research:
1) Introduce a new requirement vector research_req. Add ruleset loading time checks that no tech is in it. In my view tech tree holes don't apply to missing non tech requirements: Even if you don't have access to the ocean you can still know that a sail can be used to make a boat move.
2) Make it well supported in the ai, help system etc. Maybe use a color in the tech tree help to symbolize that a non tech requirement is needed?
3) Append the ruleset fields req1 and req2 to the research_req vector during ruleset loading. Keep them like before in the rule set. Make sure that tech req specific stuff like allow tech tree holes work as expected.
4) Remove the limit of max 2 tech requirements for a tech, allow tech requirements in the research_req requirement vector and remove the ruleset fields req1 and req2.

Root req:
Introduce a new requirement vector acquire_reqs for root requirements in the same way as research_reqs.

Prevent intentionally blocking other research to gain points from Future Techs:
Add a new effect Future_Tech to control if a player can research future techs when there is nothing else to research. When there is no techs that can be researched and this isn't active something else happens to the bulbs. Not sure if they should be saved, lost or converted to something else. An alternative is an effect that triggers the alternative behavior.

Sveinung Kvilhaugsvik <sveinung>
Project Member
Sat Oct 17 14:16:56 2009, original submission:

Remove the following fields of the struct advance:

  • require
  • num_reqs

Replace them by 'requirement_vector reqs'.

Problem would happen with future tech if no tech is available whereas some non-researched are still existing.

pepeto <pepeto>
Project Member

 

(Note: upload size limit is set to 1024 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Digest:
   patch dependencies.

Digest:
   patch dependencies.

 

Carbon-Copy List
  • -unavailable- added by sveinung (Posted a comment)
  • -unavailable- added by jtn (Updated the item)
  • -unavailable- added by dmarks (Updated the item)
  • -unavailable- added by pepeto (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Error: not logged in

     

     

    Follow 5 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Tue Mar 24 22:31:39 2015sveinungDependencies-=>Depends on patch #5944
    Mon Jun 18 01:15:29 2012jtnPlanned Release2.4.0=>
    Wed Oct 27 13:54:19 2010pepetoPlanned Release2.3.0=>2.4.0
    Sun Dec 20 09:41:13 2009dmarksPlanned Release=>2.3.0
    Sat Oct 17 14:31:36 2009pepetoDependencies-=>patch #1338 is dependent
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup