Closed oscarfonloz closed 3 years ago
FYI: My edits also adds a non-PR comment related to how the Async communicates to the RT Driver VI, since it took quite some time for us to understand it and I think it's worth capturing in the diagram. I also cleaned up the VIs I edited to implement the feature, but I did not change behavior/functions used (in case you see things moved around in the Diff)
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.
@oscarfonloz it looks like Sample File Data.vi
still has an action item?
There is a typo in the new control name on the file page (should be continuous
), and the control description is wrong.
I still need to do some functional testing, but that will be a slow process due to power outages at my house.
@oscarfonloz it looks like
Sample File Data.vi
still has an action item?
Good catch! I did implement the functionality there, just forgot to remove the comment.
There is a typo in the new control name on the file page (should be
continuous
), and the control description is wrong.
Thanks! It's a tricky word! haha
I still need to do some functional testing, but that will be a slow process due to power outages at my house.
I planned to do the documentation in a later PR, but since we're already here, I submitted a tip strip and description that we can change later.
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?
This PR adds the capability to store waveform properties in log files that are continuous (if the user chooses to do so in the configuration).
Why should this Pull Request be merged?
See User Story 1278600 for the full spec.
What testing has been done?
Manual tests and review