search

MrinalJain17 / mydia

A simple and efficient wrapper for reading videos as NumPy tensors
https://mrinaljain17.github.io/mydia/
MIT License
25 stars 3 forks source link

Feature: Option to save the processed video(s) tensor. #4

Closed MrinalJain17 closed 6 years ago

MrinalJain17 commented 6 years ago

Support to save the video tensor. Two different functionalities could be provided for this feature:

  1. Saving the tensor in .npz(or .npy) format.
  2. Writing each video back to the disk - low priority
MrinalJain17 commented 6 years ago

  1. Creating a function for saving the video tensor does not seem to be the best option. This could be achieved by the user by using the np.save or np.savez or other similar functions. A function for the same will clutter the library. But with that said, this will be referenced in the documentation in the next release.

  2. Writing videos (altered) back to the disk does make sense. But, there is certainly a lot of uncertainty:

    • What if there are multiple videos?
    • What would be the directory structure? and so on.

    Further, the user would have to again go through the process of reading videos in order to use them again, which could be time-consuming. Therefore, although this functionality could be useful, it will not be implemented in favor of the first method.