Feature "Lowband LTE 4Rx" #1

New Issue

2025-07-25T06:28:25+02:00

rtommy commented

2025-07-25 06:28:25 +02:00

(Migrated from github.com)

Minor issue on "Lowband LTE 4Rx".

Both "Lowband NR 4Rx" and "Lowband LTE 4Rx" consider EN-DC combos but the "Log misses ..." text is not fully aligned to that for LTE case.

I would recommend to add it: "Log misses EN-DC and LTE-CA capabilities."

addSupportRow(hasLteCaData || hasEndc, foundLte4Rx, [...lte4RxBands].sort((a, b) => a - b).join(', '), "No lowband LTE 4Rx support", "Lowband LTE 4Rx support", "Log misses LTE-CA capabilities.");
addSupportRow(hasEndc || hasNrca, foundNr4Rx, [...nr4RxBands].sort((a, b) => a - b).map(band => `n${band}`).join(', '), "No lowband NR 4Rx support", "Lowband NR 4Rx support", "Log misses EN-DC and NR-CA capabilities.");

Minor issue on "Lowband LTE 4Rx". Both "Lowband NR 4Rx" and "Lowband LTE 4Rx" consider EN-DC combos but the "Log misses ..." text is not fully aligned to that for LTE case. I would recommend to add it: "Log misses EN-DC and LTE-CA capabilities." ```js addSupportRow(hasLteCaData || hasEndc, foundLte4Rx, [...lte4RxBands].sort((a, b) => a - b).join(', '), "No lowband LTE 4Rx support", "Lowband LTE 4Rx support", "Log misses LTE-CA capabilities."); addSupportRow(hasEndc || hasNrca, foundNr4Rx, [...nr4RxBands].sort((a, b) => a - b).map(band => `n${band}`).join(', '), "No lowband NR 4Rx support", "Lowband NR 4Rx support", "Log misses EN-DC and NR-CA capabilities."); ```

👍 1

high3eam commented

2025-07-28 07:41:12 +02:00

(Migrated from github.com)

Fixed in c805003
Thank you!

Fixed in c805003 Thank you!

rtommy commented

2025-07-28 07:57:02 +02:00

(Migrated from github.com)

You modified it to Log misses EN-DC and NR-CA capabilities instead of "Log misses EN-DC and LTE-CA capabilities."

This is an LTE related feature, not NR. 😄

You modified it to `Log misses EN-DC and NR-CA capabilities` instead of `"Log misses EN-DC and LTE-CA capabilities."` This is an LTE related feature, not NR. 😄

😆 1

high3eam commented

2025-07-28 08:17:17 +02:00

(Migrated from github.com)

@rtommy Whoops, fixed now!

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: henrik/uecapabilityparser-features-js#1