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.

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

Remove validateitems #332

Merged
merged 1 commit into from
Nov 9, 2022
Merged

Conversation

hchagani-oasislmf
Copy link
Contributor

Remove validateitems

Development of the validateitems component does not serve a useful purpose at this moment in time, and therefore it has been removed from the ktools package.

@hchagani-oasislmf hchagani-oasislmf linked an issue Nov 9, 2022 that may be closed by this pull request
@hchagani-oasislmf hchagani-oasislmf merged commit 3df67cb into develop Nov 9, 2022
@hchagani-oasislmf hchagani-oasislmf deleted the fix/166-delete-validateitems branch November 9, 2022 12:08
@awsbuild awsbuild added this to the v3.9.5 milestone Nov 11, 2022
sambles added a commit that referenced this pull request Nov 11, 2022
* Remove validateitems (#332)

* Remove auto-merge feature (#333)

* Increase precision of return periods + use period weights in full uncertainty output in leccalc/ordleccalc (#335)

* Add extra logging info for debugging

* Add log file information to enable compilation

* Log return period calculatioins for debugging

* Increase return periods precision

* Update tests

* test store testout on build

* test store testout on build - add dbg prints

* test

* tidy up workflow

* Update tests

* Update tests

* Remove logging for debugging

* Fix call to full uncertainty with weighting calculation

* Remove logging information used for debugging

Co-authored-by: Sam Gamble <hexadessa@gmail.com>

* Fix bug in changelog create call

* Set Ktools to version 3.9.5

* Update changelog

Co-authored-by: hchagani-oasislmf <47981469+hchagani-oasislmf@users.noreply.github.com>
Co-authored-by: awsbuild <awsbuild@oasislmf.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Finish developing validateitems
2 participants