paulremo / NRPMHandbook

Repository to store and track the development on the ADS/Matrisk project for developing a jupyter book of the NRPM handbook.
Other
1 stars 1 forks source link

Review EEE interactive models #81

Closed paulremo closed 2 years ago

paulremo commented 2 years ago

The EEE interactive models are now on the website and work. The following is a list of persisting issues that should be addressed.

Process factor calculation The tool is quite involved and obviously, a lot of work went into it. Most issues I found are quite minor (see below), but I think the entry point to the tool could be improved.

The landing page is a bit detached from the rest of the tool and there is no way to return to it without reloading the page. This is a bit confusing and it should either be (1) better integrated or (2) completely detached. I would opt for the second options (because it is easier to implement). To this end, I would:

  1. Delete a majority of the introduction (it is anyways accessible later via the "help" button) and focus on the actions the user has to take before starting (i.e., choosing a configuration).
  2. Hide the planet header. To a new user, it is not immediately clear that the planet header is locked on purpose so it seems like something is broken, particularly because you refer to the planets in the third paragraph.
  3. Add a button in the main tool to abort the calculation and return to the landing page.

The load-results functionality is quite powerful and well-implemented, but it should be detached from the typical use-case that is to just start from scratch. Therefore, I would

  1. Let the "Paste answers here" field resize with content. Then you can remove the "Erase" button and turn it into a "Load" button. The load button checks the pasted content and deactivates the field after a successful load.
  2. Give it a dedicated section below the "Start" button. It is confusing for the average user why he should provide values when using the tools for the first time.

Minor issues

Process factor models

I was confused by this during the meeting, but I understand the logic behind deactivating the buttons initially. However, to make things clearer I would:

paulremo commented 2 years ago

Looks great! Thanks 😃