CenterForDigitalHumanities / SpectralRTI_Toolkit

Process Spectral RTI Images in ImageJ
GNU General Public License v3.0
1 stars 0 forks source link

Log vs. Console #20

Open thehabes opened 6 years ago

thehabes commented 6 years ago

The plugin has a hard time understanding the difference and the user definitely notices.

thehabes commented 6 years ago

Showing console instead of log as suggested by other ImageJ developers.

thanneken commented 5 years ago

"Starting" shows as a warning, should be info.

thehabes commented 5 years ago

logService.log().warn() is the cheap way of forcing the console to show up without using an ImageJ command. Natively, the scijava.log.LogService package will open the console on warn(), but not on log().

There were issues using the ImageJ commands because it wants to use its imageJ log, which is not the same as the console.

I could investigate another way of doing it. "starting" as a warning was a quick fix for this issue.

thanneken commented 5 years ago

As we move away from debugging to stable usage, we might consider the console optional. Perhaps the documentation could say something like: For information about the processing steps being executed by the plugin open the console (Window>Console).

thanneken commented 5 years ago

Similarly, if we are to imagine a user not wanting the log at all, it's not really a warning if it completes successfully, but information.

[WARNING] End of SpectralRTI Plugin.
[WARNING] Finished processing the run().
thehabes commented 5 years ago

Will do this to pull away from the unnecessary warn()s

thehabes commented 5 years ago

A line of instructions was added to the preference window.

The preference window and task selection window do not allow the user to click the imageJ panel, so they are instructed to do it during processing.

ex

thehabes commented 5 years ago

Changed text to

Tip: Select Window > Console in the ImageJ panel during processing for more information.