-
Notifications
You must be signed in to change notification settings - Fork 27
Update toolbox.adoc#117
Update toolbox.adoc#117thomas-saigre wants to merge 2 commits intofeelpp:masterfrom thomas-saigre:patch-1
Conversation
|
drag and lift are misnamed, they would require having the direction of the flow and the direction of motion. |
prudhomm
left a comment
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.
Would it be possible to remove the notion of drag and lift but add a TIP saying that the forces correspond to the drag and lift of the fluid or object motion is aligned with the x-axis ?
|
|
||
| ==== Forces | ||
| compute lift and drag | ||
| Compute lift and drag |
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.
drag and lift are misnamed, they would require having the direction of the flow and the direction of motion. Forces computes the surfacic forces (F_x, F_y) applied by the fluid to the object. It would be drag and lift if the flow or motion is aligned with the x-axis
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.
@vincentchabannes probably a discussion on forces should be done in http://docs.feelpp.org/toolboxes/0.108/cfd/theory/
| Where `markerX` are the markers of where we want to compute the forces. The results will be given in a csv file whith those columns : | ||
|
|
||
| - time | ||
| - drag_marker1 |
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.
I think drag and lift should be removed here until we provide a direction of flow or motion
|
Stale pull request message |
|
@thomas-saigre could we finish this? |
|
I do not remember what I was trying to explain here... I think it was about exports for practical work during M1 year, but it's been a while since I last used cfd toolbox... |
|
Stale pull request message |