-
-
Notifications
You must be signed in to change notification settings - Fork 120
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
Time varying ability matrix #895
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #895 +/- ##
==========================================
- Coverage 80.28% 80.26% -0.03%
==========================================
Files 18 18
Lines 4144 4175 +31
==========================================
+ Hits 3327 3351 +24
- Misses 817 824 +7
Flags with carried forward coverage won't be shown. Click here to find out more.
|
@jdebacker. I deleted my
|
@rickecon Noted about the example. That was for some testing and should be taken out. Will update. |
All test we expect to pass on Python 3.11 pass locally. The three failures are the
I was also able to run the I believe this PR is ready to merge. |
LGTM. Thanks, @jdebacker. |
This PR allows one to use an ability/labor productivity matrix that varies over time. Changing the shape of the
e
parameter object from SxJ to TxSxJ.