-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add simple gaussian model #12
Conversation
change some return errors to raise errors (mea culpa) made the dataset be an array of datasets to have the same format as the final nT likelihood (for storing and loading etc)
@kdund I just fixed some import issues but apart from that I'd now leave it at that so we can continue working with this. We already discussed a few changes in the structure last Friday but I think they come more natural once we have the Parameter class. |
sounds good-- should we start making a list of all properties a parameter might have? |
Yes I'm now opening an issue for the Parameter and ParameterList class, maybe we can then discuss it there. But I think we can merge this PR, right? |
yes agreed |
This adds the simplest model, which can serve as a very basic example and for unittests. This closes #5.
You can run the following code to check the functionality: