-
Notifications
You must be signed in to change notification settings - Fork 3
Tut8 minor corrections #55
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
Conversation
only one molecule ID for CNT makes more sense
|
please find partial-charge-corrected version here. I am not sure how to mention the image flag warning in Tutorial 8 without it being mostly irrelevant and confusing to beginners |
|
@simongravelle I like your idea of highlight backbone atoms in Figure 49, but I am unable to distinguish the colors in the manuscript-sized image also, 'Rnx' in Figure 50 should be 'Rxn' |
Corrected, thanks. |
I have increased the image so that it occupues the full width of the column, would that work for you ? |
simongravelle
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks
yes, that helps |
Tutorial 8 updates: correct for charge neutrality, single mol-ID for CNT
closes #50