Opened 3 years ago

Closed 2 years ago

#22190 closed enhancement (wontfix)

replace braid with mapclass in gap_packages

Reported by: dimpase Owned by:
Priority: major Milestone: sage-duplicate/invalid/wontfix
Component: packages: optional Keywords: gap packages
Cc: slelievre Merged in:
Authors: Reviewers: Travis Scrimshaw
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: #13426 Stopgaps:

Description (last modified by slelievre)

All the functionality of braid is in ​mapclass, according to upstream. We should replace braid then.

Change History (5)

comment:1 Changed 3 years ago by tscrim

braid was either never included or is no longer: 

sage: gap_reset_workspace(verbose=True)
Loading GAP package GAPDoc
Loading GAP package ctbllib
Loading GAP package sonata
Loading GAP package guava
Loading GAP package factint
Loading GAP package gapdoc
Loading GAP package grape
Loading GAP package design
Loading GAP package toric
Loading GAP package laguna
Loading GAP package braid
*** Error loading Gap package braid. You may want to install the gap_packages and/or database_gap SPKGs.

comment:3 Changed 2 years ago by slelievre

  • Cc slelievre added
  • Dependencies set to #13426
  • Description modified (diff)
  • Keywords gap packages added
  • Milestone changed from sage-7.6 to sage-duplicate/invalid/wontfix
  • Status changed from new to needs_review
  • Type changed from defect to enhancement

Given that braid was already removed and that mapclass is getting added in #13426, let us close this ticket after #13426 is merged.

comment:4 Changed 2 years ago by tscrim

  • Reviewers set to Travis Scrimshaw
  • Status changed from needs_review to positive_review

comment:5 Changed 2 years ago by vdelecroix

  • Resolution set to wontfix
  • Status changed from positive_review to closed

closing positively reviewed duplicates

Note: See TracTickets for help on using tickets.