-
Notifications
You must be signed in to change notification settings - Fork 242
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
Exporting Verilog code of subblock changes subblock #530
Comments
Thanks for your feedback, i'm try to reproduce this bug and fix it thanks! |
Hi! please check if in the last WIP this is fixed!, i'm waiting for your news. Thanks! |
Platform: Windows 8.1 I can still reproduce the bug in the latest WIP |
If this bug exists, it will be directed related to: #651, so I will find out during testing and will fix it |
Thanks Tim! i'm checking it , have you find a pattern to reproduce? |
No I haven't looked at it. Just reviewing the Issues list :-)
…On Thu, Apr 18, 2024 at 4:52 PM Carlos Venegas Arrabé < ***@***.***> wrote:
Thanks Tim! i'm checking it , have you find a pattern to reproduce?
—
Reply to this email directly, view it on GitHub
<#530 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA6CTIXC2OXZFRRU7IA2ATLY6AXBJAVCNFSM5CKBHOMKU5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TEMBWGUZTANBSGM3A>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
Thanks Tim! |
When exporting a subblock as verilog (requiring the user to be in the edit mode):
![afbeelding](https://user-images.githubusercontent.com/13208934/129747567-a5c9417b-3685-42cd-aeaa-295645470023.png)
Icestudio changes the subblock ports to "FPGA ports" and converts the subblock to effectively a standalone block. This occurs when going from the edit mode ( the lock symbol is unlocked) to back to the read-only mode (the lock symbol should lock again).
The lock symbol disappears, and the program enters an unusable state as the original top level module disappears. This can possibly lead to a significant loss of work.
The text was updated successfully, but these errors were encountered: