8000 [WIP] Adopt `ghi_clear` variable name for clearsky GHI by RDaxini · Pull Request #2273 · pvlib/pvlib-python · GitHub
[go: up one dir, main page]
Skip to content

[WIP] Adopt ghi_clear variable name for clearsky GHI #2273

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.

Sign up for GitHub

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

Closed
wants to merge 24 commits into from
Closed

[WIP] Adopt ghi_clear variable name for clearsky GHI #2273

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
bf1c0d8
irradiance.py: ghi_from_poa() and clearsky_index()
RDaxini Oct 21, 2024
37d17b8
clearsky.py: haurwitz(), detect_clearsky
RDaxini Oct 21, 2024
9fa9fdc
clearsky.py: detect_clearsky
RDaxini Oct 21, 2024
00ffbd9
irradiance.py: dirindex()
RDaxini Oct 21, 2024
b21e1eb
modify tests
RDaxini Oct 25, 2024
57288e7
add deprecation warning
RDaxini Nov 21, 2024
00d7b70
Update clearsky.py
RDaxini Nov 25, 2024
6c78cb8
Merge remote-tracking branch 'upstream/main' into ghi_clear
RDaxini Nov 25, 2024
5b273df
fix unrelated linting issues
RDaxini Nov 25, 2024
8000 38a409d
fix unrelated linting issue
RDaxini Nov 25, 2024
263ad4e
Update test_clearsky.py
RDaxini Nov 25, 2024
ba5ffaa
Revert "Update test_clearsky.py"
RDaxini Nov 25, 2024
f45cf19
Reapply "Update test_clearsky.py"
RDaxini Nov 25, 2024
d7a6f83
Revert "Update test_clearsky.py"
RDaxini Nov 25, 2024
fbd49e9
Revert "fix unrelated linting issue"
RDaxini Nov 25, 2024
84987b0
Revert "fix unrelated linting issues"
RDaxini Nov 25, 2024
1622571
Revert "Merge remote-tracking branch 'upstream/main' into ghi_clear"
RDaxini Nov 25, 2024
5879282
Revert "Update clearsky.py"
RDaxini Nov 25, 2024
bbe9d4f
Revert "add deprecation warning"
RDaxini Nov 25, 2024
aeab5cc
Revert "modify tests"
RDaxini Nov 25, 2024
9be20a6
Revert "irradiance.py: dirindex()"
RDaxini Nov 25, 2024
efdbbd8
Revert "clearsky.py: detect_clearsky"
RDaxini Nov 25, 2024
feab85c
Revert "clearsky.py: haurwitz(), detect_clearsky"
RDaxini Nov 25, 2024
c9c6900
Revert "irradiance.py: ghi_from_poa() and clearsky_index()"
RDaxini Nov 25, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Revert "add deprecation warning" 
This reverts commit 57288e7.
  • Loading branch information
@RDaxini
RDaxini committed Nov 25, 2024
commit bbe9d4fd3344bd56d207231b3201fa0a752cd756
4 changes: 1 addition & 3 deletions pvlib/clearsky.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,11 +12,9 @@
from scipy.linalg import hankel
import h5py

from pvlib import atmosphere, tools, renamed_kwarg_warning
from pvlib import atmosphere, tools
from pvlib.tools import _degrees_to_index

renamed_kwarg_warning('11.2', 'clearsky_ghi', 'ghi_clear', removal="12.0")
renamed_kwarg_warning('11.2', 'clearsky', 'ghi_clear', removal="12.0")

def ineichen(apparent_zenith, airmass_absolute, linke_turbidity,
altitude=0, dni_extra=1364., perez_enhancement=False):
Expand Down
4 changes: 1 addition & 3 deletions pvlib/irradiance.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,11 +16,9 @@
from pvlib import atmosphere, solarposition, tools
import pvlib # used to avoid dni name collision in complete_irradiance

from pvlib._deprecation import pvlibDeprecationWarning, renamed_kwarg_warning
from pvlib._deprecation import pvlibDeprecationWarning
import warnings

renamed_kwarg_warning('11.2', 'clearsky_ghi', 'ghi_clear', removal="12.0")
renamed_kwarg_warning('11.2', 'ghi_clearsky', 'ghi_clear', removal="12.0")

# Deprecation warning based on https://peps.python.org/pep-0562/
def __getattr__(attr):
Expand Down
0