--- name: DIALS core meeting 2021-01-14 tags: core meeting --- # DIALS core meeting 2021-01-14 [![hackmd-github-sync-badge](https://hackmd.io/LU5Trm_uQOuzqKoIolYspA/badge)](https://hackmd.io/LU5Trm_uQOuzqKoIolYspA) ## Previous Actions * [ ] MG+ND: write a proposal for ['overall architecture discussion'](https://dials.github.io/kb/core/20200903) and ['new installer'](https://dials.github.io/kb/core/20200903) * [ ] MG+ND: Come up with a proposal to [move away from all code being in header files](https://dials.github.io/kb/core/20201001) and consolidate into a single library * [ ] ND: update `CONTRIBUTING` in a (separate) pull request to include a soft-imposition of [stable master](https://dials.github.io/kb/core/20200903#stable-master-branch), [dials#1353](https://github.com/dials/dials/issues/1353). * MG to look into and see if we can make a reasonable change otherwise we can drop it * [ ] MG: organise a [typing intro lecture](https://dials.github.io/kb/core/20200917). * [ ] [cbflib conda-forge package](https://dials.github.io/kb/core/20201125#cbflib-conda-forge-package) * [ ] ND to prepare an agenda for the HDRMX meeting * [ ] ASB to organise a side-meeting next to the HDRMX workshop * [ ] ASB to explore what should happen to cbflib maintenance in the future * GW: Unlikely to happen in the next couple of weeks. In March there is an HDRMX workshop, potentially we could add a peripheral meeting there. * [ ] ND to set up a [`cbflib_adaptbx` pull request](https://dials.github.io/kb/core/20201125#cbflib_adaptbx-dependency) for MAR detector fixes. Verify with `labelit_regression`. * [ ] GW: write [dxtbx rewrite project proposal](https://dials.github.io/kb/core/20201209#discussion). * Forked dxtbx, created a `dx2` repository at https://github.com/graeme-winter/dx2/ and explored the terrain. * Python 3.6 as baseline ## Agenda ### Aaron's PR Requests Aaron requested that a slew of pull requests be reviewed by the next meeting after 2020-11-25: #### dxtbx: - ~~Allow certain classes to be labeled as 'abstract' [(#255)](https://github.com/cctbx/dxtbx/pull/255)~~ - ~~Deprecate h5rawdata and use `Format.ignore()` in the Registry [(#261)](https://github.com/cctbx/dxtbx/pull/261)~~ - ~~Make `FormatHDF5SaclaMPCCD` lazy [(#227)](https://github.com/cctbx/dxtbx/pull/227)~~ #### Dials: - ~~Redo test to support new h5 file parsing behavior in dxtbx [(#1499)](https://github.com/dials/dials/pull/1499)~~ - ~~dials.import: individually select models from reference expt [(#1371)](https://github.com/dials/dials/pull/1371)~~ - Multiple stills view [(#1463)](https://github.com/dials/dials/pull/1463) - too large to review here; but has been in use for a number of months - [DLS] Are going to look at this and merge by next core meeting - if not done by then, then it goes straight in pending news etc - Add `is_stills` parameter to the spotfinder API to allow `dials.stills_process` to work with scans [(#1508)](https://github.com/dials/dials/pull/1508) - MG to refactor the test - merge, with proviso to please add a ticket that we will review this as part of the dx2 work - ND to add a newsfragment, ASB to merge ### renaming `master` branch → `main` * As of January 2021 Github has started rolling out their [seamless branch rename](https://github.com/github/renaming#creating-a-seamless-move-for-existing-repositories-) * Agreed to migrate to `main`. Turn this into dxtbx/dials/xia2 issues and move once GitHub provides a migration path. Use xia2 as a test-bed and move dials and dxtbx two weeks later. * These tickets have been created as [`xia2/xia2#557`](https://github.com/xia2/xia2/issues/557), [`dials/dials#1546`](https://github.com/dials/dials/issues/1546), and [`cctbx/dxtbx#281`](https://github.com/cctbx/dxtbx/issues/281) * Plan is to deliberately break master, then migrate, then revert the breakage, so that anyone stuck on master will notice right away. * TBD: define breakage-message ### DIALS documentation build Should the DIALS documentation build outside a cctbx environment? * Currently you can only build the DIALS documentation inside a cctbx environment. This means a remote documentation build needs to build cctbx first, so we can't easily run the documentation build on every commit. When things break they are difficult to fix. We can't use readthedocs. * MG: I believe the main obstacles are three cctbx Sphinx plugins that may need to be extracted or removed from DIALS documentation, and some phil parsing logic. * MG: the cctbx conda-forge package may be useful here * GW & ND: It would be nice. At this time we don't have the time. * Outcome: wontfix ### move the active `dxtbx` repository into the DIALS organisation Having the main repository in cctbx_project has a few distinct disadvantages. The most annoying is the [eternal fight against the lockdown bot](https://github.com/dials/dxtbx/pulls?q=is%3Apr+is%3Aclosed+-label%3Aupcoming-release+), but we also don't get LGTM output. * This predates the `dx2` discussion. `dx2` will live in the DIALS organisation. * Outcome: Leave this as it is. ### pandas XFEL package may introduce a [pandas](https://pandas.pydata.org/) dependency. Want to use `dials.cosym` + XFEL data + MPI, pandas proved useful for bookkeeping. General enthusiasm noted. ### dials.index: create new experiments *not discussed in this meeting* dials.index should (or should not) create new experiments with crystals in rather than modifying in place existing experiments [as proposed in this issue](https://github.com/dials/dials/issues/1029) ### Next meeting Thursday, January 28th, 4pm UK time, 8am PDT.