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
ValidateAvailability implementation #270
ValidateAvailability implementation #270
Changes from 47 commits
3c24d62
1f12d6b
d8f9190
1a5f9e3
b69c0af
6dff524
25a521d
e692b0e
0d6b0f5
e4be5d4
1b2335a
28c6dee
ca8c5b2
cc7b7c4
0b8c2ca
baf2fcc
af6a815
d10e141
60afd13
4199979
bb7fd2f
f5677be
152bb82
98e1d72
ffe1719
92e4132
eba479b
0879f7a
43d420b
0510c67
69a7037
48360ca
20d4c19
50b691e
b44b303
b1276a0
ad4e5b5
cef85c7
7171843
ac324ec
cf64d14
d93f9b7
23c9d06
63e5acb
b517a38
adac1fb
0d7669e
d2fede3
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.
Optional: I think the name
generateSamples
or simplygenerate
would be clearer. Otherwise, it sounds like this would already sample the network.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.
Can't agree here. Generate sound like producing something imho, also from the name
squareSampler
the methodsample
look more idiomaticThere 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.
We are producing something: random numbers which comprise
Sample
objects. These are then used to actually sample the network (later, but not here). Doesn't really matter much as this is a private method though.