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 effective time calibration #11

Closed
dkirkby opened this issue Jul 31, 2023 · 6 comments
Closed

Update effective time calibration #11

dkirkby opened this issue Jul 31, 2023 · 6 comments

Comments

@dkirkby
Copy link
Member

dkirkby commented Jul 31, 2023

From @schlafly:

There has been a fluctuation in the value of EFFTIME_SPEC / EFFTIME_ETC such that we're now averaging something like 0.95.  We stayed at 1 between the two major shutdowns but have been at ~0.95 since then.
...
Let's go ahead and boost the amount of time we get given a particular requested time by 5%.
The goal is that exposure times after this tweak will be 5% longer than they otherwise would have been.
@dkirkby
Copy link
Member Author

dkirkby commented Jul 31, 2023

The simplest way to accomplish this is to adjust the default scale parameter of accum.get_efftime which is currently set to 1.11537 after #4. Multiplying this by 1.05 gives 1.17114.

@schlafly
Copy link

@dkirkby , can you confirm the intended sign here? We wanted to make exposures 5% longer. We have been undercooking tiles since restart; if we applied this with the wrong sign that would go a long way to explaining the current issue.

@dkirkby
Copy link
Member Author

dkirkby commented Aug 11, 2023

Arrgh, I got this backwards and increased EFFTIME_ETC by 5%.

The correct scale value to use is 1.11537 * 0.95 = 1.05960. I have fixed this in the code now and tagged 0.1.16.

@schlafly
Copy link

Thank you! Any idea how the installation in the ICS environment works? Do we need to wait for Klaus's return? Especially challenging at the moment with the network down.

@schlafly
Copy link

Sorry, I see on Slack that you already resolved this; thanks!

@daqiii
Copy link

daqiii commented Aug 11, 2023 via email

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

No branches or pull requests

3 participants