Skip to content

Update dependencies and add data contract#20

Merged
bruno-f-cruz merged 25 commits into
mainfrom
update-to-contraqctor
Apr 20, 2026
Merged

Update dependencies and add data contract#20
bruno-f-cruz merged 25 commits into
mainfrom
update-to-contraqctor

Conversation

@bruno-f-cruz
Copy link
Copy Markdown
Member

@bruno-f-cruz bruno-f-cruz commented Aug 7, 2025

Fixes #19
Closes #7

This PR needs to be tested at the rig. If someone has a chance, please collect a dataset that can be used to validate the data contract and the qc pipeline.

@bruno-f-cruz bruno-f-cruz changed the base branch from main to refactor-timestamp-gen-by-white-rabbit August 14, 2025 00:07
@bruno-f-cruz bruno-f-cruz changed the base branch from refactor-timestamp-gen-by-white-rabbit to main August 14, 2025 00:07
@tiffanyona
Copy link
Copy Markdown

Here is some data: "Z:\scratch\vr-foraging\olfactometer_test\new_PR\5B - floweter to ODOROUT"

Everything seems to run fine but the olfactometer -qc is not working anymore. The error I am getting is:

  File "C:\git\Aind.Behavior.Device.Olfactometer\.venv\Lib\site-packages\contraqctor\contract\base.py", line 205, in data
    raise ValueError("Data has not been loaded yet.")
ValueError: Data has not been loaded yet.

@DaniAlcAlv
Copy link
Copy Markdown

Fixes #19 Closes #7

This PR needs to be tested at the rig. If someone has a chance, please collect a dataset that can be used to validate the data contract and the qc pipeline.

Test OK> No significant differences were observed when comparing calibration data collected with Hw1.1/Fw2.2 and Hw2.0/Fw2.2 to Hw1.1/Fw1.4 OlfNewFw.pdf

Minor issues> A main.editor file and a main.layout file are created and must be deleted to avoid a dirty repository error on the next run. Additionally, the following error occurred on one occasion:
ErrorMessage)

@bruno-f-cruz bruno-f-cruz merged commit 0a1e6aa into main Apr 20, 2026
4 checks passed
@bruno-f-cruz bruno-f-cruz deleted the update-to-contraqctor branch April 20, 2026 22:59
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.

qc-olfactometer not working Include motor stage controller in olfactometer calibration

3 participants