Skip to content

Update bonds when atoms moved individually#48

Open
cajfisher wants to merge 8 commits into
arohl:masterfrom
cajfisher:master
Open

Update bonds when atoms moved individually#48
cajfisher wants to merge 8 commits into
arohl:masterfrom
cajfisher:master

Conversation

@cajfisher

Copy link
Copy Markdown
Contributor

Refreshes bonds in model window when atom coordinates changed in Model:Editing box (same as already done for groups of selected atoms).
Also updates dipole in Model:Content box as soon as changes made.

Refreshes bonds in model window when atom coordinates changed in Model:Editing box (same as already done for groups of selected atoms).
Enables reading and writing of structures in xsf format (including animation files)
Fixes bug in file_meta.c when parsing units
Replaces deprecated g_pattern_match_string with g_pattern_spec_match_string in file.c
Replace QE with Quantum Espresso in file type lists for clarity
Correct various spelling/punctuation errors
@ovhpa ovhpa self-assigned this Sep 4, 2022
@ovhpa

ovhpa commented Sep 4, 2022

Copy link
Copy Markdown
Collaborator

Hello,
I'm back from the hectic times...
I now work on other projects, but I will come here on GDIS sometimes ;)

About the PR:

  • gcc and other compilers (icc, clang)
  • valgrind / libasan memory checks
  • static analysis
  • review

I will review the XCrysDen file reader later with the other PR.
As per @arohl suggestion:
Consider changing the header of the new file you created (file_xsf.c) to be under your name, for example:

/*
Copyright (C) 20XX by MYNAME

myname@myprovider<mailto:mymail@myprovider>

This program is free software; you can redistribute it and/or
modify it under the terms of the GNU General Public License
as published by the Free Software Foundation; either version 2
of the License, or (at your option) any later version.

Where MYNAME is.. your name 😆

Best,

Comment thread src/file_cif.c Outdated
/* CIF loading */
/***************/
#define DEBUG_LOAD_CIF 0
#define DEBUG_LOAD_CIF 1

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

DEBUG should be OFF by default, I think.

Comment thread src/spatial.c
/* shouldn't matter when this routine is properly implemented, as we'll go */
/* through the atom_list in the usual fashoin & terminate when it is NULL, */
/* not when it's data ptr is NULL */
/* through the atom_list in the usual fashoin and terminate when it's NULL, */

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

s/fashoin/fashion/

Adds a button to "Region" window of "Model editing" dialog to calculate surface dipole of a 2D model and report it in message box
Some OSs use older versions of glib, before g_pattern_spec_match_string replaced g_pattern_match_string (deprecated since v2.70)
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.

2 participants