Repository: einsteinpy/einsteinpy

Issue:
Color options in ScatterGeodesicPlotter and StaticGeodesicPlotter is missing.
<!--

WELCOME ABOARD!

Hi and welcome to the einsteinpy project. We appreciate bug reports, questions
about documentation, and suggestions for new features.

IMPORTANT: If you are reporting a problem with einsteinpy, please follow the
template below. If it's a suggestion, a possible new addition to the library,
or just want to say "Thank you!", you can remove all this text and start
from scratch.

-->
<!--- Provide a general summary of the issue in the Title above -->

🐞 **What bugs you?**
Color options for attractor and scatter plots are not present in the classes ScatterGeodesicPlotter and StaticGeodesicPlotter.

🎯 **Goal**
This change can benefit people in the way that if they want to create many plots, It can help them in distinguishing a certain plot in a group of plots. Also it gives user to option to use his/her favorite color.
<!--- Why is this change important to you? How would you use it? -->
<!--- How can it benefit other users? -->

💡 **Possible solutions**
The possible solution is to use a parameter for the user to pass the color while making the class. If the user does not want to pass the parameter, we can default it to some color.
<!--- Not obligatory, but suggest an idea for implementing addition or change -->

📋  **Steps to solve the problem**
I have created and tested the class with added parameters. Will create a PR after writing this issue.
 * Comment below about what you've started working on.
 * Add, commit, push your changes
 * Submit a pull request and add this in comments - `Addresses #<put issue number here>`
 * Ask for a review in comments section of pull request
 * Celebrate your contribution to this project 🎉


Discussion:
shreyasbapat: You have to get the quality checks passed first. I will review the PR thoroughly after that! :)
codecov[bot]: # [Codecov](https://codecov.io/gh/einsteinpy/einsteinpy/pull/157?src=pr&el=h1) Report
> Merging [#157](https://codecov.io/gh/einsteinpy/einsteinpy/pull/157?src=pr&el=desc) into [master](https://codecov.io/gh/einsteinpy/einsteinpy/commit/6704bdb34665472d61f581f3eda62edc257fdcb3?src=pr&el=desc) will **increase** coverage by `0.02%`.
> The diff coverage is `100%`.

[![Impacted file tree graph](https://codecov.io/gh/einsteinpy/einsteinpy/pull/157/graphs/tree.svg?width=650&token=H3oDVYA11f&height=150&src=pr)](https://codecov.io/gh/einsteinpy/einsteinpy/pull/157?src=pr&el=tree)

```diff
@@            Coverage Diff             @@
##           master     #157      +/-   ##
==========================================
+ Coverage   92.35%   92.37%   +0.02%     
==========================================
  Files          18       18              
  Lines         850      853       +3     
==========================================
+ Hits          785      788       +3     
  Misses         65       65
```


| [Impacted Files](https://codecov.io/gh/einsteinpy/einsteinpy/pull/157?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [src/einsteinpy/plotting/geodesics\_static.py](https://codecov.io/gh/einsteinpy/einsteinpy/pull/157/diff?src=pr&el=tree#diff-c3JjL2VpbnN0ZWlucHkvcGxvdHRpbmcvZ2VvZGVzaWNzX3N0YXRpYy5weQ==) | `100% <100%> (ø)` | :arrow_up: |

------

[Continue to review full report at Codecov](https://codecov.io/gh/einsteinpy/einsteinpy/pull/157?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/einsteinpy/einsteinpy/pull/157?src=pr&el=footer). Last update [6704bdb...0a9afe7](https://codecov.io/gh/einsteinpy/einsteinpy/pull/157?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).

codecov[bot]: # [Codecov](https://codecov.io/gh/einsteinpy/einsteinpy/pull/157?src=pr&el=h1) Report
> Merging [#157](https://codecov.io/gh/einsteinpy/einsteinpy/pull/157?src=pr&el=desc) into [master](https://codecov.io/gh/einsteinpy/einsteinpy/commit/6704bdb34665472d61f581f3eda62edc257fdcb3?src=pr&el=desc) will **increase** coverage by `0.02%`.
> The diff coverage is `100%`.

[![Impacted file tree graph](https://codecov.io/gh/einsteinpy/einsteinpy/pull/157/graphs/tree.svg?width=650&token=H3oDVYA11f&height=150&src=pr)](https://codecov.io/gh/einsteinpy/einsteinpy/pull/157?src=pr&el=tree)

```diff
@@            Coverage Diff             @@
##           master     #157      +/-   ##
==========================================
+ Coverage   92.35%   92.37%   +0.02%     
==========================================
  Files          18       18              
  Lines         850      853       +3     
==========================================
+ Hits          785      788       +3     
  Misses         65       65
```


| [Impacted Files](https://codecov.io/gh/einsteinpy/einsteinpy/pull/157?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [src/einsteinpy/plotting/geodesics\_static.py](https://codecov.io/gh/einsteinpy/einsteinpy/pull/157/diff?src=pr&el=tree#diff-c3JjL2VpbnN0ZWlucHkvcGxvdHRpbmcvZ2VvZGVzaWNzX3N0YXRpYy5weQ==) | `100% <100%> (ø)` | :arrow_up: |

------

[Continue to review full report at Codecov](https://codecov.io/gh/einsteinpy/einsteinpy/pull/157?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/einsteinpy/einsteinpy/pull/157?src=pr&el=footer). Last update [6704bdb...0a9afe7](https://codecov.io/gh/einsteinpy/einsteinpy/pull/157?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).

shreyasbapat: The main aim of #150 was that the way the plots are made must be pleasing. As in your pull request, you can see that the attractor has a very large radius, which is wrong. There should be some strategy to keep radius of object according to the plot!
rtanmay: @shreyasbapat  Alright will fix the radius dynamically according to the dimensions of the plot. I think having a manual scaling option also will be good in case the user wants to adjust it to his/her needs.
shreyasbapat: > I think having a manual scaling option also will be good in case the user wants to adjust it to his/her needs.

Totally agreed!
shreyasbapat: @rtanmay Updates?
rtanmay: @shreyasbapat  I have written the method of auto scaling but there is a bug in that. It's almost done. Will create a PR sometime around 6AM.
rtanmay: @shreyasbapat are these proportions good?
![Screenshot from 2019-04-11 01-43-09](https://user-images.githubusercontent.com/34454817/55910155-57d4e600-5bfb-11e9-822a-2053309db6ea.png)

rtanmay: All done now. This is how it looks:
![Screenshot from 2019-04-11 04-12-31](https://user-images.githubusercontent.com/34454817/55918358-71ccf380-5c10-11e9-8311-e3abef12ad7c.png)

can you check @shreyasbapat now?
Thanks
shreyasbapat: @rtanmay Do you see the attractor going beyond the orbit? That shouldn't happen
Varunvaruns9: @rtanmay Any updates on this? Its a blocker for #189 .
rtanmay: @shreyasbapat Is this fine now?
![Screenshot from 2019-04-13 03-41-54](https://user-images.githubusercontent.com/34454817/56068899-36f8c600-5d9e-11e9-89b0-0f09be5f3a1f.png)
![Screenshot from 2019-04-13 03-41-30](https://user-images.githubusercontent.com/34454817/56068900-36f8c600-5d9e-11e9-821a-64bf79dcc132.png)

@Varunvaruns9 If shreyas says its fine now then I will just write the test for code coverage to clear them. Won't take much time.
shreyasbapat: Can you plot earth once?
rtanmay: @shreyasbapat 

Earth around sun
![Screenshot from 2019-04-13 14-50-32](https://user-images.githubusercontent.com/34454817/56077582-9638f380-5dfb-11e9-9e48-32f7ce20efab.png)


pep8speaks: Hello @rtanmay! Thanks for updating this PR. We checked the lines you've touched for [PEP 8](https://www.python.org/dev/peps/pep-0008) issues, and found:

* In the file [`src/einsteinpy/plotting/geodesics_scatter.py`](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/plotting/geodesics_scatter.py):

> [Line 14:22](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/plotting/geodesics_scatter.py#L14): [W291](https://duckduckgo.com/?q=pep8%20W291) trailing whitespace
> [Line 17:17](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/plotting/geodesics_scatter.py#L17): [W291](https://duckduckgo.com/?q=pep8%20W291) trailing whitespace
> [Line 22:20](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/plotting/geodesics_scatter.py#L22): [W291](https://duckduckgo.com/?q=pep8%20W291) trailing whitespace

* In the file [`src/einsteinpy/plotting/geodesics_static.py`](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/plotting/geodesics_static.py):

> [Line 18:55](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/plotting/geodesics_static.py#L18): [W291](https://duckduckgo.com/?q=pep8%20W291) trailing whitespace
> [Line 22:75](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/plotting/geodesics_static.py#L22): [W291](https://duckduckgo.com/?q=pep8%20W291) trailing whitespace
> [Line 23:78](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/plotting/geodesics_static.py#L23): [W291](https://duckduckgo.com/?q=pep8%20W291) trailing whitespace
> [Line 25:19](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/plotting/geodesics_static.py#L25): [W291](https://duckduckgo.com/?q=pep8%20W291) trailing whitespace
> [Line 32:20](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/plotting/geodesics_static.py#L32): [W291](https://duckduckgo.com/?q=pep8%20W291) trailing whitespace
> [Line 38:43](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/plotting/geodesics_static.py#L38): [W291](https://duckduckgo.com/?q=pep8%20W291) trailing whitespace
> [Line 39:49](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/plotting/geodesics_static.py#L39): [W291](https://duckduckgo.com/?q=pep8%20W291) trailing whitespace
> [Line 41:22](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/plotting/geodesics_static.py#L41): [W291](https://duckduckgo.com/?q=pep8%20W291) trailing whitespace
> [Line 47:20](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/plotting/geodesics_static.py#L47): [W291](https://duckduckgo.com/?q=pep8%20W291) trailing whitespace
> [Line 49:1](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/plotting/geodesics_static.py#L49): [E101](https://duckduckgo.com/?q=pep8%20E101) indentation contains mixed spaces and tabs
> [Line 49:1](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/plotting/geodesics_static.py#L49): [W191](https://duckduckgo.com/?q=pep8%20W191) indentation contains tabs
> [Line 49:1](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/plotting/geodesics_static.py#L49): [W293](https://duckduckgo.com/?q=pep8%20W293) blank line contains whitespace
> [Line 50:1](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/plotting/geodesics_static.py#L50): [E101](https://duckduckgo.com/?q=pep8%20E101) indentation contains mixed spaces and tabs
> [Line 50:14](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/plotting/geodesics_static.py#L50): [W291](https://duckduckgo.com/?q=pep8%20W291) trailing whitespace
> [Line 54:27](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/plotting/geodesics_static.py#L54): [W291](https://duckduckgo.com/?q=pep8%20W291) trailing whitespace
> [Line 125:16](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/plotting/geodesics_static.py#L125): [W291](https://duckduckgo.com/?q=pep8%20W291) trailing whitespace
> [Line 128:31](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/plotting/geodesics_static.py#L128): [W291](https://duckduckgo.com/?q=pep8%20W291) trailing whitespace
> [Line 130:26](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/plotting/geodesics_static.py#L130): [W291](https://duckduckgo.com/?q=pep8%20W291) trailing whitespace
> [Line 132:25](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/plotting/geodesics_static.py#L132): [W291](https://duckduckgo.com/?q=pep8%20W291) trailing whitespace
> [Line 154:27](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/plotting/geodesics_static.py#L154): [W291](https://duckduckgo.com/?q=pep8%20W291) trailing whitespace

* In the file [`src/einsteinpy/tests/test_plotting/test_scattergeodesicplotter.py`](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/tests/test_plotting/test_scattergeodesicplotter.py):

> [Line 10:33](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/tests/test_plotting/test_scattergeodesicplotter.py#L10): [W291](https://duckduckgo.com/?q=pep8%20W291) trailing whitespace
> [Line 25:46](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/tests/test_plotting/test_scattergeodesicplotter.py#L25): [W291](https://duckduckgo.com/?q=pep8%20W291) trailing whitespace
> [Line 27:1](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/tests/test_plotting/test_scattergeodesicplotter.py#L27): [E302](https://duckduckgo.com/?q=pep8%20E302) expected 2 blank lines, found 0
> [Line 34:1](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/tests/test_plotting/test_scattergeodesicplotter.py#L34): [E305](https://duckduckgo.com/?q=pep8%20E305) expected 2 blank lines after class or function definition, found 1
> [Line 50:1](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/tests/test_plotting/test_scattergeodesicplotter.py#L50): [E305](https://duckduckgo.com/?q=pep8%20E305) expected 2 blank lines after class or function definition, found 1
> [Line 65:1](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/tests/test_plotting/test_scattergeodesicplotter.py#L65): [E305](https://duckduckgo.com/?q=pep8%20E305) expected 2 blank lines after class or function definition, found 1
> [Line 97:1](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/tests/test_plotting/test_scattergeodesicplotter.py#L97): [E305](https://duckduckgo.com/?q=pep8%20E305) expected 2 blank lines after class or function definition, found 1
> [Line 112:1](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/tests/test_plotting/test_scattergeodesicplotter.py#L112): [W293](https://duckduckgo.com/?q=pep8%20W293) blank line contains whitespace
> [Line 112:1](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/tests/test_plotting/test_scattergeodesicplotter.py#L112): [W391](https://duckduckgo.com/?q=pep8%20W391) blank line at end of file

* In the file [`src/einsteinpy/tests/test_plotting/test_staticgeodesicplotter.py`](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/tests/test_plotting/test_staticgeodesicplotter.py):

> [Line 13:33](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/tests/test_plotting/test_staticgeodesicplotter.py#L13): [W291](https://duckduckgo.com/?q=pep8%20W291) trailing whitespace
> [Line 30:1](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/tests/test_plotting/test_staticgeodesicplotter.py#L30): [E302](https://duckduckgo.com/?q=pep8%20E302) expected 2 blank lines, found 0
> [Line 71:1](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/tests/test_plotting/test_staticgeodesicplotter.py#L71): [E305](https://duckduckgo.com/?q=pep8%20E305) expected 2 blank lines after class or function definition, found 1
> [Line 87:1](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/tests/test_plotting/test_staticgeodesicplotter.py#L87): [E305](https://duckduckgo.com/?q=pep8%20E305) expected 2 blank lines after class or function definition, found 1
> [Line 102:1](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/tests/test_plotting/test_staticgeodesicplotter.py#L102): [E305](https://duckduckgo.com/?q=pep8%20E305) expected 2 blank lines after class or function definition, found 1
> [Line 116:1](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/tests/test_plotting/test_staticgeodesicplotter.py#L116): [W293](https://duckduckgo.com/?q=pep8%20W293) blank line contains whitespace
> [Line 132:45](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/tests/test_plotting/test_staticgeodesicplotter.py#L132): [W291](https://duckduckgo.com/?q=pep8%20W291) trailing whitespace
> [Line 134:1](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/tests/test_plotting/test_staticgeodesicplotter.py#L134): [E305](https://duckduckgo.com/?q=pep8%20E305) expected 2 blank lines after class or function definition, found 1
> [Line 150:1](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/tests/test_plotting/test_staticgeodesicplotter.py#L150): [E305](https://duckduckgo.com/?q=pep8%20E305) expected 2 blank lines after class or function definition, found 1
> [Line 166:1](https://github.com/einsteinpy/einsteinpy/blob/d7e0b6597366476f4b901712e44440a77dd8c560/src/einsteinpy/tests/test_plotting/test_staticgeodesicplotter.py#L166): [E305](https://duckduckgo.com/?q=pep8%20E305) expected 2 blank lines after class or function definition, found 1


shreyasbapat: Can you rebase once? Just ignore the @pep8speaks  Comment for now :)
rtanmay: @shreyasbapat  Yes, will do that after clearing the coverage checks once.
rtanmay: Animate checks are failing after rebasing.
![Screenshot from 2019-04-15 20-05-19](https://user-images.githubusercontent.com/34454817/56141337-42c0d400-5fba-11e9-8bd9-9229debf7a94.png)

I tried adding mock patch since I thought it was not there but even that didn't solve the problem.
Can you check? @Varunvaruns9 @shreyasbapat 

shreyasbapat: Hi @rtanmay are you still working on this? Remember #177 depends on this!
rtanmay: @shreyasbapat  Yes, I am working on this. Was really busy for the past 2 days. Will fix it by today's night or by early Saturday.
rtanmay: While rebasing, something went wrong. Other's commits are also coming.
@shreyasbapat Don't review the PR now, I will let you know once the above change is reverted.
rtanmay: @shreyasbapat 
I think all the changes are done now. Can you check? Thanks
shreyasbapat: Also, I would ask you to restrict the commits to <10 or as you already know git, a number which seems reasonable to you :)
rtanmay: Yes, I will fixup the commits once I know the PR is finally ready.
shreyasbapat: This will also need a rebase!
rtanmay: > Also, I would ask you to restrict the commits to <10 or as you already know git, a number which seems reasonable to you :)

Okay, I think 1 commit per PR is good enough :)
shreyasbapat: You can write a test without mocking!
shreyasbapat: > Okay, I think 1 commit per PR is good enough :)

I can allow upto 10


Patch:
diff --git a/docs/source/api/plotting/geodesics_scatter.rst b/docs/source/api/plotting/geodesics_scatter.rst
new file mode 100644
index 0000000..ccef8ad
--- /dev/null
+++ b/docs/source/api/plotting/geodesics_scatter.rst
@@ -0,0 +1,31 @@
+Static 2D Plotting module
+=========================
+
+This module contains the basic classes for static plottings in
+2-dimensions for scatter and line:
+
+.. automodule:: einsteinpy.plotting.geodesics_scatter
+    :members:
+
+Color
+=====
+* Attractor : 
+    User can give the color to attractor of his/her choice.
+    It can be passed while making the object of geodesics_static class.
+    Default color of attractor is "black".
+
+    .. code-block:: python
+        
+        self.attractor_color = attractor_color
+        plt.scatter(0, 0, color=self.attractor_color)
+
+* Geodesic : 
+    User can give the color to the orbit of the particle moving around the attractor of his/her choice.
+    It can be passed while making the object of geodesics_scatter class.
+    Default color is "Oranges".
+    
+    .. code-block:: python
+        
+        self.cmap_color = cmap_color
+        plt.scatter(pos_x, pos_y, s=1, c=time, cmap=self.cmap_color)
+        
\ No newline at end of file
diff --git a/docs/source/api/plotting/geodesics_static.rst b/docs/source/api/plotting/geodesics_static.rst
index cff12c0..6c5285f 100644
--- a/docs/source/api/plotting/geodesics_static.rst
+++ b/docs/source/api/plotting/geodesics_static.rst
@@ -6,3 +6,60 @@ This module contains the basic classes for static plottings in
 
 .. automodule:: einsteinpy.plotting.geodesics_static
     :members:
+
+Auto and Manual scaling
+=======================
+EinsteinPy supports Automatic and Manual scaling of the attractor to make plots look better since radius of 
+attractor can be really small and not visible.
+
+* Manual_Scaling : 
+    If the user provides the attractor_radius_scale, then the autoscaling will not work.
+    This is checked by initialising the  attractor_radius_scale by -1 and if the user enters the value then 
+    it will be >0 so the value won't remain -1 which is easily checked.
+    
+    The radius is multiplied to the value given in attractor_radius_scale
+    
+    .. code-block:: python
+
+        radius = radius * self.attractor_radius_scale
+
+* Auto Scaling : 
+    If the user does not provide the attractor_radius_scale, the value will be initialised to -1 and 
+    then we will call the auto scaling function. In autoscaling, the attractor radius is first 
+    initialised to the minimum distance between the attractor and the object moving around it. 
+    Now, if this radius is greater than the 1/12th of minimum of range of X and Y coordinates then,
+    the radius is initialised to this minimum. This is done so that the plots are easy to look at.
+
+    minrad_nooverlap : Stores the minimum distance between the particle and attractor
+    .. code-block:: python
+
+        for i in range(0, len(self.xarr)):
+            minrad_nooverlap = min( minrad_nooverlap, self.mindist(self.xarr[i], self.yarr[i]))
+
+    minlen_plot : Stores the minimum of range of X and Y axis
+    
+    .. code-block:: python
+            
+            xlen = max(self.xarr) - min(self.xarr)
+            ylen = max(self.yarr) - min(self.yarr)
+            minlen_plot = min(xlen, ylen)
+    
+    multiplier : Stores the value which is multiplied to the radius to make it 1/12th of the minlenplot
+    
+    .. code-block:: python
+            
+            mulitplier = minlen_plot / (12 * radius)
+            min_radius = radius * mulitplier
+
+
+Attractor Color
+===============
+* Color Options : 
+    User can give the color to attractor of his/her choice.
+    It can be passed while calling the geodesics_static class.
+    Default color of attractor is "black".
+
+    .. code-block:: python
+        
+        self.attractor_color = attractor_color
+        mpl.patches.Circle( (0, 0), radius.value, lw=0, color=self.attractor_color)
diff --git a/docs/source/api/plotting/plotting_index.rst b/docs/source/api/plotting/plotting_index.rst
index d8dce95..8a841b4 100644
--- a/docs/source/api/plotting/plotting_index.rst
+++ b/docs/source/api/plotting/plotting_index.rst
@@ -8,3 +8,4 @@ This module contains the basic classes for static and interactive
     :maxdepth: 2
 
     geodesics_static
+    geodesics_scatter
diff --git a/src/einsteinpy/plotting/geodesics_scatter.py b/src/einsteinpy/plotting/geodesics_scatter.py
index 2475c16..d2a89d0 100644
--- a/src/einsteinpy/plotting/geodesics_scatter.py
+++ b/src/einsteinpy/plotting/geodesics_scatter.py
@@ -11,14 +11,28 @@ class ScatterGeodesicPlotter:
     Class for plotting static matplotlib plots.
     """
 
-    def __init__(self, mass, time=0 * u.s):
+    def __init__(
+        self, mass, time=0 * u.s, attractor_color="black", cmap_color="Oranges"
+    ):
+        """
+
+        Parameters
+        ----------
+        attractor_color : string, optional
+            Color which is used to denote the attractor. Defaults to black.
+        cmap_color : string, optional
+            Color used in function plot.
+
+        """
         self.mass = mass
         self.time = time
         self._attractor_present = False
+        self.attractor_color = attractor_color
+        self.cmap_color = cmap_color
 
     def _plot_attractor(self):
         self._attractor_present = True
-        plt.scatter(0, 0, color="black")
+        plt.scatter(0, 0, color=self.attractor_color)
 
     def plot(self, pos_vec, vel_vec, end_lambda=10, step_size=1e-3):
         """
@@ -51,7 +65,7 @@ class ScatterGeodesicPlotter:
         pos_x = r * np.cos(phi)
         pos_y = r * np.sin(phi)
 
-        plt.scatter(pos_x, pos_y, s=1, c=time, cmap="Oranges")
+        plt.scatter(pos_x, pos_y, s=1, c=time, cmap=self.cmap_color)
 
         if not self._attractor_present:
             self._plot_attractor()
diff --git a/src/einsteinpy/plotting/geodesics_static.py b/src/einsteinpy/plotting/geodesics_static.py
index cb8b7f0..a1d476d 100644
--- a/src/einsteinpy/plotting/geodesics_static.py
+++ b/src/einsteinpy/plotting/geodesics_static.py
@@ -1,4 +1,5 @@
 import random
+import sys
 
 import astropy.units as u
 import matplotlib as mpl
@@ -11,18 +12,56 @@ from einsteinpy.utils import schwarzschild_radius
 
 class StaticGeodesicPlotter:
     """
-    Class for plotting static matplotlib plots
+    Class for plotting static matplotlib plots.
     """
 
-    def __init__(self, mass, time=0 * u.s, ax=None):
+    def __init__(
+        self,
+        mass,
+        time=0 * u.s,
+        ax=None,
+        attractor_radius_scale=-1.0,
+        attractor_color="#ffcc00",
+    ):
+        """
+
+        Parameters
+        ----------
+        attractor_radius_scale : float, optional
+            Scales the attractor radius by the value given. Default is 1. It is used to make plots look more clear if needed.
+        attractor_color : string, optional
+            Color which is used to denote the attractor. Defaults to #ffcc00.
+
+        """
         self.ax = ax
         if not self.ax:
             _, self.ax = plt.subplots(figsize=(6, 6))
         self.time = time
         self.mass = mass
         self._attractor_present = False
+        self.attractor_color = attractor_color
+        self.attractor_radius_scale = attractor_radius_scale
+        self.__xarr = np.array([])
+        self.__yarr = np.array([])
+        self.get_curr_plot_radius = 0
 
     def plot_trajectory(self, pos_vec, vel_vec, end_lambda, step_size, color):
+        """
+
+        Parameters
+        ----------
+        pos_vec : list
+            list of r, theta & phi components along with ~astropy.units.
+        vel_vec : list
+            list of velocities of r, theta & phi components along with ~astropy.units.
+        end_lambda : float, optional
+            Lambda where iteartions will stop.
+        step_size : float, optional
+            Step size for the ODE.
+        color : string
+            Color of the Geodesic
+
+        """
         swc = Schwarzschild.from_spherical(pos_vec, vel_vec, self.time, self.mass)
 
         vals = swc.calculate_trajectory(
@@ -45,10 +84,69 @@ class StaticGeodesicPlotter:
         if not self._attractor_present:
             self._draw_attractor()
 
-    def _draw_attractor(self, min_radius=10 * u.km):
-        radius = max((schwarzschild_radius(self.mass) * 10000), min_radius.to(u.km))
-        color = "#ffcc00"
-        self.ax.add_patch(mpl.patches.Circle((0, 0), radius.value, lw=0, color=color))
+    def __get_x_y(self, pos_vec, vel_vec, end_lambda, step_size):
+        """
+
+        Parameters
+        ----------
+        pos_vec : list
+            list of r, theta & phi components along with ~astropy.units.
+        vel_vec : list
+            list of velocities of r, theta & phi components along with ~astropy.units.
+        end_lambda : float, optional
+            Lambda where iteartions will stop.
+        step_size : float, optional
+            Step size for the ODE.
+
+        """
+        swc = Schwarzschild.from_spherical(pos_vec, vel_vec, self.time, self.mass)
+
+        vals = swc.calculate_trajectory(
+            end_lambda=end_lambda, OdeMethodKwargs={"stepsize": step_size}
+        )[1]
+
+        time = np.array([coord[0] for coord in vals])
+        r = np.array([coord[1] for coord in vals])
+        theta = np.array([coord[2] for coord in vals])
+        phi = np.array([coord[3] for coord in vals])
+
+        x = r * np.cos(phi)
+        y = r * np.sin(phi)
+        self.__xarr = x
+        self.__yarr = y
+        return x, y
+
+    def mindist(self, x, y):
+        return np.sqrt(x * x + y * y)
+
+    def _draw_attractor(self):
+        radius = schwarzschild_radius(self.mass)
+        if self.attractor_radius_scale == -1.0:
+            minrad_nooverlap = self.mindist(self.__xarr[0], self.__yarr[0])
+            for i in range(0, len(self.__xarr)):
+                minrad_nooverlap = min(
+                    minrad_nooverlap, self.mindist(self.__xarr[i], self.__yarr[i])
+                )
+
+            xlen = max(self.__xarr) - min(self.__xarr)
+            ylen = max(self.__yarr) - min(self.__yarr)
+            minlen_plot = min(xlen, ylen)
+            mulitplier = minlen_plot / (12 * radius)
+            min_radius = radius * mulitplier
+
+            radius = min(min_radius, minrad_nooverlap)
+            self.get_curr_plot_radius = radius
+            self.ax.add_patch(
+                mpl.patches.Circle((0, 0), radius, lw=0, color=self.attractor_color)
+            )
+        else:
+            radius = radius * self.attractor_radius_scale
+            self.get_curr_plot_radius = radius
+            self.ax.add_patch(
+                mpl.patches.Circle(
+                    (0, 0), radius.value, lw=0, color=self.attractor_color
+                )
+            )
 
     def plot(
         self,
@@ -58,7 +156,25 @@ class StaticGeodesicPlotter:
         step_size=1e-3,
         color="#{:06x}".format(random.randint(0, 0xFFFFFF)),
     ):
-
+        """
+
+        Parameters
+        ----------
+        pos_vec : list
+            list of r, theta & phi components along with ~astropy.units.
+        vel_vec : list
+            list of velocities of r, theta & phi components along with ~astropy.units.
+        end_lambda : float, optional
+            Lambda where iteartions will stop.
+        step_size : float, optional
+            Step size for the ODE.
+        color : string
+            Color of the Geodesic
+
+        """
+        self.__xarr, self.__yarr = self.__get_x_y(
+            pos_vec, vel_vec, end_lambda, step_size
+        )
         self.plot_attractor()
         self._attractor_present = True
 
@@ -77,3 +193,6 @@ class StaticGeodesicPlotter:
 
     def save(self, name="static_geodesic.png"):
         plt.savefig(name)
+
+    def show(self):
+        plt.show()


Test patch:
diff --git a/src/einsteinpy/tests/test_plotting/test_staticgeodesicplotter.py b/src/einsteinpy/tests/test_plotting/test_staticgeodesicplotter.py
index ac8f62e..88f2fc0 100644
--- a/src/einsteinpy/tests/test_plotting/test_staticgeodesicplotter.py
+++ b/src/einsteinpy/tests/test_plotting/test_staticgeodesicplotter.py
@@ -29,8 +29,8 @@ def test_staticgeodesicplotter_has_axes(dummy_data):
     assert cl._attractor_present is False
 
 
-@mock.patch("einsteinpy.plotting.geodesics_static.StaticGeodesicPlotter.plot_attractor")
-def test_plot_calls_plot_attractor(mock_plot_attractor):
+@mock.patch("einsteinpy.plotting.geodesics_static.plt.show")
+def test_plot_calls_plt_show(mock_show):
     r = [306 * u.m, np.pi / 2 * u.rad, np.pi / 2 * u.rad]
     v = [0 * u.m / u.s, 0 * u.rad / u.s, 951.0 * u.rad / u.s]
     m = 4e24 * u.kg
@@ -38,7 +38,8 @@ def test_plot_calls_plot_attractor(mock_plot_attractor):
     ss = 0.5e-6
     cl = StaticGeodesicPlotter(m)
     cl.plot(r, v, el, ss)
-    mock_plot_attractor.assert_called_with()
+    cl.show()
+    mock_show.assert_called_with()
     assert cl._attractor_present
 
 
@@ -54,3 +55,20 @@ def test_plot_save_saves_plot(mock_save):
     name = "test_plot.png"
     cl.save(name)
     mock_save.assert_called_with(name)
+
+
+def test_plot_calls_draw_attractor_Manualscale(dummy_data):
+    r, v, _, m, _, el, ss = dummy_data
+    cl = StaticGeodesicPlotter(m, attractor_radius_scale=1500)
+    cl.plot(r, v, el, ss)
+    assert cl._attractor_present
+    assert cl.attractor_radius_scale == 1500
+    assert cl.get_curr_plot_radius != -1
+
+
+def test_plot_calls_draw_attractor_AutoScale(dummy_data):
+    r, v, _, m, _, el, ss = dummy_data
+    cl = StaticGeodesicPlotter(m)
+    cl.plot(r, v, el, ss)
+    assert cl._attractor_present
+    assert cl.get_curr_plot_radius != -1