plotly / plotly/plotly.js

omitted base property in index.d.ts

Ouverte
#6,810 0 commentaires 0 réactions 1 personne assignée Voir sur GitHub

Personne n'a encore pris cette issue.

bug P2
Langage dominant
JavaScript
Étoiles
18.3k
Forks
2k
Merge moyen
2 j 12 h
PR mergées (30 j)
28

Description

#  "plotly.js-dist-min": "^2.27.1",
# "@types/plotly.js-dist-min": "^2.3.4",
const minutes = Array.from({ length: 24 * 6 }, (_, i) => {
    const hour = Math.floor(i / 6)
    const minute = (i % 6) * 10
    return `2023-01-01 ${hour.toString().padStart(2, '0')}:${minute.toString().padStart(2, '0')}:00`
  })

  const traceDeath: Partial<Plotly.PlotData> = {
    x: minutes,
    base: minutes.map(() => Math.random() * 5),
    y: minutes.map(() => Math.random() * 5 + 3),
    type: 'bar',
    marker: { color: 'red' },
  }

In this code, please check base property.
I followed Plotly.PlotData
image
Whenever i omit base property, plotly graph changed.
But there is not base property in index.d.ts.

so I added interface

  interface extendPlotData extends Partial<Plotly.PlotData> {
    base?: number[]
  }
    const traceDeath: extendPlotData = {
    x: minutes,
    base: minutes.map(() => Math.random() * 5),
    y: minutes.map(() => Math.random() * 5 + 3),
    type: 'bar',
    marker: { color: 'red' },
  }

I know you have document
https://plotly.com/javascript/reference/bar/#bar-base

but please add missing properties.

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.