New Parent method _convert_method_map

[jdemeyer]

Factor out the logic to determine a NamedConvertMap in _generic_convert_map. This way, it can be used by _coerce_map_from_ methods.

Also clean up the logic in discover_coerce_map_from a bit and remove the unused force_use flag from DefaultConvertMap and NamedConvertMap.

CC: @videlec @tscrim

Component: coercion

Author: Jeroen Demeyer

Branch/Commit: 18d9877

Reviewer: Travis Scrimshaw

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

[jdemeyer]

Branch: u/jdemeyer/new_parent_method__convert_method_map

[jdemeyer]

Commit: cf2f643

[jdemeyer]

New commits:

cf2f643

New Parent method _convert_method_map

[jdemeyer]

Dependencies: #24372

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

375e40f	Map._extra_slots(): do not pass dict
6f46f29	New Parent method _convert_method_map

[sagetrac-git]

Changed commit from cf2f643 to 6f46f29

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

5c2f4f5

New Parent method _convert_method_map

[sagetrac-git]

Changed commit from 6f46f29 to 5c2f4f5

[jdemeyer]

Changed dependencies from #24372 to none

[jdemeyer]

comment:14

Ping? This is a dependency of #24371

[tscrim]

comment:15

Incredibly nitpicking thing:

+from . coerce_maps cimport (NamedConvertMap, DefaultConvertMap,
+        DefaultConvertMap_unique, CallableConvertMap)

The space after the . looks strange, but I guess that is not a syntax error (a bit to my surprise). Feel free to set a positive review if you don't care or on my behalf if tweaked.

[tscrim]

Reviewer: Travis Scrimshaw

[sagetrac-git]

Changed commit from 5c2f4f5 to 18d9877

[sagetrac-git]

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

18d9877

Fix cimport

[jdemeyer]

comment:17

Thanks!

[vbraun]

Changed branch from u/jdemeyer/new_parent_method__convert_method_map to 18d9877