Skip to content

FEAT: DC settings #1348

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 31 commits into from
Jul 11, 2025
Merged

FEAT: DC settings #1348

merged 31 commits into from
Jul 11, 2025

Conversation

gkorompi
Copy link
Collaborator

@gkorompi gkorompi commented Jul 8, 2025

No description provided.

@github-actions github-actions bot added the testing Anything related to testing label Jul 8, 2025
@codecov-commenter
Copy link

codecov-commenter commented Jul 8, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 76.28%. Comparing base (d256bb9) to head (d290b03).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1348      +/-   ##
==========================================
+ Coverage   76.17%   76.28%   +0.10%     
==========================================
  Files         162      162              
  Lines       23806    23812       +6     
==========================================
+ Hits        18135    18165      +30     
+ Misses       5671     5647      -24     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Collaborator

@hui-zhou-a hui-zhou-a left a comment

Choose a reason for hiding this comment

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

dc_advanced_settings It is already in there.

@gkorompi
Copy link
Collaborator Author

gkorompi commented Jul 8, 2025

dc_advanced_settings It is already in there.

Hello @hui-zhou-a i think this is in SiwaveDCSimulationSetup class, we need it in SiwaveSimulationSetup too

@gkorompi gkorompi requested a review from hui-zhou-a July 8, 2025 15:28
@hui-zhou-a
Copy link
Collaborator

dc_advanced_settings It is already in there.

Hello @hui-zhou-a i think this is in SiwaveDCSimulationSetup class, we need it in SiwaveSimulationSetup too

Can you explain why you need

dc_advanced_settings It is already in there.

Hello @hui-zhou-a i think this is in SiwaveDCSimulationSetup class, we need it in SiwaveSimulationSetup too

dc_advanced_settings It is already in there.

Hello @hui-zhou-a i think this is in SiwaveDCSimulationSetup class, we need it in SiwaveSimulationSetup too

Can you explain why you need it under SiwaveSimulationSetup?

@gkorompi gkorompi changed the title FEATURE: DC settings FEAT: DC settings Jul 9, 2025
@gkorompi
Copy link
Collaborator Author

gkorompi commented Jul 9, 2025

dc_advanced_settings It is already in there.

Hello @hui-zhou-a i think this is in SiwaveDCSimulationSetup class, we need it in SiwaveSimulationSetup too

Can you explain why you need

dc_advanced_settings It is already in there.

Hello @hui-zhou-a i think this is in SiwaveDCSimulationSetup class, we need it in SiwaveSimulationSetup too

dc_advanced_settings It is already in there.

Hello @hui-zhou-a i think this is in SiwaveDCSimulationSetup class, we need it in SiwaveSimulationSetup too

Can you explain why you need it under SiwaveSimulationSetup?

Those options exist in syz setups too:
image

@gkorompi gkorompi enabled auto-merge (squash) July 9, 2025 08:20
svandenb-dev
svandenb-dev previously approved these changes Jul 10, 2025
Copy link
Collaborator

@svandenb-dev svandenb-dev left a comment

Choose a reason for hiding this comment

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

LGTM

@hui-zhou-a
Copy link
Collaborator

@svandenb-dev @gkorompi revert to GK's commit as changing code structure is not that simple. Lots of test are failing.

@hui-zhou-a hui-zhou-a requested a review from svandenb-dev July 10, 2025 11:42
Copy link
Collaborator

@svandenb-dev svandenb-dev left a comment

Choose a reason for hiding this comment

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

LGTM

@svandenb-dev svandenb-dev self-requested a review July 10, 2025 15:22
Copy link
Collaborator

@svandenb-dev svandenb-dev left a comment

Choose a reason for hiding this comment

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

LGTM

@svandenb-dev svandenb-dev self-requested a review July 11, 2025 09:56
Copy link
Collaborator

@svandenb-dev svandenb-dev left a comment

Choose a reason for hiding this comment

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

LGTM

@gkorompi gkorompi merged commit 473507a into main Jul 11, 2025
22 checks passed
@gkorompi gkorompi deleted the dc_settings branch July 11, 2025 09:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing Anything related to testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants