Skip to content

Conversation

@Mentors4EDU
Copy link
Contributor

Added ChaosSIM, Docking@Home, and Cancer@HomeV2. Fixed library alphabetical orders.

Added ChaosSIM, Docking@Home, and Cancer@HomeV2. Fixed library alphabetical orders.
Copy link
Contributor

@Wauplin Wauplin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR @Mentors4EDU ! I've left a few comments to address but should be quick

filter: false,
countDownloads: `path:"boltzgen1_diverse.ckpt"`,
},
cancertathomev2: {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
cancertathomev2: {
"cancer-at-home-v2": {

=> to be connected with https://huggingface.co/OpenPeerAI/CancerAtHomeV2

},
cancertathomev2: {
prettyLabel: "Cancer@HomeV2",
repoName: "CancerAtHomeV2",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
repoName: "CancerAtHomeV2",
repoName: "Cancer@HomeV2",

let's keep the same as prettyLabel for consistency

repoName: "CancerAtHomeV2",
repoUrl: "https://huggingface.co/OpenPeerAI/CancerAtHomeV2",
filter: false,
countDownloads: `path:"setup.py"`,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is no setup.py in the repo. Maybe run.py? Or config.yml? Or setup.sh/setup.ps1?

filter: false,
countDownloads: `path:"asset/GPT.pt"`,
},
},
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
},
},

},
dockingathome: {
prettyLabel: "Docking@Home",
repoName: "docking-at-home",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
repoName: "docking-at-home",
repoName: "Docking@Home",

Let's keep the same for consistency

repoUrl: "https://github.com/argmaxinc/DiffusionKit",
snippets: snippets.diffusionkit,
},
dockingathome: {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
dockingathome: {
"docking-at-home": {

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants