search

ibell / pdsim

Steady-State simulation code for positive displacement machines
MIT License
38 stars 23 forks source link

PDSim GUI failed to use REFPROP fluid #79

Closed Wang-Zongwei closed 1 year ago

Wang-Zongwei commented 1 year ago

Today I installed pdsim on a new PC (windows 11) with latest github code. I found below issue, and I hope my feedback is useful and not too elementary.

  1. Installation process is very smooth except below: ModuleNotFoundError: No module named 'pandas'

    This issue is easy to solve, only need input "pip install pandas", then it's OK.

  2. PDSim GUI failed to use REFPROP fluid: The lastest version (PDSim 2.14 and CoolProp 6.5.0) cannot call REFPROP fluid in GUI. But the 2.9H windows setup version (downloaded from https://sourceforge.net/projects/pdsim/files/) seems OK to call the REFPROP fluid in GUI. Attached is related picture. (1st pic is based on PDSim GUI 2.14; 2nd pic is based on PDSim 2.9)

    Please help check if this issue could be fixed? Thanks very much.

NOK OK

ibell commented 1 year ago

How about REFPROP use without Bicubic interpolation, is that still ok?

Wang-Zongwei commented 1 year ago

For 2.9H windows setup version GUI, with or without Bicubic both OK to call REFPROP (see below pic). For 2.14 conda GUI version, with o.r without Bicubic both NOK to call REFPROP. pdsim 2 9

ibell commented 1 year ago

What error do you get?

Wang-Zongwei commented 1 year ago

For 2.14 conda GUI version, with o.r without Bicubic both NOK to call REFPROP. There is no any response if I select the "+" fluid: BICUBIC&REFPROP::R32[0.697614699375863]&R125[0.302385300624138] or REFPROP::R32[0.697614699375863]&R125[0.302385300624138] or part 1 part 2

ibell commented 1 year ago

Fixed, there was an error message in the main output log

Wang-Zongwei commented 1 year ago

Thanks so much. Below error in the main output log:

File "C:\pdsim-master\GUI\panels\pdsim_panels.py", line 2367, in OnAddFluid
  if 'BICUBIC' in Fluid or 'TTSE' in Fluid:

TypeError: a bytes-like object is required, not 'str'

Wang-Zongwei commented 1 year ago

I found latest verison from "/pdsim/tree/HEOSGUI" already solved this issue. Thanks so much!!!