Skip to content

Conversation

Tom-Willemsen
Copy link
Member

Description of work

Add your own description here

To test

Which ticket does this PR fix?

Acceptance criteria

List the acceptance criteria for the PR

  • ** If this PR changes GALIL / GALILMUL ioc are these changes applicable to both the old (galil-old branch based) and new (master branch based) drivers? If so have all appropriate PRs been created **

Code Review

Functional Tests

  • IOC responds correctly in:
    • Devsim mode
    • Recsim mode
    • Real device, if available
  • Supplementary IOCs (..._0n where n>1) run correctly
  • Log files do not report undefined macros (serach for macLib: macro to find instances of macLib: macro [macro name] is undefined...

Final steps

  • Update the IOC submodule in the main EPICS repo. See Git workflow page for details.
  • Reviewer has merged the associated PR for the release notes

@github-project-automation github-project-automation bot moved this to Flash Review in Tasks Oct 1, 2025
@FreddieAkeroyd
Copy link
Member

Another option is to set APST field to 1 so it only pushes on change (calculated via checksum) - at the moment we are getting the same value every second.

@Chsudeepta
Copy link
Contributor

@FreddieAkeroyd , you mean field(APST, "On Change")?

Copy link
Contributor

@Chsudeepta Chsudeepta left a comment

Choose a reason for hiding this comment

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

Can we use APST to trigger the logging only when there is a change? Setting the val to 5 minutes may mean that if there are multiple changes we will miss logging them.

@FreddieAkeroyd
Copy link
Member

FreddieAkeroyd commented Oct 8, 2025

@Chsudeepta Setting APST to 1 is the same as On Change as it is an enum, have pushed change

@Chsudeepta Chsudeepta merged commit 3ac881b into master Oct 8, 2025
@Chsudeepta Chsudeepta deleted the drop_triton_archive_rate branch October 8, 2025 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants