FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

LIMS-2228: Move hmap and caman out of vendors folder by ndg63276 · Pull Request #1088 · DiamondLightSource/SynchWeb · GitHub

LIMS-2228: Move hmap and caman out of vendors folder - #1088

Merged
ndg63276 merged 1 commit into
pre-release/2026-R4.1from
improvement/LIMS-2228/move-js-libs-into-package.json
Aug 18, 2026
Merged

ndg63276 merged 1 commit into
pre-release/2026-R4.1from
improvement/LIMS-2228/move-js-libs-into-package.json

Conversation

Copy link
Copy Markdown
Collaborator

JIRA ticket: LIMS-2228

Summary:

There are a few JS libraries manually inserted into the vendors folder, they should be moved to package.json if possible.

Changes:

  • Remove heatmap file, remove alias, add heatmap.js to package.json, update imports
  • Remove caman file, remove alias, add caman-dist-only to package.json, update imports and manually declare Caman
  • Don't install http or fs as part of caman, update imports-loader so caman knows we are running in a browser
  • Remove backbone.syphon file, update imports (backbone.syphon was already in package.json)
  • Remove unused luxon import, luxon was removed in LIMS-2044: Remove luxon #1022

To test:

  • heatmap.js:
    • Go to a container with grid scans (eg /containers/cid/379691, well B8d2), check the results heatmap is still displayed
    • Go to a recent grid scan (eg /dc/visit/cm44138-3/id/24524776), check the results heatmap is still displayed
  • caman:
    • Check the diffraction viewer still works on the grid scan
    • Go to a rotation data collection (eg /dc/visit/cm44138-3/id/24524800), check the diffraction image viewer still works
  • backbone.syphon
    • Go to a SAXS proposal, eg cm44173, check you can still make a shipment and a puck within it (you may need to make a protein first)

gfrn commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator

Looks good to me, but we can probably get rid of Caman (as in Evo) since we can now manipulate diffraction images with CSS

ndg63276 changed the base branch from master to pre-release/2026-R4.1 August 18, 2026 09:33
ndg63276 merged commit 324a408 into pre-release/2026-R4.1 Aug 18, 2026
2 checks passed
ndg63276 deleted the improvement/LIMS-2228/move-js-libs-into-package.json branch August 18, 2026 09:34
ndg63276 added a commit that referenced this pull request Sep 7, 2026
* LIMS-2282: Use the same callback URL for both incoming and outgoing dewars (#1087)

* LIMS-2228: Move hmap and caman out of vendors folder (#1088)

* [LIMS-2323] Fix user info endpoint in OIDC (#1091)

Co-authored-by: Guilherme Francisco <guilherme.de-freitas@diamond.ac.uk>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL