Traktorizer - Blog

MTE · September 12, 2006

Hi

also, nur fÃ¼r mein logisches VerstÃ¤ndnis...

Ich definiere die Variable

unsigned char MoniA;

ganz oben in main.c (MoniA = Taster Monitor/Deck A)

#define MoniA 0x9f

das schreib ich darunter und dieses definiert mir das der Taster NoteOn auf Midikanal 16 sendet...right ? ok kann ich dazu evtl gleich noch den Tasterpin und die NoteOn-Value dazudefinieren ? weil dann wÃ¤re es unten unter

void DIN_NotifyToggle .....

einfacher mit if und else rumzuspielen und endlich mal den Toggle Status zu verpassen....hogge mal wieder seit heut morgen dran...aber irgendwie...wie immer...no results :'(

Dont Panik about Authorenschaft ;D ...werde dich selbstverstÃ¤ndlich mit in die Firmware-Authorenliste aufnehmen....bei dem was du mir schon alles geholfen hast !!! :)

Greetz

audiocommander · September 12, 2006

na, mich irgendwo hin mit aufzunehmen ist nun wirklich nicht nÃ¶tig (...nur vielleicht die aufblasbare Midibox... ;D haha ,nein auch nicht nÃ¶tig :) )

also, ein #define ist streng genommen keine Variable, da sich ihr Wert nicht Ã¤ndert und sie niemals den Status des ausfÃ¼hrbaren Codes im PIC erreicht.

Wenn der Code kompiliert wird, tauscht der Compiler einfach jedes Vorkommen des definiertes Wortes gegen den definierten "Inhalt" aus. Suchen/Ersetzen-Prinzp; das ist alles.

also: fÃ¼r MONIA wird 0x9f eingesetzt.

Dabei darfst du natÃ¼rlich MONIA nicht als unsigned char variable deklarieren, denn das sind zwei verschiedene Dinge! Wenn du das #definierte MONIA schreibst, setzt der Compiler einfach 0x9f dahin. Du schreibst ja auch nicht myVar = unsigned char 0x9f !

NatÃ¼rlich kannst du soviele #defines definieren, bis die das Programm abstÃ¼rzt, es dient tatsÃ¤chlich nur der besseren Lesbarkeit und dass man schnell bestimmte Werte Ã¤ndern kann. Ist also auch gut fÃ¼r DIN_PIN_NUMMERN und ENCODER_NUMMERN, also alles, was sich mal so mit der Zeit Ã¤ndern kÃ¶nnte...

Es empfiehlt sich auch, sich der allgemeinen Schreibweise anzupassen, nach der defines grundsÃ¤tzlich komplett groÃŸ geschrieben werden, damit man sie leicht erkennen und von Variablen unterscheiden kann.

GrÃ¼ÃŸe,

Michael

Wild_Weasel · September 12, 2006

hi!

Wunderbar das sich hier einer mit microcontroller programmieren richtig auskennt.

Wie gesagt die LCD sachen sind zum debuggen. Allerdings plane ich bei der nÃ¤chsten Box 2 LCDs reinzutun

http://stud3.tuwien.ac.at/~e0327352/midibox/denon%20mb%202/ statt der VFDs die zwar sehr cool aussehen aber sinnlos sind weil die nur Ziffern gut darstellen kÃ¶nnen. Also ist das gut zu wissen.

Tja. WÃ¼rdest du den ganzen code sehen den ich zusammengewurschtelt hab wÃ¼rd dir schlecht werden.

So zB 200x "if" anstatt "case" usw.

Aber es hat funktioniert und das ohne wahrnehmbare Latenz. Also dÃ¼rfte der PIC doch nicht so langsam sein.

Hab nochmal Ã¼ber das scratchen nachgedacht. Ich halte ein grosses Rad und einen Mouse encoder (optisch) per Riemen angetrieben fÃ¼r sinnvoll. Weil:

Hohe AuflÃ¶sung des Mouse Encoders. Das sind gut und gerne 30 - 40 "Rasten"= Sichtfenster drauf
Noch hÃ¶here AuflÃ¶sung wegen der Ãœbersetzung des grossen Wheels auf den Encoder
Schlupffrei, weil der Gummiring nicht rutschen kann, Haftreibung

Wer baut einen Prototyp ???? ;D ;D ;D

grÃ¼sse

m.

MTE · September 12, 2006

was sollen die LCDs anzeigen ? Wenn Traktor Midiout senden wÃ¼rde wÃ¤re es gÃ¶ttlich, die Trackposition dort zu zeigen, aberso halte ich ein bzw.2 Lcds fÃ¼r Ã¼berflÃ¼ssig.

Das mit dem Lightsensor finde ich auch eine gute Idee, wobei es echt egal ist ob Encoder oder Lightsensor,haben beides ne super auflÃ¶sung

...das mit der RiemenÃ¼bersetzung mache ich jetz auch...

Wer baut einen Prototyp ?

Du ! ;D ;D

hast doch so nen Omnitronic zum ausschlachten :)

MTE · September 13, 2006

Hi

habe nunmal folgenden Code geschrieben :

unsigned char MoniA;    <- ganz oben ;)

......
...

void DIN_NotifyToggle(unsigned char pin, unsigned char pin_value) __wparam
{	
if( pin_value != 0 ) return; 
if(MoniA)
{ 
MoniA = 0;
MIOS_MIDI_BeginStream();
MIOS_MIDI_TxBufferPut(0x9f);
MIOS_MIDI_TxBufferPut(10);
MIOS_MIDI_TxBufferPut(0x00);
MIOS_DOUT_PinSet0(10);
MIOS_MIDI_EndStream();
}
else
{
MoniA = 1;
MIOS_MIDI_BeginStream();
MIOS_MIDI_TxBufferPut(0x9f);
MIOS_MIDI_TxBufferPut(10);   
MIOS_MIDI_TxBufferPut(0x7f);   
MIOS_DOUT_PinSet1(10);
MIOS_MIDI_EndStream();

MIOS_MIDI_BeginStream();
MIOS_LCD_Clear();
MIOS_LCD_PrintCString("Button ");
MIOS_LCD_PrintBCD4(pin);
MIOS_MIDI_EndStream();
}
}

aber egal welchen Taster ich drÃ¼cke, es kommt immer der gleiche Wert

timestamp 715654000 us: [9F 0A 7F] channel 16: note On A#-1 velocity: 127
timestamp 715907000 us: [9F 0A 00] channel 16: note On A#-1 velocity: 0
timestamp 717202000 us: [9F 0A 7F] channel 16: note On A#-1 velocity: 127
timestamp 717705000 us: [9F 0A 00] channel 16: note On A#-1 velocity: 0

Ich mÃ¶chte damit aber nur quasi den Monitortaster an pin 10 toggeln und die dazugehÃ¶rige Led zum leuchten bringen...die Led leuchtet mal *Fortschritt ;D ;D

Kann mir das mal jemand schnell korrigieren, dann komme ich endlich mal voran (wÃ¤r zu schÃ¶n um wahr zu sein :) )

GrÃ¼ÃŸe

MTE

audiocommander · September 13, 2006

hmm, verstehe ich nicht...

wenn du beidesmal [tt]MIOS_MIDI_TxBufferPut(10);[/tt] aufrufst, wird auch jedesmal 10 (A#-1) gesendet. Alles alles so, wie's da steht.

dann noch ein tipp: [tt] if(pin_value != 0) [/tt] kannst du auch einfach so ausdrÃ¼cken: [tt] if(pin_value) [/tt]; das wird dann bejaht, wenn pin_value grÃ¶ÃŸer als null ist, bzw. irgendeinen wert hat. I.d.F. wird es aber nur 1 sein.

grÃ¼ÃŸe,

ac

MTE · September 13, 2006

Die Zeile...

MIOS_MIDI_TxBufferPut(10);

sollte eigentlich fÃ¼r Pin 10 stehen....oh mann langsam blick ich gar nix mehr :(

audiocommander · September 13, 2006

den Pin (am DOUT) schaltest du ja mit [tt]MIOS_DOUT_PinSet0(10);[/tt]

MIOS_MIDI_TxBuffer ist natÃ¼rlich eine Midi-Funktion: am besten, du schaust mal die hervorragenden Midi-Artikel in unserem Wiki an; da ist das Midi-Protokoll super beschrieben (wenn du da nocht nicht so sicher bist):

kurz gesagt besteht eine Midi-Message aus zwei oder drei Bytes; wenn du Note_Ons sendest aus drei:

1. NOTE_ON auf CHANNEL x (i.d.F.: 0x9f => 0x90 + CH 0x0f)

2. NotenhÃ¶he (Taste) (i.d.F.: 10)

3. Velocity (NotenlautstÃ¤rke) (i.d.F.: 0x0 bzw. 0x7f)

GrÃ¼ÃŸe,

AC

MTE · September 13, 2006

ahhh ok so langsam wird es mir klar...da kann es ja net gehen

das mit der ersten zeile wusste ich....0x9f bedeutet 0x (keine Ahnung :) ) die 9 fÃ¼r noteon, 8 wÃ¤re noteoff) und f fÃ¼r den kanal 16, e wÃ¤re kanal 15 etc...

ich dachte halt immer bei der 2. zeile wird der pin definiert ::)

die valuedefinition in zeile 3 war mir auch noch klar

nungut,wie sage ich dann nun im endeffekt das das alles nur auf pin 10 zulÃ¤ssig sein soll ???

dicken Dank mal, wenn ich so weiter mache habe ich die Firmware in 2 Jahren fertig ;D

ne ernsthaft...das Galileobuch ziehe ich mir jeden tag rein,aber nach ner halben stunde raucht mir Ã¼belst die birne, bin fÃ¼r logische schaltungsfolgen irgendwie nicht gebaut, mathe war nie mein ding....aber da muss man halt jetz durch

:)

audiocommander · September 13, 2006

in Mathe war ich auch immer schlecht ;D

0x9f bedeutet 0x (keine Ahnung ) die 9 fÃ¼r noteon, 8 wÃ¤re noteoff) und f fÃ¼r den kanal 16, e wÃ¤re kanal 15 etc...

0x heiÃŸt einfach, dass es sich um eine Zahl im Hexadezimalen (16er) Zahlenraum handelt. D.h. es wird nicht wie im Dezimalen (10er) von 0 bis 9 gezÃ¤hlt und dann von 10 bis 19 usw..., sondern von 0 bis 15 (0,1,2,3,4,5,6,7,8,9,a,b,c,d,e,f). Wenn es geht, besorge dir mal einen Software-Taschenrechner, der das Umrechnen von Dezimal in Hexadezimal beherrscht, das ist bei Bitoperationen und Midi-Rechenzeugs immer von Vorteil!

nungut,wie sage ich dann nun im endeffekt das das alles nur auf pin 10 zulÃ¤ssig sein soll

Verstehe ich richtig, dass du das nur bearbeitet haben mÃ¶chtest, wenn DIN-Pin#10 gedrÃ¼ckt wird? Na, dann einfach:

[tt] if(pin == 10) [/tt]

oder:

[tt]switch(pin) {

case 10:

// do this and that

break;

case 999:

// wenn pin 999 gedrÃ¼ckt wird oder sonst ein pin, nur als beispiel...

break;

} [tt]

MTE · September 13, 2006

if(pin_value) return; // exit if button has not been pressed
if(pin == 10)
if(MoniA)
{ 
 MoniA = 0;
MIOS_MIDI_BeginStream();
MIOS_MIDI_TxBufferPut(0x9f);
MIOS_MIDI_TxBufferPut(10);
MIOS_MIDI_TxBufferPut(0x00);
MIOS_DOUT_PinSet0(10);
MIOS_MIDI_EndStream();
}
else
{
MoniA = 1;
MIOS_MIDI_BeginStream();
MIOS_MIDI_TxBufferPut(0x9f); 
MIOS_MIDI_TxBufferPut(10);  
MIOS_MIDI_TxBufferPut(0x7f);    
MIOS_DOUT_PinSet1(10);
MIOS_MIDI_EndStream();

MIOS_MIDI_BeginStream();
MIOS_LCD_Clear();
MIOS_LCD_PrintCString("Button ");
MIOS_LCD_PrintBCD4(pin);
MIOS_MIDI_EndStream();
}
}

FUNZT :D *umdenHalsfall

na endlich geht das mal, mit dem richtigen anstoss kann ich jetz endlich mal loslegen :)

DankÃ¤Ã¤Ã¤Ã¤Ã¤Ã¤Ã¤Ã¤Ã¤Ã¤Ã¤Ã¤Ã¤Ã¤Ã¤

*Fortsetzung folgt ;D

MTE · September 13, 2006

soderle mal ein paar Taster getoggelt und einige auf on/off gestellt ;)

wird halt ein riesen Code, aber ich denke das geht schon, hab mal die main.c upgeloadet

--> www.people.freenet.de/mte/Traktorizer/main.c

schaut sie euch mal an, evtl hat jemand VerbesserungsvorschlÃ¤ge um den Code ebbes zu schrumpfen

So,

nexter Schritt wÃ¤re wie hier unter Taster Wechsel-Modis fÃ¼r Cueplay.... in meiner Wiki-Sektion beschrieben

--> http://www.midibox.org/dokuwiki/doku.php?id=firmware-programmierung_in_c

jemand ne idee wie ich das sauber umsetzen kann ?

GrÃ¼ÃŸe

MTE

EDIT : haha goiiiieeel...grad noch ein feature verbaut...Jogwheel links gedreht...leuchtets grÃ¼n, rechts gedreht...leuchtets rot :D :D MIOS_DOUT_PinSet1(pin); rockt ;D

MTE · September 14, 2006

Was bedeutet diese Fehlermeldung : warning 126: unreachable code ?

kriege das beim compilieren.....die Zeile 1077 mÃ¼sste doch korrekt sein ---> siehe Attachments

GrÃ¼ÃŸe

MTE

audiocommander · September 14, 2006

unreachable code heiÃŸt, dass der code (in deinem Fall der else-Teil der Schleife) niemals erreicht werden kann.

Du hast einen ganz einfachen AnfÃ¤ngerfehler gemacht, der auch vielen anderen noch hÃ¤ufig passiert: es heiÃŸt nicht = (ist) sondern == (ist gleich); das eine ist ein sog. Zuweisungsoperator (x = 1); das andere ein logischer Vergleichsoperator ( if(x == 1) ).

;)

GrÃ¼ÃŸe,

Michael

ps: wenn Loopstep1A 1 ist, brauchst du danach eigentlich nicht Loopstep1A = 1 schreiben...

pps: noch ein Tipp: Variablen schreibt man gewÃ¶hnlich immer mit Kleinbuchstaben. Ich glaube in dem C-Buch gibt es auch ein paar coding-style Hinweise:

z.B.

- DEFINES

- var oder myVar

- MyFunction()

Das muss man natÃ¼rlich nicht machen, aber ein guter Style ist auch gut lesbar und beugt so vielen FlÃ¼chtigkeitsfehlern vor.

Du kannst deine eigenen Funktionen oder Variablen auch gerne mit einem PrÃ¤fix versehen, zb. MTE_Traktorizer_Play() oder mteCueStatus oder MTE_LCD_MESSAGE...

(das wÃ¤re auch was feines fÃ¼r deine Traktorizer-bezogenen Wiki-Files ;) )

MTE · September 15, 2006

alles klar...nun bin ich wieder ein groÃŸes StÃ¼ck weiter :)

stehe fast vor der LÃ¶sung...

if(pin == 56) if(Loopstep1A)
{
Loopstep1A = 1;
MIOS_MIDI_BeginStream();
MIOS_MIDI_TxBufferPut(0x9f);
MIOS_MIDI_TxBufferPut(pin);
MIOS_MIDI_TxBufferPut(0x7f);
MIOS_DOUT_PinSet1(pin);
MIOS_MIDI_EndStream();
}else{
Loopstep1A = 1;
MIOS_DOUT_PinSet1(56);	
Loopstep4A = 0;
MIOS_DOUT_PinSet0(57);
Loopstep8A = 0;
MIOS_DOUT_PinSet0(58);
Loopstep16A = 0;
MIOS_DOUT_PinSet0(59);
}

damit habe ich vor, im Traktor die Loopsteps zu steuern...auf deutsch....4 Taster und 4 leds...wenn 1 Taster gedrÃ¼ckt bleibt er solange an,bis er von einen der anderen 4 abgelÃ¶st wird.

Nun habe ich aber im obigen Code noch ein klitzekleines Problem zu lÃ¶sen :

wenn ich zuerst einen Taster drÃ¼cke geht zuerst die LED an und er sendet keinen Midievent....nochmal gedrÃ¼ckt folgt dann endlich der Event...gleiches ist dann wenn ich einen der anderen Taster betÃ¤tige...erst leuchtet die LEd, beim 2.drÃ¼cken dann der Event.........wo ist der Haken ?

P.S.: der obige code ist nur fÃ¼r Taster Step 1...man nehme quasi diesen code 4 mal untereinander mit den anderen Step-Variablen um es auf alle Taster zu bestimmen ;)

GrÃ¼ÃŸe

MTE

audiocommander · September 15, 2006

wenn ich zuerst einen Taster drÃ¼cke geht zuerst die LED an und er sendet keinen Midievent....nochmal gedrÃ¼ckt folgt dann endlich der Event...gleiches ist dann wenn ich einen der anderen Taster betÃ¤tige...erst leuchtet die LEd, beim 2.drÃ¼cken dann der Event.........wo ist der Haken ?

na, ich schÃ¤tze mal, dein Loopstep1A ist 0. Dann kommt die else-Schleife, der Loopstep1A wird auf 1 gesetzt, die LEDs werden geschalten und die Funktion beendet. Beim nÃ¤chsten mal ist Loopstep1A 1 und der if-Teil der Schleife wird bearbeitet. Also im Prinzip macht es das, was da steht ;D

GrÃ¼ÃŸe,

Michael

Ãœbrigens: wenn du mehrere ON/OFF switches speichern mÃ¶chtest, dann solltest du nicht mehrere unsigned chars dafÃ¼r nehmen, sondern ein sog. "Bitfeld". Damit kannst du in einer Struktur (struct) bis zu 8 AN/AUS Stadien speichern.

Der Grund dafÃ¼r ist, dass 1 unsigned char aus 8 Bytes besteht; bei sagen wir mal 8 Loopsteps wÃ¤ren das 64 Bytes. Als Bitfeld kann man das auf 8 Bytes erledigen.

Aber Achtung! SDCC-Stolperfalle: SDCC unterstÃ¼tzt maximal 8 Byte lange Bitfelder; d.h. wenn du mehr als 8 bits brauchst, dann musst du mehrere Bitfelder erstellen;

Wenn du massig Platz hast und nicht viele Features geplant sind, musst du das nicht unbedingt machen, es wÃ¤re aber die empfohlene Methode zum Speichern kleinbittiger Variablen.

Das Beispiel dafÃ¼r habe ich kÃ¼rzlich schon mal in just diesem Thread gepostet: http://www.midibox.org/forum/index.php?topic=6871.msg50985#msg50985

MTE · September 15, 2006

Suuuuupi :)

habe mittlerweile den Code um eiiiiiiiiniges schrumpfen kÃ¶nnen (hab die ODER " || " Funktion entdeckt :) )

aber dadurch entstanden kleine andere Bugs.

Mittlerweile lÃ¤sst sich aber alles sehr einfach lesen (und vorallem kapieren :D ).

Die Firmware wird eigentlich sehr winzig, sind ja nur Jogwheel, Taster und LED Funktionen....spÃ¤ter dann noch das Scratchen, das wird denke ich auch net viel Platz verschwenden....ansonsten pack ich das alles halt in einen 4620er...hab glaube ich die neue Version als Sample bekommen...muss das noch mit dem "Revision_ID"-progrÃ¤mmchen abchecken.

Auf jeden Fall flutscht es seit 2 Tagen und macht riesig SpaÃŸ es immer an der Box gleich auszuprobieren :)

Auf bald

MTE

MTE · September 15, 2006

if(pin == 56)
{
MIOS_MIDI_BeginStream();
MIOS_MIDI_TxBufferPut(0x9f);
MIOS_MIDI_TxBufferPut(pin);
MIOS_MIDI_TxBufferPut(0x7f);
MIOS_DOUT_PinSet1(pin);
MIOS_DOUT_PinSet0(57);
MIOS_DOUT_PinSet0(58);
MIOS_DOUT_PinSet0(59);
MIOS_MIDI_EndStream();
MIOS_MIDI_BeginStream();
MIOS_LCD_Clear();
MIOS_LCD_PrintCString("Button :");
MIOS_LCD_PrintBCD4(pin);
MIOS_MIDI_EndStream();
}

perfekt....funzt...tralala....luftsprung ;D

audiocommander · September 15, 2006

MIOS_MIDI_Begin und EndStream(); sind nur notwendig, wenn dazwischen auch MIOS_MIDI_TxBufferPut() kommt

;)

MTE · September 15, 2006

Codeschrumpfung immÃ¤ gut ;D :D

stryd_one · September 16, 2006

OK after 8 pages of posts I have to ask ..... What are you crazy Germans up to here? ;)

I hope there might be a very small explanation in english one day, for those of us not fortunate enough to sprechen Sie Deutsches. That last part is brought to you by babelfish ;D

audiocommander · September 16, 2006

What are you crazy Germans up to here?

of course we're planing to take over the world!

and we begin with taking over the number #2 of most-replied-threads in this forum!

uups... now I revealed our secret plans :-X ;D

well, bad jokes aside: I'm just giving some coding tipps from time to time, but MTE and Wild_Weasel are up to build a Traktor-Box /-Controller with Scratching Features and DJ-stuff (if I got that right, please correct me if I'm wrong, after 8 pages one tends to loose the oversight... ;) )

They both have set up Wiki-Pages; there's some info in English, too:

http://www.midibox.org/dokuwiki/doku.php?id=midibox_traktor

so, no world-domination until now (puhh, what a luck!) ;D

cheers!

Michael

MTE · September 16, 2006

Looooo-fuggin-oooolll ;D

no the first place for world domination is only reserved for TK ! ;) :D

Ive looked for an english & C-Code-Babelfish too, but no chance...searched all over the web >:(

but found the best substitute, called Audiocommander :P ;D 8) ... a great dude which make impressive works !!

The Traktorizer is a big challenge and of course the next step...the scratch-function, i have no plan how we will realize this.....but im on it :)

So now 2morrow (oops i forgot the time, i mean later ;D ) i will update the english section at the wiki and hope, some other ppl can give us programming-noobs some good impressions for our code :)

wet dreams :)

MTE

EDIT : Wiki pages now little updated ;D

goodnight

stryd_one · September 16, 2006

LOL Thanks guys :) I'll go back to lurking around this thread in babelfish now :)

MTE · September 19, 2006

@ Wild Weasel

ich muss sagen, deine PitchbendlÃ¶sung (im Code) ist eigentlich echt genial definiert.

SaÃŸ gerade an meiner Windencoder-Funktion und duplizierte fÃ¼r diesen deine Zeilen vom pitchbend

in Traktor 3 habe ich dann die up-down-definition auf "switch" gestellt......es klingt FAST wie gescratcht....bin dem Ziel sehr nahe...werde nun etwas mit dem Mios_Delay rumspielen und schauen, wie sich das als scratching weiterverhÃ¤lt.

Falls du (und natÃ¼rlich auch alle anderen, die das hier lesen) in den Code reinschauen wilst...der liegt hier

--> http://people.freenet.de/mte/Traktorizer/Traktorizer_V1.3.zip ;)

Der Code ist fast fertig, werde dafÃ¼r dann auch noch eine Anleitung schreiben, inwieweit man wo und wie die DIN-Pins umschreiben muss, damit jede Funktion am richtigen DIN-Pin liegt (lÃ¶tet ja nicht jeder nach meiner schematic)

Dazu liegt an jedem Button (und sogar am Pitchbend-und Windencoder) gleich die dazugehÃ¶rige LED + Status

geht ganz easy mit "MIOS_DOUT_PinSet(pin, value);" ;)

Enjoy :)

MTE

Sign In

Traktorizer - Blog

Recommended Posts

Link to comment

Share on other sites

Top Posters In This Topic

Popular Days

Top Posters In This Topic

Popular Days

Posted Images

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Join the conversation