You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Okay, ARMI is not declaring these runtime first-order dependencies, because (I am guessing) they are also second-order dependencies. But I consider this a failure of documentation:
python-dateutil - used in armi/utils/parsing.py
mpl_toolkits - NVM, this is part of mat-props
Similarly, ARMI is not declaring this first-order test dependence:
nbformat - used in test_historyTracker.py and test_notebooks.py
Lastly, ARMI has two dependencies that I find suspect because we are importing an ENTIRE dependency for one function:
Okay, ARMI is not declaring these runtime first-order dependencies, because (I am guessing) they are also second-order dependencies. But I consider this a failure of documentation:
python-dateutil
- used inarmi/utils/parsing.py
mpl_toolkits- NVM, this is part of mat-propsSimilarly, ARMI is not declaring this first-order test dependence:
nbformat
- used intest_historyTracker.py
andtest_notebooks.py
Lastly, ARMI has two dependencies that I find suspect because we are importing an ENTIRE dependency for one function:
pyDOE
- only used in one placepython-dateutil
- only used once, to read datetime strings slightly more easily.The text was updated successfully, but these errors were encountered: