patch 1.7.1 broke grids_noresponse_cc
Open
Nobody has claimed this yet.
- Dominant language
- Cuda
- Stars
- 351
- Forks
- 84
- Avg merge
- 3d 2h
- Merged PRs (30d)
- 35
Description
The patch 1.7.1 replaced grids_noresponse_cc by a stub which is currently used in the microsoft/skala repo to evaluate nuclear gradients. I think such an impactful change should not have been part of a patch.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start at the grids_noresponse_cc entry point and compare its behavior before and after patch 1.7.1. Check how microsoft/skala uses it to evaluate nuclear gradients. Done means restoring the expected non-stub behavior without the patch-level regression and confirming that this use case works.
Written by the indexing model from the issue text.
Assessment
- Domain
- hpc
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 48/100