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

BPC::Workflows:: feedback from Ewa Delmann #38

Open
2 of 6 tasks
henricasanova opened this issue Sep 10, 2020 · 1 comment
Open
2 of 6 tasks

BPC::Workflows:: feedback from Ewa Delmann #38

henricasanova opened this issue Sep 10, 2020 · 1 comment
Assignees

Comments

@henricasanova
Copy link
Contributor

henricasanova commented Sep 10, 2020

  • The workflow definition is maybe a bit narrow, assuming only file-based communications. TODO: add one of sentence.

  • It would be good to have a search capability across the various modules, and links to definitions if any—for example is the concept of a critical path defined anywhere? At some point I found the glossary, but it did not jump out at me. I think a link with the concept highlighted would be more prominent, also may have more context. TODO: tooltips to glossary definitions from links in the narratives

  • It would be good to describe the goal of the simulations, what are you looking for? Should you run with various params and compare? How would you do this? TODO: see if one explanation sentence can be added for each simulation, and make it clear that practice questions are linked to the simulation.

  • When running simulations, the parameters and the information in the graphs need to be described. The resizing of graphs is also confusing, at one time I did not notice there was more data but the original view did not show it.

    • TODO: zoom disabled by default, and button for enabling it in the graph
    • TODO: add a question mark icon next to the the graph HTML label, which pops up an explanation tooltip
@rafaelfsilva
Copy link
Member

Tooltips can now be added to simulation graphs, by simply adding the tooltip parameter:

{% include partials/host_utilization_graph.html prefixID="io" tooltip="Host utilization graph for I/O simulation" %}

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

No branches or pull requests

2 participants