Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

refractor: labeller class #813

Closed
wants to merge 1 commit into from
Closed

refractor: labeller class #813

wants to merge 1 commit into from

Conversation

kachida
Copy link

@kachida kachida commented Apr 2, 2024

  • Modified the Singleton design pattern logic in getTrainingDataModel and getLabelDataViewHelper methods.
  • refractored core logics to separate methods to improve readbility and maintainability
  • Improved encapsulation by changing access modifiers of certain fields and methods to private where appropriate
  • Added logging statements for better error handling
  • Ensured adherence to single responsibility principle by splitting responsibilities into smaller methods

@kachida
refractor: labeller class
381b638 
- [x] Modified the Singleton design pattern logic in
  getTrainingDataModel and getLabelDataViewHelper methods.
- [x] refractored core logics to separate methods to
  improve readbility and maintainability
- [x] Improved encapsulation by changing access modifiers of certain
  fields and methods to private where appropriate
- [x] Added logging statements for better error handling
- [x] Ensured adherence to single responsibility principle by splitting
  responsibilities into smaller methods
@kachida kachida closed this Apr 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@kachida