Skip to content
This repository has been archived by the owner on Jul 23, 2019. It is now read-only.

Metrics #13

Open
wants to merge 34 commits into
base: master
Choose a base branch
from
Open

Metrics #13

wants to merge 34 commits into from

Conversation

cjayross
Copy link
Contributor

@cjayross cjayross commented Jul 7, 2019

Addresses #4

This adds a total 113* different metrics that can be loaded by using the function load_metric defined in metric.py.

There are currently two issues to be addressed, the first is that the metrics aren't organized and unsearchable; meaning that a user has to manually go into the file where it's defined to actually discover what the metric actually looks like. The second issues is that citations need to be added to each metric that is defined.

>>> from sympy import *
>>> from riccipy import *
>>> g = load_metric('g', 'einstein_maxwell_2')
>>> g.as_array()

equation

* Now 115

@pep8speaks
Copy link

pep8speaks commented Jul 7, 2019

Hello @cjayross! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 267:81: E501 line too long (84 > 80 characters)

Line 12:81: E501 line too long (86 > 80 characters)

Line 12:81: E501 line too long (86 > 80 characters)

Line 15:5: W503 line break before binary operator
Line 17:9: W503 line break before binary operator
Line 19:5: W503 line break before binary operator

Line 14:81: E501 line too long (82 > 80 characters)

Line 16:5: W503 line break before binary operator
Line 17:5: W503 line break before binary operator
Line 18:5: W503 line break before binary operator

Line 16:5: W503 line break before binary operator
Line 17:5: W503 line break before binary operator
Line 18:5: W503 line break before binary operator
Line 19:81: E501 line too long (88 > 80 characters)
Line 20:9: W503 line break before binary operator

Line 16:5: W503 line break before binary operator
Line 17:5: W503 line break before binary operator

Line 12:81: E501 line too long (83 > 80 characters)

Line 2:81: E501 line too long (109 > 80 characters)

Line 2:81: E501 line too long (117 > 80 characters)

Line 2:81: E501 line too long (122 > 80 characters)

Line 15:81: E501 line too long (87 > 80 characters)

Line 2:81: E501 line too long (134 > 80 characters)
Line 14:81: E501 line too long (86 > 80 characters)

Line 15:5: W503 line break before binary operator
Line 16:5: W503 line break before binary operator
Line 17:5: W503 line break before binary operator
Line 18:5: W503 line break before binary operator
Line 19:5: W503 line break before binary operator
Line 20:5: W503 line break before binary operator

Line 13:81: E501 line too long (88 > 80 characters)

Line 10:81: E501 line too long (81 > 80 characters)

Line 20:9: W503 line break before binary operator
Line 21:9: W503 line break before binary operator
Line 22:9: W503 line break before binary operator
Line 23:9: W503 line break before binary operator
Line 24:9: W503 line break before binary operator
Line 26:5: W503 line break before binary operator
Line 27:5: W503 line break before binary operator

Line 21:9: W503 line break before binary operator
Line 22:9: W503 line break before binary operator
Line 23:9: W503 line break before binary operator
Line 24:9: W503 line break before binary operator
Line 25:9: W503 line break before binary operator
Line 27:5: W503 line break before binary operator
Line 28:5: W503 line break before binary operator

Line 17:5: W503 line break before binary operator
Line 19:9: W503 line break before binary operator
Line 20:9: W503 line break before binary operator
Line 21:9: W503 line break before binary operator
Line 22:9: W503 line break before binary operator
Line 23:9: W503 line break before binary operator
Line 25:5: W503 line break before binary operator

Line 20:9: W503 line break before binary operator
Line 21:9: W503 line break before binary operator
Line 22:9: W503 line break before binary operator
Line 23:9: W503 line break before binary operator
Line 24:9: W503 line break before binary operator
Line 25:9: W503 line break before binary operator
Line 26:9: W503 line break before binary operator
Line 28:5: W503 line break before binary operator
Line 29:5: W503 line break before binary operator

Line 15:81: E501 line too long (87 > 80 characters)
Line 21:9: W503 line break before binary operator
Line 22:9: W503 line break before binary operator
Line 23:9: W503 line break before binary operator
Line 24:9: W503 line break before binary operator
Line 25:9: W503 line break before binary operator
Line 26:9: W503 line break before binary operator
Line 28:5: W503 line break before binary operator
Line 29:5: W503 line break before binary operator

Line 2:81: E501 line too long (118 > 80 characters)

Line 12:81: E501 line too long (83 > 80 characters)

Line 12:81: E501 line too long (83 > 80 characters)

Line 12:81: E501 line too long (81 > 80 characters)

Line 12:81: E501 line too long (81 > 80 characters)

Line 13:81: E501 line too long (88 > 80 characters)

Line 2:81: E501 line too long (100 > 80 characters)

Line 15:81: E501 line too long (82 > 80 characters)

Line 14:81: E501 line too long (83 > 80 characters)

Line 15:81: E501 line too long (83 > 80 characters)

Line 13:81: E501 line too long (83 > 80 characters)

Line 14:81: E501 line too long (83 > 80 characters)

Line 15:81: E501 line too long (88 > 80 characters)

Line 2:81: E501 line too long (95 > 80 characters)

Line 2:81: E501 line too long (94 > 80 characters)

Comment last updated at 2019-07-08 09:56:19 UTC

@cjayross
Copy link
Contributor Author

cjayross commented Jul 8, 2019

I added the citations that I had available. I think there is a couple that are still missing.

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

Successfully merging this pull request may close these issues.

None yet

2 participants