Skip to content
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

Jump to bottom

Refactor aggreports class #278

Closed
hchagani-oasislmf opened this issue Mar 4, 2022 · 1 comment · Fixed by #328
Closed

Refactor aggreports class #278

hchagani-oasislmf opened this issue Mar 4, 2022 · 1 comment · Fixed by #328
Assignees
@hchagani-oasislmf
Labels
enhancement
Milestone
v3.9.4

Comments

@hchagani-oasislmf
Copy link
Contributor

hchagani-oasislmf commented Mar 4, 2022 •
edited
Loading

Issue Description

The OutputAgg*() and OutputOcc*() functions can be combined to help readability and future development. This affects leccalc and ordleccalc.

Version / Environment information

ktools v3.7.5
@hchagani-oasislmf hchagani-oasislmf self-assigned this Mar 4, 2022
@hchagani-oasislmf hchagani-oasislmf mentioned this issue Sep 5, 2022
Closed
@hchagani-oasislmf
Copy link
Contributor Author

Moved from Nazare - future developments to Nazare - current developments, as can be combined with simplifying header logic in issue #323.

@hchagani-oasislmf hchagani-oasislmf linked a pull request Oct 6, 2022 that will close this issue
Fix missing header in Wheatsheaf output and refactor aggreports class #328
Merged
@awsbuild awsbuild added this to the v3.9.4 milestone Oct 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hchagani-oasislmf hchagani-oasislmf

Labels
enhancement
Projects
None yet
Milestone
v3.9.4
Development

Successfully merging a pull request may close this issue.

Fix missing header in Wheatsheaf output and refactor aggreports class OasisLMF/ktools
2 participants
@awsbuild @hchagani-oasislmf