File updates - rc_link #523
Conversation
|
Warning Rate limit exceeded@nerdCopter has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 3 minutes and 47 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (6)
WalkthroughAdds 30+ new RC_LINK preset files and a shared defaults file under Changes
Sequence Diagram(s)(omitted — changes are configuration-only, no control-flow modifications) Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes
Possibly related PRs
Suggested reviewers
Poem
Pre-merge checks and finishing touches❌ Failed checks (2 inconclusive)
✅ Passed checks (1 passed)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
🧹 Nitpick comments (23)
presets/2025.12/rc_link/generic/150hz_freestyle.txt (1)
8-8: Minor: Redundant phrasing in warning text.The phrase "ABSOLUTELY SURE" is redundant; consider simplifying to "SURE" for conciseness. This applies to the warning text across all preset files in this PR.
presets/2025.12/rc_link/generic/50hz_hd_freestyle.txt (1)
8-8: Minor: Redundant phrasing in warning text.Line 8 contains redundant phrasing. Consider simplifying "ABSOLUTELY SURE" to "SURE" for more concise language.
presets/2025.12/rc_link/generic/500hz_cinematic.txt (1)
8-8: Minor: Redundant phrasing in warning text.Line 8 contains redundant phrasing. Consider simplifying "ABSOLUTELY SURE" to "SURE" for consistency and clarity.
presets/2025.12/rc_link/ghost/race500.txt (1)
56-56: Minor: Consider hyphenating "Ultra Smooth Cinematic".Per style guidance, "Ultra Smooth Cinematic" may read better as "Ultra-Smooth Cinematic" in the OPTION BEGIN label.
presets/2025.12/rc_link/generic/150hz_race.txt (1)
8-8: Minor: Redundant phrasing in warning text.Line 8 contains redundant phrasing. Consider simplifying "ABSOLUTELY SURE" to "SURE" for consistency and clarity.
presets/2025.12/rc_link/generic/50hz_cinematic.txt (1)
8-8: Minor: Redundant phrasing in warning text.Line 8 contains redundant phrasing. Consider simplifying "ABSOLUTELY SURE" to "SURE" for consistency and clarity.
presets/2025.12/rc_link/generic/250hz_cinematic.txt (1)
8-8: Minor: Redundant phrasing in warning text.Line 8 contains redundant phrasing. Consider simplifying "ABSOLUTELY SURE" to "SURE" for consistency and clarity.
presets/2025.12/rc_link/generic/150hz_ultra_cinematic.txt (2)
1-1: Minor: Consider hyphenating "Ultra Cinematic".Per style guidance, "Ultra Cinematic" appears in the TITLE (line 1) and DESCRIPTION (line 7) and may read better as "Ultra-Cinematic" for consistency with hyphenation conventions.
Also applies to: 7-7
8-8: Minor: Redundant phrasing in warning text.Line 8 contains redundant phrasing. Consider simplifying "ABSOLUTELY SURE" to "SURE" for consistency and clarity.
presets/2025.12/rc_link/generic/500hz_freestyle.txt (1)
8-8: Optional: Minor text clarity improvement.The phrase "make ABSOLUTELY SURE" is redundant; consider "make SURE" for conciseness in user-facing text.
presets/2025.12/rc_link/tbs/crossfire_150hz.txt (1)
16-16: Optional: Text clarity improvement in warnings."make ABSOLUTELY SURE" is redundant; consider "make SURE" for conciseness.
presets/2025.12/rc_link/generic/500hz_race.txt (1)
8-8: Optional: Text clarity improvement."make ABSOLUTELY SURE" is redundant; consider "make SURE".
presets/2025.12/rc_link/generic/500hz_hd_freestyle.txt (1)
8-8: Optional: Text clarity improvement."make ABSOLUTELY SURE" is redundant; consider "make SURE".
presets/2025.12/rc_link/generic/150hz_hd_freestyle.txt (1)
8-8: Optional: Text clarity improvement."make ABSOLUTELY SURE" is redundant; consider "make SURE".
presets/2025.12/rc_link/generic/1000hz_race.txt (1)
8-8: Optional: Text clarity improvement."make ABSOLUTELY SURE" is redundant; consider "make SURE".
presets/2025.12/rc_link/generic/500hz_ultra_cinematic.txt (2)
1-1: Optional: Text styling improvement for consistency.Consider using hyphens: "Ultra-Cinematic" and "ultra-smooth cinematic" to match standard hyphenation for compound adjectives in user-facing text.
Also applies to: 7-7
8-8: Optional: Text clarity improvement."make ABSOLUTELY SURE" is redundant; consider "make SURE".
presets/2025.12/rc_link/generic/250hz_hd_freestyle.txt (1)
8-8: Optional: Text clarity improvement."make ABSOLUTELY SURE" is redundant; consider "make SURE".
presets/2025.12/rc_link/DJI_Normal.txt (1)
68-68: Remove trailing whitespace.Line 68 has trailing spaces after the value:
set rc_smoothing_auto_factor = 250. While not breaking functionality, remove the trailing whitespace for consistency with the rest of the preset.-set rc_smoothing_auto_factor = 250 +set rc_smoothing_auto_factor = 250presets/2025.12/rc_link/defaults.txt (1)
25-25: Add newline at end of file.For consistency with typical text file conventions, add a trailing newline after the last configuration statement.
presets/2025.12/rc_link/frsky_sbus_fport.txt (1)
71-72: Remove trailing whitespace on these lines.Lines 71-72 contain trailing spaces after numeric values that should be removed for consistency.
presets/2025.12/rc_link/elrs_50hz.txt (2)
8-8: Minor: Simplify redundant language.Line 8 uses "ABSOLUTELY SURE"—the word "ABSOLUTELY" is redundant. Consider changing to just "SURE" for conciseness.
-#$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC Filter is un-checked! +#$ DESCRIPTION: WARNING: make SURE that the OpenTx or EdgeTx Hardware ADC Filter is un-checked!
52-53: Minor: Consider hyphenating compound adjectives.Lines 52-53 use "ultra smooth Cinematic"—for consistency with English style conventions, consider "ultra-smooth" when used as an attributive phrase before a noun.
-# ultra smooth Cinematic (not for racing): +# ultra-smooth Cinematic (not for racing):and
-#$ OPTION BEGIN (UNCHECKED): Ultra Cinematic +#$ OPTION BEGIN (UNCHECKED): Ultra-Cinematic
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (41)
presets/2025.12/rc_link/DJI_Normal.txt(1 hunks)presets/2025.12/rc_link/DJI_SBUS_Fast.txt(1 hunks)presets/2025.12/rc_link/defaults.txt(1 hunks)presets/2025.12/rc_link/elrs_150hz.txt(1 hunks)presets/2025.12/rc_link/elrs_250hz.txt(1 hunks)presets/2025.12/rc_link/elrs_500hz.txt(1 hunks)presets/2025.12/rc_link/elrs_50hz.txt(1 hunks)presets/2025.12/rc_link/frsky_sbus_fport.txt(1 hunks)presets/2025.12/rc_link/generic/1000hz_race.txt(1 hunks)presets/2025.12/rc_link/generic/150hz_cinematic.txt(1 hunks)presets/2025.12/rc_link/generic/150hz_freestyle.txt(1 hunks)presets/2025.12/rc_link/generic/150hz_hd_freestyle.txt(1 hunks)presets/2025.12/rc_link/generic/150hz_race.txt(1 hunks)presets/2025.12/rc_link/generic/150hz_ultra_cinematic.txt(1 hunks)presets/2025.12/rc_link/generic/250hz_cinematic.txt(1 hunks)presets/2025.12/rc_link/generic/250hz_freestyle.txt(1 hunks)presets/2025.12/rc_link/generic/250hz_hd_freestyle.txt(1 hunks)presets/2025.12/rc_link/generic/250hz_race.txt(1 hunks)presets/2025.12/rc_link/generic/250hz_ultra_cinematic.txt(1 hunks)presets/2025.12/rc_link/generic/500hz_cinematic.txt(1 hunks)presets/2025.12/rc_link/generic/500hz_freestyle.txt(1 hunks)presets/2025.12/rc_link/generic/500hz_hd_freestyle.txt(1 hunks)presets/2025.12/rc_link/generic/500hz_race.txt(1 hunks)presets/2025.12/rc_link/generic/500hz_ultra_cinematic.txt(1 hunks)presets/2025.12/rc_link/generic/50hz_cinematic.txt(1 hunks)presets/2025.12/rc_link/generic/50hz_freestyle.txt(1 hunks)presets/2025.12/rc_link/generic/50hz_hd_freestyle.txt(1 hunks)presets/2025.12/rc_link/generic/50hz_race.txt(1 hunks)presets/2025.12/rc_link/generic/50hz_ultra_cinematic.txt(1 hunks)presets/2025.12/rc_link/ghost/long_range.txt(1 hunks)presets/2025.12/rc_link/ghost/normal.txt(1 hunks)presets/2025.12/rc_link/ghost/pure_race.txt(1 hunks)presets/2025.12/rc_link/ghost/race.txt(1 hunks)presets/2025.12/rc_link/ghost/race250.txt(1 hunks)presets/2025.12/rc_link/ghost/race500.txt(1 hunks)presets/2025.12/rc_link/ghost/solid250.txt(1 hunks)presets/2025.12/rc_link/redpine_666hz.txt(1 hunks)presets/2025.12/rc_link/redpine_mpm.txt(1 hunks)presets/2025.12/rc_link/tbs/crossfire_150hz.txt(1 hunks)presets/2025.12/rc_link/tbs/crossfire_50hz.txt(1 hunks)presets/2025.12/rc_link/tbs/tracer_250hz.txt(1 hunks)
🧰 Additional context used
🧠 Learnings (4)
📓 Common learnings
Learnt from: Jaz7Mutant
Repo: betaflight/firmware-presets PR: 512
File: presets/4.5/tune/basher_jazzmutant_tune.txt:118-119
Timestamp: 2025-06-20T20:39:08.723Z
Learning: In Betaflight firmware presets, rates defaults from presets/4.3/rates/defaults.txt are used across multiple firmware versions including 4.5, as the file explicitly declares support for firmware versions 4.3, 4.4, and 4.5. There is no separate defaults.txt file in presets/4.5/rates/ directory.
Learnt from: bensonk
Repo: betaflight/firmware-presets PR: 509
File: presets/4.5/osd/bensonk.txt:138-152
Timestamp: 2025-05-25T22:26:17.328Z
Learning: BrainFPV commands in Betaflight preset files are safely ignored on non-BrainFPV flight controllers, making it safe to include them in presets even when they might be applied to different hardware.
📚 Learning: 2025-06-20T20:39:08.723Z
Learnt from: Jaz7Mutant
Repo: betaflight/firmware-presets PR: 512
File: presets/4.5/tune/basher_jazzmutant_tune.txt:118-119
Timestamp: 2025-06-20T20:39:08.723Z
Learning: In Betaflight firmware presets, rates defaults from presets/4.3/rates/defaults.txt are used across multiple firmware versions including 4.5, as the file explicitly declares support for firmware versions 4.3, 4.4, and 4.5. There is no separate defaults.txt file in presets/4.5/rates/ directory.
Applied to files:
presets/2025.12/rc_link/generic/150hz_freestyle.txtpresets/2025.12/rc_link/ghost/normal.txtpresets/2025.12/rc_link/generic/50hz_race.txtpresets/2025.12/rc_link/generic/1000hz_race.txtpresets/2025.12/rc_link/generic/250hz_cinematic.txtpresets/2025.12/rc_link/generic/500hz_race.txtpresets/2025.12/rc_link/DJI_Normal.txtpresets/2025.12/rc_link/tbs/tracer_250hz.txtpresets/2025.12/rc_link/generic/250hz_freestyle.txtpresets/2025.12/rc_link/generic/50hz_cinematic.txtpresets/2025.12/rc_link/generic/50hz_hd_freestyle.txtpresets/2025.12/rc_link/redpine_666hz.txtpresets/2025.12/rc_link/tbs/crossfire_150hz.txtpresets/2025.12/rc_link/generic/500hz_cinematic.txtpresets/2025.12/rc_link/elrs_50hz.txtpresets/2025.12/rc_link/ghost/pure_race.txtpresets/2025.12/rc_link/generic/500hz_ultra_cinematic.txtpresets/2025.12/rc_link/DJI_SBUS_Fast.txtpresets/2025.12/rc_link/generic/50hz_ultra_cinematic.txtpresets/2025.12/rc_link/generic/250hz_hd_freestyle.txtpresets/2025.12/rc_link/generic/500hz_freestyle.txtpresets/2025.12/rc_link/generic/250hz_race.txtpresets/2025.12/rc_link/generic/150hz_ultra_cinematic.txtpresets/2025.12/rc_link/tbs/crossfire_50hz.txtpresets/2025.12/rc_link/generic/150hz_race.txtpresets/2025.12/rc_link/ghost/race.txtpresets/2025.12/rc_link/elrs_150hz.txtpresets/2025.12/rc_link/elrs_250hz.txtpresets/2025.12/rc_link/ghost/long_range.txtpresets/2025.12/rc_link/generic/150hz_cinematic.txtpresets/2025.12/rc_link/ghost/race250.txtpresets/2025.12/rc_link/generic/150hz_hd_freestyle.txtpresets/2025.12/rc_link/elrs_500hz.txtpresets/2025.12/rc_link/generic/500hz_hd_freestyle.txtpresets/2025.12/rc_link/ghost/solid250.txtpresets/2025.12/rc_link/ghost/race500.txtpresets/2025.12/rc_link/frsky_sbus_fport.txtpresets/2025.12/rc_link/generic/250hz_ultra_cinematic.txtpresets/2025.12/rc_link/defaults.txtpresets/2025.12/rc_link/generic/50hz_freestyle.txtpresets/2025.12/rc_link/redpine_mpm.txt
📚 Learning: 2025-05-25T22:26:17.328Z
Learnt from: bensonk
Repo: betaflight/firmware-presets PR: 509
File: presets/4.5/osd/bensonk.txt:138-152
Timestamp: 2025-05-25T22:26:17.328Z
Learning: BrainFPV commands in Betaflight preset files are safely ignored on non-BrainFPV flight controllers, making it safe to include them in presets even when they might be applied to different hardware.
Applied to files:
presets/2025.12/rc_link/generic/150hz_freestyle.txtpresets/2025.12/rc_link/ghost/normal.txtpresets/2025.12/rc_link/generic/1000hz_race.txtpresets/2025.12/rc_link/DJI_Normal.txtpresets/2025.12/rc_link/tbs/tracer_250hz.txtpresets/2025.12/rc_link/generic/250hz_freestyle.txtpresets/2025.12/rc_link/generic/50hz_hd_freestyle.txtpresets/2025.12/rc_link/redpine_666hz.txtpresets/2025.12/rc_link/tbs/crossfire_150hz.txtpresets/2025.12/rc_link/elrs_50hz.txtpresets/2025.12/rc_link/ghost/pure_race.txtpresets/2025.12/rc_link/DJI_SBUS_Fast.txtpresets/2025.12/rc_link/generic/250hz_hd_freestyle.txtpresets/2025.12/rc_link/generic/500hz_freestyle.txtpresets/2025.12/rc_link/generic/250hz_race.txtpresets/2025.12/rc_link/generic/150hz_ultra_cinematic.txtpresets/2025.12/rc_link/tbs/crossfire_50hz.txtpresets/2025.12/rc_link/generic/150hz_race.txtpresets/2025.12/rc_link/ghost/race.txtpresets/2025.12/rc_link/elrs_150hz.txtpresets/2025.12/rc_link/elrs_250hz.txtpresets/2025.12/rc_link/ghost/long_range.txtpresets/2025.12/rc_link/generic/150hz_cinematic.txtpresets/2025.12/rc_link/ghost/race250.txtpresets/2025.12/rc_link/generic/150hz_hd_freestyle.txtpresets/2025.12/rc_link/elrs_500hz.txtpresets/2025.12/rc_link/generic/500hz_hd_freestyle.txtpresets/2025.12/rc_link/ghost/solid250.txtpresets/2025.12/rc_link/ghost/race500.txtpresets/2025.12/rc_link/frsky_sbus_fport.txtpresets/2025.12/rc_link/defaults.txtpresets/2025.12/rc_link/generic/50hz_freestyle.txtpresets/2025.12/rc_link/redpine_mpm.txt
📚 Learning: 2025-06-20T18:08:54.334Z
Learnt from: Jaz7Mutant
Repo: betaflight/firmware-presets PR: 512
File: presets/4.5/tune/basher_jazzmutant_tune.txt:1-8
Timestamp: 2025-06-20T18:08:54.334Z
Learning: In Betaflight firmware preset files, embedded double quotes in metadata fields like TITLE and KEYWORDS (e.g., `5"`, `3" to 5"`, `8"/9"`) are standard and do not need escaping. The preset parser handles this syntax correctly as evidenced by widespread usage throughout the codebase.
Applied to files:
presets/2025.12/rc_link/generic/150hz_freestyle.txtpresets/2025.12/rc_link/generic/1000hz_race.txtpresets/2025.12/rc_link/tbs/tracer_250hz.txtpresets/2025.12/rc_link/generic/250hz_freestyle.txtpresets/2025.12/rc_link/generic/50hz_hd_freestyle.txtpresets/2025.12/rc_link/tbs/crossfire_150hz.txtpresets/2025.12/rc_link/elrs_50hz.txtpresets/2025.12/rc_link/ghost/pure_race.txtpresets/2025.12/rc_link/DJI_SBUS_Fast.txtpresets/2025.12/rc_link/generic/250hz_hd_freestyle.txtpresets/2025.12/rc_link/generic/500hz_freestyle.txtpresets/2025.12/rc_link/generic/150hz_ultra_cinematic.txtpresets/2025.12/rc_link/tbs/crossfire_50hz.txtpresets/2025.12/rc_link/elrs_150hz.txtpresets/2025.12/rc_link/elrs_250hz.txtpresets/2025.12/rc_link/generic/150hz_cinematic.txtpresets/2025.12/rc_link/ghost/race250.txtpresets/2025.12/rc_link/generic/150hz_hd_freestyle.txtpresets/2025.12/rc_link/generic/500hz_hd_freestyle.txtpresets/2025.12/rc_link/ghost/solid250.txtpresets/2025.12/rc_link/ghost/race500.txtpresets/2025.12/rc_link/frsky_sbus_fport.txtpresets/2025.12/rc_link/defaults.txtpresets/2025.12/rc_link/generic/50hz_freestyle.txtpresets/2025.12/rc_link/redpine_mpm.txt
🪛 LanguageTool
presets/2025.12/rc_link/generic/150hz_freestyle.txt
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ... RC link. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
presets/2025.12/rc_link/ghost/normal.txt
[uncategorized] ~57-~57: This expression is usually spelled with a hyphen.
Context: ...PTION END #$ OPTION BEGIN (UNCHECKED): Ultra Smooth Cinematic #$ INCLUDE: presets/2025....
(ULTRA_HYPHEN)
presets/2025.12/rc_link/generic/50hz_race.txt
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ... RC link. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
presets/2025.12/rc_link/generic/1000hz_race.txt
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ...or 4.4.1+ #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
presets/2025.12/rc_link/generic/250hz_cinematic.txt
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ... RC link. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
presets/2025.12/rc_link/generic/500hz_race.txt
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ... RC link. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
presets/2025.12/rc_link/DJI_Normal.txt
[misspelling] ~6-~6: Possible spelling mistake found.
Context: ... link, transmitter #$ AUTHOR: UAV Tech (Mark Spatz) #$ PARSER: MARKED #$ DESCRIPTION: In...
(EN_MULTITOKEN_SPELLING_TWO)
presets/2025.12/rc_link/tbs/tracer_250hz.txt
[uncategorized] ~51-~51: This expression is usually spelled with a hyphen.
Context: ...PTION END #$ OPTION BEGIN (UNCHECKED): Ultra Smooth Cinematic #$ INCLUDE: presets/2025....
(ULTRA_HYPHEN)
presets/2025.12/rc_link/generic/250hz_freestyle.txt
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ... RC link. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
presets/2025.12/rc_link/generic/50hz_cinematic.txt
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ... RC link. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
presets/2025.12/rc_link/generic/50hz_hd_freestyle.txt
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ... RC link. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
presets/2025.12/rc_link/redpine_666hz.txt
[uncategorized] ~35-~35: It appears that a hyphen is missing (if ‘auto’ is not used in the context of ‘cars’).
Context: ...throttle = 25 #$ OPTION END # at 500hz auto smoothing of 150 is close to 60hz: #$ OPTION BEGI...
(AUTO_HYPHEN)
[uncategorized] ~60-~60: This expression is usually spelled with a hyphen.
Context: ...g_throttle_cutoff = 20 #$ OPTION END # ultra smooth Cinematic (not for racing): #$ OPTION B...
(ULTRA_HYPHEN)
[uncategorized] ~61-~61: This expression is usually spelled with a hyphen.
Context: ...r racing): #$ OPTION BEGIN (UNCHECKED): Ultra Cinematic set feedforward_averaging = 3_POINT set...
(ULTRA_HYPHEN)
presets/2025.12/rc_link/tbs/crossfire_150hz.txt
[uncategorized] ~52-~52: This expression is usually spelled with a hyphen.
Context: ...PTION END #$ OPTION BEGIN (UNCHECKED): Ultra Smooth Cinematic #$ INCLUDE: presets/2025....
(ULTRA_HYPHEN)
presets/2025.12/rc_link/generic/500hz_cinematic.txt
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ... RC link. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
presets/2025.12/rc_link/elrs_50hz.txt
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ...via CRSF. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
[grammar] ~16-~16: It seems that “to” is missing before the verb.
Context: ...gs in case no tuning option is selected set feedforward_averaging = OFF set feedfor...
(MISSING_TO_BETWEEN_BE_AND_VB)
[uncategorized] ~52-~52: This expression is usually spelled with a hyphen.
Context: ...g_throttle_cutoff = 20 #$ OPTION END # ultra smooth Cinematic (not for racing): #$ OPTION B...
(ULTRA_HYPHEN)
[uncategorized] ~53-~53: This expression is usually spelled with a hyphen.
Context: ...r racing): #$ OPTION BEGIN (UNCHECKED): Ultra Cinematic set feedforward_smooth_factor = 40 set ...
(ULTRA_HYPHEN)
presets/2025.12/rc_link/ghost/pure_race.txt
[uncategorized] ~56-~56: This expression is usually spelled with a hyphen.
Context: ...PTION END #$ OPTION BEGIN (UNCHECKED): Ultra Smooth Cinematic #$ INCLUDE: presets/2025....
(ULTRA_HYPHEN)
presets/2025.12/rc_link/generic/500hz_ultra_cinematic.txt
[uncategorized] ~1-~1: This expression is usually spelled with a hyphen.
Context: #$ TITLE: Generic 500Hz Ultra Cinematic #$ FIRMWARE_VERSION: 2025.12 #$ CATEGOR...
(ULTRA_HYPHEN)
[uncategorized] ~7-~7: This expression is usually spelled with a hyphen.
Context: ...SCRIPTION: Generic RC link settings for ultra smooth cinematic flying with 500Hz RC link. #$...
(ULTRA_HYPHEN)
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ... RC link. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
presets/2025.12/rc_link/DJI_SBUS_Fast.txt
[misspelling] ~6-~6: Possible spelling mistake found.
Context: ...c, link, SBUS fast #$ AUTHOR: UAV Tech (Mark Spatz) #$ PARSER: MARKED #$ DESCRIPTION: In...
(EN_MULTITOKEN_SPELLING_TWO)
presets/2025.12/rc_link/generic/50hz_ultra_cinematic.txt
[uncategorized] ~1-~1: This expression is usually spelled with a hyphen.
Context: #$ TITLE: Generic 50Hz Ultra Cinematic #$ FIRMWARE_VERSION: 2025.12 #$ CATEGOR...
(ULTRA_HYPHEN)
[uncategorized] ~7-~7: This expression is usually spelled with a hyphen.
Context: ...SCRIPTION: Generic RC link settings for ultra smooth cinematic flying with 50Hz RC link. #$ ...
(ULTRA_HYPHEN)
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ... RC link. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
presets/2025.12/rc_link/generic/250hz_hd_freestyle.txt
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ... RC link. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
presets/2025.12/rc_link/generic/500hz_freestyle.txt
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ... RC link. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
presets/2025.12/rc_link/generic/250hz_race.txt
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ... RC link. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
presets/2025.12/rc_link/generic/150hz_ultra_cinematic.txt
[uncategorized] ~1-~1: This expression is usually spelled with a hyphen.
Context: #$ TITLE: Generic 150Hz Ultra Cinematic #$ FIRMWARE_VERSION: 2025.12 #$ CATEGOR...
(ULTRA_HYPHEN)
[uncategorized] ~7-~7: This expression is usually spelled with a hyphen.
Context: ...SCRIPTION: Generic RC link settings for ultra smooth cinematic with 150Hz RC link. #$ DESCRI...
(ULTRA_HYPHEN)
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ... RC link. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
presets/2025.12/rc_link/tbs/crossfire_50hz.txt
[uncategorized] ~53-~53: This expression is usually spelled with a hyphen.
Context: ...PTION END #$ OPTION BEGIN (UNCHECKED): Ultra Smooth Cinematic #$ INCLUDE: presets/2025....
(ULTRA_HYPHEN)
presets/2025.12/rc_link/generic/150hz_race.txt
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ... RC link. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
presets/2025.12/rc_link/ghost/race.txt
[uncategorized] ~55-~55: This expression is usually spelled with a hyphen.
Context: ...PTION END #$ OPTION BEGIN (UNCHECKED): Ultra Smooth Cinematic #$ INCLUDE: presets/2025....
(ULTRA_HYPHEN)
presets/2025.12/rc_link/elrs_150hz.txt
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ...via CRSF. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
[grammar] ~16-~16: It seems that “to” is missing before the verb.
Context: ...gs in case no tuning option is selected set feedforward_averaging = OFF set feedfor...
(MISSING_TO_BETWEEN_BE_AND_VB)
[uncategorized] ~54-~54: This expression is usually spelled with a hyphen.
Context: ...g_throttle_cutoff = 20 #$ OPTION END # ultra smooth Cinematic (not for racing): #$ OPTION B...
(ULTRA_HYPHEN)
[uncategorized] ~55-~55: This expression is usually spelled with a hyphen.
Context: ...r racing): #$ OPTION BEGIN (UNCHECKED): Ultra Cinematic set feedforward_smooth_factor = 40 set ...
(ULTRA_HYPHEN)
presets/2025.12/rc_link/elrs_250hz.txt
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ...via CRSF. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
[uncategorized] ~32-~32: It appears that a hyphen is missing (if ‘auto’ is not used in the context of ‘cars’).
Context: ...tle = 25 #$ OPTION END # for freestyle auto smoothing of 52 is about 60hz: #$ OPTION BEGIN (U...
(AUTO_HYPHEN)
[uncategorized] ~56-~56: This expression is usually spelled with a hyphen.
Context: ...g_throttle_cutoff = 20 #$ OPTION END # ultra smooth Cinematic (not for racing): #$ OPTION B...
(ULTRA_HYPHEN)
[uncategorized] ~57-~57: This expression is usually spelled with a hyphen.
Context: ...r racing): #$ OPTION BEGIN (UNCHECKED): Ultra Cinematic set feedforward_smooth_factor = 65 set ...
(ULTRA_HYPHEN)
presets/2025.12/rc_link/ghost/long_range.txt
[uncategorized] ~13-~13: If this is a compound adjective that modifies the following noun, use a hyphen.
Context: ...5Hz)**. #$ DESCRIPTION: #$ DESCRIPTION: Long Range mode: For the extreme stuff, crazy rang...
(EN_COMPOUND_ADJECTIVE_INTERNAL)
presets/2025.12/rc_link/generic/150hz_cinematic.txt
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ... RC link. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
presets/2025.12/rc_link/ghost/race250.txt
[uncategorized] ~56-~56: This expression is usually spelled with a hyphen.
Context: ...PTION END #$ OPTION BEGIN (UNCHECKED): Ultra Smooth Cinematic #$ INCLUDE: presets/2025....
(ULTRA_HYPHEN)
presets/2025.12/rc_link/generic/150hz_hd_freestyle.txt
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ... RC link. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
presets/2025.12/rc_link/elrs_500hz.txt
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ...via CRSF. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
[uncategorized] ~33-~33: It appears that a hyphen is missing (if ‘auto’ is not used in the context of ‘cars’).
Context: ...throttle = 25 #$ OPTION END # at 500hz auto smoothing of 120 is close to 60hz: #$ OPTION BEGI...
(AUTO_HYPHEN)
[uncategorized] ~57-~57: This expression is usually spelled with a hyphen.
Context: ...g_throttle_cutoff = 20 #$ OPTION END # ultra smooth Cinematic (not for racing): #$ OPTION B...
(ULTRA_HYPHEN)
[uncategorized] ~58-~58: This expression is usually spelled with a hyphen.
Context: ...r racing): #$ OPTION BEGIN (UNCHECKED): Ultra Cinematic set feedforward_smooth_factor = 75 set ...
(ULTRA_HYPHEN)
presets/2025.12/rc_link/generic/500hz_hd_freestyle.txt
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ... RC link. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
presets/2025.12/rc_link/ghost/solid250.txt
[uncategorized] ~56-~56: This expression is usually spelled with a hyphen.
Context: ...PTION END #$ OPTION BEGIN (UNCHECKED): Ultra Smooth Cinematic #$ INCLUDE: presets/2025....
(ULTRA_HYPHEN)
presets/2025.12/rc_link/ghost/race500.txt
[uncategorized] ~56-~56: This expression is usually spelled with a hyphen.
Context: ...PTION END #$ OPTION BEGIN (UNCHECKED): Ultra Smooth Cinematic #$ INCLUDE: presets/2025....
(ULTRA_HYPHEN)
presets/2025.12/rc_link/frsky_sbus_fport.txt
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ...ort link. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
[uncategorized] ~77-~77: This expression is usually spelled with a hyphen.
Context: ...g_throttle_cutoff = 20 #$ OPTION END # ultra smooth Cinematic (not for racing): #$ OPTION B...
(ULTRA_HYPHEN)
[uncategorized] ~78-~78: This expression is usually spelled with a hyphen.
Context: ...r racing): #$ OPTION BEGIN (UNCHECKED): Ultra Cinematic set feedforward_smooth_factor = 45 set ...
(ULTRA_HYPHEN)
presets/2025.12/rc_link/generic/250hz_ultra_cinematic.txt
[uncategorized] ~1-~1: This expression is usually spelled with a hyphen.
Context: #$ TITLE: Generic 250Hz Ultra Cinematic #$ FIRMWARE_VERSION: 2025.12 #$ CATEGOR...
(ULTRA_HYPHEN)
[uncategorized] ~7-~7: This expression is usually spelled with a hyphen.
Context: ...SCRIPTION: Generic RC link settings for ultra smooth cinematic with 250Hz RC link. #$ DESCRI...
(ULTRA_HYPHEN)
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ... RC link. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
presets/2025.12/rc_link/generic/50hz_freestyle.txt
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ... RC link. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
presets/2025.12/rc_link/redpine_mpm.txt
[uncategorized] ~45-~45: This expression is usually spelled with a hyphen.
Context: ...g_throttle_cutoff = 20 #$ OPTION END # ultra smooth Cinematic (not for racing): #$ OPTION B...
(ULTRA_HYPHEN)
[uncategorized] ~46-~46: This expression is usually spelled with a hyphen.
Context: ...r racing): #$ OPTION BEGIN (UNCHECKED): Ultra Cinematic set feedforward_jitter_factor = 14 set ...
(ULTRA_HYPHEN)
🔇 Additional comments (38)
presets/2025.12/rc_link/generic/150hz_freestyle.txt (1)
1-17: File structure and configuration look good.All metadata headers are properly formatted, INCLUDE directive correctly references the defaults file, and configuration values are appropriately tuned for the 150Hz Freestyle profile.
presets/2025.12/rc_link/generic/50hz_freestyle.txt (1)
1-18: File structure and configuration are appropriate for 50Hz Freestyle profile.The configuration correctly uses lower smoothing factors and boost settings suitable for 50Hz operation. All metadata is properly formatted and INCLUDE directive is correct.
presets/2025.12/rc_link/generic/50hz_race.txt (1)
1-18: File structure and Race profile configuration are well-tuned.The lower jitter factor (7 vs 9-10 in freestyle) appropriately reflects the tighter responsiveness needed for racing. INCLUDE directive and metadata are correct.
presets/2025.12/rc_link/generic/250hz_ultra_cinematic.txt (1)
1-22: File structure and Ultra Cinematic profile configuration are well-structured.The high RC smoothing auto factors (250) and increased smoothing/jitter factors are appropriately tuned for ultra-smooth cinematic flying. All metadata is properly formatted and INCLUDE directive is correct. The "Ultra Cinematic" terminology (without hyphen) is standard in Betaflight presets.
presets/2025.12/rc_link/generic/50hz_ultra_cinematic.txt (1)
1-23: Configuration appropriately scaled for 50Hz Ultra Cinematic profile.The rc_smoothing_auto_factor of 90 (vs 250 at higher Hz) correctly reflects the constraints of 50Hz operation, while jitter factor 20 and smooth factor 40 provide the ultra-smooth characteristics. All metadata and INCLUDE directive are correct.
presets/2025.12/rc_link/generic/250hz_freestyle.txt (1)
1-19: 250Hz Freestyle profile configuration is well-balanced.The feedforward_smooth_factor of 45 and rc_smoothing_auto_factor of 52 provide appropriate balance for freestyle flying at 250Hz. Metadata headers and INCLUDE directive are correctly formatted.
presets/2025.12/rc_link/generic/250hz_race.txt (1)
1-21: Race profile configuration appropriately tuned for competitive performance.The lower smooth_factor (35), jitter_factor (4), and rc_smoothing_auto_factor (25) correctly prioritize responsiveness over smoothing. feedforward_boost of 18 provides adequate feedforward for race dynamics. Metadata and INCLUDE directive are correct.
presets/2025.12/rc_link/generic/150hz_cinematic.txt (1)
1-22: Cinematic profile configuration is appropriate for 150Hz smooth flying.The rc_smoothing_auto_factor of 175 and throttle_factor of 100 provide smooth cinematic behavior while setpoint_cutoff of 12 maintains responsiveness. All metadata is properly formatted and INCLUDE directive is correct.
presets/2025.12/rc_link/generic/500hz_freestyle.txt (1)
1-18: Preset structure and configuration look good.The file follows established patterns with proper INCLUDE of defaults and consistent parameter organization.
presets/2025.12/rc_link/tbs/crossfire_150hz.txt (1)
36-54: Well-structured option blocks with appropriate presets.The UNCHECKED option groups provide users clear choices across flight profiles, and the pattern follows best practices for modular preset composition.
presets/2025.12/rc_link/generic/500hz_race.txt (1)
1-20: Preset configuration is sound.Race profile parameters are well-tuned with appropriate feedforward boost and smoothing factors.
presets/2025.12/rc_link/generic/500hz_hd_freestyle.txt (1)
1-19: HD Freestyle preset is well-configured.Parameter tuning appropriately balances responsiveness with smoothness for this profile.
presets/2025.12/rc_link/generic/150hz_hd_freestyle.txt (1)
1-20: 150Hz HD Freestyle preset is correctly tuned.Frequency-appropriate parameter selection with OFF averaging and conservative smoothing factors.
presets/2025.12/rc_link/generic/500hz_ultra_cinematic.txt (1)
1-21: Ultra-cinematic preset parameters are well-tuned.Aggressive smoothing and low setpoint cutoff (6) deliver the cinematic effect while maintaining control fidelity.
presets/2025.12/rc_link/generic/250hz_hd_freestyle.txt (1)
1-20: 250Hz HD Freestyle preset is correctly configured.Frequency-appropriate parameter tuning with balanced smoothing and feedforward settings.
presets/2025.12/rc_link/ghost/long_range.txt (1)
1-37: Missing FORCE_OPTIONS_REVIEW flag compared to similar presets.While this preset is functionally correct, other similar RC_LINK presets in this PR (e.g., tbs/crossfire_50hz.txt, tbs/tracer_250hz.txt) include
#$ FORCE_OPTIONS_REVIEW: TRUE. Since this preset defines RC link tuning parameters that users should be aware of, consider adding this flag for consistency unless there's a specific reason to omit it.Is the absence of
FORCE_OPTIONS_REVIEWintentional for this Long Range preset, or should it be added for consistency with other presets in the same category?presets/2025.12/rc_link/elrs_500hz.txt (1)
1-30: Missing PARSER: MARKED tag; confirm intentional.Unlike other RC_LINK presets in this PR, this file lacks
#$ PARSER: MARKEDdeclaration (which appears early in other presets). Verify this is intentional, as the MARKED parser is used consistently throughout the other RC_LINK presets for correct parsing of metadata and options.Should this preset include
#$ PARSER: MARKEDfor consistency with other RC_LINK presets, or is the omission intentional?presets/2025.12/rc_link/ghost/race.txt (2)
5-5: Clarify 166Hz vs 150Hz naming in keywords and includes.The KEYWORDS on line 5 specify "166Hz" and the title on line 11 describes "166Hz", but the preset uses 150Hz base parameters and includes 150Hz generic presets (line 40, 44, 48, 52, 56). If Ghost Race mode is 166Hz, consider whether a dedicated 166Hz preset family should be created, or clarify in the description why 150Hz settings are being used as an approximation.
Is the use of 150Hz presets for the 166Hz Ghost Race mode intentional as the closest available frequency match, or should dedicated 166Hz presets be created?
Also applies to: 11-11
1-57: All five generic 150Hz preset files referenced in this preset are present in the PR and correctly included.presets/2025.12/rc_link/ghost/solid250.txt (1)
1-58: All referenced generic 250Hz preset files are present in the PR and correctly included.presets/2025.12/rc_link/ghost/race250.txt (1)
1-58: All referenced generic 250Hz preset files are present and included in this PR. No issues to address.presets/2025.12/rc_link/DJI_Normal.txt (1)
1-72: The defaults file is present and the include will work correctly.The verification confirms that
presets/2025.12/rc_link/defaults.txtexists in the repository. The DJI_Normal.txt preset can successfully include this file on line 32, and all RC_LINK presets will function as intended.presets/2025.12/rc_link/tbs/tracer_250hz.txt (1)
1-53: All referenced generic 250Hz preset files are present—no issues found.The five generic preset files included via
presets/2025.12/rc_link/generic/250hz_*.txtpaths are all present in the PR:250hz_race.txt,250hz_freestyle.txt,250hz_hd_freestyle.txt,250hz_cinematic.txt, and250hz_ultra_cinematic.txt. The preset structure and dependencies are complete.presets/2025.12/rc_link/tbs/crossfire_50hz.txt (1)
1-55: No issues found — all referenced generic preset files are present.All five generic 50Hz preset files referenced in the crossfire_50hz.txt preset exist in the repository:
50hz_race.txt50hz_freestyle.txt50hz_hd_freestyle.txt50hz_cinematic.txt50hz_ultra_cinematic.txtThe preset will apply without dependency failures.
presets/2025.12/rc_link/defaults.txt (1)
1-25: Well-structured defaults file with clear parameter baseline.The metadata and configuration parameters are properly formatted. This file provides a solid foundation for other RC link presets to build upon.
presets/2025.12/rc_link/ghost/pure_race.txt (1)
1-58: Well-organized Ghost Pure Race preset with comprehensive documentation.The file properly includes defaults, enables Ghost protocol features, establishes 250Hz base configuration, and provides modular options. Structure and option groups are well-formed.
presets/2025.12/rc_link/redpine_mpm.txt (1)
1-70: Well-structured Redpine MPM preset with appropriate multi-profile options.The file properly configures for multiprotocol module use, includes defaults, and provides fine-tuning groups for different flight styles. Structure and parameter ranges appear appropriate for Redpine's slower link characteristics.
presets/2025.12/rc_link/elrs_250hz.txt (1)
1-113: Comprehensive ELRS 250Hz preset with well-organized exclusive option groups.The file properly supports both connection methods (Serial/SPI), includes detailed fine-tuning profiles, and provides flexible voltage reading options. Structure and parameter selections are appropriate for 250Hz ExpressLRS links.
presets/2025.12/rc_link/redpine_666hz.txt (1)
1-89: Well-configured Redpine 666Hz preset with appropriate tuning profiles for Deviation Tx.The file properly distinguishes itself from the MPM preset (2_POINT averaging vs 3_POINT, different smoothing values) based on the specific transmission method. Fine-tuning options and cinematic rates are properly structured.
presets/2025.12/rc_link/elrs_150hz.txt (1)
1-111: Well-structured ELRS 150Hz preset with appropriate frequency-scaled tuning.The file properly adjusts parameters for lower 150Hz update rate (OFF averaging, lower smoothing factors) and maintains consistent structure with ELRS 250Hz preset. Option groups are properly formed and exclusive where appropriate.
presets/2025.12/rc_link/DJI_SBUS_Fast.txt (1)
1-72: Well-documented DJI SBUS Fast preset with appropriate configuration and profile options.The file properly enables SBUS Fast mode, includes well-formatted documentation, and provides three tuning profiles suitable for SBUS link characteristics. The reduced number of options (vs more complex presets) is appropriate for this link type.
presets/2025.12/rc_link/frsky_sbus_fport.txt (1)
1-155: Comprehensive FrSky preset with well-organized exclusive option groups for different configurations.The file properly supports both Sbus and FPort modes with appropriate parameter adjustments, includes hardware options for telemetry inversion/duplex, and provides voltage/units preferences. The warning about >8 channels with corresponding parameter adjustment is particularly valuable.
presets/2025.12/rc_link/elrs_50hz.txt (6)
16-20: LGTM.The base 50Hz feedforward settings are well-configured with appropriate defaults for this refresh rate.
23-62: LGTM.The Fine-tuning option group is well-structured with five unchecked profiles (Racing, Freestyle, HD Freestyle, Cinematic, Ultra Cinematic) that provide sensible tuning variations for different flight styles. Parameter adjustments are appropriate for each profile's intended use case.
65-77: LGTM.The EXCLUSIVE ELRS Rx connection group correctly enforces selection of one method (Serial or SPI) and applies appropriate configuration (serialrx_provider or rx_spi_protocol) for each.
80-91: LGTM.The EXCLUSIVE Voltage readings group is properly structured, allowing users to choose between single-cell or whole-pack voltage reporting via the report_cell_voltage toggle.
94-109: LGTM.The optional Cinematic Rates group provides a complete rate configuration profile with explicit ACTUAL rate type and tuned RC rates/expo/srates for smooth cinematic flying.
1-14: Dependency verified: defaults.txt exists and contains valid RC link settings.The
presets/2025.12/rc_link/defaults.txtfile is present and properly configured with base feedforward and rc_smoothing settings for all RC link presets. The INCLUDE directive on line 14 will resolve correctly. The preset is part of a consistent ELRS preset family (50Hz, 150Hz, 250Hz, 500Hz variants) and no critical issues were found.
limonspb
left a comment
limonspb
left a comment
There was a problem hiding this comment.
i know it's a lot of work, buc ould you, please, fix the padding inside the groups and options? ex:
#$ OPTION BEGIN (CHECKED): Region 1 name
<cli command n + 1>
<cli command n + 2>
...
<cli command m>
#$ OPTION END
#$ OPTION_GROUP BEGIN: This group name
#$ OPTION BEGIN (UNCHECKED): Region 2 name
<cli command m + 1>
<cli command m + 2>
#$ OPTION END
#$ OPTION BEGIN (UNCHECKED): Region 3 name
<cli command j + 1>
<cli command j + 2>
<cli command j + 3>
#$ OPTION END
#$ OPTION_GROUP END
|
not sure these are totally relevant now? @ctzsnooze here are the radio 'options' I added for the new karate race tune.. 
|
|
Smoothing factor doesn't need as much change as it used to have. It should now be quite a lot easier to treat all the different looprates in a more similar way and still have great results. |
|
ok it looks like you went a bit soft on ctz and UAV in your initial reviews @limonspb ;) |
There was a problem hiding this comment.
Actionable comments posted: 7
🧹 Nitpick comments (5)
presets/2025.12/rc_link/elrs_50hz.txt (1)
8-8: Simplify redundant warning text.The phrase "ABSOLUTELY SURE" is redundant; use "SURE" alone for clarity and brevity.
-#$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC Filter is un-checked! +#$ DESCRIPTION: WARNING: make SURE that the OpenTx or EdgeTx Hardware ADC Filter is un-checked!presets/2025.12/rc_link/elrs_250hz.txt (1)
8-8: Simplify redundant warning text.The phrase "ABSOLUTELY SURE" is redundant; use "SURE" alone for clarity and brevity.
-#$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC Filter is un-checked! +#$ DESCRIPTION: WARNING: make SURE that the OpenTx or EdgeTx Hardware ADC Filter is un-checked!presets/2025.12/rc_link/elrs_500hz.txt (2)
5-5: Remove trailing comma from KEYWORDS metadata.Line 5 has a trailing comma in the keyword list, inconsistent with the other ELRS presets in this PR. This likely represents incomplete editing.
-#$ KEYWORDS: ELRS, express, 500Hz, rc, rx, link, smoothing, +#$ KEYWORDS: ELRS, express, 500Hz, rc, rx, link, smoothing
8-8: Simplify redundant warning text.The phrase "ABSOLUTELY SURE" is redundant; use "SURE" alone for clarity and brevity.
-#$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC Filter is un-checked! +#$ DESCRIPTION: WARNING: make SURE that the OpenTx or EdgeTx Hardware ADC Filter is un-checked!presets/2025.12/rc_link/elrs_150hz.txt (1)
8-8: Simplify redundant warning text.The phrase "ABSOLUTELY SURE" is redundant; use "SURE" alone for clarity and brevity.
-#$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC Filter is un-checked! +#$ DESCRIPTION: WARNING: make SURE that the OpenTx or EdgeTx Hardware ADC Filter is un-checked!
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (11)
presets/2025.12/rc_link/DJI_Normal.txt(1 hunks)presets/2025.12/rc_link/DJI_SBUS_Fast.txt(1 hunks)presets/2025.12/rc_link/elrs_150hz.txt(1 hunks)presets/2025.12/rc_link/elrs_250hz.txt(1 hunks)presets/2025.12/rc_link/elrs_500hz.txt(1 hunks)presets/2025.12/rc_link/elrs_50hz.txt(1 hunks)presets/2025.12/rc_link/frsky_sbus_fport.txt(1 hunks)presets/2025.12/rc_link/generic/1000hz_race.txt(1 hunks)presets/2025.12/rc_link/ghost/normal.txt(1 hunks)presets/2025.12/rc_link/redpine_666hz.txt(1 hunks)presets/2025.12/rc_link/redpine_mpm.txt(1 hunks)
🚧 Files skipped from review as they are similar to previous changes (4)
- presets/2025.12/rc_link/redpine_mpm.txt
- presets/2025.12/rc_link/redpine_666hz.txt
- presets/2025.12/rc_link/ghost/normal.txt
- presets/2025.12/rc_link/DJI_SBUS_Fast.txt
🧰 Additional context used
🧠 Learnings (4)
📓 Common learnings
Learnt from: Jaz7Mutant
Repo: betaflight/firmware-presets PR: 512
File: presets/4.5/tune/basher_jazzmutant_tune.txt:118-119
Timestamp: 2025-06-20T20:39:08.723Z
Learning: In Betaflight firmware presets, rates defaults from presets/4.3/rates/defaults.txt are used across multiple firmware versions including 4.5, as the file explicitly declares support for firmware versions 4.3, 4.4, and 4.5. There is no separate defaults.txt file in presets/4.5/rates/ directory.
Learnt from: bensonk
Repo: betaflight/firmware-presets PR: 509
File: presets/4.5/osd/bensonk.txt:138-152
Timestamp: 2025-05-25T22:26:17.328Z
Learning: BrainFPV commands in Betaflight preset files are safely ignored on non-BrainFPV flight controllers, making it safe to include them in presets even when they might be applied to different hardware.
📚 Learning: 2025-05-25T22:26:17.328Z
Learnt from: bensonk
Repo: betaflight/firmware-presets PR: 509
File: presets/4.5/osd/bensonk.txt:138-152
Timestamp: 2025-05-25T22:26:17.328Z
Learning: BrainFPV commands in Betaflight preset files are safely ignored on non-BrainFPV flight controllers, making it safe to include them in presets even when they might be applied to different hardware.
Applied to files:
presets/2025.12/rc_link/frsky_sbus_fport.txtpresets/2025.12/rc_link/elrs_50hz.txtpresets/2025.12/rc_link/elrs_250hz.txtpresets/2025.12/rc_link/elrs_150hz.txtpresets/2025.12/rc_link/elrs_500hz.txtpresets/2025.12/rc_link/DJI_Normal.txt
📚 Learning: 2025-06-20T20:39:08.723Z
Learnt from: Jaz7Mutant
Repo: betaflight/firmware-presets PR: 512
File: presets/4.5/tune/basher_jazzmutant_tune.txt:118-119
Timestamp: 2025-06-20T20:39:08.723Z
Learning: In Betaflight firmware presets, rates defaults from presets/4.3/rates/defaults.txt are used across multiple firmware versions including 4.5, as the file explicitly declares support for firmware versions 4.3, 4.4, and 4.5. There is no separate defaults.txt file in presets/4.5/rates/ directory.
Applied to files:
presets/2025.12/rc_link/frsky_sbus_fport.txtpresets/2025.12/rc_link/elrs_50hz.txtpresets/2025.12/rc_link/elrs_250hz.txtpresets/2025.12/rc_link/elrs_150hz.txtpresets/2025.12/rc_link/elrs_500hz.txtpresets/2025.12/rc_link/DJI_Normal.txtpresets/2025.12/rc_link/generic/1000hz_race.txt
📚 Learning: 2025-06-20T18:08:54.334Z
Learnt from: Jaz7Mutant
Repo: betaflight/firmware-presets PR: 512
File: presets/4.5/tune/basher_jazzmutant_tune.txt:1-8
Timestamp: 2025-06-20T18:08:54.334Z
Learning: In Betaflight firmware preset files, embedded double quotes in metadata fields like TITLE and KEYWORDS (e.g., `5"`, `3" to 5"`, `8"/9"`) are standard and do not need escaping. The preset parser handles this syntax correctly as evidenced by widespread usage throughout the codebase.
Applied to files:
presets/2025.12/rc_link/frsky_sbus_fport.txtpresets/2025.12/rc_link/elrs_500hz.txtpresets/2025.12/rc_link/DJI_Normal.txt
🪛 GitHub Actions: CI - Indexer job (deploy if default branch - i.e. main)
presets/2025.12/rc_link/elrs_50hz.txt
[error] 69-69: line 69: end Option directive found but no Option to close
🪛 LanguageTool
presets/2025.12/rc_link/frsky_sbus_fport.txt
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ...ort link. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
[grammar] ~92-~92: Ensure spelling is correct
Context: ...KED): Full duplex telemetry (default is helf duplex) set tlm_halfduplex = OF...
(QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1)
presets/2025.12/rc_link/elrs_50hz.txt
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ...via CRSF. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
presets/2025.12/rc_link/elrs_250hz.txt
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ...via CRSF. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
presets/2025.12/rc_link/elrs_150hz.txt
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ...via CRSF. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
presets/2025.12/rc_link/elrs_500hz.txt
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ...via CRSF. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
presets/2025.12/rc_link/generic/1000hz_race.txt
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ... 2025.12. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
🔇 Additional comments (4)
presets/2025.12/rc_link/generic/1000hz_race.txt (1)
15-20: Parameter assignments look correct and conflict-free.The duplicate
feedforward_smooth_factorassignment issue flagged in previous review has been resolved. Current file contains no duplicate parameter assignments. The base parameters (feedforward_averaging, jitter_factor, boost, rc_smoothing factors) are well-chosen for 1000Hz racing and integrate cleanly with the included defaults.presets/2025.12/rc_link/DJI_Normal.txt (2)
8-8: Verify unfamiliar metadata field.Line 8 includes
#$ PARSER: MARKEDwhich is not documented in the provided context or learnings. Verify this is a valid Betaflight preset metadata field and that it's intended for this preset.
10-30: Well-structured preset with clear documentation and thoughtful option progression.The file provides comprehensive documentation explaining the DJI preset and three distinct flying profiles. The option blocks are logically organized—base settings on lines 41–43 establish defaults, then each option (Race, HD Freestyle, Cinematic) progressively adjusts parameters for different latency tolerances. The progression from lower rc_smoothing_auto_factor (25 in Race) to higher values (250 in Cinematic) reflects appropriate tuning strategy. SBUS configuration and feature flags are correctly set.
Also applies to: 45-69
presets/2025.12/rc_link/frsky_sbus_fport.txt (1)
1-114: Overall structure and configuration appear sound.The preset follows established patterns in the codebase:
- Proper
INCLUDEof the shared defaults baseline (line 13)- Clear exclusive option groups for protocol selection (Sbus/FPort)
- Well-structured fine-tuning profiles (Race → Ultra Cinematic) with progressive smoothing/jitter adjustments
- Optional hardware and unit configuration
FORCE_OPTIONS_REVIEW: TRUEflag appropriately set for user-facing complexityThe configuration aligns with the PR objective of providing protocol- and frequency-specific tuning for FrSky receivers at 111Hz.
There was a problem hiding this comment.
Actionable comments posted: 0
♻️ Duplicate comments (1)
presets/2025.12/rc_link/DJI_Normal.txt (1)
65-65: Remove trailing whitespace on line 65.This trailing-space issue was flagged in the previous review but remains unfixed.
Apply this diff:
- set rc_smoothing_auto_factor = 250 + set rc_smoothing_auto_factor = 250
🧹 Nitpick comments (1)
presets/2025.12/rc_link/elrs_50hz.txt (1)
73-73: Minor: Spelling inconsistency in option label.Line 73 uses British English "Centre" (
Actual, Centre = 2) while Betaflight firmware and the rest of this file use American English ("CENTER"). Standardize to "Center" for consistency.- #$ OPTION BEGIN (UNCHECKED): Actual, Centre = 2 + #$ OPTION BEGIN (UNCHECKED): Actual, Center = 2
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (5)
presets/2025.12/rc_link/DJI_Normal.txt(1 hunks)presets/2025.12/rc_link/elrs_250hz.txt(1 hunks)presets/2025.12/rc_link/elrs_50hz.txt(1 hunks)presets/2025.12/rc_link/frsky_sbus_fport.txt(1 hunks)presets/2025.12/rc_link/generic/1000hz_race.txt(1 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
- presets/2025.12/rc_link/generic/1000hz_race.txt
🧰 Additional context used
🧠 Learnings (4)
📓 Common learnings
Learnt from: Jaz7Mutant
Repo: betaflight/firmware-presets PR: 512
File: presets/4.5/tune/basher_jazzmutant_tune.txt:118-119
Timestamp: 2025-06-20T20:39:08.723Z
Learning: In Betaflight firmware presets, rates defaults from presets/4.3/rates/defaults.txt are used across multiple firmware versions including 4.5, as the file explicitly declares support for firmware versions 4.3, 4.4, and 4.5. There is no separate defaults.txt file in presets/4.5/rates/ directory.
Learnt from: bensonk
Repo: betaflight/firmware-presets PR: 509
File: presets/4.5/osd/bensonk.txt:138-152
Timestamp: 2025-05-25T22:26:17.328Z
Learning: BrainFPV commands in Betaflight preset files are safely ignored on non-BrainFPV flight controllers, making it safe to include them in presets even when they might be applied to different hardware.
📚 Learning: 2025-05-25T22:26:17.328Z
Learnt from: bensonk
Repo: betaflight/firmware-presets PR: 509
File: presets/4.5/osd/bensonk.txt:138-152
Timestamp: 2025-05-25T22:26:17.328Z
Learning: BrainFPV commands in Betaflight preset files are safely ignored on non-BrainFPV flight controllers, making it safe to include them in presets even when they might be applied to different hardware.
Applied to files:
presets/2025.12/rc_link/frsky_sbus_fport.txtpresets/2025.12/rc_link/DJI_Normal.txtpresets/2025.12/rc_link/elrs_250hz.txtpresets/2025.12/rc_link/elrs_50hz.txt
📚 Learning: 2025-06-20T20:39:08.723Z
Learnt from: Jaz7Mutant
Repo: betaflight/firmware-presets PR: 512
File: presets/4.5/tune/basher_jazzmutant_tune.txt:118-119
Timestamp: 2025-06-20T20:39:08.723Z
Learning: In Betaflight firmware presets, rates defaults from presets/4.3/rates/defaults.txt are used across multiple firmware versions including 4.5, as the file explicitly declares support for firmware versions 4.3, 4.4, and 4.5. There is no separate defaults.txt file in presets/4.5/rates/ directory.
Applied to files:
presets/2025.12/rc_link/frsky_sbus_fport.txtpresets/2025.12/rc_link/DJI_Normal.txtpresets/2025.12/rc_link/elrs_250hz.txtpresets/2025.12/rc_link/elrs_50hz.txt
📚 Learning: 2025-06-20T18:08:54.334Z
Learnt from: Jaz7Mutant
Repo: betaflight/firmware-presets PR: 512
File: presets/4.5/tune/basher_jazzmutant_tune.txt:1-8
Timestamp: 2025-06-20T18:08:54.334Z
Learning: In Betaflight firmware preset files, embedded double quotes in metadata fields like TITLE and KEYWORDS (e.g., `5"`, `3" to 5"`, `8"/9"`) are standard and do not need escaping. The preset parser handles this syntax correctly as evidenced by widespread usage throughout the codebase.
Applied to files:
presets/2025.12/rc_link/frsky_sbus_fport.txt
🪛 LanguageTool
presets/2025.12/rc_link/elrs_250hz.txt
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ...via CRSF. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
presets/2025.12/rc_link/elrs_50hz.txt
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ...via CRSF. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
🔇 Additional comments (4)
presets/2025.12/rc_link/elrs_50hz.txt (2)
67-70: Critical parser issue from previous review has been resolved.The malformed
OPTION BEGINdirective that previously triggered a pipeline error (line 67 had extra spaces in the comment marker) now has proper syntax. TheOPTION BEGINandOPTION ENDare correctly paired and the parser should accept this file.
16-48: Verify that manual feedforward/smoothing adjustments align with 2025.12 auto-detection.Per the PR objectives, feedforward smoothing is now automatically detected in firmware 2025.12, and community reviewers (sugaarK, Quick-Flash) questioned whether the extensive manual adjustments in this preset (lines 17–18, 24, 27, 30–31, 35–40, 43–48) remain necessary or beneficial. Clarify whether:
- These manual overrides disable or complement the automatic detection
- The adjustments are still recommended given the new auto-detection behavior
- The preset should be simplified to remove redundant tuning options
presets/2025.12/rc_link/frsky_sbus_fport.txt (1)
55-56: Remove trailing whitespace on lines 55–56.Both lines have trailing spaces after the numeric values.
Apply this diff:
- set rc_smoothing_auto_factor = 130 - set rc_smoothing_auto_factor_throttle = 70 + set rc_smoothing_auto_factor = 130 + set rc_smoothing_auto_factor_throttle = 70presets/2025.12/rc_link/elrs_250hz.txt (1)
41-41: Remove trailing whitespace on line 41.This line has trailing spaces after the numeric value.
Apply this diff:
- set rc_smoothing_auto_factor = 250 + set rc_smoothing_auto_factor = 250
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
presets/2025.12/rc_link/elrs_250hz.txt (1)
8-8: Minor style note: Redundant emphasis in warning text.The phrase "ABSOLUTELY SURE" is technically redundant; "SURE" alone conveys the meaning. However, given this appears in a critical safety warning about hardware ADC filter configuration, the emphasis may be intentional. Consider if the warning remains sufficiently clear with "make SURE" instead.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
presets/2025.12/rc_link/elrs_250hz.txt(1 hunks)
🧰 Additional context used
🧠 Learnings (3)
📓 Common learnings
Learnt from: Jaz7Mutant
Repo: betaflight/firmware-presets PR: 512
File: presets/4.5/tune/basher_jazzmutant_tune.txt:118-119
Timestamp: 2025-06-20T20:39:08.723Z
Learning: In Betaflight firmware presets, rates defaults from presets/4.3/rates/defaults.txt are used across multiple firmware versions including 4.5, as the file explicitly declares support for firmware versions 4.3, 4.4, and 4.5. There is no separate defaults.txt file in presets/4.5/rates/ directory.
Learnt from: bensonk
Repo: betaflight/firmware-presets PR: 509
File: presets/4.5/osd/bensonk.txt:138-152
Timestamp: 2025-05-25T22:26:17.328Z
Learning: BrainFPV commands in Betaflight preset files are safely ignored on non-BrainFPV flight controllers, making it safe to include them in presets even when they might be applied to different hardware.
📚 Learning: 2025-06-20T20:39:08.723Z
Learnt from: Jaz7Mutant
Repo: betaflight/firmware-presets PR: 512
File: presets/4.5/tune/basher_jazzmutant_tune.txt:118-119
Timestamp: 2025-06-20T20:39:08.723Z
Learning: In Betaflight firmware presets, rates defaults from presets/4.3/rates/defaults.txt are used across multiple firmware versions including 4.5, as the file explicitly declares support for firmware versions 4.3, 4.4, and 4.5. There is no separate defaults.txt file in presets/4.5/rates/ directory.
Applied to files:
presets/2025.12/rc_link/elrs_250hz.txt
📚 Learning: 2025-05-25T22:26:17.328Z
Learnt from: bensonk
Repo: betaflight/firmware-presets PR: 509
File: presets/4.5/osd/bensonk.txt:138-152
Timestamp: 2025-05-25T22:26:17.328Z
Learning: BrainFPV commands in Betaflight preset files are safely ignored on non-BrainFPV flight controllers, making it safe to include them in presets even when they might be applied to different hardware.
Applied to files:
presets/2025.12/rc_link/elrs_250hz.txt
🪛 LanguageTool
presets/2025.12/rc_link/elrs_250hz.txt
[style] ~8-~8: This phrase is redundant. Consider writing “SURE”.
Context: ...via CRSF. #$ DESCRIPTION: WARNING: make ABSOLUTELY SURE that the OpenTx or EdgeTx Hardware ADC ...
(ABSOLUTELY_SURE)
🔇 Additional comments (6)
presets/2025.12/rc_link/elrs_250hz.txt (6)
16-18: Base feedforward configuration looks correct.The foundational parameters are properly configured with valid syntax. These will serve as baseline values before option-group customizations are applied.
20-54: Fine-tuning profiles are well-structured and logically progressive.The five profiles (Racing → Cinematic → Ultra Cinematic) follow a logical progression from aggressive responsiveness to smooth handling. Syntax is valid throughout, and parameter variations are appropriate for each use case on a 250Hz ELRS link.
56-65: ELRS Rx connection method group properly structured.The EXCLUSIVE constraint and two connection options (Serial/CRSF vs SPI/ExpressLRS) correctly represent the standard ELRS architectures, with valid syntax and appropriate feature/protocol settings.
67-74: Voltage readings group is appropriately configured.The binary EXCLUSIVE choice between single cell and whole pack voltage reporting is clear and uses valid Betaflight configuration settings.
76-89: Cinematic Rates option group syntax and values are appropriate.The optional rates profile (ACTUAL type with minimal center point, zero expo, consistent rates/srates) is correctly configured for cinematic control. All syntax is valid.
14-14: Confirmed: defaults.txt file exists and is properly integrated.The file
presets/2025.12/rc_link/defaults.txtexists in the repository and contains valid preset configuration. The INCLUDE directive on line 14 is correctly structured and will load the base RC link settings without issues.
nerdCopter
left a comment
nerdCopter
left a comment
There was a problem hiding this comment.
- approving untested
- whomever merges, please make sure to SQUASH-MERGE
yes lol i did :) back in the days we didn't have a padding thing. Here just wanted to make sure this all aligned with rc link changes that im not familiar with... @Quick-Flash could you please elaborate if anything needs to eb changed? |
6 participants
Summary by CodeRabbit