sustainability-lab / polire

Spatial Interpolation in Python
https://sustainability-lab.github.io/polire/
BSD 3-Clause "New" or "Revised" License
12 stars 3 forks source link

Dev #11

Closed apoorvagnihotri closed 5 years ago

apoorvagnihotri commented 5 years ago
sdeepaknarayanan commented 5 years ago

Change the spelling of dimention to dimension. Also, make sure comments and docstrings have as minimal grammatical errors as possible.

sdeepaknarayanan commented 5 years ago

Regarding B-Spline, when there are multiple optional arguments, can you mention the argument type as (int, int) or something? kx, ky, as I see it, do not seem to have this. Also add comments mentioning what each variable means. What does "tck" mean for an example? I'll merge once you make the changes

sdeepaknarayanan commented 5 years ago

Is there any reason why function definitions are written your way? The def f(a, B, C) with indentation. I’d avoid that as long as there aren’t too many arguments (<6). No issue tho. We can change later

nipunbatra commented 5 years ago

We should use Black formatter for the code: https://github.com/python/black

apoorvagnihotri commented 5 years ago

Ran black on the following files and directories only:

usage.py
interpolation/trend
interpolation/spline
interpolation/random
interpolation/base

@sdeepaknarayanan I haven't run black on your code files as it will cause problems in merging later.