- 
                Notifications
    You must be signed in to change notification settings 
- Fork 9
topic modeling cluster script #25
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
Conversation
| @paxtonfitzpatrick looks like there's a merge conflict 😿. Also, do you want me to actually review this in detail or just merge once the conflict is resolved? | 
| @jeremymanning the conflict is just that there were two copies of my  And can you please do a quick check over the config, submit, and cruncher files? That's really all I need | 
| Oh, I also updated the remote tracking branch of the cluster tools submodule, and I think I did it in a way that when other people pull it will update for them, but I'm not certain. might have to do  | 
| @jeremymanning fixed 🎉 | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this pull request looks good-- but there are some issues with the submodule (cluster-tools-dartmouth), which is (it seems like) what you're asking me to look at. but i think you need to submit a separate pull request to get those changes into a format that i can review. or we can just go over them in person during the hackathon time.
| either works for me! I just created this ContextLab/clustrix#6 so you can compare and comment if that's easier for you | 
to run, will need to:
cluster-scriptsdirectory to/dartfs/rc/lab/D/DBIC/CDL/f0028ph/MIND-narrative-modeling/cluster-scriptshelpers.pyto/dartfs/rc/lab/D/DBIC/CDL/f0028ph/MIND-narrative-modeling/cluster-scripts