python-neo icon indicating copy to clipboard operation
python-neo copied to clipboard

Remove neo.io.nix_fr.NixIO and neo.rawio.NixRawIO

Open samuelgarcia opened this issue 5 years ago • 2 comments

NixIo have 2 implementations:

  • one non rescritive but without lazy, the "legacy one" : neo.io.nixio.NixIO
  • one rescritive, based on rawio but with lazy capability, the "rawio one" : neo.io.nixio_fr.NixIO

Having theses two classes is pain for maintenance and also for user.

Following th idea started in #895 we should "create generic lazy object". In short lazy object not only for rawio for for all io that want to implement it.

With this we could write a proper lazy object system directly in the legacy NixIO and so we could remove the NixRawIO wich is very difficult to maintain.

samuelgarcia avatar Nov 09 '20 10:11 samuelgarcia

Something similar to this existed before, right? There was a lazy loading API for data objects that any IO could implement. Is this a similar idea?

achilleas-k avatar Nov 12 '20 09:11 achilleas-k

No. Lazy objects were not existing before rawio layer. The proposal is to extend then to non rawio also.

samuelgarcia avatar Nov 12 '20 11:11 samuelgarcia