Skip to content

Handle Infinite bus on generator#510

Open
Lisrte wants to merge 6 commits into
mainfrom
infinite_bus_on_generator
Open

Handle Infinite bus on generator#510
Lisrte wants to merge 6 commits into
mainfrom
infinite_bus_on_generator

Conversation

@Lisrte
Copy link
Copy Markdown
Collaborator

@Lisrte Lisrte commented Sep 24, 2025

Please check if the PR fulfills these requirements

  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • A PR or issue has been opened in all impacted repositories (if any)

Does this PR already have an issue describing the problem?

Fixes #483

@Lisrte
Check there is no default model when a bus, line or transformer is in…
4c34cc3 
…stantiated

Fix tests

Signed-off-by: lisrte <laurent.issertial_externe@rte-france.com>
@Lisrte
Remove unused private "checkDefaultModels" method.
75b7147 
Signed-off-by: lisrte <laurent.issertial_externe@rte-france.com>
@Lisrte
Put needMandatoryDynamicModels in AbstractBus instead of its subclass
86d431c 
Signed-off-by: lisrte <laurent.issertial_externe@rte-france.com>
@Lisrte
Remove transformer setting in SetPointInfiniteBusModelXmlTest
c037dd6 
Signed-off-by: lisrte <laurent.issertial_externe@rte-france.com>
@Lisrte
Handle Infinite bus on generator
fc146af 
Signed-off-by: lisrte <laurent.issertial_externe@rte-france.com>
@Lisrte Lisrte self-assigned this Sep 24, 2025
@Lisrte
Fix var mapping
cd28e55 
Revert specific buses filter -> InfiniteBusGenerator does not trigger OmegaRef switch to SetPoint
Fix tests

Signed-off-by: lisrte <laurent.issertial_externe@rte-france.com>
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Sep 29, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
96.7% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@Lisrte Lisrte marked this pull request as ready for review October 3, 2025 08:53
Base automatically changed from mandatory_dynamic_models to main October 21, 2025 11:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@Lisrte Lisrte

Labels

PR: do-not-merge

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Lisrte