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.

Sign up for GitHub

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

Jump to bottom

ability to show markers on lines #3326

Merged
merged 5 commits into from
Aug 3, 2021
Merged

ability to show markers on lines #3326

merged 5 commits into from
Aug 3, 2021

Conversation

nicolaskruchten
Copy link
Contributor

@nicolaskruchten nicolaskruchten commented Aug 3, 2021 •
edited
Loading

Closes #2719

  • For a new feature, I have added documentation examples in an existing or
    new tutorial notebook (please see the doc checklist as well).
  • I have added a CHANGELOG entry if fixing/changing/adding anything substantial.
  • For a new feature or a change in behaviour, I have updated the relevant docstrings in the code to describe the feature or behaviour (please see the doc checklist as well).

majed282
majed282 reviewed Aug 3, 2021
View reviewed changes
Copy link

@majed282 majed282 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://guides.github.com/features/mastering-markdown/

majed282
majed282 reviewed Aug 3, 2021
View reviewed changes
Copy link

@majed282 majed282 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://guides.github.com/features/mastering-markdown/

@nicolaskruchten nicolaskruchten marked this pull request as ready for review August 3, 2021 20:46
@nicolaskruchten nicolaskruchten merged commit c67f800 into master Aug 3, 2021
@nicolaskruchten nicolaskruchten mentioned this pull request Aug 3, 2021
Closed
@archmoj archmoj deleted the px_line_marker branch November 23, 2021 23:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@majed282 majed282 majed282 left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Feature request: Simple API for lines+markers in Plotly express
2 participants
@nicolaskruchten @majed282