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

Removed GSL dependency #35

Merged
merged 2 commits into from
Jun 19, 2023
Merged

Removed GSL dependency #35

merged 2 commits into from
Jun 19, 2023

Conversation

MichaelZhao21
Copy link
Contributor

Description

  • Added custom Digamma psi function (See https://www.jstor.org/stable/2347257)
  • Because of this, I no longer have to use a debian image
  • Instead, I use an alpine image and add a couple of dependencies (eg. libgcc)
  • This decreases overall image size of the production deployment container (~55 mb)

Fixes #16

Type of Change

Delete options that do not apply:

  • Refactor (code changes that doesn't affect functionality)

Is this a breaking change?

  • Yes
  • No

@MichaelZhao21 MichaelZhao21 changed the title Wrote custom Digamma psi function Removed GSL dependency Jun 19, 2023
@MichaelZhao21 MichaelZhao21 merged commit 4ce4782 into hackutd:master Jun 19, 2023
@MichaelZhao21 MichaelZhao21 deleted the psi-function branch October 4, 2023 14:07
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.

Replace GSL
1 participant
@MichaelZhao21