Skip to content

Support NA and Inf values#14

Merged
rossellhayes merged 5 commits into
mainfrom
non-finite
Nov 2, 2021
Merged

Support NA and Inf values#14
rossellhayes merged 5 commits into
mainfrom
non-finite

Conversation

@rossellhayes

Copy link
Copy Markdown
Owner
library(fracture)

fracture(c(NA, Inf, -Inf, 0.5))
#> [1] NA       "Inf/1"  "-Inf/1" "1/2"

fracture(c(NA, Inf, -Inf, 0.5), mixed = TRUE)
#> [1] NA     "Inf"  "-Inf" "1/2"

frac_mat(c(NA, Inf, -Inf, 0.5))
#>             [,1] [,2] [,3] [,4]
#> numerator     NA  Inf -Inf    1
#> denominator   NA    1    1    2

frac_mat(c(NA, Inf, -Inf, 0.5), mixed = TRUE)
#>             [,1] [,2] [,3] [,4]
#> integer       NA  Inf -Inf    0
#> numerator     NA    0    0    1
#> denominator   NA    1    1    2

Created on 2021-11-02 by the reprex package (v2.0.1)

Closes #13.

@rossellhayes rossellhayes added the enhancement New feature or request label Nov 2, 2021
@rossellhayes rossellhayes self-assigned this Nov 2, 2021
@codecov

codecov Bot commented Nov 2, 2021

Copy link
Copy Markdown

Codecov Report

Merging #14 (b1b1695) into main (a561939) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##              main       #14   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            7         7           
  Lines          238       261   +23     
=========================================
+ Hits           238       261   +23     
Impacted Files Coverage Δ
R/frac_mat.R 100.00% <100.00%> (ø)
R/fracture.R 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a561939...b1b1695. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support NA and Inf

1 participant