Skip to content

fix(us-bf-026): rename ModelMetaData to ModelMetadata for consistent casing - #206

Merged
ooples merged 3 commits into
merge-dev2-to-masterfrom
fix/us-bf-026-modelmetadata-casing
Oct 24, 2025
Merged

ooples merged 3 commits into
merge-dev2-to-masterfrom
fix/us-bf-026-modelmetadata-casing

Conversation

@ooples

@ooples ooples commented Oct 24, 2025

Copy link
Copy Markdown
Owner

Summary

  • Renamed ModelMetaData<T> class to ModelMetadata<T> (lowercase 'd') in src/Models/ModelMetadata.cs
  • Updated IFullModel interface to reference ModelMetadata<T> instead of ModelMetaData<T>
  • Renamed GetModelMetaData() method to GetModelMetadata() in IModel interface
  • Updated all 95+ references across the codebase (neural networks, regression, time series, etc.)

Problem

The codebase had inconsistent casing: the class was named ModelMetaData<T> (capital D) but many files referenced ModelMetadata<T> (lowercase d), causing compilation errors:

  • CS0246: The type or namespace name 'ModelMetadata<>' could not be found
  • CS0535: Class does not implement interface member 'GetModelMetaData()'

Solution

Standardized all references to use ModelMetadata (lowercase 'd') which is the more conventional C# naming for "metadata" as a compound word.

Files Changed

  • Core files (3):

    • src/Models/ModelMetadata.cs - Class renamed
    • src/Interfaces/IFullModel.cs - Interface reference updated
    • src/Interfaces/IModel.cs - Method renamed
  • References updated (95+ files across):

    • Neural networks (30+ files)
    • Regression models (20+ files)
    • Time series models (20+ files)
    • Genetics algorithms
    • Cross validators
    • AutoML models
    • Transfer learning

Build Status

✅ Casing-related errors (CS0246, CS0535) resolved
⚠️ Other pre-existing build errors remain (unrelated to this fix)

Test Plan

  • Build verification shows CS0246/CS0535 errors are resolved
  • All references to ModelMetaData successfully updated to ModelMetadata
  • Method calls updated from GetModelMetaData() to GetModelMetadata()

🤖 Generated with Claude Code

…casing

- Renamed class ModelMetaData<T> to ModelMetadata<T> in src/Models/ModelMetadata.cs
- Updated IFullModel interface to use ModelMetadata<T> instead of ModelMetaData<T>
- Renamed GetModelMetaData() method to GetModelMetadata() in IModel interface
- Updated all 95 references across the codebase to use correct casing

Resolves CS0246 and CS0535 compilation errors related to type name mismatch.

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

Co-Authored-By: Claude <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings October 24, 2025 02:47

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@ooples
ooples requested a review from Copilot October 24, 2025 03:13
@ooples
ooples merged commit 8b8f31f into merge-dev2-to-master Oct 24, 2025
0 of 2 checks passed
@ooples
ooples deleted the fix/us-bf-026-modelmetadata-casing branch October 24, 2025 03:21
@ooples
ooples removed the request for review from Copilot March 23, 2026 20:23
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