Skip to content

chore: update opensta to add sdc collection support#55

Merged
akashlevy merged 1 commit intoSilimate:mainfrom
donn:sta_collections
Apr 15, 2026
Merged

chore: update opensta to add sdc collection support#55
akashlevy merged 1 commit intoSilimate:mainfrom
donn:sta_collections

Conversation

@donn
Copy link
Copy Markdown
Contributor

@donn donn commented Apr 8, 2026

No description provided.

@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented Apr 8, 2026

Greptile Summary

This PR bumps the third_party/OpenSTA submodule (Silimate fork) from commit 56337c2 to 66317a7 to pull in SDC collection support. No source files in the main repository are modified.

Confidence Score: 5/5

Safe to merge — this is a dependency-only submodule bump with no changes to main repository source code.

The entire change is a single submodule pointer update. No logic, build configuration, or source files in the main repo were modified. The risk is limited to the new OpenSTA commit introducing a regression, which is mitigated by the scoped nature of 'SDC collection support' as an additive feature.

No files require special attention.

Vulnerabilities

No security concerns identified. This is a submodule pointer update with no changes to authentication, data handling, or network-facing code in the main repository.

Important Files Changed

Filename Overview
third_party/OpenSTA Submodule pointer bumped from 56337c2 to 66317a7 to add SDC collection support; no main-repo source changes.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[silisizer-bin] --> B[silisizer library]
    B --> C[OpenSTA @ 66317a7\n+SDC collection support]
    B --> D[backward-cpp]
    C --> E[TCL runtime]
    B --> E
Loading

Reviews (1): Last reviewed commit: "chore: update opensta to add sdc collect..." | Re-trigger Greptile

@donn donn force-pushed the sta_collections branch from 2a1319e to f184afd Compare April 8, 2026 18:04
Comment thread third_party/OpenSTA
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should use OpenSTA main and have all tests passing

@donn donn force-pushed the sta_collections branch from f184afd to 12f2d72 Compare April 15, 2026 17:50
@donn donn force-pushed the sta_collections branch from 12f2d72 to cae00c3 Compare April 15, 2026 17:57
@akashlevy akashlevy merged commit d97534a into Silimate:main Apr 15, 2026
2 checks passed
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.

2 participants