Protonerd / FX-SaberOS

System code for Arduino based Lightsaber replicas for DIYino family boards
Creative Commons Zero v1.0 Universal
92 stars 42 forks source link

currentcolor issue #60

Closed whosle closed 6 years ago

whosle commented 6 years ago

Hello

, this is my 2nd attempt at a home-brew version. I am trying the set-up with string blades ( I like bright) , but will be using all types eventually.

I am using arduino nano, if it matters.

The Error I am getting is:

sketch\Light.cpp: In function 'void getColorFix(uint8_t)':

Light.cpp:974: error: 'currentColor' was not declared in this scope

 currentColor.r = MAX_BRIGHTNESS;

Thanks for any help!

Xamiuk commented 6 years ago

hi whosle, i had this problem before too, check this:

check if this line is uncommented in Config_HW.h the reason why is you define which kind of blade type are you using...

i hope if i help you...

:)

whosle commented 6 years ago

Hi There Xamiuk,

Unfortunatly, I have already tried that.

Whenever the #define LEDSTRINGS is an activated function, the sketch won't compile.

I have successfully compiled it on Star-LED and uploaded my sketch.

I am very happy I have it running, however , I would like the LED String functionality.

Cheers!

Xamiuk commented 6 years ago

hi whosle, can you show me what homebrew are you using? the diagram i mean. :)

whosle commented 6 years ago

Hi Xamiuk,

I used the same wiring diagram as provided on here/wiki. The fifth/sixth/seventh one down in wiring.

Currently the sketch is running and uploaded, and I have full functionality on all blade types except string blades, which will not compile.

Cheers!

Xamiuk commented 6 years ago

try something in Config_SW.h change this configuration:

/* COLOR CHANGE OPTIONS

/* maybe this will help you... :/

Protonerd commented 6 years ago

Try going back to earlier releases instead of using the master. Since ledstrings became a rarity, I havent checked that blade type lately, so it well might be that it doesnt compile

whosle commented 6 years ago

Dang. Ok, I will try an earlier release. I will go back and find the latest release that will work.

Do you have a guess which release I should use?

When I track down the issue, I will post my results here.

Thanks again for your effort in this!

Protonerd commented 6 years ago

no need, I fixed it, was very easy. Try now.