Skip to content

Feature: flexlogger plugins#12

Merged
alexluck-sift merged 19 commits intomainfrom
feat-flexlogger-plugin
Jun 11, 2025
Merged

Feature: flexlogger plugins#12
alexluck-sift merged 19 commits intomainfrom
feat-flexlogger-plugin

Conversation

@alexluck-sift
Copy link
Copy Markdown
Contributor

@alexluck-sift alexluck-sift commented Jun 5, 2025

Updates to Sift Client:

  • Now both gRPC and REST URLs are required (previously, these were implicit)
  • Exposes some additional properties for use in Flexlogger plugins

Adds Flexlogger plugins for:

  • Sift Stream - Live streaming of Flexlogger data
  • Sift TDMS Uploader - Uploads TDMS files to Sift after recording is complete

@alexluck-sift alexluck-sift requested a review from marc-sift June 5, 2025 01:03
Comment thread plugins/flexlogger/Sift Stream/build/Sift Stream/Sift Stream.lvlibp Outdated
Comment thread plugins/flexlogger/README.md Outdated
Comment thread src/Sift/Sift API/Remote File Service/File Upload.vi
@marc-sift
Copy link
Copy Markdown
Collaborator

marc-sift commented Jun 6, 2025

On internal VIs:

  • Run "Clean Up Diagram" (Ctrl + U) to format
  • Add docstrings so they appear when "Show Context Help" is enabled.

Comment thread plugins/flexlogger/Sift TDMS Uploader/source/SubVIs/Event Log.vi
Comment thread plugins/flexlogger/Sift TDMS Uploader/source/SubVIs/Event Log.vi
Comment thread plugins/flexlogger/Sift TDMS Uploader/source/SubVIs/Event Log.vi
Comment thread plugins/flexlogger/Sift TDMS Uploader/source/Handle Message.vi
@marc-sift
Copy link
Copy Markdown
Collaborator

Make sure we're checking output errors for all VIs that provide them. Internal functions should also check incoming errors and bypass the function if there is one where it makes sense to do so.

Comment thread plugins/flexlogger/Sift Stream/source/Sift Stream/Process.vi
Fixed multiple instance cleanup for TDMS Uploader
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Can you open a ticket to move some of these utilities to Sift Client?

@alexluck-sift alexluck-sift merged commit b27d683 into main Jun 11, 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.

2 participants