Bugs item #3438752, was opened at 2011-11-16 04:37
Message generated for change (Tracker Item Submitted) made by zmoelnig
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478070&aid=3438752&group_…
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: puredata
Group: None
Status: Open
Resolution: None
Priority: 7
Private: No
Submitted By: IOhannes m zmölnig (zmoelnig)
Assigned to: Miller Puckette (millerpuckette)
Summary: "pd -rt" crashes if pd-watchdog cannot be found
Initial Comment:
i'm using Pd-0.43 (today's git) on linux (debian wheezy/sid which comes with gcc-4.6.1)
if i start Pd in "realtime" mode (which is the default on linux) and it cannot find the "pd-watchdog" binary, Pd quits.
i stumbled upon this bug, since i had no link from Pd/src/ to Pd/bin/, and started Pd from it's source directory.
while i agree that my setup was somewhat broken as the pd-watchdog should have been in place, i strongly believe that Pd must not crash in these circumstances. it probably should drop realtime execution if it cannot find pd-watchdog, rather than crash.
zmoelnig@ferrari:~/src/git/pure-data$ ls bin/pd-watchdog
ls: cannot access bin/pd-watchdog: No such file or directory
zmoelnig@ferrari:~/src/git/pure-data$ ./src/pd
priority 6 scheduling enabled.
priority 8 scheduling enabled.
sh: ./bin/pd-watchdog: No such file or directory
pd: watchdog process died
zmoelnig@ferrari:~/src/git/pure-data$ pdsend errorname: >>error writing "sock8": broken pipe<<
zmoelnig@ferrari:~/src/git/pure-data$
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478070&aid=3438752&group_…
Patches item #3438338, was opened at 2011-11-15 07:17
Message generated for change (Comment added) made by katjav
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3438338&group_…
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: puredata
Group: None
Status: Open
Resolution: None
Priority: 7
Private: No
Submitted By: Katja (katjav)
Assigned to: Miller Puckette (millerpuckette)
Summary: change float to t_float in Pd 'extra'
Initial Comment:
0001-change-float-to-t_float-in-Pd-extra.patch converts instances of 'float' to 't_float' in all classes of vanilla Pd's 'extra'. Also, a few constants and literals are defined with more decimals, and some float suffixes removed. The patch is intended for Pd-043.1test5.
Type 't_float' is already used in most of pd core code. The changes in the 'extra' classes would make them compliant with the core in this sense.
----------------------------------------------------------------------
>Comment By: Katja (katjav)
Date: 2011-11-16 02:28
Message:
I've checked the case of bonk~ once more because the difference was
remarkable there (that is, in terms of a test situation). It is fully due
to a more precise definition of pi, 3.141592653589793 instead of 3.14159.
Since a literal without float suffix is a double anyway, 3.14159 for pi is
a waste of precision even when t_float is float. And even when this waste
is considered unimportant, a more precise definition can not spoil the
intended calculation result.
So to answer IOhannes' question: the differences can be considered an
improvement (albeit probably an unimportant one) of the situation. In
practice, no one will be dependent on an exact output of bonk~, it is even
quite hard to create a reproducible test situation. The same holds for
fiddle~ and sigmund~. In this sense, the patch can be safely applied as
is.
By the way, the test patches can be opened to see results in detail,
they're not dependent on a script to run them.
----------------------------------------------------------------------
Comment By: IOhannes m zmölnig (zmoelnig)
Date: 2011-11-16 00:41
Message:
i guess the question is: are the differences an _improvment_ of the
situation (e.g. because [bonk~] historically produced slightly "off" values
and the patch now fixes that)?
in this case i think one could argue, that your patch is fixing a minor
bug, and should be applied, even if it breaks 100% compatibility (and then:
if somebody relies on an exact value output in float-domain, they should
re-think their patch anyhow)
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2011-11-15 21:14
Message:
That answers my somewhat stupid question: your unittest patch will show the
differences.
----------------------------------------------------------------------
Comment By: Katja (katjav)
Date: 2011-11-15 11:50
Message:
The more precise constants and removal of float suffixes originate from my
attempts to make Pd work in single and double precision. Maybe I should
make a patch file without these changes for the moment, till I've reviewed
them better.
Hans I do not understand your question 'how can we see what those
differences are?' If you build a patched Pd-0.43.1test5 and run the tests
which are also attached below, you'll see that the differences are in the
analysis objects, notably bonk~.
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2011-11-15 09:20
Message:
These changes would complete the t_float/t_sample clean up that happened
with the core back in 0.42 (I think). I think it should be included in
0.43 if possible.
Katya, how can we see what those differences are?
----------------------------------------------------------------------
Comment By: Katja (katjav)
Date: 2011-11-15 07:36
Message:
correction: the reference for the unit test patches was pd-0.43.0 vanilla
release
----------------------------------------------------------------------
Comment By: Katja (katjav)
Date: 2011-11-15 07:33
Message:
Accompanying the patch file, I've produced unit test patches. The reference
was pd-043.1 vanilla release. The tests indicate slightly altered output
for some of the classes, probably due to more precise constants and removal
of float suffixes. See attached 'unittests-pdextras.zip'.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3438338&group_…
Patches item #3438338, was opened at 2011-11-15 07:17
Message generated for change (Comment added) made by zmoelnig
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3438338&group_…
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: puredata
Group: None
Status: Open
Resolution: None
Priority: 7
Private: No
Submitted By: Katja (katjav)
Assigned to: Miller Puckette (millerpuckette)
Summary: change float to t_float in Pd 'extra'
Initial Comment:
0001-change-float-to-t_float-in-Pd-extra.patch converts instances of 'float' to 't_float' in all classes of vanilla Pd's 'extra'. Also, a few constants and literals are defined with more decimals, and some float suffixes removed. The patch is intended for Pd-043.1test5.
Type 't_float' is already used in most of pd core code. The changes in the 'extra' classes would make them compliant with the core in this sense.
----------------------------------------------------------------------
>Comment By: IOhannes m zmölnig (zmoelnig)
Date: 2011-11-16 00:41
Message:
i guess the question is: are the differences an _improvment_ of the
situation (e.g. because [bonk~] historically produced slightly "off" values
and the patch now fixes that)?
in this case i think one could argue, that your patch is fixing a minor
bug, and should be applied, even if it breaks 100% compatibility (and then:
if somebody relies on an exact value output in float-domain, they should
re-think their patch anyhow)
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2011-11-15 21:14
Message:
That answers my somewhat stupid question: your unittest patch will show the
differences.
----------------------------------------------------------------------
Comment By: Katja (katjav)
Date: 2011-11-15 11:50
Message:
The more precise constants and removal of float suffixes originate from my
attempts to make Pd work in single and double precision. Maybe I should
make a patch file without these changes for the moment, till I've reviewed
them better.
Hans I do not understand your question 'how can we see what those
differences are?' If you build a patched Pd-0.43.1test5 and run the tests
which are also attached below, you'll see that the differences are in the
analysis objects, notably bonk~.
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2011-11-15 09:20
Message:
These changes would complete the t_float/t_sample clean up that happened
with the core back in 0.42 (I think). I think it should be included in
0.43 if possible.
Katya, how can we see what those differences are?
----------------------------------------------------------------------
Comment By: Katja (katjav)
Date: 2011-11-15 07:36
Message:
correction: the reference for the unit test patches was pd-0.43.0 vanilla
release
----------------------------------------------------------------------
Comment By: Katja (katjav)
Date: 2011-11-15 07:33
Message:
Accompanying the patch file, I've produced unit test patches. The reference
was pd-043.1 vanilla release. The tests indicate slightly altered output
for some of the classes, probably due to more precise constants and removal
of float suffixes. See attached 'unittests-pdextras.zip'.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3438338&group_…
Patches item #3438338, was opened at 2011-11-15 07:17
Message generated for change (Comment added) made by eighthave
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3438338&group_…
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: puredata
Group: None
Status: Open
Resolution: None
Priority: 7
Private: No
Submitted By: Katja (katjav)
Assigned to: Miller Puckette (millerpuckette)
Summary: change float to t_float in Pd 'extra'
Initial Comment:
0001-change-float-to-t_float-in-Pd-extra.patch converts instances of 'float' to 't_float' in all classes of vanilla Pd's 'extra'. Also, a few constants and literals are defined with more decimals, and some float suffixes removed. The patch is intended for Pd-043.1test5.
Type 't_float' is already used in most of pd core code. The changes in the 'extra' classes would make them compliant with the core in this sense.
----------------------------------------------------------------------
>Comment By: Hans-Christoph Steiner (eighthave)
Date: 2011-11-15 21:14
Message:
That answers my somewhat stupid question: your unittest patch will show the
differences.
----------------------------------------------------------------------
Comment By: Katja (katjav)
Date: 2011-11-15 11:50
Message:
The more precise constants and removal of float suffixes originate from my
attempts to make Pd work in single and double precision. Maybe I should
make a patch file without these changes for the moment, till I've reviewed
them better.
Hans I do not understand your question 'how can we see what those
differences are?' If you build a patched Pd-0.43.1test5 and run the tests
which are also attached below, you'll see that the differences are in the
analysis objects, notably bonk~.
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2011-11-15 09:20
Message:
These changes would complete the t_float/t_sample clean up that happened
with the core back in 0.42 (I think). I think it should be included in
0.43 if possible.
Katya, how can we see what those differences are?
----------------------------------------------------------------------
Comment By: Katja (katjav)
Date: 2011-11-15 07:36
Message:
correction: the reference for the unit test patches was pd-0.43.0 vanilla
release
----------------------------------------------------------------------
Comment By: Katja (katjav)
Date: 2011-11-15 07:33
Message:
Accompanying the patch file, I've produced unit test patches. The reference
was pd-043.1 vanilla release. The tests indicate slightly altered output
for some of the classes, probably due to more precise constants and removal
of float suffixes. See attached 'unittests-pdextras.zip'.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3438338&group_…
Patches item #3438338, was opened at 2011-11-15 07:17
Message generated for change (Comment added) made by katjav
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3438338&group_…
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: puredata
Group: None
Status: Open
Resolution: None
Priority: 7
Private: No
Submitted By: Katja (katjav)
Assigned to: Miller Puckette (millerpuckette)
Summary: change float to t_float in Pd 'extra'
Initial Comment:
0001-change-float-to-t_float-in-Pd-extra.patch converts instances of 'float' to 't_float' in all classes of vanilla Pd's 'extra'. Also, a few constants and literals are defined with more decimals, and some float suffixes removed. The patch is intended for Pd-043.1test5.
Type 't_float' is already used in most of pd core code. The changes in the 'extra' classes would make them compliant with the core in this sense.
----------------------------------------------------------------------
>Comment By: Katja (katjav)
Date: 2011-11-15 11:50
Message:
The more precise constants and removal of float suffixes originate from my
attempts to make Pd work in single and double precision. Maybe I should
make a patch file without these changes for the moment, till I've reviewed
them better.
Hans I do not understand your question 'how can we see what those
differences are?' If you build a patched Pd-0.43.1test5 and run the tests
which are also attached below, you'll see that the differences are in the
analysis objects, notably bonk~.
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2011-11-15 09:20
Message:
These changes would complete the t_float/t_sample clean up that happened
with the core back in 0.42 (I think). I think it should be included in
0.43 if possible.
Katya, how can we see what those differences are?
----------------------------------------------------------------------
Comment By: Katja (katjav)
Date: 2011-11-15 07:36
Message:
correction: the reference for the unit test patches was pd-0.43.0 vanilla
release
----------------------------------------------------------------------
Comment By: Katja (katjav)
Date: 2011-11-15 07:33
Message:
Accompanying the patch file, I've produced unit test patches. The reference
was pd-043.1 vanilla release. The tests indicate slightly altered output
for some of the classes, probably due to more precise constants and removal
of float suffixes. See attached 'unittests-pdextras.zip'.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3438338&group_…
Patches item #3438338, was opened at 2011-11-15 07:17
Message generated for change (Comment added) made by eighthave
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3438338&group_…
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: puredata
Group: None
Status: Open
Resolution: None
>Priority: 7
Private: No
Submitted By: Katja (katjav)
Assigned to: Miller Puckette (millerpuckette)
Summary: change float to t_float in Pd 'extra'
Initial Comment:
0001-change-float-to-t_float-in-Pd-extra.patch converts instances of 'float' to 't_float' in all classes of vanilla Pd's 'extra'. Also, a few constants and literals are defined with more decimals, and some float suffixes removed. The patch is intended for Pd-043.1test5.
Type 't_float' is already used in most of pd core code. The changes in the 'extra' classes would make them compliant with the core in this sense.
----------------------------------------------------------------------
>Comment By: Hans-Christoph Steiner (eighthave)
Date: 2011-11-15 09:20
Message:
These changes would complete the t_float/t_sample clean up that happened
with the core back in 0.42 (I think). I think it should be included in
0.43 if possible.
Katya, how can we see what those differences are?
----------------------------------------------------------------------
Comment By: Katja (katjav)
Date: 2011-11-15 07:36
Message:
correction: the reference for the unit test patches was pd-0.43.0 vanilla
release
----------------------------------------------------------------------
Comment By: Katja (katjav)
Date: 2011-11-15 07:33
Message:
Accompanying the patch file, I've produced unit test patches. The reference
was pd-043.1 vanilla release. The tests indicate slightly altered output
for some of the classes, probably due to more precise constants and removal
of float suffixes. See attached 'unittests-pdextras.zip'.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3438338&group_…
Bugs item #3438392, was opened at 2011-11-15 09:13
Message generated for change (Tracker Item Submitted) made by eighthave
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478070&aid=3438392&group_…
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Resolution: None
Priority: 6
Private: No
Submitted By: Hans-Christoph Steiner (eighthave)
Assigned to: mescalinum (mescalinum)
Summary: [tclpd] opening tclfile help patches crashes pd-extended
Initial Comment:
When I launch Pd-extended 0.43 2011-11-15 32-bit on Mac OS X 10.6.8/Intel, then open delete-help.pd, exists-help.pd, etc. I get this error from Tcl, then Pd-extended quits:
(Tcl) INVALID COMMAND NAME: invalid command name "Pt_Start()"
while executing
"Pt_Start() called"
("uplevel" body line 1)
invoked from within
"uplevel #0 $cmd_from_pd"
This crash seems to really be a Tcl panic triggering a quit. Ideally, tclpd would catch the panic and just have the (Tcl) dump to the Pd window. Here's the backtrace:
#0 0x910a4c5a in __kill ()
#1 0x910a4c4c in kill$UNIX2003 ()
#2 0x911375a5 in raise ()
#3 0x9114d6e4 in abort ()
#4 0x0188045f in Tcl_PanicVA ()
#5 0x0188047a in Tcl_Panic ()
#6 0x0185fc57 in Tcl_RegisterChannel ()
#7 0x01869263 in Tcl_CreateChannel ()
#8 0x0180ee08 in Tcl_ListMathFuncs ()
#9 0x0184c44e in TclStackAlloc ()
#10 0x0188c6e3 in TclObjInterpProcCore ()
#11 0x0180ee08 in Tcl_ListMathFuncs ()
#12 0x0184c44e in TclStackAlloc ()
#13 0x0188c6e3 in TclObjInterpProcCore ()
#14 0x0180ee08 in Tcl_ListMathFuncs ()
#15 0x0184c44e in TclStackAlloc ()
#16 0x0188c6e3 in TclObjInterpProcCore ()
#17 0x0180ee08 in Tcl_ListMathFuncs ()
#18 0x0180f191 in Tcl_ListMathFuncs ()
#19 0x01811b09 in Tcl_GetVersion ()
#20 0x01811eb1 in Tcl_EvalEx ()
#21 0x01887bc9 in TclFreePackageInfo ()
#22 0x01887dea in Tcl_PkgRequireProc ()
#23 0x01888a2f in Tcl_PkgProvide ()
#24 0x0180ee08 in Tcl_ListMathFuncs ()
#25 0x01811b09 in Tcl_GetVersion ()
#26 0x01811eb1 in Tcl_EvalEx ()
#27 0x0186fe5a in Tcl_FSEvalFileEx ()
#28 0x01870dce in Tcl_EvalFile ()
#29 0x0077f39e in tclpd_do_load_lib ()
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478070&aid=3438392&group_…
Patches item #3438338, was opened at 2011-11-15 07:17
Message generated for change (Comment added) made by katjav
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3438338&group_…
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: puredata
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Katja (katjav)
Assigned to: Miller Puckette (millerpuckette)
Summary: change float to t_float in Pd 'extra'
Initial Comment:
0001-change-float-to-t_float-in-Pd-extra.patch converts instances of 'float' to 't_float' in all classes of vanilla Pd's 'extra'. Also, a few constants and literals are defined with more decimals, and some float suffixes removed. The patch is intended for Pd-043.1test5.
Type 't_float' is already used in most of pd core code. The changes in the 'extra' classes would make them compliant with the core in this sense.
----------------------------------------------------------------------
Comment By: Katja (katjav)
Date: 2011-11-15 07:36
Message:
correction: the reference for the unit test patches was pd-0.43.0 vanilla
release
----------------------------------------------------------------------
Comment By: Katja (katjav)
Date: 2011-11-15 07:33
Message:
Accompanying the patch file, I've produced unit test patches. The reference
was pd-043.1 vanilla release. The tests indicate slightly altered output
for some of the classes, probably due to more precise constants and removal
of float suffixes. See attached 'unittests-pdextras.zip'.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3438338&group_…
Patches item #3438338, was opened at 2011-11-15 07:17
Message generated for change (Comment added) made by katjav
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3438338&group_…
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: puredata
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Katja (katjav)
Assigned to: Miller Puckette (millerpuckette)
Summary: change float to t_float in Pd 'extra'
Initial Comment:
0001-change-float-to-t_float-in-Pd-extra.patch converts instances of 'float' to 't_float' in all classes of vanilla Pd's 'extra'. Also, a few constants and literals are defined with more decimals, and some float suffixes removed. The patch is intended for Pd-043.1test5.
Type 't_float' is already used in most of pd core code. The changes in the 'extra' classes would make them compliant with the core in this sense.
----------------------------------------------------------------------
>Comment By: Katja (katjav)
Date: 2011-11-15 07:33
Message:
Accompanying the patch file, I've produced unit test patches. The reference
was pd-043.1 vanilla release. The tests indicate slightly altered output
for some of the classes, probably due to more precise constants and removal
of float suffixes. See attached 'unittests-pdextras.zip'.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3438338&group_…
Patches item #3438338, was opened at 2011-11-15 07:17
Message generated for change (Tracker Item Submitted) made by katjav
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3438338&group_…
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: puredata
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Katja (katjav)
Assigned to: Miller Puckette (millerpuckette)
Summary: change float to t_float in Pd 'extra'
Initial Comment:
0001-change-float-to-t_float-in-Pd-extra.patch converts instances of 'float' to 't_float' in all classes of vanilla Pd's 'extra'. Also, a few constants and literals are defined with more decimals, and some float suffixes removed. The patch is intended for Pd-043.1test5.
Type 't_float' is already used in most of pd core code. The changes in the 'extra' classes would make them compliant with the core in this sense.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3438338&group_…