/FreeOrion/Test/FreeOrion_2016-06-21.94bf367_Test_Win32_Setu

Describe your experience with the latest version of FreeOrion to help us improve it.
Forum rules
Always mention the exact version of FreeOrion you are testing.

When reporting an issue regarding the AI, if possible provide the relevant AI log file and a save game file that demonstrates the issue.
Post Reply
Message
Author
jadwin
Space Floater
Posts: 17
Joined: Sat Jun 11, 2016 6:35 am
Location: Heidelberg, Germany

/FreeOrion/Test/FreeOrion_2016-06-21.94bf367_Test_Win32_Setu

#1 Post by jadwin » Wed Jun 22, 2016 7:21 pm

I'm not able to start a new game or load a game: "Phyton Error: AI for AI_3 crashed".

The Number of AI changes, but every time it happens.

User avatar
Vezzra
Release Manager, Design
Posts: 4609
Joined: Wed Nov 16, 2011 12:56 pm
Location: Sol III

Re: /FreeOrion/Test/FreeOrion_2016-06-21.94bf367_Test_Win32_

#2 Post by Vezzra » Thu Jun 23, 2016 10:06 am

What is the exact Windows version you are using?

We also need the log files FO produces. You can find them in the same directory where config.xml is located, see this wiki page for more information. Please attach all log files you find in that directory to a post here in this thread.

jadwin
Space Floater
Posts: 17
Joined: Sat Jun 11, 2016 6:35 am
Location: Heidelberg, Germany

Re: /FreeOrion/Test/FreeOrion_2016-06-21.94bf367_Test_Win32_

#3 Post by jadwin » Thu Jun 23, 2016 3:02 pm

Window Version: Windows 10 Home, Version: 1511

Only three attatchement possible
Attachments
AI_3.log
(3.92 KiB) Downloaded 10 times
AI_2.log
(3.92 KiB) Downloaded 9 times
AI_1.log
(3.92 KiB) Downloaded 9 times

jadwin
Space Floater
Posts: 17
Joined: Sat Jun 11, 2016 6:35 am
Location: Heidelberg, Germany

Re: /FreeOrion/Test/FreeOrion_2016-06-21.94bf367_Test_Win32_

#4 Post by jadwin » Thu Jun 23, 2016 3:03 pm

More attachments
Attachments
AI_6.log
(3.79 KiB) Downloaded 14 times
AI_5.log
(3.92 KiB) Downloaded 12 times
AI_4.log
(3.92 KiB) Downloaded 10 times

jadwin
Space Floater
Posts: 17
Joined: Sat Jun 11, 2016 6:35 am
Location: Heidelberg, Germany

Re: /FreeOrion/Test/FreeOrion_2016-06-21.94bf367_Test_Win32_

#5 Post by jadwin » Thu Jun 23, 2016 3:05 pm

finished
Attachments
ogre.log
(17.71 KiB) Downloaded 10 times
freeoriond.log
(6.94 KiB) Downloaded 11 times
freeorion.log
(687.79 KiB) Downloaded 11 times

User avatar
Geoff the Medio
Programming, Design, Admin
Posts: 12248
Joined: Wed Oct 08, 2003 1:33 am
Location: Munich

Re: /FreeOrion/Test/FreeOrion_2016-06-21.94bf367_Test_Win32_

#6 Post by Geoff the Medio » Thu Jun 23, 2016 3:44 pm

Looks like you've got multiple different installs of FreeOrion that are somehow conflicting with eachother...

Code: Select all

2016-06-23 00:27:53.603347 [debug] AI : AI Python script path: C:\Program Files (x86)\FreeOrion2\default\python/AI
[...]
2016-06-23 00:27:53.860519 [debug] AI : Python paths ['C:\\Program Files (x86)\\FreeOrion2\\python27.zip', 'C:\\Program Files (x86)\\FreeOrion2\\DLLs', 'C:\\Program Files (x86)\\FreeOrion2\\lib', 'C:\\Program Files (x86)\\FreeOrion2\\lib\\plat-win', 'C:\\Program Files (x86)\\FreeOrion2\\lib\\lib-tk', 'C:\\Program Files (x86)\\FreeOrion2', 'C:\\Program Files (x86)\\FreeOrion2\\default/python', 'C:\\Program Files (x86)\\FreeOrion2\\default\\python/AI']
Have you tried deleting config.xml in the directory that contained the log files?

User avatar
Vezzra
Release Manager, Design
Posts: 4609
Joined: Wed Nov 16, 2011 12:56 pm
Location: Sol III

Re: /FreeOrion/Test/FreeOrion_2016-06-21.94bf367_Test_Win32_

#7 Post by Vezzra » Thu Jun 23, 2016 4:07 pm

Ummmmm... look at the rightmost path delimiter, the one between "python" and "AI":

Code: Select all

2016-06-23 00:27:53.603347 [debug] AI : AI Python script path: C:\Program Files (x86)\FreeOrion2\default\python/AI

Code: Select all

'C:\\Program Files (x86)\\FreeOrion2\\default\\python/AI'
...and the same here, between "default" and "python":

Code: Select all

'C:\\Program Files (x86)\\FreeOrion2\\default/python'
...it's a "/" instead of "\". Looks like the result of a very recent change to me, in PR#714, commit 0abeb96 by spike, in MultiplayerCommon.cpp line 40. Shouldn't that line be:

Code: Select all

db.Add<std::string>("ai-path",              UserStringNop("OPTIONS_DB_AI_FOLDER_PATH"),        PathString("python") / "AI",     Validator<std::string>(), false);
...? Just guessing...

jadwin
Space Floater
Posts: 17
Joined: Sat Jun 11, 2016 6:35 am
Location: Heidelberg, Germany

Re: /FreeOrion/Test/FreeOrion_2016-06-21.94bf367_Test_Win32_

#8 Post by jadwin » Thu Jun 23, 2016 4:18 pm

Geoff the Medio wrote:Looks like you've got multiple different installs of FreeOrion that are somehow conflicting with eachother...

Code: Select all

2016-06-23 00:27:53.603347 [debug] AI : AI Python script path: C:\Program Files (x86)\FreeOrion2\default\python/AI
[...]
2016-06-23 00:27:53.860519 [debug] AI : Python paths ['C:\\Program Files (x86)\\FreeOrion2\\python27.zip', 'C:\\Program Files (x86)\\FreeOrion2\\DLLs', 'C:\\Program Files (x86)\\FreeOrion2\\lib', 'C:\\Program Files (x86)\\FreeOrion2\\lib\\plat-win', 'C:\\Program Files (x86)\\FreeOrion2\\lib\\lib-tk', 'C:\\Program Files (x86)\\FreeOrion2', 'C:\\Program Files (x86)\\FreeOrion2\\default/python', 'C:\\Program Files (x86)\\FreeOrion2\\default\\python/AI']
Have you tried deleting config.xml in the directory that contained the log files?
Yep, the error still occurs.

jadwin
Space Floater
Posts: 17
Joined: Sat Jun 11, 2016 6:35 am
Location: Heidelberg, Germany

Re: /FreeOrion/Test/FreeOrion_2016-06-21.94bf367_Test_Win32_

#9 Post by jadwin » Thu Jun 23, 2016 4:26 pm

Vezzra wrote:Ummmmm... look at the rightmost path delimiter, the one between "python" and "AI":

Code: Select all

2016-06-23 00:27:53.603347 [debug] AI : AI Python script path: C:\Program Files (x86)\FreeOrion2\default\python/AI

Code: Select all

'C:\\Program Files (x86)\\FreeOrion2\\default\\python/AI'
...and the same here, between "default" and "python":

Code: Select all

'C:\\Program Files (x86)\\FreeOrion2\\default/python'
...it's a "/" instead of "\". Looks like the result of a very recent change to me, in PR#714, commit 0abeb96 by spike, in MultiplayerCommon.cpp line 40. Shouldn't that line be:

Code: Select all

db.Add<std::string>("ai-path",              UserStringNop("OPTIONS_DB_AI_FOLDER_PATH"),        PathString("python") / "AI",     Validator<std::string>(), false);
...? Just guessing...
I'm only a stupid user, if you say me step by step what to do, I'll try. Or if you release a new testing version, I will download and try.

User avatar
Geoff the Medio
Programming, Design, Admin
Posts: 12248
Joined: Wed Oct 08, 2003 1:33 am
Location: Munich

Re: /FreeOrion/Test/FreeOrion_2016-06-21.94bf367_Test_Win32_

#10 Post by Geoff the Medio » Thu Jun 23, 2016 4:32 pm

Vezzra wrote:...it's a "/" instead of "\".
Doesn't seem to cause a problem for me (Win 8.1)... generally / are properly interpreted by Windows-based programs. Eg. in cmd.exe

Code: Select all

C:\Users\Geoff\Desktop\FreeOrion_VS2013_SDK\FreeOrion>cd ./default

C:\Users\Geoff\Desktop\FreeOrion_VS2013_SDK\FreeOrion\default>
Shouldn't that line be:

Code: Select all

PathString("python") / "AI"
No, as PathString takes a path and converts it to string, while here you want to take a string, convert it to a path, concatenate it with another string to modify the path, then convert it back to a string to store. If only to look nicer, it might work instead to make the path canonical before storing, eg:

Code: Select all

boost::filesystem::canonical(GetResourceDir() / GetOptionsDB().Get<std::string>("ai-path")).string();
With that, I get:

Code: Select all

2016-06-23 18:32:00.979324 [debug] AI : AI Python script path: C:\Users\Geoff\Desktop\FreeOrion_VS2013_SDK\FreeOrion\default\AI

User avatar
Vezzra
Release Manager, Design
Posts: 4609
Joined: Wed Nov 16, 2011 12:56 pm
Location: Sol III

Re: /FreeOrion/Test/FreeOrion_2016-06-21.94bf367_Test_Win32_

#11 Post by Vezzra » Thu Jun 23, 2016 4:46 pm

Geoff the Medio wrote:No, as PathString takes a path and converts it to string, while here you want to take a string, convert it to a path, concatenate it with another string to modify the path, then convert it back to a string to store. If only to look nicer, it might work instead to make the path canonical before storing, eg:

Code: Select all

boost::filesystem::canonical(GetResourceDir() / GetOptionsDB().Get<std::string>("ai-path")).string();
With that, I get:

Code: Select all

2016-06-23 18:32:00.979324 [debug] AI : AI Python script path: C:\Users\Geoff\Desktop\FreeOrion_VS2013_SDK\FreeOrion\default\AI
Oh, I see... okay, worth a try...?

jadwin
Space Floater
Posts: 17
Joined: Sat Jun 11, 2016 6:35 am
Location: Heidelberg, Germany

Re: /FreeOrion/Test/FreeOrion_2016-06-21.94bf367_Test_Win32_

#12 Post by jadwin » Wed Jun 29, 2016 1:32 pm

FreeOrion_2016-06-28.438ed78_Test_Win32_Setup

works fine, thank you for fixing.

Post Reply