Skip to content
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

#2904 - fixed a bug editing a sorted or filtered datatable #2925

Merged
merged 1 commit into from
Nov 5, 2015

Conversation

dennisobrien
Copy link
Contributor

#2904 - fixed a bug when a table was sorted by a column other than the index then edited.

supersede #2906

This changes the semantics of getItem/setItem to use an offset into the data array, while getField/setField still take an item index. The index and the offset will be the same for a data array that is sorted by the index field and has not been filtered.

…an the index then edited.

This changes the semantics of getItem/setItem to use an offset into the data array, while getField/setField still take an item index.  The index and the offset will be the same for a data array that is sorted by the index field and has not been filtered.
@dennisobrien
Copy link
Contributor Author

I'm confused by this Travis CI build failure. The failure is reported in examples/compat/ggplot/density.py but it's not clear to me how that is connected to the changes I made in bokehjs/src/coffee/widget/data_table.coffee.

I'll try to hunt this down.

>>> Testing /home/travis/build/bokeh/bokeh/examples/compat/ggplot/density.py ...
[WARN] Timeout
[FAIL] failed to load file:///home/travis/build/bokeh/bokeh/examples/compat/ggplot/density.html

@canavandl
Copy link
Contributor

I restarted the Travis build and it passed. It looks like it was a spurious timeout.

LGTM (and thanks for contributing)

@birdsarah
Copy link
Member

Tested locally on the data_tables.py example as per instructions in #2904 - works great.

Thanks for the contribution and thanks for the knowledge that you can edit cells on DataTable - that has blown my mind :D

bryevdv added a commit that referenced this pull request Nov 5, 2015
#2904 - fixed a bug editing a sorted or filtered datatable
@bryevdv bryevdv merged commit e582049 into bokeh:master Nov 5, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants