Ticket #169 (closed defect: fixed)

Opened 10 years ago

Last modified 10 years ago

CalcTwoPhaseProp: number of phase labels

Reported by: michelpons Owned by: Michael Halloran
Priority: minor Milestone: 2.2
Component: COLoggers Version: 2.1
Keywords: CalcTwoPhaseProp Cc: michaelhalloran

Description

With a ChemSep UO, A TEA PP 1.1 and PRO/II 9.2.1, I noticed the following entries in the log for CalcTwoPhaseProp:

PropertyPackage <n-depropropanizer> : Call to CalcTwoPhaseProp

CalcTwoPhaseProp takes input Arguments:

Property Phase

SurfaceTension Vapor
SurfaceTension Liquid

I am pretty sure only one property calculation was requested so the layout of the log entries related to this should be different. There are two phase labels (CapeArrayString) instead of one per property so it should be something like:

Property Phase1 Phase2

SurfaceTension Vapor Liquid

Change History

comment:1 Changed 10 years ago by michaelhalloran

  • Status changed from new to closed
  • Resolution set to fixed

Fixed with Commit 281. Output now looks like:

PropertyPackage <HDA> : Call to CalcTwoPhaseProp

CalcTwoPhaseProp takes input Arguments:

Property:
surfaceTension
Phases:
Vapor
Liquid

MaterialObject <Material3> : Call to GetCompoundList

GetCompoundList returns:

Name CAS No Formula TBP MW
Hydrogen 1333-74-0 H2 20.39 2.01588
Methane 74-82-8 CH4 111.66 16.043
Benzene 71-43-2 C6H6 353.24 78.114
Toluene 108-88-3 C7H8 383.79 92.141
Biphenyl 92-52-4 C12H10 528.15 154.211

It's done this way because Properties could be a list of any length.

Note: See TracTickets for help on using tickets.