Update waves.py for nfreq=1

sandialabs / WecOptTool

WEC Design Optimization Toolbox

https://sandialabs.github.io/WecOptTool/

GNU General Public License v3.0

12 stars 20 forks source link

Update waves.py for nfreq=1 #312

Closed jtgrasb closed 5 months ago

jtgrasb commented 5 months ago

Description

This pull request allows for waves to be set up with nfreq = 1 without producing errors to resolve #308.

Type of PR

[X] Bug fix

coveralls commented 5 months ago

Pull Request Test Coverage Report for Build 7423551248

0 of 0 changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.08%) to 94.058%

Totals
Change from base Build 7135291663:	0.08%
Covered Lines:	2501
Relevant Lines:	2659

💛 - Coveralls

jtgrasb commented 5 months ago

I've removed the lines

if radians.size == 1: radians = radians.item()

from the degrees_to_radians() function. There doesn't seem to be a reason for this line. It seems to make the result of degrees_to_radians() 1-dimensional, but it doesn't seem to actually affect any results.