Skip to content
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

Fix file name of the bitstream specification file in the documentation. #122

Merged
merged 4 commits into from
Nov 10, 2023

Conversation

IAmMarcelJung
Copy link
Collaborator

The file "meta_data.txt" does not seem to exist even after the flow is complete and the command works with "bitStreamSpec.bin". This file is also used in the function do_gen_bitStream_binary in FABulous.py.

@IAmMarcelJung
Copy link
Collaborator Author

I revoked the change of the FASM file name since gen_bitstream_binary uses a "_gen.fasm" file for the bitstream generation, but does expect just a ".fasm" file to be specified in the command and adds the "_gen" internally.

Copy link
Contributor

@TaoBi22 TaoBi22 left a comment

Choose a reason for hiding this comment

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

Thanks for this!! I've tried to pick up all the bits of documentation that are still hanging around from before we did our big CLI implementation and refactor but I guess I've still missed bits!

docs/source/FPGA-to-bitstream/Bitstream generation.rst Outdated Show resolved Hide resolved
@TaoBi22 TaoBi22 added the documentation Improvements or additions to documentation label Nov 10, 2023
@TaoBi22
Copy link
Contributor

TaoBi22 commented Nov 10, 2023

Great, thanks!

@TaoBi22 TaoBi22 merged commit a65d8e8 into FPGA-Research:master Nov 10, 2023
@IAmMarcelJung IAmMarcelJung deleted the fix_doc_bitstream_gen branch November 10, 2023 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants