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

Remove code for unsupported Julia versions #155

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Conversation

devmotion
Copy link
Member

@devmotion devmotion commented Jan 31, 2023

When looking through tdist.jl, I noticed that there is some code left for nowadays unsupported Julia versions.

Edit: Somehow read 0.7 instead of 1.7, so clearly one of the branches is still needed...

Edit2: Maybe it would actually be better to use log2mexp here to continue performing operations in log-space.

@devmotion
Copy link
Member Author

devmotion commented Jan 31, 2023

Hmm, I think log2mexp would be a better approach - but it seems it requires adding a _expm1(::Float64) definition to LogExpFunctions? Or to use a _log2mexp function on older Julia versions.

@codecov-commenter
Copy link

codecov-commenter commented Jan 31, 2023

Codecov Report

Base: 61.58% // Head: 58.37% // Decreases project coverage by -3.21% ⚠️

Coverage data is based on head (1304710) compared to base (7217d12).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #155      +/-   ##
==========================================
- Coverage   61.58%   58.37%   -3.21%     
==========================================
  Files          14       14              
  Lines         617      543      -74     
==========================================
- Hits          380      317      -63     
+ Misses        237      226      -11     
Impacted Files Coverage Δ
src/distrs/tdist.jl 100.00% <100.00%> (+2.27%) ⬆️
src/inverse.jl 25.00% <0.00%> (-75.00%) ⬇️
src/distrs/binom.jl 75.00% <0.00%> (-3.58%) ⬇️
src/misc.jl 85.71% <0.00%> (-3.18%) ⬇️
src/tvpack.jl 14.51% <0.00%> (-1.97%) ⬇️
src/distrs/gamma.jl 97.43% <0.00%> (-0.49%) ⬇️
src/distrs/norm.jl 97.05% <0.00%> (-0.38%) ⬇️
src/rmath.jl 100.00% <0.00%> (ø)
src/distrs/beta.jl 100.00% <0.00%> (ø)
... and 2 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@devmotion
Copy link
Member Author

The remaining test errors are caused by the recent Rmath update, see #156

@devmotion devmotion closed this Jun 30, 2023
@devmotion devmotion reopened this Jun 30, 2023
@andreasnoack andreasnoack reopened this Feb 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants