Skip to content
Snippets Groups Projects

[LPD-1M] [Correct] Use parameter names instead of IDs to find constants

Merged Thomas Kluyver requested to merge feat/lpd-lookup-param-names into master

Description

Hardcoding parameter IDs can be error prone, especially when going between production & test CalCat instances. The new ability to use the parameter names makes the intent clearer.

How Has This Been Tested?

Ran notebooks interactively to retrieve constants.

Types of changes

  • Refactor (refactoring code with no functionality changes)

Checklist:

  • My code follows the code style of this project.

Reviewers

@manettim @ahmedk

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading