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

Docking Installation Instructions #138

Open
RMeli opened this issue Dec 16, 2020 · 4 comments
Open

Docking Installation Instructions #138

RMeli opened this issue Dec 16, 2020 · 4 comments
Labels
[docking] related to the docking tutorial
Milestone

Comments

@RMeli
Copy link
Member

RMeli commented Dec 16, 2020

Docking installation instructions are note very self-contained and can be non-obvious:

  • ADFR_install/bin/reduce suppose that ADFR_install/ is in the current directory
    • There is usually not the case since the ADFR folder ends up in "Downloads"
    • After ADFR is installed, the script suggest to add the folder with binaries to PATH; we should probably clarify this and change commands from ADFR_install/bin/reduce to reduce once they are in the path
@IAlibay
Copy link
Collaborator

IAlibay commented Dec 16, 2020

So @philbiggin made a mention of that a few days ago, where I think the assumption was that "ADFR_install" is just "path to ADFR install folder". I agree it's very confusing.

@philbiggin
Copy link
Contributor

Yeah - I wrote something very hastily as a stop gap which I think was included, but agree it needs to be rewritten.

@IAlibay IAlibay added the [docking] related to the docking tutorial label Dec 16, 2020
@RMeli
Copy link
Member Author

RMeli commented Dec 16, 2020

I think the main "issue" is that there is a lot of copy-paste going on. I guess the best way forward is to explain how to add the folders with binaries to the path so that everyone can type in the same commands later on.

Also, we should probably stress more the importance of reading the output of the executed scripts in order to spot errors/problems where they occur.

@philbiggin
Copy link
Contributor

Yes - copy/paste is a problem if people are not paying attention. And agree with putting emphasis on checking outputs - that is something for next year..

@RMeli RMeli changed the title Docking Installation Insturctions Docking Installation Instructions Dec 16, 2020
@IAlibay IAlibay added this to the December 2021 milestone Dec 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[docking] related to the docking tutorial
Projects
None yet
Development

No branches or pull requests

3 participants