Update jac_lin_coord methods

[arnavk23]

The linear Jacobian part is independent of x, so passing it became unnecessary.

• jac_lin_coord(nlp, x) → jac_lin_coord(nlp)
• jac_lin_coord!(nlp, x, vals) → jac_lin_coord!(nlp, vals)

Closes #404

[amontoison]

@arnavk23 It is a breaking change if we drop the signature with x, we should still support the old API and make it deprecated.

[tmigot]

Thanks @arnavk23 for the PR.
Can you also adapt jac_lin_op and jac_lin_op! https://github.com/arnavk23/NLPModels.jl/blob/504a3c29574998b892d7f598283db466a86e391e/src/nlp/api.jl#L737

[arnavk23]

@tmigot All test have passed except the breakage/upload one.

[tmigot]

Thanks @arnavk23 for these changes. I gave some thoughts about @amontoison 's comment, and it's a good idea to deprecate these functions instead of removing them directly.
Using the macro @deprecate when somebody uses the "old" variant it should receive a warning saying that the function is deprecated and point to the variant.

[tmigot]

Thanks @arnavk23 for the changes. I made some comments.
I think we also miss the functions jprod and jtprod

[tmigot]

We also need to remove the x for the function jprod_lin, jprod_lin!, jtprod_lin and jtprod_lin!

[tmigot]

Thanks @arnavk23 !