Fix config structure name in pragma for SeparableConv1D

[qberthet]

Description

Simple fix for #883

closes #883

Type of change

For a new feature or function, please create an issue first to discuss it with us before submitting a pull request.

Note: Please delete options that are not relevant.

• [x] Bug fix (non-breaking change that fixes an issue)

Tests

Tested with example code provided in bug report #883 : Fail before applying this commit, works afterward.

Checklist

• [x] I have read the guidelines for contributing.
• [ ] I have commented my code, particularly in hard-to-understand areas.
• [ ] I have made corresponding changes to the documentation.
• [x] My changes generate no new warnings.
• [x] I have installed and run pre-commit on the files I edited or added.
• [ ] I have added tests that prove my fix is effective or that my feature works.

[jmitrevs]

Is it possible to add a pytest to show that there is a problem and that this fixes it? We don't do synthesis in pytests but we do compile the HLS code.

[qberthet]

I'll try to add this later today.

[qberthet]

@jmitrevs : I've added a test, but the problem only appear during CSynth, not CSim, so not sure if that will be ok (requires vivado). Also had to catch the return code of vivado_hls command to raise exception and make the test fail. I've duplicated sepconv2d test for sepconv1d and then added a test for the build step. Let me know if you prefer that I remove the tree first commits to only have the fix.

@vloncar : your fix obviously does the job. Thanks!

[qberthet]

And I just realize that my test is not setting the output folder correctly. Let me know if you want to keep these tests before I fix it.