minhyukpark / facial-recognition Goto Github PK

Expected behaivor

Matvec multiply should give the correct answer

Actual behavior

Matvec multiply gives incorrect answer

Steps to reproduce the behavior

call Matvec multiply on a matrix and a vector

(Potential Fixes or links to pull requests that address this issue)

using vec->data as opposed to vec that is then passed into the dot_product

Expected behaivor

Be able to use break points in gdb to go line by line through code

Actual behavior

Error "linalg.c: No such file or directory"

Steps to reproduce the behavior

1. make debug
2. gdb ./facerec-debug
3. b linalg.c:(linenumber)
4. r

(Potential Fixes or links to pull requests that address this issue)

Multi-leveled compilation in MakeFile

Expected behaivor

compute_average should only compute average, taking in the vector of
images as its parameters along with the dimensions of each image

compute_average(vector* images, size_t width, size_t height);

Actual behavior

Currently the compute_average function creates a dummy data of some sort.

(Potential Fixes or links to pull requests that address this issue)

Pull Request #34 does not properly implement this

Expected behavior

Make debug creates an debug-version executable just like make

Actual behavior

Creates the debug-version executable, but also duplicates linalg.h and linalg.c into the src folder.

Steps to reproduce the behavior

• make debug

(Potential Fixes or links to pull requests that address this issue)

• Remove the duplicate files in the MakeFile

Expected behaivor

.obj directory is removed after a build, whether successful or not

Actual behavior

.obj directory is present after making release

Steps to reproduce the behavior

'make all'
.obj directory is in the root directory

(Potential Fixes or links to pull requests that address this issue)

Expected behaivor

Doxygen comments should be /** */

Actual behavior

The Doxygen Comment itself is commented

Steps to reproduce the behavior

lines 179-185 in linalg.c

(Potential Fixes or links to pull requests that address this issue)

Expected behaivor

No user config files or non-repostiory related files should be pushed

Actual behavior

.vscode was included in the newest Pull Request #36

(Potential Fixes or links to pull requests that address this issue)

Please remove the folder and recommit.