Issue 30 in cfast: Decay period parameter on EVENT line does not work correctly for long decay times.

1 view
Skip to first unread message

codesite...@google.com

unread,
Jul 23, 2008, 10:58:35 AM7/23/08
to cfast-...@googlegroups.com
Issue 30: Decay period parameter on EVENT line does not work correctly
for long decay times.
http://code.google.com/p/cfast/issues/detail?id=30

New issue report by cfastdev:
Sample input file attached.

There is a bug in the code that interpolates the opening
fraction between the beginning and end of the decay period. Since the
GUI automatically sets the decay parameter to 1 s, it did not have any
significant impact on the output for nearly all model users.

When the interval is longer (which can only be accomplished by
hand-editing the
input file), you would see a small initial drop that stayed constant
until the end of the decay period when the vent would "jump" to its
final position.

Attachments:
Long Event Decay.in 415 bytes


Issue attributes:
Status: Accepted
Owner: cfastdev
Labels: Type-Defect Priority-Medium Component-Logic

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

codesite...@google.com

unread,
Jul 23, 2008, 12:50:21 PM7/23/08
to cfast-...@googlegroups.com
Issue 30: Decay period parameter on EVENT line does not work correctly
for long decay times.
http://code.google.com/p/cfast/issues/detail?id=30

Comment #1 by cfastdev:
Interpolation routines for vent closing transition incorrect for all
EVENT types. Interpolation was <initial> +
dy/dt rather than <initial time> + dy/dt * (<time> - <initial time>).
During testing also discovered that
printout was incorrect for totally closed horizontal flow vents.
Calculation was correct, but printout output did
not set flow to zero once vent totally closed (stayed at value just
prior to total close). Corrected both problems
in Revision #49.


Issue attribute updates:
Status: Verified

Reply all
Reply to author
Forward
0 new messages