Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Operator.inheritors() returns a list rather than a set. #350

Closed
wants to merge 8 commits into from
Closed

Operator.inheritors() returns a list rather than a set. #350

wants to merge 8 commits into from

Commits on Jan 5, 2017

  1. Merge pull request #340 from rhiever/master 

    Fixed #339
    @rhiever
    rhiever committed Jan 5, 2017
    Configuration menu
    Copy the full SHA
    a24454c View commit details
    Browse the repository at this point in the history

Commits on Jan 17, 2017

  1. Update README.md

    @rhiever
    rhiever committed Jan 17, 2017
    Configuration menu
    Copy the full SHA
    736973a View commit details
    Browse the repository at this point in the history

  2. Update support.md

    @rhiever
    rhiever committed Jan 17, 2017
    Configuration menu
    Copy the full SHA
    f06fefe View commit details
    Browse the repository at this point in the history

  3. Update docs

    @rhiever
    rhiever committed Jan 17, 2017
    Configuration menu
    Copy the full SHA
    aa6f673 View commit details
    Browse the repository at this point in the history

Commits on Feb 10, 2017

  1. Operators are contained in a list rather than a set. A set is unorder… 

    …ed and this made that even with the same seed, different operators were chosen.
    @PGijsbers
    PGijsbers committed Feb 10, 2017
    Configuration menu
    Copy the full SHA
    0bb2abb View commit details
    Browse the repository at this point in the history

  2. Added a test which checks if order is indeed the same.

    @PGijsbers
    PGijsbers committed Feb 10, 2017
    Configuration menu
    Copy the full SHA
    59e18ae View commit details
    Browse the repository at this point in the history

  3. Now explicitly sorts the operators. Needed because for some reason it… 

    … did not seem to always keep order anyway on a different setup.
    @PGijsbers
    PGijsbers committed Feb 10, 2017
    Configuration menu
    Copy the full SHA
    a88aa8d View commit details
    Browse the repository at this point in the history

  4. Fixed a typo where I forgot to name the argument. Local tests all pass.

    @PGijsbers
    PGijsbers committed Feb 10, 2017
    Configuration menu
    Copy the full SHA
    36d6a73 View commit details
    Browse the repository at this point in the history