control module pipeline fixes & added control module UTs

[mraduldubey]

IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).

Fixes #384

Description

Fixed bunch of issues in control module

1. thread not joining issue
2. cyclic dependency

Added ability to print status of the control module as a serialized string

Added tests for control module (none existed)

Alternative(s) considered
None

Type

Type Choose one: (Bug fix | Feature | Testing )

Screenshots (if applicable)

Checklist

• I have read the Contribution Guidelines
• I have written Unit Tests
• I have discussed my proposed solution with code owners in the linked issue(s) and we have agreed upon the general approach

[github-actions]

Test Results Windows-cuda

  1 files  ±0    1 suites  ±0   12m 17s ⏱️ +10s
410 tests +2  293 ✅ +1  117 💤 +1  0 ❌ ±0 
293 runs  +1  176 ✅ ±0  117 💤 +1  0 ❌ ±0 

Results for commit 6f90a5d. ± Comparison against base commit 9ecea0d.

♻️ This comment has been updated with latest results.

[github-actions]

Test Results Linux

  1 files  ±0    1 suites  ±0   10m 16s ⏱️ +8s
315 tests +2  228 ✅ +1  83 💤 +1  4 ❌ ±0 
232 runs  +1  145 ✅ ±0  83 💤 +1  4 ❌ ±0 

For more details on these failures, see this check.

Results for commit 6f90a5d. ± Comparison against base commit 9ecea0d.

♻️ This comment has been updated with latest results.

[github-actions]

Test Results Win-nocuda

  1 files  ±0    1 suites  ±0   10m 28s ⏱️ -1s
309 tests +2  232 ✅ +1  77 💤 +1  0 ❌ ±0 
232 runs  +1  155 ✅ ±0  77 💤 +1  0 ❌ ±0 

Results for commit 6f90a5d. ± Comparison against base commit 9ecea0d.

♻️ This comment has been updated with latest results.

[kushaljain-apra]

Looks good to me