Skip to content
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.

Sign up for GitHub

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

Jump to bottom

remove get_build_manager dependency from validator #203

Closed
ajtritt opened this issue Nov 7, 2017 · 2 comments
Closed

remove get_build_manager dependency from validator #203

ajtritt opened this issue Nov 7, 2017 · 2 comments

Comments

@ajtritt
Copy link
Member

ajtritt commented Nov 7, 2017

pynwb/validate.py still calls get_build_manager. This code needs to be updated to call pynwb.get_manager
@oruebel
Copy link
Contributor

oruebel commented Nov 7, 2017

Pull request #187 I believe fixes that part

jcfr added a commit that referenced this issue Nov 7, 2017
@jcfr
pynwb: Update validator to use get_manager() function. See #203
cbaad41
jcfr pushed a commit that referenced this issue Nov 7, 2017
@oruebel @jcfr
pynwb: Update validator to use get_manager() function. See #203
f5d26ee
jcfr added a commit that referenced this issue Nov 7, 2017
@jcfr
Merge pull request #204 from NeurodataWithoutBorders/203-update-valid…
f2d12e6 
…ator

pynwb: Update validator to use get_manager() function. See #203
@jcfr
Copy link
Collaborator

jcfr commented Nov 7, 2017

Closing, This is fixed in f2d12e6

@jcfr jcfr closed this as completed Nov 7, 2017
@tbenst tbenst mentioned this issue Dec 10, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jcfr @ajtritt @oruebel