-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
DOC: GH3105 better error message on ndarray construction #3131
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
Conversation
I never liked that error message, will be glad to see it die. Most likely I am missing something, but are these tests asserting anything? Should each test include something similar to:
. (I'm afraid I can't help mentioning):
|
yah...prob should be asserting.... what' s the 2nd part of your message? |
Looks like python3 doesn't give .message attribute to ValueError... hmmm. Second part is simply that: |
try:
raise ValueError("foo")
except (Exception), detail:
assert(str(detail).startswith("foo")) |
@hayd Andy on a side note, you are very patient with people on stackoverflow! |
@jreback Thanks! I try not to think about it... :) |
anymore comments on this....merging soon... |
looks fine to me. |
CLN: refactor to put all block manager creation calls into internals.py so to wrap with invalid constructions TST: now py3 compatible
DOC: GH3105 better error message on ndarray construction
@jreback I have only just looked at the google groups page... right back at you! |
TOTD |
closes #3105