Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bump mermaid from 9.2.2 to 11.4.1 #365

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

faulty13
Copy link

@faulty13 faulty13 commented Dec 2, 2024

Summary of Changes

Updated the mermaid version, based on laurent22/joplin#8728, to access new chart types.

Reason for Changes

See issue: No diagram type detected for text "quadrantChart..."

Testing

I've run into some build issues, so haven't been able to test.

One test could be:

from dash import Dash, Input, Output, html

from dash_extensions import Mermaid

chart = """
quadrantChart
    title Reach and engagement of campaigns
    x-axis Low Reach --> High Reach
    y-axis Low Engagement --> High Engagement
    quadrant-1 We should expand
    quadrant-2 Need to promote
    quadrant-3 Re-evaluate
    quadrant-4 May be improved
    Campaign A: [0.3, 0.6]
    Campaign B: [0.45, 0.23]
    Campaign C: [0.57, 0.69]
    Campaign D: [0.78, 0.34]
    Campaign E: [0.40, 0.34]
    Campaign F: [0.35, 0.78]
"""
app = Dash()
app.layout = html.Div([Mermaid(id="mermaid"), html.Button("Click me", id="trigger")])


@app.callback(Output("mermaid", "chart"), Input("trigger", "n_clicks"), prevent_initial_call=True)
def set_chart(_):
    return chart


if __name__ == "__main__":
    app.run_server(port=9998)

@faulty13
Copy link
Author

faulty13 commented Dec 4, 2024

@emilhe - just checking is there anything I need to do to progress this?

@emilhe
Copy link
Owner

emilhe commented Dec 4, 2024

I just tested the change, i.e. simply bumping the version and build everything. Something goes wrong,

image

it seems like a build technical thing to me at a glance. If you get it working, please update here.

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