-
-
Notifications
You must be signed in to change notification settings - Fork 18.5k
BUG: Fixed failure in StataReader when reading variable labels in 117 #7818
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
anyway to include a test file? (small one) |
I haven't added a test yet, but will do. |
@jreback Test added, so if no typos and all pass, should be ready |
@@ -551,6 +554,18 @@ def test_bool_uint(self): | |||
written_and_read_again = written_and_read_again.set_index('index') | |||
tm.assert_frame_equal(written_and_read_again, expected) | |||
|
|||
def test_variable_labels(self): | |||
sr_115 = StataReader(self.dta16_115).variable_labels() | |||
sr_117 = StataReader(self.dta16_115).variable_labels() |
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.
should this not be the other file?
I think it is ready now. |
ok, ping when green |
Strange, a failure in a file I didn't change. |
I think that was fixed |
… files Stata's implementation does not match the online dta file format description. The solution used here is to directly compute the offset rather than reading it from the dta file. If Stata fixes their implementation, the original code can be restored. closes pandas-dev#7816
Rebased |
these look like small test files you created, yes? (which is good) |
BUG: Fixed failure in StataReader when reading variable labels in 117
@bashtage thanks for the fix! |
Stata's implementation does not match the online dta file format description.
The solution used here is to directly compute the offset rather than reading
it from the dta file. If Stata fixes their implementation, the original code
can be restored.
closes #7816