-
Notifications
You must be signed in to change notification settings - Fork 11
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use yaml for the config file and add solution specific settings #23
base: master
Are you sure you want to change the base?
Conversation
Hi Lukas, Thanks for submitting this! I'm very impressed, it looks like you have done a lot of work here. I'm also glad to hear that there are people out there using this. I'm keen to see this tool improve and progress if people find it useful and so would really like to get this update merged. I really like the idea of moving to YAML for configuration and I think if I had more knowledge at the time this is what i would have originally done, so a massive thanks for rolling that in. I also like the idea of the extended flexibility in terms of solution specific settings. You can probably tell from the lack of updates to this repo that I haven't done much with HLSCLT in the last couple of years. This is primarily because I haven't been using HLS as much for the last couple of years, either for work or personal projects. The reason I'm adding this note is really to say that it might take me a little longer than I'd like to get this PR reviewed and merged. I will try and get my Centos VM booted and pull your version for testing but can't guarantee how quickly I can do that. If I can get the code pulled and tested I might have a few questions to come back with. I noticed you pushed some extra updates after submitting the PR yesterday, do you consider the current status pretty stable? Or are you continuing to develop this further in the next few days? Can you let me know some more details about the system you use the tool on? I assume it's Linux based? You already mentioned Vivado version so that's perfect. Have you done any testing on other platforms? I had a previous issue discussing Windows 10 compatibility which I couldn't help with as I don't have a Windows 10 system to develop on. |
Hi Ben,
I am really sorry for the late reply.
I only used this tool for an University assignment and had not have
the capacity to put further work into this afterwards.
Regarding the stability: I added the ability to add solution specific files,
which is not compatible with a 'normal' workflow since added files are
kept between solutions. Therefore I inserted the `--reset` flag,
when opening the Project, which is bad because it removes all previous
artifacts generated by eg. `build` or `syn` subcommands on which
subcommands such as `sym` or `cosyn` depend.
Therefore I will remove that feature.
I did not notice any stability issues apart from that.
The bulk of changes were, PEP8 related anyways.
Regarding the System: I use Arch Linux 64bit, and used python3 instead
python2.
I have not tested it on Windows but removed linux specific stuff like superior
directory operators (eg. "/foo/bar/" replaced os.path.join("foo", "bar")),
where I found it.
…On Thu, Jun 10, 2021 at 11:34:40PM -0700, Ben Marshall wrote:
Hi Lukas,
Thanks for submitting this! I'm very impressed, it looks like you have done a lot of work here. I'm also glad to hear that there are people out there using this. I'm keen to see this tool improve and progress if people find it useful and so would really like to get this update merged.
I really like the idea of moving to YAML for configuration and I think if I had more knowledge at the time this is what i would have originally done, so a massive thanks for rolling that in. I also like the idea of the extended flexibility in terms of solution specific settings.
You can probably tell from the lack of updates to this repo that I haven't done much with HLSCLT in the last couple of years. This is primarily because I haven't been using HLS as much for the last couple of years, either for work or personal projects. The reason I'm adding this note is really to say that it might take me a little longer than I'd like to get this PR reviewed and merged. I will try and get my Centos VM booted and pull your version for testing but can't guarantee how quickly I can do that. If I can get the code pulled and tested I might have a few questions to come back with.
I noticed you pushed some extra updates after submitting the PR yesterday, do you consider the current status pretty stable? Or are you continuing to develop this further in the next few days?
Can you let me know some more details about the system you use the tool on? I assume it's Linux based? You already mentioned Vivado version so that's perfect. Have you done any testing on other platforms? I had a previous issue discussing Windows 10 compatibility which I couldn't help with as I don't have a Windows 10 system to develop on.
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
#23 (comment)
|
Created a module for the tcl commands and let the commands append to a list which is written to the file at the end. Formatting in tune with pep8.
Renamed parse_dict to pars_obj, to make it clear that the dot notation can be used (ie. `config.cflags` instead of `config['cflags'])
otherwise files from other solutions may be loaded
Hi, i just implemented said changes and polished the commit history. |
Hi, glad this tool exists!
I needed more solution specific settings, ie. setting directives not in the source but in the directives file and different subsets of the source files across solutions.
So I extended the functionality by
Changes to the codebase:
I use vivado hls 2018.1 so maybesome functionality is broken.
I tested all features on my machine and did not run into errors.