Skip to content

feat(transform): add native financial indicator dataset transform#21609

Open
Chaithanya5gif wants to merge 1 commit into
apache:masterfrom
Chaithanya5gif:feat/indicator-fix
Open

feat(transform): add native financial indicator dataset transform#21609
Chaithanya5gif wants to merge 1 commit into
apache:masterfrom
Chaithanya5gif:feat/indicator-fix

Conversation

@Chaithanya5gif
Copy link
Copy Markdown

What problem does this feature solve?

Currently, developers building financial/candlestick charts must rely on external JS libraries to calculate technical indicators before passing them to ECharts. This PR introduces a native echarts:indicator transform that computes these mathematically in the data pipeline, significantly reducing boilerplate and external dependencies.

What does the proposed API look like?

The new transform hooks seamlessly into the existing ExternalDataTransform interface:

dataset: [{
    source: stockData
}, {
    transform: {
        type: 'echarts:indicator',
        config: { 
            indicator: 'macd', // Supports: 'sma', 'ema', 'macd', 'bollinger'
            shortPeriod: 12, 
            longPeriod: 26, 
            signalPeriod: 9, 
            sourceDimension: 'Close' 
        }
    }
}]

@echarts-bot
Copy link
Copy Markdown

echarts-bot Bot commented May 9, 2026

Thanks for your contribution!
The community will review it ASAP. In the meanwhile, please checkout the coding standard and Wiki about How to make a pull request.

Please DO NOT commit the files in dist, i18n, and ssr/client/dist folders in a non-release pull request. These folders are for release use only.

To reviewers: If this PR is going to be described in the changelog in the future release, please make sure this PR has one of the following labels: PR: doc ready, PR: awaiting doc, PR: doc unchanged

This message is shown because the PR description doesn't contain the document related template.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant