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

lcf_fit bug #58

Open
tschoonj opened this issue Oct 23, 2018 · 4 comments
Open

lcf_fit bug #58

tschoonj opened this issue Oct 23, 2018 · 4 comments

Comments

@tschoonj
Copy link
Contributor

Hi @bruceravel @newville

Some of our B18 staff have run into a bug when trying to perform linear combination fitting in Athena (Demeter 0.9.26) with Larch 0.9.40 backend.

Basically the fit fails with an error dialog with contents:

06:12:34 PM: Argument "RPC::XML::Client::send_request: parse-level error: Bad f..." isn't numeric in addition (+) at /usr/lib64/perl5/vendor_perl/Demeter/LCF.pm line 458, <F> line 27.
06:12:35 PM: Argument "RPC::XML::Client::send_request: parse-level error: Bad f..." isn't numeric in sprintf at /usr/lib64/perl5/vendor_perl/Demeter/UI/Athena/LCF.pm line 610, <F> line 27.

At the same time the terminal shows the error message:

NameError
   <_ast.Module object at 0x7feff89f6890>
       ^^^
name 'demlcf' is not defined

I did some digging in the code to try and understand where this is coming from and it would appear to be coming out of the Larch lcf_fit template:

##| set up the LCF objective function
demlcf.ninfo = {$S->ninfo}

I failed to come up with a fix for it however 😢

Let me know if you need any further information!

@bruceravel
Copy link
Owner

It's 100% an @bruceravel problem. I am pretty swamped at the beamline this week, but I'll try to get to it ASAP.

@newville
Copy link
Contributor

@tschoonj I would be interested to know if you or any of the B18 staff have tried the Linear Combo capabilities in Larch's XAS Viewer (included in 0.9.40) and whether you all might have any comments on it.

@tschoonj
Copy link
Contributor Author

I'll ask around.

@tschoonj
Copy link
Contributor Author

Hi Bruce,

Have you had some time to look into this issue?

Thanks,

Tom

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

No branches or pull requests

3 participants