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.

Sign up for GitHub

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

Bokeh plotter #714

Merged
merged 16 commits into from
Nov 9, 2018
Merged

Bokeh plotter #714

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
7565a41
Created the bokeh/file_viewer tool
watsonjj Apr 20, 2018
a53dd0f
Merge branch 'master' into bokeh_plotter
watsonjj Apr 20, 2018
3a94748
Fixed formatting
watsonjj Apr 20, 2018
d6f08e8
Moved run to inside function (fixes tests)
watsonjj Apr 20, 2018
b3c6737
Replaced the rgbtohex C extension with @dneise's Python solution
watsonjj Apr 23, 2018
9709fdf
Corrected references to the old num_samples Field in inst
watsonjj Apr 23, 2018
dc2ee09
Further corrections to n_chan
watsonjj Apr 23, 2018
ff6b0b1
Improvements to setting view as suggested by @dneise
watsonjj Apr 23, 2018
31c39a2
Improved method to execute file_viewer
watsonjj Apr 23, 2018
8188310
Fixed selection of extractor
watsonjj Nov 9, 2018
0b389db
Fixed selection of pixels to work with bokeh > 1.0
watsonjj Nov 9, 2018
40c53da
Changed pixel shape from rectangular to circular to better display al…
watsonjj Nov 9, 2018
7612656
Merge branch 'master' into bokeh_plotter
watsonjj Nov 9, 2018
18da2b9
Corrected tests to use example_event
watsonjj Nov 9, 2018
a3e0d21
Corrected bokeh version
watsonjj Nov 9, 2018
b91a275
Merge branch 'master' into bokeh_plotter
kosack Nov 9, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 10 additions & 2 deletions ctapipe/tools/bokeh/file_viewer/main.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -392,5 +392,13 @@ def on_dl1_widget_change(self, _, __, ___):
self._updating_dl1 = False


exe = BokehFileViewer()
exe.run()
def main():
exe = BokehFileViewer()
exe.run()


if 'bk_script' in __name__:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is very unusual. What is this for?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To run a bokeh server you have to execute bokeh serve on the script instead of python. When you do this, __name__ equals something like "bk_script_dc88af7ef51b4e0d85854896dd662ed0", where the third term seems to be random.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

However, there may be a way to avoid having to use bokeh serve if I can understand how to utilise this: http://bokeh.pydata.org/en/latest/docs/user_guide/server.html#embedding-bokeh-server-as-a-library

main()

if __name__ == '__main__':
main()