-
Notifications
You must be signed in to change notification settings - Fork 1
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
looser symmetry tolerance for optimization #4
Comments
Alternatively, adjusting this https://github.com/psi4/psi4private/blob/master/src/bin/optking/set_params.cc#L422 |
I will look into this and find an example. |
If I recall correctly, methane might be one such? David On Mon, Feb 8, 2016 at 3:36 PM, polikw [email protected] wrote:
|
I can reproduce this error in WebMO for Two points:
I am attaching the archive of my jobs, which you can import from Job Manager: New Job: Import Job : Import Type: WebMO archive. From this you can inspect the input and output files in detail. IMPORTANT: You must FIRST unzip this *.tar.zip file to to a *.tar file, and THEN import the *.tar file into WebMO (because GitHub apparently does not support attaching tar files!). |
Seeing David's comment, I just verified that
For the details, convert the attached *.tar.zip to a *.tar file and import into WebMO. |
Ok, I think I have the fix. original input:
working input:
The main change are the So there's no actual changes to psi4. @bwb314, would you want to try incorporating these changes into the WebMO templates? |
Can you check how non-symmetric the geometry was that WebMO was providing On Thursday, April 14, 2016, Lori A. Burns [email protected] wrote:
|
No 1e-16. The general Molecule uses That said, I don't really understand what's going on with the methane. By Avogadro, it looks perfectly symmetric (1–2 decimal places). But the methane needs symmetrizing with tol<=0.66 and printing out the atom distances I think gives the below as distinct atom distances.
|
This is puzzling. Suggest looking back at H2O case -- I think that one On Thu, Apr 14, 2016 at 9:32 PM, Lori A. Burns [email protected]
|
By calculator, it's symmetric. But with only writing 8 decimal places in the input, it mayn't be possible to satisfy the default symmetry detection of 1e-8. The calibration of |
Please forgive the naivety of my comments, since I do not have a detailed understanding of the inner workings of psi4. Perhaps they are useful, through maybe not... It seems to me that two different parts of the program have different symmetry detection tolerances, first when the symmetry is initially determined, and second with the optimizer so as to be more efficient. Making these tolerances the same would seem to be a good start. However, I would not be surprised if these different sections of code even used different algorithms for symmetry detection, since they were presumably written by different people. Thus it might make sense either to make the second tolerance slightly looser than the first, or to symmetrize the molecule to a very high tolerance before starting the process (which I think is Lori's solution). Thank you for working on this issue!
|
@loriab I just checked your fix in. All of the test cases provided in this thread can now be optimized successfully using the WebMO interface. |
I think it is actually behaving rationally, though there's several things going on. In any psi4 run, the input geometry symmetry is taken as given, subject to a 1e-8 tolerance. After an optimization pass, the projected geometry is loosely symmetrized to 0.01 to clean up any numerical error. At this point if the point group changes to a higher symmetry, this is regarded as a proper error and the message you've seen gets thrown.
|
The List
21
tied with psi4/psi4#232The text was updated successfully, but these errors were encountered: