ivy icon indicating copy to clipboard operation
ivy copied to clipboard

refactor(ivy.linear): remove unneeded list()

Open lucasalavapena opened this issue 1 year ago • 1 comments

PR Description

We had an unneeded list() call, I also got rid of the slice since for two values it seemed unneeded to me at least (plus you are creating a new list with a slice operation)

Related Issue

No related issue

Checklist

  • [ ] Did you add a function?
  • [ ] Did you add the tests?
  • [x] Did you run your tests and are your tests passing?
  • [x] Did pre-commit not fail on any check?
  • [x] Did you follow the steps we provided?

Socials

lucasalavapena avatar Oct 04 '23 09:10 lucasalavapena

Thank you for this PR, here is the CI results:


Failed tests:

This PR introduces the following new failing tests: ivy_tests/test_ivy/test_functional/test_experimental/test_core/test_elementwise.py::test_diff,paddle ivy_tests/test_ivy/test_frontends/test_tensorflow/test_math.py::test_tensorflow_scalar_mul,numpy ivy_tests/test_ivy/test_frontends/test_tensorflow/test_math.py::test_tensorflow_scalar_mul,jax ivy_tests/test_ivy/test_frontends/test_tensorflow/test_math.py::test_tensorflow_scalar_mul,tensorflow ivy_tests/test_ivy/test_frontends/test_tensorflow/test_math.py::test_tensorflow_scalar_mul,torch ivy_tests/test_ivy/test_functional/test_experimental/test_core/test_elementwise.py::test_count_nonzero,torch ivy_tests/test_ivy/test_functional/test_experimental/test_core/test_manipulation.py::test_partial_vec_to_tensor,paddle

github-actions[bot] avatar Oct 04 '23 10:10 github-actions[bot]