coveralls
commented
5 months ago Pull Request Test Coverage Report for Build 8545211409
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
- For more information on this, see Tracking coverage changes with pull request builds.
- To avoid this issue with future PRs, see these Recommended CI Configurations.
- For a quick fix, rebase this PR at GitHub. Your next report should be accurate.
Details
- 5 of 6 (83.33%) changed or added relevant lines in 1 file are covered.
- No unchanged relevant lines lost coverage.
- Overall coverage increased (+0.002%) to 94.034%
| Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
|---|---|---|---|---|---|
| wecopttool/core.py | 5 | 6 | 83.33% | ||
| <!-- | Total: | 5 | 6 | 83.33% | --> |
| Totals | |
|---|---|
| Change from base Build 8545123630: | 0.002% |
| Covered Lines: | 2506 |
| Relevant Lines: | 2665 |
Description
This PR fixes a bug in in fd_to_td() when using the function without optional arguments. Now, we don't divide the DC and Nyquist component of the real valued frequency vector by 2. This solves #321
Additionally, td_to_fd() was slightly restructured, to make the scaling for single vs two-side spectrum more intuitive.
Type of PR
Checklist for PR
Additional details
Include any relevant context and describe any validation and verification efforts.