Skip to content

Update the code block for fip mapper - #488

Merged
dougollerenshaw merged 1 commit into
devfrom
fix/418-fip-code-from-physiology-fip
Sep 11, 2026
Merged

dougollerenshaw merged 1 commit into
devfrom
fix/418-fip-code-from-physiology-fip

Conversation

@dougollerenshaw

Copy link
Copy Markdown
Contributor

This PR updates the FIP mapper to hard-code the name of the FIP acquisition repo instead of the experimental orchestration repo for VR/FIP, Aind.Experiment.VrForaging-Fip.

Hardcodes code.name as "aind-physiology-fip"

Hardcodes code.url as "https://github.com/AllenNeuralDynamics/Aind.Physiology.Fip"

Sets code.version as rig.version from the FIP extracted metadata (here). This is the aind-physiology-fip package version

Sets code.parameters as session.allow_dirty_repo (example. If true, this tells us that the acquisition code is not guaranteed to match the release version.

No longer uses session.commit_hash, since that's the HEAD of the deployment repo the launcher runs from (Aind.Experiment.VrForaging-Fip) rather than the acquisition control software itself.

The resulting Code entry on the FIP data stream will look like this:

{
  "url": "https://github.com/AllenNeuralDynamics/Aind.Physiology.Fip",
  "name": "aind-physiology-fip",
  "version": "0.3.1-rc2",
  "parameters": {"allow_dirty_repo": true}
}

closes #418

@dbirman dbirman left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good

@dougollerenshaw
dougollerenshaw merged commit e163268 into dev Sep 11, 2026
2 checks passed
@dougollerenshaw
dougollerenshaw deleted the fix/418-fip-code-from-physiology-fip branch September 11, 2026 22:00
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.

Update FIP mapper to use passed code info

2 participants