-
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
activation functions added for bounded outputs #14
base: develop
Are you sure you want to change the base?
Conversation
640b623
to
da2ffcc
Compare
8eaf33f
to
373d67b
Compare
373d67b
to
85d45ae
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #14 +/- ##
========================================
Coverage 99.82% 99.83%
========================================
Files 21 22 +1
Lines 1158 1219 +61
========================================
+ Hits 1156 1217 +61
Misses 2 2 ☔ View full report in Codecov by Sentry. |
@@ -47,8 +63,10 @@ def __init__( | |||
self.data_indices = data_indices | |||
|
|||
def _process_config(self, config): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you put a comment to explain why these keys are handled specially?
Great work on this, thanks. |
This PR is related to the open issue #13 and the aifs-mono PR https://github.com/ecmwf-lab/aifs-mono/pull/224.
The basic idea behind this PR is to implement bounding strategies for specific variables, chosen by the user, via the usage of activation functions. One evident example the total precipitation accumulation variable, which is not currently bounded, therefore prone to negative value predictions.
Closes #13