deprecate import of all_max_clique, max_clique and clique_number in global namespace

[dcoudert]

As part of #25383, deprecate import of all_max_clique, max_clique and clique_number into global namespace. These methods are already methods of Graph.

Component: graph theory

Author: David Coudert

Branch/Commit: ab7d219

Reviewer: Frédéric Chapoton

Issue created by migration from https://trac.sagemath.org/ticket/26200

[dcoudert]

comment:1

I don't understand the need for the following imports in src/sage/graphs/all.py

import sage.graphs.weakly_chordal
import sage.graphs.lovasz_theta
import sage.graphs.partial_cube

I let them here for now.

---

New commits:

f96220b

trac #26200: remove methods from cliquer from global namespace

[dcoudert]

Branch: u/dcoudert/26200_cliquer_and_namespace

[dcoudert]

Commit: f96220b

[sagetrac-git]

Changed commit from f96220b to aa37776

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

8d455d9	trac #26200: fix doctests in cliquer
aa37776	trac #26200: use lazy_import and add deprecation warning instead of brutal removal

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

ab7d219

trac #26200: correct test message

[sagetrac-git]

Changed commit from aa37776 to ab7d219

[dcoudert]

comment:4

Deprecation is better than brutal removal and consistent with #26196.

[fchapoton]

comment:6

ok, good to go

[fchapoton]

Reviewer: Frédéric Chapoton

[bryangingechen]

comment:7

For the other imports you asked about above; it seems they were all added when those modules were written, without any particular comment.

• weakly_chordal: #13073 comment:4 patch
• lovasz_theta: implement Lovasz theta-function for graphs #18830, commit
• partial_cube: #19985 comment:33 commit

Was there some convention back then to add new modules to the global namespace?

[vbraun]

Changed branch from u/dcoudert/26200_cliquer_and_namespace to ab7d219