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

Patch jupyter form #62

Merged
merged 10 commits into from
Feb 13, 2019
Merged

Patch jupyter form #62

merged 10 commits into from
Feb 13, 2019

Conversation

diedpigs
Copy link
Collaborator

@diedpigs diedpigs commented Feb 7, 2019

Update launcher form:
add size of memory, number of cpu, partition.
remove customize environment, jupyter extra args

Update submit form to take the attributes added in launcher form.
Also, hardcoded -N 1 to restrict job to run on only one node.

load Anaconda3 inside job script

Remove custom environment from launcher form
Remove jupyter_extra_args form launcher form
Remove number of hour form launcher form
Remove account form launcher form
Remove queue form launcher form

Hardcoded module load anaconda3 in script.sh
Add number of cpu in launcher form
Add memory per cpu in launcher form
Add partition in launcher form
Update submit script to take attributes just added
Keeping hardcoded module load Anaconda3
 jupyter need Anaconda to be loaded in order to work
Since we have synced lmod.sh in place, no need to call module use here
Instead, we call module reset here to load all default module
Remove module load Anaconda3 which we already hardcoded in script
 add more explanation for the purpose of this block.
@jprorama
Copy link
Owner

We should add a first task to the ood_jupyter role to remove the existing git clone so it can be re-cloned. Otherwise we get an error.

- name: Remove existing jupyter app install
  shell: rm -rf /var/www/ood/apps/sys/jupyter/

@jprorama jprorama merged commit 3cf5318 into jprorama:uab-dev Feb 13, 2019
@diedpigs diedpigs deleted the patch-jupyter-form branch September 9, 2020 21:40
diedpigs pushed a commit to diedpigs/CRI_XCBC that referenced this pull request Jan 10, 2024
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