Skip to content

Conversation

@dependabot
Copy link

@dependabot dependabot bot commented on behalf of github Dec 22, 2025

Bumps third_party/OpenSTA from 14379cb to 5cb01be.

Commits

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Important

Update third_party/OpenSTA submodule to include upstream fixes and improvements.

  • Submodule Update:
    • Updates third_party/OpenSTA submodule from commit 14379cb to 5cb01be.
  • Upstream Changes:
    • Fix for generated clocks with quotes/whitespace.
    • Change in prima3 to use asap7_small.
    • Power debugging improvements.

This description was created by Ellipsis for 7b0c8a0. You can customize this summary. It will automatically update as commits are pushed.

Bumps [third_party/OpenSTA](https://github.com/Silimate/OpenSTA) from `14379cb` to `5cb01be`.
- [Commits](Silimate/OpenSTA@14379cb...5cb01be)

---
updated-dependencies:
- dependency-name: third_party/OpenSTA
  dependency-version: 5cb01be7a9dc574d46625ee22f8b157791f11f7c
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot added dependencies Pull requests that update a dependency file submodules Pull requests that update submodules code labels Dec 22, 2025
Copy link

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Important

Looks good to me! 👍

Reviewed everything up to 7b0c8a0 in 17 seconds. Click for details.
  • Reviewed 7 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 1 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. third_party/OpenSTA:1
  • Draft comment:
    Submodule update: Ensure that the new OpenSTA commit (5cb01be) is fully compatible with our build and interface. Verify that integration tests pass with this dependency bump.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50% None

Workflow ID: wflow_FOgBKXN7CaxFvFNH

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

@greptile-apps
Copy link

greptile-apps bot commented Dec 22, 2025

Greptile Summary

This PR updates the OpenSTA submodule dependency from commit 14379cb to 5cb01be, bringing in several improvements and fixes from the upstream repository.

Key changes in the OpenSTA update:

  • Fix for generated clock handling with quotes and whitespace (addresses issue Update to fix delay weighting #30)
  • Merge from upstream parallaxsw:master branch
  • Power analysis debugging improvements
  • Test configuration updates (prima3 using asap7_small)

Integration analysis:

  • The main Silisizer codebase depends on OpenSTA through CMakeLists.txt (line 125) and uses it for timing analysis
  • This is an automated dependency update via Dependabot
  • The repository has CI tests that will validate the integration
  • No breaking API changes expected based on the commit descriptions

Confidence Score: 4/5

  • This PR is safe to merge with low risk - it's a routine dependency update with targeted bug fixes
  • Score reflects that this is an automated Dependabot submodule update with well-documented changes. The OpenSTA updates include bug fixes (generated clock handling) and upstream synchronization. The main repository has CI tests that validate integration. Minor risk due to dependency updates always having potential for unexpected behavioral changes, but the commit messages indicate focused improvements rather than breaking changes.
  • No files require special attention - this is a standard submodule pointer update

Important Files Changed

Filename Overview
third_party/OpenSTA Submodule updated from 14379cb to 5cb01be, includes fix for generated clock with quote/whitespace handling, upstream merge, and power debug improvements

Sequence Diagram

sequenceDiagram
    participant D as Dependabot
    participant R as Silisizer Repo
    participant O as OpenSTA Submodule
    participant CI as CI Pipeline
    
    D->>R: Detect new OpenSTA commits
    D->>R: Create PR #50 to update submodule
    Note over R,O: Update third_party/OpenSTA<br/>14379cb → 5cb01be
    R->>O: Pull in fixes:<br/>- Generated clock quote/whitespace fix<br/>- Upstream merge from parallaxsw<br/>- Power debug improvements
    CI->>R: Checkout with submodules
    CI->>R: Build Silisizer with updated OpenSTA
    CI->>R: Run integration tests (picorv32, etc.)
    CI-->>D: Report build & test status
Loading

@dependabot @github
Copy link
Author

dependabot bot commented on behalf of github Dec 25, 2025

Looks like third_party/OpenSTA is up-to-date now, so this is no longer needed.

@dependabot dependabot bot closed this Dec 25, 2025
@dependabot dependabot bot deleted the dependabot/submodules/third_party/OpenSTA-5cb01be branch December 25, 2025 08:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file submodules Pull requests that update submodules code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant