silx icon indicating copy to clipboard operation
silx copied to clipboard

Draft: silx view: add a first way to handle linux style shell character as w…

Open payno opened this issue 3 years ago • 2 comments
trafficstars

related to https://github.com/silx-kit/silx/issues/3577

TODO

  • [x] prototype to handle simple use cases
  • [ ] check file management into more details. This use case was not really expected (having several item related to the same file). I guess this could mess up will the file management (and bring serious issues with HDF5 files). Might let the file open or stop working when we remove an item. I guess we would need some kind of a counter on top of the existing system. Not sure how complicated it can become.
  • [ ] see ways to handle data_slice For now handles simple uses cases as:
  • improve management of Viewer.appendFile to handle DataUrl directly. Avoid some translation from DataUrl to str.
  • improve wild card on the file path. For now uses glob. Might be better to use fnmatch for all. But seems less direct and might require creation of another function calling os.listdir ?
  • handle slice. Today item are created but I guess when we provide slices we would like to have it automatically "selected" and displayed ? Behavior to be discussed.
clear; silx view 5.06_crayon_W150_60_Al2_W0.25_xc1000_.nx::entry/data/image*
clear; silx view 5.06_crayon_W150_60_Al2_W0.25_xc1000_.nx::entry/*
clear; silx view 5.06_crayon_W150_60_Al2_W0.25_xc*_.nx
clear; silx view 5.06_crayon_W150_60_Al2_W0.25_xc*_.nx::entry/* # !!! fails on closing

payno avatar Apr 05 '22 13:04 payno

PR #3663 improves support for * in filename and adds support for * in data path. Adding support for slicing seems more complicated since the model manages groups and datasets... so not sure what would be the best approach to support it.

t20100 avatar Sep 08 '22 14:09 t20100

PR #3677 adds support for slicing

t20100 avatar Sep 22 '22 08:09 t20100

All has been done in other PR thank to @t20100 . We can close this PR

payno avatar May 12 '23 13:05 payno