Serials predictions fail when nominally omittable chronology caption fields not in pattern

Bug #973453 reported by Lebbeous Fogle-Weekley
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Unassigned
2.0
Fix Released
Undecided
Unassigned
2.1
Fix Released
Undecided
Unassigned

Bug Description

In Evergreen master (and probably down through 2.0):

Sorry about the title of this bug. There is simply no way to talk about how serials predictions work in plain English.

Olli-Antti Kivilahti reported trouble creating serials predictions in the #evergreen IRC channel.

He was trying to set up a weekly with any of the patterns in the following paste, without success: http://pastebin.com/CN8vFiyP

I have suggested the alternative in the following paste, which does work, but it may not be exactly what the user intended: http://pastie.org/3727277

The relevant difference is the presence of ‡i and ‡j in my pattern. You *should*, however, be able to omit those. But if you do, the middle layer routines for prediction seem to hang forever, and don't seem to emit error messages. Investigation is called for. I'll try to do what I can, but help from all sources is appreciated.

Tags: serials
Revision history for this message
Dan Wells (dbw2) wrote :

I have made some progress on this and am currently testing my fix. Barring unforeseen issues, I should have a branch for this tomorrow.

Changed in evergreen:
assignee: nobody → Dan Wells (dbw2)
Revision history for this message
Dan Wells (dbw2) wrote :

Here is the branch with the fix (just one commit):

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/dbwells/lp973453_incomplete_chron_bug

I think this can (and should) backport cleanly through 2.0. If it doesn't, just let me know and I will take care of it. Thanks!

Changed in evergreen:
assignee: Dan Wells (dbw2) → nobody
status: New → Confirmed
milestone: none → 2.2.0beta3
Revision history for this message
Lebbeous Fogle-Weekley (lebbeous) wrote :

Thanks Dan! I aim to test this by the end of today.

Revision history for this message
Lebbeous Fogle-Weekley (lebbeous) wrote :

Thanks again Dan. Sorry it took longer than I said, but this is now fixed and backported through rel_2_1. Looking into whether rel_2_0 will work now.

Revision history for this message
Lebbeous Fogle-Weekley (lebbeous) wrote :

rel_2_0 handled as well

All done!

Changed in evergreen:
status: Confirmed → Fix Committed
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.