Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Sign up
Appearance settings

added option to download an image of the most recent plot #494

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

Merged
yankev merged 40 commits into master from download-image-offline
Jun 15, 2016
Merged
Changes from 1 commit
Commits
Show all changes
40 commits
Select commit Hold shift + click to select a range
d2b9880
added option to download an image of the most recent plot
yankev Jun 2, 2016
89c53db
remove an alert, change name of function
yankev Jun 2, 2016
c769ad6
updated docstring
yankev Jun 2, 2016
555d89a
removed unused variable `selected`
yankev Jun 2, 2016
6ff77d6
more docstring updates
yankev Jun 2, 2016
78fc1d7
added the ability to download plots when calling iplot
yankev Jun 3, 2016
341e87f
fixed the inline download method
yankev Jun 3, 2016
17485bc
renamed variables so tests will pass
yankev Jun 3, 2016
8eae6bd
attempt at stopping image download on future notebook openings
yankev Jun 4, 2016
ad728a3
fixed the download on refresh
yankev Jun 4, 2016
b2f9db9
fixing code to fix tests
yankev Jun 4, 2016
bdb4b85
Revert "fixing code to fix tests"
yankev Jun 4, 2016
0effa54
removed redundant return from _plot_html
yankev Jun 4, 2016
3172843
removed unused download arguments
yankev Jun 4, 2016
151f23e
providing functionality to all offline plot calls
yankev Jun 5, 2016
cae796e
pep8 and edit message in confirmation to download image
yankev Jun 5, 2016
8b39241
fixed typo
yankev Jun 5, 2016
e20da3e
ensuring the use of the correct argument names, and proper assignment...
yankev Jun 5, 2016
6abf3f9
added new arguments to docstrings
yankev Jun 5, 2016
cac15c5
changed `download_image` parameter to `image`
yankev Jun 7, 2016
a49d903
changed default value for image_filename
yankev Jun 7, 2016
f7d972c
added warning messages indicating the option to use `py.image`
yankev Jun 7, 2016
4256891
changed default filename for `download_notebook_image`
yankev Jun 7, 2016
d447ae4
add download image examples, update more default values
yankev Jun 7, 2016
bd46038
more docstring fixes and amending examples
yankev Jun 7, 2016
1c7d517
removed `download_notebook_image()` function
yankev Jun 7, 2016
f5baa3e
removed `download_notebook_image` from module init
yankev Jun 7, 2016
dbbad22
merged image and format into one parameter: image
yankev Jun 8, 2016
5bf4dd5
added new logic to deal with the new image parameter (iplot, plot)
yankev Jun 10, 2016
1892ab6
rename some arguments, typos, and formatting
yankev Jun 10, 2016
94a55f9
added confirm message to iplot mode
yankev Jun 10, 2016
6ea4e6d
edit value error
yankev Jun 10, 2016
e60a5dd
put download scripts into a function
yankev Jun 10, 2016
f44ad45
remove old scripts replaced by function calls
yankev Jun 10, 2016
e943120
remove image server warnings
yankev Jun 10, 2016
7b4681a
address PR comments
yankev Jun 14, 2016
0582283
fixed function names, as well as added a better check for the caller ...
yankev Jun 14, 2016
8689112
Revert "fixed function names, as well as added a better check for the...
yankev Jun 14, 2016
cf90fba
renamed calls to `get_image_download_script` and fixed docstring
yankev Jun 14, 2016
4235cd4
fix ValueError message
yankev Jun 14, 2016
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
Prev Previous commit
Next Next commit
remove image server warnings
  • Loading branch information
yankev committed Jun 10, 2016
commit e943120d6838d4f3c9fbd3e2341472b1d31315b5
9 changes: 0 additions & 9 deletions plotly/offline/offline.py
View file Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -286,10 +286,6 @@ def iplot(figure_or_data, show_link=True, link_text='Export to plot.ly',
'following format types: `png`, `jpeg`, '
'`webp`, `svg`')
# if the check passes then download script injection will commence.
warnings.warn('For higher resolution images and more export '
'options, consider making requests to the '
'Plotly image servers. Type help(\'py.image\') '
'for more details.', UserWarning)
# Write script to download image of the plot
Copy link
Contributor

@theengineear theengineear Jun 13, 2016

Choose a reason for hiding this comment

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

🎎 This comment could be cleaner. Also, you might not really need the second line here, it's clear from the code.

yankev reacted with thumbs up emoji
script = image_download_script('iplot').format(format=image,
width=image_width,
Expand Down Expand Up @@ -410,11 +406,6 @@ def plot(figure_or_data,
'following format types: `png`, `jpeg`, '
'`webp`, `svg`')
Copy link
Contributor

@theengineear theengineear Jun 13, 2016

Choose a reason for hiding this comment

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

🐫! Use the actual values of __IMAGE_FORMATS here.

yankev reacted with thumbs up emoji
# if the check passes then download script is injected.
warnings.warn('For higher resolution images and more export '
'options, consider making requests to the '
'Plotly image servers. Type help(\'py.image\') '
'for more details.', UserWarning)

# write the download script:
Copy link
Contributor

@theengineear theengineear Jun 13, 2016

Choose a reason for hiding this comment

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

yankev reacted with thumbs up emoji
script = image_download_script('plot')
script = script.format(format=image,
Expand Down

AltStyle によって変換されたページ (->オリジナル) /