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

Branch1 #65

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

Branch1 #65

wants to merge 3 commits into from

Conversation

jaysonjeg
Copy link

No description provided.

@vidaurre
Copy link
Collaborator

vidaurre commented Dec 1, 2021

Hi jaysonjeg,
Thanks for your input.
About the first change, I think we want it to be stochastic here, as stochasticity can be used to our advantage (see Alonso et al., 2021 BioarXiv, for instance)
About the second change, right now it doesn't make sense but it relates to something I'm currently developing so it will hopefully make sense in the future
About the third change, that seems relevant. What's the error this is preventing?
Best,
d

@jaysonjeg
Copy link
Author

Hi, sorry I'm new to GitHub. I meant to push these change to my local fork, not a pull request to your original repository. So pls disregard this.

I'm doing a Gaussian observation model, not MAR. I was using commit 710dd64. padGamma produced an error there because d was not defined by line 27. You may have addressed this in more recent commits which I haven't noticed..

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