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
Add multi-objective global memory search algorithm #493
Add multi-objective global memory search algorithm #493
Changes from all commits
a998154
ceaa344
0f66a00
920dc12
dabf46e
28ef865
ac3aa1b
5108704
1861725
140a6ac
37c82ba
6fde3c6
7327852
add907a
825a0c9
3c39f8c
afcaa43
ca577e3
99b4d7e
08072ac
d496816
b1abaeb
1820eea
83ca529
98d20fe
ca88495
c13ba1a
dbe249f
6455550
4c4194c
1e859d9
3ff36e0
5bbc208
f8250cf
2f84e30
fa1a1de
a1b34d8
1d5bf42
4ae92c1
22c7cf9
f17a50c
dc71164
e68de3e
394dae6
c9c7293
4bbeae7
42877ff
41719ec
cda8ad8
02c9fe7
f36d10c
092468c
bc0340d
a71ec09
4a283bd
46f9c9c
734bcb0
2f88cae
7813c4d
7dff97c
cd53288
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.
Could we get a more useful name here? Maybe
usage_in_bytes
or something?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.
Hmm, it's not necessarily in bytes I think. I agree that a more verbose name might be better, but a simple "num" should be easy-to-understand that it is holding a floating point number of the usage. So, it's fine to keep this as is.
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.
What units is
num
in then, if not bytes?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.
Note that it's a float, not size_t. Also, changing the name needs a lot of effort and I prefer to leave it in a future PR. I can possibly change this in the next PR.
Thanks
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.
Great, can you create an issue for this?
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.
#620