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.

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

Move __init__.py utility functions to a utils.py module. #428

Closed
pvk-developer opened this issue Oct 15, 2024 · 0 comments · Fixed by #430
Closed

Move __init__.py utility functions to a utils.py module. #428

pvk-developer opened this issue Oct 15, 2024 · 0 comments · Fixed by #430
Assignees
Labels
maintenance Tasks related to infrastructure & dependencies
Milestone

Comments

@pvk-developer
Copy link
Member

pvk-developer commented Oct 15, 2024

Problem Description

The __init__.py file currently contains utility functions that are not directly related to package initialization. To improve code organization and maintain a clean separation of concepts, these utility functions should be moved to a dedicated utils.py module.

Expected behavior

Everywhere that we are importing from copulas import <utility function> should be from copulas.utils import <utility function> instead.

Additional context

  • Verify that this change does not break any existing functionality.
  • Consider adding any missing documentation for these utility functions while moving them.
  • Use functools to create the decorators if the utility function is meant to be a wrapper.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Tasks related to infrastructure & dependencies
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant