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

[BUG] LTIME is not translated properly #170

Closed
PTKu opened this issue May 4, 2023 · 2 comments · Fixed by #171
Closed

[BUG] LTIME is not translated properly #170

PTKu opened this issue May 4, 2023 · 2 comments · Fixed by #171
Labels
question Further information is requested s7-webapi

Comments

@PTKu
Copy link
Contributor

PTKu commented May 4, 2023

  • We are off by two orders of magnitude
@PTKu PTKu added question Further information is requested s7-webapi labels May 4, 2023
@PTKu
Copy link
Contributor Author

PTKu commented May 4, 2023

/cib

@PTKu
Copy link
Contributor Author

PTKu commented May 4, 2023

Branch 170-_BUG_LTIME_is_not_translated_properly created!

PTKu added a commit that referenced this issue May 4, 2023

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
@PTKu PTKu closed this as completed in #171 May 4, 2023
PTKu added a commit that referenced this issue May 4, 2023

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
* Create draft PR for #168

* add polling for rcc on init and releases on dispose as before

* Create draft PR for #170

* LTIME to ms range unable to use full range not settable in plc

---------

Co-authored-by: PTKu <PTKu@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested s7-webapi
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant