Closed dbendele closed 3 years ago
Bleep bloop!
LabVIEW Diff Robot here with some diffs served up hot for your pull request.
Notice something funny? Help fix me on my GitHub repo.
Bleep bloop!
LabVIEW Diff Robot here with some diffs served up hot for your pull request.
Notice something funny? Help fix me on my GitHub repo.
@buckd, I added you as a reviewer to get preliminary feedback on this draft PR. The PR will continue to grow as we add Rx and Tx Execution Unit classes.
Bleep bloop!
LabVIEW Diff Robot here with some diffs served up hot for your pull request.
Notice something funny? Help fix me on my GitHub repo.
Bleep bloop!
LabVIEW Diff Robot here with some diffs served up hot for your pull request.
Notice something funny? Help fix me on my GitHub repo.
the must override VIs added are still stubs and have not been implemented as part of this PR.
Bleep bloop!
LabVIEW Diff Robot here with some diffs served up hot for your pull request.
Notice something funny? Help fix me on my GitHub repo.
Is "AsynchronousLoopTime" supposed to represent status channels or something else?
Yes, the asynchronous loop time is the one status channel that the released CD creates. It is mirrored here.
Bleep bloop!
LabVIEW Diff Robot here with some diffs served up hot for your pull request.
Notice something funny? Help fix me on my GitHub repo.
What does this Pull Request accomplish?
Adds more scripting and shared functionality to parse the system definition and construct execution units.
Why should this Pull Request be merged?
Execution units transfer channel and configuration information for inline and asynchronous work from configuration to engine.
What testing has been done?
Scripting VIs are verified through automated unit tests. The Engine has not yet been implemented.