llama.cpp
hparam comparison uses memcpy for floating point comparison, which is in general a bad idea
#3446
Merged

hparam comparison uses memcpy for floating point comparison, which is in general a bad idea #3446

l3utterfly
l3utterfly fixed floating point comparison issues
66f2063d
cebtenzzre
cebtenzzre commented on 2023-10-02
ggerganov
ggerganov approved these changes on 2023-10-03
l3utterfly updated implementation for hparam comparison to handle inf and NaN
16f45c4d
cebtenzzre
cebtenzzre commented on 2023-10-04
l3utterfly fixed code review comments
d9cb48f0
cebtenzzre minor simplification
bde943e6
cebtenzzre
cebtenzzre approved these changes on 2023-10-04
cebtenzzre rename is_float_eq -> is_float_close
4cffcb84
ggerganov ggerganov merged 16820a5a into master 2 years ago
l3utterfly l3utterfly deleted the hparam-compare branch 2 years ago

Login to write a write a comment.

Login via GitHub

Reviewers
Assignees
No one assigned
Labels
Milestone