[request] message concept for stdout/stderr --- use 1.1.4+

Bug #697604 reported by RaiMan
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
SikuliX
Fix Released
Undecided
RaiMan

Bug Description

1.1.0: many improvements, but ongoing effort
1.2. new logging concept
----------------------------------------------------------------
Sikuli X all platforms

Especially when running Sikuli scripts in a way, that you have access to stdout/stderr (e.g. using sikuli-script.jar standalone) you find differently styled messages.

This is an example:
Sikuli vision engine loaded.
Text Recgonizer inited.
Mac OS X utilities loaded.
VDictProxy loaded.
EventManager loaded.
[safarihelpers log] imported
PyramidTemplateMatcher: source is smaller than the target
[log] highlight Match[4,26 61x13] score=0,72, target=center for 2.0 secs
2011-01-05 09:34:48.682 java[35565:d07] bringWindowToFront: <CocoaAppWindow: 0x10018c230>
PyramidTemplateMatcher: source is smaller than the target
[log] highlight Match[4,26 61x13] score=0,72, target=center for 2.0 secs
2011-01-05 09:34:50.873 java[35565:d07] bringWindowToFront: <CocoaAppWindow: 0x1006718b0>
[debug] close all ScreenHighlighter

The only user generated message is [safarihelpers log] imported

proposal:
every Sikuli generated message look like
[sikuli type] time-stamp generating-source message

where "type" signals the severity (log, debug, info, warning, error, ...)
time-stamp and generating-source only when needed for whatever purpose

advantages:
- sikuli messages could be filtered out (find messages not starting with [sikuli)
- a user could adopt the systematic and have e.g. a filter for errors (e.g. find error messages with regex ^.*?error\])
- even in the IDE message area it would be easier to identify simple user generated print output

BTW:
the stderr message
PyramidTemplateMatcher: source is smaller than the target
does not really "talk" to a user ;-)
see https://answers.launchpad.net/sikuli/+question/140198

Tags: general
RaiMan (raimund-hocke)
Changed in sikuli:
importance: Undecided → Wishlist
RaiMan (raimund-hocke)
Changed in sikuli:
status: New → In Progress
milestone: none → x1.0-rc2
Revision history for this message
Tsung-Hsiang Chang (vgod) wrote :

In RC2, most messages have a prefix that can be turned off. Some low-level messages in the vision engine will be removed in RC3.

Changed in sikuli:
milestone: x1.0-rc2 → x1.0-rc3
Revision history for this message
Denise Brooks (denisebrooks3) wrote :

Yay! This is finally going to get fixed. Btw, I tried turning off every message that I could, but there is currently no way to turn off the PyramidTemplateMatcher message. I'm watching it scroll across my console right now. I am really looking forward to being able to see my own print output without scrolling through thousands and thousands of PyramidTemplateMatcher messages.

Changed in sikuli:
status: In Progress → Fix Committed
RaiMan (raimund-hocke)
Changed in sikuli:
importance: Wishlist → Low
assignee: nobody → RaiMan (raimund-hocke)
milestone: x1.0-rc3 → x1.0
RaiMan (raimund-hocke)
tags: added: general
RaiMan (raimund-hocke)
Changed in sikuli:
status: Fix Committed → In Progress
milestone: x1.0 → none
importance: Low → Medium
importance: Medium → Wishlist
RaiMan (raimund-hocke)
Changed in sikuli:
importance: Wishlist → High
milestone: none → x1.1
Revision history for this message
RaiMan (raimund-hocke) wrote :

*** already available with SikuliX-1.0

* timestamp
Settings.LogTime = True

each Sikuli message will look like this:

[type (date time)] message text

* log messages to file additionally

* a user log type with options: type-text, timestamp, to own file

RaiMan (raimund-hocke)
Changed in sikuli:
status: In Progress → Fix Committed
RaiMan (raimund-hocke)
Changed in sikuli:
milestone: 1.1.0 → 1.2.0
description: updated
RaiMan (raimund-hocke)
Changed in sikuli:
milestone: 2.0.0 → none
importance: High → Undecided
status: Fix Committed → Won't Fix
status: Won't Fix → Fix Released
summary: - [request] message concept for stdout/sterr
+ [request] message concept for stdout/sterr --- use 1.1.4+
summary: - [request] message concept for stdout/sterr --- use 1.1.4+
+ [request] message concept for stdout/stderr --- use 1.1.4+
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.