Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhanced processtime #73

Closed
wants to merge 9 commits into from
Closed

Enhanced processtime #73

wants to merge 9 commits into from

Conversation

sidey79
Copy link
Contributor

@sidey79 sidey79 commented Nov 6, 2017

@Ralf9

Ich habe ein paar Sachen angepasst

calcHisto();
for (uint8_t idx = 0; idx<patternLen-1; idx++)

//calcHisto();
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

erst mal calchisto entfernt, da ich es nicht mehr vor dem compress brauche. Sollte sich nichts komprimieren lassen, wird erin calchisto noch am Ende aufgerufen

@@ -265,29 +276,38 @@ void SignalDetectorClass::compress_pattern()

if (inTol(pattern[idx2], pattern[idx], tol)) // Pattern are very equal, so we can combine them
{
idx_a[count] = idx;
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hier wird sich nur gemerkt, welche pattern zusammengefasst werden können

compiler Fehler behoben
@sidey79
Copy link
Contributor Author

sidey79 commented Nov 6, 2017

@Ralf9

Durch meine Anpassungen kommt es nun manchmal zu so einer Ausgabe:

�MS;P1=458;P3=-9156;D=131;CP=1;SP=3;R=8;m2;�
�MS;P1=458;P3=-9156;D=131;CP=1;SP=3;R=8;m1;�

Das war ja vorher bestimmt nicht :(

@Ralf9
Copy link
Contributor

Ralf9 commented Nov 6, 2017

die BAUDRATE 115200 ist zu hoch. Habe ich auch schon versucht, es aber nicht stabil hinbekommen.

mend Berechnung korrigiert
Suche nach validem sync in getSync() durch eine schnellere Variante ersetzt
@sidey79
Copy link
Contributor Author

sidey79 commented Nov 6, 2017

Bei einem 8 Mhz oder 16 Mhz Arduino. Der läuft eigentlich auch mit 1 Megabaud ganz gut.
Bei 2 Mbit/s ist dann Schluss.

Das Problem ist auch eher die Taktrate vom FTDI Chip, die muss gut passen, sonst gibt es Übertragungsfehler

@sidey79
Copy link
Contributor Author

sidey79 commented Nov 7, 2017

@Ralf9 Ich habe mir das mit der Baudrate noch mal genauer angesehen.

http://wormfood.net/avrbaudcalc.php?bitrate=57.6k%2C76.8k%2C115.2k%2C230.4k%2C250k%2C.5m%2C1m%2C2m&clock=8%2C16&databits=8

Wenn man das mal so nimmt wie es steht, kann man auf 1 Megabaud gehen und hat weniger Fehler.
Gut, 76.8K wäre auch ein Kompromiss, wenn man das UBRR Register setzt. 115200 oder aber auch die heute verwendeten 57600 scheinen nicht so gut geeignet zu sein

@Ralf9
Copy link
Contributor

Ralf9 commented Nov 7, 2017

Ja, 76.8K würde für 8 und 16 MHz passen, aber ob diese krumme Baudrate von den USB seriell Wandlern oder dem USR TCP232 unterstützt wird?
https://www.mikrocontroller.net/topic/214163

250 kBaud habe ich auch schon getestet, ich habe es aber nicht fehlerfrei hinbekommen,

@sidey79
Copy link
Contributor Author

sidey79 commented Nov 7, 2017

Ich lass mal meinen auf 500k laufen :) Die gängigen USB Chips können das.

Jaja ich weiss, ich hab auch schon mal behauptet das gibt nur Probleme... Aber ich weiss nicht wieso überhaupt.

@sidey79
Copy link
Contributor Author

sidey79 commented Nov 8, 2017

@Ralf9
Habs jetzt seit gestern 12 Uhr auf 500Kbaud laufen. Seit > 22 Stunden am Stück. Das sieht gut aus finde ich :)

@sidey79
Copy link
Contributor Author

sidey79 commented Nov 12, 2017

Ob man die Suchfunktion vielleicht in die Bitstore Klasse einbaut. Hmm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants