Skip to content

Add deprecation warnings for parameter sweep wrapper functions#21

Closed
k1nshuk wants to merge 3 commits into
watertap-org:mainfrom
k1nshuk:dpr_ps_fn
Closed

Add deprecation warnings for parameter sweep wrapper functions#21
k1nshuk wants to merge 3 commits into
watertap-org:mainfrom
k1nshuk:dpr_ps_fn

Conversation

@k1nshuk

@k1nshuk k1nshuk commented Dec 26, 2024

Copy link
Copy Markdown
Collaborator

This PR:

  1. Adds deprecation warnings for parameter sweep wrapper functions as they will be removed in future versions.
  2. Updates line breaks in existing deprecation warnings to prevent extra spaces in actual printed outputs.

@k1nshuk k1nshuk self-assigned this Dec 26, 2024
@codecov

codecov Bot commented Dec 26, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.30%. Comparing base (518572f) to head (664accd).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #21   +/-   ##
=======================================
  Coverage   94.30%   94.30%           
=======================================
  Files          38       38           
  Lines        3686     3690    +4     
=======================================
+ Hits         3476     3480    +4     
  Misses        210      210           

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

@ksbeattie

Copy link
Copy Markdown
Contributor

@k1nshuk any news on this?

@ksbeattie

Copy link
Copy Markdown
Contributor

No current plans to deprecate these functions

@ksbeattie ksbeattie closed this Mar 19, 2026
@ksbeattie ksbeattie added the Priority:Normal Normal Priority Issue or PR label Mar 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Priority:Normal Normal Priority Issue or PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants