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

Deprecate and possibly remove unused functions and conditions #79

Closed
kessler-frost opened this issue Feb 2, 2022 · 0 comments · Fixed by #102
Closed

Deprecate and possibly remove unused functions and conditions #79

kessler-frost opened this issue Feb 2, 2022 · 0 comments · Fixed by #102
Assignees
@kessler-frost
Labels
bug Something isn't working covalent-dispatcher Related to the dispatcher module lattice Related to the lattice component

Comments

@kessler-frost
Copy link
Member

There are certain functions in covalent which are not being used currently and may as well be removed. Some examples include check_consumable, check_constraint_specific_sum in lattice.py, some utility functions, the schedule condition in _plan_workflow (_plan_workflow may itself be removed but that isn't that big a deal).
@kessler-frost kessler-frost added bug Something isn't working covalent-dispatcher Related to the dispatcher module lattice Related to the lattice component labels Feb 2, 2022
@kessler-frost kessler-frost mentioned this issue Feb 10, 2022
Merged
3 tasks
@kessler-frost kessler-frost self-assigned this Feb 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kessler-frost kessler-frost

Labels
bug Something isn't working covalent-dispatcher Related to the dispatcher module lattice Related to the lattice component
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Removing unused functions AgnostiqHQ/covalent
1 participant
@kessler-frost