Skip to content

#14/streamline startup info output#114

Merged
yunlishao merged 1 commit intomainfrom
maintenance/#14
Jan 28, 2026
Merged

#14/streamline startup info output#114
yunlishao merged 1 commit intomainfrom
maintenance/#14

Conversation

@yunlishao
Copy link
Copy Markdown
Contributor

Summary

Streamlines TrafficLayer.exe startup console output by suppressing verbose default value messages and providing cleaner, more concise status information. Addresses incorrect/excessive info shown during startup.

Related Issues / Tasks

Closes #14

Type of Change

  • Bug fix
  • New feature
  • Maintenance / Refactor
  • Documentation
  • Test case / scenario update

Affected Modules / Components

  • CommonLib/ConfigHelper.cpp - Added SuppressDefaultMessages flag to control verbose output
  • CommonLib/ConfigHelper.h - Added SuppressDefaultMessages member variable
  • TrafficLayer/TrafficLayer/mainTrafficLayer.cpp - Streamlined startup messages, removed redundant VISSIM selection messages, added consolidated client ports summary

Test Cases

Manual verification of TrafficLayer.exe startup output with various config.yaml configurations.

Environment

  • Visual Studio 2022

Checklist

  • Code compiles/runs as expected
  • Tests pass locally
  • Documentation is updated (if applicable)
  • Relevant issues are linked
  • Version-specific changes are noted (if any)

Additional Notes (optional)

Changes include:

  • Default "Will use X as default" messages are now suppressed by default via SuppressDefaultMessages = true
  • TrafficLayer startup output is more concise (e.g., "Traffic Simulator: SUMO" instead of "Selected Traffic Simulator SUMO")
  • Client ports are now displayed in a single summary line instead of individual messages
  • Removed redundant VISSIM driver model selection messages

- Add SuppressDefaultMessages flag to ConfigHelper to reduce verbose default value messages
- Streamline TrafficLayer startup output with cleaner, more concise format
- Remove redundant VISSIM driver model selection messages
- Add consolidated client ports summary display
@yunlishao yunlishao requested a review from cjffly November 20, 2025 04:17
@yunlishao yunlishao merged commit 7fafc3a into main Jan 28, 2026
@yunlishao yunlishao deleted the maintenance/#14 branch January 28, 2026 17:46
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

Successfully merging this pull request may close these issues.

Incorrect info shown on TrafficLayer.exe startup

2 participants