Skip to content

Dev->Main#150

Merged
nextcen-dgemoets merged 3 commits intomainfrom
development
Sep 12, 2025
Merged

Dev->Main#150
nextcen-dgemoets merged 3 commits intomainfrom
development

Conversation

@nextcen-dgemoets
Copy link
Copy Markdown
Contributor

@nextcen-dgemoets nextcen-dgemoets commented Sep 12, 2025

This deployment contains:

  • ITM-1009 and ITM 1065: Eliminate Phase 1 Config Triage Hack and Hardcoding In ADEPT TA 1 Controller
  • ITM-1067: Glob and Regex Definitions For Config Variable Assignments
  • ITM-1102: Added September 2025 configuration and tweaks

To test, update your configuration and run a variety of sessions, some connected to TA1, and some without TA1 (-t server command line switch).

Let me know if you need the September scenarios.

NeilDaniel07 and others added 3 commits August 1, 2025 16:59
…oding In ADEPT TA 1 Controller (#146)

* full functionality

* Use set for O(1) membership test in get_target_ids()

* Fixed (ancient) test scenarios so that --test session works

* Change eval_number from String to int based to benefit downstream processes.
… Multi-Purpose Utils Script (#147)

* full functionality

* glob and regex support implementation

* moved functionality to util file and completed testing:

* filepath refactoring and newline patches

* seen set optimization for filename loading

* extended support to scenario IDs

* extend support for alignment targets and increase logic abstraction

* disable alignment support in testing mode, prediction based logging, hashset optimization

* refactor config files to use glob

* use list instead of test for scenario testing

* remove Globs for Soartech config vars

* eliminate unsupported glob syntax in config templates

* consistent spacing in config files

* code style fixes

* whitespace diff fix
Copy link
Copy Markdown

@joshuale-caci joshuale-caci left a comment

Choose a reason for hiding this comment

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

Saw no issues running with the new September items and running with phase1 in test mode worked as well. Just had issues with my own phase1 adept server which I don't think is a big thing.

@nextcen-dgemoets
Copy link
Copy Markdown
Contributor Author

Saw no issues running with the new September items and running with phase1 in test mode worked as well. Just had issues with my own phase1 adept server which I don't think is a big thing.

Phase 1? The new September items are Phase 2, and so should be the ADEPT server. <confused>

@joshuale-caci
Copy link
Copy Markdown

joshuale-caci commented Sep 12, 2025

Saw no issues running with the new September items and running with phase1 in test mode worked as well. Just had issues with my own phase1 adept server which I don't think is a big thing.

Phase 1? The new September items are Phase 2, and so should be the ADEPT server.

Ahh I didn't run them together. I ran the september materials and then reverted to testing if the phase 1 stuff still worked.

Copy link
Copy Markdown
Contributor

@NeilDaniel07 NeilDaniel07 left a comment

Choose a reason for hiding this comment

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

I tested running through scenarios as well as generation of the September scenario YAML files from the CSV files. Everything look good to me. I ran the automated testing script I developed in #149 on all possible configurations defined in the updated phase 2 config.ini file besides OW (DEFAULT, SUBSET_ONLY, FULL_NO_SUBSET, MULTI_KDMA, MULTI_KDMA_SUBSET, and MULTI_KDMA_FULL_NO_SUBSET) without TA1 server connection. The resultant output files showed no issues exercising the scenarios. Using the script, I also tested the DEFAULT and MULTI_KDMA configs with prod TA1 server connection. I was able to get session alignment scores and encountered no errors here either.

@nextcen-dgemoets
Copy link
Copy Markdown
Contributor Author

Ahh I didn't run them together. I ran the september materials and then reverted to testing if the phase 1 stuff still worked.

Oh I see-- extra credit for you!

@nextcen-dgemoets nextcen-dgemoets merged commit 976c118 into main Sep 12, 2025
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.

3 participants