Skip to content

rm plotting from tests #48

Description

@sneakers-the-rat

part of the purpose of having automated tests is so that one doesn't need to run a bunch of stuff and manually inspect the output to check if it looks right. plotting should be a separate feature of the package, and the tests should make assertions about the functionality of its components, rather than a means of creating inspectable output.

plotting is also super expensive, so it should probably just be removed, but if it is useful as a debugging step, then it should be moved into some decorator or pytest hook that wraps the tests, and then can be enabled when desired with like pytest --with-plots

Activity

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions