-
Notifications
You must be signed in to change notification settings - Fork 6
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
Refactor branch Addressing issues #15
Comments
faders work only from Reaper to ProC Reaper is receiving I switched the ReaperOSC file to read possibly the same thing is happening with the scribstrips? which are also not showing anything |
pans cause an error: |
all displays (VU's, Clock, Automode LEDs, Scribstrips) which were working in the previous version are all not working |
Scrub Wheel also not working: |
regarding the displays the address for control24 appears to be 0xF0, 0x13, 0x01 whereas for ProC it is 0xF0, 0x13, 0x00 which would explain why the displays are not working. I made this change to my ReaCommon.py and got my Vus, pan LEDs and automode LEDs working but still no scribs or clock. not sure if @phunkyg had tried to implement this address change elsewhere but it got em working |
I am also getting this error in terminal I am not sure why Reaper is sending from that port (57672 -- ...0.15 is my computer running Reaper the ReaControl.py is running on the pi at ...0.34 ) I don't remember seeing this before. |
I got a bunch of stuff working again except the jog wheel by editing my procontrolosc.py file to mimic the changes made py @phunkyg to control24osc.py in commit https://github.com/phunkyg/ReaControl24/commit/55f0beb7009e025e1027dad7ac70888148887d46 that fixed the fader, auto and pan addressing issue. --edit automode is not working properly: leds respond but auto mode does not change in reaper when pushing automode on the desk. ReaControl.py sends I also edited into ReaCommon.py this was the only way I could get the displays going. I manually editing the addresses which would surely break control24 compatability but I presume there is a way to get procontrolosc.py to change the address from If I can figure out how to push from my pi I will push the changes to my fork or I will transfer the files later today and copy them. |
jog wheel is still being identified as reavpot track 28 instead of reajpot:
Response: This is OK, it gets parsed as a vpot on channel 28 but the class that is actually instantiated is a jpot so the instructions go to the right code. |
Hey @lazlooose So after the Control24 session the ProControl stuff needs catching up. You change will need something like I mentally notes I need to come back to it but I was too tired to pick it up right away. Most of your issues with addressing might be you haven't picked up the OSC file or I haven't updated it for procontrol. As for the jog wheel, I need to see which track the Pro Control addresses it as, and adjust accordingly. IIRC I shoved it on 9 i.e. the first after 'real' tracks but that was a guess. I need to refer back to your map. |
Nothing to worry about here. Your PC uses 'outgoing' ports for all sorts of comms, even going to webpages and such. You hardly ever see them, they are picked by the system. the 'unexpected source address' happens if comms were lost with Reaper. There is no code to disconnect/reconnect yet. Just restart reaper and if that don't work everything and you'll be back in business. |
awesome thanks!
I created a pull request https://github.com/phunkyg/ReaControl24/pull/16 for the fixes to get the faders working I did in procontrolosc.py after that change faders and pans work.
the terminal window is identifying it as 28
wicked, I had tried |
I tried Adding first I tried editing the ProCDesk Class like this:
since I interpreted that as what you were saying but I get an attribute error:
if I put it in the class procscribstrip no error, but no scribs either:
|
regarding the jpot there I checked the map and it is indeed track 28 But there is some kind of attribute error which leads me to believe there is a mistake in the code? it points to line 1666, lol, of Reacommon is the jpot working on the control24?
|
So I've merged your pull requrest to get those changes in and corrected the cmdbytes bit. You were almost there, but I think it needs to be index 2 as they begin at zero. Jpots are added to track 28 for all desks as they are in the _ReaTrack class which is common to both, so in theory should be working on both. We tested on Control24 and they seemed good, are they still a problem on ProC? I could do with a bit of a round-up on this thread to see what's still outstanding edit: looking above, did vpot led's, clocks and VU's come back? Do I need to compare back to the working branch for these to see if there's any missing 'specifics'? |
Something about the most recent merges has completely broken communication for me between ReaControl.py and everything. the desk is recognized ,the logs receive the announcement transimission from the board, but that its it. Reaper can neither receive or send anything. (nothing appearing in the logs. Also Fader movements and button pushes on the ProC do not show up in the logs.
lol, noob mistake! I believe that did it. In order to test I went back to a previous version. the presumably we can use a similar thing to access the other row of scrib strips...(he says excitedly!) Do we need similar classes for the clock, pan leds, Vus and auto LEDs? is it as simple as copying the basic format of what we have tthere and doing the cmdbytes edit? |
Yes exactly, we just need to set the 'bank' byte accordingly but decide how we are going to add them to the desk model. All the other classes I'm assuming are going to be standard, I don't' recall if we had to tweak them for ProControl? Can you remind me? If so it should be easy enough to do this base class/specific class pattern. |
OK I've pushed all the cmdbyte[2] changes |
I can tell that we are making progress since more displays are popping on in my tests but the "can't quit fix" bug makes it hard to test since the desk doesn't really respond to anything in real time and the displays appear randomly. |
Also, from my notes the Clock has an additional byte that is different from C24:
so presumably that could be fixed via |
Yep exactly, stick it in see if it flies |
ok will do, I might need to create a branch without the "quit fix" to keep testing for now though. I think I can go bak to the commit before that and then add the addressing stuff |
now that the osc communication issue related to the quit fix has been resolved for now I can continue testing. working:
not working:
that is all I have noticed for now. |
Testing for the new Refactor branch is underway so far there are some addressing issues for faders, scribs panpots and displays.
The text was updated successfully, but these errors were encountered: