-
Notifications
You must be signed in to change notification settings - Fork 888
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
Image for "Basic plots - plot" should hint at markers #16
Comments
You're right. I think it might be nicer to add markers along the sine with marker and markevery. Can you make a PR? |
import matplotlib.pyplot as plt |
IMHO it's important to show independent lines and markers. |
Somewhere on the cheatsheet it's mentioned that plot is a better option for very specific types of scatter situations and I think it's OK to leave this as general case plot is for lines and scatter is for points. |
Can anyone plzzz tell me my code will work or not ...as I am new to open source contribution so I am understanding things slowly.... |
|
for smaller datasets the difference is negligible & so you may as well not overload a function with two semantically distinct (continuous versus discrete) use cases. @Raghibshams456 can you be more specific? How is your code not working? |
I argue the other way round: From the data semantics:
|
Currently the image is just sine line, which could trick users in thinking that
plot
is for lines andscatter
is for markers.I propose to additionally show another set of values with markers, e.g. something like:
The text was updated successfully, but these errors were encountered: