Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Remove pomegranate #230

Closed
amontanez24 opened this issue May 23, 2023 · 0 comments · Fixed by #241
Closed

Remove pomegranate #230

amontanez24 opened this issue May 23, 2023 · 0 comments · Fixed by #241
Assignees
@amontanez24
Labels
maintenance Tasks related to infrastructure & dependencies
Milestone
0.7.0

Comments

@amontanez24
Copy link
Contributor

Problem Description

Since the CLBN synthesizer was removed, we no longer need pomegranate. We should remove it from the dependencies.

Expected behavior

  • Pomegranate is no longer a required dependency.

Additional context

  • Investigate if this speeds up the install time for sdgym. If not, take note of what other dependencies are taking long.
@amontanez24 amontanez24 added the maintenance Tasks related to infrastructure & dependencies label May 23, 2023
@amontanez24 amontanez24 added this to the 0.7.0 milestone May 23, 2023
@amontanez24 amontanez24 mentioned this issue May 30, 2023
Merged
@amontanez24 amontanez24 self-assigned this Jun 13, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@amontanez24 amontanez24

Labels
maintenance Tasks related to infrastructure & dependencies
Projects
None yet
Milestone
0.7.0
Development

Successfully merging a pull request may close this issue.

remove pomegranate sdv-dev/SDGym
1 participant
@amontanez24