René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Use of imported schema object","articleBody":"Currently, when you create a schema object in one module (i.e. Python file) using `locals()` and then import that schema object and use it to decorate a DataJoint user relation class, the lookup of foreign key names does not work. For example:\r\n\r\n`module1.py`\r\n```python\r\nimport datajoint as dj\r\n\r\nschema = dj.schema('my_schema', locals())\r\n\r\n@schema\r\nclass A(dj.Manual):\r\n    definiton = \"\"\"\r\n    id:  int    # id\r\n    \"\"\"\r\n\r\n# this works just fine\r\n@schema\r\nclass B(dj.Manual):\r\n    definition  = \"\"\"\r\n    -\u003e A\r\n    \"\"\"\r\n```\r\n\r\n`module2.py`\r\n```python\r\nimport datajoint as dj\r\nfrom .module1 import schema   # importing schema object\r\n\r\n# this still works because no foreign key reference made\r\n@schema\r\nclass C(dj.Manual):\r\n    definition = \"\"\"\r\n    id: int    # id\r\n    \"\"\"\r\n\r\n# however, this doesn't work because imported schema object cannot look at the *current local* context\r\n@schema\r\nclass D(dj.Manual):\r\n    definition = \"\"\"\r\n    -\u003e C\r\n    \"\"\"\r\n```\r\n\r\nAlthough this behavior is completely consistent and thus is expected based on how `locals()` work, the result is certainly counter-intuitive and may be error prone - especially because some cases of using imported schema object (e.g. when defining class `C` in the above example) actually works.\r\n\r\nIn conjunction to fixing #368 as aimed by #412 , I suggest that we also let `dj.schema('schema_name')` to be aware of the local context it is currently present **regardless of where it was defiend**.","author":{"url":"https://github.com/eywalker","@type":"Person","name":"eywalker"},"datePublished":"2017-12-14T02:19:09.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":1},"url":"https://github.com/413/datajoint-python/issues/413"}