Skip to content

Conversation

@ChrisRackauckas
Copy link
Member

Summary

  • Enhanced spell checking configuration to prevent 42 false positive flags
  • Added comprehensive coverage for symbolic indexing and Julia/SciML terminology
  • No genuine typos found - all flagged issues were legitimate technical terms

Technical Terms Added

  • setp: Set parameter function (core API functionality)
  • setp_oop: Set parameter out-of-place function (core API functionality)
  • Standard Julia/SciML terms: getp, getu, setu, indexin, mathematical terminology
  • Variable patterns: Common abbreviations used across the SciML ecosystem

Before/After

  • Before: 42 false positive flags for legitimate function names
  • After: Clean spell checking with proper recognition of domain-specific terminology

Background

The setp and setp_oop functions are legitimate exported API functions that:

  • Are documented in the official API documentation
  • Have comprehensive docstrings
  • Follow Julia naming conventions (parallel to existing getp function)
  • Are used consistently throughout the codebase

This enhancement maintains spell checking effectiveness while eliminating false positives for domain-specific technical terminology.

🤖 Generated with Claude Code

- Add comprehensive spell checking configuration for symbolic indexing terminology
- Allow legitimate technical terms: setp (set parameter), setp_oop (set parameter out-of-place), getp, getu, setu
- Include standard Julia/SciML terms to prevent false positives across the ecosystem
- Covers mathematical terms, variable patterns, abbreviations, and scientific terminology

This prevents the 42 false positive flags for legitimate `setp` and `setp_oop` function names
while maintaining spell checking effectiveness for genuine typos.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@ChrisRackauckas ChrisRackauckas merged commit 3790452 into master Jul 29, 2025
9 of 10 checks passed
@ChrisRackauckas ChrisRackauckas deleted the enhance-typos-config branch July 29, 2025 18:15
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.

3 participants