search

ScPo-CompEcon / CourseMatch.jl

MIT License
1 stars 9 forks source link

fix clearing function #5

Closed floswald closed 5 years ago

floswald commented 5 years ago

this needs to be cleaned up and put in a proper unit test in the /test/ dir of the repo

ghost commented 5 years ago

i will work on this issue with Xiaojie

jeannesorin commented 5 years ago

@lorenzokaaks and I have a clue if you need

ghost commented 5 years ago

As @jeannesorin and @lorenzokaaks suggested I changed the output of the function to be the squareroot of the sum of squares instead of being an array of errors

ghost commented 5 years ago

created new branch with new function and test

floswald commented 5 years ago

when you are ready to share this addition you need to create a pull request from that new branch.