Skip to content
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

Review and tidy-up notes July 2023 #11

Open
52 of 59 tasks
phockett opened this issue Jul 10, 2023 · 3 comments
Open
52 of 59 tasks

Review and tidy-up notes July 2023 #11

phockett opened this issue Jul 10, 2023 · 3 comments

Comments

@phockett
Copy link
Owner

phockett commented Jul 10, 2023

Final pass for most docs, hopefully, on branch reviewJuly2023, https://github.com/phockett/Quantum-Metrology-with-Photoelectrons-Vol3/tree/reviewJuly2023

31/07/23 Released.

General todo

  • Update text. Versions to new date-stamp as revised, and update TOC to reflect.

    • 58f3773 Adds frame defintion section (needs some work) and alignment section (stub).
    • Theory
      • Swap section ordering? Or add note/figure in intro to this effect.
      • Finish observables by type sections.
      • Promote symmetry subsubsection to subsection (may also fix maths rendering issues in "photoionziation dynamics" parent subsection). UPDATE: did this anyway, but rendering issues also fixed with pkg versions in 15/07/23 builds.
      • Clarify LF/AF coupling term - this currently mixes LF and MF definitions for latter case (now fixed in code, see also MF recon manuscript)?
      • Add some notes on alignment? Currently introduced numerically later. (E.g. alignment tensor section). Should include some plots.
        • NOTE: issues with Matplotlib plots in builds 19/07/23, seem to be related to MolPlot routine triggering errors (Arrow3D issues). BUT OK for <v3.6. pip install matplotlib==3.5.3 is working, now set in Dockerfile_qm3_local_install_from_base.
        • This version OK in Jupyter directly in testing, but failing in book build 21/0723 - still some Matplotlib issue.
        • Ah, looks like issue is Arrow3D class, needs patching for Matplotlib >v3.5, see stackoverflow.com/a/74122407 TODO: add to ePSproc with conditional patching depending on MPL version.
        • Current Docker build to MPL 3.7 after fixing other plotting issues (conda install -c conda-forge firefox geckodriver did this, annoyingly), but so far seems OK, although have WARNINGS OFF.
        • Do now see some issues again, reran sudo pip install matplotlib==3.5.3 and pip install --user matplotlib==3.5.3 in container to fix (hopefully), seemed to have some pip/Conda mismatch issues too.
        • Add 'glue' refs to page for builds.
    • Density matrix page needs a tidy. May move QuTip stuff to info content page or appendix (still giving rendering issues).
      • Build 1458_280723 with QuTip section removed for testing. Didn't fix missing figures, maybe other issues?
      • Captions to debug, check logs but maybe missing glue items?
      • Figures OK after Docker Holoviews/Bokeh export fix per Docker section below.
      • NOTE: didn't retest QuTip output in LaTex builds.
    • Info content page also still to do. Updated in f27e2cb
    • Numerics intro section. Updated 24/07/23, passable - content better, but quite a bit of overlap with Chpt. 2; some parts could move to PARTII?
  • Check/test individual notebooks in current Docker build.

  • Debug HTML and PDF builds.

    • Density matrix chpt. no figures in PDF builds (as of 11/07/23)? May be related to QuTip issues (see Qutip Latex matrix output #8 ), although those cells are hidden in PDF build. NOTE: was previously working, and do have static renders in pdf-build dir, although multiple versions, so may just need a clean/non-cached build to fix.
      • Full build (no cache/force) 26/07/23 pm didn't fix. Should debug in single page build.
      • NOW FIXED in builds 30/07/23 after container update as below, figure export was silently failing for HV/Bokeh on this page previously.
      • Retesting QuTip section inclusion... OK
    • MFPAD plots (Plotly) cropping after setting height=800 in plotting routine, but NOT FIXED after setting back to 1200px and clean build. Not sure what is going on there... maybe need bigger and had something cached accidentally previously? Should debug in single page build. NOW FIXED in 27/07/23 builds, back to 1200x800. ALSO SET other figures to None for default height.
      • Updated notebooks in c2125e7, sectioned PAD plotting seems more robust.
      • Also testing separate script with independent figure settings for case studies.
      • Generally working well now, all pages only have 1 row or 2 row layout so can get away with config hack.
    • Added get_ipython().magic('matplotlib inline') to general setup script in 77be80e, hopefully only will do good things. SHOULD ALSO CHECK FIG BUILD CHAIN GENERALLY.
    • In latest Docker container getting RuntimeError: Neither firefox and geckodriver nor a variant of chromium browser and chromedriver are available on system PATH. You can install the former with 'conda install -c conda-forge firefox geckodriver'., in builds AND NOTEBOOK IN LAB, even though installed at container build? Conda env issues, and/or user paths?
    • Part labelling - need to add explicitly for HTML builds, but this messes up PDF builds currently. (Which adds additional "Part" labels.). For linking to Part II in text can use {ref}Part II chpt:extracting-matrix-elements-overview``.
    • Outstanding issues with data.params (lmfit object) data return in basis sets page.
  • Wrap up case studies.

    • 27/07/23 consolidating general setup script. Note this is from mid-July Docker builds (runs on Fock), and still includes phase convention fix (which may be incorrect...). See GeomFunc todo & tidy-up ePSproc#26 (comment)
    • 28/07/23 N2 case study in place. Code a little messy, but otherwise OK.
    • Case study data files maybe >100Mb, breaks Github file size limit. UPDATE: OK as archive, 0d0dd3f
    • TODO: post to Zenodo.
    • 29/07/23 OCS and C2H4 in place. Code as N2. All OK, except MFPADs for C2H4 case, need to fix remapping/downsampling.
      • Build 29/07/23 includes these for testing/debug.
      • Odd - now get C2H4 MFPADs OK, but not OCS case, maybe plotting bugs?
      • Also N2 case not picking up dataset? No idea why, just grabs an old one since deleted? Issues with buildTmp source dir TBC...
      • Ref runs now working, and saved, for both cases. Manual export to HTML OK. Also note automated figure export to static seems buggy.
  • General Docker builds for readers, and push to Docker hub. (see also build chain notes below)

    • Current working version with Jupyter Stacks builds circa May 2023, issues with Jupyter Lab builds in new images (?).
      • Now running tests with v2023-05-15 - but note this is Python 3.10, not Python 3.9.7 as previously!
      • Now running tests with v2023-05-19 - Matplotlib issues in Alignment plotting (v3.7 vs. 3.4 previously).
    • Build 2023-07-22 now deployed to Github. Fixed to Sect. 3.6. NEW ISSUES with Plotly figure rendering however. Due to changes to setup script? Thought this had been undone, but maybe stuck in cached files.
    • Build 2023-07-30 pm now deployed to GH. Still a bit of debugging to do, but generally looking pretty good at a glance.
    • Images to DockerHub (22/08/23 rebuilds), https://hub.docker.com/r/epsproc/quantum-met-vol3
  • Tidy/remove/archive drafts

    • Build 2023-07-29_00-19-57_preDraftsCleanUp archived with all drafts.
    • TODO: remove duplicate/unnecessary files from archives, e.g. .ipynb_checkpoints etc. Lazy.
    • Part1 drafts to notes/_drafts/part1
    • Part 2 drafts to notes/_drafts/part2
    • Fitting sections still to finish and tidy.
  • Tidy scripts

  • Tidy build chain and finalise Docker build files and notes.

    • Note issues with clean PDF builds in current case, seeing data persisting in buildTmp/_latest_build/pdf and buildTmp/_latest_build/html. 30/07/23: manual reset of dir structure for testing.
    • Minor Dockerfile tidy and build test (local only, 22/08/23, see local build notes for details)
    • Added Docker build files and notes, 1b880b6
    • Images to DockerHub (22/08/23 rebuilds), https://hub.docker.com/r/epsproc/quantum-met-vol3
    • Script this for local Python installs, and also for full Docker build.
  • Installation guide in main readme, and also at end of "Platform intro" chpt. (Old version here.) UPDATED version 07/07/23, OK aside from Docker details to update.

To do:

@phockett
Copy link
Owner Author

UGH - main comment above overwritten by Firefox cached version... here's full previous text...

Final pass for most docs, hopefully, on branch reviewJuly2023, reviewJuly2023

  • Update text. Versions to new date-stamp as revised, and update TOC to reflect.
    58f3773 Adds frame defintion section (needs some work) and alignment section
    (stub).
    Theory:
    Swap section ordering? Or add note/figure in intro to this effect.
    Finish observables by type sections.
    Promote symmetry subsubsection to subsection (may also fix maths rendering issues in "photoionziation dynamics" parent subsection). UPDATE: did this anyway, but rendering issues also fixed with pkg versions in 15/07/23 builds.
    Clarify LF/AF coupling term - this currently mixes LF and MF definitions for latter case (now fixed in code, see also MF recon manuscript)?
    Add some notes on alignment? Currently introduced numerically later. (E.g. alignment tensor section). Should include some plots. NOTE: issues with Matplotlib plots in builds 19/07/23, seem to be related to MolPlot routine triggering errors (Arrow3D issues). BUT OK for <v3.6. pip install matplotlib==3.5.3 is working, now set in Dockerfile_qm3_local_install_from_base.
    This version OK in Jupyter directly in testing, but failing in book build 21/0723 - still some Matplotlib issue.
    Ah, looks like issue is Arrow3D class, needs patching for Matplotlib >v3.5, see stackoverflow.com/a/74122407 TODO: add to ePSproc with conditional patching depending on MPL version.
    Add 'glue' refs to page for builds.
    Numerics intro section. Updated 24/07/23, passable - content better, but quite a bit of overlap with Chpt. 2; some parts could move to PARTII?

    Check/test individual notebooks in current Docker build.
    Fix fig object return in epsproc.sphFromBLMPlot for Plotly case.
    Fix data.padPlot for basic case, already did this...? Currently issues with missing defaults due to (recent?) changes in code, see broken example here. Should now be fixed (in next build), see Plotting routines ePSproc#27 (comment)
    Issue in Density Mat notebook with setup script (%run '../scripts/setup_symmetry_basis_tensors.py') in 19/07/23 builds - but rest of notebook seems OK however. UPDATE: removed from script in 59edb77.
    Info content notebook now returning all zero BLMs... no idea why
    Test change in default phaseConventions
    Test change in default sumDims
    AH - likely due to addition of dipole filtering in calcs? Need to add per phockett.github.io/Quantum-Metrology-with-Photoelectrons-Vol3/part2/sym-fitting-intro_220423.html#mapping-symmetrized-harmonics-to-fit-parameters. Quick test unsuccessful, will need a more careful revisit.

    Debug HTML and PDF builds.
    Density matrix chpt. no figures in PDF builds (as of 11/07/23)? May be related to QuTip issues (see Qutip Latex matrix output #8 ), although those cells are hidden in PDF build.
    Part labelling - need to add explicitly for HTML builds, but this messes up PDF builds currently. (Which adds additional "Part" labels.). For linking to Part II in text can use {ref}Part II chpt:extracting-matrix-elements-overview``.
    Build 2023-07-22 now deployed to Github. Fixed to Sect. 3.6. NEW ISSUES with Plotly figure rendering however. Due to changes to setup script? Thought this had been undone, but maybe stuck in cached files.

    Wrap up case studies.

    General Docker builds for readers, and push to Docker hub.
    Current working version with Jupyter Stacks builds circa May 2023, issues with Jupyter Lab builds in new images (?).
    Now running tests with v2023-05-15 - but note this is Python 3.10, not Python 3.9.7 as previously!
    Now running tests with v2023-05-19 - Matplotlib issues in Alignment plotting (v3.7 vs. 3.4 previously).

    Tidy/remove/archive drafts

    Tidy scripts

    Tidy build chain and finalise Docker build files and notes.

To do:

Installation guide in main readme, and also at end of "Platform intro" chpt. (Old version here.)
    Review current Docker build.
    Script this for local Python installs, and also for full Docker build.
    General routine for locating setup-notebook.py script? Still OK to use relative paths here? Now added a note to the Installation section on this. May want to add as an appendix?
Numerical implementations/examples
    Add MF case either in outline or in detail, see pemtk.readthedocs.io/en/latest/fitting/PEMtk_fitting_demo_multi-fit_tests_130621-MFtests_120822-tidy-retest.html and pemtk.readthedocs.io/en/latest/fitting/PEMtk_fitting_backends_demo_010922.html. 24/07/23 now mentioned in passing in Sects. 2.5 and 4.1.3/4.2 (in progress).

@phockett
Copy link
Owner Author

AND AGAIN FFS

Final pass for most docs, hopefully, on branch reviewJuly2023, reviewJuly2023

Update text. Versions to new date-stamp as revised, and update TOC to reflect.
    58f3773c662e461f4c0f768f0805b99de1be2ae6 Adds frame defintion section (needs some work) and alignment section (stub).
    Theory
        Swap section ordering? Or add note/figure in intro to this effect.
        Finish observables by type sections.
        Promote symmetry subsubsection to subsection (may also fix maths rendering issues in "photoionziation dynamics" parent subsection). UPDATE: did this anyway, but rendering issues also fixed with pkg versions in 15/07/23 builds.
        [Clarify LF/AF coupling term](https://phockett.github.io/Quantum-Metrology-with-Photoelectrons-Vol3/part1/theory_tensor_formalism_201122.html#molecular-frame-projection-term-lambda) - this currently mixes LF and MF definitions for latter case (now fixed in code, see also MF recon manuscript)?
        Add some notes on alignment? Currently introduced numerically later. (E.g. [alignment tensor section](https://phockett.github.io/Quantum-Metrology-with-Photoelectrons-Vol3/part1/theory_tensor_formalism_201122.html#alignment-tensor-delta-l-m-k-q-s)). Should include some plots.
            NOTE: issues with Matplotlib plots in builds 19/07/23, seem to be related to MolPlot routine triggering errors (Arrow3D issues). BUT OK for <v3.6. pip install matplotlib==3.5.3 is working, now set in Dockerfile_qm3_local_install_from_base.
            This version OK in Jupyter directly in testing, but failing in book build 21/0723 - still some Matplotlib issue.
            Ah, looks like issue is Arrow3D class, needs patching for Matplotlib >v3.5, see [stackoverflow.com/a/74122407](https://stackoverflow.com/a/74122407) TODO: add to ePSproc with conditional patching depending on MPL version.
            Add 'glue' refs to page for builds.
    Numerics intro section. Updated 24/07/23, passable - content better, but quite a bit of overlap with Chpt. 2; some parts could move to PARTII?
    Fitting intro updating 26/07/23. NOTE: keep Matplotlib plots for now, since these render directly in both PDF and HTML. Add HV plots as optional in HTML format.

Check/test individual notebooks in current Docker build.
    Fix fig object return in epsproc.sphFromBLMPlot for Plotly case.
    Fix data.padPlot for basic case, already did this...? Currently issues with missing defaults due to (recent?) changes in code, [see broken example here](https://phockett.github.io/Quantum-Metrology-with-Photoelectrons-Vol3/part1/theory_observables_intro_211122.html). Should now be fixed (in next build), see https://github.com/phockett/ePSproc/issues/27#issuecomment-1468435262
    Issue in Density Mat notebook with setup script (%run '../scripts/setup_symmetry_basis_tensors.py') in 19/07/23 builds - but rest of notebook seems OK however. UPDATE: removed from script in 59edb777677b25f06f137421339d5bf4e971f366.
    Info content notebook now returning all zero BLMs... no idea why
        Test change in default phaseConventions
        Test change in default sumDims
        AH - likely due to addition of dipole filtering in calcs? Need to add per [phockett.github.io/Quantum-Metrology-with-Photoelectrons-Vol3/part2/sym-fitting-intro_220423.html#mapping-symmetrized-harmonics-to-fit-parameters](https://phockett.github.io/Quantum-Metrology-with-Photoelectrons-Vol3/part2/sym-fitting-intro_220423.html#mapping-symmetrized-harmonics-to-fit-parameters). Quick test unsuccessful, will need a more careful revisit.
        Now fied in 9c60eafce2c7faaa8938d32156cde9179346c265, updated setup script with full symmetry routine and tested info content and density matrix chpts.

Debug HTML and PDF builds.
    Density matrix chpt. no figures in PDF builds (as of 11/07/23)? May be related to QuTip issues (see #8 ), although those cells are hidden in PDF build. NOTE: was previously working, and do have static renders in pdf-build dir, although multiple versions, so may just need a clean/non-cached build to fix.
        Full build (no cache/force) 26/07/23 pm didn't fix. Should debug in single page build.
    MFPAD plots (Plotly) cropping after setting height=800 in plotting routine, but NOT FIXED after setting back to 1200px and clean build. Not sure what is going on there... maybe need bigger and had something cached accidentally previously? Should debug in single page build.
    Part labelling - need to add explicitly for HTML builds, but this messes up PDF builds currently. (Which adds additional "Part" labels.). For linking to Part II in text can use {ref}Part II chpt:extracting-matrix-elements-overview``.
    Outstanding issues with data.params (lmfit object) data return in [basis sets page](https://phockett.github.io/Quantum-Metrology-with-Photoelectrons-Vol3/part2/sym-fitting-intro_220423.html#mapping-to-fitting-parameters-and-reduction).
        OK in HTML for raw cell output, but no glue() success so far, not sure how to return PD object? Tested a few variants (text and HTML reprs), but no joy. See notebook for more comments, also https://github.com/phockett/Quantum-Metrology-with-Photoelectrons-Vol3/issues/6.
        AH, see also https://github.com/lmfit/lmfit-py/discussions/827. NOW added to setup script, should be OK on next build.

Wrap up case studies.
    27/07/23 consolidating general setup script. Note this is from mid-July Docker builds (runs on Fock), and still includes phase convention fix (which may be incorrect...). See https://github.com/phockett/ePSproc/issues/26#issuecomment-1640541159

General Docker builds for readers, and push to Docker hub.
    Current working version with Jupyter Stacks builds circa May 2023, issues with Jupyter Lab builds in new images (?).
    Now running tests with v2023-05-15 - but note this is Python 3.10, not Python 3.9.7 as previously!
    Now running tests with v2023-05-19 - Matplotlib issues in Alignment plotting (v3.7 vs. 3.4 previously).
    Build 2023-07-22 now deployed to Github. Fixed to Sect. 3.6. NEW ISSUES with Plotly figure rendering however. Due to changes to setup script? Thought this had been undone, but maybe stuck in cached files.

Tidy/remove/archive drafts

Tidy scripts

Tidy build chain and finalise Docker build files and notes.

To do:

Installation guide in main readme, and also at end of "Platform intro" chpt. ([Old version here](https://phockett.github.io/Quantum-Metrology-with-Photoelectrons-Vol3/part1/platform_intro_071122.html).)
    Review current Docker build.
    Script this for local Python installs, and also for full Docker build.
    General routine for locating setup-notebook.py script? Still OK to use relative paths here? Now added a note to the Installation section on this. May want to add as an appendix?
    General notes on pkg versions, and link to build versions doc (end of book), also to add. Should release current versions and match to Docker build at completion.

Numerical implementations/examples
    Add MF case either in outline or in detail, see [pemtk.readthedocs.io/en/latest/fitting/PEMtk_fitting_demo_multi-fit_tests_130621-MFtests_120822-tidy-retest.html](https://pemtk.readthedocs.io/en/latest/fitting/PEMtk_fitting_demo_multi-fit_tests_130621-MFtests_120822-tidy-retest.html) and [pemtk.readthedocs.io/en/latest/fitting/PEMtk_fitting_backends_demo_010922.html](https://pemtk.readthedocs.io/en/latest/fitting/PEMtk_fitting_backends_demo_010922.html). 24/07/23 now mentioned in passing in Sects. 2.5 and 4.1.3/4.2 (in progress).

@phockett
Copy link
Owner Author

REFERENCE CASE 27/07/23 pm

Final pass for most docs, hopefully, on branch reviewJuly2023, https://github.com/phockett/Quantum-Metrology-with-Photoelectrons-Vol3/tree/reviewJuly2023

  • Update text. Versions to new date-stamp as revised, and update TOC to reflect.

    • 58f3773 Adds frame defintion section (needs some work) and alignment section (stub).
    • Theory
      • Swap section ordering? Or add note/figure in intro to this effect.
      • Finish observables by type sections.
      • Promote symmetry subsubsection to subsection (may also fix maths rendering issues in "photoionziation dynamics" parent subsection). UPDATE: did this anyway, but rendering issues also fixed with pkg versions in 15/07/23 builds.
      • Clarify LF/AF coupling term - this currently mixes LF and MF definitions for latter case (now fixed in code, see also MF recon manuscript)?
      • Add some notes on alignment? Currently introduced numerically later. (E.g. alignment tensor section). Should include some plots.
        • NOTE: issues with Matplotlib plots in builds 19/07/23, seem to be related to MolPlot routine triggering errors (Arrow3D issues). BUT OK for <v3.6. pip install matplotlib==3.5.3 is working, now set in Dockerfile_qm3_local_install_from_base.
        • This version OK in Jupyter directly in testing, but failing in book build 21/0723 - still some Matplotlib issue.
        • Ah, looks like issue is Arrow3D class, needs patching for Matplotlib >v3.5, see stackoverflow.com/a/74122407 TODO: add to ePSproc with conditional patching depending on MPL version.
        • Add 'glue' refs to page for builds.
    • Density matrix page needs a tidy.
    • Info content page also still to do
    • Numerics intro section. Updated 24/07/23, passable - content better, but quite a bit of overlap with Chpt. 2; some parts could move to PARTII?
  • Check/test individual notebooks in current Docker build.

  • Debug HTML and PDF builds.

    • Density matrix chpt. no figures in PDF builds (as of 11/07/23)? May be related to QuTip issues (see Qutip Latex matrix output #8 ), although those cells are hidden in PDF build. NOTE: was previously working, and do have static renders in pdf-build dir, although multiple versions, so may just need a clean/non-cached build to fix.
      • Full build (no cache/force) 26/07/23 pm didn't fix. Should debug in single page build.
    • MFPAD plots (Plotly) cropping after setting height=800 in plotting routine, but NOT FIXED after setting back to 1200px and clean build. Not sure what is going on there... maybe need bigger and had something cached accidentally previously? Should debug in single page build.
    • Part labelling - need to add explicitly for HTML builds, but this messes up PDF builds currently. (Which adds additional "Part" labels.). For linking to Part II in text can use {ref}Part II chpt:extracting-matrix-elements-overview``.
    • Outstanding issues with data.params (lmfit object) data return in basis sets page.
  • Wrap up case studies.

  • General Docker builds for readers, and push to Docker hub.

    • Current working version with Jupyter Stacks builds circa May 2023, issues with Jupyter Lab builds in new images (?).
    • Now running tests with v2023-05-15 - but note this is Python 3.10, not Python 3.9.7 as previously!
    • Now running tests with v2023-05-19 - Matplotlib issues in Alignment plotting (v3.7 vs. 3.4 previously).
    • Build 2023-07-22 now deployed to Github. Fixed to Sect. 3.6. NEW ISSUES with Plotly figure rendering however. Due to changes to setup script? Thought this had been undone, but maybe stuck in cached files.
  • Tidy/remove/archive drafts

  • Tidy scripts

  • Tidy build chain and finalise Docker build files and notes.

To do:

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

No branches or pull requests

1 participant