Skip to content

Conversation

@braden6521
Copy link
Contributor

Purpose

Brought SofastCommandLIneInterface up to OpenCSP standards.

Summary of changes

  • Moved the common elements of the old SofastCommandLineInterface from /contrib/ into /common/.
  • Added an example in the example/sofastFringe directory that uses this new class.
  • NOTE: this example requires a working camera, screen, and mirror to run. Thus, I did not set it up to run automatically in the nightlies.

Implementation notes

Since this requires a physical Sofast setup to run, there are no unit tests for this module. Sofast was run in the lab using this module and associated example successfully.

Submission checklist

  • Target branch is develop, not main
  • Existing tests are updated or new tests were added
  • opencsp/test/test_DocStringsExist.py are verified to include this change or have been updated accordingly
  • .rst file(s) under doc/ are verified to include this change or have been updated accordingly

Additional information

None

@braden6521 braden6521 requested review from e10harvey and rcbrost March 14, 2025 18:35
@e10harvey e10harvey force-pushed the sofast_interface_class branch from cd489f0 to 6ce4331 Compare April 16, 2025 13:38
@e10harvey e10harvey self-assigned this May 6, 2025
@e10harvey e10harvey merged commit eb71369 into sandialabs:develop May 6, 2025
4 checks passed
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