Add vector.random method to FreeModule for consistency with matrix.random

[eslteacher902010]

Updated

This PR moves vector_random(R, n, *args, **kwargs) into the FreeModule_ambient class as a method and NOT a function:

random(self, *args, **kwargs) in src/sage/modules/free_module.py.

—Also, I did my best to eliminate verbosity and follow the project guidelines. I hope this satisfies.

I’m still new to contributing here, so I really appreciate your guidance and patience. Thank you!

Ps. I will probably be stepping away from this project. I feel a little in too deep. Thanks again.

[eslteacher902010]

Small note: The earlier commit mentions a test file, but that was just temporary while checking behavior. The final PR only includes doctests inside vector_random(), as discussed — no standalone test file added.

[whoami730]

• As I said earlier, please reduce or avoid AI slop. It looks like everything including code, commit/PR message etc are AI generated. If you wish to do that, please do a self review of your own PR before proceeding to ask for review.
• Fix PR title and avoid raising new PRs for each change, this loses history.
• We already have random_vector function. Having another vector_random function is redundant. What is instead needed to have is a vector.random method akin to matrix.random method. Consider looking at the implementation of the same.
• Both random_vector and vector.random should ideally share implementation as well.

[user202729]

the function should be accessible as vector.random(), not V.random() or whatever. I recommend trying out the existing functions to get a better understanding on why the proposal is consistent with the existing functions.

Since you're not working on this, I suppose I will just close the pull request.