Deprecate Surface_list - Githubissues

flatsurf / sage-flatsurf

Flat surfaces in Sage

https://flatsurf.github.io/sage-flatsurf/

GNU General Public License v2.0

10 stars 10 forks source link

Deprecate Surface_list #212

Closed saraedum closed 1 year ago

saraedum commented 1 year ago

We deprecate (or probably just remove) Surface_list as it provides no real benefit over Surface_dict. The performance gains are minimal in our timing experiments.

Checklist

[ ] Added an entry in doc/news/.
[ ] Added a test for this change.
[ ] Added new .py files to the documentation in doc/geometry or doc/graphical.

saraedum commented 1 year ago

This is just a very rough draft. We might not end up doing this at all and we certainly won't be as extreme about enforcing labels everywhere. But adding all these deprecation warnings made it easy to check that this is actually feasible and does not really break anything.