fixed linting errors and updated linting guide

[justinmadison]

Description

Brief description of what this PR does.

Closes #(issue number)

Type of Change

• Bug fix (non-breaking change that fixes an issue)
• New feature (non-breaking change that adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Documentation update
• Performance improvement
• Code refactoring
• Tests
• Build/CI changes

Changes Made

• List key changes
• One per line
• Be specific

Testing

• Tests pass locally (pytest tests/)
• Added new tests for new features
• Tested manually (describe below)
• No regressions in existing functionality

Manual Testing:
Describe how you tested this change manually.

Performance Impact

• No performance impact
• Improves performance
• May impact performance (explain below)

Documentation

• Updated relevant documentation
• Added code comments for complex logic
• Updated CHANGELOG (if applicable)
• Added/updated docstrings

Checklist

• My code follows the project's style guidelines
• I have performed a self-review of my code
• I have commented my code, particularly in hard-to-understand areas
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes

Screenshots (if applicable)

Add screenshots to help reviewers understand your changes.

Additional Notes

Any additional information reviewers should know.