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

Improve datamodel dictionary 'update' handling of recursion #66

Open
dmggh opened this issue Mar 15, 2021 · 0 comments
Open

Improve datamodel dictionary 'update' handling of recursion #66

dmggh opened this issue Mar 15, 2021 · 0 comments

Comments

@dmggh
Copy link
Contributor

dmggh commented Mar 15, 2021

The dictionary function ‘update’ for datamodels does not elegantly handle recursion. A brute force implementation can be used as a workaround, but the goal is to implement an override for the function that will recursively apply ‘update’ to objects within a tree. The use case is to update all the metadata of a file (under the key ‘meta’), which is a dictionary that may contain sub dictionaries (hence the recursion). This problem was discovered while creating unit tests for the dark current step.

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

No branches or pull requests

1 participant