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

EnergyPlus simulation crashes above a certain BaseDepth value for GroundHeatTransfer #10847

Open
nadjabaden opened this issue Dec 6, 2024 · 4 comments · May be fixed by #10850
Open

EnergyPlus simulation crashes above a certain BaseDepth value for GroundHeatTransfer #10847

nadjabaden opened this issue Dec 6, 2024 · 4 comments · May be fixed by #10850
Assignees
Labels
AuxiliaryTool Related to an auxiliary tool, not EnergyPlus itself (readvars, preprocessor, ep-launch, etc.) Defect Includes code to repair a defect in EnergyPlus

Comments

@nadjabaden
Copy link

Simulation crashes with a certain basement depth (the maximum is 7.39 m).

Link to issue posted on unmet hours: https://unmethours.com/question/100599/energyplus-simulation-crashes-above-a-certain-basedepth-value-for-groundheattransfer/

Details:

Link to idf file: https://drive.google.com/file/d/1xPwj9DqAPkokQ0jRaxU6_t6uWJM5mAf1/view?usp=drive_link

@jmarrec jmarrec self-assigned this Dec 9, 2024
@jmarrec jmarrec added Defect Includes code to repair a defect in EnergyPlus AuxiliaryTool Related to an auxiliary tool, not EnergyPlus itself (readvars, preprocessor, ep-launch, etc.) labels Dec 9, 2024
jmarrec added a commit to jmarrec/EnergyPlus that referenced this issue Dec 10, 2024
jmarrec added a commit to jmarrec/EnergyPlus that referenced this issue Dec 10, 2024
jmarrec added a commit to jmarrec/EnergyPlus that referenced this issue Dec 10, 2024
fixup! Add a ctest for NREL#10847
jmarrec added a commit to jmarrec/EnergyPlus that referenced this issue Dec 10, 2024
ZFACE is -35:100, positive indicates Below ground (NZBG).
A, B, C,R and X are used in the 1 to NZBG range, so they should be 100 elements not 50 (Note: they are 1-indexed)
@jmarrec
Copy link
Contributor

jmarrec commented Dec 10, 2024

@nadjabaden fixed it in #10850, I ran the Basement utility on it and it took 1h15. unfortunately, I didn't use the same weather file as you, so I can't just give you the results, and I'm not going to waste another 1h15.

@nadjabaden
Copy link
Author

Thanks @jmarrec. So, I'm assuming this should be resolved in the next version of EnergyPlus? Do you know when this will be released?

@jmarrec
Copy link
Contributor

jmarrec commented Dec 11, 2024

I reran it with Paris Montsouris TMYX, here are the results of the basement sim

EPObjects.TXT

Run ExpandObjects on your IDF (C:\EnergyPlus-X-y-z\ExpandObjects, your idf should be named in.idf for it to work...), it'll create an expanded.idf, then append at the end the content of that EPObjects.TXT to that expanded.idf, and you'll have a file that no longer needs basement and can be simulated in E+

@jmarrec
Copy link
Contributor

jmarrec commented Dec 11, 2024

Here is a fully expanded.idf for 23.1.0 for convenience:;

expanded.idf.txt

Change extension back to .idf, not .idf.txt (did that so I github would let me upload it)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AuxiliaryTool Related to an auxiliary tool, not EnergyPlus itself (readvars, preprocessor, ep-launch, etc.) Defect Includes code to repair a defect in EnergyPlus
Projects
None yet
2 participants