-
Notifications
You must be signed in to change notification settings - Fork 244
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
implement annotate/drop in terms of select_entries #3200
implement annotate/drop in terms of select_entries #3200
Conversation
|
||
ids = [] | ||
for e in exprs: |
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.
It doesn't look like anything in exprs
is getting passed to _select_entries
. I might be missing something, but if not you should add a test that fails, and then fix select_entries
to make it pass.
python/hail/matrixtable.py
Outdated
# need to drop entry fields | ||
m = MatrixTable(m._jvds.dropEntries(entry_fields)) | ||
|
||
|
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.
Why so much space?
good call. thanks for catching that!
* rewrite MatrixTable.selectEntries to take single expr * remove annotateEntriesExpr and dropEntries from scala MatrixTable * python wip * wip * wip * fixed selectEntries bug * cleanup * fix * fix select_entries
* rewrite MatrixTable.selectEntries to take single expr * remove annotateEntriesExpr and dropEntries from scala MatrixTable * python wip * wip * wip * fixed selectEntries bug * cleanup * fix * fix select_entries
* rewrite MatrixTable.selectEntries to take single expr * remove annotateEntriesExpr and dropEntries from scala MatrixTable * python wip * wip * wip * fixed selectEntries bug * cleanup * fix * fix select_entries
No description provided.