slac207 / cs207project

MIT License
0 stars 3 forks source link

#35 Fixed TimeSeries constructor #39

Closed cocochrane closed 8 years ago

cocochrane commented 8 years ago

Changed the TimeSeries constructor based on our meeting consensus. TimeSeries has the precondition that times is a monotonically increasing sequence and we don't explicitly check for this behavior. We do ensure that times and values are sequences and that they have the same length (if any of these checks fail, a TypeError with message is raised).

Removed sort inside Interpolate function as this is unnecessary.

coveralls commented 8 years ago

Coverage Status

Coverage remained the same at 100.0% when pulling c5fad1ecf12ea65b71e1b974dd7650943e9edecd on cocochrane7 into 35ca5012809302079609780e2126b41d149a90a5 on master.

cocochrane commented 8 years ago

Note: edited out the ArrayTimeSeries doctests because they were not written when we necessitated that times and values be the same length. Will rewrite soon when I rewrite the ArrayTimeSeries class