snoplusuk / echidna

MIT License
4 stars 12 forks source link

Updates limit setting code #49

Closed ashleyrback closed 9 years ago

ashleyrback commented 9 years ago

Includes:

Bug fixes/updates:

New:

ashleyrback commented 9 years ago

@jwaterfield here are the bug fixes, updates and some new features for the limit setting code. I haven't run the unittests or added any new ones yet or checked the docs. I'll do that now and push any changes.

jwaterfield commented 9 years ago

@ashleyrback cheers reviewing now...

ashleyrback commented 9 years ago

Note: unittests currently give

ValueError: The truth value of an array with more than one element is ambiguous. Use a.any() or a.all()

This will be fixed in the next commit that is pushed

ashleyrback commented 9 years ago

@jwaterfield pushed some updates, hopefully this responds to all comments. Let me know if there is anything else.

jwaterfield commented 9 years ago

Merging