Skip to content

use faster algorithm in lir_basis #37

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

PikasXYZ
Copy link

The implementation includes optional parameters, such as target_ratio, ratio_tolerance, and target_center_coord. However, these parameters are hardcoded in the source code and can only be modified by directly altering the code, as no external interface or configuration mechanism is currently provided for dynamic adjustment.

👇different target_center_coord (coordinate of blue dot) got different rectangle (red frame) within fixed target_ratio (width/height) in the grid (black area)
image

PikasXYZ and others added 2 commits January 23, 2025 23:05
The implementation includes optional parameters, such as target_ratio=0.00, ratio_tolerance=0.01, and target_center_coord=None. However, these parameters are hardcoded in the source code and can only be modified by directly altering the code, as no external interface or configuration mechanism is currently provided for dynamic adjustment.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant