Skip to content

Exclude streak map errors#80

Merged
jeanconn merged 1 commit intomasterfrom
streak
Jan 8, 2026
Merged

Exclude streak map errors#80
jeanconn merged 1 commit intomasterfrom
streak

Conversation

@jeanconn
Copy link
Contributor

@jeanconn jeanconn commented Jan 7, 2026

Description

Exclude astromon errors like "make_images: 29999 acis_streak_map failed" .

According to the developer, these errors are non-impacting and will be removed in a future release.

Interface impacts

Testing

Unit tests

  • Linux
(ska3-latest) jeanconn-kady> pytest
=============================================================================== test session starts ================================================================================
platform linux -- Python 3.12.8, pytest-8.3.4, pluggy-1.5.0
rootdir: /proj/sot/ska/jeanproj/git
configfile: pytest.ini
plugins: anyio-4.7.0, timeout-2.3.1
collected 9 items                                                                                                                                                                  

jobwatch/tests/test_jobwatch.py .........                                                                                                                                    [100%]

================================================================================ 9 passed in 1.33

Independent check of unit tests by [REVIEWER NAME]

  • [PLATFORM]:

Functional tests

No functional testing.

Copy link
Contributor

@javierggt javierggt left a comment

Choose a reason for hiding this comment

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

I'm ok with this. Maybe we could remove it once the issue is fixed in astromon.

@jeanconn jeanconn merged commit 907b93a into master Jan 8, 2026
1 check failed
@jeanconn jeanconn deleted the streak branch January 8, 2026 00:59
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