Re: [EPx Developers] Contribute to code EPx?

52 views
Skip to first unread message

Jeremiah Crossett

unread,
Feb 3, 2014, 10:32:23 PM2/3/14
to epx-dev...@googlegroups.com
Neal, 
I did not hear back from you-- is it possible to contribute to the project? 
  Not sure-- has there been any changes from standard E+ by EPx yet? 



Jeremiah D. Crossett
  | Senior Analyst  |  Phase Change Energy Solutions
120 E. Pritchard St.  | Asheboro, NC 27203 
 | Mobile 503-688-8951
  





On Thu, Jan 30, 2014 at 1:27 PM, Jeremiah Crossett <jcro...@phasechange.com> wrote:
Neal, 
I have attached my code in a zip file.  Most of the changes are to HeatBalFiniteDifferenceManager.f90, with small additions to DataHeatBalancef90 and SurfaceGeometry.f90. 

Also included is documentation in word doc format. 

Please let me know if this work can be contributed to EPx? 



Jeremiah D. Crossett
  | Senior Analyst  |  Phase Change Energy Solutions
120 E. Pritchard St.  | Asheboro, NC 27203 
 | Mobile 503-688-8951
  





On Wed, Oct 30, 2013 at 8:27 AM, Neal Kruis <neal....@bigladdersoftware.com> wrote:
Jeremiah,

We welcome anyone interested in contributing to EPx. Currently there are a few people with read/write access to the SVN repository. For those without read/write access you can send us files you have changed and we will review them before committing them to the repository. After a few submittals we can look into getting you read/write access as well. 

We are still establishing a vetting process for adding new committers to the repository, but for now feel free to post any changes you make to the code and we will look into merging them into the development branch.

Thanks for your interest!

Neal


On Tuesday, October 29, 2013 3:42:44 PM UTC-6, NRGSIM wrote:
Mengsong, I'm not sure about the tutorial, but I am compiling just fine with a GNU compiler bundled into an IDE called simply fortran. A free trial can be downloaded on their website: http://simplyfortran.com/  This is for my custom version that is modified from the development team version 8.1 and under my own licence (not actually EPX)  With simply fortran I use the following compiler flags: -cpp -ffree-line-length-none -Wtabs -fbacktrace -Wno-align-commons -w 

Also I wonder how I might contribute my code to EPX? (I did not notice any way to contribute on the site...)



Jeremiah D. Crossett  | Senior Analyst  |  Phase Change Energy Solutions
120 E. Pritchard St.  | Asheboro, NC 27203  | Mobile 503-688-8951
  





On Tue, Oct 29, 2013 at 2:25 PM, Mengsong Zou <mengs...@gmail.com> wrote:
Hi,

This is the first time I've been trying to work with EnergyPlus. I downloaded the source code, but have no idea of how to compile and run it in Linux(Scientific linux). Is there anyone can help me or give me a link to some tutorial?

I looked this page:
http://bigladdersoftware.com/epx/developer-tutorial/build-from-command-line.html
But the method they provide is not working because when I'm doing
cmake -G "Unix Makefiles" -DCMAKE_BUILD_TYPE:STRING=RELEASE ../../SourceCode/

I got an error:
CMake Error: The source directory "~/Research/EnergyPlus/energyplus-8.0.0.008-source/SourceCode" does not appear to contain CMakeLists.txt.

So basically I'm stuck here. Any help is really appreciated!

Thanks.

Best,
Mengsong



On Wednesday, July 18, 2012 4:54:32 AM UTC-4, Clayton Miller wrote:
If you're a student or an academic then check out Intel's student rates - $50-150!:


You can then get Microsoft Visual Studio for free as a student or startup:


On Tuesday, July 17, 2012 10:07:40 PM UTC+8, Neal Kruis wrote:
In the top directory you should find a document titled "SourceReadme.pdf". This provides a brief discussion on compiling EnergyPlus.

Currently the EnergyPlus development team uses Intel Visual Fortran to build their releases. In the future, we hope to post instructions on compiling with a free, open-source compiler such as GFortran.

Neal Kruis

On Tuesday, July 17, 2012 6:26:33 AM UTC-6, Dragan Cvetkovic wrote:
I have a problem with compiling a EnergyPlus opensource code.

Which compiler to use?

--
You received this message because you are subscribed to the Google Groups "EPx Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to epx-developer...@googlegroups.com.

To post to this group, send email to epx-dev...@googlegroups.com.

--
You received this message because you are subscribed to the Google Groups "EPx Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to epx-developer...@googlegroups.com.
To post to this group, send email to epx-dev...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/epx-developers/c1b285a8-8e76-4706-b95b-9a19b0e3c24b%40googlegroups.com.

For more options, visit https://groups.google.com/groups/opt_out.


Peter Ellis

unread,
Feb 9, 2014, 4:09:12 AM2/9/14
to epx-dev...@googlegroups.com

Hi Jeremiah,

Thanks for posting your code. And sorry for the delay. If you would like to contribute code, you need to start by telling us the following:

1. What is the new feature or bug fix that you are addressing?

2. Why is it needed?

3. What testing have you done to verify that the code is correct?

Nonetheless, I have given your code a preliminary review. You have a TON of changes and new code in there. It's hard to tell what you are trying to do. You might want to diff your code against the code released for EnergyPlus 8.1.0.008 (same as the EPx trunk). It might be that your changes were based on an earlier code base, so some of the diffs could actually be reintroducing old code.

Thanks,
P

Jeremiah Crossett

unread,
Feb 9, 2014, 12:39:16 PM2/9/14
to epx-dev...@googlegroups.com
Peter, 
Thanks for the reply.  Quick answers to your questions in line.. Yes this work is based on the code explained in paper I gave you a copy of last time we met in Denver, and presented by the original developer at last years summer ASHRAE conference. I have attached this paper and a couple of model report that where done using this code. 

1. What is the new feature or bug fix that you are addressing?
  • PCM Hysteresis modeling capabilities. 
  • A method to count the physical state of the PCM, very useful for determining PCM performance. 

2. Why is it needed?
  • Some PCM's express hysteresis and thus can not be properly modeled with the single linear curve approach employed in the current version of Energy Plus,  this work solves that issue by providing hysteresis modeling capabilities. 
  • Without a counting system for the physical state of the PCM it is impossible to know how the PCM is performing, the PCM state output solves this issue. 

3. What testing have you done to verify that the code is correct?
  • I have done ASHRAE 140 tests that show the new model to be very close the current model.
  • I have used the new code on a few residential and a couple commercial models that indicate the new code is performing as intended. 
Nonetheless, I have given your code a preliminary review. You have a TON of changes and new code in there. It's hard to tell what you are trying to do. You might want to diff your code against the code released for EnergyPlus 8.1.0.008 (same as the EPx trunk). It might be that your changes were based on an earlier code base, so some of the diffs could actually be reintroducing old code.
  • I have done extensive DIFF using an advanced diff/merge software, it is fully EnergyPlus 8.1.0.008, with the new code added to it.
  • The new code is segmented from the current code by commented text brackets, so !VE START (new code starts below this line)  !VE END (new code ends at this line) and !VE START@xyz (new code starts at  "xyz" text)
  • The only changes to the EnergyPlus 8.1.0.008 code are cleanup of messy/misspelled comments. 

The code may have some issues, or could possibly be further enhanced by other users, however I have been using it for the last few months and it seems to work fine----
Given the open source nature of the Epx project, my assumption is that if the code is contributed to the project that it may be further modified and that it would not need to be 100% perfect in order to be a development branch. If it was a branch, then it would be at the discretion of the users to use it, or modify it as they see fit.  From what I can tell, there have not been any other contributions from Epx users, so I had imagined that this contribution would be welcomed by the group. I expect that I will be able to get this code into the C++ version of EnergyPlus within the next couple of years, and hope that I can contribute this FORTRAN version to Epx now so that other users have the opportunity to use and/or modify it as they see fit. 

Please let me know if you need more from me in order to contribute my work, or if anyone has further questions. 

Kindest of regards 



--
You received this message because you are subscribed to the Google Groups "EPx Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to epx-developer...@googlegroups.com.
To post to this group, send email to epx-dev...@googlegroups.com.
An_Enhanced_Simulation_Model_for_Building_Envelopes_with_Phase_Change_Materials.pdf
BelgianPassivhausVillaWithAdvanced_bioPCM_Envelope.pdf
LEFF-InfoGraphicPoster_24x30_.pdf

Jeremiah Crossett

unread,
Feb 9, 2014, 2:16:25 PM2/9/14
to epx-dev...@googlegroups.com
Peter, 
Probably the most interisting / useful element of my proposed contribution to Epx is the new output for PCM state. 
Below is the IO refrence to this, and time bin results for a typical surface and charts of the same surface. Also attached is an HTML report with PCM state output in time bins, and a CSV with hourly PCM state typical surfaces used in this model.  Hopefully this example helps to illustrate how the output is useful, as I find the output very useful for designing with PCM's, and think that other would as well. 

Inline image 1



Report: CONDFD SURFACE PHASE CHANGE STATE NODE 5 []

For: SOUTH WALL

Timestamp: 2014-01-30 13:17:02

Values in table are in hours.

Time Bin Results

-12345--
Interval Startless than-2.0<=-1.0<=0.00<=1.00<=2.00<= equal to or more thanRow
Interval End-2.0-1.0>0.00>1.00> 2.00>3.00>3.00Total
January0.000.000.00 0.00186.70557.300.00744.00
February0.001.05 1.400.00278.25391.300.00672.00
March0.00 1.201.500.00382.80358.500.00744.00
April 0.0023.1028.700.00449.10219.100.00720.00
May0.0038.2066.300.00486.35153.150.00 744.00
June0.00157.05288.900.00191.2582.80 0.00720.00
July0.00256.45449.900.0024.25 13.400.00744.00
August0.00277.75405.850.00 42.7017.700.00744.00
September0.00157.00202.20 0.00280.1580.650.00720.00
October0.0047.25 72.600.00449.55174.600.00744.00
November0.00 0.000.000.00320.90399.100.00720.00
December 0.000.000.000.00170.25573.750.00744.00
12:01 to 1:00 am0.0080.5511.900.00190.5582.000.00 365.00
1:01 to 2:00 am0.0072.4010.500.00193.0089.10 0.00365.00
2:01 to 3:00 am0.0069.557.300.00 188.7599.400.00365.00
3:01 to 4:00 am0.0067.755.30 0.00178.75113.200.00365.00
4:01 to 5:00 am0.00 62.352.450.00173.45126.750.00365.00
5:01 to 6:00 am 0.0051.154.500.00171.90137.450.00365.00
6:01 to 7:00 am0.0035.8516.150.00172.60140.40 0.00365.00
7:01 to 8:00 am0.007.0544.950.00156.65 156.350.00365.00
8:01 to 9:00 am0.000.7052.75 0.00121.85189.700.00365.00
9:01 to 10:00 am0.000.00 66.400.0052.10246.500.00365.00
10:01 to 11:00 am 0.000.0078.800.0028.85257.350.00365.00
11:01 to 12:00 pm0.000.0097.400.0019.60248.000.00 365.00
12:01 to 1:00 pm0.000.10121.450.0017.95225.50 0.00365.00
1:01 to 2:00 pm0.001.00140.150.00 30.85193.000.00365.00
2:01 to 3:00 pm0.002.80147.40 0.0066.75148.050.00365.00
3:01 to 4:00 pm0.00 3.70153.350.00123.3084.650.00365.00
4:01 to 5:00 pm 0.0016.70142.950.00152.0553.300.00365.00
5:01 to 6:00 pm0.0033.40121.700.00162.1547.75 0.00365.00
6:01 to 7:00 pm0.0043.35104.050.00165.85 51.750.00365.00
7:01 to 8:00 pm0.0053.6082.50 0.00171.3057.600.00365.00
8:01 to 9:00 pm0.0072.85 53.750.00175.9562.450.00365.00
9:01 to 10:00 pm 0.0094.1527.750.00176.9566.150.00365.00
10:01 to 11:00 pm0.0098.7013.450.00182.6570.200.00 365.00
11:01 to 12:00 am0.0091.3510.450.00188.4574.75 0.00365.00
Total0.00959.051517.350.00 3262.253021.350.008760.00


Statistics

Statistic
Minimum -2.0
Mean minus two standard deviations-2.1
Mean0.67
Mean plus two standard deviations 3.44
Maximum2.00
Standard deviation1.38



Inline image 3

Inline image 4

Inline image 3

image.png
image.png
image.png
image.png
BioPCMM27Q23swr_R30r_R19sw_R30ne_PCMOUTPUTSTable.html
BioPCMM27Q23swr_R30r_R19sw_R30neTable.csv

Jeremiah Crossett

unread,
Feb 9, 2014, 2:18:12 PM2/9/14
to epx-dev...@googlegroups.com
Sorry, I attached the wrong csv-- correct one attached here.. 
image.png
image.png
image.png
image.png
BioPCMM27Q23swr_R30r_R19sw_R30ne_PCMOUTPUTS.csv

Peter Ellis

unread,
Feb 11, 2014, 1:00:02 PM2/11/14
to epx-dev...@googlegroups.com

Hi Jeremiah,

I'm afraid it's beyond our current capacity to evaluate so many changes and so much new code. Your changes modify so much of the finite difference solution method that it would require a significant effort to test and ensure the validity of the code.

At this stage, contributions to EPx would ideally be surgical changes such as a neatly delineated bug fix or an isolated new feature that doesn't interfere with any existing functionality, e.g., break the interface or change the answers for other objects. For bigger changes, it would be better to work with you from the beginning to help you define the scope of your modifications.

If you want to discuss further, you can email me directly.

Thanks,
P

Reply all
Reply to author
Forward
0 new messages