Skip to content

Conversation

@deyanmihaylov
Copy link
Member

No description provided.

x0,
args=(mol, basis_str, exp_array, use_Jacobian),
method="SLSQP",
method="Nelder-Mead",
Copy link
Member

Choose a reason for hiding this comment

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

Why is SLSQP changed to Nelder-Mead? Please always remember to "pull" the latest version of a code before pushing any commits!

print(x0)
bnds = ((1e-9, None) for _ in range(exp_array.shape[0]))
def minimize_energy(mol, basis_str, exp_array=None, use_Jacobian=False):
x0 = exp_array[np.where(exp_array[:, 1] == 0), 0]
Copy link
Member

Choose a reason for hiding this comment

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

Good, so it is meant to check that the exponent's freeze/free flag is ==0 before sending it to the optimizer. I will check to see if it works soon!

@ndattani
Copy link
Member

Thank you @deyanmihaylov!
I think any progress in this project now is not worth pursuing if we don't have the scaled Newton-Raphson method (or something as good) available. What do you think an approximate timeline for getting that available would be? Do you want help from Vandan and/or a student? Would you like me to do it myself? If you want me to do it myself I can try it in Python though would probably be able to get it working in Octave or Julia faster (PySCFad would still provide the energies and derivatives, but would be called from an Octave or Julia program).

@deyanmihaylov
Copy link
Member Author

I would suggest a timeline of having the code adapted to do this is end of this week. I am already working on this (but I haven't got anything ready yet).

@ndattani
Copy link
Member

@deyanmihaylov Is there anything specific that caused this code to take so much longer than the original estimated timeline? In Discord you asked me about the meaning of the X variable and I provided you the paper and the paragraph that introduces X.

@deyanmihaylov
Copy link
Member Author

@ndattani I apologise for the delay, I had an emergency which kept me away from my work for the past few days, today I am back to work. I will discuss on Discord and follow up accordingly.

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