smiths / aorta

Other
2 stars 0 forks source link

Module guide #53

Closed JovieL25 closed 1 year ago

JovieL25 commented 1 year ago

The newest Module Guide can be merged with the main branch.

The main updates are:

  1. Modified Anticipated Changes.
  2. Modified Modules.
  3. Included Traceability Matrix for Requirements and Modules.
  4. Added mapping between each module and the corresponding code.
  5. Added Use Hierarchy.
smiths commented 1 year ago

M2 is shown using M10, M9 and M11. Is that right? I definitely agree that M2 uses M3. Does M2 have multidimensional arrays?

If you end up editing the uses hierarchy, it would be nice if the modules were written in ascending order in each layer. For instance, for the second layer from the top the modules are currently listed in the order M6, M5, M8, M7, M2. A more logical ordering would be M2, M5, M6, M7, M8. You don't need to bother fixing this, unless you are already editing the figure for another reason.

JovieL25 commented 1 year ago

M2 is shown using M10, M9 and M11. Is that right? I definitely agree that M2 uses M3. Does M2 have multidimensional arrays?

If you end up editing the uses hierarchy, it would be nice if the modules were written in ascending order in each layer. For instance, for the second layer from the top the modules are currently listed in the order M6, M5, M8, M7, M2. A more logical ordering would be M2, M5, M6, M7, M8. You don't need to bother fixing this, unless you are already editing the figure for another reason.

Actually, M2 doesn't use M9, M10, and M11, it just uses M3. I will update the picture after, and I can fix the ordering to make it look nicer.

smiths commented 1 year ago

Great! The way the figure was drawn, it looked like M2 used those other modules. Should be an easy fix. :smile:

JovieL25 commented 1 year ago

Hello Dr. @smiths,

The new use hierarchy is updated on MG. Here is a snapshot: UsesHierarchy