Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Triage inaccessible, undocumented, and/or unused proposals and decide which to fix and which to remove #288

Open
krivit opened this issue May 5, 2021 · 2 comments

Comments

@krivit
Copy link
Member

krivit commented May 5, 2021

A number of proposals have accumulated, particularly in the C code, many of which are one or more of the following:

Some need to be salvaged, others removed. I have compiled what I am pretty sure is all the proposal functions we have and created a checklist for each of them. @drh20drh20 , @handcock , @CarterButts , you are probably the only ones who remember how and why some of these ended up in the package, and whether or not they still have any value. If you could take a minute to go through the list and tell me if any of these are no longer worth keeping, that would save a lot of time.

Edit: It appears that the checklists in the table isn't working as expected. Here they are in a flat list. Apologies for the length.

BipartiteHammingConstantEdges

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

BipartiteHammingTNT

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

BipartiteCondDegreeDist

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

randomtoggle

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

TNT

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

BDStratTNT

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

TNT10

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

ConstantEdges

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

CondDegreeDist

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

CondOutDegreeDist

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

CondInDegreeDist

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

TwoRandomToggles

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

randomnode

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

ConstrainedCondOutDegDist

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

NodePairedTiesToggles

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

OneRandomTnTNode

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

ReallocateWithReplacement

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

AllTogglesForOneNode

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

SwitchLabelTwoNodesToggles

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

ConstrainedCondDeg

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

ConstrainedSwitchLabelTwoNodesToggles

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

ConstrainedCondDegDist

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

ConstrainedNodePairedTiesToggles

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

ConstrainedReallocateWithReplacement

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

ConstrainedAllTogglesForOneNode

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

ConstrainedTwoRandomToggles

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

ConstantEdgesToggles

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

CondDegSwitchToggles

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

CondDegreeTetrad

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

CondDegreeMix

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

CondDegreeTetradMixMore

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

CondDegreeTetradMixLess

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

CondDegreeMixChangeOrig

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

CondDegreeMix

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

CondDegreeHexad

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

CondDegree

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

CondOutDegree

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

CondInDegree

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

CondB1Degree

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

CondB2Degree

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

dyadnoiseTNT

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

dyadnoisemTNT

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

dyadnoise

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

dyadnoisem

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

StdNormal

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

Unif

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

UnifNonObserved

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

DiscUnif

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

DiscUnif2

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

DiscUnifNonObserved

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved

DistRLE

  • Does not use defunct APIs
  • Is accessible from R
  • Is in the lookup table
  • Is tested
  • Resolve: Keep
  • Resolve: Remove
  • Resolved
@krivit
Copy link
Member Author

krivit commented May 5, 2021

P.S. @handcock , @drh20drh20 , @CarterButts , @mbojan , @chad-klumb , if you know the status of any of these off the top of your head (e.g., you wrote the test or you use it regularly) please take a moment to tick the corresponding boxes.

@chad-klumb
Copy link
Contributor

I checked the one I wrote (BDStratTNT), and don't know much about the others.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants