ProtProtocols / IsoProt

Protocol of the analysis of iTRAQ/TMT proteomics data including quantification, statistical analysis and maybe clustering
https://protprotocols.github.io
11 stars 4 forks source link

create a run.bat #40

Closed gvinterhalter closed 5 years ago

gvinterhalter commented 6 years ago

Create batch script equivalent of run.sh

gvinterhalter commented 6 years ago

Docker for windows has some user experience problems:

  1. Docker must be started first
  2. First time running requires approving folder sharing. (might not start the browser if too much time passes)
  3. Pulling the image crashed 3 times saying no privileges or something. But i wasn't using the batch script.

Currently I have to relay on vbs file. Ps I hate windows.

gvinterhalter commented 6 years ago

@veitveit & @jgriss , maybe switching to PowerShell is a better long term investment? Can someone try it on Windows as well?

veitveit commented 6 years ago

I can try on Windows 7 (using the Docker Toolbox as Docker does not run natively there) on a virtual machine as an ultimate test but no experience with windows here. So we probably even need to distinguish old and new Windows machines.

@jgriss how is it with you?

jgriss commented 6 years ago

I am going to set up a virtual windows 10 machine to give the current docker installer a try.

But I think that we need to updated our Readme.md file to add explicit steps for windows users. @gvinterhalter could you write this for win 7?

gvinterhalter commented 6 years ago

@jgriss, I originally tried it on win 7 virtual machine without successes. Not sure if it's win 7, virtual machine or just me not knowing how to run it. Then I tried with win 10 on a regular PC and got it working.

On Mon, Aug 6, 2018 at 12:29 PM, Johannes Griss notifications@github.com wrote:

I am going to set up a virtual windows 10 machine to give the current docker installer a try.

But I think that we need to updated our Readme.md file to add explicit steps for windows users. @gvinterhalter https://github.com/gvinterhalter could you write this for win 7?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/ProtProtocols/IsoLabeledProtocol/issues/40#issuecomment-410663797, or mute the thread https://github.com/notifications/unsubscribe-auth/AF6yEfhQhSsLLeYGlioOZD0KTXXBnKXIks5uOBpsgaJpZM4VMkV6 .

-- Goran Vinterhalter

veitveit commented 6 years ago

I tried with Windows 7 on a virtual machine and it did not work either. But the reason could be the virtual machine. At least my trial gave a complaint about a wrong bios setting (in the virtual machine) which was set correctly but probably not passed to the docker.

On Mon, 6 Aug 2018, 12:58 Goran, notifications@github.com wrote:

@jgriss, I originally tried it on win 7 virtual machine without successes. Not sure if it's win 7, virtual machine or just me not knowing how to run it. Then I tried with win 10 on a regular PC and got it working.

On Mon, Aug 6, 2018 at 12:29 PM, Johannes Griss notifications@github.com wrote:

I am going to set up a virtual windows 10 machine to give the current docker installer a try.

But I think that we need to updated our Readme.md file to add explicit steps for windows users. @gvinterhalter < https://github.com/gvinterhalter> could you write this for win 7?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub < https://github.com/ProtProtocols/IsoLabeledProtocol/issues/40#issuecomment-410663797 , or mute the thread < https://github.com/notifications/unsubscribe-auth/AF6yEfhQhSsLLeYGlioOZD0KTXXBnKXIks5uOBpsgaJpZM4VMkV6

.

-- Goran Vinterhalter

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/ProtProtocols/IsoLabeledProtocol/issues/40#issuecomment-410670889, or mute the thread https://github.com/notifications/unsubscribe-auth/APEZhcID_sr3xzwRxDFt8xU-cKJj29Scks5uOCFpgaJpZM4VMkV6 .

jgriss commented 5 years ago

run.bat will no longer be maintained but replaced by ProtProtocols/docker-launcher