cords icon indicating copy to clipboard operation
cords copied to clipboard

Replace apricot by submodlib in cords

Open rishabhk108 opened this issue 3 years ago • 2 comments

rishabhk108 avatar Jun 26 '21 18:06 rishabhk108

@krishnatejakk I can help out with this if you'd like.

ghost avatar Mar 08 '22 17:03 ghost

@MichaelSpencerA Sure that sounds good. Please go ahead and make the necessary changes. While making the changes, please keep the following things in mind:

  • Update the documentation wherever necessary while replacing apricot with submodlib.
  • Things that currently use apricot in CORDS are submodular strategies and CRAIG strategy in SL and SSL settings.
  • Apart from this, make the greedy algorithm option in submodlib an option of CRAIG data loader and strategy.

Once you are done with the required changes, please raise a PULL request and one of us will review the code and merge it to main branch.

krishnatejakk avatar Mar 08 '22 20:03 krishnatejakk