Motion_Control_Examples icon indicating copy to clipboard operation
Motion_Control_Examples copied to clipboard

Voltage not updated when using PFM450_pythonnet.py with Kinesis Simulator

Open bng919 opened this issue 1 year ago • 1 comments

When using PFM450_pythonnet.py with Kinesis Simulator the following error is recorded in the simulator log, and voltage is not updated. Simulator Log attached.

Comms Processing Error: Object reference not set to an instance of an object.
System.NullReferenceException: Object reference not set to an instance of an object.
   at Thorlabs.MotionControl.KinesisSimulator.Model.GenericPiezo.PrecisionPiezoSimulator.SimulateControlMode()
   at Thorlabs.MotionControl.KinesisSimulator.Model.GenericModules.GenericDeviceType1.<>c__DisplayClass18_0.<AddGetterSetterWithChannel>b__0(UInt16 cmd, Byte destination, Byte[] bytes)
   at Thorlabs.MotionControl.KinesisSimulator.Model.GenericModules.GenericDeviceBase.ProcessCommand(UInt16 cmdID, Byte destination, Byte[] data)

This issue occurs with the following two lines uncommented: https://github.com/Thorlabs/Motion_Control_Examples/blob/088a9dfc0d1ae5f3b8b51bb06ae230bc6445b0dc/Python/Benchtop/PPC00X/PFM450_pythonnet.py#L29 https://github.com/Thorlabs/Motion_Control_Examples/blob/088a9dfc0d1ae5f3b8b51bb06ae230bc6445b0dc/Python/Benchtop/PPC00X/PFM450_pythonnet.py#L102

Simulator settings Simulator type: PPC001 Precision Piezo Mode: Standard connection Stage: PFM450E

EventLog -20240905-125826.Log

Screenshot 2024-09-05 125815

bng919 avatar Sep 05 '24 19:09 bng919

Hello, Thank you for bringing this to our attention. We're currently investigating this occurrence. I will post another comment once we've discovered the source of this issue.

Bests,

Truman Metz

TrumanMetz avatar Sep 19 '24 20:09 TrumanMetz

Hi, Thank you for your patience on this matter. We have updated the PPC example to fix that error. Let us know if you continue to have issues with the new version, but I will be closing this issue for now.

Bests, Truman Metz

TrumanMetz avatar Nov 07 '24 15:11 TrumanMetz