2020-11-16 Core Devs Meeting notes

Date

Nov 16, 2020

Participants

  • @Jeffrey Wagner

  • @Pavan Behara

  • @Matt Thompson

  • @David Dotson

  • @Simon Boothroyd

  • @Jaime Rodríguez-Guerra (Deactivated)

  • @Joshua Horton

Discussion topics

Item

Notes

Item

Notes

Jaime Rodríguez-Guerra

  • OMM-Quansight helped a lot

  • Significant complexity around CUDA toolkit – Need nvcc metapackage, build matrix with definitions for new versions, ways to install windows drivers (not included in docker image).

    • We’ve emailed NVidia – JRG contacted JC, who forwarded request to NVidia.

  • Added requirements to conda-forge. Maybe some revisiting OpenCL policy issue with Isuru.

  • CUDA toolkit packages are not being built on conda-forge, listed on testing label. Been iterating on this, looks promising. These packages will be going onto main shortly.

  • JC and PE are working on 7.5.0 release, working on OpenCL stuff to ensure that we don’t lead incompaible deps.

  • JRG is working on PPC package, not ready yet, don’t anticipate issues.

  • Now we need to test c-f OpenMM package

    • Would really help to have GPU tests run

    • SB – I do GPU-enabled tests on lilac periodically. I’ll test out the new OpenMM there.

    • SB – Will module add be sufficient for making CUDA available during OpenMM install?

      • JRG – If the driver is available at install-time, it will cause a metapackage to be added to the conda env and ensure that the correct OpenMM is installed. If multiple CUDA versions are available, it will pick the highest one. After/during conda installation, theere should be a CUDA toolkit package queued for download/installed that will sow the version that it thinks your computer has.

  • Once OMM tests pass, then we’ll move whole OpenFF stack to c-f.

  • How to handle OpenEye deps?

  • MT – How to handle nonessential deps?

    • “Developer deps” like pytest

    • “Example deps” like ParmEd, other notebook goodies

    • “Serialization deps” like bson

    • SB – Agree – Would like a minimum openforcefield package, without RDKit, OpenEye, OpenMM, etc. Could have metapackages for more elaborate deps.

      • SB – So openff-toolkit would pull openff-toolkit-core and rdkit and friends, whereas it would be possible openff-toolkit-core only install.

    • JRG – This could work like matplotlib on c-f, where the core version is very stripped down.

  • JRG – Namespace planning – We need to be careful with this.

    • SB – Could follow pattern of openff-packagename

    • MT – Plan is for everything to be imported as openff.packagename, currently everything follows this except toolkit (which is currently openforcefield, but we plan to migrate to openff.toolkit)

    • JRG – If we make an openforcefield package on conda-forge, we can never “take it back”, and if we make a…

    • (General) – We will never upload an openforcefield package to conda-forge

      • 0.8.1 release will beupladoed to c-f as openff-toolkit, but import in python will still be from openforcefield…

      • 0.8.1 release will ALSO be uploaded on omnia, but will importtime deprecation warning that next release will only be on c-f

        • JW will manually patch omnia package to have deprecation warning

  • (General) – JRG will move forward “full steam” with openmm migration, not be concerned about whole omnia ecosystem migration.

  • (General) – We should never release openforcefield or openforcefieldds package to conda-forge

  • JW – Still unresolved about how to handle OE deps in OFF-toolkit repo, won’t be able to accept PRs from forks until reoslved

    • JRG – Maybe look at patterns in sphinx-contrib

  • DD – Do we want to make a formal process/planning document for the omnia migration?

@Jeffrey Wagner will take the lead on making an omnia migration checklist, espeically WRT openff packages. Will loop in @David Dotson and @Jaime Rodríguez-Guerra (Deactivated)

Simon Boothroyd

  • Worked with @Joshua Horton on rebuilding wavefunctions from QCA records

    • SEEMS TO BE WORKING!!!!!

    • Thanks to @David Dotson as well for working on the QCA side of this.

    • SB is planning to integrate this into recharge, JH will try to integrate it into psi4

    • SB – Should this functionality live in psi4 or QCEngine?

    • JH – QCEngine would be the best home

    • DD – Slightly favoring QCNG, though psi4 would also work

    • (General) – The wavefunction storage should be engine-independent, so it shouldn’t be restricted to psi4.

    • (General) – We’re relatively confident that the current implementation is working, but there’s limited bandwidth on the QC developer side to validate it.

    •  

@Simon Boothroyd will loop in @Pavan Behara to review wavefunction reconstruction PR when it comes
  • SB – JRG, I had some trouble with plotly, but it turns out that it’s hard to make responsive plots WRT web browser resizing. Though I’ve since figured out how to put in some customization to make it smoother.

Matt Thompson

  • Added dependabot for GH actions to many repos

David Dotson

  • Running lots of datasets on qca-dataset-submission

    • Debugging Lilac errors

    • May need larger nodes for large molecules, could add filtering to not error-cycle element coverage problems or convergence step errors. Could also route to openff-highmem tag.

  • Driving partner benchmarking work – Delegated among @Joshua Horton @David Hahn @Jeffrey Wagner , and @David Dotson

Action items

Decisions