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.

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

pass neurodata_type_def refinements into ObjectMapper.build #633

Open
4 of 11 tasks
ajtritt opened this issue Sep 27, 2018 · 0 comments
Open
4 of 11 tasks

pass neurodata_type_def refinements into ObjectMapper.build #633

ajtritt opened this issue Sep 27, 2018 · 0 comments
Assignees
Labels
help wanted: deep dive request for community contributions that will involve many parts of the code base priority: low alternative solution already working and/or relevant to only specific user(s)
Milestone

Comments

@ajtritt
Copy link
Member

ajtritt commented Sep 27, 2018

Problem/Use Case

When using an existing neurodata_type (using neurodata_type_inc), you are allowed to further refine the neurodata_type specification without declaring a new neurodata_type (using neurodata_type_def).

Currently, the TypeMap and ObjectMappers do not know about these further refinements when building the Builder representation.

  • modify BuildManager.build, TypeMap.build to accept these further refinements.
  • modify ObjectMapper.build to accept these further refinements
    • modify ObjectMapper.__add_groups
    • modify ObjectMapper.__add_datasets
    • modify ObjectMapper.__add_containers
    • modify ObjectMapper.__add_attributes
  • ensure the logic recurses

Checklist

  • Have you ensured the feature or change was not already reported ?
  • Have you included a brief and descriptive title?
  • Have you included a clear description of the problem you are trying to solve?
  • Have you included a minimal code snippet that reproduces the issue you are encountering?
@ajtritt ajtritt added the help wanted: deep dive request for community contributions that will involve many parts of the code base label Sep 27, 2018
@ajtritt ajtritt added this to the NWB 2.x milestone Sep 27, 2018
@stephprince stephprince added the priority: low alternative solution already working and/or relevant to only specific user(s) label Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted: deep dive request for community contributions that will involve many parts of the code base priority: low alternative solution already working and/or relevant to only specific user(s)
Projects
None yet
Development

No branches or pull requests

4 participants