Conversation
| export default function ChartsViz({chartComp}) { | ||
| export default function ChartsViz({chartComp,setChartComp}) { | ||
| const remover = (key) => { | ||
| setChartComp(chartComp.filter((el) => el.key !==key)); |
There was a problem hiding this comment.
instead of filter use splice() to delete the chart component.
|
|
||
| export default function DataTables({datacomp, setCompIndex, setSidePlane,setDataComp}) { | ||
| const remover = (keys) => { | ||
| setDataComp(datacomp.filter((el) => el.keys !==keys)); |
There was a problem hiding this comment.
same thing here instead of filter we can delete the element totally.
There was a problem hiding this comment.
i did filter so the component can be deleted individually
There was a problem hiding this comment.
I understand but this not removing the element from the component state. can you try creating multiple charts and see if the deleting will work
src/App.js
Outdated
|
|
||
| read_csv(url, {start: 0, end: 20}).then(df => { | ||
| read_csv("https://raw.githubusercontent.com/plotly/datasets/master/finance-charts-apple.csv", {start: 0, end: 20}).then(df => { |
There was a problem hiding this comment.
remove this link and add back the default url as read_csv(URL) before pushing
There was a problem hiding this comment.
should it be capital URL?
There was a problem hiding this comment.
no the former variable before adding this
There was a problem hiding this comment.
have you resolve the error in the danfo js lib
README.md
Outdated
|
|
||
|  | ||
|
|
||
|  |
There was a problem hiding this comment.
remove  from here add it to your pull request message
|
|
No description provided.