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

Initial steps. #341

Conversation

aoifeboyle
Copy link
Contributor

Just creating a PR so we can work on this together @wkerzendorf. Been trying to figure out where to start using the IonNumberDensity module as an example. I'm not sure if we want to keep the output properties from the same function grouped together or separate? If you can show me how to get started when you have time that would be great.

@wkerzendorf
Copy link
Member

@aoifeboyle so there are now two PRs with the new outputs structure, right?

@aoifeboyle
Copy link
Contributor Author

@wkerzendorf No, I haven't added the new outputs stuff to PR #340. Is that the other one you're referring to? You said to create a new branch for this stuff. I was planning to go back to #340 when I get this working.

@wkerzendorf
Copy link
Member

@aoifeboyle let's quickly skype if you have time.

@aoifeboyle aoifeboyle closed this Jul 1, 2015
@aoifeboyle aoifeboyle deleted the plasma/multiple_inputs branch July 1, 2015 11:08
@aoifeboyle aoifeboyle restored the plasma/multiple_inputs branch July 1, 2015 15:44
@aoifeboyle aoifeboyle reopened this Jul 1, 2015
@aoifeboyle aoifeboyle closed this Jul 1, 2015
@aoifeboyle aoifeboyle deleted the plasma/multiple_inputs branch July 1, 2015 15:46
This was referenced Jul 1, 2015
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