Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Provide email address to NEOS solvers #1782

Merged
merged 3 commits into from
Jan 11, 2021

Conversation

jsiirola
Copy link
Member

Fixes #N/A

Summary/Motivation:

NEOS has updated its requirements and not rejects any jobs that do not provide an email address as part of the XML problem definition. This PR adds an the <email> field to the document specification. The interface uses the contents of the EMAIL environment variable (if present) otherwise reverts to user@{fully-qualified hostname}. If neither of those appear to be a valid email address, then the Kestrel interface raises an RuntimeError

Changes proposed in this PR:

  • add <email> section to the NEOS XML document
  • switch the username to be the email address (instead of {user} on {host}). This may help address NEOS interface broken for certain solvers  #910.
  • retrieve the email address from the EMAIL environment variable, or use user@{fully-qualified hostname}

Legal Acknowledgement

By contributing to this software project, I have read the contribution guide and agree to the following terms and conditions for my contribution:

  1. I agree my contributions are submitted under the BSD license.
  2. I represent I am authorized to make the contributions and grant the license. If my employer has rights to intellectual property that includes these contributions, I represent that I have received permission to make contributions and grant the required license on behalf of that employer.

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

Successfully merging this pull request may close these issues.

2 participants