-
Notifications
You must be signed in to change notification settings - Fork 197
Scripts: Modify reference warning? #2541
Description
Been contemplating the terminal message that is currently generated upon executing a Python script for which there are citations registered: https://github.com/MRtrix3/mrtrix3/blob/3.0.3/lib/mrtrix3/app.py#L668-L675
In the case where there are MRtrix3 contributor references registered, this message will be issued without the addition of the text indicating that there are external references. This is however not consistent with the MRtrix3 C++ binaries, where there is no such message issued. Further, the fact that such messages are issued so regularly may dilute the importance of this message in cases where there is an external dependency, which is the case where we want to make the most noise in order to not obscure the use of software / methods that are not ours.
Therefore, what I'm thinking is that this message should not be issued if there are only MRtrix3 references. This will "increase the volume" of the issuing of that message when there are external dependencies involved.
Thoughts?