Skip to content

LUMO Toolbox: Introducing a loading function for 2D layouts and Improving Graph Responsiveness#24

Open
Giuseppe-1993 wants to merge 11 commits into
DOT-HUB:masterfrom
Giuseppe-1993:master
Open

LUMO Toolbox: Introducing a loading function for 2D layouts and Improving Graph Responsiveness#24
Giuseppe-1993 wants to merge 11 commits into
DOT-HUB:masterfrom
Giuseppe-1993:master

Conversation

@Giuseppe-1993
Copy link
Copy Markdown

@Giuseppe-1993 Giuseppe-1993 commented May 31, 2024

This version of the DOTHUB toolbox introduces a series of new features in the LUMO_toolbox subfolder. These are:

  1. Loading: A load button has been added to the bottom-left corner of the GUI, enabling the user to load an existing 2D layout in .SD format. A function called Load2Dfile handles the following:
  • It presents a warning dialog box if there is a mismatch between the specified number of tiles and the number contained in the loaded file, prompting the user to choose whether to proceed.
  • The 2D positions of the tiles are estimated from the sources and detectors array available in the loaded file. The rotation is handled by a separate function called DOTHUB_LUMOcomputeRotations. Simple trigonometry is used to estimate the angle between the initial vector (from the centre of the tile to source A) and the loaded one (from the centre of the tile to the rotated source A).
  1. Responsiveness: Based on user feedback, the text for sources and detectors (e.g., S1, S2, …) has been removed. Instead, for clarity, the number of each tile is introduced. This change improves the responsiveness of the graph, allowing users to zoom in and move around the graph more easily.

LumoUser and others added 11 commits May 16, 2024 14:38
…ious 2DSD layouts.

- Remove the sources and detectors text (e.g. 'S1') to make the graph more responsive. 

- Add number of tile to provide useful information to the user.
Adding SLG options and associated offset calculations
Change variable names
Add options for sprung light guides and calculate the offset
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

Successfully merging this pull request may close these issues.

1 participant