Tutorial scenario reports script errors

[Matthew]

Steps to reproduce

1. Install Simutrans with pak128.
2. Load the tutorial scenario.
3. Note that all the tabs of the scenario briefing only contain the word "(null)"
4. Click on a building.

Expected result

The building's Info Window opens.

Actual result

The building's Info Window opens and so does an error window reporting "An error occurred within a script!"

Animated example



Setup
Simutrans 123.0.1 r10421
Steam
Kubuntu 22.04

Notes

I was alerted to this problem by this Reddit post from a new player.

I suspect that the tutorial is supposed to start with some kind of instruction, but it's so long since I've opened it that I'm not certain.

I don't know whether this bug is specific to the Steam version or not; I just happened to have the Steam version installed.

P.S. Nightly not affected?

I tried to reproduce this using the Nightly version provided via Steam, r11101. I could not, but I got a message saying "This Scenario is not compatible with your pakset or version of simutrans."

[Yona-TYT]

I'm afraid you are using a very old simutras revision, according to the title bar it is using "r10421".

[Yona-TYT]

I tried to reproduce this using the Nightly version provided via Steam, r11101. I could not, but I got a message saying "This Scenario is not compatible with your pakset or version of simutrans."

I have a concern with the version comparator, it is possible that it may crash on steam or future versions.


I don't know how to test since I would only know when trying a new version of simutrans.

[Matthew]

I'm afraid you are using a very old simutras revision, according to the title bar it is using "r10421".

This is 123.0.1, which is the latest official release.

According to the scenario.nut file, Steam is distributing tutorial version 1.6.20.

Since they are both older versions, they should work together, shouldn't they?

[Yona-TYT]

This is 123.0.1, which is the latest official release.

According to the scenario.nut file, Steam is distributing tutorial version 1.6.20.

Since they are both older versions, they should work together, shouldn't they?

Yes they should, but if the version comparison says that it is not compatible, it may be that in that version there was case sensitivity in the version names and pak128 changed to Pak128.

[Matthew]

I tried to play the tutorial on the new Simutrans 124 r11164 via Steam, running pak128 with tutorial version 1.6.51 beta, but I was unsuccessful:



This is on Linux.

[Andarix]

You can change the version in pak128/scenario/tutorial/scenario.nut line 116.

The tutorial in pak64 also contains version 123.0.1 instead of 124.0.

pak192.comic -> 122.0.1

[Roboron]

Is it really necessary to prevent the tutorial from loading if the game version is more recent?

[Yona-TYT]

The expected behavior should be that all versions higher than that are loaded, for some reason this is not fulfilled

I'm going to check, this was really what I was afraid of.

Edit

Unfortunately there was a problem in the text analyzer that determines if the version is correct or not, this is a basic function present in the other tutorials so I had to update all of them.

I feel very sorry for this, since until they update the script in the paksets the tutorial will not work.

https://github.com/simutrans/tutorial_pak64/commit/04859d2b59d8cfed7d3583f91b48633ddecccc6c

[Andarix]

In many cases, users can also replace one file. Make the file available and indicate where it needs to be saved.

If this file is replaced in the zips of the package sets as soon as possible, that would be an advantage. The package sets do not have to be recreated for this.

Then a reinstallation is enough to fix it.

[Roboron]

I have updated the tutorial scenarios on Steam.

But this is probably worth a small bugfix release for those paksets.

Nevertheless, players getting paksets through the installer will still install the version released with Simutrans :-(

It would be cool if the installer actually fetched a remote pakset list, so you always install the latest version of a pakset without the need for us to update Simutrans as well. But that's a development for another day.

[Roboron]

A bug was reported on the steam forums https://steamcommunity.com/app/434520/discussions/0/4357872562129203939/

[Yona-TYT]

A bug was reported on the steam forums https://steamcommunity.com/app/434520/discussions/0/4357872562129203939/

It would be great if a savegame was provided to replicate the problem easily.

[Matthew]

Roboron, thank you for fixing this so quickly.

I have never seen the tutorial before. It's really great to see how cleverly it detects the learner's successes and failures. I have played games from big studios where the tutorial could not do that. Thank you to Dwachs (for the scripting system), Yona (for the tutorial), and anyone else responsible. I think it is a great help to new players, especially on Steam.

[Yona-TYT]

A bug was reported on the steam forums https://steamcommunity.com/app/434520/discussions/0/4357872562129203939/

I have analyzed the situation and from what it says it seems that the [5/5] section was not displayed. I've already fixed it, I didn't find anything else out of the ordinary.

https://github.com/simutrans/pak128/commit/fcaa31b0434ff870aec0f28479b5ede48ff5eeb9

So far there have only been minor details with the texts, I thank the commands with which I do something similar to automated tests hehehe.

[Roboron]

Fantastic, I've updated the tutorial on Steam once again to include the recent fixes (for pak128 only).

[Yona-TYT]

In underground constructions, when the tunnel entrance is placed, it should show:

Code Select Expand

translate("Press [Ctrl] to build a tunnel entrance here")+" ("+start_tunn.tostring()+".)"
However this stopped working due to some changes in the simutrans code and the way the tunnels are built.


In order to get this working again I'm going to need a new function that lets me know if the [Ctrl] key is pressed.

[Andarix]

pak64 - 124.0

The class_chapter_07.nut file is different between pak64 SVN and Release Zip file.

[Yona-TYT]

pak64 - 124.0

The class_chapter_07.nut file is different between pak64 SVN and Release Zip file.

This is strange, but I'm sure Prissi is now using the version directly from github, so there should be no problems next time.

[prissi]

The language update script now also updates the tutorial of pak64.

[Yona-TYT]

I have updated the tutorial scenarios on Steam.

But this is probably worth a small bugfix release for those paksets.

Nevertheless, players getting paksets through the installer will still install the version released with Simutrans :-(

It would be cool if the installer actually fetched a remote pakset list, so you always install the latest version of a pakset without the need for us to update Simutrans as well. But that's a development for another day.

I completely agree with that idea, in fact supertux already does it like that! (maybe it's a good idea to look at their code for inspiration), let's hope that's the way forward for simutrans too.

[Yona-TYT]

You can change the version in pak128/scenario/tutorial/scenario.nut line 116.

The tutorial in pak64 also contains version 123.0.1 instead of 124.0.

pak192.comic -> 122.0.1

Today I just sent a significant improvement in the function that analyzes simutrans versions, it is much more robust and I hope it will remain functional for many years (unless the version numbering style in simutrans changes).

I have carried out various tests playing with version numbers and simulating all possible cases, but if anyone wants to help with more tests, welcome.

pak64: https://github.com/simutrans/tutorial_pak64/commit/250781ec803cf751419a51b69a197bf2fe734839
pak128: https://github.com/simutrans/pak128/commit/45fba552488b43f72d8ec21ce0672a0ac0db796d
pak192.comic: https://github.com/simutrans/tutorial_pak192.comic/commit/7cd172c5cc92f3b126a3a06a268f9472a543184f

[Roboron]

Another error (and general feedback) https://steamcommunity.com/app/434520/discussions/0/4358999171570733692/?tscn=1715609064

[Yona-TYT]

Another error (and general feedback) https://steamcommunity.com/app/434520/discussions/0/4358999171570733692/?tscn=1715609064

It seems that there are problems with the coordinates, I thought I had solved them all but it seems that at some point some change in the API broke something, I'm going to investigate.


I think it mentions more problems, but I'll start by fixing this one.

[Yona-TYT]

There is an error in the text variables in the pak128 tutorial: "chapter_03/08_3-3.txt"

Code Select Expand

{tx} Set the layer to <em>{lev}</em>, and connect the <em>tunnel</em> {t4} with the other entrance in {t5}.
To:

Code Select Expand

{tx} Set the layer to <em>{lev}</em>, and connect the <em>tunnel</em> {t1} with the other entrance in {t2}.
Edit.

I have fixed some problems with the list that shows the coordinates and layer levels, the mechanics worked wonderfully but the problem was only the visual part (the displayed texts).





Edit 2.

Chapter 6

"/scenario/tutorial/en/chapter_06/goal.txt"
"/scenario/tutorial/es/chapter_06/goal.txt"

Command Script Link is not in its place.

Code Select Expand

<p>
{scr}
</p>

Edit 3.

He also discovered one of the strangest bugs that was in the tutorial, this has to do with the coordinates obtained from the city limits.

It seems that the coordinates obtained when the map is rotated are incorrect, to solve this I have had to implement a function to know in which direction the rotation of the current map is located.

https://github.com/simutrans/pak128/commit/9a355ac764d288c363ac9854dbfc70f40a6d5725
https://github.com/simutrans/pak128/commit/41de627c410eda58fb162e4809cf1ac692358692

Code Select Expand

function my_compass()
{
local c_max = {x = map_siz.x-1, y = map_siz.y-1}
local c = coord(0,0)
local text = c.tostring()

local res_c = {x = 0, y = 0}
local ttx = ""
local siz = text.len()
for(local j=0;j<siz;j++){
local tx = format("%c",text[j])
try {
tx.tointeger()
}
catch(ev) {
if(tx==","){
res_c.x = ttx.tointeger()
ttx = ""
continue
}
continue
}
ttx+=tx
if(j == siz-1){
res_c.y = ttx.tointeger()
}
}
//gui.add_message("Res: "+ res_c.x +" -- "+res_c.y)
//gui.add_message("MAX: "+ c_max.x +" -- "+c_max.y)
if(res_c.x == 0 && res_c.y == 0){
gui.add_message("N")
return 0
}
else if(res_c.x == c_max.y && res_c.y == 0){
gui.add_message("W")
return 1
}
else if(res_c.x == c_max.x && res_c.y == c_max.y){
gui.add_message("S")
return 2
}
else if(res_c.x == 0 && res_c.y == c_max.x){
gui.add_message("E")
return 3
}
return null
}

[Yona-TYT]

Completed v1.6.60 testing for all tutorials.

Change Log:

Code Select Expand

ADD Following Convoys in Chapter 2 Step D
Minor fixes in Chapter 3
Fix Chapter 7
Fix error when get coord form city limits (Chapter 7)
Update simutrans Version and fix string_analyzer()
(Using in Pak192.comic only) New functions to get build and way names



On github you can find the most recent one.

https://github.com/simutrans/pak128/tree/master/pak128.prototype/scenario/tutorial
https://github.com/simutrans/tutorial_pak192.comic
https://github.com/simutrans/tutorial_pak64

[prissi]

SHould you mess with the toolbar on pak128: The numbers of all toolbars will change shortly, same for pak64 as part of menu restructuring.

[Roboron]

This is happening again with latest pak128 (nightly, not release)

[Yona-TYT]

Is that the Steam version? I tried it last night on Android and the version checker doesn't fail.

[Roboron]

Yes, it is.

[Yona-TYT]

Yes, it is.

Can you test if it works in the normal version of simutrans? Now I can't use the PC since there is no electricity.

I can only speculate that the version nomenclature is different on simutrans steam.

[Roboron]

Yes, it also happens with the "normal" version.

[Yona-TYT]

The stupid "0" in the version was not being recognized as integral, this seems to solve the problem.

https://github.com/simutrans/pak128/commit/48afddd11995b9edd3329bfcf1211a8abeffb417

Please check if this really solves the problem as I can't do it myself right now.

[Roboron]

[Yona-TYT]

I can't see this picture 


404 File Not Found