Re: [PD-dev] initbang and friends WAS: run-up to release 0.43
Hi, On Fri, Aug 20, 2010 at 02:02:08PM -0400, Hans-Christoph Steiner wrote: On Aug 20, 2010, at 5:42 AM, IOhannes m zmölnig wrote: I'm saying I like the interface of having a suite of objects called *bang rather than [loadbang close], etc. it makes them super easy to use and remember. [initbang] [loadbang] [propertybang] [closebang] The only issue I have with this is the difference between initbang and loadbang. In several patches posted to this list in the past I observed, that sometimes people tended to use initbang where a simple loadbang would be sufficient, i.e. they were doing nothing that would actually require initbang.(*) I assume this is because they actually didn't know or understand the difference. That's where a loadbang object that somehow combined initbang into it with an argument *may* be preferable. I don't see any reason to combine load- and closebang (or propertybang, but I don't really know that. I assume it will fire when Help-Properties is selected.) (*) A typical example were abstractions using initbang because their loadbang would not fire after dynamic patching. Here initbang is not the correct solution, but a loadbang-message to the dynamic patching canvas. Ciao -- Frank ___ Pd-dev mailing list Pd-dev@iem.at http://lists.puredata.info/listinfo/pd-dev
[PD-dev] [ pure-data-Patches-3050324 ] pdp_mix giving imperfect copy for crossfade=1 (no mmx)
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=detailatid=478072aid=3050324group_id=55736 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=detailatid=478072aid=3050324group_id=55736 ___ Pd-dev mailing list Pd-dev@iem.at http://lists.puredata.info/listinfo/pd-dev
[PD-dev] [ pure-data-Patches-3050325 ] pdp_mix perfect copy for crossfade=1: portable (mmx)
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=detailatid=478072aid=3050325group_id=55736 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=detailatid=478072aid=3050325group_id=55736 ___ Pd-dev mailing list Pd-dev@iem.at http://lists.puredata.info/listinfo/pd-dev
[PD-dev] [ pure-data-Patches-3050371 ] [bugfix] print version number
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=detailatid=478072aid=3050371group_id=55736 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=detailatid=478072aid=3050371group_id=55736 ___ Pd-dev mailing list Pd-dev@iem.at http://lists.puredata.info/listinfo/pd-dev
Re: [PD-dev] run-up to release 0.43
On 08/19/2010 02:06 AM, Miller Puckette wrote: I'll try not to add new features but just fix bugs for the next 2 weeks -- here is one bug (+ fix): pd -version prints garbage. https://sourceforge.net/tracker/?func=detailaid=3050371group_id=55736atid=478072 mgfasdr IOhannes signature.asc Description: OpenPGP digital signature ___ Pd-dev mailing list Pd-dev@iem.at http://lists.puredata.info/listinfo/pd-dev
[PD-dev] [ pure-data-Patches-2835752 ] floor, ciel functions in expr misdeclared
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=detailatid=478072aid=2835752group_id=55736 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=detailatid=478072aid=2835752group_id=55736 ___ Pd-dev mailing list Pd-dev@iem.at http://lists.puredata.info/listinfo/pd-dev
[PD-dev] [ pure-data-Patches-1917574 ] fix absolute path support for [declare] and #X declare
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=detailatid=478072aid=1917574group_id=55736 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=revrevision=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
[PD-dev] [ pure-data-Patches-1952001 ] on OSX, made canvas windows show dirty in OS-native style
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=detailatid=478072aid=1952001group_id=55736 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=detailatid=478072aid=1952001group_id=55736 ___ Pd-dev mailing list Pd-dev@iem.at http://lists.puredata.info/listinfo/pd-dev
Re: [PD-dev] run-up to release 0.43
On 08/21/2010 06:32 PM, IOhannes zmölnig wrote: On 08/19/2010 02:06 AM, Miller Puckette wrote: I'll try not to add new features but just fix bugs for the next 2 weeks -- here is one bug and of course there are several simple bugfixes still pending in the tracker: e.g. 2806049: [bugfix] renamed arrays do not get used by dsp 2835752: floor, ceil functions in expr misdeclared those are really simple, about 2 lines code each. gmasdr IOhannes signature.asc Description: OpenPGP digital signature ___ Pd-dev mailing list Pd-dev@iem.at http://lists.puredata.info/listinfo/pd-dev
[PD-dev] [ pure-data-Patches-1952001 ] on OSX, made canvas windows show dirty in OS-native style
Patches item #1952001, was opened at 2008-04-25 19:57 Message generated for change (Comment added) made by eighthave You can respond by visiting: https://sourceforge.net/tracker/?func=detailatid=478072aid=1952001group_id=55736 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: Closed 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: Hans-Christoph Steiner (eighthave) Date: 2010-08-21 14:01 Message: Yes indeed, its true! -- Comment By: IOhannes m zmlnig (zmoelnig) Date: 2010-08-21 12: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 12: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-27 20: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-26 20: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=detailatid=478072aid=1952001group_id=55736 ___ Pd-dev mailing list Pd-dev@iem.at http://lists.puredata.info/listinfo/pd-dev
[PD-dev] template Makefile
i cleaned up the template Makefile (externals/template/Makefile) a bit. namely i switched to using $@, $ and $^, which makes a lot of things way more elegant. other cleanups: removed the SOURCES_android since this is not used anywhere and is misleading use PD_PATH where appropriate added SOURCES_LIB for .c files holding shared functions use HELPPATCHES to enumerate -help.pd (usually this info is gathered automatically; the average user will never see that) these changes should make zero difference to the user experience of the templ Makefile. remaining questions: DIST PATH: why does the makefile enforce the manual and examples to be examples/ resp. manual/ folders _in the source directory_ ? i understand that they should be laid out standardized in the install directory, but why in the source folder? why not just: MANUAL = doc/manual.pdf manuals/anothermanual.pd and install that into .../$(LIBRARY_NAME)/manual/ INSTALL PATH why the heck is the default installation path: $(libdir)/pd-externals ?? i cannot remember that this was agreed on by anybody but hans. i do remember that ~/pd-externals was discussed (and i still would prefer ~/.pd/extra/ over this unnecessary homedirectory cluttering), but $(libdir)/pd-externals is news to me. instead i do remember, that $(libdir)/pd/extra was meant to be the canonical name for all flavours of Pd (e.g. with PdX and pd-vanilla, in debian pd-vanilla would look into both /usr/lib/puredata/extra and /usr/lib/pd/extra, whereas PdX would look into /usr/lib/pd-extended/extra and /usr/lib/pd/extra (in this order)) i also find, that a related patch has made it into puredata.git. unfortunately, it is inconsinstent with the template Makefile, as puredata.git has hardcoded /usr/local/lib/pd-externals/ whereas the tmpl uses $(libdir)/pd-externals which could well be /usr/lib/pd-externals/ which in turn is never ever searched by Pd. in case, i haven't said it yet: i don't like this. ,asdr IOhannes signature.asc Description: OpenPGP digital signature ___ Pd-dev mailing list Pd-dev@iem.at http://lists.puredata.info/listinfo/pd-dev
Re: [PD-dev] iemguts build issues WAS: dynamic patching: getting info on objects
On 08/17/2010 01:59 AM, Hans-Christoph Steiner wrote: Try using the template Makefile yourself, it should be pretty easy to plug in: http://puredata.info/docs/developer/MakefileTemplate Basically, you just need to list the C files in the right place, at the top. for what it is worth, iem/iemguts/Makefile is now template-based, so it should build on a numbe of platforms wihtout too much hazzle. fgmasdr IOhannes signature.asc Description: OpenPGP digital signature ___ Pd-dev mailing list Pd-dev@iem.at http://lists.puredata.info/listinfo/pd-dev
Re: [PD-dev] iemguts build issues WAS: dynamic patching: getting info on objects
On Aug 21, 2010, at 3:59 PM, IOhannes zmölnig wrote: On 08/17/2010 01:59 AM, Hans-Christoph Steiner wrote: Try using the template Makefile yourself, it should be pretty easy to plug in: http://puredata.info/docs/developer/MakefileTemplate Basically, you just need to list the C files in the right place, at the top. for what it is worth, iem/iemguts/Makefile is now template-based, so it should build on a numbe of platforms wihtout too much hazzle. To be pedantic, I'd say its _derived_ from the Makefile library template, not really based on it since you have organized the files into different directories. But it does seem to build as universal on Mac OS X, which is an improvement. .hc Man has survived hitherto because he was too ignorant to know how to realize his wishes. Now that he can realize them, he must either change them, or perish.-William Carlos Williams ___ Pd-dev mailing list Pd-dev@iem.at http://lists.puredata.info/listinfo/pd-dev