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

Write design document for Device object #50

Closed
thesamovar opened this issue Jun 11, 2013 · 5 comments
Closed

Write design document for Device object #50

thesamovar opened this issue Jun 11, 2013 · 5 comments

Comments

@thesamovar
Copy link
Member

See title and issue #49.

@ghost ghost assigned thesamovar Jun 11, 2013
@thesamovar
Copy link
Member Author

I started writing this document on the wiki.

@mstimberg
Copy link
Member

Regarding saving the arrays and the specifier system: I see two ways of implementing this:

  1. Specifiers themselves are device dependent, e.g. to create a new array variable you would do something along the lines of device.create_array_variable('v', unit, len(self), ...)
  2. Specifiers are device independent, but delegate array access to a device-dependent object, so creating a specifier would pass a device: ArrayVariable('v', unit, len(self), device, ...)

With the first option you have re-implement all specifiers for each device, but maybe we need the flexibility that comes with it? The classes are basically simple data container, anyway.

@thesamovar
Copy link
Member Author

I slightly prefer option (2) but only very slightly so let's see what seems more convenient as we start implementing stuff?

@thesamovar
Copy link
Member Author

Oh, and let's keep the discussion on issue #49.

@thesamovar
Copy link
Member Author

OK I've updated the wiki page on devices, we should work from this now. It's not very detailed, but it has the core ideas. I think now we need to actually implement C++ standalone mode and then people can use that as a reference when implementing other modes.

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

2 participants