Hardy Cross Looped Network Solver & O&G Review Update#1849
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR introduces a new Hardy Cross looped pipeline network solver for handling ring mains, parallel pipelines, and complex gathering systems. It also includes a comprehensive update to the Oil & Gas Design Operations Review document with improved documentation of existing functionality.
New Features
1. Hardy Cross Looped Pipeline Network Solver
A complete implementation of the classic Hardy Cross iterative method for solving looped pipe networks.
New Classes
Key Features
Algorithm
The Hardy Cross method calculates flow corrections for each loop:
Where:
Usage Example
Applications
2. Example Notebook
Notebook Contents:
Documentation Updates
3. Oil & Gas Design Operations Review
Major update to NEQSIM_OIL_GAS_DESIGN_OPERATIONS_REVIEW.md:
Updated Sections
New Documentation of Existing Features
Well Performance (IPR) - WellFlow.java:
Reservoir Coupling - ReservoirCouplingExporter.java:
Erosion-Corrosion - ManifoldMechanicalDesignCalculator.java:
Compressor Drivers:
Condition Monitoring:
Flare System - FlareStack.java:
4. Reference Manual Index Update
Updated REFERENCE_MANUAL_INDEX.md:
5. Pipeline Network Enhancement Proposal
Updated PIPELINE_NETWORK_SOLVER_ENHANCEMENT.md:
Test Results
All 12 unit tests pass:
Test Coverage:
testTriangleLoopDetection- Basic loop detectiontestTwoLoopDetection- Multiple independent loopstestTreeNetworkNoLoops- Tree topology (no loops)testRingMainNetworkCreation- Ring main topologytestHardyCrossConvergence- Solver convergencetestNetworkLoopClass- Loop member managementtestSolutionSummary- Output formattestJsonOutput- JSON serializationtestParallelPipes- Parallel pipe loop detectiontestSolverTypeSelection- Solver configurationtestRelaxationFactor- Parameter settingtestOffshoreRingNetwork- Realistic offshore topologyFiles Changed
New Files (5)
Modified Files (3)
Breaking Changes
None. This PR adds new functionality without modifying existing APIs.
Migration Guide
No migration required. Existing code continues to work unchanged.
Future Work
Related Issues
Reviewers
Please review:
Checklist