-
Notifications
You must be signed in to change notification settings - Fork 2
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
Update VM #4
Comments
@bethac07 I have update the provisioning script for DCP Control Node by looking up instructions on the [wiki](# https://github.com/CellProfiler/Distributed-CellProfiler/wiki/Before-you-get-started%3A-setting-up) I have copied it nearly verbatim here https://github.com/cytomining/cytominer-vm/blob/issues/4/python.sh#L68 so I think we should be all set, but can you confirm that there's nothing extra you do to set up the node that is not documented in the DCP wiki? Also, a minor question: do you know why you do this |
DCP isn't tested in Python3 yet. |
Ah I missed that. I think I had confused it for the fact that CPv3 is now in DCP. PS - I just saw DistributedScience/Distributed-CellProfiler#83 so its certainly more than a few hours of work. I'll go ahead and add back Python2 |
I don't know how non-trivial it is yet.
I don't think there's any urgent need to update the VM until that is ready.
…On Mon, May 4, 2020 at 4:24 PM Shantanu Singh ***@***.***> wrote:
DCP isn't tested in Python3 yet.
Ah I missed that. I think I had confused it for the fact that CPv3 is now
in DCP. I assume it is non-trivial to do so? If so I'll re-add Python2
support in this VM (I had some ssl issues in doing so but should be easily
fixable)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#4 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABTI72YAVXIT56QOQMC2X5LRP4QA5ANCNFSM4MXKJ2GQ>
.
--
Beth Cimini, PhD
Senior Computational Biologist, Imaging Platform
Broad Institute
415 Main St Room 5011
Cambridge, MA 02142
Current office number- (617) 714-8189
Pronouns - She/her/hers
I will sometimes send or respond to emails outside of my local office
hours, but I never expect responses outside of your local office hours.
|
Sounds good. I'll leave this PR open |
DCP now does work on Python 3 (or 2, still). Only dependency for the login node is |
@BolekZapiec asked:
|
I haven't tested DCP on Python 3.9, but IIRC it should support anything 2.7-3.8, and i have no reason to think it doesn't support 3.9. It's not a huge codebase, and the only dependency now is boto3. I think we can probably pretty easily python3-ize pe2loaddata, I can prioritize that if need be. |
@BolekZapiec – just checking if you trying working further with this VM and if you had any notes to share? |
Maybe it's outside the scope of this, but I think it might be worth explaining the security group setup for the EFS and how to select an IAM role that allows bucket access (whether we're using it at build rather than runtime) - if only for the benefits of our future selves, because it definitely took me a while to figure out the EFS setup. |
Also, this assumes an R-cytominer world, do we currently live in a py-cytominer world and if so, we should add in whatever we need for that. |
@shntnu Thanks for the reminder - I got sidetracked with getting the ECS (finally!) working on our internal subnet and forgot to update this. Below are my notes for the necessary changes tool.sh change change Change python.sh add change change docker.sh change change delete cytominer_ami.json change change For the S3, for some reason, the command in fstab didn't work for me, instead I first need to generate rights file: Before pushing these changes I'd like to re-create a cytominer VM with all these changes implemented so that I can be sure there was no additional debugging steps I did post-hoc that made things work and that the edits listed are comprehensive. |
ami_name
(latest Ubuntu) and related items inbuilders
cytominer-database
on PyPi. Add back Python2cytominer
on CRANs3fs-fuse
installation procedureapt
instead ofapt-get
*** System restart required ***
on launch. Maybe read this New provisionner: posix-restart hashicorp/packer#4555 (comment)Notes:
@connersk said: Yup used https://github.com/cytomining/cytominer-vm but I had to tweak a few things, the VM runs Ubuntu 14.04 which is no longer supported, so I updated it to Ubuntu 18.04, changed the R and python installations, and used a different means of creating a python 3 virtual environment
The text was updated successfully, but these errors were encountered: