Opened 6 years ago

Closed 6 years ago

#15637 closed defect (fixed)

TypeError: object of type 'listreverseiterator' has no len()

Reported by: darij Owned by:
Priority: major Milestone: sage-duplicate/invalid/wontfix
Component: website/wiki Keywords: trac
Cc: ohanar Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description

How to Reproduce

While doing a POST operation on /ticket/15576, Trac issued an internal error.

(please provide additional details here)

Request parameters:

{'__EDITOR__1': u'textarea',
 '__EDITOR__2': u'textarea',
 '__FORM_TOKEN': u'3dd5a53b3fddfbbb88a5a311',
 'action': u'leave',
 'comment': u'Attention: branch change!\r\n\r\nI\'m a bit surprised that you changed the definite articles back to the indefinites, so I\'m suggesting to change them back in my commit. (Also, "the group of degree n over a ring R" sounds better than "the group over a ring R of degree n" to my ears.)\r\n\r\nIf my edits are OK to you, please set this to positive_review. Thanks for your work!',
 'field_author': u'',
 'field_branch': u'public/ticket/15576',
 'field_cc': u'tfeulner',
 'field_commit': u'6155cf9ce606b77b6c70c84a106430e9068e15c2',
 'field_component': u'group theory',
 'field_dependencies': u'',
 'field_description': u"As detailed in #14885, it is not healthy for code to rely on the `__mul__` operation on permutations, since this operation depends on the `Permutations().global_options()['mul']` variable which can change at runtime. It is better to use the `left_action_product` and `right_action_product` methods introduced in #15174 (formerly known as `_left_to_right_multiply_on_left` and `_left_to_right_multiply_on_right`).\r\n\r\nMy tests show some dependence on the `__mul__` method in `sage/groups/semimonomial_transformations/semimonomial_transformation.pyx` and `sage/groups/semimonomial_transformations/semimonomial_transformation_group.py`, although it might be that only one of these files depends on it and the other depends on the first file. Unfortunately I don't have time to study this in detail, as I'd first have to read up on the definitions.",
 'field_keywords': u'permutation, semimonomial transformation, groups',
 'field_merged': u'',
 'field_milestone': u'sage-6.1',
 'field_priority': u'major',
 'field_reviewer': u'',
 'field_stopgaps': u'',
 'field_summary': u'Semimonomial transformation groups code is sensitive to Permutations global options',
 'field_type': u'defect',
 'field_upstream': u'N/A',
 'field_work_issues': u'',
 'id': u'15576',
 'replyto': u'',
 'start_time': u'1388826242393549',
 'submit': u'Submit changes',
 'view_time': u'1388826242393549'}

User agent: Mozilla/5.0 (X11; Ubuntu; Linux i686; rv:26.0) Gecko/20100101 Firefox/26.0

System Information

System information not available

Enabled Plugins

Plugin information not available

Python Traceback

Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/Trac-1.1.2dev-py2.7.egg/trac/web/main.py", line 497, in _dispatch_request
    dispatcher.dispatch(req)
  File "/usr/local/lib/python2.7/dist-packages/Trac-1.1.2dev-py2.7.egg/trac/web/main.py", line 214, in dispatch
    resp = chosen_handler.process_request(req)
  File "/usr/local/lib/python2.7/dist-packages/Trac-1.1.2dev-py2.7.egg/trac/ticket/web_ui.py", line 185, in process_request
    return self._process_ticket_request(req)
  File "/usr/local/lib/python2.7/dist-packages/Trac-1.1.2dev-py2.7.egg/trac/ticket/web_ui.py", line 623, in _process_ticket_request
    valid = self._validate_ticket(req, ticket, not valid) and valid
  File "/usr/local/lib/python2.7/dist-packages/Trac-1.1.2dev-py2.7.egg/trac/ticket/web_ui.py", line 1321, in _validate_ticket
    for field, message in manipulator.validate_ticket(req, ticket):
  File "/srv/trac/sage_trac/plugins/ticket_branch.py", line 288, in validate_ticket
    return []
TypeError: object of type 'listreverseiterator' has no len()

Change History (8)

comment:1 Changed 6 years ago by darij

That actually came from the attempt at changing the branch...

comment:2 Changed 6 years ago by tscrim

  • Cc ohanar added

comment:3 Changed 6 years ago by ohanar

  • Branch set to u/ohanar/fix_csage_deps
  • Commit set to 144d006f9185299c4121eb94bd606ee1fb16d992

comment:4 Changed 6 years ago by ohanar

  • Branch changed from u/ohanar/fix_csage_deps to u/ohanar/prereq
  • Commit changed from 144d006f9185299c4121eb94bd606ee1fb16d992 to 43b696f7f53f21e9a08f8a01a3fe0480e1c3a448

Last 10 new commits:

73f867dadded openssl and dpkg-architecture check
ac675c6make dpgk-architecture check error out, its at least in Ubuntu 8.04
48b54aaTrac #13329: small reviewer fixes
e63837bTrac #13329: Add -lcrypto and -ldl (needed for -lssl), support SAGE_LOCAL
e98db82trac 13385: Remove check for openssl.
cb5e072Trac #14406: remove sqrtl() and SAGE_FORTRAN/SAGE_FORTRAN_LIB checks
0881c74Merge in prereq-1.2
63e1300Integrate prereq in the build system
143e5e4Don't use cp -p in sage-clone-source
68ac77fMerge branch 'u/jdemeyer/ticket/15596' of git://trac.sagemath.org/sage into ticket/15580

comment:5 Changed 6 years ago by ohanar

  • Status changed from new to needs_review

Ok, should be fixed.

comment:6 Changed 6 years ago by ohanar

  • Branch u/ohanar/prereq deleted
  • Commit 43b696f7f53f21e9a08f8a01a3fe0480e1c3a448 deleted

comment:7 Changed 6 years ago by tscrim

  • Milestone changed from sage-6.1 to sage-duplicate/invalid/wontfix
  • Status changed from needs_review to positive_review

I haven't had it come up again, so I believe it's fixed.

comment:8 Changed 6 years ago by vbraun

  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.