Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Added a Plotting example that uses TekHSI, tm_devices, and tm_data_types #69

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Added a Plotting example that uses TekHSI, tm_devices, and tm_data_types #69

wants to merge 4 commits into from

Conversation

abruno-tek
Copy link

@abruno-tek abruno-tek commented Oct 14, 2024 •
edited by dwyban
Loading

Pull Request Checklist

Please review the Contribution Guidelines before submitting.

  • Pulling against the master branch (left side).
  • You have previously submitted a Contributor License Agreement or have contacted a maintainer to request one.

Type (Select only one)

  • Bug fix
  • Working example
  • In-progress example (need Tektronix help)
  • In-progress example (does not need Tektronix help)

Description

Added an example for simple Plotting using TekHSI, tm_devices, and tm_data_types.

@OceanSBlueTyphoon21
Created tekhsi_SimplePlotExample
43d7882 
added the tekshsi_SimplePlotExample Folder and the tekhsi_simplePlotExample.py file.
@OceanSBlueTyphoon21
moved the tekhsi_simpleplotexample,.py
48e17a7 
moved to a different folder in the repo.
@OceanSBlueTyphoon21
added tskHSI simple plot example to readMe
01ef521 
Added to the readme file with TekHSI Simple Plot Example.
@OceanSBlueTyphoon21
added 3.12 to tekHSI
8a0ab64 
Added the link for python 3.12 for the tekHSI plot example in the ReadME file.
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

github-advanced-security[bot]
github-advanced-security bot found potential problems Oct 14, 2024
View reviewed changes
.../TekSeriesScopes_HighSpeedDigitizers/src/tekhsiSimplePlotExample/tekhsi_SimplePlotExample.py
# import time # ONLY for code line 58
import matplotlib.pyplot as plt
from tm_devices import DeviceManager # https://pypi.org/project/tm-devices/
from tm_devices.drivers import MSO4B, MSO5, MSO5B, MSO6, MSO6B # Importing Drivers for Tektronix MSO58B, https://pypi.org/project/tm-devices/

Check notice

Code scanning / CodeQL

Unused import Note

Import of 'MSO4B' is not used.
Import of 'MSO5' is not used.
Import of 'MSO6' is not used.
Import of 'MSO6B' is not used.
.../TekSeriesScopes_HighSpeedDigitizers/src/tekhsiSimplePlotExample/tekhsi_SimplePlotExample.py
import matplotlib.pyplot as plt
from tm_devices import DeviceManager # https://pypi.org/project/tm-devices/
from tm_devices.drivers import MSO4B, MSO5, MSO5B, MSO6, MSO6B # Importing Drivers for Tektronix MSO58B, https://pypi.org/project/tm-devices/
from tm_devices.drivers import DPO70K, DPO70KC, DPO70KD, DPO70KDX, DPO70KSX

Check notice

Code scanning / CodeQL

Unused import Note

Import of 'DPO70K' is not used.
Import of 'DPO70KC' is not used.
Import of 'DPO70KD' is not used.
Import of 'DPO70KDX' is not used.
Import of 'DPO70KSX' is not used.
.../TekSeriesScopes_HighSpeedDigitizers/src/tekhsiSimplePlotExample/tekhsi_SimplePlotExample.py
from tm_devices import DeviceManager # https://pypi.org/project/tm-devices/
from tm_devices.drivers import MSO4B, MSO5, MSO5B, MSO6, MSO6B # Importing Drivers for Tektronix MSO58B, https://pypi.org/project/tm-devices/
from tm_devices.drivers import DPO70K, DPO70KC, DPO70KD, DPO70KDX, DPO70KSX
from tm_devices.drivers import MSO70K, MSO70KC, MSO70KDX

Check notice

Code scanning / CodeQL

Unused import Note

Import of 'MSO70K' is not used.
Import of 'MSO70KC' is not used.
Import of 'MSO70KDX' is not used.
@dwyban
Copy link
Member

dwyban commented Oct 23, 2024

Hi @abruno-tek. Did you submit a CLA? The CLA Assistant indicates that one has not yet been submitted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bit-lift bit-lift Awaiting requested review from bit-lift bit-lift is a code owner

@adametry adametry Awaiting requested review from adametry adametry is a code owner

@dwyban dwyban Awaiting requested review from dwyban dwyban is a code owner

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.
4 participants
@abruno-tek @CLAassistant @dwyban @OceanSBlueTyphoon21