Skip to content

Add signal tests using m2k#52

Open
jpineda3 wants to merge 9 commits intomainfrom
jpineda3/tests
Open

Add signal tests using m2k#52
jpineda3 wants to merge 9 commits intomainfrom
jpineda3/tests

Conversation

@jpineda3
Copy link
Collaborator

@jpineda3 jpineda3 commented Mar 18, 2025

Pull Request Description

The signal tests need libm2k which I managed to work only using R2022b. There are no changes in the device classes.

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 performed a self-review of the changes
  • I have commented my code, at least hard-to-understand parts
  • I have build all projects affected by the changes in this PR
  • I have tested in hardware affected projects, at the relevant boards
  • I have signed off all commits from this PR
  • I have updated the documentation by running /docs/source/reference_api/gen_sysobj_doc.m script locally, if applies.

SaikiranGudla
SaikiranGudla previously approved these changes Mar 27, 2025
@jpineda3 jpineda3 marked this pull request as draft April 7, 2025 04:05
@jpineda3
Copy link
Collaborator Author

jpineda3 commented Apr 7, 2025

I converted this to draft to fix libm2k use on 23b

Signed-off-by: Julia Pineda <Julia.Pineda@analog.com>
Signed-off-by: Julia Pineda <Julia.Pineda@analog.com>
Signed-off-by: Julia Pineda <Julia.Pineda@analog.com>
Signed-off-by: Julia Pineda <Julia.Pineda@analog.com>
Signed-off-by: Julia Pineda <Julia.Pineda@analog.com>
Signed-off-by: Julia Pineda <Julia.Pineda@analog.com>
Signed-off-by: Julia Pineda <Julia.Pineda@analog.com>
Signed-off-by: Pineda <Julia.Pineda@analog.com>
@jpineda3 jpineda3 force-pushed the jpineda3/tests branch 2 times, most recently from c06ec6c to 38e9788 Compare April 21, 2025 04:08
@jpineda3 jpineda3 marked this pull request as ready for review April 21, 2025 04:09
Signed-off-by: Pineda <Julia.Pineda@analog.com>
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