Skip to content

Conversation

@paxtonfitzpatrick
Copy link
Member

to run, will need to:

  • move cluster-scripts directory to /dartfs/rc/lab/D/DBIC/CDL/f0028ph/MIND-narrative-modeling/cluster-scripts
  • add @jeroenvanbaar's helpers.py to /dartfs/rc/lab/D/DBIC/CDL/f0028ph/MIND-narrative-modeling/cluster-scripts
  • set number of nodes, processors per node, and wall time
  • make sure conda environment has all necessary modules installed

@jeremymanning
Copy link
Member

@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?

@paxtonfitzpatrick
Copy link
Member Author

@jeremymanning the conflict is just that there were two copies of my get_events.ipynb notebook and I deleted the older one... not sure why it doesn't like that 😕

And can you please do a quick check over the config, submit, and cruncher files? That's really all I need

@paxtonfitzpatrick
Copy link
Member Author

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 git submodule update

@paxtonfitzpatrick
Copy link
Member Author

@jeremymanning fixed 🎉

Copy link
Member

@jeremymanning jeremymanning left a 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.

@paxtonfitzpatrick
Copy link
Member Author

either works for me! I just created this ContextLab/clustrix#6 so you can compare and comment if that's easier for you

@jeremymanning jeremymanning merged commit 3460310 into ContextLab:master Aug 13, 2019
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