segfault16 / modular-led-controller-workstation

:musical_note: :rainbow: Real-time LED strip music visualization using Python and the ESP8266 or Raspberry Pi
MIT License
23 stars 9 forks source link

Running With DotStar-apa102c #121

Open maxycatz opened 5 years ago

maxycatz commented 5 years ago

How is the server run for DotStar or APA102C leds? I have both adafruit's DotStar and tinue APA102 libraries installed.

When running the server with $ sudo python3 server.py -D DotStar I get: server.py: error: argument -D/--device: invalid choice: 'DotStar' (choose from 'FadeCandy', 'RaspberryPi')

Many thanks.

segfault16 commented 5 years ago

@maxycatz Hi, we haven't had any dotstar LEDs to test yet. I can prepare a branch for you if you're willing to test it.

maxycatz commented 5 years ago

DotStar is listed as a class in audioled./devices.py?

But, yes; certainly will test.

segfault16 commented 5 years ago

@maxycatz If prepared a PR #122 for DotStar LED's. Please feel free to test it.

maxycatz commented 5 years ago

Wow, cheers. Will test today.

maxycatz commented 5 years ago

Thought I had updated this; maybe I forgot to send.

Although the apa102 library is installed, I can only get apa102 library to load if I transfer tinue's colorsheme and driver folders into this project folder and then call via 'import from driver' .

Is this down to lack of understanding?

segfault16 commented 5 years ago

@maxycatz I've updated the import path. Can you test again?

segfault16 commented 5 years ago

@maxycatz Any updates?

maxycatz commented 5 years ago

Sorry about delay; holidays and kids out of school; will try to asap.

ChasonDeshotel commented 4 years ago

usage: server.py [-h] [-N NUM_PIXELS] [-R NUM_ROWS] [--device_candy_server DEVICE_CANDY_SERVER] [--device_panel_mapping DEVICE_PANEL_MAPPING] [-A AUDIO_DEVICE_INDEX] [-p PORT] [-C CONFIG_LOCATION] [--no_conf] [--no_store] [-D {FadeCandy,RaspberryPi}] [-DC DEVICE_CONFIG] [-P] [--strand] server.py: error: argument -D/--device: invalid choice: 'DotStar' (choose from 'FadeCandy', 'RaspberryPi')

DotStar is not listed in "allowed_devices". If you add it, you get other errors.

Sort of related:

I'm using the GenericUDP device I made a while back (for APA102s driven by ESP8266)... somewhere along the way I didn't push all of the changes or it broke (haven't touched this project in months) After pulling the new code and trying to determine where all I need to add a device... in the end I just renamed my GenericUDP method to FadeCandy and it works like a champ

I suppose I should make an issue but I don't understand why those would be hardcoded instead of "allowed_devices" checking globals to see what all methods are available in audioled/devices.py

segfault16 commented 4 years ago

Hi @ChasonDeshotel They are hardcoded since I only want tested devices to be available on the cmd line options. See https://github.com/segfault16/modular-led-controller-workstation/pull/122