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

possible bug in tensor evaluation #15

Closed
svigerske opened this issue Feb 22, 2020 · 3 comments
Closed

possible bug in tensor evaluation #15

svigerske opened this issue Feb 22, 2020 · 3 comments
Labels
bug Something isn't working examples minor

Comments

@svigerske
Copy link
Member

Issue created by migration from Trac.

Original creator: chrschul

Original creation time: 2016-09-21 00:52:33

Assignee: @awalther1

Version: 2.0

Keywords: tensor bug

Hi,
I use windows, c++ and visual studio. The tensor_eval function does not work for the taylor example. I could run it by changing the following lines (858-864) in the function definition in taylor.c:

old:
free((char*) jm);
free((char*) X);
free((char
) X);
free((char
) X);
free((char
) *Y);
free((char
) Y);
free((char
) Y);

new:

free((char*) jm);
//free((char*) X);
//free((char
) X);
free((char
) X);
//free((char
) *Y);
//free((char
) Y);
free((char
) Y);

@svigerske svigerske added bug Something isn't working examples minor labels Feb 22, 2020
@svigerske
Copy link
Member Author

Comment by chrschul created at 2016-09-21 00:57:12

the formatting is wrong, this is what helped:

old:

free((char*) jm);
free((char*) X);
free((char*) X);
free((char
) X);
free((char*) Y);
free((char*) Y);
free((char
) Y);

new:

free((char*) jm);
/* free((char*) X); /
/
free((char
) X); /
free((char
) X);
/
free((char
) *Y); /
/
free((char
) *Y); /
free((char
) Y);

@svigerske
Copy link
Member Author

Comment by @awalther1 created at 2016-09-21 13:29:05

The memory deallocation is now fixed in the trunk version of ADOL-C

Thanks for the hint!

@svigerske
Copy link
Member Author

Comment by @awalther1 created at 2016-09-21 13:29:05

Resolution: fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working examples minor
Projects
None yet
Development

No branches or pull requests

1 participant