Minor cleanup/documentation cleanup of schema and models#887
Minor cleanup/documentation cleanup of schema and models#887VisLab merged 4 commits intohed-standard:developfrom
Conversation
|
spec_tests failure is expected, pending PR approval on hed-specification |
| Returns: | ||
| list: A list of issues. Each issue is a dictionary. | ||
| """ | ||
| """Check if the single tag is a partnered schema tag""" |
There was a problem hiding this comment.
Why is the documentation of the Parameters and Returns being removed?
There was a problem hiding this comment.
Because it was the exact same on all of those functions, there's a template at the top of the file(I just fixed a mistake in it). Users would never be calling these functions directly.
I'm happy to add the parameter/return value doc strings back if desired, they just didn't seem helpful to me.
There was a problem hiding this comment.
Please put them back in. When you jump to a particular function from a link in the docs you don't see the template.
VisLab
left a comment
VisLab
left a comment
There was a problem hiding this comment.
Mostly looks good -- but the documentation of Parameters and Returns were removed in a number of places --- I didn't think this documentation should be removed.
|
I went ahead and added disabling owl related code to this PR |
2 participants
No description provided.