qiskit-nature
qiskit-nature copied to clipboard
qiskit-community
`GaussianForcesDriver.from_molecule` improvements
What is the expected enhancement?
The GaussianForcesDriver.from_molecule method currently has two limitations:
- It hard-codes the
B3LYPfunctional which should really be a configurable option. - It ignores
driver_kwargs(which needs to be changed if the above functional should be made configurable in an easy fashion)
On another note: technically it would be possible to use the VibrationalStructureMoleculeDriver wrapper to construct a GaussianForcesDriver from a molecule while also providing a logfile via the kwarg dict (once this gets no longer ignored). While it may seem like there is no point in doing that, it would allow for a more streamlined approach when running Gaussian calculations on another machine but using the same interface. I.e.:
Machine A:
driver = VibrationalStructureMoleculeDriver(molecule, basis, driver_type)
driver.run() # produces log written to some file
Machine B:
driver = VibrationalStructureMoleculeDriver(molecule, basis, driver_type, {"logflle": "path/to/copied/logfile.txt"})
driver.run() # reads produced log file
Instead one currently needs to use the GaussianlogDriver directly on machine B.
If this were to be changed, the GaussianForcesDriver would need to relax it's check_installed method when reading from a logfile (in which case Gaussian does not need to be installed)
Summary
There are three tasks:
- [ ] make XC functional in
GaussianForcesDriverconfigurable - [ ] do not ignore
driver_kwargsinGaussianForcesDriver.from_moleculemethod - [x] relax
GaussianForcesDriver.check_installedmethod in case of usinglogfile
Hi, I am new to the contribution, I think this is a good place to start. Can I try to solve this?
Hi @bharath1996-hub! Sure thing, please go ahead :slightly_smiling_face:
Please make sure to apply your changes to the class in the qiskit_nature.drivers.second_quantization module (and not in qiskit_nature.drivers).
@bharath1996-hub Are you still interested in working on this?
Just a heads-up: #701 outlines the new direction which we are taking with Qiskit Nature which likely means that the drivers as they exist today will not be relevant in some future. It is hard to predict now in what form they will remain but if you still want to add this contribution to Qiskit Nature you are more than welcome to do so! Just let me know :+1:
