Skip to content

Add RBV signals to PVA transport#170

Merged
GDYendell merged 8 commits intomainfrom
153_add_rbv_signals_in_pva
Jul 24, 2025
Merged

Add RBV signals to PVA transport#170
GDYendell merged 8 commits intomainfrom
153_add_rbv_signals_in_pva

Conversation

@shihab-dls
Copy link
Contributor

Fixes #153

This PR splits out make_shared_pv for read and write signals, creating two shared PV instances for AttrRW, where one is a _RBV.

@codecov
Copy link

codecov bot commented Jul 23, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.27%. Comparing base (e3e15ea) to head (325d670).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #170      +/-   ##
==========================================
- Coverage   92.58%   92.27%   -0.31%     
==========================================
  Files          40       40              
  Lines        1968     1980      +12     
==========================================
+ Hits         1822     1827       +5     
- Misses        146      153       +7     

☔ 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.

@shihab-dls shihab-dls marked this pull request as ready for review July 23, 2025 16:13
@shihab-dls shihab-dls requested a review from GDYendell July 23, 2025 16:19
@shihab-dls shihab-dls requested a review from GDYendell July 24, 2025 11:22
@GDYendell GDYendell merged commit aec213b into main Jul 24, 2025
17 of 18 checks passed
@GDYendell GDYendell deleted the 153_add_rbv_signals_in_pva branch July 24, 2025 14:18
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.

Create _RBV records for PVA transport

2 participants