Replies: 2 comments 3 replies
-
I completely agree with you that the package needs to be better organized. Since I have been the main contributor to Chemics, I haven't worried about the structure of the package. My main priority has been to keep adding to it as I have time. But yes, it needs to be better organized to encourage others to contribute and to prevent confusion. Two ideas come to mind on how to organize the package's components. The first is to group the modules (files) into sub packages which are related to a specific type of reactor. For example, functions and modules that relate to a bubbling fluidized bed reactor would be in a bfb_reactor sub package or functions/modules for a fixed bed reactor would be in a fixed_bed sub package. The second idea is to group functions and modules according to their specific area of chemical engineering which is what you have suggested. This approach would probably be more suitable and prevent feature overlap. Currently, I don't think it would be much work to reorganize as long as we agree on a package structure and don't make any more contributions until after everything is refactored. But this should be done sooner rather than later because it will be more work as the package gets bigger. |
Beta Was this translation helpful? Give feedback.
-
Here's my attempt at a better file structure for the Chemics package. The names of the modules in the sub-packages could be simplified as part of the reorganization.
|
Beta Was this translation helpful? Give feedback.
-
Hey.
This might be a bit premature but I'll leave a suggestion on the organization of the package. As the package starts to cover areas of Chemical Engineering other than fluidization/reaction (I am currently focusing in adsorption) it might be useful to create sub-packages. Here is a suggestion:
Also, I'm anticipating some concepts can appear in slightly different forms in different areas of Chem. Eng.. Having them side-by-side or "unorganized" might cause confusion. Just an example: The Reynolds number can often appear with porosity in packed bed problems. However adding a second Reynolds to the same module might be confusing.
I understand this might be a major change so this is mostly just food for though at the moment.
Beta Was this translation helpful? Give feedback.
All reactions