Skip to content

Changed version of adtimepix3#210

Open
Juan Marulanda (jmaruland) wants to merge 2 commits into
NSLS2:mainfrom
jmaruland:downgrade-serval-version-for-timepix3
Open

Changed version of adtimepix3#210
Juan Marulanda (jmaruland) wants to merge 2 commits into
NSLS2:mainfrom
jmaruland:downgrade-serval-version-for-timepix3

Conversation

@jmaruland

Copy link
Copy Markdown
Contributor

This PR modifies the yml files that pick the latest version of adtimepix3 to use an earlier version in order to make it compatible with the current version that is being run with the timepix3 at CHX.

Copilot AI review requested due to automatic review settings April 9, 2026 19:44

Copilot AI left a comment

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.

Pull request overview

This PR pins the ADTimePix3 module used by the Timepix3 IOC deployment to an earlier Git commit to maintain compatibility with the CHX Timepix3 runtime.

Changes:

  • Updated the ADTimePix3 module variable key and pinned git version to 15009fe.
  • Switched the Timepix3 IOC deployment to require adtimepix3_15009fe instead of adtimepix3_2faf970.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
roles/install_module/vars/adtimepix3_15009fe.yml Pins ADTimePix3’s git version and aligns the YAML variable key with the pinned module identifier.
roles/deploy_ioc/vars/adtimepix3.yml Updates the required module identifier so the deploy role installs/uses the pinned ADTimePix3 version.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jwlodek

Copy link
Copy Markdown
Collaborator

Actually, before merging, we need to fix the build failures on the test deployment.

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.

3 participants