-
Notifications
You must be signed in to change notification settings - Fork 19
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
Fco #51
Fco #51
Conversation
…d unaggregated, so not quite U_Mined yet)
_uschema = [('SimId', ts.UUID), ('QualId', ts.INT), | ||
('TransactionId', ts.INT), ('ResourceId', ts.INT), | ||
('ObjId', ts.INT), ('TimeCreated', ts.INT), | ||
('FCO_U_Mined', ts.DOUBLE)] |
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.
Bad column name. Should be something like FcoUMined
Thanks @scopatz! All changes are in. |
u = u.groupby(level=['TimeCreated']).sum() | ||
u.name = 'FcoUMined' | ||
# sum by years (12 time steps) | ||
u_year = u.groupby(pd.cut(u.index, np.arange(0, 12, 12)))['FcoUMined'].sum() |
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.
@scopatz, here is the gist of my error regarding this line:
https://gist.github.com/opotowsky/71d6c2ea168128fa6045
Can I not do this because u
is a Series?
FYI doing:
u_year = u.groupby(pd.cut(u.index, np.arange(0, 12, 12))).sum()
u_year.name = 'FcoUMined'
returns an empty series
Anyone want to put me through the ringer? I'd like this to inform my near-future metrics development :-) |
Thanks! |
No description provided.