You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
These classes were implemented before Crunch had a dedicated attribute for streaming. Now that it does the distinction of these streaming states as dedicated classes is not needed. Instead, protected methods can reference the streaming flag directly.
We do want to provide some sort of compatibility layer. Probably concentrate methods in the Dataset class and continue having Streaming and Mutable as subclasses of it, but raise a DeprecationWarning when making an instance out of them.
These classes were implemented before Crunch had a dedicated attribute for streaming. Now that it does the distinction of these streaming states as dedicated classes is not needed. Instead, protected methods can reference the streaming flag directly.
This should be done prior to working on #370.
The text was updated successfully, but these errors were encountered: