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
flight state to parsing #139
Conversation
|
||
return [Re == (state["V"]*state["\\rho"]*l/state["\\mu"]), | ||
return [Re == (V*rho*l/mu), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove parens
gpkitmodels/GP/aircraft/wing/wing.py
Outdated
|
||
if fd["d"] == 2: | ||
independentvars = [self.CL, self.Re] | ||
elif fd["d"] == 3: | ||
independentvars = [self.CL, self.Re, static.planform.tau] | ||
|
||
return [Cd >= cdp + CL**2/np.pi/AR/e, | ||
Re == (state["\\rho"]*state["V"]*cmac/state["\\mu"]), | ||
Re == (rho*V*cmac/mu), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove parens
@@ -1,21 +1,22 @@ | |||
" wing test " | |||
from gpkitmodels.GP.aircraft.wing.wing import Wing | |||
from gpkitmodels.GP.aircraft.wing.boxspar import BoxSpar | |||
from gpkit import Variable, Model | |||
from gpkit import Variable, Model, parse_variables |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
does Variable still need to be imported?
path = os.path.dirname(__file__) | ||
fd = pd.read_csv(path + os.sep + "tail_dragfit.csv").to_dict( | ||
orient="records")[0] | ||
|
||
constraints = [ | ||
Re == (state["V"]*state["\\rho"]*S/b | ||
/ state["\\mu"]), | ||
Re == (V*rho*S/b/mu), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove parens
test this please |
test models please |
test models please |
@mjburton11 ready to merge this if it passes model tests |
@bqpd they won't pass unfortunately until I get convexengineering/solar#13 to pass |
merging now, will fix solar, gassolar and jho after. |
No description provided.