Jump to content

Periodic trigger lossless version


Recommended Posts

I was using the OpenG Periodic Trigger in an application and notice that my time base was drifting. After spending some time on it and rewritting to make it a little more readable I made a change to fix what seems to be a time leak to me. The change is made where it gets the new time after a trigger has occurred. If someone would take a look and give me feedback that would be great. Could this be an update to the OpenG Time library?Periodic Trigger.zip

  • Like 2
Link to comment
  • 2 months later...

OK, I figured out what the essential difference between the two implementation is.

The original OpenG Periodic trigger resets the phase upon every trigger. Your implementation is maintaining the phase.

I have a little table showing this:

post-2399-0-08085700-1351436224.png

Where I had a Time (ms) of 500 ms and waited 450, 100, 400, 100 and 10 ms. The column Next Trigger is showing the output state of the upper Shift Register which basically contains when the next rollover of the timer will happen.

I think we should add both implementations, making yours the default for new VIs, the old functionality should be maintained via deprecation. So currently already code isn't changing current behaviour.

Here's my proposal:

post-2399-0-80425800-1351437434.png

And it's documentation:

post-2399-0-12966800-1351437677.png.

Losing phase has functionality if you alter the 'Time to wait'. However we might need to auto-reset then as well.

EDIT:

Please note that you might get unexpected Triggers with Maintain Phase On:

post-2399-0-22557800-1351447327.png

Ton

Link to comment

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.