Repository navigation
Add verification report for freedv-ka9q (re-target to main) - #86
Merged
Merged
Conversation
PR #77 was accidentally merged into the stale dr-radev2 branch instead of main; this carries the report over. Co-authored-by: Mooneer Salem <mooneer@gmail.com>
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
PR #77 (verification report for freedv-ka9q) was accidentally merged into
the stale
dr-radev2branch instead ofmain. This carries the samereport over so it lands where it should.
No code changes — just the two report files from #77.
@tmiw - this is pass, good work. However I encourage you to try an OTC test of freedv-gui or freedv-ka9q. It's a good experience and will be a useful skill to have, e.g. as we move forward with V2 and have to configure it to run with real radios. No rush, I appreciate you are busy with life.