Skip to content
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

Visual C++ Compile error #483

Closed
theSundayProgrammer opened this issue Nov 23, 2015 · 5 comments
Closed

Visual C++ Compile error #483

theSundayProgrammer opened this issue Nov 23, 2015 · 5 comments

Comments

@theSundayProgrammer
Copy link
Contributor

Method: RSModel::BuildModel
Error: .\src\mlpack\methods\range_search\rs_model.cpp(72): error C2899: typename can
not be used outside a template declaration

The fix, I think, would be to remove said keyword

@rcurtin
Copy link
Member

rcurtin commented Nov 23, 2015

Hi there,

I don't have a setup to reproduce this. If you'd like this fixed, could you please test that fix and provide a pull request? Once you do that, I can test the fix on Linux (with gcc and clang) and then we can merge it in.

Thanks!

@theSundayProgrammer
Copy link
Contributor Author

I have implemented it and put it in my own fork https://github.com/theSundayProgrammer/mlpack.git and it works.

The change is simple: just remove the keyword 'typename' from rs_model.cpp.

@rcurtin
Copy link
Member

rcurtin commented Nov 24, 2015

Yep, I merged in your commit. Thanks! Do you mind if I add your name/email to the list of contributors?

Also, it looks like your git configuration is a little off; your commits have the email 'joe.mariadassou@gmail.cvom' (I amended this in each of the commits I cherry-picked from your fork to be 'gmail.com').

@theSundayProgrammer
Copy link
Contributor Author

I will honoured to be in the list of contributors.
II am not sure why github uses the wrong email address. My Profile is
correct.

On Wed, Nov 25, 2015 at 3:41 AM, Ryan Curtin notifications@github.com
wrote:

Closed #483 #483.


Reply to this email directly or view it on GitHub
#483 (comment).

Joseph Chakravarti Mariadassou
http://thesundayprogrammer.com

@rcurtin
Copy link
Member

rcurtin commented Nov 24, 2015

Sure, added in 4f97fdf.

Your Github profile is just fine -- I think the issue is with your git configuration on the system you are using to develop.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants