Opened 8 years ago

Closed 5 years ago

#11730 closed enhancement (invalid)

do not load pkg_resources on startup

Reported by: saraedum Owned by: jason, mpatel, was
Priority: minor Milestone: sage-duplicate/invalid/wontfix
Component: notebook Keywords: sd32
Cc: Merged in:
Authors: Julian Rueth Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description

The attached patch removes pkg_resources from being loaded on sage startup: 

$ ./sage --startuptime|grep pkg_resources
                  pkg_resources: 0.029

The patch improves the situation from 

$ dumbbench -- ./sage --startuptime
Ran 22 iterations (2 outliers).
Rounded run time per iteration: 9.433e-01 +/- 7.2e-03 (0.8%)

to 

$ dumbbench -- ./sage --startuptime
Ran 23 iterations (3 outliers).
Rounded run time per iteration: 8.903e-01 +/- 8.3e-03 (0.9%)

Attachments (2)

trac_11730_pkg_resources_sage.patch​ (577 bytes) - added by saraedum 8 years ago.
patch for sage
trac_11730_pkg_resources_sagenb.patch​ (15.2 KB) - added by saraedum 8 years ago.
patch for sagenb

Download all attachments as: .zip

Change History (10)

Changed 8 years ago by saraedum

patch for sage

Changed 8 years ago by saraedum

patch for sagenb

comment:1 Changed 8 years ago by saraedum

  • Status changed from new to needs_review

comment:2 Changed 8 years ago by saraedum

  • Authors changed from Julian Rüth to Julian Rueth

comment:3 Changed 8 years ago by was

  • Keywords sd32 added

comment:4 Changed 6 years ago by jdemeyer

  • Milestone changed from sage-5.11 to sage-5.12

comment:5 Changed 5 years ago by vbraun_spam

  • Milestone changed from sage-6.1 to sage-6.2

comment:6 Changed 5 years ago by vbraun_spam

  • Milestone changed from sage-6.2 to sage-6.3

comment:7 Changed 5 years ago by saraedum

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

pgk_resources are not loaded anymore.

comment:8 Changed 5 years ago by vbraun

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