Skip to content

#122/Fixed Python test script environment calling#124

Open
vkatari10 wants to merge 1 commit intomainfrom
#122/docs
Open

#122/Fixed Python test script environment calling#124
vkatari10 wants to merge 1 commit intomainfrom
#122/docs

Conversation

@vkatari10
Copy link

Summary

Setup of tests/Python on my local machine. Made slight modifications to some testing scripts, and added some comments to explain where port changes were made.

Related Issues / Tasks

PR 1 for #122

Type of Change

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

Affected Modules / Components

Directory tests/Python : https://github.com/ORNL-Real-Sim/FIXS/tree/main/tests/Python

Test Cases

Already implemented test scripts in the tests/Python sub module

Environment

Only fill in the tools used for testing this PR:

  • Python version: 3.10
  • MATLAB/Simulink/dSPACE version:
  • SUMO version: 1.26
  • VISSIM version:
  • IPG CarMaker version:

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)

Still need to figure out why SimpleTrafficLight test fails to run independently without running the other SimpleEchoClient test script first

Other notes I took:

  • When running the simple traffic client it fails due to a port connection issue
  • However running the simple echo client first and then running the simple traffic client works
  • So most likely an issue with the script with the simple traffic client .bat file, or a configuration issue with the port
  • Sumo is stuck on trying to load the .cfg file
  • Changing the port on the config file inside the folder to 1337 allows SUMO to start with the simulation but the command prompt does not show up and the entire process freezes after
  • Simple echo client test .bat script has the wrong anaconda environment call, but can still run anyways, I fixed it so that it calls the right conda environment

@vkatari10 vkatari10 changed the title Fixed Python test script environment calling #122/Fixed Python test script environment calling Mar 3, 2026
@yunlishao
Copy link
Contributor

@vkatari10 The issues is that there is a hardcoded path of TrafficLayer in the SimpleTrafficLight/run_test.bat. Please revise it to be consistent with SimpleEchoClient. Also, please update the root/environment.yaml to have consistent environment name and add any missing packages there. Thanks

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please do not change anything on this .yaml file

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.

2 participants