-
Notifications
You must be signed in to change notification settings - Fork 59
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Domains lost when using TimeSeries.operation method #169
Comments
Ah I see that domains have been removed for some reason on master (making this somewhat moot). Searching around I couldn't seem to find a reason for this. Could someone provide a reference or reason? |
Hi, thanks for opening the issue. We removed domains because people were finding them confusing, and we were finding that they were rarely used. It didn't seem like the minimal value they were adding was worth the confusion in explaining them. Would you mind sharing a more complete use case of how you are using domains and why they are useful? (and perhaps how ugly it would be without them) |
@stringertheory I am developing a library py-stl for working within Metric Temporal Logic and Signal Temporal Logic. These are often used to determine if a time series signal meets some specification (for filtering perhaps). For example, one may write
My library would take the two signals For example, if It's worth noting that I made the design decision to make |
Minimal Example using traces v0.3.1
It seems to me that
y
's domain should still be(0, 20)
. From an API standpoint, if applying a binary operation onx1
andx2
, I think that it'd be simpler to have the resulting domain bex1.domain & x2.domain
The text was updated successfully, but these errors were encountered: