Skip to content

use fc00 and f144 instead of rf5k and f142 respectively#401

Merged
iangillingham-stfc merged 13 commits intomasterfrom
update_streaming_data_types
Jan 9, 2025
Merged

use fc00 and f144 instead of rf5k and f142 respectively#401
iangillingham-stfc merged 13 commits intomasterfrom
update_streaming_data_types

Conversation

@rerpha
Copy link
Contributor

@rerpha rerpha commented Dec 2, 2024

Description of work

This uses the latest schemas from https://github.com/ess-dmsc/streaming-data-types to send forwarder updates as well as set up sample environment (f142 or nowadays f144) streams.

This currently isn't relied on, so won't break anything, but we should do some testing with this when we have a central forwarder again.

To test

Which ticket does this PR fix?

Acceptance criteria

List the acceptance criteria for the PR


Code Review

  • Is the code of an acceptable quality?
  • Has the author taken into account the multi-threaded nature of the code?
  • Have the changes been recorded appropriately in a PR for release notes?
  • Has the manual system tests spreadsheet been updated?

Functional Tests

  • Do changes function as described? Add comments below that describe the tests performed.

Final steps

  • Reviewer has updated the submodule in the main EPICS repo? See Reviewing work for the subModules of EPICS in the Git workflow page for details.
  • Reviewer has merged the associated PR for the release notes

@rerpha
Copy link
Contributor Author

rerpha commented Dec 12, 2024

CI is failing as it currently doesn't install https://pypi.org/project/ess-streaming-data-types/ on setup. we could fix this, or we could split up inst_servers and sort out requirements per-module instead. I'm in favour of the latter.

@iangillingham-stfc iangillingham-stfc self-assigned this Jan 9, 2025
@iangillingham-stfc iangillingham-stfc merged commit 47ac9b0 into master Jan 9, 2025
2 checks passed
@iangillingham-stfc iangillingham-stfc deleted the update_streaming_data_types branch January 9, 2025 11:45
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.

2 participants