Lokasi ngalangkungan proxy:   [ UP ]  
[Ngawartoskeun bug]   [Panyetelan cookie]                
Skip to content

Highlight "distinguished" classes in the ERD #378#557

Merged
dimitri-yatsenko merged 10 commits into
datajoint:masterfrom
kimehta:master
Jan 22, 2019
Merged

Highlight "distinguished" classes in the ERD #378#557
dimitri-yatsenko merged 10 commits into
datajoint:masterfrom
kimehta:master

Conversation

@kimehta

@kimehta kimehta commented Jan 14, 2019

Copy link
Copy Markdown
Contributor

still need to update docs and test

Comment thread datajoint/heading.py Outdated
Comment thread datajoint/heading.py Outdated
code for distinguished is now limited to erd and contained within erd.py
table heading no longer has is_distinguished property.
table attributes in heading no longer have is_foreign_key property.
table heading no longer has is_distinguished property.
table attributes in heading no longer have is_foreign_key property.
table heading no longer has is_distinguished property.
table attributes in heading no longer have is_foreign_key property.
nx.drawing.nx_pydot.to_pydot(graph) appears to casting (at least bool) attributes to str.
see line #132 and #275
Comment thread datajoint/erd.py Outdated
Comment thread datajoint/erd.py Outdated
Comment thread datajoint/erd.py
Comment thread datajoint/erd.py Outdated
@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.03%) to 90.848% when pulling d4254e7 on kimehta:master into 85aa404 on datajoint:master.

1 similar comment
@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.03%) to 90.848% when pulling d4254e7 on kimehta:master into 85aa404 on datajoint:master.

@coveralls

coveralls commented Jan 17, 2019

Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.02%) to 90.833% when pulling 40280ad on kimehta:master into 85aa404 on datajoint:master.

Comment thread datajoint/erd.py Outdated
Comment thread datajoint/erd.py Outdated

@dimitri-yatsenko dimitri-yatsenko left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

minor about ready to merge

Comment thread datajoint/erd.py Outdated
Comment thread datajoint/erd.py Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants