Skip to content

Docs: clarify interval_unit definition and use in lookback/batch size#3613

Merged
treysp merged 4 commits intomainfrom
trey/int-unit-lookback
Jan 13, 2025
Merged

Docs: clarify interval_unit definition and use in lookback/batch size#3613
treysp merged 4 commits intomainfrom
trey/int-unit-lookback

Conversation

@treysp
Copy link
Copy Markdown
Contributor

@treysp treysp commented Jan 10, 2025

No description provided.

@treysp treysp requested review from sungchun12 and tobymao January 10, 2025 01:10
Comment thread docs/concepts/models/overview.md Outdated
Comment thread docs/concepts/models/overview.md
Comment thread docs/concepts/models/overview.md
Comment thread docs/concepts/models/overview.md Outdated
@treysp treysp force-pushed the trey/int-unit-lookback branch from 5289e16 to 2ffdfc9 Compare January 10, 2025 23:31
Comment thread docs/concepts/models/overview.md Outdated
Copy link
Copy Markdown
Contributor

@sungchun12 sungchun12 left a comment

Choose a reason for hiding this comment

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

Made a couple suggestions, and this is a welcome improvement to these config definitions. Great use of examples.

Comment thread docs/concepts/models/overview.md Outdated
Comment thread docs/concepts/models/overview.md Outdated
@treysp treysp force-pushed the trey/int-unit-lookback branch from 2ffdfc9 to 161fc39 Compare January 13, 2025 17:59
@treysp treysp force-pushed the trey/int-unit-lookback branch from 161fc39 to 6d6fab2 Compare January 13, 2025 18:00
@treysp treysp merged commit e95a604 into main Jan 13, 2025
@treysp treysp deleted the trey/int-unit-lookback branch January 13, 2025 18:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants