PyRate icon indicating copy to clipboard operation
PyRate copied to clipboard

Reduce cropping options

Open adeane-ga opened this issue 3 years ago • 2 comments

Simple Change to clarify usage Currently PyRate has 4 options to crop, would be better if we reduce to 2 because the others are redundant and confusing.

Current Options: # ifgcropopt: 1 = minimum extent 2 = maximum extent 3 = crop 4 = no cropping

Proposed Change: # ifgcropopt: 1 = crop 2 = no cropping

adeane-ga avatar Oct 18 '21 06:10 adeane-ga

I agree with the proposed change, but I would suggest a binary-style switch:

# ifgcropopt: 0 = do not crop; 1 = crop

mcgarth avatar Oct 29 '21 03:10 mcgarth

I agree with the proposed change, but I would suggest a binary-style switch:

# ifgcropopt: 0 = do not crop; 1 = crop

Yeah I agree, makes sense.

adeane-ga avatar Nov 03 '21 01:11 adeane-ga