Skip to content

Conversation

@bennybp
Copy link
Contributor

@bennybp bennybp commented Jun 26, 2025

Description

Implements native files protocols for optimization and torsiondrive

This refactors the protocols enum to common_models.py, and uses it for OptimizationSpecification and TorsiondriveSpecification.

Conversion function for optimization specifications is also updated.

Validators have been simply copied from AtomicInput. Eventually we will likely refactor out a base class for some of this, but not today.

Changelog description

Status

  • Code base linted
  • Ready to go

@codecov
Copy link

codecov bot commented Jun 26, 2025

Codecov Report

Attention: Patch coverage is 44.44444% with 30 lines in your changes missing coverage. Please review.

Please upload report for BASE (next2025@ced0474). Learn more about missing BASE report.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

)
stderr: Optional[str] = Field(None, description="The standard error of the program execution.")
native_files: Dict[str, Any] = Field({}, description="DSL files.")
native_files: Dict[str, Any] = Field({}, description="Other program-specific files returned from the computation.")
Copy link
Collaborator

Choose a reason for hiding this comment

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

Oh, I remember why native_files wasn't fully implemented (result field but not protocol) now: the target programs (geometric, optking, torsiondrive, qcmanybody) don't really have a history of random text or binary file in/outputs. Anything extra they did comm. could probably be absorbed into schema another way.

r"""Program-specific to keep from a computation."""

all = "all"
input = "input"
Copy link
Collaborator

Choose a reason for hiding this comment

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

And since geometric/TD/QCMB are API or schema input, they have no "input" file generated

Copy link
Collaborator

@loriab loriab left a comment

Choose a reason for hiding this comment

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

Also add a note (unorganized is fine) to the changelog, or we'll never keep track :-)



class NativeFilesProtocolEnum(str, Enum):
r"""Program-specific to keep from a computation."""
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
r"""Program-specific to keep from a computation."""
r"""Program-specific to keep from a computation.
Less likely to be used when program is API- or schema-based (e.g., geomeTRIC)
"""

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