-
Notifications
You must be signed in to change notification settings - Fork 99
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
added rootSetOption and error estimation #220
Conversation
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.
Just minor typos
Codecov Report
@@ Coverage Diff @@
## main #220 +/- ##
==========================================
+ Coverage 40.78% 41.04% +0.26%
==========================================
Files 13 13
Lines 3769 3793 +24
==========================================
+ Hits 1537 1557 +20
- Misses 2232 2236 +4
📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more |
adflow/pyADflow.py
Outdated
@@ -4682,7 +4682,7 @@ def solveErrorEstimate(self, aeroProblem, funcError, evalFuncs=None): | |||
-------- | |||
>>> funcs = {} | |||
>>> CFDsolver(ap) | |||
>>> CFDsolver.solveErrorEstimate(ap1, funcError, ['cl', 'cd']) | |||
>>> CFDsolver.solveErrorEstimate(ap, funcError, ['cl', 'cd']) | |||
>>> funcs | |||
>>> # Result will look like (if aeroProblem, ap1, has name of 'wing'): |
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.
ap
adflow/pyADflow.py
Outdated
Parameters | ||
---------- | ||
aeroProblem : pyAero_problem class | ||
The aerodynamic problem to to get the error for |
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.
typo 'to to'
Purpose
This PR adds two things:
Expected time until merged
Soon.
Type of change
Testing
There are two tests.
Checklist
flake8
andblack
to make sure the code adheres to PEP-8 and is consistently formatted