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

feat(asset.provider): Added type for Scale and Offset in Channels [backport release-5.6.0] #5426

Merged
merged 1 commit into from
Oct 7, 2024

Conversation

eclipse-kura-bot
Copy link

Backport c65d800 from #5418.

)

* Added new field for Channel, scaleoffset.type which assign a type (Double, Float, Integer or Long) to scale and offset value used in the channel.

Fixed bug on applyscaleoffset condition

Now offsetScale Type is not mandatory to preserver backward compatibility

Updated expected OCD

rewritten adding a new ScaleOffsetType with DEFINED_BY_VALUE_TYPE

minor fixes

Moved ScaleOffsetType into org.eclipse.kura.channel

Removed throwing IAE for old scale/offset getters

Removed all ScaleOffsetType except for DOUBLE

fixed tests

fixed test

Reverted org.eclipse.kura.type version change

* Restore getKuraServicePid change from develop.

* fix: Sonar fixes

Signed-off-by: MMaiero <[email protected]>

* fix: Added another sonar suggested improvement

Signed-off-by: MMaiero <[email protected]>

---------

Signed-off-by: MMaiero <[email protected]>
Co-authored-by: MMaiero <[email protected]>
(cherry picked from commit c65d800)
@MMaiero
Copy link
Contributor

MMaiero commented Oct 7, 2024

Forcing merge. Currently the Eclipse Jenkins instance is not working properly. We are going to merge the PRs and then perform a full build on Eclipse release-5.6.0 maintenance branch with, eventually, a following PR to fix remaining issues

@MMaiero MMaiero merged commit e3ad041 into release-5.6.0 Oct 7, 2024
2 checks passed
@MMaiero MMaiero deleted the backport-5418-to-release-5.6.0 branch October 7, 2024 08:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants