Add files via upload #301
Open
Add files via upload #301
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Extension of the encoder to half and quarter steps for encoders that have 2 or 4 detent positions per pulse period. The configuration takes place when the class instance is created using the constructor. The new parameters are optional, so existing code can continue to be used unchanged. A special feature emerged with double or quadruple evaluation when using hardware debouncing of the encoder signals via RC element: If the interrupt release occurs during initialization immediately after the port mode has been defined, the capacitor is still empty and is charging. This leads to an unwanted detection of an alleged rotary movement. Therefore the interrupt release must be delayed. For the sake of simplicity, I used the delay() method here, since this is called only once during boot and only during setup.
|
Hast du bei der |
|
Na irgendwas mußte ja schiefgehen. Ich habe die files mit Merge und UltraEdit bearbeitet. Ich kann mir schon vorstellen, dass das in UltraEdit passiert sein kann. Blöd. Merge zeigt aber nur wirklich die geänderten Zeilen als Differenz an ...
Was mach ich jetzt am besten? Original hernehmen und die Änderungen nochmal hineinmergen in der Hoffung, dass das dann darin korrekt gemacht wird?
Aber wenn diese Zeilen aus meinem file stammen, das wahrscheinlich schon verändert ist, wird’s nichts bringen.
Hätte es wohl besser in Visual Studio machen soll (dessen Lizenz abgelaufen ist).
Matthias
From: Jérôme
Sent: Wednesday, July 20, 2022 12:03 PM
To: pa-pa/AskSinPP
Cc: ChMatsch ; Author
Subject: Re: [pa-pa/AskSinPP] Add files via upload (PR #301)
Hast du bei der Button.h die Line-Endings verändert? Im Diff wird das komplette File als Unterschied erkannt
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
|
Wenn du Notepad++ hast, kannst du dort im Menü Bearbeiten --> Format Zeilenende --> UNIX (LF) wählen, und dann als weitere Version hochladen. So sieht man den tatsächlichen Unterschied: |
Cool, den Splitview und Hide Whitespaces im Github kannte ich noch nicht |
|
Ich sehe schon, die Originaldateien sind mit LF (0x0A) abgeschlossen, meine mit CR/LF (0x0D 0x0A).
Was muß ich jetzt mit den korrigierten Dateien tun? Einfach in meinem Fork austauschen? Muß ich dann einen neuen Pullrequest machen?
Matthias
From: Marco
Sent: Wednesday, July 20, 2022 1:10 PM
To: pa-pa/AskSinPP
Cc: ChMatsch ; Author
Subject: Re: [pa-pa/AskSinPP] Add files via upload (PR #301)
Wenn du Notepad++ hast, kannst du dort im Menü Bearbeiten --> Format Zeilenende --> UNIX (LF) wählen, und dann als weitere Version hochladen.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
|
Einfach die Datei in deinem fork mit nem neuen commit reparieren. |
Line ending fixed
|
Und jetzt noch das andere file 😉 |
|
Ich bin aktuell am rätseln. Ich habe beide Files ausgetauscht, doch Remote.h zeigt noch immer das alte Problem.
Ich habe daraufhin diese Datei nochmal aus Github heruntergeladen und untersucht. Und da ist komischerweise alles korrekt und alle Zeilenenden nur auf LF.
Ich versteh’s jetzt nicht, wo die Differenz herkommt.
Weiß nicht, ob ich mit einem Binärvergleich was rausfinde.
From: Christoph Wiechert
Sent: Wednesday, July 20, 2022 3:22 PM
To: pa-pa/AskSinPP
Cc: ChMatsch ; Author
Subject: Re: [pa-pa/AskSinPP] Add files via upload (PR #301)
Und jetzt noch das andere file 😉
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
file format fixing
|
Sehe gerade im Binärvergleich, dass es bei Remote.h genau andersrum ist wie bei Button.h! Dort sind original CR/LF drin und in der neuen Version nun nur noch LF! Krrr. |
|
Jetzt aber! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Extension of the encoder to half and quarter steps for encoders that have 2 or 4 detent positions per pulse period.
The configuration takes place when the class instance is created using the constructor. The new parameters are optional, so existing code can continue to be used unchanged.
A special feature emerged with double or quadruple evaluation when using hardware debouncing of the encoder signals via RC element: If the interrupt release occurs during initialization immediately after the port mode has been defined, the capacitor is still empty and is charging. This leads to an unwanted detection of an alleged rotary movement. Therefore the interrupt release must be delayed. For the sake of simplicity, I used the delay() method here, since this is called only once during boot and only during setup.