-
Notifications
You must be signed in to change notification settings - Fork 528
-
Notifications
You must be signed in to change notification settings - Fork 528
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
[ERROR MPL-0040] Failed on cluster #5655
Comments
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
Should this be an OR issue or is it related to the setup of megaboom? |
Unknown. I have the reproduction case this morning, but I don't know anything about what is going on. Please advice. |
If you want OR developers to look at then it is best to file with OR. We don't track megaboom issues. |
This comment was marked as outdated.
This comment was marked as outdated.
@AcKoucher please give this high priority (a workaround or a solution) |
@AcKoucher @maliberty Found a workaround, tweak initial conditions |
From the megaboom PR:
|
It looks like there's a combination of things that make this somewhat peculiar.
Now, the actual problem seems to be that when we get to the point of placing the children of the cluster 4 in the first image - 7 in the second image after dead space filling - even with the target util variation SA can't fit the clusters in the outline. Apparently this happens, because the outline penalty never wins the fight against the boundary penalty. However there's something going on with the wire length, because for all the steps, I see zero at the debug report (perhaps it's too small I have to check).
My first suggestion would be to try decreasing the halos as @oharboe already did or decrease the boundary penalty. |
Thanks! Sounds like this is in good hands and well understood. No longer urgent for my part as we have a workaround. |
@oharboe Ok :-) I'm investigating what is going on with the clustering so we can have a proper fix. |
Another workaround I'm trying out is to save a macro placement. With a saved macro placement, I should avoid rtlmp errors due to slight changes in initial conditions, like changed PLACE_DENSITY.
|
@AcKoucher Please confirm that the fixes work on the full testcase of 1 hour I included a faster, 13 minute, testcase here, that I produced from the full testcase with deltaDebug.py. There is a risk that deltaDebug.py identified other bugs than the original bug... |
ah, the full test-case still fails... |
Can you re-delta? |
@maliberty New deltadeug: this test case takes ca. 13 minutes and fails on master: https://drive.google.com/file/d/1klYn7s2_uJBk2Wi-vfPKK_ol8Kwv02sY/view?usp=sharing |
13 minutes to reproduce:
untar https://drive.google.com/file/d/18n0z4_Bk9Gscy3RRCiU6FiNvghIb6zIG/view?usp=drive_link
Originally posted by @oharboe in The-OpenROAD-Project/megaboom#97 (comment)
The text was updated successfully, but these errors were encountered: