Closed jsiirola closed 5 years ago
Merging #45 into master will increase coverage by
0.01%
. The diff coverage is100%
.
@@ Coverage Diff @@
## master #45 +/- ##
==========================================
+ Coverage 64.92% 64.94% +0.01%
==========================================
Files 87 87
Lines 8725 8726 +1
==========================================
+ Hits 5665 5667 +2
+ Misses 3060 3059 -1
Impacted Files | Coverage Δ | |
---|---|---|
pyutilib/misc/config.py | 94.96% <100%> (ø) |
:arrow_up: |
pyutilib/misc/comparison.py | 88.56% <0%> (+0.36%) |
:arrow_up: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update ea05379...f910407. Read the comment docs.
Summary/Motivation:
When using ConfigBlocks for configuring complex objects (like solvers in Pyomo), it becomes convenient to create temporary ConfigBlocks for holding transient options. We do that by calling the ConfigBlock. The catch is that if the block permits implicit keys, then we will want to preserve those implicit keys.
This PR adds a new option to
__call__
to cause the copy to preserve all implicitly-defined keys.Changes proposed in this PR:
preserve_implicit
option toConfigBase.__call__()
Legal Acknowledgement
By contributing to this software project, I agree to the following terms and conditions for my contribution: