Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Edit plot notebook to include results of cases with re-distribute internal heat gain (IHG) #24

Open
2 tasks
JayHuLBL opened this issue Oct 24, 2017 · 8 comments
Assignees

Comments

@JayHuLBL
Copy link
Contributor

  • add plot showing room temperature of south and north zone, for base/G36 case, spring/summer/winter season, with/without IHG re-distribute. It will all show in one plot.
  • add two more columns (base/G36 case with redistribute IHG) in plot of annual energy use.
@JayHuLBL JayHuLBL self-assigned this Oct 24, 2017
@JayHuLBL
Copy link
Contributor Author

@mwetter I have updated the post_process.ipynb based on the latest update from yesterday (branch issue24_plot_IHG). It is fine to generate plot showing room temperature of south and north zone for short term simulation, but it will have error to read annual simulation results. Although I have allocated 14 GB memory to my virtual machine, it still gives me the error even at reading the 1st annual result.

@milicag
Copy link
Contributor

milicag commented Oct 25, 2017

@JayHuLBL did you try running outside of the notebook?

@mwetter
Copy link
Member

mwetter commented Oct 25, 2017

@JayHuLBL I will need to check when I am back at the lab and have access to the results.

@JayHuLBL
Copy link
Contributor Author

JayHuLBL commented Oct 25, 2017

@milicag when running outside of the notebook, it can successfully run first annual case but not the second one.

Read annual G36 without diverse loads
Traceback (most recent call last):
  File "run_test.py", line 117, in <module>
    r_base_annual_without_diverse_loads=get_results('annual_base')
  File "run_test.py", line 47, in get_results
    return Reader(file_name, "dymola")
  File "/home/jianjun/GitFolder/BuildingsPy/buildingspy/io/outputfile.py", line 179, in __init__
    self._data_ = DyMatFile(fileName)
  File "/home/jianjun/GitFolder/BuildingsPy/buildingspy/thirdParty/dymat/DyMat/__init__.py", line 44, in __init__
    self.mat = scipy.io.loadmat(fileName, chars_as_strings=False)
  File "/usr/lib/python2.7/dist-packages/scipy/io/matlab/mio.py", line 135, in loadmat
    matfile_dict = MR.get_variables(variable_names)
  File "/usr/lib/python2.7/dist-packages/scipy/io/matlab/mio4.py", line 399, in get_variables
    mdict[name] = self.read_var_array(hdr)
  File "/usr/lib/python2.7/dist-packages/scipy/io/matlab/mio4.py", line 374, in read_var_array
    return self._matrix_reader.array_from_header(header, process)
  File "/usr/lib/python2.7/dist-packages/scipy/io/matlab/mio4.py", line 137, in array_from_header
    arr = self.read_full_array(hdr)
  File "/usr/lib/python2.7/dist-packages/scipy/io/matlab/mio4.py", line 207, in read_full_array
    return self.read_sub_array(hdr)
  File "/usr/lib/python2.7/dist-packages/scipy/io/matlab/mio4.py", line 184, in read_sub_array
    arr = arr.copy()
MemoryError

@milicag
Copy link
Contributor

milicag commented Oct 25, 2017

@JayHuLBL Is there a way to increase the memory allocated to the python kernel? Are you saving in two separate variables (you could merge them later)? I'd also try to keep only 2 weeks per season of data before reading in the other case.

@JayHuLBL
Copy link
Contributor Author

@milicag I checked the Ubuntu system memory usage when running the code. It indicates that the memory easily use out, reduced from 12GB to less than 1GB and then becomes insufficient.
I did save the two results reading into two separate variables.

@milicag
Copy link
Contributor

milicag commented Oct 26, 2017

@JayHuLBL Let's see what Michael suggests. I don't know enough about *.mat files and whether any other data storing files I'm familiar with would be more memory efficient (storing as something else, reading in the second case, etc.).

@milicag
Copy link
Contributor

milicag commented Oct 26, 2017

@JayHuLBL One other thing to consider is that python 64 bit is much better at memory allocation.

JayHuLBL added a commit that referenced this issue Oct 26, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants