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.

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

Labview Examples for Kinesis Motion Control Products #78

Open
wants to merge 14 commits into
base: main
Choose a base branch
from

Conversation

movagharnematiThorlabs
Copy link
Contributor

Formatted the following examples to new github friendly formatting (screenshot of block diagram, front panel, vi and a read me) from older examples:
Kinesis - KNA - Latch Trak
Kinesis - KNA - Set Circle Diameter
Kinesis - KST101 - Move Relative
Kinesis - KIM101 - Set Jog
Kinesis - BNT - Set Circle Home Position
Kinesis - BNT - Connect
Formatted the new examples I made in the same way
Kinesis - BBD30X - Connect Abs Rel Move
Kinesis - M30XY - Set Velocity Step
Kinesis - BSC20X - Connect Abs Rel Move
Kinesis - KBD101 - Connect Move
Kinesis - KBD101 - Connect Jog

All saved for labview 2015.

Doneill-Thorlabs and others added 13 commits November 7, 2024 10:48

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Fixed ctypes example for the BBD
LoadMotorConfiguration(serialNo); leads to "object reference not set to an instance of an object" error.
device.GetIOPortConfigParams(0x01) also leads to error. As IOPortConfigParams isn't really used in this example so I delete the corresponding commands.
The example doesn't contain Enable/Disable Device and Start/Stop polling commands.
Block,Front,Readme,VI for Kinesis Labview Examples
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks good, but we need to make sure we are initializing a channel for the benchtops as well.

Copy link
Contributor

Choose a reason for hiding this comment

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

Should be formatted and saved as a .md file so github knows to display this. You can download and open the other ones we have to get a sense of how you to make titles and bolden text.

Copy link
Contributor

Choose a reason for hiding this comment

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

Should be formatted and saved as a .md file

Copy link
Contributor

Choose a reason for hiding this comment

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

Should be formatted and saved as a .md file

Copy link
Contributor

@TrumanMetz TrumanMetz left a comment

Choose a reason for hiding this comment

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

Good job Far. There are a few things that need to be tweaked but once these have been addressed this will be good to go.

Copy link
Contributor

Choose a reason for hiding this comment

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

Looks good, but we need to make sure we are initializing a channel for the benchtops as well.

Copy link
Contributor

Choose a reason for hiding this comment

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

Should be formatted and saved as a .md file

Copy link
Contributor

Choose a reason for hiding this comment

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

Should be formatted and saved as a .md file

Copy link
Contributor

Choose a reason for hiding this comment

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

Should be formatted and saved as a .md file

Copy link
Contributor

Choose a reason for hiding this comment

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

Should be formatted and saved as a .md file

Copy link
Contributor

Choose a reason for hiding this comment

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

Should be formatted and saved as a .md file

Copy link
Contributor

Choose a reason for hiding this comment

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

Should be formatted and saved as a .md file

Copy link
Contributor

Choose a reason for hiding this comment

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

Should be formatted and saved as a .md file

Copy link
Contributor

Choose a reason for hiding this comment

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

Should be formatted and saved as a .md file

Copy link
Contributor

Choose a reason for hiding this comment

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

We'll want to change this to KST201.

changed LoadMotorConfiguration
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.

None yet

5 participants