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

raven_framework interfaceCheck error #659

Closed
10 tasks done
jchen56 opened this issue May 21, 2018 · 7 comments · Fixed by #660
Closed
10 tasks done

raven_framework interfaceCheck error #659

jchen56 opened this issue May 21, 2018 · 7 comments · Fixed by #660
Labels
defect devel issues in current devel priority_normal

Comments

@jchen56
Copy link

jchen56 commented May 21, 2018


Issue Description

What did you expect to see happen?

Using the following command line to do interface check:

raven_framework inputFile.xml interfaceCheck

Supposed to see if the interface is functional without invoking the executables.

What did you see instead?

IOError: There were failed runs; aborting RAVEN

It seems that RAVEN submits the jobs, ignoring the keyword interfaceCheck.

Do you have a suggested fix for the development team?

A workaround suggested is using:

python ../../../../framework/Driver.py inputFile.xml interfaceCheck

In this way RAVEN recognizes the keyword and tests the interface without invoking the executables of the codes linked by the interface.

Suggestion would be checking the argument processing in the raven_framework.

Please attach the input file(s) that generate this error. The simpler the input, the faster we can find the issue.

The input files are from the tests in the codes. Location is:

~/raven/tests/framework/CodeInterfaceTests/Scale/


For Change Control Board: Issue Review

This review should occur before any development is performed as a response to this issue.

  • 1. Is it tagged with a type: defect or improvement?
  • 2. Is it tagged with a priority: critical, normal or minor?
  • 3. If it will impact requirements or requirements tests, is it tagged with requirements?
  • 4. If it is a defect, can it cause wrong results for users? If so an email needs to be sent to the users. no impact on the results.
  • 5. Is a rationale provided? (Such as explaining why the improvement is needed or why current code is wrong.)

For Change Control Board: Issue Closure

This review should occur when the issue is imminently going to be closed.

  • 1. If the issue is a defect, is the defect fixed?
  • 2. If the issue is a defect, is the defect tested for in the regression test system? (If not explain why not.) This is a defect in the test system, not the raven framework, we may need to test our test system in the future
  • 3. If the issue can impact users, has an email to the users group been written (the email should specify if the defect impacts stable or master)?
  • 4. If the issue is a defect, does it impact the latest stable branch? If yes, is there any issue tagged with stable (create if needed)?
  • 5. If the issue is being closed without a merge request, has an explanation of why it is being closed been provided?
@PaulTalbot-INL
Copy link
Collaborator

Out of curiosity, what is the branch (and date of last commit) that you're working with when you see this?

@jchen56
Copy link
Author

jchen56 commented May 21, 2018

Actually I haven't created a branch but I'm working with Andrea on the interface between RAVEN and SCALE. I did a git pull just before I ran the above tests, so I assume the code is up to date.

@alfoa
Copy link
Collaborator

alfoa commented May 21, 2018

@PaulTalbot-INL yes it is in devel...i checked it

@PaulTalbot-INL
Copy link
Collaborator

PaulTalbot-INL commented May 21, 2018

I just tried the same thing on (current) devel and I did see failed runs.

Oddly, there is code in raven_framework to preserve the arguments in the $ARGS variable.

@wangcj05
Copy link
Collaborator

wangcj05 commented May 21, 2018 via email

@PaulTalbot-INL
Copy link
Collaborator

I found the fix.

When listing the arguments to the python call, in bash you have to use ${ARGS[@]} not $ARGS or it only passes through the first entry. Kind of like our PointSets, actually...

I'll have a PR for this shortly.

@alfoa
Copy link
Collaborator

alfoa commented May 21, 2018

Closure checklist passed...

No email is required since the defect was affecting a "testing" feature that we currently do not expose to the users

@alfoa alfoa added the devel issues in current devel label May 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
defect devel issues in current devel priority_normal
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants