Patches item #1962700, was opened at 2008-05-12 16:18
Message generated for change (Comment added) made by millerpuckette
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1962700&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: Hans-Christoph Steiner (eighthave)
Assigned to: Miller Puckette (millerpuckette)
Summary: two embedded prefs for Mac: default and override all
Initial Comment:
This patch creates two different kinds of embedded org.puredata.pd.plist prefs files on Mac OS X. The first is the default settings that come with Pd-extended.app, the second is a prefs file that overrides all others, so that people can make their own custom apps based on the Pd.app, and be sure that the user's prefs aren't going to override things.
----------------------------------------------------------------------
>Comment By: Miller Puckette (millerpuckette)
Date: 2010-07-28 17:38
Message:
I think we should fix the preferences system so that pd extended doesn't
rely on any preferences at all -- anything short of that will make for
maintenance headaches later on.
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2008-05-13 10:06
Message:
Logged In: YES
user_id=27104
Originator: YES
That could work. I don't think this will cover every possibility. I
think a more straightforward approach would be to handle very special
hardware setups in the patch rather than the preferences. I think the
preferences should be general things, since you can't really query or test
for failures with the preferences system.
For the built-in audio, it almost always just works, so specific
configuration isn't needed.
----------------------------------------------------------------------
Comment By: IOhannes m zmlnig (zmoelnig)
Date: 2008-05-13 00:37
Message:
Logged In: YES
user_id=564396
Originator: NO
hmm, but how should the author of a custom app know about the
hardware-settings of the end-user?
wouldn't it be better to split the configuration files into
machine-dependencies (soundcard,...) and user-dependencies (libs,...) which
could then be overridden seperately?
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1962700&group_…
Patches item #1961878, was opened at 2008-05-11 06:28
Message generated for change (Settings changed) made by millerpuckette
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1961878&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: Closed
>Resolution: Accepted
Priority: 9
Private: No
Submitted By: Hans-Christoph Steiner (eighthave)
Assigned to: Miller Puckette (millerpuckette)
Summary: add ~ and env var support to paths when opening files
Initial Comment:
Created sys_expandpath() to replace ~ with $HOME and expand env vars on Windows. This function is then used in sys_trytoopenone() so that ~ works with all opening operations.
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2010-07-19 12:15
Message:
updated using git-format-patch for 0.43.0-test1
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2008-05-14 10:24
Message:
Logged In: YES
user_id=27104
Originator: YES
updated to fix crasher bug on GNU/Linux
File Added: add_tilde_support_toopen-0.41.4.patch
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1961878&group_…
Patches item #1948287, was opened at 2008-04-21 14:50
Message generated for change (Comment added) made by millerpuckette
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1948287&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: Closed
>Resolution: Accepted
Priority: 5
Private: No
Submitted By: Stephen Sinclair (radarsat1)
Assigned to: Nobody/Anonymous (nobody)
Summary: Throttle speed of mouse motion events whhen moving objects.
Initial Comment:
Use a clock to throttle mouse motion events when objects are being moved in the editor.
This improves GUI responsiveness when moving many objects at once. Perhaps there is a better way than adding a clock instance to each and every canvas.
----------------------------------------------------------------------
>Comment By: Miller Puckette (millerpuckette)
Date: 2010-07-28 17:19
Message:
cool - I should have done this 10 years ago.
----------------------------------------------------------------------
Comment By: Stephen Sinclair (radarsat1)
Date: 2008-04-21 18:20
Message:
Logged In: YES
user_id=616837
Originator: YES
It would probably help to use 'move', but this doesn't really conflict
with that idea. Either way I think it's somewhat wasteful to redraw the
screen on every mouse move event. It's only necessary to redraw as fast
as:
- a human can move the mouse
- the screen can refresh
Some tests I did with moving lots of objects (and large arrays) were much
smoother using this clock. Probably one global clock could be used, since
only one canvas can receive mouse motion events at a time. But that might
require more complicated changes. I wasn't sure about adding the e_xnew
and e_ynew variables, but I couldn't think of a better way. I'm open to
ideas.
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2008-04-21 16:51
Message:
Logged In: YES
user_id=27104
Originator: NO
This in interesting idea, I think some of the IEMGUIs have built-in clocks
for drawing, like [vu]. IIRC. I wonder if it would be necessary if the
displace_fn stuff was cleaned up to use 'move' instead of 'delete' then
'create', like in patch #1948146.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1948287&group_…
Patches item #1914851, was opened at 2008-03-15 11:45
Message generated for change (Comment added) made by eighthave
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1914851&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: Closed
Resolution: Accepted
Priority: 5
Private: No
Submitted By: marius schebella (mariusschebella)
Assigned to: Miller Puckette (millerpuckette)
Summary: menubar - window filename update
Initial Comment:
hi,
I found a minor bug. when you open a patch (or create new patch) and then "save as" (or "save" with a new patch) then that name is not updated in the menubar-window menu.
the patch is still listed under the old name or "untitled".
maybe that's only on mac 10.5. I tried pd-extended 0.40.3(080222) and the current pd 0.41-2 from millers site.
marius.
----------------------------------------------------------------------
>Comment By: Hans-Christoph Steiner (eighthave)
Date: 2010-07-28 14:53
Message:
git commit 842230bb50000c90dbaab46c9e529e6020ef10fc
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2008-04-28 16:45
Message:
Logged In: YES
user_id=27104
Originator: NO
Ok, found the source of the problem. I changed it so whenever saving,
update the window list for the Window menu in case the save was a "Save As"
that changed the name of the window.
File Added: update_window_list_on_saveas-0.41.4.patch
----------------------------------------------------------------------
Comment By: marius schebella (mariusschebella)
Date: 2008-04-28 08:09
Message:
Logged In: YES
user_id=1856112
Originator: YES
sorry, if I was not clear. to reproduce create a new patch and look in the
menubar under "Windows". It will be listed as Untitled-1 or something. then
save it as somename.pd and look again under "Windows". it is still listed
as untitled. this does not get updated until you open a new patch.
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2008-04-28 00:57
Message:
Logged In: YES
user_id=27104
Originator: NO
I cannot reproduce his bug on Pd-0.40.3-extended-20080427 or Pd-0.41-4 on
Mac OS X 10.4.11. If you saveas an abstraction that is in use, it will not
change the title , and it will indeed remain as that abstraction, and just
save a copy, IIRC.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1914851&group_…
Patches item #1881907, was opened at 2008-01-29 11:24
Message generated for change (Comment added) made by eighthave
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1881907&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: Closed
Resolution: Accepted
Priority: 5
Private: No
Submitted By: IOhannes m zmlnig (zmoelnig)
Assigned to: Miller Puckette (millerpuckette)
Summary: more t_float/t_sample fixes
Initial Comment:
pd-0.41 has a lot of cleanup regarding the types of numbers (see patch-#1822001 'use of t_float/t_sample instead of "float"')
however, some things have been left out by the original patches, here are some more:
m_pd.h:
seems like the t_float-enabling of postfloat() was forgotten in the declaration
d_math.c:
here it is the other way round: while the declarations in m_pd.h use t_float, the definitions do not
g_array.c g_template.c:
here my patch was a bit rash: scanf() doesn't like to get pointers to double (if "t_float" is set to "double") if the format string is really "%f" (or "%g"); the new patches use an internal _double_ variable and the format string "%lf" ("%lg"); the _double_ variable is then cast to (t_float).
this should be on the save side; the alternatives would have been using an internal _float_ variable, and/or to change the format-string depending on the used type.
d_fftroutine.c: this needed a bit more changes, but luckily enough, the original code already was aware of possible type-changes; i have currently used "t_float" (in line with the m_pd.h declaration of "pd_fft()"), but i am not sure whether this really should be "t_sample".
changes should be rather simple: just change the defines of FLOAT and SAMPLE in the beginning... (and don't forget the declaration in m_pd.h)
with these changes, Pd compiles without (additional) warnings with t_float/t_floatarg and t_sample set to "double".
(there is still work to do in d_ugen.c though, to make it really usable...)
----------------------------------------------------------------------
>Comment By: Hans-Christoph Steiner (eighthave)
Date: 2010-07-28 14:52
Message:
git commit 842230bb50000c90dbaab46c9e529e6020ef10fc
----------------------------------------------------------------------
Comment By: Miller Puckette (millerpuckette)
Date: 2010-07-24 22:17
Message:
I adopted changes where they looked right.. I then tried actually running
Pd in 64 bit and found
lots of problems. So I'll mark this 'accepted' but there's more to do
here.
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2010-01-01 20:43
Message:
it seems that some of this stuff has been accepted and some not, perhaps
they should be reformated?
----------------------------------------------------------------------
Comment By: IOhannes m zmlnig (zmoelnig)
Date: 2008-01-29 12:54
Message:
Logged In: YES
user_id=564396
Originator: YES
File Added: sigmund~.c.diff
----------------------------------------------------------------------
Comment By: IOhannes m zmlnig (zmoelnig)
Date: 2008-01-29 12:12
Message:
Logged In: YES
user_id=564396
Originator: YES
added a fix for extra/sigmung~
File Added: sigmund~.c.diff
----------------------------------------------------------------------
Comment By: IOhannes m zmlnig (zmoelnig)
Date: 2008-01-29 12:11
Message:
Logged In: YES
user_id=564396
Originator: YES
added a fix for pique
File Added: pique.c.diff
----------------------------------------------------------------------
Comment By: IOhannes m zmlnig (zmoelnig)
Date: 2008-01-29 12:09
Message:
Logged In: YES
user_id=564396
Originator: YES
added a fix for extra/lrshift~
File Added: lrshift~.c.diff
----------------------------------------------------------------------
Comment By: IOhannes m zmlnig (zmoelnig)
Date: 2008-01-29 12:08
Message:
Logged In: YES
user_id=564396
Originator: YES
added a fix for extra/loop~
(the core code originally was not dependent on Pd; i therefore introduced
an additional "#define t_sample float" in case it is not compiled for Pd)
File Added: loop~.c.diff
----------------------------------------------------------------------
Comment By: IOhannes m zmlnig (zmoelnig)
Date: 2008-01-29 12:06
Message:
Logged In: YES
user_id=564396
Originator: YES
added a fix for extra/choice
File Added: choice.c.diff
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1881907&group_…
Patches item #1862178, was opened at 2008-01-02 08:40
Message generated for change (Comment added) made by eighthave
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1862178&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: Closed
Resolution: Accepted
Priority: 1
Private: No
Submitted By: IOhannes m zmlnig (zmoelnig)
Assigned to: Miller Puckette (millerpuckette)
Summary: [feature] make makefilename accept floats in %s
Initial Comment:
while fiddling with the bug-fix for the missing "%c" in [makefilename] (see #1862168), i found that
[makefilename %s] will safely ignore any number that it receives.
in theory this is nice and consistent behaviour.
in practice i find it weird that
[32(
|
[makefilename %s]
outputs nothing(!).
and
[32(
|
[makefilename _%s+]
outputs "_+"
the attached patch uses "%g" to convert a number into a string which is then inserted as %s.
----------------------------------------------------------------------
>Comment By: Hans-Christoph Steiner (eighthave)
Date: 2010-07-28 14:51
Message:
git commit 842230bb50000c90dbaab46c9e529e6020ef10fc
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1862178&group_…
Patches item #1851193, was opened at 2007-12-15 00:23
Message generated for change (Comment added) made by eighthave
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1851193&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: Closed
Resolution: Accepted
Priority: 9
Private: No
Submitted By: Roman Haefeli (reduzent)
Assigned to: Miller Puckette (millerpuckette)
Summary: [symbolatom\ : doesn't allow to type spaces (ascii 32)
Initial Comment:
i would like to read a textfile into an object [textfile] by using the symbolatom as a gui to specify the filename. however, it fails when the filename of the file contains a space.
example patch:
[symbolatom\
|
[read $1(
|
[textfile]
----------------------------------------------------------------------
>Comment By: Hans-Christoph Steiner (eighthave)
Date: 2010-07-28 14:50
Message:
git commit 842230bb50000c90dbaab46c9e529e6020ef10fc
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2010-07-15 18:32
Message:
I'm elevating the priority since this is a trivial fix and has been in
Pd-extended since 2007.
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2009-03-20 23:16
Message:
Wow, such a simple fix, I wish I thought of trying years ago. This does
make me think that we should have a discussion at PdCon just about spaces
and symbols. I included this change in Pd-extended 0.41. Here's the
commit:
http://pure-data.svn.sourceforge.net/viewvc/pure-data?view=rev&revision=108…
----------------------------------------------------------------------
Comment By: IOhannes m zmlnig (zmoelnig)
Date: 2008-04-06 13:42
Message:
Logged In: YES
user_id=564396
Originator: NO
the fix for this is rather trivial (see attached patch)
i'm wondering why spaces have been disabled in the first place...
File Added: spacesymbol.diff
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1851193&group_…
I meant to keep it on dev -- must have hit the wrong flavor of 'reply' by
accident, will try to remember to CC dev back in.
I think pd_path is sent to tcl when pd starts up, and then afterward
any changes to it are initiated by the gui (which can then simply store
the new value it sends down to Pd, if it's going to need it for anything).
This is all OK unless there's some mechanism I don't know about for changing
the path from within pd (in which case I'd propose that pd send the new
value up to the GUI on its own initiative).
Ditto the 'startup' path -- pd volunteers it at startup so the GUI always
knows what it is.
Problem with waiting for pd_path is that pd sends the path up in a sequence
of messages like:
set ::pd_path {}
lappend ::pd_path {foo}
and you have no way of ascertaining which 'lappend' terminates the sequence so
that ::pd_path is now correct. In my version, when you see "pdtk_pd_startup"
you know that ::pd_path and ::startup_libraries are no longer going to get
any lappends.
Unless I'm really missing something, plugins (or anyone else) can access
::pd_path and ::startup_libraries at will, no?
cheers
Miller
On Wed, Jul 21, 2010 at 10:28:01PM -0400, Hans-Christoph Steiner wrote:
>
> Quick question, do you prefer to keep this off pd-dev? I'd prefer it
> on the list so others can weigh in.
>
> Removing the set-startup almost completely breaks the loading of
> plugins, for example. pd_path is also used in the Help Browser so that
> it will display all libraries that are installed anywhere in the
> path. It also means that plugins can't search the path like 'pd' can,
> so things like a installed class browser plugin would no longer be
> possible.
>
> I am not married to the implementation, but I do see the functionality
> as essential. I don't see the difference between waiting on
> "pdtk_pd_startup" to be sent via a network socket, or waiting the
> contents of pd_path being sent over the same network socket. You can
> even set pd_path to blank and that will trigger the vwait properly.
> So your current startup procedure relies on:
>
> sys_vgui("pdtk_pd_startup %d %d %d {%s} %s %s {%s} %s\n",
>
> the pd-gui-rewrite/0.43 startup procedure relies on:
>
> sys_gui("set ::pd_path {}\n");
>
> Seems pretty close to me.
>
> .hc
>
> On Jul 21, 2010, at 9:54 PM, Miller Puckette wrote:
>
> >Hmm -- the only place I saw set-startup was in the startup tcl code --
> >which I think could have only been called once. Is there an easy-to-
> >exhibit
> >problem in the code in sf/git (or is the problem in another branch
> >somewhere?)
> >
> >Anyhow I'm just about sure there's no way code could work robustly
> >waiting on
> >pd_path... some other means of syncing would be needed.
>
> >
> >cheers
> >M
> >
> >
> >On Wed, Jul 21, 2010 at 04:21:00PM -0400, Hans-Christoph Steiner
> >wrote:
> >>
> >>Its already being actively used in the GUI rewrite, which is
> >>currently
> >>somewhat broken without it. I think the only client for that message
> >>really would be pd-gui, and I'd happily do the work to change it if
> >>need
> >>be. That's a much better option than omitting it.
> >>
> >>.hc
> >>
> >>Miller Puckette wrote:
> >>>My usual strategy on stuff like this is wait until someone is
> >>>actively
> >>>using it - because one's guess as to how the feature should work
> >>>is never
> >>>how it's really needed when the time comes and then you're stuck
> >>>maintaining
> >>>two different versions of the same thing :)
> >>>
> >>>M
> >>>
> >>>On Sat, Jul 17, 2010 at 12:15:30AM -0400, Hans-Christoph Steiner
> >>>wrote:
> >>>>Yeah, I added the "pd set-startup", etc. stuff to m_glob.c and
> >>>>s_path.c. Regardless of the startup procedure, I think pd-gui
> >>>>should
> >>>>be able to request state from pd so it can stay in sync. A dialog
> >>>>shouldn't be required to get this info. This is akin to IOhannes'
> >>>>idea that pd-gui's state should be settable by pd at any point
> >>>>while
> >>>>running.
> >>>>
> >>>>Then it makes sense to reuse the same procedure for getting the
> >>>>state
> >>>>in the startup procedure, rather than having a custom method for
> >>>>only
> >>>>the startup procedure.
> >>>>
> >>>>.hc
> >>>>
> >>>>On Jul 16, 2010, at 10:59 PM, Miller Puckette wrote:
> >>>>
> >>>>>Hi Hans --
> >>>>>
> >>>>>the "vwait" line didn't seem correct to me (could return right
> >>>>>after
> >>>>>Pd says to clear pd_path, before the other messages bubble up to
> >>>>>append
> >>>>>the various directories to it).
> >>>>>
> >>>>>I ended up fixing Pd to volunteer the path and startup stuff
> >>>>>before
> >>>>>sending
> >>>>>pdtk_pd_startup so that none of the tcl side of things should be
> >>>>>needed
> >>>>>any longer. OTOH if you were planning to send "pd set-startup"
> >>>>>etc at
> >>>>>some later point in the tcl code, this would need to be
> >>>>>supported in
> >>>>>Pd...(?)
> >>>>>
> >>>>>cheers
> >>>>>Miller
> >>>>>
> >>>>>On Fri, Jul 16, 2010 at 03:58:24PM -0400, Hans-Christoph Steiner
> >>>>>wrote:
> >>>>>>Hey Miller,
> >>>>>>
> >>>>>>I just noticed that you left out a key part of pd-gui-rewrite in
> >>>>>>0.43. There is the 'set-startup' and 'set-path' messages which
> >>>>>>allows
> >>>>>>pd-gui to get the state of those things without having to open
> >>>>>>the
> >>>>>>respective preference dialog panels. The changes are in
> >>>>>>m_glob.c and
> >>>>>>s_path.c, as well as these lines in pd-gui.tcl:
> >>>>>>
> >>>>>> pdsend "pd set-startup" ;# get ::startup_libraries
> >>>>>>and ::startup_flags lists
> >>>>>> pdsend "pd set-path" ;# get the ::pd_path list
> >>>>>> vwait ::pd_path ;# wait for 'pd' to respond
> >>>>>>
> >>>>>>.hc
> >>>>>>
> >>>>>>----------------------------------------------------------------------------
> >>>>>>
> >>>>>>"We have nothing to fear from love and commitment." - New York
> >>>>>>Senator
> >>>>>>Diane Savino, trying to convince the NY Senate to pass a gay
> >>>>>>marriage
> >>>>>>bill
> >>>>>>
> >>>>>>
> >>>>>>_______________________________________________
> >>>>>>Pd-dev mailing list
> >>>>>>Pd-dev(a)iem.at
> >>>>>>http://lists.puredata.info/listinfo/pd-dev
> >>>>
> >>>>
> >>>>
> >>>>
> >>>>----------------------------------------------------------------------------
> >>>>
> >>>>Programs should be written for people to read, and only
> >>>>incidentally
> >>>>for machines to execute.
> >>>>- from Structure and Interpretation of Computer Programs
>
>
>
> ----------------------------------------------------------------------------
>
> "We have nothing to fear from love and commitment." - New York Senator
> Diane Savino, trying to convince the NY Senate to pass a gay marriage
> bill
Patches item #3035455, was opened at 2010-07-27 14:10
Message generated for change (Comment added) made by eighthave
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3035455&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: bugfix
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Vladimir Gorsunov ()
Assigned to: Nobody/Anonymous (nobody)
Summary: Correct error messages for ALSA device setup
Initial Comment:
-
----------------------------------------------------------------------
>Comment By: Hans-Christoph Steiner (eighthave)
Date: 2010-07-27 15:18
Message:
can you give more information on the problem this patch is addressing?
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3035455&group_…