GUI: replace sta::Delay with float for numeric_limits

[luis201420]

Summary

Replacing sta::Delay with float for numeric_limits, since sta::Delay is not a primitive class, causes the min() and max() functions ​​to both return 0. This solves the current problem of the master regarding the visualization of the clock trees.
I think the bug is integrated into the tool after updating sta, since there are modifications in class Delay.

Type of Change

• Bug fix

Impact

It enables the visualization of clock trees in the GUI.

Verification

• I have verified that the local build succeeds (./etc/Build.sh).
• I have run the relevant tests and they pass.
• My code follows the repository's formatting guidelines.
• I have signed my commits (DCO).

[gemini-code-assist]

Code Review

This pull request updates the initialization of delay and arrival variables in src/gui/src/staGuiInterface.cpp by replacing sta::Delay limits with float limits. The review feedback correctly identifies a logic error in the maximum arrival calculation where std::numeric_limits::min() was used instead of a truly negative value, and suggests using the project-specific sta::INF constant for better consistency and correctness across all modified functions.

[gemini-code-assist]

Using std::numeric_limits::min() here is a logic error. In C++, std::numeric_limits::min() for floating-point types returns the smallest positive normalized value (approximately 1.17e-38), not the most negative value.

If all arrival times are 0.0, std::max(maximum, arrival) will compare 1.17e-38 with 0.0 and keep 1.17e-38, leading to an incorrect result. To initialize a maximum search, you should use a value that is guaranteed to be less than or equal to any possible value, such as -sta::INF or std::numeric_limits::lowest(). Given the existing code style, -sta::INF is the preferred choice.

Suggested change

	sta::Delay maximum = std::numeric_limits<float>::min();
	sta::Delay maximum = -sta::INF;

References

1. Define tunable parameters as named constants instead of using hardcoded magic numbers.

[gemini-code-assist]

Instead of using std::numeric_limits::max(), it is more idiomatic in this codebase to use sta::INF. This constant is already used for similar purposes elsewhere in this file (e.g., lines 1027 and 1204).

Suggested change

	sta::Delay minimum = std::numeric_limits<float>::max();
	sta::Delay minimum = sta::INF;

References

1. Define tunable parameters as named constants instead of using hardcoded magic numbers.

[gemini-code-assist]

Use sta::INF for consistency with other parts of the timing interface implementation.

Suggested change

	sta::Delay minimum = std::numeric_limits<float>::max();
	sta::Delay minimum = sta::INF;

References

1. Define tunable parameters as named constants instead of using hardcoded magic numbers.

[github-actions]

clang-tidy review says "All clean, LGTM! 👍"

[maliberty]

@dsengupta0628 FYI