Skip to content

Add real32 (Beckhoff float) and real64 (Beckhoff double) on jazzy - #197

Merged
tpoignonec merged 8 commits into
ICube-Robotics:jazzyfrom
christophfroehlich:real32-float
Dec 17, 2025
Merged

Add real32 (Beckhoff float) and real64 (Beckhoff double) on jazzy#197
tpoignonec merged 8 commits into
ICube-Robotics:jazzyfrom
christophfroehlich:real32-float

Conversation

@christophfroehlich

Copy link
Copy Markdown
Contributor

I tried to merge the current jazzy branch into #132 from @JensVanhooydonck

I applied my comments from #132 (review) and fixed some compiler warnings.

As of now, I only tested compilation and smoke test with double PDO interfaces, hardware tests hopefully tomorrow.

@tpoignonec tpoignonec left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hi,
Thanks for the PR, everything looks good to me

@tpoignonec

Copy link
Copy Markdown
Member

I tried to merge the current jazzy branch into #132 from @JensVanhooydonck

I applied my comments from #132 (review) and fixed some compiler warnings.

As of now, I only tested compilation and smoke test with double PDO interfaces, hardware tests hopefully tomorrow.

Do you confirm it works on your hardware?
We do not have anything that requires float as far as I know.

@tpoignonec tpoignonec self-assigned this Dec 17, 2025
@JensVanhooydonck

Copy link
Copy Markdown
Contributor

I tried to merge the current jazzy branch into #132 from @JensVanhooydonck
I applied my comments from #132 (review) and fixed some compiler warnings.
As of now, I only tested compilation and smoke test with double PDO interfaces, hardware tests hopefully tomorrow.

Do you confirm it works on your hardware? We do not have anything that requires float as far as I know.

I didn't test this branch, but the changes I made works for floats. Can't quickly test this for now, because I have a branch with to much changes in for now to quickly swap and test.

@christophfroehlich

Copy link
Copy Markdown
Contributor Author

I used it for reading PDOs with floats, that worked perfectly

@tpoignonec

Copy link
Copy Markdown
Member

I used it for reading PDOs with floats, that worked perfectly

Alright, thank you both!
I'll merge it now then

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.

3 participants