-
Notifications
You must be signed in to change notification settings - Fork 24
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
Fix Multivariate MODE to log it's output to a single log file. #1593
Labels
MET: Library Code
priority: high
High Priority
requestor: NCAR
National Center for Atmospheric Research
type: enhancement
Improve something that it is currently doing
Milestone
Comments
JohnHalleyGotway
added
type: enhancement
Improve something that it is currently doing
component: library code
requestor: NCAR
National Center for Atmospheric Research
alert: NEED ACCOUNT KEY
Need to assign an account key to this issue
labels
Dec 3, 2020
JohnHalleyGotway
changed the title
Fix multi-variate MODE to log it's output to a single log file.
Fix Multivariate MODE to log it's output to a single log file.
Dec 3, 2020
lindsayrblank
added
the
alert: NEED CYCLE ASSIGNMENT
Need to assign to a release development cycle
label
Jan 27, 2021
TaraJensen
added
required: FOR OFFICIAL RELEASE
Required to be completed in the official release for the assigned milestone
and removed
alert: NEED CYCLE ASSIGNMENT
Need to assign to a release development cycle
labels
May 20, 2021
JohnHalleyGotway
removed
the
required: FOR OFFICIAL RELEASE
Required to be completed in the official release for the assigned milestone
label
Sep 21, 2022
JohnHalleyGotway
added
the
alert: NEED CYCLE ASSIGNMENT
Need to assign to a release development cycle
label
Sep 21, 2022
14 tasks
Closing this issue as it was fixed by refactoring multivariate mode to not spawn multiple instance of traditional mode. The newer design does not do this, it does sequential steps that depend on each other (so the original design no longer made sense). |
JohnHalleyGotway
removed
alert: NEED ACCOUNT KEY
Need to assign an account key to this issue
alert: NEED CYCLE ASSIGNMENT
Need to assign to a release development cycle
labels
Oct 17, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
MET: Library Code
priority: high
High Priority
requestor: NCAR
National Center for Atmospheric Research
type: enhancement
Improve something that it is currently doing
Describe the Enhancement
This task is closely related to MET #1283. If all Multivariate MODE functionality is run by a single process, then the logging issue effectively goes away.
MET #1184 was included in the met-10.0.0-beta2 release to add the multi-variate MODE option. This version of the code spawns new processes to execute individual calls to the standard MODE prior to combining the output into a super object. This task is to figure out how to handle the writing of a log file in this context. Since multiple processes are spawned, having them write to the same output log file is non-trivial.
Since the calls to MODE are done sequentially, consider adding a no-clobber option to the logger so that each call to single MODE can open and append to the same log file without overwriting it.
Time Estimate
Estimate the amount of work required here.
Issues should represent approximately 1 to 3 days of work.
Sub-Issues
Consider breaking the enhancement down into sub-issues.
No sub-issues required.
Relevant Deadlines
List relevant project deadlines here or state NONE.
Funding Source
Define the source of funding and account keys here or state NONE.
Define the Metadata
Assignee
Labels
Projects and Milestone
Define Related Issue(s)
Consider the impact to the other METplus components.
No impacts.
Enhancement Checklist
See the METplus Workflow for details.
Branch name:
feature_<Issue Number>_<Description>
Pull request:
feature <Issue Number> <Description>
Select: Reviewer(s), Project(s), Milestone, and Linked issues
The text was updated successfully, but these errors were encountered: