c't-Lab

dg1vs hat geschrieben:ich habe mich beim DDS Funktionsgenerator immer etwas geärgert, dass man mit dem kleinen Display immer „ewig“ scrollen muss. Deshalb habe ich vor einiger Zeit mal einen Versuch gestartet, ein größeres „PM8“ zu entwerfen.

dg1vs hat geschrieben:Und nun kommt mein „Angstgegener“ . Da ich vor über 15 Jahren beschlossen das Thema MMI konsequent zu meiden, stehe ich vor dem Problem, was packt man nun in die 80 Zeichen.... Die unterste Zeile sollte die Icons für Hoch/Runter (Rechts/Links außen) und die Bedeutungen der zwei „neuen“ Softkeys enthalten. Vielleicht ist es auch sinnvoll die Bedeutung von allen 4 Tastern zu ändern. Hier wäre ich auf Eure Ideen angewiesen. Meine Bitte: wenn jemand Ideen zur Gestaltung hat, mir bitte mitteilen.

dg1vs hat geschrieben:Die Software habe ich angepasst, aber sicher was vergessen.

psclab38 hat geschrieben: Das Scrollen war für mich eigentlich nie so das große Problem, die etwas eingeschränkte Übersichtlichkeit schon eher. Meistens war das Display mit 8 Zeichen einfach nur zu kurz. Andererseits wollte ich keine Einschubslots im Gehäuse mit breiten Frontplatten versperren.

psclab38 hat geschrieben: Ich hab' jetzt eine Weile überlegen müssen, was ich Dir antworten soll. Einerseits bist Du ja schon mit der Hardware recht weit, aber ich muß zugeben, daß ich bei dem bereits geleisteten Aufwand schon eine grundsätzliche Idee zur MMI-Gestaltung auch der oberen drei Zeilen erwartet hätte. Manchmal muß man seinen Angstgegner einfach angreifen, um weiterzukommen...

psclab38 hat geschrieben: Zeigt es denn schon das "Standardmenü" an? Was den Code angeht, die Menüführung ist im C-Code ja recht gut getrennt vom Rest. Abhängig vom endgültigen MMI könnte man quasi eine eigene Datei für Deine Variante aufmachen.

dg1vs hat geschrieben:Ich habe es aber nicht geschaftt in folgende Zeile reinzulaufen (mit Debugger)

Code: Alles auswählen

LCDClearLine(1);

	{
		if(null!=(void*)pgm_read_word(&g_currentMenuItem->pSubmenu))
			LCDOverwrite_P(7, 1, 1, PSTR("\x05"));
		else
			LCDClearLine(1);
	}

psclab38 hat geschrieben:Hi Karsten,
Welchen Debugger verwendest Du? Den JTAGICEmk2? Im AVR Studio kann man nicht an jeder Anweisung einen Brechpunkt setzen, wenn der compilierte Code zu gut
optimiert ist.

	{
		if(null!=(void*)pgm_read_word(&g_currentMenuItem->pSubmenu))
			LCDOverwrite_P(7, 1, 1, PSTR("\x05"));
		else
			LCDClearLine(1);
	}

psclab38 hat geschrieben:denn das Dreieck in Sonderzeichen 0x05 erscheint brav in Abhängigkeit davon, ob ein Untermenü vorhanden ist oder nicht. Inwiefern die mit "LCDClearLine(1);" gelöschte Zeile nicht gleich wieder mit Inhalten überschrieben wird, hab ich mir jetzt nicht direkt angeschaut. Mein Debugger ist im Moment abgeklemmt.

dg1vs hat geschrieben: Ich werde heute Abend mit den den Optimierungen spielen (solange mich Frau und Kinder lassen ). Vielleicht habe ich auch was übersehen.

LCDClearLine(1);

dg1vs hat geschrieben: Naja das Spielen hat etwas länger gedauert. Ich habe alle möglichen und unmöglichen Compileroptimierungsoptionen ausprobiert, aber ich laufe nicht in die Zeile:

      if(null!=UpdateDisplayFunction)
            UpdateDisplayFunction(1,
                    (void*)pgm_read_word(&g_currentMenuItem->pParam));

        // if there's no EncoderFunction, display triangle to indicate if there
        // is a submenu or output a blank line if not
        else
        {
            if(null!=(void*)pgm_read_word(&g_currentMenuItem->pSubmenu))
                LCDOverwrite_P(7, 1, 1, PSTR("\x05"));
            [b]else[/b]
                LCDClearLine(1);
        }

dg1vs hat geschrieben:anbei mal der Start-Screen des Panels.

dg1vs hat geschrieben:Der "Rest" ist auf dem Stand des kleinen Display, zusätzlich gibt es eine Statusleiste, die die aktuelle Signalform, Sweep an/aus und Burst an/aus darstellen. Eine Taste habe ich mit dem Rücksetzen auf Defaultwerte belegt. Geht so ganz schnell .

dg1vs hat geschrieben:Beim Lesen der TODO ist mir noch aufgefallen, dass PWM, AM und FM geplant gewesen war/ist. Wie ist da der Stand?

dg1vs hat geschrieben:PS: Dank an Paul für die Motivation mit dem "Angstgegner"

psclab38 hat geschrieben: [...] PWM hat Thoralt mal angefangen und versprochen, es auch fertig zu machen. @Thoralt: was fehlt da eigentlich noch?

• Eingabe der PWM-Frequenz und des PWM-Verhältnisses
• Umrechnung der Frequenz in Vorteiler/Teiler des Timers im AVR, hier immer Auswahl des optimalen Vorteilers beachten, damit die Auflösung der PWM möglichst groß bleibt
• Zurückrechnen in tatsächliche Frequenz (Rundung wegen der ganzzahligen Teilerverhältnisse im Timer), Anzeige der tatsächlichen Frequenz; Alternative: Einrasten der Frequenzeingabe auf "krummen" Werten, wie sie durch die Timer-Teilung entstehen
• Anlöten des kleinen Drahtes vom Timerausgang an die Verstärkerstufe, damit die ganze Zauberei auch hinten rauskommt

psclab38 hat geschrieben: Hast Du da auch noch ein Bildchen davon? So wie Du es beschreibst, wäre das ein guter Kompromiss zwischen Aufwand und besserer Darstellung!

dg1vs hat geschrieben:Ist zwar ein mächtiges #define-Monster geworden. Scheint aber gut zu funktionieren.

dg1vs hat geschrieben:Die Darstellung ist nicht exakt die, die ich möchte (und schon umgesetzt hatte), aber so beschränken sich die Änderungen zunächst auf kleine Dinge.

dg1vs hat geschrieben:Bei einem kleinen Display funktioniert alles wie bekannt.

dg1vs hat geschrieben:Ich würde gerne die Sachen einchecken, den Zugang hat mir Hartmut eingerichtet. Auch würde ich mich "besser" fühlen, wenn jemand die Änderungen reviewen würde, bevor ich sie einchecke. Im Anhang also alle geänderten File zum SF-Stand.

dg1vs hat geschrieben:Eine Sache habe ich noch offen, Macros für "Char" in sprintf oder ähnlichen Funktionen. Hat jemand einen guten Link zu dem Thema. Es geht um die "Sonderzeichen" die sich in Abhängigkeit vom benutzten Display unterscheiden. Und da sind wir auch bei den 2 Symbolen, wo eine Macroersetzung extrem hilfreich wäre. Auf die Schnelle ist mir nichts eingefallen.

#ifdef COMPILE_WITH_DISPLAY204
#else
#endif

switch(Params.ucSweepSlope)
{
	case SWEEP_UP:
		LCDOverwrite_P(2, DISPLAY_LINE_1, 6, PSTR("Up   \x05"));
		break;
	case SWEEP_DOWN:
		LCDOverwrite_P(2, DISPLAY_LINE_1, 6, PSTR("Down \x05"));
		break;
	case SWEEP_UPDOWN:
		LCDOverwrite_P(0, DISPLAY_LINE_1, 8, PSTR("Up+Down\x05"));
		break;
}