From 562aeddf7efa09e0930f6b3db5bc665fe6e321e4 Mon Sep 17 00:00:00 2001 From: mmatera Date: Sun, 6 Nov 2022 16:51:44 -0300 Subject: [PATCH 1/2] fix cleanup after loading autoload definitions --- mathics/core/definitions.py | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/mathics/core/definitions.py b/mathics/core/definitions.py index def6b1a93..3a6575c07 100644 --- a/mathics/core/definitions.py +++ b/mathics/core/definitions.py @@ -169,7 +169,13 @@ def __init__( if name.startswith("Global`"): raise ValueError("autoload defined %s." % name) - self.builtin.update(self.user) + # Move symbols defined in autoload modules + # to Builtin definitions. + # This seems important for Export and Import... + # TODO: check why + for name in self.user: + self.builtin[name] = self.get_definition(name) + self.user = {} self.clear_cache() From d95f5019cd059eeb9f2ceceb5f88eaa5b988ebe1 Mon Sep 17 00:00:00 2001 From: mmatera Date: Mon, 7 Nov 2022 09:38:39 -0300 Subject: [PATCH 2/2] improving comment --- mathics/core/definitions.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/mathics/core/definitions.py b/mathics/core/definitions.py index 3a6575c07..81a156b0b 100644 --- a/mathics/core/definitions.py +++ b/mathics/core/definitions.py @@ -171,8 +171,9 @@ def __init__( # Move symbols defined in autoload modules # to Builtin definitions. - # This seems important for Export and Import... - # TODO: check why + # This is important to avoid that reset_user_definitions + # getting rid of the autoloaded definitions (including) + # the needed by Import/Export. for name in self.user: self.builtin[name] = self.get_definition(name)