Patches item #1952001, was opened at 2008-04-26 01:57
Message generated for change (Comment added) made by zmoelnig
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1952001&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: feature
>Status: Pending
Resolution: Works For Me
Priority: 5
Private: No
Submitted By: Luke Iannini (lukeiannini)
Assigned to: Miller Puckette (millerpuckette)
Summary: on OSX, made canvas windows show dirty in OS-native style
Initial Comment:
This is a patch to add proxy icons (a tiny representation of the file being worked on in the window's titlebar; it can be dragged to grab a reference to the file for drag and drop, or right/command-clicked to get the parent folders of the file). The patch also adds a dot in the close button when the Pdpatch is dirty, consistent with other OS X apps.
The patch is untested; Pd-Vanilla doesn't seem to support the Tk attributes necessary (for reasons I don't understand), but Pd-Extended does (hardwiring wm attributes $name -modified 0 -titlepath
{insertPathHere} in pd.tk's pdtk_canvas_new works in pd-e but not in pd-v).
----------------------------------------------------------------------
Comment By: IOhannes m zmlnig (zmoelnig)
Date: 2010-08-21 18:54
Message:
This patch has either been incorporated into pd-gui-rewrite pd-devel 0.43
or made irrelevant by it.
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2008-04-28 18:41
Message:
Logged In: YES
user_id=27104
Originator: NO
added to branches/pd-extended/0.40
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2008-04-28 02:20
Message:
Logged In: YES
user_id=27104
Originator: NO
I uploaded a version of Luke Iannini's proxy_icon_and_modified_state.patch
fixed to eliminate crash (caused by using %f in sys_vgui for an int value).
Also, made the title bar fully Mac OS X-style.
Pd-vanilla uses an old version of Tcl/Tk, that's probably why it doesn't
work there. If Pd-vanilla is built against a recent version of Tcl/Tk,
then I'll bet it'd work.
File Added: proxy_icon_and_modified_state-0.41.4.patch
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2008-04-27 02:34
Message:
Logged In: YES
user_id=27104
Originator: NO
I tried this with the latest from branches/pd-extended/v0-40 and I am
getting a lot of crashes. It didn't seem to change the window behavior
either. Are there some changes to u_main.tk that are missing from the
patch, perhaps? I'll try to troubleshoot it more later.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1952001&group_…
Patches item #1917574, was opened at 2008-03-18 02:01
Message generated for change (Comment added) made by zmoelnig
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1917574&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: Pending
Resolution: None
Priority: 5
Private: No
Submitted By: marius schebella (mariusschebella)
Assigned to: Miller Puckette (millerpuckette)
Summary: fix absolute path support for [declare] and #X declare
Initial Comment:
I am not sure if declare is working the way it is supposed to on OSX
the object I am trying to create is [OSCroute].
I can create [oscx/OSCroute], but what I want to accomplish is to avoid the oscx prefix, so I tried
[declare -stdpath extra/oscx]
[declare -stdpath oscx]
[declare -path extra/oscx]
[declare -path oscx]
[declare -stdlib extra/oscx]
[declare -stdlib oscx]
[declare -lib extra/oscx]
[declare -lib oscx]
I even tried [declare -stdpath extra/oscx] in combination with import oscx. and I get a console printout saying [import] loaded library: oscx,
but OSCroute
... couldn't create
don't know how to track down the error, or if I am maybe doing something wrong.
pd-extended-0.40.3-20080222.
----------------------------------------------------------------------
>Comment By: IOhannes m zmlnig (zmoelnig)
Date: 2010-08-21 18:51
Message:
is this stil an issue with 0.43?
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2008-05-22 21:26
Message:
Logged In: YES
user_id=27104
Originator: NO
(repost of my pd-dev post)
The realpath() is definitely related. If you start Pd using a relative
path, like I do when doing dev work (e.g. make && ../bin/pd), then
sys_libdir will be a relative path. That means it is impossible to make
absolute paths using sys_libdir, which is what I need to do to make
[import] work, or loading libdirs with [declare -lib] and the libdir
loader.
I can't see any problems that realpath() might cause.
----------------------------------------------------------------------
Comment By: Miller Puckette (millerpuckette)
Date: 2008-05-22 19:21
Message:
Logged In: YES
user_id=313747
Originator: NO
I patched 0.42 with th epart of it that dealt with declare... there's
something else in the patch that invokes a "realpath" function I don't
know about (and I assume won't compile as is) so I ignored that bit.
I think I should also update 0.41 since this is a fairly important bug fix
and also a safe-looking one.
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2008-05-21 20:19
Message:
Logged In: YES
user_id=27104
Originator: NO
This patch should add support for absolute paths in [declare] and the #X
declare lines in the .pd files. Here's the related commit for
Pd-extended:
http://pure-data.svn.sourceforge.net/viewvc/pure-data?view=rev&revision=9862
File Added: fix_absolute_paths_for_declare-0.41.4.patch
----------------------------------------------------------------------
Comment By: marius schebella (mariusschebella)
Date: 2008-05-06 18:45
Message:
Logged In: YES
user_id=1856112
Originator: YES
that's how -stdpath works on os x. it does not work for absolute paths,
and seems to be relative to some directory outside of /Applications. I
tried all relations with less ../ and from inside /Applications.
declare -stdpath
../../../../../../../../Applications/Pd-0.40.3-extended-20080428.app/Contents/Resources/extra/boids
maybe this helps.
m.
----------------------------------------------------------------------
Comment By: marius schebella (mariusschebella)
Date: 2008-05-04 19:35
Message:
Logged In: YES
user_id=1856112
Originator: YES
I just realized that I made some mistakes in the past and that declare
actually is working better than I thought. here is what I forgot:
1) declare only takes effect when you open the patch. stupid, I think I
read it, but... Some alleged "bugs" were caused by the assumption that it
is enough to create a declare object and paths will be set...
2) no absolute paths! absolute paths work for -lib, but not for
-path/-stdpath. at least on OS X it is not possible to set absolute paths
(or at least I don't know how), all paths have to be relative to Pd
(-stdpath) or to the patch (-path).
----------------------------------------------------------------------
Comment By: Roman Haefeli (reduzent)
Date: 2008-03-20 14:38
Message:
Logged In: YES
user_id=1895440
Originator: NO
oops...
from some quick fix from before i accidently still had a symbolic link
from /home/roman/extra pointing to /usr/local/lib/pd/extra/, that is why it
found [bar] in that funny place:
/home/roman/extra/foo/bar.pd
tried again without the symlink:
--------------------------------
tried /home/roman/declare_test.pd and succeeded
tried /home/roman/../extra/foo/bar.l_i386 and failed
tried /home/roman/bar.l_i386 and failed
tried /usr/local/lib/pd/extra/bar.l_i386 and failed
tried /home/roman/../extra/foo/bar.pd_linux and failed
tried /home/roman/bar.pd_linux and failed
tried /usr/local/lib/pd/extra/bar.pd_linux and failed
tried /home/roman/../extra/foo/bar/bar.l_i386 and failed
tried /home/roman/bar/bar.l_i386 and failed
tried /usr/local/lib/pd/extra/bar/bar.l_i386 and failed
tried /home/roman/../extra/foo/bar/bar.pd_linux and failed
tried /home/roman/bar/bar.pd_linux and failed
tried /usr/local/lib/pd/extra/bar/bar.pd_linux and failed
tried /home/roman/../extra/foo/bar.pd and failed
tried /home/roman/bar.pd and failed
tried /usr/local/lib/pd/extra/bar.pd and failed
tried /home/roman/../extra/foo/bar.pat and failed
tried /home/roman/bar.pat and failed
tried /usr/local/lib/pd/extra/bar.pat and failed
----------------------------------------------------------------------
Comment By: Roman Haefeli (reduzent)
Date: 2008-03-20 12:49
Message:
Logged In: YES
user_id=1895440
Originator: NO
i just did the same test again on my box, ubuntu dapper i686:
tried /home/roman/declare_test.pd and succeeded
tried /home/roman/extra/foo/bar.l_i386 and failed
tried /home/roman/bar.l_i386 and failed
tried /usr/local/lib/pd/extra/bar.l_i386 and failed
tried /home/roman/extra/foo/bar.pd_linux and failed
tried /home/roman/bar.pd_linux and failed
tried /usr/local/lib/pd/extra/bar.pd_linux and failed
tried /home/roman/extra/foo/bar/bar.l_i386 and failed
tried /home/roman/bar/bar.l_i386 and failed
tried /usr/local/lib/pd/extra/bar/bar.l_i386 and failed
tried /home/roman/extra/foo/bar/bar.pd_linux and failed
tried /home/roman/bar/bar.pd_linux and failed
tried /usr/local/lib/pd/extra/bar/bar.pd_linux and failed
tried /home/roman/extra/foo/bar.pd and succeeded
which means:
- there is a difference between linux ppc and linux intel
- currently linux intel is the _only_ platform where [declare] works as
documented.
----------------------------------------------------------------------
Comment By: Roman Haefeli (reduzent)
Date: 2008-03-20 12:20
Message:
Logged In: YES
user_id=1895440
Originator: NO
could we give this issue a higher priority? the longer [declare] is not
working correctly, the more confusion it will cause. also it currently
cannot be used to create portable patches, that work on every platform with
every pd-distro, which i find would be the strong point in using
[declare].
----------------------------------------------------------------------
Comment By: Roman Haefeli (reduzent)
Date: 2008-03-20 12:11
Message:
Logged In: YES
user_id=1895440
Originator: NO
the same test on a ubuntu ppc machine with pd 0.41-4 compiled from source
and installed to the default location gave similar results:
output from pd-window:
----------------------
tried /home/olivia/declare_test.pd and succeeded
tried /home/olivia/extra/foo/bar.l_i386 and failed
tried /home/olivia/bar.l_i386 and failed
tried /usr/local/lib/pd/extra/bar.l_i386 and failed
tried /home/olivia/extra/foo/bar.pd_linux and failed
tried /home/olivia/bar.pd_linux and failed
tried /usr/local/lib/pd/extra/bar.pd_linux and failed
tried /home/olivia/extra/foo/bar/bar.l_i386 and failed
tried /home/olivia/bar/bar.l_i386 and failed
tried /usr/local/lib/pd/extra/bar/bar.l_i386 and failed
tried /home/olivia/extra/foo/bar/bar.pd_linux and failed
tried /home/olivia/bar/bar.pd_linux and failed
tried /usr/local/lib/pd/extra/bar/bar.pd_linux and failed
tried /home/olivia/extra/foo/bar.pd and failed
tried /home/olivia/bar.pd and failed
tried /usr/local/lib/pd/extra/bar.pd and failed
tried /home/olivia/extra/foo/bar.pat and failed
tried /home/olivia/bar.pat and failed
tried /usr/local/lib/pd/extra/bar.pat and failed
bar
... couldn't create
instantiating [foo/bar] works fine, which means, that installation of
[bar] is correct and pd finds it in extra/foo.
----------------------------------------------------------------------
Comment By: Roman Haefeli (reduzent)
Date: 2008-03-20 11:48
Message:
Logged In: YES
user_id=1895440
Originator: NO
while [declare -stdlib] seems to work, [declare -stdpath] doesn't work on
windows as well. i created a folder 'foo' into the extra directory and put
a file 'bar.pd' in there.
my test setup:
--------------
pd installation path:
D:\programme\pd-0.41\
pd executable:
D:\programme\pd-0.41\bin\pd.exe
test-abstraction:
D:\programme\pd-0.41\extra\foo\bar.pd
test-patch:
D:\home\declare_test.pd
test-patch contains:
[declare -stdpath extra/foo]
[bar]
pd start command:
D:\programme\pd-0.41\bin\pd.exe -verbose -open D:\home\declare_test.pd
output from pd-window:
----------------------
tried D:\\home\\declare_test.pd and succeeded
tried D:\\home\\..\\extra\\foo\\bar.m_i386 and failed
tried D:\\home\\bar.m_i386 and failed
tried D:\\programme\\pd-0.41\\extra\\bar.m_i386 and failed
tried D:\\home\\..\\extra\\foo\\bar.dll and failed
tried D:\\home\\bar.dll and failed
tried D:\\programme\\pd-0.41\\extra\\bar.dll and failed
tried D:\\home\\..\\extra\\foo\\bar\\bar.m_i386 and failed
tried D:\\home\\bar\\bar.m_i386 and failed
tried D:\\programme\\pd-0.41\\extra\\bar\\bar.m_i386 and failed
tried D:\\home\\..\\extra\\foo\\bar\\bar.dll and failed
tried D:\\home\\bar\\bar.dll and failed
tried D:\\programme\\pd-0.41\\extra\\bar\\bar.dll and failed
tried D:\\home\\..\\extra\\foo\\bar.pd and failed
tried D:\\home\\bar.pd and failed
tried D:\\programme\\pd-0.41\\extra\\bar.pd and failed
tried D:\\home\\..\\extra\\foo\\bar.pat and failed
tried D:\\home\\bar.pat and failed
tried D:\\programme\\pd-0.41\\extra\\bar.pat and failed
bar
... couldn't create
it looks like [declare -stdpath] is adding the search path relative to the
patch instead of relative to pd.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1917574&group_…
Patches item #2835752, was opened at 2009-08-11 20:41
Message generated for change (Settings changed) made by zmoelnig
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=2835752&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: bugfix
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Chris McCormick (chr15m)
Assigned to: Miller Puckette (millerpuckette)
>Summary: floor, ceil functions in expr misdeclared
Initial Comment:
Low-hanging fruit bugfix from notes.txt plus expr-help.pd with a list of all of the builtin functions. Included the scripts to generate that doc.
----------------------------------------------------------------------
Comment By: IOhannes m zmlnig (zmoelnig)
Date: 2010-08-21 18:39
Message:
btw, the patch is reversed.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=2835752&group_…
Patches item #2835752, was opened at 2009-08-11 20:41
Message generated for change (Comment added) made by zmoelnig
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=2835752&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: bugfix
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Chris McCormick (chr15m)
Assigned to: Miller Puckette (millerpuckette)
Summary: floor, ciel functions in expr misdeclared
Initial Comment:
Low-hanging fruit bugfix from notes.txt plus expr-help.pd with a list of all of the builtin functions. Included the scripts to generate that doc.
----------------------------------------------------------------------
>Comment By: IOhannes m zmlnig (zmoelnig)
Date: 2010-08-21 18:39
Message:
btw, the patch is reversed.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=2835752&group_…
Patches item #3050371, was opened at 2010-08-21 18:31
Message generated for change (Tracker Item Submitted) made by zmoelnig
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3050371&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: bugfix
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: IOhannes m zmlnig (zmoelnig)
Assigned to: Miller Puckette (millerpuckette)
Summary: [bugfix] print version number
Initial Comment:
in 0.43.test2, the veresion number is not printed correctly when running "pd -version".
rather than the version number, it prints "(null)", since the version string is uninitialized (and luckily set to NULL)
the attached patch fixes this.
(it's a git patch, so apply with "gem am 0001-provide-a-version-string.patch")
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3050371&group_…
Patches item #3050325, was opened at 2010-08-21 10:08
Message generated for change (Tracker Item Submitted) made by whyameye
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3050325&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: externals
Group: bugfix
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: whyameye (whyameye)
Assigned to: Nobody/Anonymous (nobody)
Summary: pdp_mix perfect copy for crossfade=1: portable (mmx)
Initial Comment:
when crossfade is set to 1 for [pdp_mix], outlet should return exact copy of middle inlet. Instead [pdp_mix] still tries to mix the left and middle inlet, resulting in slight degradation of the resulting video. Consequently, a feedback loop connecting [pdp_mix] outlet to its middle inlet through a [pdp_del] with crossfade set to 1 will eventually result in a video that is completely black.
This patch adds a trivial copy from the middle inlet to the outlet if the crossfade is set to 1. This patch works for the mmx version of the code. The portabkle (no mmx) version of the code has the same flaw and the patch for this has been submitted with ID 3050324
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3050325&group_…
Patches item #3050324, was opened at 2010-08-21 10:04
Message generated for change (Settings changed) made by whyameye
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3050324&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: externals
>Group: bugfix
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: whyameye (whyameye)
Assigned to: Nobody/Anonymous (nobody)
Summary: pdp_mix giving imperfect copy for crossfade=1 (no mmx)
Initial Comment:
when crossfade is set to 1 for [pdp_mix], outlet should return exact copy of middle inlet. Instead [pdp_mix] still tries to mix the left and middle inlet, resulting in slight degradation of the resulting video. Consequently, a feedback loop connecting [pdp_mix] outlet to its middle inlet through a [pdp_del] with crossfade set to 1 will eventually result in a video that is completely black.
This patch adds a trivial copy from the middle inlet to the outlet if the crossfade is set to 1. This patch works for the portable (no mmx) version of the code. The mmx version of the code has the same flaw and the patch for this will be submitted separately.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3050324&group_…
Patches item #3050324, was opened at 2010-08-21 10:04
Message generated for change (Tracker Item Submitted) made by whyameye
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3050324&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: externals
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: whyameye (whyameye)
Assigned to: Nobody/Anonymous (nobody)
Summary: pdp_mix giving imperfect copy for crossfade=1 (no mmx)
Initial Comment:
when crossfade is set to 1 for [pdp_mix], outlet should return exact copy of middle inlet. Instead [pdp_mix] still tries to mix the left and middle inlet, resulting in slight degradation of the resulting video. Consequently, a feedback loop connecting [pdp_mix] outlet to its middle inlet through a [pdp_del] with crossfade set to 1 will eventually result in a video that is completely black.
This patch adds a trivial copy from the middle inlet to the outlet if the crossfade is set to 1. This patch works for the portable (no mmx) version of the code. The mmx version of the code has the same flaw and the patch for this will be submitted separately.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3050324&group_…
Hey Miller,
I'm in the process of sorting thru all of the patches in Pd-extended
and what you have accepted. Everything looks good except for one
patch. it seems the commit message points to this patch, but the
actual changes come from something else entirely:
http://sourceforge.net/tracker/index.php?func=detail&aid=2952880&group_id=5…
.hc
----------------------------------------------------------------------------
If nature has made any one thing less susceptible than all others of
exclusive property, it is the action of the thinking power called an
idea, which an individual may exclusively possess as long as he keeps
it to himself; but the moment it is divulged, it forces itself into
the possession of everyone, and the receiver cannot dispossess himself
of it. - Thomas Jefferson
Patches item #3042790, was opened at 2010-08-10 15:35
Message generated for change (Comment added) made by millerpuckette
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3042790&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: feature
>Status: Closed
>Resolution: Accepted
Priority: 8
Private: No
Submitted By: Hans-Christoph Steiner (eighthave)
Assigned to: Miller Puckette (millerpuckette)
Summary: create USEAPI_DUMMY/s_audio_dummy.c for libpd
Initial Comment:
I set this to high priority since its a trivial change and the only hurdle left before we can use the pure-data.git 0.43 source directly in the libpd/Pd-for-Android project.
This patch creates a "dummy" audio API in the file s_audio_dummy.c, like the existing s_midi_dummy.c. The libpd implementation calls the Pd callback code directly, so this dummy API is needed to keep various other parts of Pd happy, like the GUI.
----------------------------------------------------------------------
>Comment By: Miller Puckette (millerpuckette)
Date: 2010-08-18 14:42
Message:
bizarre but I can't see a better way to do it right now.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3042790&group_…