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

Implementation of 6K model #8

Open
wants to merge 14 commits into
base: master
Choose a base branch
from

Conversation

AdrianBeersingVasquez
Copy link

No description provided.

Include exception in RhodopsinModel base class when nStates='6K' and cannot be converted to string
Remove assertion that 'nStates' is an int, and account for this when initialising 'states'
Include the 6K model in the ordered dict containing all models
Include Kuhne model within the list of default model parameters
Define default parameters for 6K model
Include 6K model in 'modelParams'
Describe fitting procedure for 6K model
Redefine list of non-optimised parameters for 6K model.

Warning: This removes the check to ensure only one of the possible models (3, 4, 6) has been selected.

To do: Ensure only one of the four models is selected.
Include constrained parameters for 6K model.
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.

1 participant