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

size of movie images #175

Open
mjberger opened this issue Mar 13, 2016 · 5 comments
Open

size of movie images #175

mjberger opened this issue Mar 13, 2016 · 5 comments

Comments

@mjberger
Copy link

is set to 500 px in html_writer. But it appears to be initialized at 650 then this variables isn't used. What about using the size of png itself, which has a variable that controls it in setplot.py

@mandli
Copy link
Member

mandli commented Mar 13, 2016

The offending line is

myHTMLwriter=myHTMLWriter(embed_frames=False, frame_dir=os.getcwd(), \
                    add_html=pre_html, frame_width=500,file_names=filenames)

at line 2995 in plotpages.py.

@rjleveque
Copy link
Member

@mjberger: Can you be more specific what you're trying to control? Do you mean the size of the image shown in JSAnimation plots? Can you link to an example?

@mandli
Copy link
Member

mandli commented Feb 24, 2019

I have a suggested fix in this branch that simply adds an option plotdata.html_movie_width. This maybe is not ideal as this would be a value used for all movies.

@rjleveque
Copy link
Member

This seems to work. I suggest we also change to

plt.savefig(fname, bbox_inches='tight')

in line 1052 of frametools.py.

@mandli
Copy link
Member

mandli commented Feb 24, 2019

Just added the bbox_inches option.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants