-
Notifications
You must be signed in to change notification settings - Fork 84
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix #698 and #699 #706
Fix #698 and #699 #706
Conversation
dimitri-yatsenko
commented
Nov 19, 2019
•
edited
Loading
edited
- Fix help does not work on tables #698 -- Help now works for DataJoint classes
- Fix Append table definition to docstring #699 -- table definition is appended to doc string
tests/test_relation.py
Outdated
@@ -36,6 +36,12 @@ def setup_class(cls): | |||
cls.img = schema.Image() | |||
cls.trash = schema.UberTrash() | |||
|
|||
def test_class_help(self): | |||
help(schema.TTest) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
would it make sense to also e.g.:
assert schema.TTest.definition in schema.TTest.__doc__
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done. Also moved into test_declare
tests/test_relation.py
Outdated
def test_class_help(self): | ||
help(schema.TTest) | ||
|
||
def test_instance_help(self): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
as before r.e. testing for definition in doc.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done. Also moved into test_declare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you update CHANGELOG.md
and docs-parts/intro/Releases_lang1.rst
based on these updates?
@dimitri-yatsenko Can you also update |
datajoint/blob.py
Outdated
@@ -253,6 +253,9 @@ def pack_recarray(self, array): | |||
def read_sparse_array(self): | |||
raise DataJointError('datajoint-python does not yet support sparse arrays. Issue (#590)') | |||
|
|||
def read_scalar(selfs): | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks like a typo... Does not raise/return anything?