Skip to content

Conversation

@jmperafan
Copy link

Here is a small sample of Excel functions. I am using average as an example to create (averagea, averageif, averagex), but once they have been validated, we can do the same with other aggregations.

@jmperafan jmperafan added the enhancement New feature or request label Mar 2, 2023
@jmperafan jmperafan requested a review from JCZuurmond March 2, 2023 21:15
@jmperafan jmperafan self-assigned this Mar 2, 2023
Copy link
Contributor

@JCZuurmond JCZuurmond left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, one comment on averageif

(
select avg({{ average_range }})
from {{ table }}
where range = criteria
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is nog going to work

["macro.dbt_excel.enforce_string"],
indirect=True,
)
def test_enforce_string(macro_generator: MacroGenerator, config) -> None:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jmperafan : Could you add more tests for the macros?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants