[Bug] MarkLine does not render

[frilox042]

Version

5.6.0

Link to Minimal Reproduction

https://jsfiddle.net/pL8u97sz/4/

Steps to Reproduce

1. call setOption with series without markLine
2. call setOption again but spread the previous options and adds series with the markLine

chart.setOption({
  xAxis: {
    type: "value",
  },
  yAxis: {
    type: "value"
  },
  series: [{
    data: [[10,10],[20,20],[30,15],[40,10],[50,20],[60,15],],
    type: "line",
  }]
});

chart.setOption({
  ...chart.getOption(),
  series: [{
    data: [[10,10],[20,20],[30,15],[40,10],[50,20],[60,15],],
    type: "line",
    markLine: {
      data: [{ xAxis: 15 }, { xAxis:  25},]
    },
  }]
});

Current Behavior

The chart is rendered without the marklines

Expected Behavior

The chart is rendered with the marklines

Environment

- OS: linux
- Browser: firefox 139.0.4 
- Framework: react@18

Any additional comments?

Note that in my use case, I am using react.
I am updating the chart from different events.
Each does a React.useEffect where they call setOption and spread the previous option.

When I inspect the option from getOption, it has the desired configuration for the chart. Thus, I expect it render the markline

[helgasoft]

chart.getOption() is redundant in the code, remove or comment out
📌 please close issue if problem solved.

[frilox042]

Removing char.getOption() does not work in my use case.

In my minimal reproduction, the issue seemed simpler, and removing getOption() appeared to solve the problem. However, here's another reproducible example where that approach does not work: https://jsfiddle.net/d65wt98c/

Note that I'm calling setOption(option, true) with true as the second parameter.
This ensures that the restore button only resets the zoom state.

Without the true parameter, the restore action also reverts the data to a previous state. Specifically, a state where not all the data in the series is present.

[helgasoft]

There may be a bug when mixing newly defined series with series from getOption on replace.
One workaround is to keep track of the original option object and just update it before replace - Demo.

[frilox042]

Thanks, I got workaround for my case

In react, it does not feel natural to keep track of the ref of original option.
Usually, I rebuild the object by spreading previous values in a new one.
I guess in this case, it is ok to work with the ref.
The rendering is done by echarts, not by react.

I will keep this in mind in my adventure with echarts.

---

There may be a bug when mixing newly defined series with series from getOption on replace.

Should we close this issue or keep it open until there is a fix ?

I would like to contribute to fix it myself, but I feel very lost within the codebase of echarts

[helgasoft]

Should we close this issue or keep it open until there is a fix ?

Better keep it open.

I would like to contribute to fix it myself, but I feel very lost within the codebase of echarts

You are not alone for sure 🤖🤔🤪