Skip to content

Add cast to Bool for EPICS bool records#168

Merged
GDYendell merged 1 commit intomainfrom
handle-epics-bool
Jul 9, 2025
Merged

Add cast to Bool for EPICS bool records#168
GDYendell merged 1 commit intomainfrom
handle-epics-bool

Conversation

@GDYendell
Copy link
Contributor

@GDYendell GDYendell commented Jul 7, 2025

Fixes #169

@codecov
Copy link

codecov bot commented Jul 7, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.58%. Comparing base (77189df) to head (ce8826b).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #168      +/-   ##
==========================================
+ Coverage   92.35%   92.58%   +0.22%     
==========================================
  Files          40       40              
  Lines        1962     1968       +6     
==========================================
+ Hits         1812     1822      +10     
+ Misses        150      146       -4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@GDYendell GDYendell force-pushed the handle-epics-bool branch from ccbd510 to 2c730b3 Compare July 8, 2025 14:32
Copy link
Contributor

@jsouter jsouter left a comment

Choose a reason for hiding this comment

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

Looks good, I'd seen this error with fastcs-odin but didn't realise it was a coming from the CA backend, assumed it was some weirdness on the odin side.

@GDYendell GDYendell merged commit e3e15ea into main Jul 9, 2025
18 checks passed
@GDYendell GDYendell deleted the handle-epics-bool branch July 9, 2025 08:13
@LuisFSegalla
Copy link
Contributor

A bit late for the party, but I tested this against fastcs-xspress and it worked pretty well!! It solved an issue I was having for a long time!

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.

Epics CA transport does not handle Bool values correctly

3 participants