Skip to content

Comments

feat(metadata_schema): add scan_name and comment fields to basic schema#727

Open
wakonig wants to merge 2 commits intomainfrom
feature/scan_info_metadata
Open

feat(metadata_schema): add scan_name and comment fields to basic schema#727
wakonig wants to merge 2 commits intomainfrom
feature/scan_info_metadata

Conversation

@wakonig
Copy link
Member

@wakonig wakonig commented Jan 10, 2026

This PR adds two new fields to the default scan metadata schema: scan name and comment. Need to be merged together with

bec-project/bec_widgets#1002

@codecov
Copy link

codecov bot commented Jan 10, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@wakonig wakonig self-assigned this Jan 14, 2026
@wakonig wakonig marked this pull request as ready for review January 14, 2026 09:55
Copilot AI review requested due to automatic review settings January 14, 2026 09:55
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds two new optional metadata fields to the BasicScanMetadata schema: scan_name (for a human-friendly scan identifier) and comment (for scan annotations). The docstring is updated to reflect that no fields are strictly required, and the sample_name field title is capitalized for consistency.

Changes:

  • Added scan_name field with empty string default for identifying individual scans
  • Added comment field with empty string default for scan annotations
  • Updated docstring to remove "Only requires a sample name" phrasing
  • Standardized field titles to title case

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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