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

Support Lazy Loading for Large Signals. #267

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

CSSFrancis
Copy link
Member

@CSSFrancis CSSFrancis commented Aug 1, 2024

I was going to go through and make this better for large signals so that you could potentially use this as a way to view and check 4D STEM data. That requires lazy loading to be an option.

This also really would benefit from better saving of navigator signals.

To Do:

  • virtual imaging (non live/ add compute button)
  • better display compute bar to know how long computation should take
  • Fix zspy file loading
  • Fix 5D sliders being too small on Mac (test on windows)

@ericpre
Copy link
Member

ericpre commented Aug 3, 2024

(I thought that I already commented here but I must have lost it somehow so I am having another go!)

I am wondering if it would be possible to support reading zspy through the existing "open" action - I guess at the moment, it doesn't work because it is a folder and not a file, but this could be workaround?
This would have the advantage of also easily supporting the drag and drop functionality for zspy?

Regarding loading lazily, would it be worth setting preference setting along a modal dialog that asks what to do the first time being used with the drag and drop functionality?

Can we do separate PRs for the different items of the list, please?

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.

2 participants