gap-packages / qpa

GAP package for quivers and path algebras
https://folk.ntnu.no/oyvinso/QPA/
GNU General Public License v2.0
30 stars 13 forks source link

Nextpathfix #82

Closed HereComesTheMoon closed 1 year ago

HereComesTheMoon commented 1 year ago

See #76.

  1. Added a test to ensure that the paths in example quivers are enumerated in the right order.
  2. Rewrote NextPath to have much higher performance. (This matters for certain computations with path algebras or quivers with 10+ arrows).
  3. Rewrote the \[\] method for QuiverEnumerators, and the QuiverIterator wrappers to ensure they still work nicely. All tests pass.
codecov[bot] commented 1 year ago

Codecov Report

Merging #82 (07c4594) into master (4c8d801) will increase coverage by 0.09%. The diff coverage is 79.59%.

@@            Coverage Diff             @@
##           master      #82      +/-   ##
==========================================
+ Coverage   60.62%   60.72%   +0.09%     
==========================================
  Files          74       74              
  Lines       22577    22513      -64     
==========================================
- Hits        13688    13670      -18     
+ Misses       8889     8843      -46     
Impacted Files Coverage Δ
lib/quiver.gi 74.74% <79.59%> (+3.46%) :arrow_up:
sunnyquiver commented 1 year ago

Thank you very much for improving the performance of the quiver enumerator.