-
Notifications
You must be signed in to change notification settings - Fork 7
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
Address Warnings from pandas in METplotpy ROC diagram code #296
Labels
component: external dependency
External dependency issue
priority: high
High Priority
reporting: DTC NOAA BASE
NOAA Office of Atmospheric Research DTC Project
reporting: DTC NOAA R2O
NOAA Research to Operations DTC Project
requestor: METplus Team
METplus Development Team
requestor: NOAA/other
NOAA Laboratory, not otherwise specified
type: bug
Fix something that is not working
Milestone
Comments
bikegeek
added
type: bug
Fix something that is not working
priority: high
High Priority
requestor: NOAA/other
NOAA Laboratory, not otherwise specified
alert: NEED ACCOUNT KEY
Need to assign an account key to this issue
alert: NEED MORE DEFINITION
Not yet actionable, additional definition required
alert: NEED CYCLE ASSIGNMENT
Need to assign to a release development cycle
component: external dependency
External dependency issue
requestor: METplus Team
METplus Development Team
labels
Dec 23, 2022
bikegeek
added a commit
that referenced
this issue
Dec 23, 2022
bikegeek
added a commit
that referenced
this issue
Dec 24, 2022
…oints to address FutureWarning with Series.append. Replaced Series.append logic with pd.concat
bikegeek
added a commit
that referenced
this issue
Dec 24, 2022
bikegeek
added a commit
that referenced
this issue
Dec 24, 2022
bikegeek
removed
alert: NEED MORE DEFINITION
Not yet actionable, additional definition required
alert: NEED CYCLE ASSIGNMENT
Need to assign to a release development cycle
labels
Jan 17, 2023
bikegeek
added a commit
that referenced
this issue
Jan 17, 2023
bikegeek
added a commit
that referenced
this issue
Jan 18, 2023
* replace pd.append with pd.concat * replace pd.append with pd.concat * Issue #296 replace pd.append with pd.concat to address future deprecation in pandas * Issue #296 updates to reflect changes made to address pandas future deprecation in roc_diagram_series * Issue #296 add data and configuration to test future deprecation warnings
bikegeek
added a commit
that referenced
this issue
Jan 18, 2023
* Issue #296 Add yaml for testing PCT data * Issue #296 PCT sample data * Issue #296 for update to pandas 1.5.x modify code in _create_series_points to address FutureWarning with Series.append. Replaced Series.append logic with pd.concat * Issue #296 Didn't remove one of the replaced lines in the PCT linetype code block * Issue #296 Added two more tests, using PCT linetype data * replace pd.append with pd.concat * replace pd.append with pd.concat
bikegeek
removed
the
alert: NEED ACCOUNT KEY
Need to assign an account key to this issue
label
Jan 19, 2023
github-project-automation
bot
moved this to 📋 Backlog
in Coordinated METplus-5.0 Support
Jan 25, 2023
github-project-automation
bot
moved this to 📋 Backlog
in METplus-Analysis-5.1.0 Development
Jan 25, 2023
TaraJensen
added
reporting: DTC NOAA BASE
NOAA Office of Atmospheric Research DTC Project
reporting: DTC NOAA R2O
NOAA Research to Operations DTC Project
labels
Jan 25, 2023
bikegeek
added a commit
that referenced
this issue
Jan 26, 2023
some words were cut off for #296, missing the ROC diagram code portion of the issue
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
component: external dependency
External dependency issue
priority: high
High Priority
reporting: DTC NOAA BASE
NOAA Office of Atmospheric Research DTC Project
reporting: DTC NOAA R2O
NOAA Research to Operations DTC Project
requestor: METplus Team
METplus Development Team
requestor: NOAA/other
NOAA Laboratory, not otherwise specified
type: bug
Fix something that is not working
Describe the Problem
Related to METcalcpy issue 249:
dtcenter/METcalcpy#249
This is observed with pandas 1.5 but not with 1.2.3
Expected Behavior
When generating a ROC diagram with PCT data, the FutureWarning is generated from the roc_diagram_series.py code in the _create_series_points() method:
Remedy:
Replace pody.append with pd.concat([pody, pd.Series([0])]) in above code block and in the code for the CTC line type.
Environment
Describe your runtime environment:
*1. Machine: mac laptop 'blenny'
*2. OS: MacOS
3. Software version number(s) v2.0
To Reproduce
Describe the steps to reproduce the behavior:
1. Go to '...'
2. Click on '....'
3. Scroll down to '....'
4. See error
Post relevant sample data following these instructions:
https://dtcenter.org/community-code/model-evaluation-tools-met/met-help-desk#ftp
Relevant Deadlines
List relevant project deadlines here or state NONE.
Funding Source
2792542, 2773542
Define the Metadata
Assignee
Labels
Projects and Milestone
Define Related Issue(s)
Consider the impact to the other METplus components.
Bugfix Checklist
See the METplus Workflow for details.
Branch name:
bugfix_<Issue Number>_main_<Version>_<Description>
Pull request:
bugfix <Issue Number> main_<Version> <Description>
Select: Reviewer(s) and Development issues
Select: Organization level software support Project for the current coordinated release
Select: Milestone as the next bugfix version
Branch name:
bugfix_<Issue Number>_develop_<Description>
Pull request:
bugfix <Issue Number> develop <Description>
Select: Reviewer(s) and Development issues
Select: Repository level development cycle Project for the next official release
Select: Milestone as the next official version
The text was updated successfully, but these errors were encountered: