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
ARMI has some migration scripts from the dawn of time. But after a couple of years of trying, I can't find any examples of these old databases to test and make sure these scripts actually work. (I had to delete a couple that were overtly broken at some point.)
Also, these scripts are fundamentally untestable without example databases to test them on.
For the scripts to remain useful, we need them tested via CI so we can see when they go stale.
My current thinking:
Delete the current scripts.
a. Worry about the future, not the past.
Add a new script (if necessary) every time ARMI releases a new, official version.
Ensure the scripts are fully unit-tested, or they don't count.
What I haven't figured out yet is:
The scripts are currently importable in ARMI. Should they be part of the standard import armi namespace? This seems, I guess, convenient. But I've also never seen migration scripts importable as part of a project. They are usually their own directory of helpful tools.
The text was updated successfully, but these errors were encountered:
ARMI has some migration scripts from the dawn of time. But after a couple of years of trying, I can't find any examples of these old databases to test and make sure these scripts actually work. (I had to delete a couple that were overtly broken at some point.)
Also, these scripts are fundamentally untestable without example databases to test them on.
For the scripts to remain useful, we need them tested via CI so we can see when they go stale.
My current thinking:
a. Worry about the future, not the past.
What I haven't figured out yet is:
import armi
namespace? This seems, I guess, convenient. But I've also never seen migration scripts importable as part of a project. They are usually their own directory of helpful tools.The text was updated successfully, but these errors were encountered: