Updates for semester S21A

naojsoft / qplan

Other

2 stars 4 forks source link

Updates for semester S21A #28

Closed ejeschke closed 3 years ago

ejeschke commented 3 years ago

accounting time per OB includes overheads
correct scheduling (accounting time for executed OBs)
add DB username/password capability

rdk1024 commented 3 years ago

I tested the qplan new-overhead-time branch and it is working ok. I didn't notice any errors or problems and it does seem to be using the "Total Time" column and the "extra_overhead_factor" in the scheduling calculations. Note that I didn't test with the queue database. That would take a bit more time to test.

ejeschke commented 3 years ago

Note that I didn't test with the queue database.

I just pushed up some changes for that. I'm not sure offhand that it changes anything, since most of these OBs should not be in the database yet. Anyway, thanks for testing and we'll see what the queue team finds.