Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Add type hints to scene_file_writer.py, section.py, zoomed_scene.py #4133

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Add type hints to scene_file_writer.py, section.py, zoomed_scene.py #4133

wants to merge 6 commits into from
+126 −117

Conversation

fmuenkel
Copy link
Contributor

Overview: What does this pull request change?

  • Add type hints
  • Minor changes to docstrings

Motivation and Explanation: Why and how do your changes improve the library?

Progress on #3375

Further Information and Comments

Reviewer Checklist

  • The PR title is descriptive enough for the changelog, and the PR is labeled correctly
  • If applicable: newly added non-private functions and classes have a docstring including a short summary and a PARAMETERS section
  • If applicable: newly added functions and classes are tested

@fmuenkel fmuenkel changed the title Add type hints to scene_file_write.py, section.py, zoomed_scene.py Add type hints to scene_file_writer.py, section.py, zoomed_scene.py Jan 22, 2025
@JasonGrace2282 JasonGrace2282 added the typehints For adding/discussing typehints label Feb 9, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
typehints For adding/discussing typehints
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@fmuenkel @JasonGrace2282