Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

allow midi map/learn for taper parameters #1241

Closed
catfact opened this issue Nov 15, 2020 · 12 comments
Closed

allow midi map/learn for taper parameters #1241

catfact opened this issue Nov 15, 2020 · 12 comments
Assignees

Comments

@catfact
Copy link
Collaborator

@catfact catfact commented Nov 15, 2020

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
Copy link
Member

@ngwese ngwese commented Nov 16, 2020

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.

@ngwese ngwese self-assigned this Nov 16, 2020
@jasonw22
Copy link

@jasonw22 jasonw22 commented Nov 21, 2020

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
Copy link
Member

@ngwese ngwese commented Nov 21, 2020

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
Copy link

@jasonw22 jasonw22 commented Nov 24, 2020

@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
Copy link
Member

@tehn tehn commented Nov 24, 2020

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

@jasonw22
Copy link

@jasonw22 jasonw22 commented Nov 24, 2020

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
Copy link
Member

@tehn tehn commented Nov 25, 2020

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

@jasonw22
Copy link

@jasonw22 jasonw22 commented Nov 27, 2020

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
Copy link
Member

@tehn tehn commented Nov 27, 2020

@jasonw22
Copy link

@jasonw22 jasonw22 commented Dec 1, 2020

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
Copy link
Member

@tehn tehn commented Dec 1, 2020

@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
Copy link

@jasonw22 jasonw22 commented Dec 4, 2020

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

4 participants