Skip to content

Add natural gas pipelines one-shot extraction plugin - #522

Open
bpulluta wants to merge 4 commits into
mainfrom
natural-gas-pipelines-extraction
Open

Add natural gas pipelines one-shot extraction plugin#522
bpulluta wants to merge 4 commits into
mainfrom
natural-gas-pipelines-extraction

Conversation

@bpulluta

@bpulluta bpulluta commented Aug 6, 2026

Copy link
Copy Markdown
Collaborator

Stacked on #521. Adds the natural_gas_pipelines one-shot plugin (schema + config) and registers the extractor.

bpulluta and others added 3 commits August 6, 2026 08:07
Restore geothermal schema/plugin to the latest tuned recall state (v2.2.2 + collection prompt tuning), and add adaptive OpenAI service retry behavior that removes provider-rejected call kwargs (e.g., temperature) and retries safely.

Also adds runtime-focused tests for unsupported-kwarg recovery and non-recoverable error passthrough.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Address PR review feedback: correct the module docstring typo and
bound the number of unsupported top-level kwarg drops per request so
provider rejections cannot trigger unbounded paid retries.
Add a schema-driven one-shot plugin covering natural gas pipelines and
compressor stations, including the extraction schema and plugin config,
and register the extractor so the `natural_gas_pipelines` technology is
available in the plugin registry.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@codecov-commenter

codecov-commenter commented Aug 6, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 73.07692% with 14 lines in your changes missing coverage. Please review.
✅ Project coverage is 64.06%. Comparing base (09173ed) to head (2bbdc1f).

Files with missing lines Patch % Lines
compass/services/openai.py 73.07% 7 Missing and 7 partials ⚠️

❌ Your patch status has failed because the patch coverage (73.07%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #522      +/-   ##
==========================================
+ Coverage   63.82%   64.06%   +0.23%     
==========================================
  Files          78       78              
  Lines        7445     7489      +44     
  Branches      755      766      +11     
==========================================
+ Hits         4752     4798      +46     
+ Misses       2532     2525       -7     
- Partials      161      166       +5     
Flag Coverage Δ
unittests 64.06% <73.07%> (+0.23%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Base automatically changed from geothermal-v222-litellm-runtime-fix to main August 7, 2026 23:35
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.

3 participants