You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I first checked if there were any open issues regarding matching design and off-design combustor areas and was going to push a PR myself, when I saw PR#62.
I am documenting here the issue to support that PR.
Descripion
When the exemple given in "pyCycle/example_cycles/high_bypass_turbofan.py" is run, I get the following result, showing a calculated area for DESIGN.burner.Fl_O.stat:area=169.041. Further down in the results, for the Off-Design results, I get OD_part_pwr.burner.Fl_O.sta:area=1550.003 (see excerpt from hbtf_view results file):
Further investigation showed that in class HBTF(pyc.Cycle), the Combustor element is added on line 54 of "high_bypass_turbofan.py":
self.add_subsystem('burner', pyc.Combustor(fuel_type=FUEL_TYPE))
Then, on line 370 of "high_bypass_turbofan.py", class MPhbtf() creates an instance of HBTF (line 298) and on line 370, the pyc_use_default_des_of_conns() method is invoked.
Looking in "mp_cycle.py", where class MPCycle is defined, the method expects a list of default connections for a given element to be in "default_des_od_conns"
Proposed fix:
Add the following to Combustor class' "initialize" method:
I first checked if there were any open issues regarding matching design and off-design combustor areas and was going to push a PR myself, when I saw PR#62.
I am documenting here the issue to support that PR.
Descripion
When the exemple given in "pyCycle/example_cycles/high_bypass_turbofan.py" is run, I get the following result, showing a calculated area for DESIGN.burner.Fl_O.stat:area=169.041. Further down in the results, for the Off-Design results, I get OD_part_pwr.burner.Fl_O.sta:area=1550.003 (see excerpt from hbtf_view results file):
Further investigation showed that in class HBTF(pyc.Cycle), the Combustor element is added on line 54 of "high_bypass_turbofan.py":
self.add_subsystem('burner', pyc.Combustor(fuel_type=FUEL_TYPE))
Then, on line 370 of "high_bypass_turbofan.py", class MPhbtf() creates an instance of HBTF (line 298) and on line 370, the pyc_use_default_des_of_conns() method is invoked.
Looking in "mp_cycle.py", where class MPCycle is defined, the method expects a list of default connections for a given element to be in "default_des_od_conns"
Proposed fix:
Add the following to Combustor class' "initialize" method:
Results:
After implementing this fix, I get the following results from running the same example given in "pyCycle/example_cycles/high_bypass_turbofan.py":
Recommendation
Merge PR#62
The text was updated successfully, but these errors were encountered: