Renaming backend methods related to vector and matrix norms

[renatomello]

Closes #1518

Checklist:

• Reviewers confirm new code works as expected.
• Tests are passing.
• Coverage does not decrease.
• Documentation is updated.

[renatomello]

@MatteoRobbiati @BrunoLiegiBastonLiegi do you know what is this error in the tests regarding importing tensorflow from qiboml?

https://github.com/qiboteam/qibo/actions/runs/11794483631/job/32852177151?pr=1520

[BrunoLiegiBastonLiegi]

Mmmh weird, but the tests under tensorflow are still running?

[MatteoRobbiati]

I think what is strange is that it didn't fail before: tensorflow is no more a dependency, thus shouldn't this line always set tensorflow to False?

[MatteoRobbiati]

Here is the problem! This modification has to be removed. We need "tensorflow": False

Suggested change

	"qiboml": {"tensorflow": True, "pytorch": True},
	"qiboml": {"tensorflow": False, "pytorch": True},

[renatomello]

It was like that before and it was passing on Windows and Mac but failing on Ubuntu.

…

On Tue, 10 Dec 2024 at 19:19, Matteo Robbiati ***@***.***> wrote: ***@***.**** commented on this pull request. ------------------------------ In tests/test_backends.py <#1520 (comment)>: > @@ -123,7 +123,7 @@ def test_list_available_backends(): "qibolab": False, "qibo-cloud-backends": False, "qibotn": {"cutensornet": False, "qutensornet": True}, - "qiboml": {"tensorflow": False, "pytorch": True}, + "qiboml": {"tensorflow": True, "pytorch": True}, Here is the problem! This modification has to be removed. We need "tensorflow": False ⬇️ Suggested change - "qiboml": {"tensorflow": True, "pytorch": True}, + "qiboml": {"tensorflow": False, "pytorch": True}, — Reply to this email directly, view it on GitHub <#1520 (review)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABH5QVRNRFRIX73PSBR5LJT2E4BBZAVCNFSM6AAAAABRTIBE56VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDIOJSGY3TQMZUHE> . You are receiving this because you were assigned.Message ID: ***@***.***>

[MatteoRobbiati]

It was like that before and it was passing on Windows and Mac but failing on Ubuntu.

I think because, for some reason, tensorflow is still a dependence here: see pyproject.toml. It has to be removed.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.67%. Comparing base (ce6dc20) to head (ce0592f).
Report is 28 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #1520   +/-   ##
=======================================
  Coverage   99.67%   99.67%           
=======================================
  Files          76       76           
  Lines       11234    11234           
=======================================
  Hits        11198    11198           
  Misses         36       36           

Flag	Coverage Δ
unittests	99.67% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[MatteoRobbiati]

Thanks!

[renatomello]

Reminder that this PR needs merge coordination with the equivalent qibojit PR.

[MatteoRobbiati]

Reminder that this PR needs merge coordination with the equivalent qibojit PR.

Do you need reviewers in Qibojit? Feel free to add me in case

[renatomello]

Reminder that this PR needs merge coordination with the equivalent qibojit PR.

Not needed after all, so I'm merging this PR.