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

Hallo,
Adafruit verwendet für Farben einen 4-stelligen hex-Code, der nicht mit den 6-stelligen hex-Codes identisch ist, wie man sie von html kennt.
Kennt jemand eine Konvertierungs-Routine oder Vorschrift, um die R,G,B-Werte (jew. 0-255) in den Adafruit Code umzurechnen?

Code:

---


#define ILI9341_BLACK      0x0000  ///<  0,  0,  0
#define ILI9341_NAVY        0x000F  ///<  0,  0, 123
#define ILI9341_DARKGREEN  0x03E0  ///<  0, 125,  0
#define ILI9341_DARKCYAN    0x03EF  ///<  0, 125, 123
#define ILI9341_MAROON      0x7800  ///< 123,  0,  0
#define ILI9341_PURPLE      0x780F  ///< 123,  0, 123
#define ILI9341_OLIVE      0x7BE0  ///< 123, 125,  0
#define ILI9341_LIGHTGREY  0xC618  ///< 198, 195, 198
#define ILI9341_DARKGREY    0x7BEF  ///< 123, 125, 123
#define ILI9341_BLUE        0x001F  ///<  0,  0, 255
#define ILI9341_GREEN      0x07E0  ///<  0, 255,  0
#define ILI9341_CYAN        0x07FF  ///<  0, 255, 255
#define ILI9341_RED        0xF800  ///< 255,  0,  0
#define ILI9341_MAGENTA    0xF81F  ///< 255,  0, 255
#define ILI9341_YELLOW      0xFFE0  ///< 255, 255,  0
#define ILI9341_WHITE      0xFFFF  ///< 255, 255, 255
#define ILI9341_ORANGE      0xFD20  ///< 255, 165,  0
#define ILI9341_GREENYELLOW 0xAFE5  ///< 173, 255,  41
#define ILI9341_PINK 0xFC18 ///< 255, 130, 198

---

Die Farben sind MSB>LSB 5bit R, 6bit G und 5bit B codiert

du könntest also ganz simpel mit maskieren und shiften 3 bytes daraus machen wie du es gewohnt bist

R = (0b1111100000000000 & input) >> 11;
G = (0b0000011111100000 & input) >> 5;
B = (0b0000000000011111 & input);

https://en.wikipedia.org/wiki/List_o...tes#16-bit_RGB

Zitat:

---

Zitat von Ceos

Die Farben sind MSB>LSB 5bit R, 6bit G und 5bit B codiert

du könntest also ganz simpel mit maskieren und shiften 3 bytes daraus machen wie du es gewohnt bist

R = (0b1111100000000000 & input) >> 11;
G = (0b0000011111100000 & input) >> 5;
B = (0b0000000000011111 & input);

https://en.wikipedia.org/wiki/List_o...tes#16-bit_RGB

---

aha, danke!
verstehe ich das richtig:
also z.B. für LightHeliotrop (220, 139, 231):

uint16_t R = (0b1111100000000000 & 220) >> 11;
uint16_t G = (0b0000011111100000 & 139) >> 5;
uint16_t B = (0b0000000000011111 & 231);

uint16_t colorcode=R+G+B;

?

- - - Aktualisiert - - -

hmm, nee, scheint genau anders rum zu arbeiten...

mit der Bitarithmetik komme ich aber nicht zurecht - wie sähe die Umkehrfunktion aus, aus den RGW-Einzelwerten die hex colorcode Zahl?

einfach
R =220 << 11;
G = 139 << 5;
B = 231;
colorcode=R+G+B;

?

PS,
ja, schein zu klappen so rum...
Danke!

ne das war die umrechnung von 16bit (sog. 65k Farbraum) auf R G B

wolltest du es umgekehrt? Sorry :)

RGB16 = ((uint16_t)R << 11) | (((uint16_t)G << 5) & 0b0000011111100000) | ((uint16_t)B & 0b0000000000011111);

(uint16_t) muss zwingend vorher stehen und castet den 8bit auf einen 16bit, sonst fallen die bits nach dem 8ten ins nirvana :D

ich kenne gerade nicht den 16bit unsigned int typ für arduino sorry

(PS: Das ist auch der Grund weswegen ich so einen sprechenden typen verwende, egal welcher compiler oder welche umgebnung jeder macht es anders)

danke vielmals, so passts! 8)

das erste in eine Funktion zu packen mit Rückgabewert, ist ja einfach -

Code:

---

//---------------------------------------------------------
// ColorRGB2color16bit
//---------------------------------------------------------

int16_t ColorRGB2color16bit(uint16_t R, uint16_t G, uint16_t B) {

  return  ((uint16_t)R << 11) | (((uint16_t)G << 5) & 0b0000011111100000) | ((uint16_t)B & 0b0000000000011111);
]

---

kann man die Rückwärtstransformierung so programmieren (bin mir unsicher wegen pass per reference):

Code:

---

//---------------------------------------------------------
// Color16bit2RGB
//---------------------------------------------------------
 
void Color16bit2RGB(uint16_t color16, uint16_t &R, uint16_t &G, uint16_t &B) {

  R = (0b1111100000000000 & color16) >> 11;
  G = (0b0000011111100000 & color16) >> 5;
  B = (0b0000000000011111 & color16);
}

---

im zweifelsfall mit pointer

Code:

---

void Color16bit2RGB(uint16_t color16, uint8_t* R, uint8_t* G, uint8_t* B) {

  *R = (uint8_t)((0b1111100000000000 & color16) >> 11);
  *G = (uint8_t)((0b0000011111100000 & color16) >> 5);
  *B = (uint8_t)((0b0000000000011111 & color16));
}

---

muss gestehen dass ich solche reference calls meide, weils mir leicht spanisch ist :D
pointer sind für mich persönlich einfacher zu handhaben

beim aufruf musst du dann natürlich dioe adressen auflösen

Code:

---

uint8_t myR,myG,myB;
Color16bit2RGB(some16bitColor, &myR, &myG, &myB);
printf("<font color=\"#%00h%00h%00h\">",myR,myG,myB);

---

PS: ich habe die einzelnen farben mal auf 8 bit reduziert und einen expliziten cast mit eingebaut damit alles sauber aussieht :)

danke für die Mühe!
Ich muss allerdings auch etwas gestehen...: dass mir das mit diese Pointern mit * und dann doch wieder & im Hauptprogramm noch viel unklarer ist, mit Pointer hin und vorher und hinterher referenzieren oder dereferenzieren oder wie auch immer das heißt - wenn möglich würde ich lieber eindeutige Variablennamen im Hauptprogramm verwenden, die dann per Referenz übergeben werden und auf die man dann nach Ende der Funktion wieder als aktuaisierte Werte per Variabennamen zugreifen kann.
Aber auch da komme ich immer ins Schleudern. Hatte gehofft, das könnte irgendwer aus dem Stegreif... 8)

so, nochmal getestet - das mit dem "&" klappt...
Rest nochmal in Arbeit...



funktioniert noch nicht ganz perfekt...

- - - Aktualisiert - - -

Code:

---


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

  return  ((uint16_t)R << 11) | (((uint16_t)G << 5) & 0b0000011111100000) | ((uint16_t)B & 0b0000000000011111);
}


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

  R = (uint8_t)((0b1111100000000000 & color16) >> 11);
  G = (uint8_t)((0b0000011111100000 & color16) >> 5);
  B = (uint8_t)((0b0000000000011111 & color16));
  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;              // Dark Pink (255,102,78)


  Serial.begin(115200);
 
  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 Einzelwerte dann 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:

}

---

Ausgabe:

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

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

Unterprogramm color16 zu RGB:
color16=64718
R=31
G=38
B=14

Hauptprogramm nach col16 zu rgb:
col16=64718
r=31
g=38
b=14

kann es sein, dass die zurück-Richtung clolor16 => r,g,b nicht richtig rechnet? Zumindest schein es so nach direkter Kontrolle der (lokalen) Rechenschritte
R = (uint8_t)((0b1111100000000000 & color16) >> 11);
G = (uint8_t)((0b0000011111100000 & color16) >> 5);
B = (uint8_t)((0b0000000000011111 & color16));

denn die geben ja andere Ergebnisse als die hinwärts-Richtung r,g,b => color16:


die hinwärts-Richtung ermittelt ja aus den 3 rgb-Werten
r=255
g=102
b=78
die color16-Zahl 64718

die Rückwärts-Funktion dann aber aus color16=64718
die (abweichenden) rgb-Werte
R=31
G=38
B=14

Hatte ich etwas falsch c+p'tet ?

Sorry dass ich nicht früher antworten konnte, hatte nicht viel gelegenheit überhaupt mal länger als 5 Minuten ins Forum zu schauen.

Mir ist da ein Denkfehler passiert, bei der Rückrechnung der einzelnen Komponenten stimmt zumindest schonmal deine Formel, bei der Umrechnung von Komponenten auf RGB16 allerdings ist rigendwas schief, da komme ich auf einen anderen Wert.

Der primäre Denkfehler liegt jedoch beim Rückrechnen von RGB16 auf R G B

G wird hier mit 6Bit aufgelöst, also müsste man streng genommen >> 6 statt >> 5 machen ... was allerdings der größte Denkfehler von mir war ... die Werte sind doch nur 5 bzw. 6Bit groß und müssten ja nochmal auf 8Bit aufgeblasen werden (inklusive eines Verlust der Auflösung selbsttverständlich)

Und ich glaube beim Maskieren und schieben ist auch noch ein dreher drinne ... ich habe immer die MSB ausmaskiert und da kann nur blödsinn bei rauskommen XD

Ich versuche heute Nachmittag mal eine Korrektur, da ich jetzt noch was wegschaffen muss bevor hier alles abbrennt :)

besten Dank schon im vorraus - mach dir aber keinen Stress, es ist absolut nicht eilig, es hatte mich nur stutzig gemacht!

Code:

---

Das mit dem Konvertieren der Farben ist so eine Sache, das kann man machen, es kommt aber nicht unbedingt das gewünschte Ergebnis heraus. Da die Farbgewichtung auch eine Rolle spielt. Ohne das weiter kompliziert zu machen, kommt es bei der einfachen Farbanpassung per Logikoperatoren teils zu falschen Farbdarstellungen beim Ziel. Das sollte man nur mal so im Hinterkopf haben.
Ansonsten ist RGB-5-6-5, 5Bit für Rot, 6Bit für Grün und 5Bit für Blau.
Bei RGB-8-8-8, jeweils 8Bit für eine Farbe.

RGB 8-8-8: 11111111-11111111-11111111
RGB 5-6-5: xxx11111-xx111111-xxx11111

Schiebe die ersten 8 Bit nach rechts, um 3 Stellen.
Schiebe die zweiten 8 Bit nach rechts, um 2 Stellen.
Schiebe die dritten 8 Bit nach rechts, um 3 Stellen.

int rot1 = rot/8; int gruen1 = gruen/4; int blau1 = blau/8;

Das Problem dabei: Grün ist anders gewichtet, das kann sich in einem Grünstich des Bildes bemerkbar machen.

Rückwärts geht es genau andersrum:

Schiebe die ersten 5 Bit nach links, um 3 Stellen.
Schiebe die zweiten 6 Bit nach links, um 2 Stellen.
Schiebe die dritten 5 Bit nach links, um 3 Stellen.

int rot = rot1*8; int gruen = gruen1*4; int blau = blau1*8;

--------------------------

Die einzelnen Farben aus rgb-8-8-8 nach rgb-5-6-5:

r = rgb888/524288;
g = (rgb888/1024)&63;
b = (rgb888/8)&31;

Ganze RGB-Werte umrechnen:

rgb565 =  ((  ((rgb888/524288)*64) | ((rgb888/1024)&63)  )*32) | ((rgb888/8)&31);

10111000 10101100 10101000 =>  ((1472 | 43) * 32) | 21 => 1011110101110101

und zurück (als Probe):

rgb888 = ((rgb565 & 63488) * 256) | ((rgb565 & 2016) * 32) | ((rgb565 & 31) * 8)

1011110101110101 => (12058624 | 44032) | 168 => 10111000 10101100 10101000

---

danke für den Post!
ich habe jetzt neu codiert:

Code:

---


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

  return  ((uint16_t)R << 11) | (((uint16_t)G << 5) & 0b0000011111100000) | ((uint16_t)B & 0b0000000000011111);
}


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

  R = (uint8_t)(color16/524288);
  G = (uint8_t)(color16/1024);
  B = (uint8_t)((color16/8)&31);
  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:

}

---

und bekomme jetzt den output:

Code:

---

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

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

Unterprogramm color16 zu RGB:
color16=64718
R=0
G=63
B=25

Hauptprogramm nach col16 zu rgb:
col16=64718
r=0
g=63
b=25

---

hmmmm... :confused:

Code:

---

G = (uint8_t)(color16/1024);

muss, wenn dann so:

G = (uint8_t)(color16/1024)&63;

---

Allerdings schreibst Du "color16". sind das als Quelle 16Bit-Farben? RGB-8-8-8 sind 24Bit-Farben. RGB-5-6-5 sind 16Bit-Farben.

ja, es handelt sich immer um die 16-bit Zahlen
r,g,b haben immer die volle range von 0-255, die dann nur anders in die 16- bzw. auch optional 24-bit oder 32-bit truecolor umkodiert werden sollen.

Ich hatte gelesen, Du schriebst, dass nicht so einfach ist, wie in HTML - Farben für Webseiten. Die sind dort 8bit-8bit-8bit (style="color:#452f63"), 45h - 2fh - 63h. Macht 24Bit.

Von welchem Farbformat, nach welchem muss es denn?

"r,g,b haben immer die volle range von 0-255, die dann nur anders in die 16- bzw. auch optional 24-bit oder 32-bit truecolor umkodiert werden sollen."

Dann ist es 8-8-8 nach 5-6-5. Wo ist der Fehler?

Code:

---

sinngemäß wäre so richtig:

G = (uint8_t)(color24/1024)&63;

---

Allerdings kann man die 24Bit-Farbe natürlich nur in ein 32Bit-Register packen. Um in einem Stück zu rechnen.
Dumm gelaufen, wenn der Arduino nur 16Bit-Register hat ;-) Vor Jahren noch hätte ich das jetzt einfach auf 16Bit-Operationen aufgeteilt, als ich noch in Übung war. So muss ich nochmal drüber nachdenken.

Nachtrag:

Vorher war der ganze 24Bit-Farbwert: rgb888

Wenn die Farbwerte einzeln vorliegen, ist es so, wie ich Anfangs schrieb:

int rot1 = rot/8; int gruen1 = gruen/4; int blau1 = blau/8;

So wird aus Rot mit 8 Bit, Rot mit 5 Bit, aus Grün mit 8 Bit, wird Grün mit 6 Bit, aus Blau mit 8 Bit, wird Blau mit 5 Bit.

also noch mal von vorn:
r,g,b-Werte sind ja als Vorgabe universell,
weiß ist immer (255, 255,255)
und rot immer (255,0,0 )
und "Dark Pink" z.B. (255,102,78 ) etc.

in html habe ich meist 24-bit, das ist einfach: color24 = (r<<16)+(g<<8 )+b

hier geht es aber jetzt um die color16 - Kodierung (hin und zurück).

Demnach müsste es so sein, von R,G,B Einzelwerten zu 16Bit-Werten:

Code:

---

rgb =  ((rot/8)*2048) | ((grün/4)*32) | (blau/8);

---

und zurück müsste dann so sein:

Code:

---

rot = (rgb/2048)*8;
grün = (rgb&2016)/8;
blau = (rgb&31)*8;

---

hast du das mal selber getestet? Ich hatte doch den Arduino Code gepostet...

Code:

---

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

  return  ((R/8)*2048) | ((G/4)*32) | (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:

}

---

ergibt:

Code:

---

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

---

Teste doch bitte mal selber, sei so nett... 8)

Ich guck mal, ob noch ein Fehler drin ist.

Hier mal die Hin-Rechnung:

Code:

---

r=255, g=102, b=78

r=11111111, g=01100110, b=01001110

muss ergeben: 11111 011001 01001 = 64297

rgb =  ((rot/8)*2048) | ((grün/4)*32) | (blau/8);

rgb    = 11111 000000 00000
        | 00000 011001 00000
        | 00000 000000 01001
        --------------------
          11111 011001 01001

        = 64297

---

und das Ganze zurück:

Code:

---

rgb = 64297;

rot = (rgb/2048)*8;    =>      11111000  => 248
grün = (rgb&2016)/8;    =>      01100100  => 100
blau = (rgb&31)*8;      =>      01001000  => 72

---

So weit ich das sehe, ist das richtig so, wenn ich die Bitmuster vergleiche.

Mit den Werten Rot=248, Grün=100 und Blau=72 müsste das Hin und Zurück dann stimmen.
Auf die Originalwerte kann man nicht kommen, weil man 24Bit-Farbe auf 16Bit-Farbe komprimiert, das ist verlustbehaftet - die unteren Bits gehen hier verloren und können nicht rekonstruiert werden. 16Bit-Farben haben einen kleineren Farbraum als 24Bit-Farben.

Allerdings benötigt man für diese Berechnungen:

Code:

---

rot = (rgb/2048)*8; ... etc.

---

16 Bit. Die Berechnung muss mit 16 Bit stattfinden und dann am Ende könnte man auf 8 Bit umwandeln, weil im Ergebnis nur noch die unteren 8 Bit der Berechnung belegt sind.

Ich denke Dein Problem liegt hier:

Code:

---

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

---

sieht für mich so aus, als ob die gesamte Berechnung mit 8 Bit durchgeführt werden soll(!?).

Eventuell so besser (ohne jede Gewähr):

Code:

---

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

---

Weiß ich aber nicht, weil ich diese sprachspezifischen Sachen hier nicht gut so kenne. Aber ich denke, Du weißt, was ich meine.

Oder mach es so:

Code:

---

int R = (color16 / 2048)*8;

---

Die obersten 8 Bit sind nach erfolgter Berechnung sowieso nicht belegt. Normalerweise switcht man auch zwischen den Datentypen nicht so oft hin und her. Das dürfte Zeit kosten, wenn aus 16 Bit-Operanden 8 Bit-Operanden gemacht werden und umgekehrt. Da sind bestimmt unnütze Operationen, die der Compiler für Dich einbaut. Ich würde es bei den Datentypen für die Farben immer bei 16 Bit belassen. Vielleicht kann man das später aus Platzgründen optimieren, aber erst mal sollte man schauen, dass es funktioniert.

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

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.

leider nein, stimmt immer noch nicht.

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

Code:

---

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:

}

---

Code:

---

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

---

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

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? :

Code:

---

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

---

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

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

mach bitte so:

Code:

---

 
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!

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!

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.

Code:

---

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

Code:

---

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

---

Aber das funktioniert nicht!

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

Code:

---

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);
}

---

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.

Schau noch mal im letzten Beitrag vor Deinem. Habe es geändert.

Allerdings glaube ich auch, dass das

Code:

---

R = (uint8_t)R;

---

überflüssig ist. Weil der kleinste Typ, den der Arduino kennt, ist: integer
Er würde den von Dir geglaubten 8-Bit-Wert vermutlich als Integer ablegen/speichern. So, wie er nach der Berechnung schon vorliegt.

das habe ich verstanden und auch so gemacht.
aber nein, der kleinste Wert ist uint16_t , nicht int, denn int ist bei mir int32_t (ARM Cortex M0 = Arduino M0/Zero), deshalb ist es immer explizit auf uint16_t oder uint8_t gecastet.

Tipps sind wirklich sehr willkomen, aber bitte immer vorher selber testen!

Na ja, ich hatte geschrieben: "vermutlich" und vorher schon, dass ich mich mit den Typen nicht so genau bei den Teilen auskenne.

Aber um das aufzuklären: Gibt es noch ein Problem?
Und welchen Tipp sollte ich testen - was soll ich da testen? - k.A.

:-)

Der Code soll mit dem expliziten Casting damit eben auch auf 8bit Arduinos lauffähig sein.

Teste einfach meinen Testcode, in den du deine Algorithmen einsetzt, du bekommst dann ja über Serial die Debug-Informationen zur Kontrolle.

Da habe ich noch byte gefunden:

Code:

---

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

---

das ist doch wurscht, byte == uint8_t

bitte testen und Serial Meldungen posten!

Spuckt genau dasselbe aus.

Falls der Controller nur 8 Bit beherrschen würde, müsste man die 16-Bit-Farbwerte getrennt in Low- und High-Byte (MSB und LSB) ablegen und anders rechnen.

Code:

---

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

---

Der Grund ist doch klar:
ein 16bit Integer dividiert durch 2048 ist gleich >>11, und damit immer ein 5bit Wert, und passt damit immer in ein 8bit Byte.

warum sollte die MCU nur 8 bit beherrschen? die 8bit AVRs rechnen bis zu int32_t, und die 32bit MCUs bis int64_t.
Ich bin mal auf Ceos' Lösung gespannt, aber ist ja eh nicht so schrecklich eilig.

Im Übrigen war es die falsche Schreibweise, die zu den merkwürdigen Ergebnissen führte:

Code:

---

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

---

was immer die Schreibweise (uint8_t) bewirken sollte. Hier hast Du offenbar uint8_t nur auf die Folgeausdrücke angewandt. Also:

Code:

---

uint8_t(color16 & 2016)

---

und das dann durch 8 dividiert. Du stampfst

Code:

---

(color16 & 2016)

---

auf 8 Bit ein (obwohl Du an der Stelle noch 16 Bit brauchst - eigentlich nur 11). Richtig wäre die Schreibweise und Funktion, die Du wolltest, so:

Code:

---

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

---

Allerdings habe ich woanders gelesen, dass man statt uint8_t() lieber byte() verwenden soll. Wäre der bessere Stil (kann man sehen wie man möchte):

"An unsigned char data type that occupies 1 byte of memory. It is the same as the byte datatype. The unsigned char datatype encodes numbers from 0 to 255. For consistency of Arduino programming style, the byte data type is to be preferred."
Arduino: Difference in “Byte” VS “uint8_t” VS “unsigned char”

nein, das stimmt doch nicht, was du schreibst:
ich habe deine gesammelten Codes in allen erdenklichen Klammerungen getestet, auch der obigen, und es hat nie funktioniert. Da du nie vollständigen Code gepostet hast, habe ich das tatsächlich hier und da ggf missverstanden, aber deine andere Klammerung von oben per
R = uint8_t((color16 / 2048 )*8 );
G = uint8_t((color16 & 2016)/8 );
B = uint8_t((color16 & 31)*8 );
hat ja auch nicht funktioniert.

Wenn du allerdings einen funktionierenden, kompilierfähigen Arduino-Sketch hast, selber von dir getestet, stelle ihn gerne ein, ich bin sehr gespannt.

byte ist i.Ü.auch nicht "besser", es ist einfach nur Arduino Kauderwelsch,
hingegen uin8_t usw. sind die stdint-Dateitypen u.a. auch für C++11 ,
und weil Arduino den gpp mit C++11 verwendet, funktioniert das selbstverständlich 100% standardgemäß.

Und das funktioniert dann auch ohne weiteres auf dem Raspi mit gcc/g++, wenn man will - im Gegensatz zu byte, das müsste man dann erst mal neu definieren als #define byte uint8_t.

übrigens heißt es nicht
uint8_t ((color16 / 2048 )*8 );
sondern
(uint8_t) ((color16 / 2048 )*8 );
denn für type casting schreibt man in C den Ziel-Datentyp in runde Klammern.
Aber auch DAS habe ich getestet, ebenfalls leider mit falschem Ergebnis, aber trotzdem herzlichen Dank für deine Ideen!

Und wie gesagt, ich freue mich auf einen funktionierenden, kompilierfähigen Arduino-Sketch von dir, vorher selber von dir getestet... 8)