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

Wiki: Some improvements to Development Workflow Wiki #14728

Open
htrivedi04 opened this issue May 20, 2018 · 1 comment
Open

Wiki: Some improvements to Development Workflow Wiki #14728

htrivedi04 opened this issue May 20, 2018 · 1 comment

Comments

@htrivedi04
Copy link

I was following the Development Workflow Wiki page as I'm new to contributing to this project. It is a good source of knowledge for newbies. However, I found some outdated / misleading information in the Development Workflow Wiki page, which can be fixed easily -

  • In the Set up Git - Clone SymPy section - The download size of SymPy should now be 88.5MB instead of 77MB.
  • In the Setup Virtual Environments section - The conda create -n sympy-dev-py27 python=2.7 mpmath should also ideally include numpy package.
  • In the Be sure that all tests pass section - A little info on what 's', 'f' and 'w' signify in the test case results would be good for new contributors.

I can fix these if you approve of these, although I might need some information about the last point. Let me know what suits you best.

@asmeurer
Copy link
Member

Feel free to edit the wiki.

Note that numpy is only required for a very small subset of SymPy (lambdify basically). But it should be clear there that you should add whatever other packages you need, like ipython, jupyter, matplotlib, and so on. Or you can install them in the environment after you activate it with conda install jupyter and so on.

Regarding the tests, s means the test is skipped (you can see the reason using --verbose. F means the test failed. f means the test failed, but it was expected to fail (XFAIL). wmeans the test was skipped because it is too slow (it can be run with--slow). X` means the test was expected to fail (XFAIL), but it passed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants