butler Config merge doesn't with dict despite docstring saying it does.

Description

Ran into problems trying to use it to replace an explicit loop in ctrl_bps code.  Here's a simplified example:

Checklist

Lucidchart Diagrams

Issue Matrix

hide

Activity

Show:
Michelle Gower
February 8, 2021 at 4:39 PM

A question and some comments about the test case.  Approved for merging.

Tim Jenness
February 5, 2021 at 12:21 AM

Trivial fix

Tim Jenness
November 9, 2020 at 9:32 PM

Hopefully that will get you through in the short term. It's definitely fixable to make it look for a Mapping.

Michelle Gower
November 9, 2020 at 9:30 PM

Missing opening paren, but yes that works 

Tim Jenness
November 9, 2020 at 8:25 PM
(edited)

Does it work if you say

?

Done
Pinned fields
Click on the next to a field label to start pinning.

Details

Assignee

Reporter

Labels

Reviewers

Michelle Gower

Story Points

RubinTeam

Components

Checklist

Created November 9, 2020 at 6:45 PM
Updated February 8, 2021 at 5:03 PM
Resolved February 8, 2021 at 5:03 PM