monome / norns

norns is many sound instruments.
http://monome.org
GNU General Public License v3.0
621 stars 144 forks source link

allow midi map/learn for taper parameters #1241

Closed catfact closed 3 years ago

catfact commented 3 years ago

i haven't been keeping up with these features, but it seems like parameter types now need to be explicitly made mappable in the param type class. this isn't happening for Taper, which is a pretty serious limitation.

seems like easy fix... is it enough to add something like

Taper.new(...)  
   local o =... 
   ...
   if allow_pmap == nil then o.allow_pmap = true else o.allow_pmap = allow_pmap end

?

i also may be misunderstanding something, this report is hearsay

ngwese commented 3 years ago

i’ll take a look, likely due to the bit i added to which allows one to prevent params from being mapped if need be. i thought i had done it in a compatible manner but will double check.

jasonw22 commented 3 years ago

I wonder if there's a larger issue? Just got a new 16n and went to try mapping grain size and density in angl but they weren't mappable.

ngwese commented 3 years ago

it looks like angl uses taper parameters almost exclusively.

https://github.com/tehn/ash/blob/master/angl.lua#L120

EDIT: will try to get this investigated/fixed this weekend

jasonw22 commented 3 years ago

@ngwese i tried applying the patch from this commit manually to my local norns (which is up to date) but the angl params still aren't mappable. Is there anything else I should do?

tehn commented 3 years ago

@jasonw22 i just checked and mine seems to be mappable. did you do a SYSTEM > RESTART after git pull?

jasonw22 commented 3 years ago

If you mean SYSTEM > RESET, yes. However I didn't do a git pull, I just patched the two files mentioned in the commit, manually.

tehn commented 3 years ago

can you try a pull so your system is on latest main? seems to be working fine here.

jasonw22 commented 3 years ago

git pull in /home/we/norns, followed by SYSTEM > RESET resulted in a hang on the resulting screen saying reset. Did a hard reset and the menu never comes up, but I can still SSH in.

tehn commented 3 years ago

you'll need to:

cd norns git pull ./waf

or, since you already pulled, try

norns/edge.sh

(which automates the above)

On Fri, Nov 27, 2020 at 5:52 PM Jason Wehmhoener notifications@github.com wrote:

git pull in /home/we/norns, followed by SYSTEM > RESET resulted in a hang on the resulting screen saying reset. Did a hard reset and the menu never comes up, but I can still SSH in.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/monome/norns/issues/1241#issuecomment-735006213, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAB4I4CFGWULG47ZHHQI6O3SSAUTTANCNFSM4TWM3URA .

jasonw22 commented 3 years ago

Thanks for the extra help. Unfortunately, I seem to be having some issues.

10.0.0.23 ~/norns $ ./edge.sh
Already up-to-date.
Setting top to                           : /home/we/norns 
Setting out to                           : /home/we/norns/build 
Checking for 'gcc' (C compiler)          : /usr/bin/gcc 
Checking for 'g++' (C++ compiler)        : /usr/bin/g++ 
Checking for program 'pkg-config'        : /usr/bin/pkg-config 
Checking for 'alsa'                      : yes 
Checking for 'libudev'                   : yes 
Checking for 'libevdev'                  : yes 
Checking for 'liblo'                     : yes 
Checking for 'cairo'                     : yes 
Checking for 'cairo-ft'                  : yes 
Checking for 'lua53'                     : yes 
Checking for 'nanomsg'                   : yes 
Checking for 'avahi-compat-libdns_sd'    : yes 
Checking for 'sndfile'                   : yes 
Checking for libmonome                   : yes 
Checking for supercollider               : yes 
Checking for program 'dpkg-architecture' : /usr/bin/dpkg-architecture 
Checking boost includes                  : 1.62.0 
'configure' finished successfully (2.465s)
Waf: Entering directory `/home/we/norns/build'
[ 1/58] Compiling third-party/link-c/ableton_link.cpp
[34/58] Compiling maiden-repl/src/page.c
[35/58] Compiling maiden-repl/src/pages.c
[36/58] Compiling maiden-repl/src/ui.c
In file included from ../maiden-repl/src/pages.c:1:0:
../maiden-repl/src/page.h:7:21: fatal error: ncurses.h: No such file or directory
 #include <ncurses.h>
                     ^
compilation terminated.

../maiden-repl/src/page.c:2:21: fatal error: ncurses.h: No such file or directory
 #include <ncurses.h>
                     ^
compilation terminated.

../maiden-repl/src/ui.c:9:19: fatal error: panel.h: No such file or directory
 #include <panel.h>
                   ^
compilation terminated.
tehn commented 3 years ago

@jasonw22 apologies for all of the trouble, i didn't realize this was such a pain

get ncurses this way:

sudo apt-get install libncurses5-dev libncursesw5-dev

then re-run the compile

jasonw22 commented 3 years ago

Thanks so much! Norns is functional, and taper parameters are mappable.