search

clawpack / visclaw

Clawpack visualization tools
http://www.clawpack.org
BSD 3-Clause "New" or "Revised" License
29 stars 48 forks source link

Frametools imports #205

Closed ketch closed 7 years ago

ketch commented 7 years ago

This is intended to replace #204. It moves almost all the imports in frametools to the top of the file, avoiding the reload issue discussed there. One import (plotpages) is left in a function because putting it at the top throws an exception due to a circular import. That circular import could be resolved in a better way by making set_show() a method of ClawPlotData(). I didn't do that here since it is really a separate issue and deserves its own PR.

ketch commented 7 years ago

I have tested this for PyClaw applications. Please test it for other applications before merging.

mandli commented 7 years ago

I just tested this for GeoClaw, AMRClaw, and Classic and everything seems to be working.