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

Create new class for result of simdex.get() #14

Open
saroele opened this issue May 16, 2012 · 0 comments
Open

Create new class for result of simdex.get() #14

saroele opened this issue May 16, 2012 · 0 comments
Assignees

Comments

@saroele
Copy link
Owner

saroele commented May 16, 2012

I think we could benefit from an additional class. Now, simdex.get() returns a dictionary {sid:value}. I find it cumbersome to do operations on this dictionary, like plotting all of the values with their legend etc. Of course, it is possible, but if I made an additional class of this, I could also implement some very useful features.

Suppose we call this class Result. The core of a result would be the {sid:value} dictionary. Maybe it's even a better idea to extend Result from the dictionary class in order to inherit some of the methods?

Some needed functionality:

  • Result.values(): values in SORTED order, according to SID
  • basic operators (+, -, *, /)
  • integration
  • aggregation? Or this could also be in the simdex.get() method
  • plot: should replace the simdex.plot() , internally (simdex.plot will use result.plot)
  • therefore, I need two more things in my Result class: the Time, and the identifiers (legends for plotting).

It seems like the most easy way will be to make my own class an have a few dictionaries as attributes

@ghost ghost assigned saroele May 16, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant