Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.26%. Comparing base (031a4f4) to head (0e7430e).

@@           Coverage Diff           @@
##             main    #1019   +/-   ##
=======================================
  Coverage   87.25%   87.26%           
=======================================
  Files           9        9           
  Lines        4928     4930    +2     
=======================================
+ Hits         4300     4302    +2     
  Misses        628      628           

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

Draft for now as I'd like:

• doc: add a page that lists the deprecated and removed APIs #1016 to be merged
• potentially the docstrings MR from @MarcSkovMadsen to be merged to (I'd like to update the docstring of Parameter to mark pickle_default_value as deprecated

I think the problem that pickle_default_value was meant to address would still apply in 2025, in that pickling a value that is dependent on a particular run on a particular system can cause problems when unpickling, but if it's not been supported for some time, may as well worry about this issue next time it comes up in actual practice.

Yes, I agree that it sounds like it might be useful. But in practice, it seems like it was never implemented in Param, and no one reported setting pickle_default_value=False has no effect. So I agree that we can worry about it when this comes up again. In fact, deprecating it might lead someone to come here and ask for that feature to be implemented.

In the meantime, I'd like to push adding a metadata slot to Parameters (#898). I think that would allow users to implement their own special Parameter handling without having to create their own Parameter class (I've had the need a few times to label Parameters differently in a class and abused precedence).