Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Multi-surface crash issue. #34

Open
credomane opened this issue Feb 27, 2020 · 0 comments
Open

Multi-surface crash issue. #34

credomane opened this issue Feb 27, 2020 · 0 comments

Comments

@credomane
Copy link

credomane commented Feb 27, 2020

To reproduce:

  1. have LTN Train Stops on multiple surfaces
  2. Open LTN Tracker GUI
  3. Click a LTN Train stop name in the LTN Tracker GUI that is NOT on the nauvis surface.
  4. Factorio display a script error about expecting surface nauvis but entity is on "insert different surface name".

Looking though the code the issue seems to be a hardcoded value of "nauvis.

local render_id = rendering.draw_circle({
color = marker_color,
radius = 3,
width = 10,
surface = "nauvis",
filled = false,
target = stop.entity,
target_offset = {-1, -1},
time_to_live = 300,
players = {pind},
})

Instead of surface = "nauvis" it should be probably be surface = stop.entity.surface

I don't know if that will fix the issue for certain but it has to at least be better than before right?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant