patchFreeciv - Patches: patch #1340, Technology requirements

 
 
Show feedback again

patch #1340: Technology requirements

Submitted by:  pepeto <pepeto>
Submitted on:  Sat 17 Oct 2009 02:16:56 PM UTC  
 
Category: rulesetsPriority: 1 - Later
Status: NonePrivacy: Public
Assigned to: NoneOpen/Closed: Open
Planned Release: 

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

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

 

Sun 09 Mar 2014 04:45:02 PM UTC, 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 17 Oct 2009 02:16:56 PM UTC, 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

 

Depends on the following items: None found

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.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Mon 18 Jun 2012 01:15:29 AM UTCjtnPlanned Release2.4.0=>
    Wed 27 Oct 2010 01:54:19 PM UTCpepetoPlanned Release2.3.0=>2.4.0
    Sun 20 Dec 2009 09:41:13 AM UTCdmarksPlanned Release=>2.3.0
    Sat 17 Oct 2009 02:31:36 PM UTCpepetoDependencies-=>patch #1338 is dependent
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup