Diplomatic_CenterAgricultural_Station Not Found Fix

Find any bugs in Vega Strike? See if someone has already found it, or report them here!
Post Reply
whraven
Explorer
Explorer
Posts: 8
Joined: Fri Mar 20, 2009 7:21 pm
Location: Edgerton, WI, USA
Contact:

Diplomatic_CenterAgricultural_Station Not Found Fix

Post by whraven »

After hitting this problem, also reported in another thread (http://vegastrike.sourceforge.net/forum ... =2&t=10428), in several systems, I dug through the code and found a missing comma in units/faction_ships.py. Patch is attached, but faction_ships.pyc will need to be recompiled. I do have a version recompiled with python 2.4 (yes, I need to update my system) if anyone needs it.

I do believe, due to some indexing issues that it may break existing save-files, but I haven't tested it with an old save-file.
You do not have the required permissions to view the files attached to this post.
loki1950
The Shepherd
Posts: 5841
Joined: Fri May 13, 2005 8:37 pm
Location: Ottawa
Contact:

Re: Diplomatic_CenterAgricultural_Station Not Found Fix

Post by loki1950 »

So it wasn't a missing comma in units.csv this time but a misplaced one in the Python this time.

Enjoy the Choice :)
my box::HP Envy i5-6400 @2Q70GHzx4 8 Gb ram/1 Tb(Win10 64)/3 Tb Mint 19.2/GTX745 4Gb acer S243HL K222HQL
Q8200/Asus P5QDLX/8 Gb ram/WD 2Tb 2-500 G HD/GF GT640 2Gb Mint 17.3 64 bit Win 10 32 bit acer and Lenovo ideapad 320-15ARB Win 10/Mint 19.2
ace123
Lead Network Developer
Lead Network Developer
Posts: 2560
Joined: Sun Jan 12, 2003 9:13 am
Location: Palo Alto CA
Contact:

Re: Diplomatic_CenterAgricultural_Station Not Found Fix

Post by ace123 »

I committed your change. Good catch--in fact I didn't realize that python let you do string concatenation like that.
whatch1
Bounty Hunter
Bounty Hunter
Posts: 182
Joined: Sat Jul 21, 2007 5:14 am
Location: Western MA

Re: Diplomatic_CenterAgricultural_Station Not Found Fix

Post by whatch1 »

Close but no Cigar

Just tested the fix

unless 2 agracultural stations named "Agracultural station Agracultural station" with no Diplomatic Center was the desired result.
Machine: P4 prescott 3.2 Ghz, 2Gb ddr ram, 2 80g seagate sata drives in raiad stripe config nvidia 6600 gt oc card sidwinder pro FF2
whraven
Explorer
Explorer
Posts: 8
Joined: Fri Mar 20, 2009 7:21 pm
Location: Edgerton, WI, USA
Contact:

Re: Diplomatic_CenterAgricultural_Station Not Found Fix

Post by whraven »

whatch1 wrote:2 agracultural stations named "Agracultural station Agracultural station" with no Diplomatic Center was the desired result.
Is this with an old save file? I believe the fix will cause an index conflict with old save files. I've tried it with a brand new game without seeing the problem you mentioned so far.
whatch1
Bounty Hunter
Bounty Hunter
Posts: 182
Joined: Sat Jul 21, 2007 5:14 am
Location: Western MA

Re: Diplomatic_CenterAgricultural_Station Not Found Fix

Post by whatch1 »

I was afraid that would if i used the my old save file

i was looking for a fix saw the your post, brought down the latest svn with your fix and started a fresh campain that is when i saw the "agracurlural station agracurlural station" at two different places in the system.


Bill
Machine: P4 prescott 3.2 Ghz, 2Gb ddr ram, 2 80g seagate sata drives in raiad stripe config nvidia 6600 gt oc card sidwinder pro FF2
loki1950
The Shepherd
Posts: 5841
Joined: Fri May 13, 2005 8:37 pm
Location: Ottawa
Contact:

Re: Diplomatic_CenterAgricultural_Station Not Found Fix

Post by loki1950 »

Did you also clean out all the stuff in the .vegastrike folder before starting that new game that's something that i have been doing for a couple of years now whenever i test a new change as the generated files kept there can conflict with the new data or code.

Enjoy the Choice :)
my box::HP Envy i5-6400 @2Q70GHzx4 8 Gb ram/1 Tb(Win10 64)/3 Tb Mint 19.2/GTX745 4Gb acer S243HL K222HQL
Q8200/Asus P5QDLX/8 Gb ram/WD 2Tb 2-500 G HD/GF GT640 2Gb Mint 17.3 64 bit Win 10 32 bit acer and Lenovo ideapad 320-15ARB Win 10/Mint 19.2
whraven
Explorer
Explorer
Posts: 8
Joined: Fri Mar 20, 2009 7:21 pm
Location: Edgerton, WI, USA
Contact:

Re: Diplomatic_CenterAgricultural_Station Not Found Fix

Post by whraven »

loki1950 wrote:Did you also clean out all the stuff in the .vegastrike folder before starting that new game ...
Wouldn't surprise me if that was part of the problem. There is a lot of generated stuff in the sectors and serialized.xml folder that have referents to outside data structures. When those structures change, the referents break.

Now, I'm not sure this is the case. I just haven't seen it yet with a fully clean .vegastrike folder after the change. If I do, I'll certainly try to track it down.
Post Reply