Skip to content

fix(config): load manim.cfg beside input file - #4971

Open
emerardd wants to merge 1 commit into
ManimCommunity:mainfrom
emerardd:fix/load-scene-local-config
Open

fix(config): load manim.cfg beside input file#4971
emerardd wants to merge 1 commit into
ManimCommunity:mainfrom
emerardd:fix/load-scene-local-config

Conversation

@emerardd

Copy link
Copy Markdown
Contributor

Closes #4963.

Summary

  • Reload folder-wide configuration from the input file's directory before applying CLI arguments.
  • Ignore a manim.cfg in the execution directory when rendering a scene elsewhere.
  • Preserve explicit --config_file, stdin/Jupyter handling, and directory arguments whose config supplies input_file.
  • Add regression coverage for scene-adjacent config lookup and directory-based input config.

Validation

  • 8 focused configuration and CLI tests
  • Ruff check and formatting
  • Targeted mypy check for manim/_config/utils.py
  • git diff --check

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.

manim.cfg reading not working as expected

1 participant