Skip to content

RCAL-1075: use env variable for pickle model in roman_photoz.#24

Merged
mairanteodoro merged 4 commits intospacetelescope:mainfrom
mairanteodoro:RCAL-1075
Jun 25, 2025
Merged

RCAL-1075: use env variable for pickle model in roman_photoz.#24
mairanteodoro merged 4 commits intospacetelescope:mainfrom
mairanteodoro:RCAL-1075

Conversation

@mairanteodoro
Copy link
Collaborator

Resolves RCAL-1075

This PR refactors roman_catalog_process.py to allow the user to provide the path and filename to the pickle model (created by RomanCatalogProcess.create_informer_stage()).

The optional path to the model is provided as an environment variable named INFORMER_MODEL_PATH (otherwise it defaults to LEPHAREWORK), whereas the optional model filename is provided through the model_filename keyword argument during the class instantiation (if not provided, it defaults to roman_model.pkl).

Copy link
Collaborator

@schlafly schlafly left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@mairanteodoro mairanteodoro merged commit cdb28b7 into spacetelescope:main Jun 25, 2025
2 checks passed
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