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.

Sign up for GitHub

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

Jump to bottom

v11 - Trace in RELEASE build #454

Closed
originalfoo opened this issue Jul 24, 2019 · 4 comments · Fixed by #455 or #499
Closed

v11 - Trace in RELEASE build #454

originalfoo opened this issue Jul 24, 2019 · 4 comments · Fixed by #455 or #499
Assignees
@kvakvs
Labels
11 ALPHA TM:PE v11 alpha edition code cleanup Refactor code, remove old code, improve maintainability
Milestone
11.0

Comments

@originalfoo
Copy link
Member

Some Trace is being logged in TMPE.log in the v11 RELEASE build (built from master branch prior to #451 merge).

Trace 185.2295256: TrafficLightSimulation.Update(): called for node 2623
Trace 186.6951208: TrafficLightSimulation.Update(): called for node 4469
Trace 186.7477688: TrafficLightSimulation.Update(): called for node 5008
Trace 186.7613966: TrafficLightSimulation.Update(): called for node 9883

Full log, savegame, etc., can be found in #452
@originalfoo originalfoo added BUG Defect detected confirmed Represents confirmed issue or bug 11 ALPHA TM:PE v11 alpha edition labels Jul 24, 2019
@originalfoo originalfoo added this to the 11.0 milestone Jul 24, 2019
@kvakvs
Copy link
Collaborator

kvakvs commented Jul 24, 2019

I'll pay Log class a visit in a couple hours

@kvakvs kvakvs mentioned this issue Jul 24, 2019
Merged
@originalfoo originalfoo added code cleanup Refactor code, remove old code, improve maintainability and removed BUG Defect detected confirmed Represents confirmed issue or bug labels Jul 24, 2019
@TLHeart60
Copy link

Still have the trace showing up in my log file. Don't know if the V11 alpha 6 has been updated to remove those... and for information, all of those traces are for traffic lights that have more than one node to control.
TMPE.log

@kvakvs
Copy link
Collaborator

kvakvs commented Aug 23, 2019

Sub-project TMPE.API has TRACE defined for Release build.

@kvakvs kvakvs reopened this Aug 23, 2019
@kvakvs kvakvs mentioned this issue Aug 23, 2019
Merged
@originalfoo
Copy link
Member Author

I'll push another v11 alpha to workshop tonight with the trace removed. Thanks @kvakvs and thanks for the report @TLHeart60 👍

@kianzarrin kianzarrin mentioned this issue May 12, 2020
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kvakvs kvakvs

Labels
11 ALPHA TM:PE v11 alpha edition code cleanup Refactor code, remove old code, improve maintainability
Projects
None yet
Milestone
11.0
3 participants
@kvakvs @originalfoo @TLHeart60