Replies: 14 comments 4 replies
-
I say go for a release. I don't think any of the open bugs are serious enough to delay it. I did have a quick look at #1041. I think the problem is that vpi_put_value sends the value to the output of the net, whereas it needs to send it to the input. I may have some free time this weekend to work on it. |
Beta Was this translation helpful? Give feedback.
-
I agree, this is a good time for a new release. I have removed the release specific bits from the ivtest suite, so releases should be even easier now. |
Beta Was this translation helpful? Give feedback.
-
One other thing that should be synced is the list of constructs not supported is incorrect in the local documentation. This had been updated on the external web page, but those changes never made it back into the repo documentation. We probably should do a pass over the entire documentation set to make sure things are as expected before the release. I'm going to try to finish #1038 this weekend and I need to look at the algorithm for time rounding in the context of my recent FreeBSD fix since it is numerically very unstable if being off by a single bit in the pow() function causes a test failure. |
Beta Was this translation helpful? Give feedback.
-
It does, since creating a new release would have required creating a ne
regression-<N>.list. That file was selected automatically.
And yes, lots of cruft was also cleaned out.
…On Thu, Jan 4, 2024 at 9:39 AM Cary R. ***@***.***> wrote:
I think once the tests were integrated into the repo the TB was then
synced. The recent removals is mostly getting ride of cruft that no longer
serves a purpose, but does not impact a release.
—
Reply to this email directly, view it on GitHub
<#1058 (reply in thread)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAACKMXT7M77CAC5FEYZ2ETYM3SNFAVCNFSM6AAAAABBG4YEVKVHI2DSMVQWIX3LMV43SRDJONRXK43TNFXW4Q3PNVWWK3TUHM4DAMJVGQ3TG>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
--
Steve Williams
***@***.*** ***@***.***>
|
Beta Was this translation helpful? Give feedback.
-
A newcomer here, so feel free to ignore me. I think I have a modified version of #667 that is ready to become a replacement PR. But there is no demonstrated use case, as that was blocked by #1041. It seems that there may now be progress on this, so for me it would be nice if there was time to validate #667 and have it considered for the next release. On the other hand #667 rearranges fundamental code in vvp and I would completely understand the response "That needs to simmer for months before being considered for release." The only reason this might matter is that my application is intended as a new feature in mixed-signal simulator ngspice (like this). Ngspice typically has three releases a year (with one last week). So if "vvp as shared library" misses the boat, I will have to remember to remove "a development build of iverilog must be used" from the Ngspice manual, a year or two from now. Thanks, |
Beta Was this translation helpful? Give feedback.
-
I see there is currently one unexpected failure in the vlog95 target tests. |
Beta Was this translation helpful? Give feedback.
-
Yes, it is related to the changes done for SDF support. I've not looked into if this is a bug in the compiler or a limitation in the vlog95 code generator. |
Beta Was this translation helpful? Give feedback.
-
Time is slipping by. It would be good to get a new release out soon. |
Beta Was this translation helpful? Give feedback.
-
I hope to have some time in a week or two to finish the task I was trying to get done for the release. My hectic day job is getting in the way of all the Icarus fun. |
Beta Was this translation helpful? Give feedback.
-
As we still haven't made a release, I'm minded to at least create a snapshot. However, now we are publishing on GitHub instead of icarus.com, there is a new problem. Previously Steve would include a suitable It would be nice if that process could be automated, but I don't know a way to do it. |
Beta Was this translation helpful? Give feedback.
-
I've been thinking about a snapshot as well. Is there some kind of github action we can do to create the version information, build the gperf files, etc. and then tar all that up? I would assume other repos have some way to deal with this so maybe we need to do a little research before adding another file repo. |
Beta Was this translation helpful? Give feedback.
-
I already did some research. AFAICT, it's possible to use github actions to create additional binary files that are published with the release (which could include a source code tarball), but I don't see any way to hide the automatically-generated source code bundles (tgz and zip) which are generated directly from a tagged git commit (I think they are generated on demand, not at the point of release). I'll be happy to be proved wrong though. We could use github actions to generate tarballs that are published elsewhere, but I think it would be best to publish them as github releases - that's where users will look for them. |
Beta Was this translation helpful? Give feedback.
-
P.S. I still think we should just go ahead and release v13. There will always be open bugs. Provided they aren't regressions from the previous release, they should not be a release blocker. |
Beta Was this translation helpful? Give feedback.
-
At this point I'm okay releasing v13. I would say it looks like work may not be so overwhelming soon so I can take care of the things I wanted for V13, but I've been proven wrong too many times to continue holding up the release. |
Beta Was this translation helpful? Give feedback.
-
A large number of changes have gone into the repo since out last release almost a year ago, so I believe it is time to start thinking about a V13 release.
I'm currently working on fixing a VPI issue #1038 and there is one more I would like to look at and fix if possible #1041. Are there any other bugs or enhancements people are working on that they would like to get into the repo before a new release?I'm trying to focus on bugs with the limited time I have right now. Also if someone else wants to look at #1041 that would just make it easier.
@steveicarus do you have time to roll a release once we decide it is completed? Hopefully in no more than a couple weeks.
I hope everyone is doing well. Take care, Cary
Beta Was this translation helpful? Give feedback.
All reactions