Skip to content
This repository was archived by the owner on Oct 15, 2025. It is now read-only.
This repository was archived by the owner on Oct 15, 2025. It is now read-only.

Reduce cropping options #364

Open
Open

Description

@adeane-ga

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

Activity

  1. mcgarth commented on Oct 29, 2021

    @mcgarth
    Contributor

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

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

  2. adeane-ga commented on Nov 3, 2021

    @adeane-ga
    ContributorAuthor

    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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions