Update sun tracker refresh time #297
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
When running the same schedule with what appear to be the same versions for basically all packages (
scheduler
,scheduler-scripts
,SAT-scan-schedules
,numpy
,ocs
,socs
, etc) there is a difference in when some scans are considered sun-safe between running on site and running on my Mac with the latter being more restrictive by a few minutes. This reduces the refresh rate of the sun safety tracker to one hour instead of 6, which makes the site agree with my Mac.Also makes the gap finding re-use the same sun tracker object when possible. Won't be a big improvement for most cases, but when we run into sun-safety difficulties, this can speed that part of the code up by a large factor.
Example config where this problem occurs: