search

daphne-eu / daphne

DAPHNE: An Open and Extensible System Infrastructure for Integrated Data Analysis Pipelines
Apache License 2.0
67 stars 62 forks source link

`seq()` DaphneDSL built-in: Make `inc` argument optional #666

Closed pdamme closed 5 months ago

pdamme commented 9 months ago

The seq() built-in function in DaphneDSL creates an increasing sequence of numbers from a given start to a given end in increments of a given inc. In fact, the increment is very often 1, so for convenience, it would be great to make the increment an optional argument.

This can and should be done purely at the DSL level (no need for a deeper integration into DaphneDSL or the kernels). The change can be implemented in src/parser/daphnedsl/DaphneDSLBuiltins.cpp. It should be accompanied by some script-level test cases.

saminbassiri commented 5 months ago

Hi, please assign me to this issue.

pdamme commented 5 months ago

Sure, go ahead!