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

Shrinking strength does not work #22

Closed
yueqiw opened this issue Jul 27, 2018 · 4 comments
Closed

Shrinking strength does not work #22

yueqiw opened this issue Jul 27, 2018 · 4 comments

Comments

@yueqiw
Copy link

yueqiw commented Jul 27, 2018

Hi,

First of all, Slingshot is a great package!

In the documentation, the shrink parameter is described as follows:

shrink: logical or numeric between 0 and 1, determines whether and how much to shrink branching lineages toward their average prior to the split.

However, when I try shrink = 0.1, it produces the same result as shrink = TRUE. Looking at the code, I don't see how a numeric shrink is used. Is there another way to set the strength of shrink? Thanks!

@kstreet13
Copy link
Owner

Thanks!
And thank you very much for pointing this out. You're absolutely right that the shrink parameter was not doing what it says in the documentation. I'm not sure where along the way that functionality was lost, but I just pushed a new update to the develop branch that should restore it (and I will add it to master once it passes the usual checks).

@yueqiw
Copy link
Author

yueqiw commented Jul 31, 2018

Thanks! I made a similar change in my fork. I've also been trying to use RNA velocity to guide the initialization of Slingshot, so that the fitted trajectories are better aligned with RNA velocity vectors. I've got it to work to some extent but I feel that it can be done better. If you're interested, we can discuss it at some point.

@kstreet13
Copy link
Owner

@yueqiw
Yeah, this sounds very interesting! I'm going to close this issue, but feel free to email me at kstreet@berkeley.edu if you want to discuss it further.

@yueqiw
Copy link
Author

yueqiw commented Aug 3, 2018

@kstreet13 Sure. I'm doing manuscript writing but will email you in the next week or so.

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

2 participants