Note: This PR should go in after #45, if it is approved.
Goal / Problem
The adjust-interval function had an API that was designed for when we were anticipating not knowing yet how we would get the index. We know that now because we have the index on the column in tmd.
Proposed Solution
This PR does the following to the adjust-interval API:
The index column is now "auto" detected via the get-column-name-or-error fn added in #45;
The idea of "keys" (taken from R) that was previously there with the idea that we might add mimick it here has been turned into a hopefully explicit option :also-group-by in lieu of us supporting the idea of keys officially (#44);
The column name of the index column that this fn adjust can for now not be changed. This is because we currently do not yet maintain state of the :index meta carefully enough. We had a solution for this that we need to re-introduce (i.e. see @daslu's #5 )
Goal / Problem
The
adjust-interval
function had an API that was designed for when we were anticipating not knowing yet how we would get the index. We know that now because we have the index on the column in tmd.Proposed Solution
This PR does the following to the adjust-interval API:
get-column-name-or-error
fn added in #45;:also-group-by
in lieu of us supporting the idea of keys officially (#44);So usage now is like this: