/FreeOrion/Test/FreeOrion_2016-06-21.94bf367_Test_Win32_Setu

Describe your experience with the latest version of FreeOrion to help us improve it.

Moderator: Oberlus

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: 32
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 »

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: 6095
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 »

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: 32
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 »

Window Version: Windows 10 Home, Version: 1511

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

jadwin
Space Floater
Posts: 32
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 »

More attachments
Attachments
AI_6.log
(3.79 KiB) Downloaded 50 times
AI_5.log
(3.92 KiB) Downloaded 43 times
AI_4.log
(3.92 KiB) Downloaded 41 times

jadwin
Space Floater
Posts: 32
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 »

finished
Attachments
ogre.log
(17.71 KiB) Downloaded 45 times
freeoriond.log
(6.94 KiB) Downloaded 50 times
freeorion.log
(687.79 KiB) Downloaded 43 times

User avatar
Geoff the Medio
Programming, Design, Admin
Posts: 13587
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 »

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: 6095
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 »

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: 32
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 »

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: 32
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 »

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: 13587
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 »

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: 6095
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 »

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: 32
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 »

FreeOrion_2016-06-28.438ed78_Test_Win32_Setup

works fine, thank you for fixing.

Post Reply