Kyle Mandli
Kyle Mandli
I imagine `kind=8` is probably the right way to do this even for the `ceiling` call. We really need to address this more robustly so we are not tied to...
I am not sure if we want to rely on this. I would suggest maybe keeping this issue open at least for the time being and point out this workaround...
As I mentioned in a comment, if #5 is looked as a coverage test we may be able to get a better idea of what is and is not used.
Do you know if there's a replacement available? Is the intention that another package should pick up the functionality that we should switch to?
Ok, so we are maybe ok?
At some point I had looked into passing down `file_format` as an argument down the call stack in visclaw but it got a bit too convoluted. Doing that would probably...
I think the major problem here is that there are places where VisClaw will lose this information in the calling sequences. It is definitely fixable, just needs a bit of...
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](https://github.com/clawpack/visclaw/blob/master/src/python/visclaw/plotpages.py#L2995).
I have a suggested fix in [this branch](https://github.com/mandli/visclaw/tree/add-animation-size-opt) that simply adds an option `plotdata.html_movie_width`. This maybe is not ideal as this would be a value used for all movies.
Just added the `bbox_inches` option.