Skip to content

Adapt Roman code to new required data files - #162

Open
ojustino wants to merge 3 commits into
spacetelescope:developfrom
ojustino:roman-opd-integration
Open

Adapt Roman code to new required data files#162
ojustino wants to merge 3 commits into
spacetelescope:developfrom
ojustino:roman-opd-integration

Conversation

@ojustino

@ojustino ojustino commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

Cleans up my test branch with code changes to handle the new structure of the required Roman data from GSFC – OPDs, pupils, and expanded Zernike coefficients.

Let's see if changing DATA_VERSION_MIN in stpsf/__init__.py is enough to enforce the use of the new files as we test this branch. If we merge this branch before the release is ready, we may need to revert this change so others who want to use the develop branch but don't have the files can still run STPSF.

I included @mperrin's quick fix to prevent large slowdowns from the number of Zernike coefficients exceeding the lru_cache's default maximum size, but we could still benefit from either aligning the calculation more with the approach taken in STPSF's JWST models (future task) or by making the R() function from poppy's zernike.py more efficient.

@ojustino ojustino changed the title Roman opd integration Adapt Roman code to new required data files Aug 5, 2026
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.

1 participant