create common materials models #141
Conversation
|
@bqpd not sure of the best way to go about doing this, see use case in convexengineering/solar@47040c1 |
|
@bqpd One problem I'm having that you'll see the in the solar commit, is that I want the wing skin material, htail skin and vtail skin materials models to all be the same. I can do that with this approach, but the materials model that is used is like the 5th one that is created. I think because importing a model causes the materials model to be created? I'm wondering if there's a better way to do this. |
|
(noting here that we met and discussed in lab doing this with instances created on import of gplibrary) |
|
test models please |
| @@ -77,45 +75,10 @@ def box_spar(): | |||
| W, fs, perf, loading]) | |||
| m.solve(verbosity=0) | |||
|
|
|||
| def materials(): | |||
There was a problem hiding this comment.
This tested that I could swap out the materials models to have a common material model for any use of CFRPFabric, but this became irrelevant once you import created models from the materials __init__.py
| @@ -0,0 +1,2 @@ | |||
| from gpkit import Variable | |||
| g = Variable("g", 9.81, "m/s^2", "gravitational acceleration") | |||
There was a problem hiding this comment.
add "near the earth's surface" ?
There was a problem hiding this comment.
I pushed directly to master. Sorry I should have asked you to review this again before merging...
|
naw worries!
…On Tue, Dec 5, 2017 at 11:17 AM, Michael Burton ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In gpkitmodels/__init__.py
<#141 (comment)>
:
> @@ -0,0 +1,2 @@
+from gpkit import Variable
+g = Variable("g", 9.81, "m/s^2", "gravitational acceleration")
I pushed directly to master. Sorry I should have asked you to review this
again before merging...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#141 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABagGMcoYpopbOrVb4-cj1K8DXL3l_yZks5s9WyrgaJpZM4Qxvhn>
.
|
No description provided.