LuaTrain.go_to_station()

[Reika]

Exactly what it looks like - a function to call on a LuaTrain to send it to a given station.

[quyxkh]

Does

Code: Select all

/c _=game.player.selected.train _.schedule={current=1,records={{station='NE Iron Plate'}}} _.manual_mode=false

do what you want?

[Reika]

Does

Code: Select all

/c _=game.player.selected.train _.schedule={current=1,records={{station='NE Iron Plate'}}} _.manual_mode=false

do what you want?

No, because I am not interested in deleting the train's normal schedule.

[Bilka]

Okay, added for 0.17.

[Fractaliste]

Okay, added for 0.17.

Will the new station appears into the ScheduleRecords array?

[Bilka]

Will the new station appears into the ScheduleRecords array?

See, https://lua-api.factorio.com/0.17.0-pre ... to_station - the station is given by a schedule index, so it must already be in the schedule.

[Optera]

Isn't this implementation the same as setting the current schedule record?

e.g. if you know you want to send it to the 2nd station in the schedule records:
train.schedule.current = 2

[Bilka]

Isn't this implementation the same as setting the current schedule record?

e.g. if you know you want to send it to the 2nd station in the schedule records:
train.schedule.current = 2

Yes but it's on the C++ side so you don't have to do what the post further up is doing: copying the entire schedule, setting automatic mode etc.

[Optera]

This function would be really sleek if it took a backer_name as parameter.
Set the index to the corresponding entry or add a new entry for it, potentially with additional parameter taking an list of WaitCondition in case a new entry is generated.


Just for setting current and mode it feels redundant.
I can't think of a use case where you didn't have to find the stop name from the schedule in lua aka read the whole train.schedule.
Already having the schedule in a local table changing current and mode and writing it back to train is simpler to do right there. Even more so if you have to add the desired station in the first place e.g. for something like a player shuttle.

[Fractaliste]

This function would be really sleek if it took a backer_name as parameter.
Set the index to the corresponding entry or add a new entry for it, potentially with additional parameter taking an list of WaitCondition in case a new entry is generated.

Is was this way I initially understand the new LuaTrain.go_to_station() functionnality.

[Reika]

This function would be really sleek if it took a backer_name as parameter.
Set the index to the corresponding entry or add a new entry for it, potentially with additional parameter taking an list of WaitCondition in case a new entry is generated.

Is was this way I initially understand the new LuaTrain.go_to_station() functionnality.

And it was how I intended it.

[Bilka]

Please make a new request and describe exactly what you want.

[Reika]

Please make a new request and describe exactly what you want.

Done. viewtopic.php?f=28&t=71679