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

Submitting CrabNet #23

Merged
merged 3 commits into from
Jul 28, 2021
Merged

Submitting CrabNet #23

merged 3 commits into from
Jul 28, 2021

Conversation

MahamadSalah74
Copy link
Contributor

Matbench Pull Request Template

Thanks for making a PR to Matbench! We appreciate your contribution (like, a lot). To make things run smoothly, check out the following templates,
depending on what kind of PR you are making.

If you are making a benchmark submission (i.e., you have tried an algorithm on Matbench and want to appear on the leaderboard),
please use the template under Benchmark submissions.

If you are making changes to the core matbench code, data, or docs, please use the template under Core code/data/docs changes.

Benchmark submissions

Benchmark submissions can include a full benchmark on any of the benchmarks Matbench submits, as well as any subset of tasks within a benchmark (e.g., 3 Matbench v0.1 tasks your algorithm supports).

Brief description of your algorithm

You should a brief description of your algorithm in the pull request body. This can include any details you'd like.

Included files

If you are making a benchmark submission, please only include the submission as a folder in the /benchmarks directory with the format <benchmark_name>_<algorithm_name>. Your PR should have no other changes to the core code.
The submission should have these three required files, as indicated in the
docs:

Example

-- benchmarks
---- matbench_v0.1_my_algorithm
------ results.json.gz             # required filename
------ notebook.ipynb              # required filename
------ info.json                   # required filename

Please make sure each of these files has the information specified in the docs.

If you have other short/small files required for the notebook, please give a brief overview of what each one is used for and how to use it.

Label the pull request

Label the pull request with the new_benchmark label.

Core code/data/docs changes

Brief description of changes

Please include a brief description of the changes you are making, in bullet point format.

Tests

Indicate if your code requires new tests and whether they are included with your PR. ALL core code/data/docs changes adding new features must have new tests for them.

Closed issues or PRs

Indicate if your PR closes any currently open issues or supercedes any other currently open PRs.

Label the pull request

Label the pull request with the code or docs labels, depending on which one (or both) applies.

@ardunn
Copy link
Collaborator

ardunn commented Jul 21, 2021

Hey @MahamadSalah !

Thanks for submitting! If tests pass I will merge and results will appear on the website immediately after.

Do you have any thoughts on using the matbench python package to record results? Areas for improvement or things that could be done better?

@ardunn
Copy link
Collaborator

ardunn commented Jul 21, 2021

@MahamadSalah it looks like a notes field is needed in info.json. This can just be blank if you don't want to add any actual notes but the field is required.

Also for requirements field in info.json, could you pin the version of crabnet used?

@MahamadSalah74
Copy link
Contributor Author

Hey @ardunn, thanks for your feedback. I have updated the info.json file. Please let me know if any other problems arise while running the tests.

@ardunn ardunn merged commit e283b41 into materialsproject:main Jul 28, 2021
@ardunn
Copy link
Collaborator

ardunn commented Jul 28, 2021

@MahamadSalah I had to fix a few things with the test, not your fault the check failed again! It has been corrected on my side and Crabnet should be appearing on the website momentarily

@MahamadSalah74
Copy link
Contributor Author

Awesome, thanks a lot @ardunn!

@ardunn
Copy link
Collaborator

ardunn commented Jul 28, 2021

Looks like I got it to build correctly, see screenshots:

image

image

image

@ardunn
Copy link
Collaborator

ardunn commented Jul 28, 2021

and thank you @MahamadSalah !!

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.

None yet

2 participants