Skip to content

update process_fdn fucntion: add positional filter arguments - #202

Open
almahova wants to merge 2 commits into
mainfrom
refactor/process-fdn-api
Open

update process_fdn fucntion: add positional filter arguments#202
almahova wants to merge 2 commits into
mainfrom
refactor/process-fdn-api

Conversation

@almahova

Copy link
Copy Markdown
Collaborator
  1. Add positional filter arguments to process_fdn() function. The function now accepts td objects:
    post_delay: Any | None = None,
    post_matrix: Any | None = None,
    post_output: Any | None = None,

  2. Update every declaration related to process_fdn() function.

Testing

  • pytest passes locally
  • New tests added for new functionality
  • Coverage not decreased (pytest --cov=pyFDN)

Checklist

  • [ x] Pre-commit hooks pass (pre-commit run --all-files)
  • Docstrings updated for any changed public API
  • HISTORY.rst updated if this is a user-facing change

almahova added 2 commits July 21, 2026 14:07
…pt td objects (post_delay, post_matrix, post_output). update all declerations

@SebastianJiroSchlecht SebastianJiroSchlecht left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have read all the changes, and I think they are good as is. I have not executed the code, though. It's ok to merge in my view.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants