News:

Congratulations!
 You've won the News Item Lottery! Your prize? Reading this news item! :)

Tutorial scenario reports script errors

Started by Matthew, April 02, 2024, 07:19:57 PM

Previous topic - Next topic

0 Members and 1 Guest are viewing this topic.

Yona-TYT

#35
I had forgotten to add elsewhere, there should no longer be non-integral values in the arrays.

https://github.com/simutrans/pak128/commit/9fe7195674f5e47898b09be6ce69be1405e6848c


I'm making minor modifications from my phone so I can't test well.

Edit.
I had to fix the logic.

Yona-TYT

#36
Well, actually something curious was happening, the logic I had worked with the official versions and not with the nightly versions, but never with both at the same time.  :o

Now I have improved that logic.  ;)

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


Thanks for your help with the tests. ;D

Please try again and verify with official release and night

Roboron

It works now with both version :-)

But will it work when the version is increased again?

Yona-TYT

#38
Quote from: Roboron on June 04, 2024, 12:50:28 PMIt works now with both version :-)

But will it work when the version is increased again?
Yes, in fact it already worked before, but I made a modification so that it worked with the nightly versions but clumsily that messed up the verification for official versions.


I had a really bad time fixing this, 😞 but I think it should work fine now.


Edit.
Quote from: Roboron on June 04, 2024, 12:50:28 PMBut will it work when the version is increased again?
Maybe an experiment with making compilations with modified versions can clarify those doubts


prissi

The pak64 tutorial is working less than ever. Now, after clicking "this is a >>link" (just ">>link" would work much better in most languages) I am asked to build a railway for wood. But I should click on "{pos2}"

Please give a zip of a working tutorial, as the one on your github does not work. (I downloaded https://github.com/Yona-TYT/tutorial_pak64/archive/refs/heads/main.zip)

Yona-TYT

#41
Quote from: prissi on June 23, 2024, 09:07:55 AMI am asked to build a railway for wood. But I should click on "{pos2}"
The wood train is working well, I have had no problems getting to this point.

Could it be that you are using an old version of the tutorial?.

try here: tutorial_pak64.zip

Captura desde 2024-06-24 08-54-42.png



Quote from: prissi on June 23, 2024, 09:07:55 AM(just ">>link" would work much better in most languages)

Good idea, I think this can be changed from the translator.  8)

prissi

Then please state where to find the most recent version of the tutorial.  https://github.com/Yona-TYT/tutorial_pak64 main does not work. Have I to use a branch?

Yona-TYT

I use the repo from the simutrans organization on Github, I don't know why the main one doesn't work, I don't know what I should do, I would like to leave only the repo from the simutrans organization on Github.

prissi

OK, somehow some files were missing from the copy. pak64 should work now.

Yona-TYT

Quote from: prissi on June 25, 2024, 07:03:25 AMOK, somehow some files were missing from the copy. pak64 should work now.
It's a relief to read that. :)

Yona-TYT

This is a proof of concept or a rough draft of how the menus in chapter 2 of the tutorial would look using the new hide menus feature.

Please let me know what you think about it.

tutorial_pak64-test2.zip
Captura desde 2024-06-30 19-09-00.png


Yona-TYT

#48
Quote from: Roboron on September 25, 2024, 10:12:13 AMAnother bug of the scenario was reported on Steam https://steamcommunity.com/app/434520/discussions/0/4843148117772753026/
It's not a code error, it's just that in the Steam version the texts are not updated. Remember that there is a new action which is "Follow the Vehicle".
Edit:

Captura de pantalla -2024-10-07 08-45-00.png

Yona-TYT

Tutorial for pak128 has also been updated to show disabled tools, following @prissi's recommendations to not remove tools from menus.

tutorial_pak128-v1.6.70.zip

Captura de pantalla -2024-10-10 12-20-08.png

Andarix

chapter_02/goal_step_01.txt

The km/h specifications no longer match those in pak64 124.2.2.

At the time of the tutorial there is no 50 km/h road.

The marking of the streets before the construction of the depot also does not seem to be working correctly.

Andarix

class_chapter_03.nut

line 66
  c_way3 = {a = coord3d(106,158,-1), b = coord3d(96,151,1), dir = 6}  //Inicio, Fin de la via y direccion(fullway)

The field at a should be 102,158,0.

106,158,-1 is the starting square for the construction of the bridge.

line 58
  c_brge1 = {b = coord(106,158), a = coord(103,158)}

Yona-TYT

Quote from: Andarix on October 14, 2024, 10:51:10 PMclass_chapter_03.nut

line 66
  c_way3 = {a = coord3d(106,158,-1), b = coord3d(96,151,1), dir = 6}  //Inicio, Fin de la via y direccion(fullway)

The field at a should be 102,158,0.

106,158,-1 is the starting square for the construction of the bridge.

line 58
  c_brge1 = {b = coord(106,158), a = coord(103,158)}

Yes, that was because I had a lot of problems with that bridge, even @Prissi broke the script right there, there was always the possibility that the rails would not connect to the bridge.

However, that is in the past and I think changing the coordinates should not cause problems:
https://github.com/simutrans/tutorial_pak64/commit/dca6f09c3f928a40510f49f6d8a9e80145deffc6

Yona-TYT

Quote from: Andarix on October 12, 2024, 08:20:53 PMchapter_02/goal_step_01.txt

The km/h specifications no longer match those in pak64 124.2.2.

At the time of the tutorial there is no 50 km/h road.

Is "25 km/h to 130 km/h" ok? I could also get the values dynamically but I'm too lazy  ;D .



Quote from: Andarix on October 12, 2024, 08:20:53 PMThe marking of the streets before the construction of the depot also does not seem to be working correctly.
Are you referring to the orange markings? I think it has to do with the flashing delay. Is there a way to create delays in milliseconds?, That would be great 8) .

Andarix

#54
Quote from: Yona-TYT on October 14, 2024, 11:53:38 PM...
Are you referring to the orange markings? I think it has to do with the flashing delay. Is there a way to create delays in milliseconds?, That would be great 8) .

The sidewalk covers the marking at the edge of the field.



Quote from: Yona-TYT on October 14, 2024, 11:53:38 PMIs "25 km/h to 130 km/h" ok? I could also get the values dynamically but I'm too lazy  ;D .
...

I would generally avoid such fixed information, especially given that the paksets different.

the following should be sufficient
QuoteThere are different roads with different maximum speeds.

Andarix

chapter_03/08_4-5.txt

The change of level in the tunnel directly at the portal is somewhat unfavorable.

map 180° rotate
Screenshot 2024-10-15 235726.jpg

Yona-TYT

#56
Quote from: Andarix on October 15, 2024, 10:03:56 PMchapter_03/08_4-5.txt

The change of level in the tunnel directly at the portal is somewhat unfavorable.

map 180° rotate
Screenshot 2024-10-15 235726.jpg
The solution was simple, just decrement 1 value in each coord x of the list, try again :) :

https://github.com/simutrans/tutorial_pak64/commit/defbb7a31c30c183b72c380b0e5e24b99998685b

Edi.

Quote from: Andarix on October 15, 2024, 06:27:40 AMI would generally avoid such fixed information, especially given that the paksets different.

Also updated the text about roads/tracks:

https://github.com/simutrans/tutorial_pak64/commit/6e30f1eb4e2ae8ab2690dda3ae523043dde4c27f

Andarix

124.2.2 r11400 and 124.2.3 r11422 - pak64 tutorial from Git repo

Screenshot 2024-10-16 110955.jpg


Andarix

#59
chapter_03/08_4-5.txt

When raising the track to level 7 the text changes incorrectly. It is possible that this is a consequence of yesterday's field change.

Screenshot 2024-10-16 153551.jpg



chapter_03/08_5-5.txt

mmh, you're supposed to connect the tunnel entrance to yourself?

Screenshot 2024-10-16 154837.jpg

Yona-TYT

Quote from: Andarix on October 16, 2024, 01:43:27 PMchapter_03/08_4-5.txt

When raising the track to level 7 the text changes incorrectly. It is possible that this is a consequence of yesterday's field change.

Screenshot 2024-10-16 153551.jpg



chapter_03/08_5-5.txt

mmh, you're supposed to connect the tunnel entrance to yourself?

Screenshot 2024-10-16 154837.jpg

Is fix here:
https://github.com/simutrans/tutorial_pak64/commit/392a1bfffcbabf313230697931696473b1bb2b3d

Andarix

#61
chapter_03/goal_step_11.txt

The specified stopping fields (same fields there and back) are not suitable for efficient train operation, since the trains will make a full circuit through the signals for each stop.

Screenshot 2024-10-17 160405.jpg



Another mistake is clicking on copy. It only needs to be done twice to get 3 moves and not 3 times as stated.

I assumed that the Spanish texts were the most current. However, this does not seem to be the case as the English texts differ.

Yona-TYT

Quote from: Andarix on October 17, 2024, 02:08:51 PMchapter_03/goal_step_11.txt

The specified stopping fields (same fields there and back) are not suitable for efficient train operation, since the trains will make a full circuit through the signals for each stop.

Screenshot 2024-10-17 160405.jpg
You say I should remove stations 6, 7 and 8 from the schedule?.

Quote from: Andarix on October 17, 2024, 02:08:51 PMAnother mistake is clicking on copy. It only needs to be done twice to get 3 moves and not 3 times as stated.

Screenshot 2024-10-17 at 12-32-48 SimuTranslator - Editar traducción del objeto.png
https://github.com/simutrans/tutorial_pak64/commit/8729b22edec750e3aacf4fa35c4bac25c03181cb

Quote from: Andarix on October 17, 2024, 02:08:51 PMI assumed that the Spanish texts were the most current. However, this does not seem to be the case as the English texts differ.
Unfortunately the texts in pak64 tutorial have not been reviewed too much, I think you are one of the first to do a thorough review.  ;)



Andarix

Quote from: Yona-TYT on October 17, 2024, 04:37:04 PMYou say I should remove stations 6, 7 and 8 from the schedule?.
...

No.

You enter coordinates in the list. If you click on these, a message appears on the way back saying that it should be a different field.

I would find it less confusing if the list contained the coordinates for the stopping fields.

Yona-TYT

Quote from: Andarix on October 17, 2024, 04:49:05 PMNo.

You enter coordinates in the list. If you click on these, a message appears on the way back saying that it should be a different field.

I would find it less confusing if the list contained the coordinates for the stopping fields.
It is fix here:
https://github.com/simutrans/tutorial_pak64/commit/daa21789ee5d781106ec54935aa08c988482ccd0
https://github.com/simutrans/tutorial_pak64/commit/0685ad1c311c42b6f3e7381f643dbf55ae0f5bf8

Andarix

tool_alias not translated in pak64 tutorial

pak64 scenario.nut line 87
// placeholder for tools names in simutrans
tool_alias  <- {inspe = "Abfrage", road= "ROADTOOLS", rail = "RAILTOOLS", ship = "SHIPTOOLS", land = "SLOPETOOLS", spec = "SPECIALTOOLS"}


pak128 scenario.nut line 120
// placeholder for tools names in simutrans
tool_alias  <- {inspe = translate("Abfrage"), road= translate("ROADTOOLS"), rail = translate("RAILTOOLS"), ship = translate("SHIPTOOLS"), land = translate("SLOPETOOLS"), spec = translate("SPECIALTOOLS")}

translate remove here
https://github.com/simutrans/tutorial_pak64/commit/e78f814ab481b249f21648c416b7645be1b8ab40#diff-3568f64b9cd2f0a269e9320c00f268f721bf2dc5f7ba8c8514b3060c6b1f2df7

Yona-TYT

Quote from: Andarix on October 19, 2024, 05:49:56 AMtool_alias not translated in pak64 tutorial

pak64 scenario.nut line 87
// placeholder for tools names in simutrans
tool_alias  <- {inspe = "Abfrage", road= "ROADTOOLS", rail = "RAILTOOLS", ship = "SHIPTOOLS", land = "SLOPETOOLS", spec = "SPECIALTOOLS"}


pak128 scenario.nut line 120
// placeholder for tools names in simutrans
tool_alias  <- {inspe = translate("Abfrage"), road= translate("ROADTOOLS"), rail = translate("RAILTOOLS"), ship = translate("SHIPTOOLS"), land = translate("SLOPETOOLS"), spec = translate("SPECIALTOOLS")}

translate remove here
https://github.com/simutrans/tutorial_pak64/commit/e78f814ab481b249f21648c416b7645be1b8ab40#diff-3568f64b9cd2f0a269e9320c00f268f721bf2dc5f7ba8c8514b3060c6b1f2df7
There is a problem with the translations of those names that I could not solve, for some reason when changing the language the text is not translated.

Curiously, the table for the names of load types is being translated correctly.

I must investigate this problem further.

Edit.
Well it seems that I only have to translate them when the text is processed.
text.tool1 = translate(tool_alias.inspe)
text.tool2 = translate(tool_alias.road)
text.tool3 = translate(tool_alias.spec)

Andarix

Quote from: Yona-TYT on October 19, 2024, 06:15:34 AMThere is a problem with the translations of those names that I could not solve, for some reason when changing the language the text is not translated.
...

This is probably because this file is only read at startup.

I assume that the language will rarely be changed during the tutorial.

Yona-TYT


Andarix

chapter_03/02_1-3.txt

I think the field position should be 102,168,2 and not 107,158.

Screenshot 2024-10-19 124435.jpg

[edit]

Sorry, this is a text change by me.

<p>
Bauen Sie das Bahngleis von {cbor} nach {w2}.
</p>