Skip to content

docs: calibration#141

Merged
dbirman merged 6 commits into
mainfrom
117-calibration
May 28, 2026
Merged

docs: calibration#141
dbirman merged 6 commits into
mainfrom
117-calibration

Conversation

@dbirman
Copy link
Copy Markdown
Member

@dbirman dbirman commented May 24, 2026

Adds information about how to upload calibration data


📚 Documentation preview 📚: https://scicomp-docs--141.org.readthedocs.build/en/141/

@dbirman dbirman linked an issue May 24, 2026 that may be closed by this pull request
@dbirman dbirman requested a review from arielleleon May 26, 2026 21:41

procedures = Procedures(subject_id=subject_id)

subject.write_standard_file()
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Will this leave fields not specified in subject empty or are there default values?

Copy link
Copy Markdown
Member Author

@dbirman dbirman May 28, 2026

Choose a reason for hiding this comment

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

Subject only has three fields and the required ones are being set in the code here https://aind-data-schema.readthedocs.io/en/latest/subject.html


### Manual calibration metadata

If the processing pipeline that will run on your test data asset requires certain fields in the subject or procedures metadata to be set you need to create an actual `subject.json` and `procedures.json` and upload these alongside your data asset.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Can this be in bold?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

added bold

Copy link
Copy Markdown
Collaborator

@arielleleon arielleleon left a comment

Choose a reason for hiding this comment

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

Just a couple things then it's g2g

@dbirman dbirman merged commit bd130a6 into main May 28, 2026
1 check passed
@dbirman dbirman deleted the 117-calibration branch May 28, 2026 21:37
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.

Add a short section on how to upload calibration data

2 participants