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
feat(config)!: Improved configuration and data structures #79
base: main
Are you sure you want to change the base?
feat(config)!: Improved configuration and data structures #79
Changes from all commits
639025b
ffbdff4
ad9c178
bae570c
76cd43c
194bac8
3cb472a
cbe84bd
01d7bc0
7d5a047
83fc4d2
a623237
6f23137
3ffaca2
de33e0c
b2f192e
9a05ef3
c98a90b
25d2c2f
54c731d
f953c31
b352fa1
caf7b1b
0c4568f
c44d231
674e2d7
4d3ef0c
0e319a0
076617b
d3d3674
561ec39
98a16c6
c704906
2e894c1
0ef7471
f449c36
61fd8c7
5c3c066
e6c3fdd
2c7b3a9
bde85aa
af855f5
b9a1690
c9ee0af
8b16f64
872b3eb
c50aa3a
5cdb0be
a7d95f3
3df5eb2
895a3c7
4412a19
de4f32f
f7fb892
9e5ef60
86a8a9a
a9b64e7
edaa60e
091ca0b
f445c1b
5ade973
844cd24
ee3f362
a9f0a3c
b6cbab2
e3f2ecf
6bd4fac
937536f
d7a8d93
e46a015
16310f9
8e780dd
2571f59
cd89e42
3e0be01
80a5d4e
aab3e0b
5490158
65839ad
5fa2d61
fe4dae9
1297328
fc25fa2
cf3475f
26a4340
2524d78
8425db9
6e0913f
ecb6233
6107d77
24257d7
fc628ac
f3e37e3
1cd4a04
c435e6a
956e061
4625ed4
87d781b
3aa7247
98edf14
74510e8
5bf4b27
4a7bbbb
2450d15
3da44eb
3be6a0d
adbc3c6
8e446e9
573f249
a69c4c9
017bb8d
4c35659
ec1c59d
27ddb7c
07157d3
20cd3e1
60b9a5c
6a5b96c
71f807c
87ff410
05ee7da
aca728e
4c6c1cb
17ada22
f646da3
c8a22ef
980debb
d19389c
17eb470
0f1e04a
6bd7b3a
5eec586
bf89a65
2dc4bf4
b19e167
c3baa42
1900b39
3837e4d
4234d74
5126b3b
625595a
dcd019b
15bf0e4
52f5641
ffbed46
fcfcca8
d6baeb9
979b672
0f4455c
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.
I think this new section of the docs will be really useful Helen!
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.
Do you think we should also have some documentation/explanation regarding how custom classes outside of anemoi should be suppported?
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 should be solved by the
no-validation
flag, but that needs clarification in the docs.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 here I'd propose to have a short config example with the
no_validation: True
andno_validation: False
to explain Pydantic behaviour (happy to help I you'd like)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.
Could you add that example to the docs, please?