search

BCDA-APS / mdaviz

Data visualization for mda
https://bcda-aps.github.io/mdaviz/
Other
3 stars 0 forks source link

Refactor MDAFolderTableModel #88

Closed rodolakis closed 7 months ago

rodolakis commented 7 months ago

Very inefficient:

  1. reads the MDA file 5 times to get pts/dim/pos/date/size, instead of reading it once to gather all 5 info.
  2. It also seems to have too many get/set method that might not be even used.
rodolakis commented 7 months ago

Removed most of unnecessary methods (2 above) in 3704fa2b8fd38105ba867c8c64cca8db595ff1d9

Still need to improve efficiency (1 above)

rodolakis commented 7 months ago

(1) done with https://github.com/BCDA-APS/mdaviz/commit/d086d2e5e1f06244e0ea84f3a293d410df8da374