datatree: Use new numpy random number generator in docs example
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 4.2k
- Forks
- 1.4k
- Avg merge
- 2d 15h
- Merged PRs (30d)
- 14
Description
What is your issue?
Hi folks, just getting started with datatree and hoping it's the solution i've been searching for :) Just opening a very minor PR I noticed while going through the intro tutorial hands-on. I usually keep my Ruff linting settings pretty strict and it chimed in with the NPY002 linting rule. I have a patch branch for on my fork that addresses this change, and I'd be happy to open a PR with a little more contributors' guidance and encouragement 🚀
Originally posted by @schlich in https://github.com/xarray-contrib/datatree/issues/290#issue-2027711482
Great! Please let us know any other thoughts you have as you explore the package :)
I'd be happy to open a PR with a little more contributors' guidance and encouragement 🚀
Definitely PRs welcome! Most things from xarray's contributing guide apply here also. Feel free to ping me with questions.
Originally posted by @TomNicholas in https://github.com/xarray-contrib/datatree/issues/290#issuecomment-1842136357
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 with the datatree intro tutorial example mentioned in the issue and compare it with the contributor's patch branch. Check Ruff's NPY002 guidance, update the example to use NumPy's newer random-number generator, and verify the documentation example no longer triggers that lint rule.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- numpy, python
- Domain
- documentation
- Issue type
- Documentation
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100