-
Notifications
You must be signed in to change notification settings - Fork 415
Automatically generate floorplan test files after placement, and enhancements to cluster attraction groups #1938
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
Merged
Merged
Changes from 15 commits
Commits
Show all changes
60 commits
Select commit
Hold shift + click to select a range
2276efb
Fixed bug in region tile count by intersecting region with grid befor…
sfkhalid bbaa785
Added ability to generate different constraints file by creating part…
sfkhalid 0b71f8b
Added ability to print multiple constraints files at once and add deb…
sfkhalid 15ee41d
Commented out debug line
sfkhalid 1280f59
Adding changes to way molecule gain is updated according to attractio…
sfkhalid b24ce10
Changed way molecule gain is updated when it is part of an attraction…
sfkhalid 0aa6e7a
Added new error checking routines during clustering and placement, ad…
sfkhalid 2702e13
When a cluster fails routing, its PartitionRegion must be reset set t…
sfkhalid 066a996
Merge branch 'master' into generate_test_constraints_files
sfkhalid b521467
Improved comment for new routine in initial placement
sfkhalid e370bbf
Cleaned up control routine for generating constraints files
sfkhalid 2f654cd
Replaced numbers in get_molecule_gain with defined constants and adde…
sfkhalid 1def2e3
Ran make format
sfkhalid 9cab9b2
Reversed some changes during clustering which were degrading the qor …
sfkhalid 7dcdeb8
Added more detailed comments about the --floorplan_split vpr option, …
sfkhalid 9c37438
Corrected a mistake where one of the constraint generating function w…
sfkhalid 5830eaf
Added comment to explain bug fix in do_clustering
sfkhalid f479b76
Added more detail to try_pack_molecule comment
sfkhalid 87ac95e
Improved constraints report routine - every region in PartitionRegion…
sfkhalid 4e761ed
Added more detailed commenting to constraints report header file
sfkhalid 7735f74
Improved error message that is printed when some blocks are not place…
sfkhalid 581c498
Took out loop in do_clustering that counted number of failed molecule…
sfkhalid 5f4685e
Added hash function and equal operator to the Region class. Refactore…
sfkhalid ea133c1
Added constructor to GridTileLookup class which initializes the data …
sfkhalid 7107b45
Ran make format
sfkhalid 7d2abcc
Overhauled the way molecules are pulled from attraction groups during…
sfkhalid 88150c9
Added a fix to the while loop in do_clustering, which counts how many…
sfkhalid 746bf8a
Added return line to make sure that when a molecule is removed it is …
sfkhalid fd636f3
Added a re-try loop that makes the the cluster's attraction group be …
sfkhalid 9607452
Removed unused code
sfkhalid ae969b1
Added routine for filling attraction groups based on overfull regions…
sfkhalid 5bcc77d
Made changes to constraints writer to ensure that it works in all cases
sfkhalid 04a2b28
Added comments and removed unused code
sfkhalid a806ffb
Resolved cluster.cpp merge conflicts
sfkhalid 7375979
Ran make format
sfkhalid fbdd65c
Fixed parameter type in writer setup function
sfkhalid 8f92765
Fixed variable type that was causing compiler errors and warnings
sfkhalid 609d16b
Minor fixes to get rid of compiler warnings
sfkhalid 839d256
Added a routine to be able to constrain half of all blocks with the c…
sfkhalid 07727d1
Added comments and removed unnecessary lines
sfkhalid 7bc8468
Added a helper function to record molecule failures during clustering
sfkhalid b3cf6b8
Edited the comment which explain how to use the floorplan constraints…
sfkhalid 954d60f
Removed outdated code from the constraints generator
sfkhalid 968e82c
Changed packing flow when running with floorplan constraints. Before,…
sfkhalid 7097a30
ran make format
sfkhalid 553c323
Reversed a minor change with adding constrained atoms because it was …
sfkhalid 0567b68
Ran make format
sfkhalid 4dccac4
Initialized att_group_pulls value, as it was causing a memory leak
sfkhalid a449463
Added comments to further clarify how the attraction groups are worki…
sfkhalid 8a5951c
Changed the formation of attraction groups - only create them for ove…
sfkhalid 545ca65
Merge branch 'master' into generate_test_constraints_files
sfkhalid b79865b
Changed routine that gets molecule candidates from the attraction gro…
sfkhalid 553766b
Added more detailed commenting to clustering stage
sfkhalid 9bdc971
Refactored the way attraction group molecules are added when looking …
sfkhalid 9f14605
Changed vpr command line options used for generating test constraints…
sfkhalid f26768a
cleaned up names of routines which check whether floorplan regions ar…
sfkhalid f1506e8
Ran make format
sfkhalid 20097c5
Added a comment to state which pack iteration we are on when we are d…
sfkhalid 39b78b5
Added to-do note for routine that needs to be enhanced in another pul…
sfkhalid ca7d60a
Merge branch 'master' into generate_test_constraints_files
sfkhalid File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.