Deprecation warnings are shown even if the field is not accessed in the task

[poautran]

In GitLab by @loichuder on Feb 12, 2026, 17:38 GMT+1:

When marking a field as deprecated using Pydantic deprecated mechanism, a warning is always printed even if the field is not accessed.

from pydantic import Field

from ewoks import BaseInputModel, Task


class Inputs(BaseInputModel):
    a: int
    b: int = Field(deprecated="DO NOT USE B", default=2)


class TheTask(Task, input_model=Inputs, output_names=["result"]):
    def run(self):
        self.outputs.result = self.inputs.a


task = TheTask(inputs={"a": 2})
task.execute()

Migrated from GitLab: https://gitlab.esrf.fr/workflow/ewoks/ewokscore/-/issues/86

[poautran]

In GitLab by @woutdenolf on Feb 12, 2026, 20:05 GMT+1:

For reference: the warning comes from getting, not setting

python -W error test.py 

Traceback (most recent call last):
  File "/home/denolf/.local/wdnbashutils/virtualenvs/ewoks/lib/python3.12/site-packages/ewokscore/task.py", line 688, in execute
    self._validate_inputs()
  File "/home/denolf/.local/wdnbashutils/virtualenvs/ewoks/lib/python3.12/site-packages/ewokscore/task.py", line 161, in _validate_inputs
    self.__inputs[name].value = getattr(model, name)
                                ^^^^^^^^^^^^^^^^^^^^
  File "/home/denolf/.local/wdnbashutils/virtualenvs/ewoks/lib/python3.12/site-packages/pydantic/_internal/_model_construction.py", line 747, in __get__
    warnings.warn(self.msg, DeprecationWarning, stacklevel=2)
DeprecationWarning: DO NOT USE B

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/denolf/projects/blissoda/test.py", line 19, in <module>
    task.execute()
  File "/home/denolf/.local/wdnbashutils/virtualenvs/ewoks/lib/python3.12/site-packages/ewokscore/task.py", line 700, in execute
    self._raise_task_error(
  File "/home/denolf/.local/wdnbashutils/virtualenvs/ewoks/lib/python3.12/site-packages/ewokscore/task.py", line 651, in _raise_task_error
    raise exc_class(err_msg) from cause
RuntimeError: Execution failed for ewoks task (id: None, task: '__main__.TheTask'): DO NOT USE B

[poautran]

In GitLab by @woutdenolf on Feb 12, 2026, 20:08 GMT+1:

The ideal solution would be to support deprecation in VariableContainer.

[poautran]

In GitLab by @woutdenolf on Feb 12, 2026, 20:11 GMT+1:

Of course we could go even further and merge VariableContainer with pydantic like redis-om does.

As a reminder, the functionality VariableContainer provides is caching variables with a dynamically resolved URL based on hashing. It allows variables to be in-memory or provided as URL's (of different types).

[woutdenolf]

Of course we could ignore this

with warnings.catch_warnings():
    warnings.simplefilter("ignore", DeprecationWarning)
    self.__inputs[name].value = getattr(model, name)

but then using deprecated is pointless. Instead we could capture this warning and then let the variable container produce the a deprecation warning when getting or setting values.

Note that other functions like get_input_values() should not provide warnings either when the input value is MISSING_DATA.

[maxenceprog]

Implem could be something like that ?

def _validate_outputs(self) -> None:
       ...
        for name, field_info  in self._OUTPUT_MODEL.model_fields.items():
              with warnings.catch_warnings():
                  warnings.simplefilter("ignore", DeprecationWarning)
                  self.__inputs[name].value = getattr(model, name)
              if  field_info.deprecated:
                  self.__outputs[name].set_deprecated(field_info.deprecated)
                  

class Variable):
    def set_deprecated(deprecated):
        ...
    def is deprecated():
      ....
    @property
    def value(self):
       if deprecated -> do something

@loichuder @woutdenolf ?

[loichuder]

Looks good to me. Wait for @woutdenolf input first, though.

[woutdenolf]

Good idea! Maybe a simple Variable.deprecated attribute, no need for getter and setter.

[maxenceprog]

Ok thank you !