Skip to content

Convert Modeldata to Dict#12

Merged
zeryx merged 1 commit into
developfrom
convert-modeldata-to-dict
Nov 18, 2021
Merged

Convert Modeldata to Dict#12
zeryx merged 1 commit into
developfrom
convert-modeldata-to-dict

Conversation

@zeryx
Copy link
Copy Markdown
Contributor

@zeryx zeryx commented Nov 16, 2021

This PR removes the user_data and system_data objects of ModelData, and instead converts the modelData object itself into a dictionary type! This dramatically simplifies interactions in load functions, and allows for easy use by algorithm developers to store data that can later be used at runtime, without having to create a separate object, or reference a special member or method.

Now if you want to export everything what was recorded in userspace, you simply need to call the .data() method, which returns a dict containing the key/value pairs stored.

@zeryx zeryx requested a review from aslisabanci November 16, 2021 18:14
@zeryx zeryx self-assigned this Nov 16, 2021
Copy link
Copy Markdown
Contributor

@aslisabanci aslisabanci left a comment

Choose a reason for hiding this comment

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

i like this simplification :) nothing stood out for me, so 👍

@zeryx zeryx merged commit 9ddd7f8 into develop Nov 18, 2021
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.

2 participants