Skip to content
Snippets Groups Projects

AST-943 Put central frequency directly in WorkTable

Closed AST-943 Put central frequency directly in WorkTable
3 unresolved threads
Closed Maik Nijhuis requested to merge ast-943-use-central-frequency-only into main
3 unresolved threads

Since Radler only uses the central frequency, there is no need for storing separate band start and end frequencies.

Also fixes a bug when printing the WorkTable.

Merge request reports

Merged results pipeline #32233 passed

Merged results pipeline passed for f240a903

Test summary results are being parsed

Closed by Maik NijhuisMaik Nijhuis 2 years ago (Jun 23, 2022 7:13am UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
123 120 Deconvolution groups 1
124 121 Channel index 0
125 122 # Pol Ch Interval Weight Freq(MHz)
126 0 Q 0 4 1.234 10-10
127 1 I 1 8 1.01 2-2
128 2 U 0 16 1.1 200-200
  • 72 67 << " " << std::setw(2) << entry.original_channel_index << " "
    73 68 << std::setw(8) << entry.original_interval_index << " "
    74 69 << std::setw(6) << entry.image_weight << " "
    75 << round(entry.band_end_frequency * 1e-6) << "-"
  • 70 70 auto e = std::make_unique<WorkTableEntry>();
    71 71 e->original_channel_index = outChannel;
    72 72 e->polarization = pol;
    73 e->band_start_frequency = frequencyMHz;
    • Resolved by Maik Nijhuis

      Storing just the central frequency does not give enough information. Some bands could be wider than others (giving useful information for interpolation). So yes, this metadata could matter.

  • Well spotted bugs!

  • closed

  • Maik Nijhuis mentioned in merge request !63 (merged)

    mentioned in merge request !63 (merged)

  • Please register or sign in to reply
    Loading