Skip to content

Add Tsuchinoko agent - #49

Merged
maffettone merged 23 commits into
bluesky:mainfrom
ronpandolfi:tsuchinoko
Jul 10, 2025
Merged

Add Tsuchinoko agent#49
maffettone merged 23 commits into
bluesky:mainfrom
ronpandolfi:tsuchinoko

Conversation

@ronpandolfi

@ronpandolfi ronpandolfi commented Apr 22, 2025

Copy link
Copy Markdown
Contributor

Description

Adding a TsuchinokoAgent that communicates with Tsuchinoko over zmq to request new targets and report back measurements. This is an abstract class that must be subclassed.

Motivation and Context

Adds capability to use gpCAM and Tsuchinoko in the bluesky-adaptive framework in response to interest from ILLUMINE and Jana Thayer at LCLS

Summary of Changes for Release Notes

Fixed

Added

  • GPCamTsuchinokoAgent and supporting classes
  • Test for GPCamTsuchinokoAgent

Changed

Removed

How Has This Been Tested?

See test_tsuchinoko_agents.py

TODO:

  • Complete test_tsuchinoko_agents.py

Comment thread bluesky_adaptive/agents/tsuchinoko.py Outdated
Comment thread bluesky_adaptive/agents/tsuchinoko.py Outdated
Comment thread bluesky_adaptive/agents/tsuchinoko.py Outdated
Comment thread bluesky_adaptive/agents/tsuchinoko.py Outdated
Comment thread bluesky_adaptive/tests/test_tsuchinoko_agents.py Outdated
Comment thread bluesky_adaptive/tests/test_tsuchinoko_agents.py Outdated
@maffettone

Copy link
Copy Markdown
Contributor

Making a quick note here that may relieve some blocking with tests etc.

I've finished some work on getting the upstream main branch to use pyproject.toml, ruff, and a more clear set of CI/CD steps.

A rebase may make some things a little awkward (e.g., no more requirements.txt), but hopefully it makes things easier in the long run.

@ronpandolfi
ronpandolfi marked this pull request as ready for review July 9, 2025 22:03
@maffettone
maffettone merged commit c8113a0 into bluesky:main Jul 10, 2025
5 checks passed
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