Create guidance for memory-intensive use cases, introduce Python memory profiler #1183
Closed
22 tasks
Labels
component: code optimization
Code optimization issue
component: documentation
Documentation issue
priority: high
High Priority
requestor: NOAA/EMC
NOAA Environmental Modeling Center
required: FOR OFFICIAL RELEASE
Required to be completed in the official release for the assigned milestone
type: enhancement
Improve something that it is currently doing
Milestone
Replace italics below with details for this issue.
Describe the Enhancement
During the creation of the use case for issue #1004, it was found that Github Actions automated testing has a limited capacity for memory. As a result of this use case requiring a vast memory storage for its Python script, the automated testing fails. Even after adjustments to try and limit the amount of data held in memory, the automated testing continues to exit unsuccessfully.
The goal of this issue is twofold: the first is to document in the Contributor's Guide what to do when this type of scenario happens.
The second goal is the introduction of a Python memory logger/profiler that becomes standard when creating use cases. By introducing this tool, along with a memory "cap", contributors can know what to do if their use case will exceed the Github automated testing before pushing the branch into automated testing.
Time Estimate
Documentation goal: 1 day
Python memory profiler: 2-4 days
Sub-Issues
Consider breaking the enhancement down into sub-issues.
Relevant Deadlines
None
Funding Source
2792541 and 2702691
Define the Metadata
Assignee
Labels
Projects and Milestone
Define Related Issue(s)
Consider the impact to the other METplus components.
Enhancement Checklist
See the METplus Workflow for details.
Branch name:
feature_<Issue Number>_<Description>
Pull request:
feature <Issue Number> <Description>
Select: Reviewer(s) and Linked 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: