Skip to content

Handle > 160 one shots without skipping#19

Merged
jeanconn merged 2 commits intomasterfrom
large-one-shot
Apr 28, 2026
Merged

Handle > 160 one shots without skipping#19
jeanconn merged 2 commits intomasterfrom
large-one-shot

Conversation

@jeanconn
Copy link
Copy Markdown
Contributor

@jeanconn jeanconn commented Apr 10, 2026

Description

Handle one shots larger than allowed from proseco's get_p_man_err function.
Internally, this just assigns p=0.0001 for > 160 one shots.

Fixes

/proj/sot/ska3/aca/data/aca_weekly_report/logs/aca_weekly_report.log
** ERROR - line 1: Skip 32288 at 2026:098:02:37:06.286.  Error processing
** ERROR - line 2: Skip 30152 at 2026:098:06:08:11.185.  Error processing
** ERROR - line 3: Skip 32288 at 2026:098:02:37:06.286.  Error processing
** ERROR - line 4: Skip 30152 at 2026:098:06:08:11.185.  Error processing

Interface impacts

Testing

Unit tests

  • No unit tests

Functional tests

Ran this locally and it no longer skipped 32288 (183 arcsec one shot after) and 30152 (has the 183 arcsec one shot).

@jeanconn jeanconn requested review from javierggt and taldcroft April 10, 2026 17:41
@jeanconn jeanconn merged commit 626e858 into master Apr 28, 2026
2 checks passed
@jeanconn jeanconn deleted the large-one-shot branch April 28, 2026 11:15
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