RiverWare Output Canvas: Teacup Storage and Flow Animation: Design
3 Review (8-25-2014)
Phil Weinstein, David Neumann, CADSWES -- See
Project Index
David's review notes for the 8-24-2014 draft of the Design 3 document, and Phil's responses. Some of David's notes were conveyed as hard copy markups.
![]() |
![]() |
![]() |
1 | I don't think we need Text Groups and Image Groups. Add Text and Images directly to the canvas? The group seems superfluous as there are almost no settings at that level and will confuse the user. The only exception is font which could be moved to the Text Item and be similar to the color setting. | |
|
||
David and I agreed to these provisions:
|
||
2 | I'm confused about Flow Line Intervals. They are supposed to show the color based on that slot's value vs the interval. I don't see how that is a group level setting as you've described with the "Flow Interval Specification". A bigger question is how does the user know what a color means? If you have two blue lines, they could be very different flows, so how does the user know? Is there a meaningful legend we could create for Flow lines? These need some more thought... | |
|
||
David and I agreed to keep the document with the currently documented Flow Line design. We may want to revisit this later. | ||
3 | Add Text Item: Generated Date/Time (i.e. put the wall clock date/time on the viewer) seems unnecessary. We don't do that elsewhere within RiverWare output. Maybe add that to the printing/export functionality, like plots. | |
This was in one of the examples / use cases. But we don't need it, I'll remove it. | ||
4 | In section (4) Output Canvas Config, a lot of those file actions should be moved to the viewer dialog or the output manager. I'll give you my marked up version. I think we also need a setting "Show Timestep Controls" to the output configuration. Maybe they always want it at the specified date. The default should be "Y" show controls. | |
These changes seem fine. The markups also suggest elimination of the ability to hide the configuration panels. I'll remove that as well. |
||
5 | The Percent text item should be called "Computed Fraction" and use the unit scheme's settings for "Fraction", either decimal or percent. Showing on Percent with no precision is not general enough. We have a way for the user to express their desired user unit and precision, so let's use it. | |
|
||
We're adopting this change. Other or special provisions can be considered later. |
![]() |
![]() |
![]() |
6 | (2.4) David is suggesting this minor change to the Teacup Legend graphic (position of Current storage label). | |
I've adopted this change (first image changed to second image): |
--- (end) ---