Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

jwst/no-vars-fallback-to-latest #1106

Conversation

alphasentaurii
Copy link
Collaborator

@alphasentaurii alphasentaurii commented Jan 14, 2025

Resolves CCD-1540 and JP-3838

This PR addresses a bug introduced by the recent cal version-based context changes where users internal to the STScI network who have no CRDS environment variables configured are no longer able to retrieve the default context. The client will now fallback to the "latest" context if all else fails, and also default to latest if crds determines a local configuration (which occurs automatically when no server url is preset). A new test as well as a note in the relevant documentation has been added. Note - it is possible to retrieve the cached "build_context" variable but this is not necessarily reliable since the user may have updated their jwst cal version since the last crds sync took place.

Tasks

  • update or add relevant tests
  • update relevant docstrings and / or docs/ page
  • Does this PR change any API used downstream? (if not, label with no-changelog-entry-needed)
    • write news fragment(s) in changes/: echo "changed something" > changes/<PR#>.<changetype>.rst (see below for change types)
news fragment change types...
  • changes/<PR#>.hst.rst: HST reference files
  • changes/<PR#>.jwst.rst: JWST reference files
  • changes/<PR#>.roman.rst: Roman reference files
  • changes/<PR#>.doc.rst: documentation change
  • changes/<PR#>.testing.rst: change to tests or test automation
  • changes/<PR#>.general.rst: infrastructure or miscellaneous change

@alphasentaurii alphasentaurii requested a review from a team as a code owner January 14, 2025 20:22
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