Skip to content

Conversation

@jrgissing
Copy link
Collaborator

@jrgissing jrgissing commented Jan 4, 2025

Tutorial 8 updates: correct for charge neutrality, single mol-ID for CNT

closes #50

@jrgissing
Copy link
Collaborator Author

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

@jrgissing
Copy link
Collaborator Author

@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'

@simongravelle
Copy link
Member

also, 'Rnx' in Figure 50 should be 'Rxn'

Corrected, thanks.

@simongravelle
Copy link
Member

simongravelle commented Jan 4, 2025

but I am unable to distinguish the colors in the manuscript-sized image

I have increased the image so that it occupues the full width of the column, would that work for you ?

@simongravelle simongravelle self-requested a review January 4, 2025 21:28
Copy link
Member

@simongravelle simongravelle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks

@jrgissing
Copy link
Collaborator Author

but I am unable to distinguish the colors in the manuscript-sized image

I have increased the image so that it occupues the full width of the column, would that work for you ?

yes, that helps

@simongravelle simongravelle merged commit cfaed05 into main Jan 5, 2025
@simongravelle simongravelle deleted the tut8_minor_corrections branch August 11, 2025 13:10
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.

Tutorial 8: System is not charge neutral

3 participants