Skip to content
This repository has been archived by the owner on Oct 24, 2023. It is now read-only.

FSV instrument nodes and peak detection #337

Open
wants to merge 9 commits into
base: develop
Choose a base branch
from
Open

FSV instrument nodes and peak detection #337

wants to merge 9 commits into from

Conversation

dstrande
Copy link
Contributor

@dstrande dstrande commented Oct 20, 2023 •
edited
Loading

Description

  • I've included a concise description of what each node does

Styleguide

  • My node adheres to the styleguide for Flojoy nodes

Docs

  • I've submitted a PR for a documentation page for the new node(s) that contains usage examples (see docs.flojoy.io)

Testing

  • This PR includes a unit test (example here and/or ideally a screenshot of the node's output on an example app.

@linear
Copy link

linear bot commented Oct 20, 2023

NOD-86 RS FSV signal analyzer initial nodes

Will likely add more nodes later separately

@flojoy-bot flojoy-bot bot mentioned this pull request Oct 20, 2023
Draft
@dstrande dstrande requested a review from 39bytes October 24, 2023 14:30
dstrande and others added 2 commits October 24, 2023 07:30
@dstrande @39bytes
Update IO/INSTRUMENTS/SIGNAL_ANALYZERS/ROHDE_SCHWARZ/FSV/BASIC/INIT_S…
2b78354 
…WEEP_FSV/INIT_SWEEP_FSV.py

Co-authored-by: Jeff Zhang <[email protected]>
@dstrande @39bytes
Update IO/INSTRUMENTS/SIGNAL_ANALYZERS/ROHDE_SCHWARZ/FSV/BASIC/SWEEP_…
def44ff 
…SETTINGS_FSV/SWEEP_SETTINGS_FSV.py

Co-authored-by: Jeff Zhang <[email protected]>
@dstrande dstrande mentioned this pull request Oct 24, 2023
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@39bytes 39bytes Awaiting requested review from 39bytes

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dstrande @39bytes