Skip to content

Conversation

@BCapota
Copy link
Collaborator

@BCapota BCapota commented Feb 19, 2025

PR Description

Added the documentation page for fmcomms2 FIR filter example.

PR Type

  • Bug fix (change that fixes an issue)
  • New feature (change that adds new functionality)
  • Breaking change (has dependencies in other repos or will cause CI to fail)

PR Checklist

  • I have followed the code style guidelines
  • I have performed a self-review of changes
  • I have compiled all hdl projects and libraries affected by this PR
  • I have tested in hardware affected projects, at least on relevant boards
  • I have commented my code, at least hard-to-understand parts
  • I have signed off all commits from this PR
  • I have updated the documentation (wiki pages, ReadMe files, Copyright etc)
  • I have not introduced new Warnings/Critical Warnings on compilation
  • I have added new hdl testbenches or updated existing ones

@CLAassistant
Copy link

CLAassistant commented Feb 19, 2025

CLA assistant check
All committers have signed the CLA.

@IuliaCMoldovan
Copy link
Contributor

Did some minor changes on the text, wrapped it and some rephrasing.
Added links to new documentation.
Added some TODO notes for @BCapota to check and do in another commit.
Afterwards, we can do a rebase on latest main.

@BCapota BCapota marked this pull request as ready for review September 19, 2025 08:15
@BCapota BCapota force-pushed the doc_fir_filter branch 4 times, most recently from 1b1a3dc to ff0508d Compare September 26, 2025 08:08
@BCapota BCapota force-pushed the doc_fir_filter branch 2 times, most recently from 5306d8d to 0f0a857 Compare September 26, 2025 08:44
@BCapota BCapota force-pushed the doc_fir_filter branch 7 times, most recently from 1940953 to f31901b Compare December 3, 2025 09:07
@BCapota BCapota force-pushed the doc_fir_filter branch 3 times, most recently from 438082d to 827e3a2 Compare December 3, 2025 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants