Skip to content

Add polyhedral-analysis to docs dependencies#58

Merged
bjmorgan merged 1 commit intomainfrom
fix-docs-polyhedral-analysis
Apr 4, 2026
Merged

Add polyhedral-analysis to docs dependencies#58
bjmorgan merged 1 commit intomainfrom
fix-docs-polyhedral-analysis

Conversation

@bjmorgan
Copy link
Copy Markdown
Owner

@bjmorgan bjmorgan commented Apr 4, 2026

Summary

  • Adds polyhedral-analysis>=0.3 to the docs optional dependencies in pyproject.toml
  • The SrTiO3 tilt animation in generate_images.py requires this package for octahedral tilt angle computation, without which the RTD build fails

The SrTiO3 tilt animation in generate_images.py requires
polyhedral-analysis for octahedral tilt angle computation.
Without this dependency the RTD docs build fails.
Copilot AI review requested due to automatic review settings April 4, 2026 12:32
Copy link
Copy Markdown

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 updates the project’s documentation build dependencies to ensure Read the Docs can successfully run docs/_static/generate_images.py, specifically the SrTiO3 octahedral tilt animation that imports polyhedral_analysis.

Changes:

  • Add polyhedral-analysis>=0.3 to the docs optional dependency group in pyproject.toml.

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

@bjmorgan bjmorgan merged commit e6c50d3 into main Apr 4, 2026
9 checks passed
@bjmorgan bjmorgan deleted the fix-docs-polyhedral-analysis branch April 4, 2026 12:35
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