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

Error after downloading latest version #50

Closed
bdcoelho opened this issue May 30, 2019 · 2 comments
Closed

Error after downloading latest version #50

bdcoelho opened this issue May 30, 2019 · 2 comments
Assignees
Labels
Milestone

Comments

@bdcoelho
Copy link

bdcoelho commented May 30, 2019

I updated to eplusr version ‘0.10.2’ because of an issue with the model$del() function. It kept telling me that it is not a function. However this is not the main issue (I'll see if it persists after the update). After I updated to the latest version, the idf will not read into R. I get the following error:

model<-read_idf("Test.idf")

IDD v9.0.1 has not been parsed before.
Try to locate Energy+.idd in EnergyPlus v9.0.1 installation folder C:/EnergyPlusV9-0-1.
IDD file found: C:\EnergyPlusV9-0-1\Energy+.idd.
Start parsing...
Parsing completed.
Error in vecseq(f__, len__, if (allow.cartesian || notjoin || !anyDuplicated(f__, :
Join results in 90646 rows; more than 50512 = nrow(x)+nrow(i). Check for duplicate key values in i each of which join to the same group in x over and over again. If that's ok, try by=.EACHI to run j for each group to avoid the large allocation. If you are sure you wish to proceed, rerun with allow.cartesian=TRUE. Otherwise, please search for this error message in the FAQ, Wiki, Stack Overflow and data.table issue tracker for advice.

It doesn't happen with all IDFs but the one I'm using is a working idf (See attached IDF)
Do you have an idea of what it is? After a few attempts with different IDFs, the problem seems to occur with larger IDFs (approximately 90000 lines and over). This issue didn't exist before I updated.

Test.txt

Regards,
Benson

@hongyuanjia hongyuanjia self-assigned this May 30, 2019
@hongyuanjia hongyuanjia added this to the 0.10.3 milestone May 30, 2019
@bdcoelho
Copy link
Author

Quick update: model$del() works in the latest version, so no worries there!

@hongyuanjia
Copy link
Owner

Thanks for reporting.

I can confirm this is a bug. it should be fixed now. Please try to install the development version by remotes::install_github("hongyuanjia/eplusr").

Feel free to reopen it if you still get an error.

By the way, in the development version, if you are using RStudio, auto-completion of class names and field naems should be toggled when typing idf$ and idfobj$ respectively. Please give it a try.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants