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

Equations for IMs #13

Closed
peter-michalski opened this issue Oct 10, 2019 · 4 comments
Closed

Equations for IMs #13

peter-michalski opened this issue Oct 10, 2019 · 4 comments
Assignees

Comments

@peter-michalski
Copy link
Collaborator

I am wondering if there should be equations for the IMs (derived from GD1). How does the programmer transform the inputs into the outputs listed in the IMs? I am not familiar with optics myself, if I had to program this I would not be sure how to proceed (even from an abstract viewpoint; that is, not considering the data structures or algorithms that will come during design).

I would like to get @smiths input on this as well in case I am wrong here - hoping to expand my understanding.

@smiths
Copy link
Collaborator

smiths commented Oct 19, 2019

@peter-michalski, this is a great test of the documentation. If someone cannot implement the IMs, at least "abstractly", then the document isn't done. @sorainsm, I haven't finished my review of your document, but I can see what @peter-michalski is getting at. Your Output field lists the output variables, but it doesn't say anything about how they are calculated. It looks like information is missing.

@sharyuwu
Copy link
Collaborator

@sorainsm I also agree with the point doctor smith and peter points. It looks confusing without the equation/relationship between input and output in your IM. Moreover, it a bit hard to tell how does the data definition drive the Instance Model.
I will post an issue about this question.

@sorainsm
Copy link
Owner

sorainsm commented Nov 5, 2019

Equations added to IMs in cb95e52.
Also revised their structure to make the input and output types explicit.

@peter-michalski can you take a look at this new structure and let me know if it's more clear how the IMs would be implemented?

@smiths can you take a look at the structures and let me know if it's too close to code?

@sorainsm
Copy link
Owner

Closing issue as the IMs have been resolved and I'm unlikely to get feedback at this point.

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

4 participants