Open huges84 opened 3 years ago
To resolve the issue running the GUI, I ran the setup.py file in pdsim directory using powershell. (There was no instructions to do so, I just found that file made reference to quantities folder and gave it a shot). That didn't seem to work, because I still got errors about the quantities module.
Then I installed the quantities package using "conda install -c conda-forge quantities'. That then allowed pdsimgui.py to launch without error. I also ran the default inputs and solver without issue.
I use Py3.8 through Anaconda and MSVC2019 for compiling and everything works well. I would skip Py3.7.
The main conda repo does not have a version of wxpython compatible with 3.8. Also the latest release of PDSim (v2.13) states "Hold off on python 3.8 for now". However, I think that by following the directions I am using the latest GitHub code. Would you recommend that I use the conde-forge version of wxpython and python 3.8 with the latest PDSim code?
Not true anymore. wxpython 4.1.1 works like a charm. There could be a few minor syntax issues but mostly they have been taken care of. I am running Py3.8.5 and I have no issues.
@huges84 I too am a mechanical engineer ! So is @davideziviani !
I pushed some changes to the docs, hopefully they help to address some of your issues you ran into.
Regarding your questions: the pip package for CoolProp is always up to date, not always the case with the conda package (I think). So always better to use that.
Yeah, the tutorial is indeed out of date. Once you get a bit up to speed with PDSim, want to write one? You clearly have the right mindset to contribute that, and it is better if it is written by a non-expert because then all the things that are not obvious to you get documented.
I am happy to contribute however I can. I will need to poke around a bit first.
What is the best way to contact you for questions? I will try to respect your time by researching python questions before asking those type of questions and looking at the code, papers and thesis before asking PDSim questions.
If anybody else is having issues with Conda make sure that you don't have another python installed and make conda your default python. that worked for me.
Hello, First off, let me start by saying I am a mechanical engineer and good with computers but very novice with programming and python, so hopefully my feedback is useful and not too elementary. I had some difficulties getting up and running following the Read The Docs information, so I wanted to give my feedback. BTW, I was doing all of this in Windows 10 x64 with admin rights.
If I understand correctly, conda is used instead of pip to avoid compiling, handle version conflicts and create environments. However, I ran into errors using it that I think others will as well.
So my recommendations to update Read The Docs are: A. If there is a specific version of Python to use, indicate so in the install instructions and have the user install the Miniconda version that packages the correct python version in the installer. B. Make clear that the conda install step should not be broken into pieces, instead it should be run in one big install. If there's errors then the user should try an older python version that all packages might be compatible with. C. Perhaps make use of the conda-forge version of wxpython, because it seems that the normal conda wxpython is not being maintained. D. I had to do some activation of conda to get it to work in the PowerShell after installing Miniconda. I have forgotten the steps, but some guidance or at least mention of it might be good. E. I would add some basic instructions on getting to the point that you can run conda and not have the command line immediately close (open PowerShell by holding shift and right clicking in Miniconda3\Scripts folder). F. For pulling the repo from github with Git, you might want to mention that all options can be left at defaults (no boxes checked). G. For the step to compile PDSim, I believe that "python setup.py install" has to be run from the pdsim directory containing the code. If so, that should be mentioned. I opened my PowerShell window in that directory before running the command and it worked. H. The step after step 8 is labeled as 1 and it says "Now go into the doc folder, start IPython, .." This is the first mention of IPython. At first I thought it was a typo for Python, but I see it's not. Installing IPython or maybe Jupiter should be mentioned first. I. The Linux instructions on RTD also mentions Python 2.X. J. The instructions for creating an environment and running PDSim should be much higher on the page, IMO. I would put them right after the Linux instructions. I missed them a few times.
The "pdsim\doc\tutorial" folder has really outdated information, referring to Python 2.x and VS 2008. It's probably best to delete this entire folder and just point to the RTD page. pdsim_compile_log_2021-02-04-14_34.txt pdsimgui_error_2021-02-04-15_31.txt