Farben nach R,G,B umwandeln in 4-stell. hex-code?

[HaWe]

...danke für die Idee..und, schon getested...?

[Moppi]

Nein, ich habe das von Hand gerechnet. Du hattest ja nur auf dem Rückweg Probleme, mit den Berechnungen, daher denke ich, dass es was mit dem 8 Bit-Gedöhns zu tun hat. Ich habe die Rechnung genau aufgeschlüsselt. Kann man nachvollziehen und überprüfen.

[HaWe]

leider nein, stimmt immer noch nicht.

Bitte teste es doch vorher selber, bevor du einen falschen Lösungsweg postest.

uint16_t ColorRGB216bit(uint16_t R, uint16_t G, uint16_t B) {

   return  (uint16_t)((R/8)*2048) | (uint16_t)((G/4)*32) | (uint16_t)(B/8);
}


void Color16bit2colorRGB(uint16_t color16, uint16_t &R, uint16_t &G, uint16_t &B) {

   R = (uint8_t)(color16 / 2048)*8; ;
   G = (uint8_t)(color16 & 2016)/8;;
   B = (uint8_t)(color16 & 31)*8;
   Serial.println("");
   Serial.println("Unterprogramm color16 zu RGB:");
   Serial.println( (String)"color16="+(String)color16);
   Serial.println( (String)"R="+(String)R);
   Serial.println( (String)"G="+(String)G);
   Serial.println( (String)"B="+(String)B); 
}

void setup() {
  // put your setup code here, to run once:

   uint16_t r=255, g=102, b=78, 
            col16=0;               // 0xFF66B2;  // Dark Pink (255,102,78)


   Serial.begin(115200);
   delay(3000);
   
   Serial.println("Hauptprogramm vor Aufruf:");
   Serial.println( (String)"col16="+(String)col16);
   Serial.println( (String)"r="+(String)r);
   Serial.println( (String)"g="+(String)g);
   Serial.println( (String)"b="+(String)b);
   Serial.println("");
   
   col16=ColorRGB216bit(r,g,b);
   r=g=b=0; // Rücksetzen!

   Serial.println("Hauptprogramm nach rgb zu col16-Berechnung (rgb gelöscht):");
   Serial.println( (String)"col16="+(String)col16);
   Serial.println( (String)"r="+(String)r);
   Serial.println( (String)"g="+(String)g);
   Serial.println( (String)"b="+(String)b);
   
   Color16bit2colorRGB(col16, r, g, b);
   Serial.println("");
   Serial.println("Hauptprogramm nach col16 zu rgb:");
   Serial.println( (String)"col16="+(String)col16);
   Serial.println( (String)"r="+(String)r);
   Serial.println( (String)"g="+(String)g);
   Serial.println( (String)"b="+(String)b);   
    
}

void loop() {
  // put your main code here, to run repeatedly:

}

Hauptprogramm vor Aufruf:
col16=0
r=255
g=102
b=78

Hauptprogramm nach rgb zu col16-Berechnung (rgb gel⸮scht):
col16=64297
r=0
g=0
b=0

Unterprogramm color16 zu RGB:
color16=64297
R=248
G=4
B=72

Hauptprogramm nach col16 zu rgb:
col16=64297
r=248
g=4
b=72

[Ceos]

Ich habe die Berechnung gerade einmal in einem Excel Blatt angelegt um es verständlich zu machen

https://puu.sh/BsLSg/6fe9cf4a10.png

ich liefere später noch etwas code nach, wenn mein kopf sich von dem krampf auf arbeit erholt hat XD

ich sehe im moment nur miraculi vor augen

man kann den leichten farbunterschied durch die skallierung von 5 bzw. 6 bit doch besser erkennen als mir lieb ist XD

[Moppi]

HaWe, soll das auf Arduino laufen?

Nächstes mal komme ich vorbei und haue Dich! Schreibt der Kerl "...bevor du einen falschen Lösungsweg postest". Tse, tse, tse!

Was hältst Du denn von dieser Ausgabe mit Deinem Programm? Sieht das besser aus? :

Hauptprogramm vor Aufruf:
col16=0
r=255
g=102
b=78

Hauptprogramm nach rgb zu col16-Berechnung (rgb gelöscht):
col16=64297
r=0
g=0
b=0

Unterprogramm color16 zu RGB:
color16=64297
R=248
G=100
B=72

Hauptprogramm nach col16 zu rgb:
col16=64297
r=248
g=100
b=72

[HaWe]

da bin ich jetzt gespantn (ich habs noch nicht hingekriegt, evtl was falsch verstanden)

[Moppi]

Ich habe doch geschrieben, wo ich denke, dass das Problem liegt. Dass Du die Berechnung auf 8 Bit erzwingst.

mach bitte so:

   
void Color16bit2colorRGB(uint16_t color16, uint16_t &R, uint16_t &G, uint16_t &B) {

   R = (color16 / 2048)*8; ;
   G = (color16 & 2016)/8;;
   B = (color16 & 31)*8;
   Serial.println("");
   Serial.println("Unterprogramm color16 zu RGB:");
   Serial.println( (String)"color16="+(String)color16);
   Serial.println( (String)"R="+(String)R);
   Serial.println( (String)"G="+(String)G);
   Serial.println( (String)"B="+(String)B); 
}

Ich hoffe, dass es jetzt besser ist!

[HaWe]

hatte ich das nicht gemacht?

R = (uint8_t)(color16 / 2048 )*8; ;
G = (uint8_t)(color16 & 2016)/8;;
B = (uint8_t)(color16 & 31)*8;

ach so, du meinst ich soll das nicht machen?
Das wäre nicht korrekt, denn es sind ja uint8_t Zahlen, die zurückgegeben werden sollen.
Probier doch mal bitte selber!

[Moppi]

Ich hatte das hier schon geschrieben (Beirag #20): https://www.roboternetz.de/community...l=1#post646541

Ich denke, er führt die Berechnung mit 8 Bit aus.

so:  (uint8_t)(color16 / 2048 )*8;

Du brauchst aber 16 Bit während der Berechnung, weil zumindest Dein Farbwert color16 ja 16 Bit umfasst.

Du müsstest hinterher eine Umwandlung vornehmen

(uint8_t)R = (color16 / 2048 )*8;

Aber das funktioniert nicht!

Deshalb habe ich jetzt das probiert, da gibt es kein Gemecker:

void Color16bit2colorRGB(uint16_t color16, uint16_t &R, uint16_t &G, uint16_t &B) {

   R = (color16 / 2048)*8; ;
   G = (color16 & 2016)/8;;
   B = (color16 & 31)*8;
   R = (uint8_t)R;
   G = (uint8_t)G;
   B = (uint8_t)B;
   
   Serial.println("");
   Serial.println("Unterprogramm color16 zu RGB:");
   Serial.println( (String)"color16="+(String)color16);
   Serial.println( (String)"R="+(String)R);
   Serial.println( (String)"G="+(String)G);
   Serial.println( (String)"B="+(String)B); 
}

[HaWe]

bittebittebitte mal selber testen!

- - - Aktualisiert - - -

...ach so, jetzt habe ich deinen Einwand verstanden!
Klammer ums ganze, dann erst casten!

- - - Aktualisiert - - -

nein, stimmt auch nicht dann.