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

Update run_poe_model.rsc #2

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Update run_poe_model.rsc #2

wants to merge 3 commits into from

Conversation

chiplam
Copy link

@chiplam chiplam commented Aug 6, 2020

Add variable max_iter for the maximum feedback iteration at the beginning of the code (Better readability)
Update macro "CreateBatchFile" to 1. add title to the window 2. correct quotation issue

chiplam added 3 commits August 4, 2020 18:16
Add variable max_iter for the maximum feedback iteration
Update macro "CreateBatchFile"
Replace Java dependent package "xlsx" with non-Java dependent "openxlsx"
Remove hard coded 0.9 factor on first iteration (it is already specified in config input)
Use join statements in calculating hourly processed volume
Correct fatal error under these two situations:
1. When there are initial queues specified in configuration
2. When there are unprocessed vehicle at the end of the day
des_outcomes_df <- Compute_Weighted_Average_Wait_Time(des_outcomes_df, prior_des_outcomes_df, 0.9)
# Chi Ping Lam 7/23/2020
# For the first iteration, the des_outcomes_df should be simply the rsesult of first simulation
# and there is need to compute a weighted average (no previous result!)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(no) need to compute?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It should be computed. Thanks.

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

Successfully merging this pull request may close these issues.

2 participants