[#14602] laplace_transform() does not validate negative s_value despite documented non-negative requirement

[jgohel9902]

Feature description

Description

The laplace_transform() function documentation says that s_value must be non-negative, but the implementation currently does not validate this condition.

Because of this, users can pass a negative s_value, even though the function docstring states that only real-valued, non-negative Laplace parameters are supported.

File

maths/laplace_transformation.py

Relevant lines

Around line 17–18, the docstring says:

This implementation supports only real-valued, non-negative Laplace
parameters `s`.

[psyduck1203]

Hi @jgohel9902, This issue looked very interesting to me and I wanted to contribute to it but I'm not able to locate maths/laplace_transformation.py file.

[TTAAAN]

Hi @jgohel9902, This issue looked very interesting to me and I wanted to contribute to it but I'm not able to locate maths/laplace_transformation.py file.

I'm also not able to locate the file. I've performed search in the codebase and still not able to locate the file name or locate the line that says.

This implementation supports only real-valued, non-negative Laplace parameters `s`.

[tempoo04]

Same here. I think the file is not yet tracked by git.

[TTAAAN]

I've ran additional cli

git log --all -- maths/laplace_transformation.py

No output

git log --all --name-only --pretty=format: | sort -u | grep -i -F "laplace_transformation"

No output

git log --all --name-only --pretty=format: | sort -u | grep -i -F "transformation" 

Output:
linear_algebra/src/transformations_2d.py
machine_learning/data_transformations.py
physics/lorentz_transformation_four_vector.py
physics/lorenz_transformation_four_vector.py


git log --all --name-only --pretty=format: | sort -u | grep -i -F "lap"    

Output:
digital_image_processing/filters/laplacian_filter.py

No result is found.

Could you clarify if this issue is coming from a different branch, fork, or older version of the code?

[TTAAAN]

Just to understand the context better. Was this reported based on what you found manually, or generated by a tool like LLM?