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

Add a few useful dict, set, and string commands #27

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

Conversation

zchen088
Copy link
Contributor

No description provided.

@tbelahi
Copy link
Contributor

tbelahi commented Jul 22, 2016

thanks for contributing.
Before we can merge, could you correct the typos in line 114 and 115 with regard to quotation marks. I think the LaTex formatting example in the strings section should be removed, it is a feature from matplotlib not of the String class. The lines on string formatting should be, at least, moved to the end of the section if not removed also. Line 114 is not complete and also maybe to specific to be in the cheat sheet. So my opinion is that in the string section, you could only add the example on string formatting and remove the rest.
In the dictionnary section, could you please rename the a2 dict to e as this is more in line with the style of the cheatsheet, and then change the following example.
For a scientific python cheatsheet, I wonder if we need to discuss sets (if yes, then we should also include tuples I think). this needs to be discussed with @MMesch.

@MMesch
Copy link
Member

MMesch commented Jul 22, 2016

Agree completely. I think we can fill up the python section until we have a full A4/A3 page in the printed version. Right now it is 2/3 full. Once we reach this limit, it is time to think about limiting the commands to the most essential ones and/or open subsections that can be added optionally.

it would be nice to have 1 full page python, 1 full page numpy soonish. I think this also gives a reasonable limit to the web interface.

@zchen088
Copy link
Contributor Author

I find sets mostly useful for the trick of checking uniqueness in a list, though I agree that it's not essential. I can remove it if you like.

I think the LaTeX formatting is actually quite useful and use it all the time when plotting. Would you be fine with it if I move it to the matplotlib section?

@tbelahi
Copy link
Contributor

tbelahi commented Jul 25, 2016

Given, @MMesch's comment, I suggest we keep the section on sets and even add a section on tuples later on, to be comprehensive when it comes to containers in Python.

The LaTeX formatting should be moved to the matplotlib section yes, it is a nice addition.

So to sum up, changes to be done before merge are:

  • keep only the basic string formatting example in the string section
  • move the latex formatting example to the matplotlib section
  • edit the examples in the dictionnary section to match the style

let's keep the section on sets as it is for now. it is a nice one.

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