Skip to content

Backport commits that fix rate function giving incorrect results#1286

Merged
tonyqus merged 1 commit into
nissl-lab:masterfrom
superrnovae:rate
Sep 22, 2024
Merged

Backport commits that fix rate function giving incorrect results#1286
tonyqus merged 1 commit into
nissl-lab:masterfrom
superrnovae:rate

Conversation

@superrnovae
Copy link
Copy Markdown
Contributor

This pretty much updates Rate function to the latest upstream commit.
Testcases are included.

Closes: #1226

@tonyqus tonyqus added this to the NPOI 2.7.1 milestone Mar 6, 2024
@Bykiev
Copy link
Copy Markdown
Collaborator

Bykiev commented Apr 3, 2024

Can you please fix test errors and link to related bugfix commit in POI?

@tonyqus tonyqus modified the milestones: NPOI 2.7.1, NPOI 2.7.2 May 6, 2024
@tonyqus
Copy link
Copy Markdown
Member

tonyqus commented May 6, 2024

Test case 'TestMicrosoftExample1' failed. Can you fix it?

@tonyqus
Copy link
Copy Markdown
Member

tonyqus commented Jun 22, 2024

Can you merge from master and rerun the unit tests?

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CalculateRate(double,double,double,double,double,double)@Rate can calculate a wrong rate.

3 participants