Tickets/dm 13345 #179
Merged
Tickets/dm 13345 #179
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| calcErrorFromInputVariance = pexConfig.Field( | ||
| dtype=bool, | ||
| doc="Calculate coadd variance from input variance by stacking statistic." | ||
| "Passed to StatisticsControl.setCalcErrorFromInputVariance()", |
There was a problem hiding this comment.
Does this work? Two strings on consecutive lines without any explicit concatenation?
There was a problem hiding this comment.
Contained within the function parens, yes.
There was a problem hiding this comment.
Interesting! Never seen that before.
* Use calcErrorFromInputVariance=False to calc template variance plane * Use doScaleWarpVariance=True to rescale warp variance using empirical noise to improve detection the difference between the two.
AssembleCoadd's config doWrite exists to prevent data duplication when coadd calexps are stored too. The only config that should control whether an NImage is created and stored is doNImage.
yalsayyad
force-pushed
the
tickets/DM-13345
branch
from
d017d16
to
b40d293
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.