Skip to content

Enable PVA support in OPIs & add basic data streaming OPI#1834

Merged
rerpha merged 5 commits intomasterfrom
enable_pva_support
Apr 27, 2026
Merged

Enable PVA support in OPIs & add basic data streaming OPI#1834
rerpha merged 5 commits intomasterfrom
enable_pva_support

Conversation

@Tom-Willemsen
Copy link
Copy Markdown
Member

@Tom-Willemsen Tom-Willemsen commented Jan 9, 2026

Description of work

Enable PVA support in OPIs & add an initial, basic, datastreaming OPI.

Ticket

ISISComputingGroup/DataStreaming#26

Acceptance criteria

  • An OPI with a pva:// pv in it works.
  • Data streaming OPI works

Code Review

Final Steps

-XX:ErrorFile=C:\Instrument\Scratch\CrashDumps\ibex_client_%p.log
-Dlog4j2.formatMsgNoLookups=true
-Djdk.util.zip.disableZip64ExtraFieldValidation=true
"-DEPICS_PVA_ADDR_LIST=127.0.0.1 130.246.51.255"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

is this intended to be committed on this PR/is it an artifact of a branch that is yet to be rebased?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

actually sorry im being stupid - it's PVA support isn't it? how do we set EPICS_CA_ADDR_LIST though normally?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yes, this is PVA support.

We set epics CA address lists via a .properties file, but as far as I could tell no such properties exist for PVA and env/system variables were the only way to configure this for PVA.

I also found the PVA support in CS-Studio a bit flaky especially on reconnection - would be curious if you find the same. If so it's probably something we need to look into...

Copy link
Copy Markdown
Contributor

@rerpha rerpha left a comment

Choose a reason for hiding this comment

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

have created ISISComputingGroup/DataStreaming#43 to look into the weird pva disconnect issue but this looks a good start.

@rerpha
Copy link
Copy Markdown
Contributor

rerpha commented Apr 27, 2026

although could you add the dummy widget pls @Tom-Willemsen

@rerpha rerpha merged commit 2aaa8e3 into master Apr 27, 2026
8 checks passed
@rerpha rerpha deleted the enable_pva_support branch April 27, 2026 13:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants