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.

Sign up for GitHub

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

Jump to bottom

$run failed if the eplus path was not installed in default path ( C:/EnergyPlusV8-X-X ) #5

Closed
jywang2016 opened this issue Jan 8, 2018 · 3 comments
Closed

$run failed if the eplus path was not installed in default path ( C:/EnergyPlusV8-X-X ) #5

jywang2016 opened this issue Jan 8, 2018 · 3 comments
Labels
bug

Comments

@jywang2016
Copy link

jywang2016 commented Jan 8, 2018 •
edited
Loading

Hi! The $run function seems to work only if the Eplus is installed in the default path of the eplusr package. For example, the run function failed in my PC with Eplus installed in disk D, although I specified the eplus_home argument. BTW, I'm a Windows user.
@jywang2016 jywang2016 changed the title $run failed when the eplus path is installed in default path ( C:/EnergyPlusV8-X-X ) $run failed if the eplus path was not installed in default path ( C:/EnergyPlusV8-X-X ) Jan 8, 2018
hongyuanjia added a commit that referenced this issue Jan 9, 2018
@hongyuanjia
Fix #5
7f51630
@hongyuanjia
Copy link
Owner

It is a bug in the logic of finding EnergyPlus location in eplusr. It should be fixed now. If not, please let me know.

@jywang2016
Copy link
Author

This bug has been fixed with the latest version.

@hongyuanjia
Copy link
Owner

Great to hear that.

hongyuanjia added a commit that referenced this issue Apr 26, 2018
@hongyuanjia
Change the logic of finding EnergyPlus location. Fix #5
37621b9
hongyuanjia added a commit that referenced this issue Apr 26, 2018
@hongyuanjia
Fix #5
e6b0258
hongyuanjia added a commit that referenced this issue Apr 29, 2018
@hongyuanjia
Change the logic of finding EnergyPlus location. Fix #5
5842bb0
hongyuanjia added a commit that referenced this issue Apr 29, 2018
@hongyuanjia
Fix #5
9e73d93
hongyuanjia added a commit that referenced this issue Jul 4, 2018
@hongyuanjia
Change the logic of finding EnergyPlus location. Fix #5
7eddc54
hongyuanjia added a commit that referenced this issue Jul 4, 2018
@hongyuanjia
Fix #5
0dc489b
hongyuanjia added a commit that referenced this issue Aug 16, 2018
@hongyuanjia
Change the logic of finding EnergyPlus location. Fix #5
f5fc953
hongyuanjia added a commit that referenced this issue Aug 16, 2018
@hongyuanjia
Change the logic of finding EnergyPlus location. Fix #5
71436b2
hongyuanjia added a commit that referenced this issue Aug 16, 2018
@hongyuanjia
Fix #5
69e3ab8
hongyuanjia added a commit that referenced this issue Aug 16, 2018
@hongyuanjia
Fix #5
860d614
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@hongyuanjia @jywang2016