Opened 2 years ago
Last modified 4 months ago
#29336 needs_work enhancement
Algorithm run time and implementation
Reported by:  ghsurajkrmodi  Owned by:  

Priority:  major  Milestone:  sage9.7 
Component:  graph theory  Keywords:  Diameter 
Cc:  Merged in:  
Authors:  Reviewers:  
Report Upstream:  N/A  Work issues:  
Branch:  u/ghsurajkrmodi/algorithm_run_time_and_implementation (Commits, GitHub, GitLab)  Commit:  df8b76fad50b286f4363a429d06ad6adf56a20df 
Dependencies:  Stopgaps: 
Description (last modified by )
I have included an experimental package for sagemath and package of graph diameter has been successfully installed
Change History (24)
comment:1 Changed 2 years ago by
 Component changed from PLEASE CHANGE to graph theory
 Description modified (diff)
 Keywords Diameter added
 Status changed from new to needs_review
 Type changed from PLEASE CHANGE to enhancement
comment:2 Changed 2 years ago by
 Branch set to u/ghsurajkrmodi/algorithm_run_time_and_implementation
comment:3 Changed 2 years ago by
 Commit set to b607344b9ab51a5f743f6c695e6ba4be61a3c18d
 Status changed from needs_review to needs_work
comment:4 Changed 2 years ago by
 Commit changed from b607344b9ab51a5f743f6c695e6ba4be61a3c18d to f5d36fde2ac5ac0583af3a9b8bd79e1fc833a58a
Branch pushed to git repo; I updated commit sha1. New commits:
f5d36fd  On branch t/29336/algorithm_run_time_and_implementation

comment:5 Changed 2 years ago by
 Commit changed from f5d36fde2ac5ac0583af3a9b8bd79e1fc833a58a to 533c7a41be00c48e3ce605a68ff64a3076ead668
Branch pushed to git repo; I updated commit sha1. New commits:
533c7a4  deleted: sample/lol.tsv

comment:6 Changed 2 years ago by
 Commit changed from 533c7a41be00c48e3ce605a68ff64a3076ead668 to 8306b28b4cec72ef3f69342d10e914e1a5740c0d
Branch pushed to git repo; I updated commit sha1. New commits:
8306b28  deleted: myscript.py

comment:7 Changed 2 years ago by
 Commit changed from 8306b28b4cec72ef3f69342d10e914e1a5740c0d to 29d11ec78631597d99ba65b205100654def92ad4
Branch pushed to git repo; I updated commit sha1. New commits:
29d11ec  modified: src/sage/graphs/bipartite_graph.py

comment:8 Changed 2 years ago by
 Commit changed from 29d11ec78631597d99ba65b205100654def92ad4 to 391d37acf43c3ec13c1f3dcc39434d40a2118058
Branch pushed to git repo; I updated commit sha1. New commits:
391d37a  new file: build/pkgs/graphdiameter/SPKG.txt

comment:9 Changed 2 years ago by
 Description modified (diff)
 Status changed from needs_work to needs_review
comment:10 Changed 2 years ago by
error i got while running sage package sagerun received unknown option: package usage: sage [options] Try 'sage h' for more information.
comment:11 Changed 2 years ago by
 Commit changed from 391d37acf43c3ec13c1f3dcc39434d40a2118058 to 68139bcd5da3c331428cad87d00c4d42e163328c
Branch pushed to git repo; I updated commit sha1. New commits:
68139bc  modified: build/pkgs/graphdiameter/checksums.ini

comment:12 Changed 2 years ago by
 Commit changed from 68139bcd5da3c331428cad87d00c4d42e163328c to 7d22f2700d9711a22c540b872af2b90e82207045
Branch pushed to git repo; I updated commit sha1. New commits:
7d22f27  modified: build/pkgs/graphdiameter/spkginstall

comment:13 Changed 2 years ago by
 Description modified (diff)
waiting for review and further instructions
comment:14 Changed 2 years ago by
 Commit changed from 7d22f2700d9711a22c540b872af2b90e82207045 to df8b76fad50b286f4363a429d06ad6adf56a20df
Branch pushed to git repo; I updated commit sha1. New commits:
df8b76f  modified: build/pkgs/graphdiameter/SPKG.txt

comment:15 Changed 2 years ago by
I don't have the expertise to check if this is the right way to make an experimental package. I will call for help. Meanwhile, where can I find the tarball ?
comment:16 Changed 2 years ago by
 Milestone changed from sage9.1 to sage9.2
Batch modifying tickets that will likely not be ready for 9.1, based on a review of the ticket title, branch/review status, and last modification date.
comment:17 Changed 2 years ago by
 Status changed from needs_review to needs_work
Information how to make a new package is here: https://doc.sagemath.org/html/en/developer/packaging.html
Note that a new feature is to provide the upstream_url
directly in the checksums file, see for instance #30342.
Because the upstream url is not provided, I change the status to needs work.
comment:18 Changed 2 years ago by
Also it seems the branch is doing a chmod
change to the file src/sage/graphs/centrality.pyx
which is weird.
comment:19 Changed 2 years ago by
 Milestone changed from sage9.2 to sage9.3
comment:20 Changed 18 months ago by
 Milestone changed from sage9.3 to sage9.4
Setting new milestone based on a cursory review of ticket status, priority, and last modification date.
comment:21 Changed 14 months ago by
May be it's time to move this ticket to wontfix.
comment:22 Changed 13 months ago by
 Milestone changed from sage9.4 to sage9.5
Setting a new milestone for this ticket based on a cursory review.
comment:23 Changed 8 months ago by
 Milestone changed from sage9.5 to sage9.6
comment:24 Changed 4 months ago by
 Milestone changed from sage9.6 to sage9.7
This is not the right direction. If I understand well, you have copy/paste the code of Yuki Kawata, and do various edits in different files.
I you want to use the code of Yuki Kawata, you can propose an interface to this code and make it an optional package of Sagemath (at first an experimental package). See the developper manual.
New commits:
First commit