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

Ahmed semah #198

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
17 changes: 15 additions & 2 deletions numpy_questions.py
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,11 @@ def max_index(X):
j = 0

# TODO

if not isinstance(X, np.ndarray):
raise ValueError("Input is not a numpy array")
if X.ndim != 2:
raise ValueError("Input is not a 2D array")
i, j = np.unravel_index(np.argmax(X), X.shape)
return i, j


Expand All @@ -64,4 +68,13 @@ def wallis_product(n_terms):
"""
# XXX : The n_terms is an int that corresponds to the number of
# terms in the product. For example 10000.
return 0.

if n_terms < 0:
raise ValueError("n_terms should be a positive integer")
if n_terms == 0:
return 2.

pi = 1.
for i in range(1, n_terms+1):
pi *= 4*i**2/(4*i**2-1)
return 2*pi
Loading