Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Hypothesis strategy for generating Variable objects #8404
Hypothesis strategy for generating Variable objects #8404
Changes from all commits
587ebb8
acbfa69
73d763f
db2deff
746cfc8
03cd9de
2fe3583
4db3629
14d11aa
418a359
c8a7d0e
d48aceb
a20e341
3a4816f
d0406a2
65a222d
e1d718a
57d0f5b
82c734c
029f19a
46895fe
50c62e9
e21555a
1688779
0a29d32
3259849
717fabe
d76e5b6
c25940c
cd7b065
742b18c
8e548b1
d1487d4
c8b53f2
8bac610
cf3beb5
d991357
a6405cf
400ae3e
3609a34
63ad529
4ffbcbd
469482d
472de00
ced1a9f
a3c9ad0
b387304
404111d
3764a7b
9723e45
2e44860
1cc073b
603e6bb
63bb362
69ec230
e5c7e23
fd3d357
52f2490
9b96470
41fe0b4
0e53aa1
f659b4b
d1be3ee
e88f5f0
4b88887
2a1dc66
9bddcec
b2887d4
3b8e8ae
0980061
0313b3e
e6ebb1f
4da8772
e6d7a34
5197d1b
15812fd
3dc9c7b
4374681
0f0c4fb
6a30af5
cac46dc
177d908
5424e37
c871273
7730a27
24549bc
3082a09
5df60dc
01078de
53290e2
bd2cb6e
c5e83c2
de26b2f
f81e14f
129e2c3
601d9e2
af24af5
9777c2a
4dcbc60
7841dd5
968ee72
fd6aa06
df3341e
de4de8f
af14dc2
d001dbb
d4c9cb5
7983e34
2ad7bb0
a9f7cd5
49a1c64
c1f1974
6482ad3
95cab79
839c4f0
ded711a
f3c80ed
010f28c
4b07992
ba99afa
700d652
0e01d76
79f40f0
4ff57ec
959222e
78825c4
2418a61
331f521
adca1d2
14998c1
71f01f9
9c10895
2833f01
1ddc515
618bfea
fe1ff1a
2e038ea
04c3dc1
cf35fb9
4811e8a
a036253
054a0dc
ececa07
0fa090d
a9ac6f1
43831ce
62dbe88
af5eb25
dc78254
5822390
eeb6b32
e13c6ac
a169e1f
46b36b9
00ed3d6
d265ddb
0e872a8
a941e60
3d43ed6
6c912d2
bdf3aed
afd526d
6bbd13b
29ecd7d
631e810
c613027
4412d98
1ea0dcf
cf1a45e
ea738cd
79b0094
c6d43ca
00079bd
cbcd486
69ddd08
ea90162
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The intention of creating this page is that material on using
xarray.testing.assert_equal
etc. should go here, before the material on hypothesis.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perhaps this whole page should go under the xarray internals section of the docs instead of the user guide? Because this realistically is only going to be used by other library developers, not most users.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure. It is true that the page has a different target audience than the other pages in the user guide, but then again applications can also be tested. And, so far the "internals" section describes implementation details or extension mechanisms that affect the internals.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we can work this out later. This location seems fine for now, and changing it isn't a backwards-compatibility issue.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This feels like a great place to introduce the
shared()
strategy, which is the idiomatic way to meet a common need like "variables might have a z-dimension, which can vary, but if present must be the same size in all variables". It's a complication, but one which I expect many readers will benefit from.