Skip to content

Update bonds when atoms moved individually #48

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

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

cajfisher
Copy link
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
Copy link
Collaborator

ovhpa commented Sep 4, 2022

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,

src/file_cif.c Outdated
@@ -182,7 +182,7 @@ return(0);
/***************/
/* CIF loading */
/***************/
#define DEBUG_LOAD_CIF 0
#define DEBUG_LOAD_CIF 1
Copy link
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.

@@ -399,8 +399,8 @@ struct core_pak *core1, *core2, *core3;
/* FIXME - this only uses the 1st 3 points, in future - an average for n pts */
/* http://www.geocities.com/CollegePark/Campus/1278/JBook/LSQ_Plane.html */
/* 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
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