Time |
Nick |
Message |
01:09 |
|
jonadab joined #evergreen |
06:01 |
pinesol |
News from qatests: Testing Success <http://testing.evergreen-ils.org/~live> |
07:28 |
|
rjackson_isl_hom joined #evergreen |
07:29 |
|
collum joined #evergreen |
08:38 |
|
mmorgan joined #evergreen |
08:47 |
|
rfrasur joined #evergreen |
08:53 |
|
mantis joined #evergreen |
09:39 |
mmorgan |
Still trying to understand standing penalties. We have a new grp_penalty_threshold for PATRON_EXCEEDS_LOST_COUNT with threshold 1 at the consortium level. |
09:41 |
mmorgan |
After the fine generator ran, we have a patron that has 2 Lost items, but also 2 items that got fines. The patron now has 2 rows in actor.usr_standing_penalty for PATRON_EXCEEDS_LOST_COUNT, so the message shows twice on the account. |
11:01 |
jeff |
Is the patron subject to two different thresholds at two different levels of the org tree? |
11:01 |
jeff |
Is one of the standing penalties present from a threshold that has changed org units or been removed completely? |
11:04 |
mmorgan |
jeff: No, and No. Only one threshold applies to this patron. Both rows in the usr_standing_penalty table were applied at almost the same time. Here are the timestamps: |
11:04 |
mmorgan |
2021-11-05 00:30:23.032548-04 |
11:04 |
mmorgan |
2021-11-05 00:30:23.006111-04 |
11:06 |
jeff |
do both of those two standing penalties have the same value for org_unit? |
11:08 |
mmorgan |
jeff: Yes, they do, 1. |
11:12 |
csharp_ |
mmorgan: parallel fine generation maybe hit the two items at the same moment? |
11:12 |
jeff |
If you refresh (open-ils.actor.user.penalties.update) the patron, do you end up with only one? If you delete them both and refresh the patron, do you end up with one or two? |
11:12 |
csharp_ |
er... circs not items |
11:13 |
jeff |
Does the parallelization in the generator not group on users? Maybe it should... |
11:18 |
csharp_ |
jeff: from what I can tell, it doesn't - but I may be missing something |
11:18 |
csharp_ |
but that still might not be the cause of mmorgan's issue |
11:20 |
mmorgan |
jeff: So removing the 2 penalties and refreshing the patron record does not bring back any penalties. So open-ils.actor.user.penalties.update I would need to run using srfsh? |
11:22 |
mmorgan |
Regarding refreshing in the client, bug 1823225 seems relevant - and annoying. |
11:22 |
pinesol |
Launchpad bug 1823225 in Evergreen "Web client has no way to recalculate patron penalties" [Undecided,Confirmed] https://launchpad.net/bugs/1823225 |
11:27 |
* jeff |
nods |
11:27 |
jeff |
(to both: run via srfsh and annoying that there isn't a way to refresh from the web client) |
11:28 |
jeff |
s/refresh/recalculate/ |
11:43 |
mmorgan |
jeff: Refreshing the user's penalties results in just one row in actor.usr_standing_penalty. |
11:45 |
* mmorgan |
will try a few queries to see if this was just a one-off. |
11:47 |
mmorgan |
I've destroyed my evidence, but can see that the create_dates in the patron's two billings exactly match the set_date in usr_standing_penalty |
12:00 |
jeff |
mmorgan: the circs were marked lost early this morning by A/T? |
12:04 |
mmorgan |
jeff: No, overdue fines on 2 circs were generated this morning. One circ was marked Lost in 2019. New grp_penalty_threshold was added yesterday, so the patron would not have been previously blocked. |
12:10 |
|
jvwoolf joined #evergreen |
12:10 |
pinesol |
[evergreen|Mike Rylander] LP#1947173: Speed up the symspell part of ingest - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=32c880d> |
12:10 |
pinesol |
[evergreen|Mike Rylander] LP#1947173: Clean up bad cataloging pot hole - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=2d0a459> |
12:10 |
pinesol |
[evergreen|Galen Charlton] LP#1947173: stamp schema update - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=d6d9388> |
12:14 |
pinesol |
[evergreen|Jason Stephenson] LP1948594: Fix pgtap live test - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=1d56670> |
12:15 |
mmorgan |
So only one other patron with a Lost item got overdue fines for 2 circs this morning. This patron got only one row in actor.usr_standing_penalty. Guess I'll just monitor to see if it happens again. |
12:15 |
mmorgan |
The dups, that is. |
12:42 |
pinesol |
[evergreen|Galen Charlton] update Czech translations for Angular staff client - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=11575e6> |
12:46 |
|
stephengwills left #evergreen |
13:09 |
|
collum joined #evergreen |
13:10 |
|
collum joined #evergreen |
13:10 |
pinesol |
[evergreen|Jeff Davis] LP#1830089: close transaction and update copy status on payment or adjust to zero - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=ea247ab> |
15:35 |
jvwoolf |
Just saw open-ils.auth drones and open-ils.actor drones max out at the same time - I don't think I've seen that particular thing happen before |
16:15 |
pinesol |
[evergreen|Galen Charlton] update upgrade script for 3.7.1-3.8.0 - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=9287045> |
16:23 |
pinesol |
[evergreen|Galen Charlton] final update to release notes for 3.8.0 - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=534eb91> |
16:25 |
pinesol |
[evergreen|Galen Charlton] Translation updates - po files - 3.8.0 - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=a7a9faf> |
16:45 |
|
Keith-isl joined #evergreen |
17:01 |
|
mmorgan left #evergreen |
17:09 |
gmcharlt |
https://evergreen-ils.org/evergreen-3-8-0-released/ |
17:09 |
alynn26 |
gmcharlt++ |
17:25 |
pinesol |
[evergreen|Galen Charlton] docs: add 3.8 to target branches for Antora - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=2a51ce6> |
17:35 |
gmcharlt |
docs.evergreen-ils.org is now building docs for 3.8 |
17:35 |
|
jvwoolf left #evergreen |
18:01 |
pinesol |
News from qatests: Testing Success <http://testing.evergreen-ils.org/~live> |
18:21 |
pinesol |
[evergreen|Terran McCanna] LP1830387 Add access to juvenile flag in print templates - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=cb54628> |
18:21 |
pinesol |
[evergreen|Galen Charlton] LP#1830387: (follow-up) normalize whitespace - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=ddc6710> |
18:21 |
pinesol |
[evergreen|Galen Charlton] LP#1830387: add release notes entry - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=dfa2ecc> |
18:27 |
* gmcharlt |
claims 1037 |
18:27 |
gmcharlt |
er, 1307 |
18:33 |
pinesol |
Showing latest 5 of 7 commits to Evergreen... |
18:33 |
pinesol |
[evergreen|Jeff Davis] LP#1831803: release note for removing old staged search stuff from db - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=33ea045> |
18:33 |
pinesol |
[evergreen|Jason Boyer] LP1831803: Remove unused data types - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=ab080c4> |
18:33 |
pinesol |
[evergreen|Galen Charlton] LP#1831803: drop the disused search-related types during upgrade - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=8c91148> |
18:33 |
pinesol |
[evergreen|Galen Charlton] LP#1831803: (follow-up) update release notes formatting - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=065e24b> |
18:33 |
pinesol |
[evergreen|Galen Charlton] LP#1831803: stamp schema update - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=a868113> |