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

Anonymous community feedback #10

Closed
5 tasks done
Irratzo opened this issue Jun 13, 2023 · 3 comments
Closed
5 tasks done

Anonymous community feedback #10

Irratzo opened this issue Jun 13, 2023 · 3 comments
Labels
configuration Issue refers to the best-of list configuration

Comments

@Irratzo
Copy link
Member

Irratzo commented Jun 13, 2023

Configuration Change:

Anonymous community feedback 2023-06-13.

  • Disable license warnings. Which licenses are risky is a matter of opinion.
  • Add an explanation of the "combined project-quality score".
  • MD category: Add LAMMPS & all the lammps-xxx repo integrating a given potential with lammps; as well as OpenMM and TorchMD, both usable with ML potentials.

Improvement suggestions to the best-of-generator template.

  • Allow projects in multiple categories.
    • For now, the template only allows one category per project. However, one can implement this also by duplicating a project entry for different categories. That should not require a change to the template. The question is, then, where to draw the line of duplication, in order to stay fair to every package. Ultimately, this shows the deficit of the category (tree-based) vs. label (tag-based) classification schemes. Simplest would be to just do this case-by-case, whenever there is a package developer / community request for it. In that case:
    • Put equisolve in multiple categories: math, mlp, general-tools. Developer request.
  • If the way the project score is calculated raises concern, create an issue with improvement suggestions for .
@Irratzo Irratzo added the configuration Issue refers to the best-of list configuration label Jun 13, 2023
@Irratzo
Copy link
Member Author

Irratzo commented Jun 13, 2023

Suggestion "Disable license warnings" implemented in e84fe5d.

@Irratzo
Copy link
Member Author

Irratzo commented Jun 13, 2023

Suggestion 'Add an explanation of the "combined project-quality score"' implemented in PR #18

@Irratzo
Copy link
Member Author

Irratzo commented Aug 23, 2023

Moved OP request wrt "MD category" to issues #77, #78, #79. This action closes this issue.

@Irratzo Irratzo closed this as completed Aug 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
configuration Issue refers to the best-of list configuration
Projects
None yet
Development

No branches or pull requests

1 participant