-
Notifications
You must be signed in to change notification settings - Fork 87
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.
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
Closed #2651: series_test.py
conversion for new framework
#2671
Closed #2651: series_test.py
conversion for new framework
#2671
Conversation
I added some updates to this, marking as a draft until those make it in |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mostly looked at parameterization for better coverage.
ff93492
to
648de5d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Few things still are not all the way there.
a5d8076
to
0165900
Compare
ethan's conversations have been resolved and he's moved on to another project
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Didn't see anything besides Pierce's comments
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All I see is the leftover prob_size parameter
bdc00a2
to
9189f6a
Compare
This PR (Closes #2651) updates
series_test.py
to the new testing framework.*This is just a direct conversion. I would appreciate some help identifying any places where some parameterization could be added.