Skip to content

Improve docstrings for design execution and candidate classes#1041

Closed
gjmulhol wants to merge 1 commit intomainfrom
docs/design-execution-classes
Closed

Improve docstrings for design execution and candidate classes#1041
gjmulhol wants to merge 1 commit intomainfrom
docs/design-execution-classes

Conversation

@gjmulhol
Copy link
Copy Markdown

Summary

  • DesignExecution: explain lifecycle and link to Score/candidates methods
  • candidates(), hierarchical_candidates(), predict(): add full Parameters/Returns sections (these had NO docstrings beyond one line)
  • PredictRequest: add full Parameters section for all 5 constructor arguments (previously undocumented)
  • DesignCandidate: explain primary_score semantics (higher = better)
  • DesignVariable: list all 5 subtypes and when each is used
  • DesignCandidateComment: add missing class docstring

Test plan

  • All 1342 tests pass (docstring-only changes)

🤖 Generated with Claude Code

Docs PR 3 of 8 in the documentation improvement series.

- DesignExecution: explain the execution lifecycle and link to
  Score, candidates(), and hierarchical_candidates()
- candidates(): document return ordering (by score), per_page,
  and return type with full Parameters/Returns sections
- hierarchical_candidates(): explain the difference from
  candidates() (includes process history and sub-materials)
- predict(): document PredictRequest input and DesignCandidate
  return with full Parameters/Returns sections
- PredictRequest: add full Parameters section documenting all
  5 constructor arguments
- DesignCandidate: explain primary_score semantics
- DesignVariable: list all subtypes and when each is used
- DesignCandidateComment: add missing class docstring

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@gjmulhol
Copy link
Copy Markdown
Author

Consolidated into docs/improve-sdk-documentation branch.

@gjmulhol gjmulhol closed this Mar 25, 2026
@gjmulhol gjmulhol deleted the docs/design-execution-classes branch March 25, 2026 22:26
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.

1 participant