From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 30345 invoked by alias); 11 Sep 2004 05:05:50 -0000 Mailing-List: contact xconq7-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: xconq7-owner@sources.redhat.com Received: (qmail 30321 invoked from network); 11 Sep 2004 05:05:49 -0000 Received: from unknown (HELO smtp812.mail.sc5.yahoo.com) (66.163.170.82) by sourceware.org with SMTP; 11 Sep 2004 05:05:49 -0000 Received: from unknown (HELO ?192.168.1.101?) (sampln@sbcglobal.net@67.121.168.201 with plain) by smtp812.mail.sc5.yahoo.com with SMTP; 11 Sep 2004 05:05:49 -0000 Subject: Re: Possible bug in side_can_research From: Lincoln Peters To: Eric McDonald Cc: Xconq list In-Reply-To: <414266B6.3030106@phy.cmich.edu> References: <1094789127.4338.103479.camel@localhost> <414266B6.3030106@phy.cmich.edu> Content-Type: text/plain Message-Id: <1094879273.28085.18040.camel@localhost> Mime-Version: 1.0 Date: Sat, 11 Sep 2004 19:27:00 -0000 Content-Transfer-Encoding: 7bit X-SW-Source: 2004/txt/msg01153.txt.bz2 On Fri, 2004-09-10 at 19:45, Eric McDonald wrote: > Lincoln Peters wrote: > > > When I re-wrote ai_plan_research, I had assumed that the > > side_can_research function would return false for advances that have > > already been researched. However, having run advances.g several times > > That should be the case, and seems to do exactly that in the places it > is used in the new side research code in the SDL UI. I am surprised that > it is causing you problems. So maybe there's a logic error in the new ai_plan_research function that produces an effect comparable to a logic error in side_can_research? It's certainly possible. > > > Would it be reasonable to modify side_can_research so that it returns > > false for advances that the side already has? Or would it be better to > > place the necessary code in ai_plan_research instead? > > Actually, there is a function to update the research vector and it gets > called whenever a topic is actually researched in the kernel run code. I'm a bit unclear on this. Do you mean that it selects a new advance to research at the beginning of every turn? I'd say that such behavior is rather inefficient. --- Lincoln Peters There's a way out of any cage. -- Captain Christopher Pike, "The Menagerie" ("The Cage"), stardate unknown.