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

Use NTScalarArray for 1D arrays. #977

Merged
merged 3 commits into from
Oct 31, 2023
Merged

Use NTScalarArray for 1D arrays. #977

merged 3 commits into from
Oct 31, 2023

Conversation

omar-moreno
Copy link
Collaborator

This PR updates the protocol epicsV4 to use the normative type NTScalar when a 1D array is encountered. Currently, the type NTNDArray is being used but that is meant to encode a data frame from a detector or an image from a camera.

Description

Make use of the normative type NTScalar when encoding 1D arrays in pyrogue.protocols.epicsV4.

JIRA

https://jira.slac.stanford.edu/browse/ESROGUE-640

Related

This PR addresses the following issue: slaclab/darpa-accel-llrf-phase-1p5#6

@codecov-commenter
Copy link

Codecov Report

Merging #977 (9202a3f) into main (64ac9ce) will decrease coverage by 0.01%.
The diff coverage is 55.38%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@            Coverage Diff             @@
##             main     #977      +/-   ##
==========================================
- Coverage   30.02%   30.01%   -0.01%     
==========================================
  Files          65       65              
  Lines        6634     6636       +2     
  Branches     1013     1014       +1     
==========================================
  Hits         1992     1992              
- Misses       4485     4487       +2     
  Partials      157      157              
Files Coverage Δ
python/pyrogue/protocols/epicsV4.py 54.66% <55.38%> (-0.50%) ⬇️

@omar-moreno omar-moreno changed the title Use NTScalars for 1D arrays. Use NTScalarArray for 1D arrays. Oct 18, 2023
@ruck314 ruck314 changed the base branch from main to pre-release October 24, 2023 20:01
@ruck314
Copy link
Contributor

ruck314 commented Oct 27, 2023

@omar-moreno Confirmed this worked with your array-dev during as the P4P server
image

@ruck314 ruck314 merged commit 11db7f7 into pre-release Oct 31, 2023
5 checks passed
@ruck314 ruck314 deleted the array-dev branch October 31, 2023 20:58
@ruck314 ruck314 mentioned this pull request Dec 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants