-
-
Notifications
You must be signed in to change notification settings - Fork 42
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 in the 5G Vignette - base::split() function leads to compatiblity issue with get_mon_arrivals() #210
Comments
|
Also, please try to determine at which point the error is issued by running just arrivals <- envs %>%
get_mon_arrivals() then arrivals <- envs %>%
get_mon_arrivals() %>%
left_join(rowid_to_column(cases, "replication")) etc. |
My
Error occurs directly here
Output of
Can I get you anything else to help? |
I tried a complete fresh run with the code from here: https://raw.githubusercontent.com/r-simmer/simmer/master/vignettes/includes/5G-1.R Error persists. I assume the code worked some time. So the question is wheather this is specific to my machine and my R Setup or maybe due to a newer R version ? Other ideas or anything I can do to help? Just let me know. |
Thanks, the issue has to do with the names of the elements. The minimal reproducible example would be: library(simmer)
get_mon_arrivals(list(simmer()))
#> [1] name start_time end_time activity_time finished
#> <0 rows> (or 0-length row.names)
get_mon_arrivals(list(some_name=simmer()))
#> Error in (function (x) : unused argument (some_name = <environment>) A workaround until this is fixed would be: arrivals <- unname(envs) %>%
get_mon_arrivals() %>%
... etc. |
I thought in this direction but could not narrow it down. Thanks for the workaround and quick response. Also thumbs up for the package in general. |
Many thanks for the report. Fixed in master. |
I tried to execute the 5G vignette code. Simulation runs fine, but upon retrieval of the monitored arrivals there happens a code error
I think it has something to do how
base::split()
function transform the cases dataframe into a list.Sadly I could not find a solution for this problem.
R Version Information:
The text was updated successfully, but these errors were encountered: