Opened 4 years ago

Last modified 4 years ago

#21222 new defect

ECM.one_curve can give misleading output

Reported by: embray Owned by:
Priority: minor Milestone:
Component: interfaces Keywords: ecm
Cc: Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description (last modified by embray)

Right now the ECM interface is broken for me on Cygwin, such that calls to ecm are segfaulting. That's a separate issue though (no ticket yet). What bothers me in this case is that the implementation of ECM.one_curve hides lower-level errors like this, by catching any ValueError from ECM._parse_output and returning a valid output as if no factors were found (when in fact the ecm process itself crashed so yes technically no factors were found...).

Really an error should have been raised much earlier, probably from ECM._run_ecm, but it only checks for errors by checking for output on stderr. It does not check the return code, though it probably should.

Change History (1)

comment:1 Changed 4 years ago by embray

  • Description modified (diff)
Note: See TracTickets for help on using tickets.