Logan Adams
Logan Adams
> > @loadams @arashb @tjruwase @mrwyattii @awan-10 Can anyone trigger the unit-tests please? > > Yes, though we have a known issue in CI now, so some tests will still...
@oelayan7 - I've got to push a fix for nv-accelerate tests, will get this merged shorlty.
This should be fixed now, let us know if you're still seeing this, but I see it fixed on my side now.
> So, is this bug solved or not? Are you able to test with the latest version from source to confirm?
> > Hi @tjruwase, Could you review this pls? > > @i4never, thanks for this PR. This is very old code, which is documented as hacky, and unfortunately the author...
> sure, I'm working on this. Thanks @i4never - ping me whenever this needs review/tests/etc.
> @loadams this PR was removed from merge queue for some reason. can you check? Hi @nelyahu - yes, I removed it because I wanted to make the pydantic changes...
> Changing this line has been associated with several bugs #5422, #5538 @nelyahu - thoughts on this comment, seems last time this line was modified users ran into issues?
> > i did offline debugging of those failure and improved the code change so it will pass > > @nelyahu, it great that you narrowed this down. Do you...
@twaka Can you try #5577?