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

AStar runtimes #13307

Closed
Probe1 opened this issue Jan 8, 2017 · 5 comments
Closed

AStar runtimes #13307

Probe1 opened this issue Jan 8, 2017 · 5 comments
Labels
Runtime Log Something causing a runtime in this issue / This PR fixes a runtime

Comments

@Probe1
Copy link
Member

Probe1 commented Jan 8, 2017

[19:35:46] Runtime in AStar.dm, line 195: cannot append to list


proc name: AStar (/proc/AStar)
src: null
call stack:
AStar(the floor (242,241,1) (/turf/simulated/floor), the floor (227,254,1) (/turf/simulated/floor), /turf/proc/CardinalTurfsWithAc... (/turf/proc/CardinalTurfsWithAccess), /turf/proc/Distance_cardinal (/turf/proc/Distance_cardinal), 0, 120, null, null, the identification card (/obj/item/weapon/card/id), null)
Officer Beepsky (/obj/machinery/bot/secbot/beepsky): calc path(null)
Officer Beepsky (/obj/machinery/bot/secbot/beepsky): process()

[19:30:39] Runtime in AStar.dm, line 195: cannot append to list


proc name: AStar (/proc/AStar)
src: null
call stack:
AStar(the floor (281,257,1) (/turf/simulated/floor/wood), the floor (242,254,1) (/turf/simulated/floor), /turf/proc/CardinalTurfsWithAc... (/turf/proc/CardinalTurfsWithAccess), /turf/proc/Distance (/turf/proc/Distance), 0, 120, null, null, the identification card (/obj/item/weapon/card/id), null)
Buddy (/obj/machinery/bot/cleanbot): process()

#13301
@Iamgoofball

(This kills the mob pathing)

@Probe1 Probe1 added the Runtime Log Something causing a runtime in this issue / This PR fixes a runtime label Jan 8, 2017
@Iamgoofball
Copy link
Contributor

huh, that's weird we haven't gotten that before

@Probe1
Copy link
Member Author

Probe1 commented Jan 8, 2017

Look at our AStar.dm file and if it's worth the effort, or put up a revert (or I will in a few hours)

@Iamgoofball
Copy link
Contributor

Iamgoofball commented Jan 8, 2017

i'd just port heaps and our current A* at tg entirely imo, the time spent will be well worth the effort lag-fix wise

@Probe1
Copy link
Member Author

Probe1 commented Jan 8, 2017

Do it.

@Probe1
Copy link
Member Author

Probe1 commented Jan 8, 2017


Infinite loop suspected--switching proc to background.
If it is not an infinite loop, either do 'set background=1' or set world.loop_checks=0.
proc name: AStar (/proc/AStar)
  source file: AStar.dm,195
  usr: Oxuzi Opusa (/mob/living/carbon/human)
  src: null
  usr.loc: the floor (215,253,1) (/turf/simulated/floor)
  call stack:
AStar(the floor (256,243,1) (/turf/simulated/floor), the floor (196,231,1) (/turf/simulated/floor), /turf/proc/CardinalTurfsWithAc... (/turf/proc/CardinalTurfsWithAccess), /turf/proc/Distance_cardinal (/turf/proc/Distance_cardinal), 0, 250, null, null, the identification card (/obj/item/weapon/card/id), null)
the Mulebot (#1) (/obj/machinery/bot/mulebot): calc path(null)
the Mulebot (#1) (/obj/machinery/bot/mulebot): receive signal(/datum/signal (/datum/signal), 1, 1400)
/datum/radio_frequency (/datum/radio_frequency): post signal(the navigation beacon (/obj/machinery/navbeacon), /datum/signal (/datum/signal), "5", null)
the navigation beacon (/obj/machinery/navbeacon): post signal()
the navigation beacon (/obj/machinery/navbeacon): receive signal(/datum/signal (/datum/signal), 1, 1400)
[16:33:03] Runtime in AStar.dm,195: cannot append to list
  proc name: AStar (/proc/AStar)
  usr: Oxuzi Opusa (abumblingidiot) (/mob/living/carbon/human)
  usr.loc: The floor (211, 253, 1) (/turf/simulated/floor)
  src: null
  call stack:
  AStar(the floor (256,243,1) (/turf/simulated/floor), the floor (196,231,1) (/turf/simulated/floor), /turf/proc/CardinalTurfsWithAc... (/turf/proc/CardinalTurfsWithAccess), /turf/proc/Distance_cardinal (/turf/proc/Distance_cardinal), 0, 250, null, null, the identification card (/obj/item/weapon/card/id), null)
  the Mulebot (#1) (/obj/machinery/bot/mulebot): calc path(null)
  the Mulebot (#1) (/obj/machinery/bot/mulebot): receive signal(/datum/signal (/datum/signal), 1, 1400)
  /datum/radio_frequency (/datum/radio_frequency): post signal(the navigation beacon (/obj/machinery/navbeacon), /datum/signal (/datum/signal), "5", null)
  the navigation beacon (/obj/machinery/navbeacon): post signal()
  the navigation beacon (/obj/machinery/navbeacon): receive signal(/datum/signal (/datum/signal), 1, 1400)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Runtime Log Something causing a runtime in this issue / This PR fixes a runtime
Projects
None yet
Development

No branches or pull requests

2 participants