Skip to content

Commit 657338a

Browse files
Add issue and PR templates (ddmms#29)
1 parent b15d1c8 commit 657338a

File tree

10 files changed

+283
-0
lines changed

10 files changed

+283
-0
lines changed
Lines changed: 49 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,49 @@
1+
name: New benchmark
2+
description: Propose or discuss a new benchmark for existing models
3+
labels: ["new benchmark"]
4+
body:
5+
6+
- type: textarea
7+
attributes:
8+
label: Summary
9+
description: |
10+
A clear and concise summary of the new benchmark. What application and properties are of interest?
11+
validations:
12+
required: true
13+
14+
- type: textarea
15+
attributes:
16+
label: Interactive features
17+
description: |
18+
How would you like to explore the results of your benchmark?
19+
validations:
20+
required: false
21+
22+
- type: textarea
23+
attributes:
24+
label: Category
25+
description: |
26+
Which category does the new benchmark belong in? This can already exist, or you may propose a new one.
27+
validations:
28+
required: false
29+
30+
- type: textarea
31+
attributes:
32+
label: Data availability
33+
description: What reference data is available? If this is theoretical, at what level of theory?
34+
validations:
35+
required: false
36+
37+
- type: textarea
38+
attributes:
39+
label: Computational cost
40+
description: If possible, estimate how long the benchmark would take to run on a typical model.
41+
validations:
42+
required: false
43+
44+
- type: textarea
45+
attributes:
46+
label: Additional context
47+
description: Add any other context, or example implementations.
48+
validations:
49+
required: false
Lines changed: 44 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,44 @@
1+
name: New model
2+
description: Propose or discuss benchmarking a new model.
3+
labels: ["new model"]
4+
body:
5+
6+
- type: textarea
7+
attributes:
8+
label: Summary
9+
description: |
10+
A clear and concise description of the model. Please include any relevant references.
11+
validations:
12+
required: true
13+
14+
- type: textarea
15+
attributes:
16+
label: Model availability
17+
description: |
18+
Is the model publicly available? Please share any links.
19+
validations:
20+
required: true
21+
22+
- type: textarea
23+
attributes:
24+
label: Training data
25+
description: |
26+
What data was used to train the model? Is this publicly available?
27+
validations:
28+
required: true
29+
30+
- type: textarea
31+
attributes:
32+
label: Model architecture
33+
description: |
34+
Does the model require a new calculator class, or are other models of similar
35+
architecture already supported?
36+
validations:
37+
required: false
38+
39+
- type: textarea
40+
attributes:
41+
label: Additional context
42+
description: Add any other context, or examples of use.
43+
validations:
44+
required: false
Lines changed: 29 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,29 @@
1+
name: Bug report
2+
description: Create a report to help us improve
3+
labels: ["bug"]
4+
body:
5+
- type: textarea
6+
attributes:
7+
label: Summary
8+
description: |
9+
A clear and concise description of the bug, including a minimal reproducible example.
10+
11+
Please include your full output with the complete error message.
12+
validations:
13+
required: true
14+
15+
- type: input
16+
attributes:
17+
label: Platform
18+
description: What operating system and architecture are you using? (e.g. `uname -orsm`)
19+
placeholder: e.g., macOS 14 arm64, Windows 11 x86_64, Ubuntu 20.04 amd64
20+
validations:
21+
required: true
22+
23+
- type: input
24+
attributes:
25+
label: Python version
26+
description: What version of Python are you using? (e.g. `python3 --version`)
27+
placeholder: e.g., Python 3.12.6
28+
validations:
29+
required: false
Lines changed: 39 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,39 @@
1+
name: Feature request
2+
description: Suggest a new feature or improvement
3+
labels: ["enhancement"]
4+
body:
5+
- type: markdown
6+
attributes:
7+
value: |
8+
**Please select the `New benchmark` issue template to propose or discuss new benchmarks.**
9+
10+
- type: textarea
11+
attributes:
12+
label: Summary
13+
description: |
14+
A clear and concise description of what new feature or behaviour you would like to see.
15+
16+
If applicable, please describe the current behaviour as well.
17+
validations:
18+
required: true
19+
20+
- type: textarea
21+
attributes:
22+
label: Example
23+
description: Provide an example of how the user experience would change or how the new feature would be used.
24+
validations:
25+
required: false
26+
27+
- type: textarea
28+
attributes:
29+
label: Alternatives
30+
description: A clear and concise description of any alternative solutions or features you've considered.
31+
validations:
32+
required: false
33+
34+
- type: textarea
35+
attributes:
36+
label: Additional context
37+
description: Add any other context or screenshots about the feature request here.
38+
validations:
39+
required: false
Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
name: Question
2+
description: Ask a question about mlip-testing
3+
labels: ["question"]
4+
body:
5+
- type: markdown
6+
attributes:
7+
value: |
8+
**Please select the `New benchmark` issue template to propose or discuss new benchmarks.**
9+
10+
- type: textarea
11+
attributes:
12+
label: Question
13+
description: Describe your question in detail.
14+
validations:
15+
required: true
16+
17+
- type: input
18+
attributes:
19+
label: Platform
20+
description: What operating system and architecture are you using? (e.g. `uname -orsm`)
21+
placeholder: e.g., macOS 14 arm64, Windows 11 x86_64, Ubuntu 20.04 amd64
22+
validations:
23+
required: false

.github/ISSUE_TEMPLATE/config.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
blank_issues_enabled: true
2+
contact_links:
3+
- name: Documentation
4+
url: https://ddmms.github.io/mlip-testing/
5+
about: Please consult the documentation before creating an issue.

.github/PULL_REQUEST_TEMPLATE.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
Please click the `Preview` tab and select a PR template:
2+
3+
- [New benchmark for existing models](?expand=1&template=new_benchmark.md&labels=new+benchmark)
4+
- [Benchmark a new model](?expand=1&template=new_model.md&labels=new+model)
5+
- [All other proposed changes](?expand=1&template=custom.md)
Lines changed: 26 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,26 @@
1+
<!--
2+
Thank you for contributing! To help us out with reviewing, please consider the following:
3+
4+
- Does this pull request include a summary of the change (see below)?
5+
- Does this pull request include a descriptive title?
6+
- Does this pull request link to an issue (see below)?
7+
-->
8+
9+
## Pre-review checklist for PR author
10+
11+
PR author **must** check the checkboxes below when creating the PR.
12+
13+
- [ ] I've confirmed the [contribution guidelines](https://github.com/ddmms/mlip-testing/blob/main/contributing.md).
14+
15+
## Summary
16+
17+
<!-- Describe your proposed changes. This can be brief, as most information can be in the linked issue. -->
18+
19+
## Linked issue
20+
21+
<!-- Enter the number of the issue this resolves. -->
22+
Resolves #
23+
24+
## Testing
25+
26+
<!-- How have your proposed changes been tested? -->
Lines changed: 37 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,37 @@
1+
<!--
2+
Thank you for contributing! To help us out with reviewing, please consider the following:
3+
4+
- Does this pull request include a summary of the change (see below)?
5+
- Does this pull request include a descriptive title?
6+
- Does this pull request link to a "new benchmark" issue (see below)?
7+
-->
8+
9+
## Pre-review checklist for PR author
10+
11+
PR author **must** check the checkboxes below when creating the PR.
12+
13+
- [ ] I've confirmed the [contribution guidelines](https://github.com/ddmms/mlip-testing/blob/main/contributing.md).
14+
15+
## Summary
16+
17+
<!-- Summarise the new benchmark. This can be brief, as most information should be in the linked issue. -->
18+
19+
## Linked issue
20+
21+
<!-- Enter the number of the issue this resolves. This should be labelled as "new benchmark" -->
22+
Resolves #
23+
24+
## Progress
25+
26+
<!-- Which aspects of the new benchmark are complete? -->
27+
- [ ] Calculations
28+
- [ ] Analysis
29+
- [ ] Application
30+
31+
## Testing
32+
33+
<!-- Which models have you tested your benchmark on? -->
34+
35+
## New decorators/callbacks
36+
37+
<!-- Are new callbacks required for the interactivity you need? -->
Lines changed: 26 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,26 @@
1+
<!--
2+
Thank you for contributing! To help us out with reviewing, please consider the following:
3+
4+
- Does this pull request include a summary of the change (see below)?
5+
- Does this pull request include a descriptive title?
6+
- Does this pull request link to a "new model" issue (see below)?
7+
-->
8+
9+
## Pre-review checklist for PR author
10+
11+
PR author **must** check the checkboxes below when creating the PR.
12+
13+
- [ ] I've confirmed the [contribution guidelines](https://github.com/ddmms/mlip-testing/blob/main/contributing.md).
14+
15+
## Summary
16+
17+
<!-- Describe the new model. This can be brief, as most information should be in the linked issue. -->
18+
19+
## Linked issue
20+
21+
<!-- Enter the number of the issue this resolves. This should be labelled as "new model" -->
22+
Resolves #
23+
24+
## Testing
25+
26+
<!-- Which benchmarks have you tested your model against? -->

0 commit comments

Comments
 (0)